Skip to content

Commit 7bebb5e

Browse files
committed
Fix testRepositoryAnalysis (elastic#69538)
This test would occasionally generate an invalid request in which the max total data size cannot be satisfied. This commit ensures the max total data size is always sufficiently large. Relates elastic#67247 Closes elastic#69219
1 parent d93aaf3 commit 7bebb5e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ protected Collection<Class<? extends Plugin>> nodePlugins() {
7373
);
7474
}
7575

76-
@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/69219")
7776
public void testRepositoryAnalysis() {
7877

7978
createRepositoryNoVerify("test-repo", TestPlugin.ASSERTING_REPO_TYPE);
@@ -106,7 +105,9 @@ public void testRepositoryAnalysis() {
106105
}
107106

108107
if (usually()) {
109-
request.maxTotalDataSize(new ByteSizeValue(between(1, 1 << 20)));
108+
request.maxTotalDataSize(
109+
new ByteSizeValue(request.getMaxBlobSize().getBytes() + request.getBlobCount() - 1 + between(0, 1 << 20))
110+
);
110111
blobStore.ensureMaxTotalBlobSize(request.getMaxTotalDataSize().getBytes());
111112
}
112113

0 commit comments

Comments
 (0)