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] Adds data frame messages to data frames list #39609
[ML] Adds data frame messages to data frames list #39609
Conversation
Pinging @elastic/ml-ui |
...ns/ml/public/data_frame/pages/job_management/components/job_list/transform_messages_pane.tsx
Outdated
Show resolved
Hide resolved
💚 Build Succeeded |
💚 Build Succeeded |
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.
Great stuff overall, just added a comment about a missing translation and how to refresh the messages tab.
...egacy/plugins/ml/public/data_frame/pages/job_management/components/job_list/expanded_row.tsx
Outdated
Show resolved
Hide resolved
...ns/ml/public/data_frame/pages/job_management/components/job_list/transform_messages_pane.tsx
Outdated
Show resolved
Hide resolved
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.
Latest changes LGTM 🍾
💔 Build Failed |
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.
Just one question, otherwise LGTM
@@ -10,3 +10,4 @@ export const ML_ANNOTATIONS_INDEX_PATTERN = '.ml-annotations-6'; | |||
|
|||
export const ML_RESULTS_INDEX_PATTERN = '.ml-anomalies-*'; | |||
export const ML_NOTIFICATION_INDEX_PATTERN = '.ml-notifications'; | |||
export const ML_DF_NOTIFICATION_INDEX_PATTERN = '.data-frame-notifications-1'; |
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.
Just double checking, but do the data frame notifications always go in .data-frame-notifications-1
(with the 1
)?
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.
Yes - that's the index it will go into
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
*/ | ||
|
||
|
||
import { ML_DF_NOTIFICATION_INDEX_PATTERN } from '../../../common/constants/index_patterns'; |
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.
this should probably be typescript
💚 Build Succeeded |
💔 Build Failed |
retest |
💚 Build Succeeded |
* kibana server endpoint for fetching DataFrame messages * create Messages tab in expanded row * update snapshot * translate error message * ensure messages fetched on refresh interval * add tab translation * update lastUpdate every time getJobs is run * update expanded row snapshot
Summary
Related issue: #38540
This PR adds a
Messages
tab to the data frames list expanded row and adds an endpoint for fetching the data frame messages.Messages in tab:
If there's a problem loading the messages:
Follow up:
Adding icon at message level in a follow up PR
NOTE:
As data frame creation time is not yet available, messages are currently not filtered by time. This can be added in a follow-up once that information becomes available.
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.- [ ] Documentation was added for features that require explanation or tutorialsFor maintainers
- [ ] This was checked for breaking API changes and was labeled appropriately- [ ] This includes a feature addition or change that requires a release note and was labeled appropriately