Skip to content

[7.4] [APM] Fix indefinite loading state in agent settings for unauthorized user roles (#44970)#45207

Merged
ogupte merged 3 commits intoelastic:7.4from
ogupte:backport/7.4/pr-44970
Sep 11, 2019
Merged

[7.4] [APM] Fix indefinite loading state in agent settings for unauthorized user roles (#44970)#45207
ogupte merged 3 commits intoelastic:7.4from
ogupte:backport/7.4/pr-44970

Conversation

@ogupte
Copy link
Contributor

@ogupte ogupte commented Sep 9, 2019

Backports the following commits to 7.4:

@ogupte ogupte added the backport This PR is a backport of another PR label Sep 9, 2019
@ogupte
Copy link
Contributor Author

ogupte commented Sep 10, 2019

retest

1 similar comment
@tylersmalley
Copy link
Member

retest

… user roles (elastic#44970)

* - handle unauthorized error to return empty list of agent settings
- display to user when a failure occurred in settings initialization
- fix incorrect settings link path

* - moved agent config index creation to the plugin setup step
- make failure text reflect a general failure to load settings

* - Rename Settings/SettingsList -> Settings/index
- Add Settings/SettingsList to render only the list/messaging
- Mention permissions issue in failure message
@ogupte ogupte force-pushed the backport/7.4/pr-44970 branch from 5913eed to ea4c0d5 Compare September 11, 2019 16:04
@elasticmachine

This comment has been minimized.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@ogupte
Copy link
Contributor Author

ogupte commented Sep 11, 2019

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💔 Build Failed

@ogupte
Copy link
Contributor Author

ogupte commented Sep 11, 2019

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@ogupte ogupte merged commit af67243 into elastic:7.4 Sep 11, 2019
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants