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

[7.0] [ML] Fix Job Audit Messages filter. (#30490) #30515

Merged
merged 1 commit into from
Feb 15, 2019

Conversation

walterra
Copy link
Contributor

@walterra walterra commented Feb 8, 2019

Backports the following commits to 7.0:

@walterra walterra self-assigned this Feb 8, 2019
@walterra walterra added the :ml label Feb 8, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

@elasticmachine
Copy link
Contributor

💔 Build Failed

@walterra
Copy link
Contributor Author

retest

1 similar comment
@spalger
Copy link
Contributor

spalger commented Feb 14, 2019

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@walterra
Copy link
Contributor Author

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@walterra
Copy link
Contributor Author

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

The way job audit messages were fetched didn't retrieve the expected results if there were deleted jobs with messages still present for these jobs.
This fix allows to specify a list of job IDs to filter the audit messages on. For the jobs list UI, the currently existing job IDs will be passed on to ignore messages from deleted jobs.
@walterra walterra force-pushed the backport/7.0/pr-30490 branch from 094fd8c to 6c5a4e0 Compare February 15, 2019 00:24
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@walterra walterra merged commit bf14948 into elastic:7.0 Feb 15, 2019
@walterra walterra deleted the backport/7.0/pr-30490 branch February 15, 2019 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants