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.

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

[7.x][ML] Avoid direct action on system index in evaluate REST tests … #72074

Conversation

dimitris-athanasiou
Copy link
Contributor

…(#72069)

Refreshing all indices seems to potentially touch on the .ml-config
index which is a system index. That can cause a depracation warning
for directly accessing system indices. This commit changes the
refresh request to just be on the data index the test uses.

Closes #72046

Backport of #72069

…lastic#72069)

Refreshing all indices seems to potentially touch on the `.ml-config`
index which is a system index. That can cause a depracation warning
for directly accessing system indices. This commit changes the
refresh request to just be on the data index the test uses.

Closes elastic#72046

Backport of elastic#72069
@dimitris-athanasiou dimitris-athanasiou added >test Issues or PRs that are addressing/adding tests :ml Machine learning backport labels Apr 22, 2021
@elasticmachine elasticmachine added the Team:ML Meta label for the ML team label Apr 22, 2021
@elasticmachine
Copy link
Collaborator

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

@dimitris-athanasiou
Copy link
Contributor Author

This failed with a similar error as in #62439. I reopened that issue.

@dimitris-athanasiou
Copy link
Contributor Author

run elasticsearch-ci/1

@dimitris-athanasiou dimitris-athanasiou merged commit 4ed85fe into elastic:7.x Apr 22, 2021
@dimitris-athanasiou dimitris-athanasiou deleted the avoid-direct-action-on-system-index-evaluate-rest-tests-7x branch April 22, 2021 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :ml Machine learning Team:ML Meta label for the ML team >test Issues or PRs that are addressing/adding tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants