Skip to content

Conversation

@MichelLosier
Copy link

@MichelLosier MichelLosier commented Aug 12, 2025

As per the discussion in the linked issue, this PR undeprecates the legacy ILM policies metrics, logs, and synthetics. Package installs have not used the current @lifecycle named versions of these, and since users can and have modified the legacy ILM policies, migrating users to the @lifecycle would introduce too much risk.

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against main? Unless there is a good reason otherwise, we prefer pull requests against main and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS and architecture that we support?
  • If you are submitting this code for a class then read our policy for that.

@elasticsearchmachine elasticsearchmachine added v9.2.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Aug 12, 2025
@MichelLosier MichelLosier marked this pull request as ready for review August 12, 2025 18:32
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Aug 12, 2025
@MichelLosier MichelLosier added >enhancement :Data Management/Indices APIs APIs to create and manage indices and templates labels Aug 12, 2025
@elasticsearchmachine elasticsearchmachine added Team:Data Management Meta label for data/management team and removed needs:triage Requires assignment of a team area label labels Aug 12, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Hi @MichelLosier, I've created a changelog YAML for you.

@felixbarny
Copy link
Member

This un-deprecates a lot more than just the ILM policies we talked about.

@MichelLosier
Copy link
Author

MichelLosier commented Aug 13, 2025

You're right, my mistake this PR does affect other deprecated templates -- I misread their relationship here with the ILM policies.

Aside from that, I original thought the scope of this could be any ILM policy type declared in a package that follows the @lifecycle convention, and so all legacy policies replaced by @lifecycle ones would be implicated, but reviewing package-spec we do enumerate specifically what ILM policies can be used.

Reviewing current integrations, I just see metrics, logs, and synthetics in use. So I can rescope this to just metrics, logs and synthetics ILM policies. On the kibana side, we can append @lifecycle to all other ILM policies (traces, profiling et c.).

Next steps:

I'm going to put this back in draft, and just move the deprecated metrics, logs, and synthetics into StackTemplateRegistry and revert deprecated back to true for LegacyStackTemplateRegistry.

@MichelLosier MichelLosier changed the title Undeprecate LegacyStackTemplateRegistry Undeprecate metrics, logs and synthetics ILM policies Aug 13, 2025
@MichelLosier MichelLosier marked this pull request as draft August 13, 2025 14:31
@dakrone
Copy link
Member

dakrone commented Aug 13, 2025

Before we merge this, I'd like to discuss with Felix about if there is a better way we can support this without having two copies of the policies existing in perpetuity.

@MichelLosier
Copy link
Author

After further discussion, we don't want to put ourselves in a position where we have both legacy and current ILM policies in place which may also confuse the user. We'll look to provide a migration experience instead for users and possibly look at removal for the next major ver.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Indices APIs APIs to create and manage indices and templates >enhancement external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants