Skip to content

Conversation

@pxsalehi
Copy link
Member

The inference index added to the delete index call doesn't exist in all 8.x versions. AFAICT, since this is not a pattern, the wipeAllIndices call fails since it is not able to find that index. Using a wildcard instead seems to resolve the issue.

Closes #114994

@pxsalehi pxsalehi added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs labels Oct 21, 2024
@pxsalehi pxsalehi requested review from prwhelan and ywangd October 21, 2024 13:30
@elasticsearchmachine elasticsearchmachine added v9.0.0 Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. labels Oct 21, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

…ersionRepositoryAccessIT.testUpgradeMovesRepoToNewMetaVersion
@pxsalehi pxsalehi changed the title Use pattern in wipeAllIndices and unmute testUpgradeMovesRepoToNewMetaVersion Unmute testUpgradeMovesRepoToNewMetaVersion Oct 21, 2024
@pxsalehi
Copy link
Member Author

The inference index and the corresponding line from wipeAllIndices was removed in #115023.

@pxsalehi pxsalehi requested a review from davidkyle October 21, 2024 14:47
Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@pxsalehi pxsalehi merged commit 477f0cd into elastic:main Oct 22, 2024
16 checks passed
georgewallace pushed a commit to georgewallace/elasticsearch that referenced this pull request Oct 25, 2024
…aVersion (elastic#115232)

The inference index added to the delete index call doesn't exist in all 8.x versions. AFAICT, since this is not a pattern, the wipeAllIndices call fails since it is not able to find that index. Using a wildcard instead seems to resolve the issue.

Closes elastic#114994
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request Nov 4, 2024
…aVersion (elastic#115232)

The inference index added to the delete index call doesn't exist in all 8.x versions. AFAICT, since this is not a pattern, the wipeAllIndices call fails since it is not able to find that index. Using a wildcard instead seems to resolve the issue.

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

Labels

:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. >test Issues or PRs that are addressing/adding tests v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] MultiVersionRepositoryAccessIT testUpgradeMovesRepoToNewMetaVersion failing

3 participants