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] Better handling for permission errors during analytics job run time #55692

Closed
pheyos opened this issue Jan 23, 2020 · 2 comments
Closed
Assignees
Labels
enhancement New value added to drive a business result Feature:Data Frame Analytics ML data frame analytics features :ml v7.7.0

Comments

@pheyos
Copy link
Member

pheyos commented Jan 23, 2020

When permission issues occur during the data frame analytics execution, the UI doesn't inform the user properly.
A lot of the needed permissions are already checked before creating/starting the job, but the permission model gets more complex over time, so we should also include graceful error handling during run time.

Example:

  • Try to give a user all needed permissions to create and run data frame analytics jobs based on the errors the user sees during job creation (assuming that when no error is displayed any more, all needed permissions are granted).
  • You end up with permissions close to the documented ones (https://www.elastic.co/guide/en/elasticsearch/reference/7.6/start-dfanalytics.html) but in this case the permission indices:admin/refresh (contained in manage index permission) is missing.
  • The job can be created and started without issues and it finished phase 1 (reindex) fine.
  • Then the job doesn't continue to do anything and no error message is displayed.
  • The error is only logged to the ES log:
org.elasticsearch.ElasticsearchSecurityException: action [indices:admin/refresh] is unauthorized for user [analytics_admin]
@pheyos pheyos added enhancement New value added to drive a business result :ml labels Jan 23, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@peteharverson peteharverson added v7.7.0 Feature:Data Frame Analytics ML data frame analytics features labels Jan 23, 2020
@peteharverson peteharverson self-assigned this Mar 11, 2020
@peteharverson
Copy link
Contributor

Fixed by #60249. The failure reason for the job is displayed in the EuiBadge tooltip in the data frame analytics job list (previously tooltip was not displaying the failure reason), and an EuiBadge has been added for the display of the job state to highlight errors in the expanded row for the job.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Data Frame Analytics ML data frame analytics features :ml v7.7.0
Projects
None yet
Development

No branches or pull requests

3 participants