Skip to content

Commit

Permalink
Increase ensureGreen() timeout in CloseWhileRelocatingShardsIT (#53981)
Browse files Browse the repository at this point in the history
The test in CloseWhileRelocatingShardsIT failed recently 
multiple times (3) when waiting for initial indices to be 
become green. Looking at the execution logs from  #53544
 it appears at the very beginning of the test and when 
the WindowsFS file system is picked up (which is known 
to slow down tests).

This commit simply increases the timeout for the first 
ensureGreen() to 60 seconds. If the test continues to fail, 
we might want to test a larger timeout or disable 
WindowsFS for this test.

Closes #53544
  • Loading branch information
tlrx committed Mar 23, 2020
1 parent 53769f9 commit 6ddf5bf
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
import org.elasticsearch.cluster.routing.allocation.decider.ThrottlingAllocationDecider;
import org.elasticsearch.cluster.service.ClusterService;
import org.elasticsearch.common.settings.Settings;
import org.elasticsearch.common.unit.TimeValue;
import org.elasticsearch.common.util.concurrent.ConcurrentCollections;
import org.elasticsearch.index.shard.ShardId;
import org.elasticsearch.indices.recovery.PeerRecoverySourceService;
Expand Down Expand Up @@ -117,7 +118,7 @@ public void testCloseWhileRelocatingShards() throws Exception {
indices[i] = indexName;
}

ensureGreen(indices);
ensureGreen(TimeValue.timeValueSeconds(60L),indices);
assertAcked(client().admin().cluster().prepareUpdateSettings()
.setTransientSettings(Settings.builder()
.put(EnableAllocationDecider.CLUSTER_ROUTING_REBALANCE_ENABLE_SETTING.getKey(), Rebalance.NONE.toString())));
Expand Down

0 comments on commit 6ddf5bf

Please sign in to comment.