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

Remove the es_admin proxy #13000

Merged
merged 2 commits into from
Jul 20, 2017
Merged

Remove the es_admin proxy #13000

merged 2 commits into from
Jul 20, 2017

Conversation

epixa
Copy link
Contributor

@epixa epixa commented Jul 20, 2017

Saved object access is now handled through the saved object client
rather than by making direct requests to the admin cluster through a
proxy. The indices module and scanner module were both still relying on
the proxy, so temporary "legacy" endpoints have been created for their
specific use-cases. We should add more purposeful endpoints to the
public API in the future to replace these.

The admin cluster itself is still readily available on the server and
should still be used for all requests for internal kibana indices.

@epixa epixa force-pushed the esindices branch 2 times, most recently from 0ebbc8b to 47f6d82 Compare July 20, 2017 14:29
@epixa epixa added the WIP Work in progress label Jul 20, 2017
@epixa epixa self-assigned this Jul 20, 2017
@epixa epixa force-pushed the esindices branch 5 times, most recently from d83ae6e to b9e8897 Compare July 20, 2017 17:07
Saved object access is now handled through the saved object client
rather than by making direct requests to the admin cluster through a
proxy. The indices module and scanner module were both still relying on
the proxy, so temporary "legacy" endpoints have been created for their
specific use-cases. We should add more purposeful endpoints to the
public API in the future to replace these.

The admin cluster itself is still readily available on the server and
should still be used for all requests for internal kibana indices.
@epixa
Copy link
Contributor Author

epixa commented Jul 20, 2017

I think a green build is blocked by #13014, or at least I can't get further in the tests than the issue that PR is fixing, but it should be good for review otherwise.

@epixa epixa added review and removed WIP Work in progress labels Jul 20, 2017
@epixa epixa requested review from tylersmalley and spalger July 20, 2017 20:05
Copy link
Contributor

@tylersmalley tylersmalley left a comment

Choose a reason for hiding this comment

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

LGTM - set pageSize to 1 to functionally test continue.

@tylersmalley
Copy link
Contributor

Merged master since #13014 went in

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM

@epixa epixa merged commit fac1935 into elastic:master Jul 20, 2017
@epixa epixa deleted the esindices branch July 20, 2017 23:54
@jimgoodwin jimgoodwin added the Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc label Jul 25, 2017
Bargs added a commit that referenced this pull request Aug 10, 2017
Bargs added a commit to Bargs/kibana that referenced this pull request Aug 10, 2017
Bargs added a commit to Bargs/kibana that referenced this pull request Aug 10, 2017
Bargs added a commit that referenced this pull request Aug 10, 2017
Bargs added a commit that referenced this pull request Aug 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Plugins release_note:breaking review Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v6.0.0-beta1 v6.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants