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] Missing job list error icon - follow-up #28364

Closed
pheyos opened this issue Jan 9, 2019 · 2 comments · Fixed by #30490
Closed

[ML] Missing job list error icon - follow-up #28364

pheyos opened this issue Jan 9, 2019 · 2 comments · Fixed by #30490
Assignees
Labels
bug Fixes for quality problems that affect the customer experience :ml usability v6.7.0

Comments

@pheyos
Copy link
Member

pheyos commented Jan 9, 2019

Follow-up for #24348

Found in version

  • 6.6.0-bc2

Browser

  • Firefox, Chrome

Steps to reproduce

  • Create a number of jobs, run them and delete them all again
  • Create 1 job with a memory limit so low that it triggers the soft_limit or hard_limit
  • Run the job and look at the error icon in the list

Expected result

  • The error icon is displayed correctly

Actual result

  • In most cases the error icon is not displayed

Additional information

  • With [ML] Fixes Missing job list error icon #27575 the size of the terms aggregation [is set] to the length of available jobs. But the audit messages are stored also for deleted jobs, so the size is not always enough to collect the messages for all existing jobs.
  • A filter in the query for existing jobs might be a solution for this.
@pheyos pheyos added bug Fixes for quality problems that affect the customer experience :ml usability v6.7.0 labels Jan 9, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

@walterra
Copy link
Contributor

walterra commented Feb 15, 2019

@pheyos Fix backported to 6.7: #30516

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience :ml usability v6.7.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants