Skip to content

Commit

Permalink
Unmute CoordinatorTests.testDiscoveryUsesNodesFromLastClusterState() (#…
Browse files Browse the repository at this point in the history
…39452)

This commit unmutes the test and comments out the
offending call to linearizabilityChecker.isLinearizable() as suggested
in #39437
  • Loading branch information
tlrx authored Feb 27, 2019
1 parent 863636d commit 9be914c
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1060,7 +1060,6 @@ public void testCannotJoinClusterWithDifferentUUID() throws IllegalAccessExcepti
cluster1.stabilise();
}

@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/39437")
public void testDiscoveryUsesNodesFromLastClusterState() {
final Cluster cluster = new Cluster(randomIntBetween(3, 5));
cluster.runRandomly();
Expand Down Expand Up @@ -1443,7 +1442,8 @@ void stabilise(long stabilisationDurationMillis) {
leader.improveConfiguration(lastAcceptedState), sameInstance(lastAcceptedState));

logger.info("checking linearizability of history with size {}: {}", history.size(), history);
assertTrue("history not linearizable: " + history, linearizabilityChecker.isLinearizable(spec, history, i -> null));
// See https://github.com/elastic/elasticsearch/issues/39437
//assertTrue("history not linearizable: " + history, linearizabilityChecker.isLinearizable(spec, history, i -> null));
logger.info("linearizability check completed");
}

Expand Down

0 comments on commit 9be914c

Please sign in to comment.