Skip to content

[9.3] Remove ILM-stopped-check before the cluster update in migrateToDataTiersRouting (#140885) (#140892)#140949

Merged
elasticsearchmachine merged 1 commit intoelastic:9.3from
gmarouli:backport/9.3/pr-140892
Jan 20, 2026
Merged

[9.3] Remove ILM-stopped-check before the cluster update in migrateToDataTiersRouting (#140885) (#140892)#140949
elasticsearchmachine merged 1 commit intoelastic:9.3from
gmarouli:backport/9.3/pr-140892

Conversation

@gmarouli
Copy link
Contributor

…iersRouting` (elastic#140885) (elastic#140892)

In order to run the migrate data tiers request, we need to ensure that
ILM is stopped. In the `TransportMigrateToDataTiersAction` we performed
this check twice:

1. The first time was when the API was called to avoid creating cluster state updates that were bound to fail.
2. The second time was during the cluster state update, to ensure that the cluster state has the correct conditions when applying the change.

It is possible that the first check is applied on an outdated cluster
state causing the request to fail while it could succeed.

In this PR, considering the usage patterns of this API, we believe the
benefit of the first check is very small, so we keep only the second
which is the up-to-date one.

Relevant test:

https://github.com/elastic/elasticsearch/blob/main/x-pack/plugin/ilm/qa/multi-node/src/javaRestTest/java/org/elasticsearch/xpack/MigrateToDataTiersIT.java#L82-L87

Fixes elastic#140885
@gmarouli gmarouli added :StorageEngine/ILM Index Lifecycle Management >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:StorageEngine labels Jan 20, 2026
@elasticsearchmachine elasticsearchmachine merged commit 84d8879 into elastic:9.3 Jan 20, 2026
35 checks passed
@gmarouli gmarouli deleted the backport/9.3/pr-140892 branch January 20, 2026 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :StorageEngine/ILM Index Lifecycle Management Team:StorageEngine v9.3.1

2 participants