Skip to content

Conversation

@davidkyle
Copy link
Member

@davidkyle davidkyle commented Nov 14, 2025

#135270 changed the template but without a corresponding version bump the template is not updated.

The version test is here

if (templateConfig != null && templateConfig.version().equals(installedTemplate.version()) == false) {

@davidkyle davidkyle added the :ml Machine learning label Nov 14, 2025
Copy link
Contributor

@valeriy42 valeriy42 left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticsearchmachine
Copy link
Collaborator

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

*/
@SuppressWarnings("unchecked")
public static void assertLegacyTemplateMatchesIndexMappings(
Copy link
Member Author

Choose a reason for hiding this comment

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

This function was not use

@davidkyle davidkyle marked this pull request as ready for review November 14, 2025 16:01
@davidkyle davidkyle enabled auto-merge (squash) November 14, 2025 16:01
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Nov 14, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

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

Labels

auto-backport Automatically create backport pull requests when merged >bug :ml Machine learning Team:ML Meta label for the ML team v8.19.8 v9.1.8 v9.2.2 v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants