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.

Sign up for GitHub

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

Jump to bottom

BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackgroundMerges #89412

Closed
astefan opened this issue Aug 17, 2022 · 1 comment · Fixed by #89416
Closed

BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackgroundMerges #89412

astefan opened this issue Aug 17, 2022 · 1 comment · Fixed by #89416
Assignees
@original-brownbear
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-failure Triaged test failures from CI

Comments

@astefan
Copy link
Contributor

astefan commented Aug 17, 2022

CI Link

https://gradle-enterprise.elastic.co/s/pzjicoqh7nuhq

Repro line

./gradlew ':server:internalClusterTest' --tests "org.elasticsearch.snapshots.BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackgroundMerges" -Dtests.seed=DEA1040A4A18200C -Dtests.locale=ar-KW -Dtests.timezone=Etc/Universal -Druntime.java=17

Does it reproduce?

Yes

Applicable branches

main, 8.4

Failure history

First time it fails in the last 7 days: https://gradle-enterprise.elastic.co/scans/tests?search.timeZoneId=Europe/Bucharest&tests.container=org.elasticsearch.snapshots.BlobStoreIncrementalityIT&tests.test=testRecordCorrectSegmentCountsWithBackgroundMerges

Failure excerpt

org.elasticsearch.snapshots.BlobStoreIncrementalityIT > testRecordCorrectSegmentCountsWithBackgroundMerges FAILED
    java.lang.AssertionError: 
    Expected: a value less than <3L>
         but: <3L> was equal to <3L>
        at __randomizedtesting.SeedInfo.seed([DEA1040A4A18200C:C81912C0892C8B7B]:0)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
        at org.junit.Assert.assertThat(Assert.java:956)
        at org.junit.Assert.assertThat(Assert.java:923)
        at org.elasticsearch.snapshots.BlobStoreIncrementalityIT.lambda$testRecordCorrectSegmentCountsWithBackgroundMerges$0(BlobStoreIncrementalityIT.java:236)
        at org.elasticsearch.test.ESTestCase.assertBusy(ESTestCase.java:1104)
        at org.elasticsearch.snapshots.BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackgroundMerges(BlobStoreIncrementalityIT.java:234)
@astefan astefan added :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test-failure Triaged test failures from CI labels Aug 17, 2022
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Aug 17, 2022
@original-brownbear original-brownbear self-assigned this Aug 17, 2022
@original-brownbear original-brownbear mentioned this issue Aug 17, 2022
Merged
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this issue Aug 17, 2022
@original-brownbear
Fix BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackg…
f15ff12 
…roundMerges (elastic#89416)

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

closes elastic#89412
elasticsearchmachine pushed a commit that referenced this issue Aug 17, 2022
@original-brownbear
Fix BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackg…
28a3b72 
…roundMerges (#89416) (#89423)

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

closes #89412
elasticmachine pushed a commit to mushao999/elasticsearch that referenced this issue Aug 17, 2022
@original-brownbear
Fix BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackg…
2a08258 
…roundMerges (elastic#89416)

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

closes elastic#89412
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this issue Nov 30, 2024
@original-brownbear
Fix BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackg…
ce7ad0e 
…roundMerges

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

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

@original-brownbear original-brownbear

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-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
3 participants
@astefan @original-brownbear @elasticsearchmachine