[Obs AI Assistant] Remove assistant settings page for basic licenses by eleonoramicozzi · Pull Request #230036 · elastic/kibana

eleonoramicozzi · 2025-07-31T08:14:27Z

Summary

AI Assistants are only available for users with enterprise or trial license. Right now, if a user has a basic license, they can still navigate to the AI Assistant settings page. This PR removes the "AI Assistants" menu item for licenses < enterprise, preventing the user from seeing a disabled page.

Note

The decision to remove the menu item completely, instead of disabling the observability Assistant tile, was made together with @isaclfreire , see https://elastic.slack.com/archives/C0932G71NDA/p1753867908124919?thread_ts=1753810253.575439&cid=C0932G71NDA

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
Review the backport guidelines and apply applicable backport:* labels.

Testing

Enterprise license, AI settings page is available

Basic license, AI settings page is not available (directly navigating to the url is also blocked)

github-actions · 2025-07-31T08:14:37Z

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

/oblt-deploy : Deploy a Kibana instance using the Observability test environments.
run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

elasticmachine · 2025-07-31T08:15:18Z

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

… remove_assistant_settings_for_basic_license

…no-cache --fix'

… remove_assistant_settings_for_basic_license

…/github.com/eleonoramicozzi/kibana into remove_assistant_settings_for_basic_license

elasticmachine · 2025-08-05T19:21:14Z

💛 Build succeeded, but was flaky

Buildkite Build
Commit: 1421dc3
Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-230036-1421dc39d463

Failed CI Steps

FTR Configs #71

Test Failures

[job] [logs] FTR Configs #71 / InfraOps App Metrics UI Home page with metrics present sort nodes by ascending value

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
`aiAssistantManagementSelection`	78.1KB	78.2KB	+112.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id	before	after	diff
`aiAssistantManagementSelection`	5.6KB	5.9KB	+366.0B
`observabilityAiAssistantManagement`	5.7KB	5.9KB	+214.0B
total			+580.0B

Unknown metric groups

ESLint disabled line counts

id	before	after	diff
`aiAssistantManagementSelection`	5	4	-1

Total ESLint disabled count

id	before	after	diff
`aiAssistantManagementSelection`	5	4	-1

History

cc @eleonoramicozzi

neptunian · 2025-08-06T17:08:15Z

...d/ai_assistant_management/selection/public/routes/components/ai_assistant_selection_page.tsx

+  const isObservabilityAIAssistantEnabled =
+    observabilityAIAssistantCapabilityEnabled &&
+    observabilityAIAssistantEnabled &&
+    aiAssistantManagementSelection;


Do we need this check for aiAssistantManagementSelection if the app is already disabled? They can't get to this page anyway?

Resolves: #224922 This PR: * Introduces a new `AI` section in Stack Management * Moves all AI Assistant management pages out of the `Kibana` section into the new `AI` setion. * Introduces the `genAISettings` plugin with a `GenAI Settings` page under the new `AI` section to manage generic configurations like connectors and feature visibility. * ~~AI Assistants settings now depend on the `aiAssistantManagementSelection` setting instead of the availability of individual assistants.~~ * ~~Enabled read mode for Security and Observability/Search assistant settings based on the setting above ^~~ It appears that the Assistant settings cannot directly rely on the `AI Assistant Settings` privilege. There are still ongoing discussions about whether we should remove the `AI Assistant Settings` privilege or take a different approach. With that in mind, I reverted some changes in this PR to just create a new section and move `AI Assistants` under it, along with adding the new `GenAI Settings` page. I left the existing privileges checks as they are, they will be updated once the discussions are resolved. <img width="1477" height="954" alt="image" src="https://github.com/user-attachments/assets/8b50bcbc-28df-42fa-82c5-5a8c262b2c9f" /> ### `AI Assistants` scenarios with different privileges: 1. `Management > AI Assistant Settings` - All/Read `Security Assistant` and `Observability Assistant` - All <img width="1717" height="1363" alt="image" src="https://github.com/user-attachments/assets/dcea9c33-4f2d-4ea4-818e-ba6ffaf1a812" /> 2. `Management > AI Assistant Settings` - All/Read, `Security Assistant` and `Observability Assistant` - none <img width="1716" height="1357" alt="image" src="https://github.com/user-attachments/assets/6b26e954-e201-475b-bee9-4a7ca20bd120" /> 3. `Management > AI Assistant Settings` - None <img width="1719" height="1313" alt="image" src="https://github.com/user-attachments/assets/9c38ecc1-693b-4c3b-8c09-f227ba539202" /> 4. No `Enterprise` license. Currently buggy, will be fixed by #230036 <img width="1718" height="1356" alt="image" src="https://github.com/user-attachments/assets/39e07e33-baa3-4eaf-8fa4-c31895605517" /> TBD: - [ ] Manage Connectors should apply filters, however, the connectors page does not currently save filters in the URL state. - [ ] In solution views, take the user directly to the individual Assistant settings - I've decided to hold off on this until the ongoing privilege discussions are resolved, as the implementation may vary depending on the outcome. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>

Resolves: elastic#224922 This PR: * Introduces a new `AI` section in Stack Management * Moves all AI Assistant management pages out of the `Kibana` section into the new `AI` setion. * Introduces the `genAISettings` plugin with a `GenAI Settings` page under the new `AI` section to manage generic configurations like connectors and feature visibility. * ~~AI Assistants settings now depend on the `aiAssistantManagementSelection` setting instead of the availability of individual assistants.~~ * ~~Enabled read mode for Security and Observability/Search assistant settings based on the setting above ^~~ It appears that the Assistant settings cannot directly rely on the `AI Assistant Settings` privilege. There are still ongoing discussions about whether we should remove the `AI Assistant Settings` privilege or take a different approach. With that in mind, I reverted some changes in this PR to just create a new section and move `AI Assistants` under it, along with adding the new `GenAI Settings` page. I left the existing privileges checks as they are, they will be updated once the discussions are resolved. <img width="1477" height="954" alt="image" src="https://github.com/user-attachments/assets/8b50bcbc-28df-42fa-82c5-5a8c262b2c9f" /> ### `AI Assistants` scenarios with different privileges: 1. `Management > AI Assistant Settings` - All/Read `Security Assistant` and `Observability Assistant` - All <img width="1717" height="1363" alt="image" src="https://github.com/user-attachments/assets/dcea9c33-4f2d-4ea4-818e-ba6ffaf1a812" /> 2. `Management > AI Assistant Settings` - All/Read, `Security Assistant` and `Observability Assistant` - none <img width="1716" height="1357" alt="image" src="https://github.com/user-attachments/assets/6b26e954-e201-475b-bee9-4a7ca20bd120" /> 3. `Management > AI Assistant Settings` - None <img width="1719" height="1313" alt="image" src="https://github.com/user-attachments/assets/9c38ecc1-693b-4c3b-8c09-f227ba539202" /> 4. No `Enterprise` license. Currently buggy, will be fixed by elastic#230036 <img width="1718" height="1356" alt="image" src="https://github.com/user-attachments/assets/39e07e33-baa3-4eaf-8fa4-c31895605517" /> TBD: - [ ] Manage Connectors should apply filters, however, the connectors page does not currently save filters in the URL state. - [ ] In solution views, take the user directly to the individual Assistant settings - I've decided to hold off on this until the ongoing privilege discussions are resolved, as the implementation may vary depending on the outcome. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>

eleonoramicozzi · 2025-08-27T20:58:40Z

This was addressed in #231989

Remove assistant settings page for basic licenses

f63dbf1

eleonoramicozzi requested review from a team as code owners July 31, 2025 08:14

botelastic bot added the ci:project-deploy-observability Create an Observability project label Jul 31, 2025

eleonoramicozzi self-assigned this Jul 31, 2025

eleonoramicozzi added the Team:Obs AI Assistant Observability AI Assistant label Jul 31, 2025

eleonoramicozzi added release_note:fix backport:version Backport to applied version labels labels Jul 31, 2025

sorenlouv changed the title ~~Remove assistant settings page for basic licenses~~ Jul 31, 2025

eleonoramicozzi added 2 commits July 31, 2025 16:47

Merge branch 'main' of https://github.com/eleonoramicozzi/kibana into…

c6e1fcf

… remove_assistant_settings_for_basic_license

Remove unused import

e7ba812

eleonoramicozzi force-pushed the remove_assistant_settings_for_basic_license branch from 7111d76 to e7ba812 Compare July 31, 2025 15:58

kibanamachine and others added 6 commits July 31, 2025 16:18

[CI] Auto-commit changed files from 'node scripts/notice'

f925c97

[CI] Auto-commit changed files from 'node scripts/eslint_all_files --…

497ee03

…no-cache --fix'

Merge branch 'main' of https://github.com/eleonoramicozzi/kibana into…

07cdef3

… remove_assistant_settings_for_basic_license

Fix test

9920663

Merge branch 'remove_assistant_settings_for_basic_license' of https:/…

43383eb

…/github.com/eleonoramicozzi/kibana into remove_assistant_settings_for_basic_license

Merge branch 'main' into remove_assistant_settings_for_basic_license

1421dc3

neptunian reviewed Aug 6, 2025

View reviewed changes

rbrtj mentioned this pull request Aug 11, 2025

[ML] Adds an AI section into Stack Management #227289

Merged

2 tasks

eleonoramicozzi closed this Aug 27, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Obs AI Assistant] Remove assistant settings page for basic licenses#230036

[Obs AI Assistant] Remove assistant settings page for basic licenses#230036
eleonoramicozzi wants to merge 9 commits intoelastic:mainfrom
eleonoramicozzi:remove_assistant_settings_for_basic_license

eleonoramicozzi commented Jul 31, 2025 •

edited

Loading

github-actions bot commented Jul 31, 2025

elasticmachine commented Jul 31, 2025

elasticmachine commented Aug 5, 2025 •

edited

Loading

ESLint disabled line counts

Total ESLint disabled count

neptunian Aug 6, 2025 •

edited

Loading

eleonoramicozzi commented Aug 27, 2025

Labels

4 participants

Conversation

eleonoramicozzi commented Jul 31, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Note

Checklist

Testing

Enterprise license, AI settings page is available

Basic license, AI settings page is not available (directly navigating to the url is also blocked)

github-actions bot commented Jul 31, 2025

🤖 GitHub comments

elasticmachine commented Jul 31, 2025

elasticmachine commented Aug 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

Metrics [docs]

Async chunks

Page load bundle

ESLint disabled line counts

Total ESLint disabled count

History

neptunian Aug 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

eleonoramicozzi commented Aug 27, 2025

Labels

4 participants

eleonoramicozzi commented Jul 31, 2025 •

edited

Loading

elasticmachine commented Aug 5, 2025 •

edited

Loading

neptunian Aug 6, 2025 •

edited

Loading