Skip to content

[Agent Builder] Update agent builder RBAC docs#5313

Open
machadoum wants to merge 3 commits intoelastic:mainfrom
machadoum:al-update-rbac-docs
Open

[Agent Builder] Update agent builder RBAC docs#5313
machadoum wants to merge 3 commits intoelastic:mainfrom
machadoum:al-update-rbac-docs

Conversation

@machadoum
Copy link
Member

@machadoum machadoum commented Feb 27, 2026

Summary

Update agent builder RBAC docs to include sub-features (agent and tools) added by elastic/kibana#254464

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No
  1. If you answered "Yes" to the previous question, please specify the tool(s) and model(s) used (e.g., Google Gemini, OpenAI ChatGPT-4, etc.).

Cursor

@github-actions
Copy link
Contributor

github-actions bot commented Feb 27, 2026

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@machadoum machadoum requested a review from leemthompo February 27, 2026 13:12
@github-actions
Copy link
Contributor

github-actions bot commented Feb 27, 2026

🔍 Preview links for changed docs

**Kibana privileges section**
- Rewrite lead-in to orient users upfront: "Assign either `Read` or `All`..."
- Replace flat bullet list with subheadings (`Read`, `All`) for scannability
- Reorder applies-switch: 9.4+/serverless first, 9.2–9.3 second
- Simplify `All` description — remove circular "base read and write... full access" wording
- Clarify sub-features placement: listed under `Read` with explicit "instead of `All`" framing
- Move AI connectors requirement outside the version switch — applies to all versions
- Standardize privilege name formatting to backticks throughout (was mixed `"Read"` / **Read**)

**Roles section**
- Convert read-only tip to an inline code callout on the relevant privilege line
- Drop redundant "sub-features not available in 9.3" note
- Replace nested applies-switch + tip with a single versioned `note` admonition using `applies_to`
Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for driving this @machadoum. We should wait until changes are live on serverless to merge this PR. When ready to merge, can just update the branch and set to auto-merge. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants