Skip to content

[7.x] Adjust SearchableSnapshotsBlobStoreCacheIntegTests.testBlobStoreCache#77843

Merged
elasticsearchmachine merged 1 commit intoelastic:7.xfrom
tlrx:adjust-testBlobStoreCache-7.x
Sep 16, 2021
Merged

[7.x] Adjust SearchableSnapshotsBlobStoreCacheIntegTests.testBlobStoreCache#77843
elasticsearchmachine merged 1 commit intoelastic:7.xfrom
tlrx:adjust-testBlobStoreCache-7.x

Conversation

@tlrx
Copy link
Copy Markdown
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
Copy Markdown
Member Author

tlrx commented Sep 16, 2021

@elasticmachine run elasticsearch-ci/part-1

@tlrx
Copy link
Copy Markdown
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