Skip to content

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Jan 30, 2025

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

Relates to elastic/elasticsearch#118993

This PR updates the remote info and resolve cluster specifications to (1) copy some missing descriptions from the docs (https://www.elastic.co/guide/en/elasticsearch/reference/master/cluster-remote-info.html and https://www.elastic.co/guide/en/elasticsearch/reference/master/indices-resolve-cluster-api.html) and (2) incorporate the changes from elastic/elasticsearch#118993

Co-authored by @lcawl

(cherry picked from commit 5b17efd)
Copy link
Contributor

Following you can find the validation results for the APIs you have changed.

API Status Request Response
cluster.remote_info 🟢 3/3 3/3
indices.resolve_cluster 🔴 10/12 12/12

You can validate these APIs yourself by using the make validate target.

@lcawl lcawl enabled auto-merge (squash) January 30, 2025 18:18
num_proxy_sockets_connected: integer
/** The maximum number of socket connections to the remote cluster when proxy mode is configured. */
max_proxy_socket_connections: integer
/** This field presents and has value of ::es_redacted:: only when the remote cluster is configured with the API key based model. Otherwise, the field is not present. */
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, I thought I fixed this to say "This field is present"? Sorry that fix that lost somehow.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in #3670

@lcawl lcawl merged commit 2eadbbd into 8.x Jan 30, 2025
7 checks passed
@lcawl lcawl deleted the backport/8.x/pr-3655 branch January 30, 2025 18:31
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.

2 participants