Remove assertion on Settings

Signed-off-by: Shivansh Arora <[email protected]>
shiv0408 · Oct 28, 2024 · 953282e · 953282e
1 parent 1fa4eb2
commit 953282e
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/qa/rolling-upgrade/src/test/java/org/opensearch/upgrades/ClusterStateIT.java b/qa/rolling-upgrade/src/test/java/org/opensearch/upgrades/ClusterStateIT.java
@@ -21,7 +21,6 @@
 
 @SuppressWarnings("unchecked")
 public class ClusterStateIT extends AbstractRollingTestCase{
-    private static final Logger logger = LogManager.getLogger(ClusterStateIT.class);
     private static final String templateName = "my_template";
     private static final String indexName = "test_cluster_state";
     private static final String componentTemplateName = "test_component_template";
@@ -69,7 +68,8 @@ public void testUpgradeWithRemotePublicationEnabled() throws Exception {
             verifyTemplateMetadataInClusterState(response);
             verifyComponentTemplateInClusterState(response);
             verifyComposableTemplateInClusterState(response);
-            verifySettingsInClusterState();
+//            Settings are required to wiped after Tests, so Settings are not present after rolling upgrade starts
+//            verifySettingsInClusterState();
         }
     }
 
@@ -187,7 +187,7 @@ private static void addPersistentSettings() throws Exception {
             "}";
 
         putSettings.setJsonEntity(settingsJson);
-        client().performRequest(putSettings);
+        assertOK(client().performRequest(putSettings));
     }
 
     private static void addTransientSettings() throws Exception {
@@ -207,7 +207,7 @@ private static void addTransientSettings() throws Exception {
             "}";
 
         putSettings.setJsonEntity(settingsJson);
-        client().performRequest(putSettings);
+        assertOK(client().performRequest(putSettings));
     }
 
     private static void verifyIndexInClusterState(Response clusterStateResponse) throws Exception {
@@ -221,7 +221,6 @@ private static void verifyIndexInClusterState(Response clusterStateResponse) thr
         Map<String, Object> index = (Map<String, Object>) indices.get(indexName);
         assertNotNull("Index " + indexName + " should exist in cluster state", index);
 
-        logger.info(index);
         // Verify index settings
         Map<String, Object> settings = (Map<String, Object>) index.get("settings");
         assertNotNull("Settings should exist in index", settings);