Skip to content

[7.5] [ML] Transform: Fix permissions. Fix error messages. (#49025)#49055

Merged
walterra merged 2 commits intoelastic:7.5from
walterra:backport/7.5/pr-49025
Oct 24, 2019
Merged

[7.5] [ML] Transform: Fix permissions. Fix error messages. (#49025)#49055
walterra merged 2 commits intoelastic:7.5from
walterra:backport/7.5/pr-49025

Conversation

@walterra
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.5:

- Updated internal permissions docs.
- Fixes regression to change data_frame to transform for privileges.
- Fixes error messages for wizard source table. An error triggered by Elasticsearch would not properly be passed through, just a generic JS error would be reported. Now the error for both source table and pivot preview get passed through.
- Fixes correctly reporting an error if the wizard fails to create the transform, previously a success toast would show in any case.
- The screenshot below also shows the fix for elastic#46434: A snipped of the full error message is now shown within the toast.
@walterra walterra added the backport This PR is a backport of another PR label Oct 23, 2019
@walterra walterra self-assigned this Oct 23, 2019
@walterra walterra added the :ml label Oct 23, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@walterra
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@walterra walterra merged commit 96bdc33 into elastic:7.5 Oct 24, 2019
@walterra walterra deleted the backport/7.5/pr-49025 branch October 24, 2019 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants