Skip to content

Commit

Permalink
Fix full cluster restart test recovery (#29545)
Browse files Browse the repository at this point in the history
The test was using a parameter on GET /_cluster/health that older nodes
do not understand. Yet, we do no even need to make this call here, we
can use ensure green for the index.
  • Loading branch information
jasontedor committed Apr 18, 2018
1 parent a7c9857 commit a548a7f
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -687,8 +687,7 @@ public void testEmptyShard() throws IOException {
* Tests recovery of an index with or without a translog and the
* statistics we gather about that.
*/
@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/29544")
public void testRecovery() throws IOException {
public void testRecovery() throws Exception {
int count;
boolean shouldHaveTranslog;
if (runningAgainstOldCluster) {
Expand All @@ -701,7 +700,7 @@ public void testRecovery() throws IOException {
indexRandomDocuments(count, true, true, i -> jsonBuilder().startObject().field("field", "value").endObject());

// make sure all recoveries are done
ensureNoInitializingShards();
ensureGreen(index);
// Explicitly flush so we're sure to have a bunch of documents in the Lucene index
client().performRequest("POST", "/_flush");
if (shouldHaveTranslog) {
Expand Down

0 comments on commit a548a7f

Please sign in to comment.