Skip to content

Conversation

@edsavage
Copy link
Contributor

@edsavage edsavage commented Oct 13, 2025

Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be .ml-anomalies-shared. This may not always be the case going forward in which case REST compatibility tests will break.

The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.

Relates #136065

Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
@edsavage edsavage requested a review from valeriy42 October 13, 2025 02:00
@edsavage edsavage added >non-issue >test Issues or PRs that are addressing/adding tests :ml Machine learning auto-backport Automatically create backport pull requests when merged v8.19.6 v9.1.6 v9.0.9 v9.2.1 v9.3.0 labels Oct 13, 2025
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Oct 13, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@davidkyle davidkyle 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

💚 Backport successful

Status Branch Result
8.19
9.1
9.0
9.2
edsavage added a commit to edsavage/elasticsearch that referenced this pull request Oct 19, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
edsavage added a commit to edsavage/elasticsearch that referenced this pull request Oct 19, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
edsavage added a commit to edsavage/elasticsearch that referenced this pull request Oct 19, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
elasticsearchmachine pushed a commit that referenced this pull request Oct 19, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
elasticsearchmachine pushed a commit that referenced this pull request Oct 19, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
elasticsearchmachine pushed a commit that referenced this pull request Oct 19, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
@edsavage
Copy link
Contributor Author

Removing backport label for 9.0 as that branch is now "closed"

@edsavage edsavage removed the v9.0.9 label Oct 19, 2025
chrisparrinello pushed a commit to chrisparrinello/elasticsearch that referenced this pull request Oct 24, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
fzowl pushed a commit to voyage-ai/elasticsearch that referenced this pull request Nov 3, 2025
Some ML REST tests make explicit reference to the AD results index, making the assumption that the index name will be `.ml-anomalies-shared`. This may not always be the case going forward in which cae REST compatibility tests will break. The bulk of the references can be handled by the yamlRestCompatTest transformation rules but a small number must be manually rewritten.
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 :ml Machine learning >non-issue Team:ML Meta label for the ML team >test Issues or PRs that are addressing/adding tests v8.19.6 v9.1.6 v9.2.1 v9.3.0

3 participants