Skip to content

Commit 0713b58

Browse files
authored
Tighten up assertion in shard write load test (elastic#131659)
1 parent f3d74f3 commit 0713b58

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

server/src/internalClusterTest/java/org/elasticsearch/index/shard/IndexShardIT.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -396,16 +396,16 @@ public void testShardWriteLoadsArePresent() {
396396
// Verify that each shard has write-load reported.
397397
final ClusterState state = getInstanceFromNode(ClusterService.class).state();
398398
assertEquals(state.projectState(ProjectId.DEFAULT).metadata().getTotalNumberOfShards(), shardWriteLoads.size());
399-
double maximumLoadRecorded = 0;
400399
for (IndexMetadata indexMetadata : state.projectState(ProjectId.DEFAULT).metadata()) {
400+
double maximumLoadRecorded = 0;
401401
for (int i = 0; i < indexMetadata.getNumberOfShards(); i++) {
402402
final ShardId shardId = new ShardId(indexMetadata.getIndex(), i);
403403
assertTrue(shardWriteLoads.containsKey(shardId));
404404
maximumLoadRecorded = Math.max(shardWriteLoads.get(shardId), maximumLoadRecorded);
405405
}
406+
// Each index should have seen some write-load
407+
assertThat(maximumLoadRecorded, greaterThan(0.0));
406408
}
407-
// And that at least one is greater than zero
408-
assertThat(maximumLoadRecorded, greaterThan(0.0));
409409
} finally {
410410
updateClusterSettings(
411411
Settings.builder().putNull(WriteLoadConstraintSettings.WRITE_LOAD_DECIDER_ENABLED_SETTING.getKey()).build()

0 commit comments

Comments
 (0)