Skip to content

[8.19] Fix tests in IndexLifecycleServiceTests (#129449) #129542

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

Conversation

nielsbauman
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

These two tests were ineffective because the result of the countdown
latch was ignored. Correctly asserting the output of the latch showed
that the test was failing, which was due to missing test setup - the
policies were not defined in the registry and the lifecycle runner
didn't execute the cluster state steps due to a missing task queue.
Finally, some test objects were constructed incorrectly likely due to
typos.

(cherry picked from commit d384e2f)
@nielsbauman nielsbauman added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Data Management/ILM+SLM Index and Snapshot lifecycle management >test Issues or PRs that are addressing/adding tests Team:Data Management Meta label for data/management team labels Jun 17, 2025
@elasticsearchmachine elasticsearchmachine merged commit f33ac4b into elastic:8.19 Jun 17, 2025
22 checks passed
@nielsbauman nielsbauman deleted the backport/8.19/pr-129449 branch June 17, 2025 11:47
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 :Data Management/ILM+SLM Index and Snapshot lifecycle management Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants