Unmute more memory-related tests after the fix in elastic#68542

danhermann · Feb 19, 2021 · 8fb0949 · 8fb0949
1 parent 9b81a7e
commit 8fb0949
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 21 deletions.
diff --git a/docs/reference/ml/anomaly-detection/apis/get-ml-info.asciidoc b/docs/reference/ml/anomaly-detection/apis/get-ml-info.asciidoc
@@ -126,4 +126,3 @@ This is a possible response:
 // TESTRESPONSE[s/"build_hash": "99a07c016d5a73"/"build_hash": "$body.native_code.build_hash"/]
 // TESTRESPONSE[s/"effective_max_model_memory_limit": "28961mb"/"effective_max_model_memory_limit": "$body.limits.effective_max_model_memory_limit"/]
 // TESTRESPONSE[s/"total_ml_memory": "86883mb"/"total_ml_memory": "$body.limits.total_ml_memory"/]
-// TESTRESPONSE[skip:"AwaitsFix https://github.com/elastic/elasticsearch/issues/66629"]
diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.stats/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/cluster.stats/10_basic.yml
@@ -1,9 +1,5 @@
 ---
 "cluster stats test":
-  - skip:
-      version: "all"
-      reason: "https://github.com/elastic/elasticsearch/issues/66629"
-
   - do:
       cluster.stats: {}
 
@@ -37,10 +33,8 @@
 ---
 "get cluster stats returns cluster_uuid at the top level":
   - skip:
-      version: "all"
-      reason: "https://github.com/elastic/elasticsearch/issues/66629"
-      #version: " - 6.99.99"
-      #reason:  "cluster stats including cluster_uuid at the top level is new in v6.5.0 and higher"
+      version: " - 6.99.99"
+      reason:  "cluster stats including cluster_uuid at the top level is new in v6.5.0 and higher"
 
   - do:
       cluster.stats: {}

diff --git a/server/src/test/java/org/elasticsearch/monitor/os/OsProbeTests.java b/server/src/test/java/org/elasticsearch/monitor/os/OsProbeTests.java
@@ -8,15 +8,8 @@
 
 package org.elasticsearch.monitor.os;
 
-import static org.hamcrest.Matchers.allOf;
-import static org.hamcrest.Matchers.anyOf;
-import static org.hamcrest.Matchers.both;
-import static org.hamcrest.Matchers.equalTo;
-import static org.hamcrest.Matchers.greaterThan;
-import static org.hamcrest.Matchers.greaterThanOrEqualTo;
-import static org.hamcrest.Matchers.is;
-import static org.hamcrest.Matchers.lessThanOrEqualTo;
-import static org.hamcrest.Matchers.notNullValue;
+import org.apache.lucene.util.Constants;
+import org.elasticsearch.test.ESTestCase;
 
 import java.io.IOException;
 import java.math.BigInteger;
@@ -26,8 +19,15 @@
 import java.util.Locale;
 import java.util.stream.Collectors;
 
-import org.apache.lucene.util.Constants;
-import org.elasticsearch.test.ESTestCase;
+import static org.hamcrest.Matchers.allOf;
+import static org.hamcrest.Matchers.anyOf;
+import static org.hamcrest.Matchers.both;
+import static org.hamcrest.Matchers.equalTo;
+import static org.hamcrest.Matchers.greaterThan;
+import static org.hamcrest.Matchers.greaterThanOrEqualTo;
+import static org.hamcrest.Matchers.is;
+import static org.hamcrest.Matchers.lessThanOrEqualTo;
+import static org.hamcrest.Matchers.notNullValue;
 
 public class OsProbeTests extends ESTestCase {
 
@@ -73,7 +73,6 @@ List<String> readOsRelease() throws IOException {
         assertThat(info.getAvailableProcessors(), equalTo(Runtime.getRuntime().availableProcessors()));
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/66629")
     public void testOsStats() {
         final OsProbe osProbe = new OsProbe();
         OsStats stats = osProbe.osStats();