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

Run TransportClusterGetSettingsAction on local node #119831

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nielsbauman
Copy link
Contributor

@nielsbauman nielsbauman commented Jan 9, 2025

This action solely needs the cluster state, it can run on any node. Additionally, it needs to be cancellable to avoid doing unnecessary work after a client failure or timeout. The ?local parameter becomes a no-op and is marked as deprecated.

Relates #101805
Relates #107984

This action solely needs the cluster state, it can run on any node.
Additionally, it needs to be cancellable to avoid doing unnecessary
work after a client failure or timeout.
@nielsbauman nielsbauman added >enhancement :Core/Infra/Settings Settings infrastructure and APIs Team:Core/Infra Meta label for core/infra team v9.0.0 labels Jan 9, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine
Copy link
Collaborator

Hi @nielsbauman, I've created a changelog YAML for you.

@nielsbauman
Copy link
Contributor Author

I initially mistakenly thought Data Management owned this API but I just realized we don't. Since I had the changes already, I just opened a PR anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Settings Settings infrastructure and APIs >enhancement Team:Core/Infra Meta label for core/infra team v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants