Skip to content

Run TransportGetStatusAction on local node #129367

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

Merged
merged 2 commits into from
Jun 12, 2025

Conversation

nielsbauman
Copy link
Contributor

This action solely needs the cluster state, it can run on any node.

Relates #101805

This action solely needs the cluster state, it can run on any node.

Relates elastic#101805
@nielsbauman nielsbauman added >enhancement :Data Management/ILM+SLM Index and Snapshot lifecycle management labels Jun 12, 2025
@elasticsearchmachine elasticsearchmachine added Team:Data Management Meta label for data/management team v9.1.0 labels Jun 12, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@nielsbauman nielsbauman enabled auto-merge (squash) June 12, 2025 22:14
@nielsbauman nielsbauman merged commit 47032b5 into elastic:main Jun 12, 2025
15 of 17 checks passed
@nielsbauman nielsbauman deleted the get-status-ilm-local branch June 12, 2025 23:21
nielsbauman added a commit to nielsbauman/elasticsearch that referenced this pull request Jun 13, 2025
Since elastic#129367 we run the `_ilm/status` API on the local node, which
could cause issues in tests that assume the API runs on the master node
- i.e. they assumed that once the assertion passed, all nodes in the
cluster would have that cluster state, which is not true.
nielsbauman added a commit to nielsbauman/elasticsearch that referenced this pull request Jun 13, 2025
Since elastic#129367 we run the `_ilm/status` API on the local node, which
could cause issues in tests that assume the API runs on the master node
(i.e. they assumed that once the assertion passed, all nodes in the
cluster would have that cluster state, which is not true).
nielsbauman added a commit that referenced this pull request Jun 13, 2025
Since #129367 we run the `_ilm/status` API on the local node, which
could cause issues in tests that assume the API runs on the master node
(i.e. they assumed that once the assertion passed, all nodes in the
cluster would have that cluster state, which is not true).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >enhancement Team:Data Management Meta label for data/management team v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants