Skip to content

Commit d2339da

Browse files
committed
RepositoryAnalysisFailureIT: disrupt earlier
Fixes #127029 The fix to #126747 was only for one test. This applies that change to all the tests in this suite that need it.
1 parent 6335391 commit d2339da

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -411,9 +411,6 @@ tests:
411411
- class: org.elasticsearch.packaging.test.DockerTests
412412
method: test024InstallPluginFromArchiveUsingConfigFile
413413
issue: https://github.com/elastic/elasticsearch/issues/126936
414-
- class: org.elasticsearch.repositories.blobstore.testkit.analyze.RepositoryAnalysisFailureIT
415-
method: testFailsOnReadError
416-
issue: https://github.com/elastic/elasticsearch/issues/127029
417414

418415
# Examples:
419416
#

x-pack/plugin/snapshot-repo-test-kit/src/internalClusterTest/java/org/elasticsearch/repositories/blobstore/testkit/analyze/RepositoryAnalysisFailureIT.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ public void testFailsOnReadError() {
134134
request.maxBlobSize(ByteSizeValue.ofBytes(10L));
135135
request.abortWritePermitted(false);
136136

137-
final CountDown countDown = new CountDown(between(1, request.getBlobCount()));
137+
final CountDown countDown = new CountDown(between(1, request.getBlobCount() / 2));
138138
blobStore.setDisruption(new Disruption() {
139139
@Override
140140
public byte[] onRead(byte[] actualContents, long position, long length) throws IOException {
@@ -158,7 +158,7 @@ public void testFailsOnNotFoundAfterWrite() {
158158
request.abortWritePermitted(false);
159159
request.rareActionProbability(0.0); // not found on an early read or an overwrite is ok
160160

161-
final CountDown countDown = new CountDown(between(1, request.getBlobCount()));
161+
final CountDown countDown = new CountDown(between(1, request.getBlobCount() / 2));
162162

163163
blobStore.setDisruption(new Disruption() {
164164
@Override
@@ -212,7 +212,7 @@ public void testFailsOnChecksumMismatch() {
212212
// leads to CI failures. Therefore, we disable rare actions to improve CI stability.
213213
request.rareActionProbability(0.0);
214214

215-
final CountDown countDown = new CountDown(between(1, request.getBlobCount()));
215+
final CountDown countDown = new CountDown(between(1, request.getBlobCount() / 2));
216216

217217
blobStore.setDisruption(new Disruption() {
218218
@Override

0 commit comments

Comments
 (0)