Skip to content
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

Merged
merged 5 commits into from
Feb 12, 2025

Conversation

jgowdyelastic
Copy link
Member

@jgowdyelastic jgowdyelastic commented Feb 11, 2025

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
image

After
image

Copy link
Contributor

@peteharverson peteharverson left a 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?

@jgowdyelastic
Copy link
Member Author

Can you hide the query controls from the data visualizer page too?

Updated in 01c5eeb
This also affects Data Drift

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

Tested and LGTM

@jgowdyelastic jgowdyelastic self-assigned this Feb 12, 2025
@jgowdyelastic jgowdyelastic added release_note:fix :ml backport:version Backport to applied version labels v9.1.0 v8.19.0 Feature:ML/AIOps ML AIOps features: Change Point Detection, Log Pattern Analysis, Log Rate Analysis labels Feb 12, 2025
@jgowdyelastic jgowdyelastic marked this pull request as ready for review February 12, 2025 16:56
@jgowdyelastic jgowdyelastic requested review from a team as code owners February 12, 2025 16:56
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #8 / X-Pack Accessibility Tests - Group 1 uptime Accessibility detail page

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
aiops 606.4KB 606.5KB +78.0B
dataVisualizer 721.8KB 721.8KB +26.0B
unifiedSearch 372.5KB 372.6KB +151.0B
total +255.0B

History

cc @jgowdyelastic

@jgowdyelastic jgowdyelastic merged commit 36c764e into elastic:main Feb 12, 2025
22 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/13293684222

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 210556

Questions ?

Please refer to the Backport tool documentation

@jgowdyelastic
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

jgowdyelastic added a commit to jgowdyelastic/kibana that referenced this pull request Feb 12, 2025
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**

![image](https://github.com/user-attachments/assets/a26a73d0-63b4-47c1-bfa2-8579b08dfe24)

**After**

![image](https://github.com/user-attachments/assets/4fe6937e-3c7d-4d15-98da-fdf472fc21ee)

(cherry picked from commit 36c764e)

# Conflicts:
#	src/platform/plugins/shared/unified_search/public/query_string_input/query_bar_menu_panels.tsx
jgowdyelastic added a commit that referenced this pull request Feb 13, 2025
# 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![image](https://github.com/user-attachments/assets/a26a73d0-63b4-47c1-bfa2-8579b08dfe24)\r\n\r\n**After**\r\n\r\n![image](https://github.com/user-attachments/assets/4fe6937e-3c7d-4d15-98da-fdf472fc21ee)","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![image](https://github.com/user-attachments/assets/a26a73d0-63b4-47c1-bfa2-8579b08dfe24)\r\n\r\n**After**\r\n\r\n![image](https://github.com/user-attachments/assets/4fe6937e-3c7d-4d15-98da-fdf472fc21ee)","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![image](https://github.com/user-attachments/assets/a26a73d0-63b4-47c1-bfa2-8579b08dfe24)\r\n\r\n**After**\r\n\r\n![image](https://github.com/user-attachments/assets/4fe6937e-3c7d-4d15-98da-fdf472fc21ee)","sha":"36c764ebf722455a412416d48d48c737feb30e71"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels Feature:ML/AIOps ML AIOps features: Change Point Detection, Log Pattern Analysis, Log Rate Analysis :ml release_note:fix v8.19.0 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ML] Unable to load saved query in the log pattern analysis page
5 participants