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 http_addresses from Remote Cluster Info API #29207

Closed
jkakavas opened this issue Mar 22, 2018 · 3 comments
Closed

Remove http_addresses from Remote Cluster Info API #29207

jkakavas opened this issue Mar 22, 2018 · 3 comments
Assignees
Labels
:Core/Infra/Core Core issues without another label >deprecation

Comments

@jkakavas
Copy link
Member

We currently make node info API requests to remote clusters as part of the Remote Cluster Info API in order to return information about http_addresses of the remote nodes.
http_addresses was originally introduced for kibana to do _field_caps but this is not used in any actionable way and could be removed from the response or replaced with transport addresses, so that the the remote nodes info API calls can be skipped.

Worth noting here that these API calls also require additional privileges for users running x-pack security.

@jkakavas jkakavas added the :Core/Infra/Core Core issues without another label label Mar 22, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@jasontedor
Copy link
Member

@nik9000 Can you investigate this one and if appropriate, make the necessary changes?

@nik9000
Copy link
Member

nik9000 commented Mar 29, 2018

@nik9000 Can you investigate this one and if appropriate, make the necessary changes?

I'll take it!

nik9000 added a commit to nik9000/elasticsearch that referenced this issue Apr 17, 2018
They are expensive to fetch and no longer needed by Kibana so they
*shouldn't* be needed by anyone else either.

Closes elastic#29207
nik9000 added a commit that referenced this issue Apr 27, 2018
They are expensive to fetch and no longer needed by Kibana so they
*shouldn't* be needed by anyone else either.

Closes #29207
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >deprecation
Projects
None yet
Development

No branches or pull requests

5 participants