-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[8.11] [ML] AIOps: Fix Change point embeddable reporting (#169962) #170046
Conversation
## Summary Fixes elastic#169733 #### Reporting fix Change point detection embeddable was incorrectly reporting render completion. It was relying on the `onLoad` callback from the Lens embeddable responsible for chart rendering, which only indicates that data fetching is complete, but not the actual rendering. Current implementation relies on the `renderComplete` event from each child embeddable. Both PNG and PDF exports tested and work as expected. ![DASHBOARDDDD](https://github.com/elastic/kibana/assets/5236598/fb718f31-5862-43ab-82e3-60ebb795b8eb) #### Additional fixes - Fixes the metric and split field controls states when editing existing Change point embeddable from a dashboard - Fixes `filter` query if partitions input is initialized as an empty array. ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios (cherry picked from commit 80d382a) # Conflicts: # .eslintrc.js
💚 Build Succeeded
Metrics [docs]Async chunks
To update your PR or re-run it, just comment with: cc @darnautov |
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.
LGTM
@@ -248,6 +249,7 @@ const FieldPanel: FC<FieldPanelProps> = ({ | |||
disabled: removeDisabled, | |||
}, | |||
], | |||
'data=test-subj': 'aiopsChangePointDetectionContextMenuPanel', |
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.
The =
should be -
I guess?
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.
Agreed with @darnautov to fix in a separate PR as this needs correcting on main
too, and this data-test-subj
is not currently being used.
Backport
This will backport the following commits from
main
to8.11
:Questions ?
Please refer to the Backport tool documentation