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.14] Fix testBlobStoreCache (#79747) #79889

Closed
wants to merge 2 commits into from
Closed

Conversation

tlrx
Copy link
Member

@tlrx tlrx commented Oct 27, 2021

The attempt to fix this test in #78616 wasn't complete: we need to wait
for all mounted indices, not only the last restored one, before deleting
the indices. Otherwise one of the mounted index might still be
initializing and recreates the cached documents in
.snapshot-blob-cache

Closes #78993

The attempt to fix this test in elastic#78616 wasn't complete: we need to wait
for all mounted indices, not only the last restored one, before deleting
the indices. Otherwise one of the mounted index might still be
initializing and recreates the cached documents in
`.snapshot-blob-cache`  Closes elastic#78993
@tlrx tlrx added backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v7.14.3 labels Oct 27, 2021
    I backported some fixes in elastic#79888 but the fix relies
    on index names that should start by "restored-" but
    this change was part of another change () that was
    not backported, on purpose, to 7.14.

    This pull request adapts the index names so that they
    can beneficiate from the earlier fix elastic#79888.
@tlrx tlrx removed the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Oct 29, 2021
@tlrx tlrx closed this Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant