Skip to content

Run TransportGetStatusAction on local node#129367

Merged
nielsbauman merged 2 commits intoelastic:mainfrom
nielsbauman:get-status-ilm-local
Jun 12, 2025
Merged

Run TransportGetStatusAction on local node#129367
nielsbauman merged 2 commits intoelastic:mainfrom
nielsbauman:get-status-ilm-local

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 DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. labels Jun 12, 2025
@elasticsearchmachine elasticsearchmachine added Team:Data Management (obsolete) DO NOT USE. This team no longer exists. 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 DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. >enhancement Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v9.1.0

3 participants