Skip to content

Conversation

@nielsbauman
Copy link
Contributor

This is part of an iterative process to make ILM project-aware.

This is part of an iterative process to make ILM project-aware.
@nielsbauman nielsbauman added >non-issue :Data Management/ILM+SLM Index and Snapshot lifecycle management Team:Data Management Meta label for data/management team v9.1.0 labels Jun 3, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Comment on lines 130 to 132
state = ClusterState.builder(state)
.putProjectMetadata(
IndexLifecycleTransition.moveProjectToStep(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This looks a bit clunky for now, but it'll get improved in a future change. Same goes for some other placesin this PR.

Copy link
Member

@PeteGillinElastic PeteGillinElastic left a comment

Choose a reason for hiding this comment

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

LGTM, except there's one naming question I think would benefit from some thought.

@nielsbauman nielsbauman enabled auto-merge (squash) June 3, 2025 16:13
@nielsbauman nielsbauman merged commit 7d62ea6 into elastic:main Jun 3, 2025
17 of 18 checks passed
@nielsbauman nielsbauman deleted the ilm-move-step-mp branch June 3, 2025 21:09
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 >non-issue 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