-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ML] AIOps Hiding saved query controls #210556
[ML] AIOps Hiding saved query controls #210556
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you hide the query controls from the data visualizer page too?
…c/kibana into hiding-saved-query-controls
Updated in 01c5eeb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and LGTM
Pinging @elastic/ml-ui (:ml) |
💛 Build succeeded, but was flaky
Failed CI StepsTest FailuresMetrics [docs]Async chunks
History
|
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
The AIOps pages in ML make use of the shared query bar component, but do not support saved queries. By having the load and save query options in the menu, it can give the impression that these options are generally available but for some reason cannot be used. This PR adds a new optional flag which allows the load and save options to be hidden from the menu. The flag defaults to always showing the options. Fixes elastic#209152 **Before**  **After**  (cherry picked from commit 36c764e) # Conflicts: # src/platform/plugins/shared/unified_search/public/query_string_input/query_bar_menu_panels.tsx
# Backport This will backport the following commits from `main` to `8.x`: - [[ML] AIOps Hiding saved query controls (#210556)](#210556) <!--- Backport version: 9.6.4 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"James Gowdy","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-12T19:52:30Z","message":"[ML] AIOps Hiding saved query controls (#210556)\n\nThe AIOps pages in ML make use of the shared query bar component, but do\r\nnot support saved queries.\r\nBy having the load and save query options in the menu, it can give the\r\nimpression that these options are generally available but for some\r\nreason cannot be used.\r\nThis PR adds a new optional flag which allows the load and save options\r\nto be hidden from the menu.\r\nThe flag defaults to always showing the options.\r\n\r\nFixes https://github.com/elastic/kibana/issues/209152\r\n\r\n**Before**\r\n\r\n\r\n\r\n**After**\r\n\r\n","sha":"36c764ebf722455a412416d48d48c737feb30e71","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix",":ml","Feature:ML/AIOps","backport:version","v9.1.0","v8.19.0"],"title":"[ML] AIOps Hiding saved query controls","number":210556,"url":"https://github.com/elastic/kibana/pull/210556","mergeCommit":{"message":"[ML] AIOps Hiding saved query controls (#210556)\n\nThe AIOps pages in ML make use of the shared query bar component, but do\r\nnot support saved queries.\r\nBy having the load and save query options in the menu, it can give the\r\nimpression that these options are generally available but for some\r\nreason cannot be used.\r\nThis PR adds a new optional flag which allows the load and save options\r\nto be hidden from the menu.\r\nThe flag defaults to always showing the options.\r\n\r\nFixes https://github.com/elastic/kibana/issues/209152\r\n\r\n**Before**\r\n\r\n\r\n\r\n**After**\r\n\r\n","sha":"36c764ebf722455a412416d48d48c737feb30e71"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210556","number":210556,"mergeCommit":{"message":"[ML] AIOps Hiding saved query controls (#210556)\n\nThe AIOps pages in ML make use of the shared query bar component, but do\r\nnot support saved queries.\r\nBy having the load and save query options in the menu, it can give the\r\nimpression that these options are generally available but for some\r\nreason cannot be used.\r\nThis PR adds a new optional flag which allows the load and save options\r\nto be hidden from the menu.\r\nThe flag defaults to always showing the options.\r\n\r\nFixes https://github.com/elastic/kibana/issues/209152\r\n\r\n**Before**\r\n\r\n\r\n\r\n**After**\r\n\r\n","sha":"36c764ebf722455a412416d48d48c737feb30e71"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT-->
The AIOps pages in ML make use of the shared query bar component, but do not support saved queries.
By having the load and save query options in the menu, it can give the impression that these options are generally available but for some reason cannot be used.
This PR adds a new optional flag which allows the load and save options to be hidden from the menu.
The flag defaults to always showing the options.
Fixes #209152
Before

After
