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.

Sign up for GitHub

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

Jump to bottom

[8.x] #101193 Preserve Step Info Across ILM Auto Retries (#113187) #113786

Merged
merged 1 commit into from
Sep 30, 2024
Merged

[8.x] #101193 Preserve Step Info Across ILM Auto Retries (#113187) #113786

merged 1 commit into from
Sep 30, 2024

Conversation

lukewhiting
Copy link
Contributor

Backports the following commits to 8.x:

@lukewhiting @gmarouli
elastic#101193 Preserve Step Info Across ILM Auto Retries (elastic#11…
a9c2310 
…3187)

* Add new Previous Step Info field to LifecycleExecutionState

* Add new field to IndexLifecycleExplainResponse

* Add new field to TransportExplainLifecycleAction

* Add logic to IndexLifecycleTransition to keep previous setp info

* Switch tests to use Java standard Clock class

for any time based testing, this is the recommended method

* Fix tests for new field

Also refactor tests to newer style

* Add test to ensure step info is preserved

Across auto retries

* Add docs for new field

* Changelog Entry

* Update docs/changelog/113187.yaml

* Revert "Switch tests to use Java standard Clock class"

This reverts commit 241074c.

* PR Changes

* PR Changes - Improve docs wording

Co-authored-by: Mary Gouseti <[email protected]>

* Integration test for new ILM explain field

* Use ROOT locale instead of default toLowerCase

* PR Changes - Switch to block strings

* Remove forbidden API usage

---------

Co-authored-by: Mary Gouseti <[email protected]>
@lukewhiting lukewhiting added :Data Management/ILM+SLM Index and Snapshot lifecycle management >enhancement auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Data Management Meta label for data/management team labels Sep 30, 2024
@elasticsearchmachine elasticsearchmachine mentioned this pull request Sep 30, 2024
Merged
@github-actions GitHub Actions
Copy link
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine merged commit c6e3397 into elastic:8.x Sep 30, 2024
16 checks passed
@lukewhiting lukewhiting deleted the backport/8.x/pr-113187 branch September 30, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Data Management/ILM+SLM Index and Snapshot lifecycle management >enhancement Team:Data Management Meta label for data/management team v8.16.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lukewhiting @elasticsearchmachine