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] Adjust SearchableSnapshotsBlobStoreCacheIntegTests.testBlobStoreCache #77843

Merged

Conversation

tlrx
Copy link
Member

@tlrx tlrx commented Sep 16, 2021

This test sometimes fails (#77753) when it tries to refresh the
snapshot blob cache index while it wasn't created at all due to
the test index being completely empty with 0 documents indexed.

Having an empty index in this test does not make much sense
because the index will contain no segments and therefore no
segments file to cache in the system index.

This commit adjusts the test to always index at least
some docs.

Backport of #77758

…elastic#77758)

This test sometimes fails (elastic#77753) when it tries to refresh the
snapshot blob cache index while it wasn't created at all due to
the test index being completely empty with 0 documents indexed.

Having an empty index in this test does not make much sense
because the index will contain no segments and therefore no
segments file to cache in the system index.

This commit adjusts the test to always index at least
some docs.

Backport of elastic#77758
@tlrx tlrx added >test Issues or PRs that are addressing/adding tests backport v7.16.0 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Sep 16, 2021
@tlrx
Copy link
Member Author

tlrx commented Sep 16, 2021

@elasticmachine run elasticsearch-ci/part-1

@tlrx
Copy link
Member Author

tlrx commented Sep 16, 2021

@elasticmachine run elasticsearch-ci/packaging-tests-windows-sample

@elasticsearchmachine elasticsearchmachine merged commit 3bf1b9b into elastic:7.x Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >test Issues or PRs that are addressing/adding tests v7.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants