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

[Management] Saved objects UI should use the Saved Objects client #19193

Merged

Conversation

chrisronline
Copy link
Contributor

Fixes #19185

#17426 brought in a few poor choices of how to query for saved objects, notably going outside of the saved objects api. This PR addresses that. There should be no functional changes.

cc @kobelb

@epixa
Copy link
Contributor

epixa commented May 18, 2018

To be clear, this isn’t strictly about RBAC. It’s about supporting all of the capabilities that the saved object abstraction provides, like seamless major version upgrades/migrations/pluggability for things like tags/etc.

@chrisronline chrisronline changed the title Saved objects rbac compatibility [Management] Saved objects UI should use the Saved Objects client May 18, 2018
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@jen-huang jen-huang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested that functionality remains the same.

LGTM!

Copy link
Contributor

@bmcconaghy bmcconaghy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chrisronline chrisronline merged commit fcec107 into elastic:master May 18, 2018
chrisronline added a commit to chrisronline/kibana that referenced this pull request May 18, 2018
…astic#19193)

* Ensure we always go through the saved objects client

* Fix a couple UI glitches

* Update these tests too

* Update snapshots
@chrisronline chrisronline deleted the saved_objects_rbac_compatibility branch May 18, 2018 19:21
chrisronline added a commit that referenced this pull request May 18, 2018
…9193) (#19227)

* Ensure we always go through the saved objects client

* Fix a couple UI glitches

* Update these tests too

* Update snapshots
@chrisronline
Copy link
Contributor Author

Backport:

6.x: 7e0e77b

epixa added a commit to epixa/kibana that referenced this pull request May 19, 2018
epixa added a commit to epixa/kibana that referenced this pull request May 19, 2018
epixa added a commit to epixa/kibana that referenced this pull request May 19, 2018
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.

5 participants