Attempt to fix broken ITs

Signed-off-by: Peter Alfonsi <petealft@amazon.com>
peteralfonsi · Aug 30, 2024 · bb66699 · bb66699
1 parent 58170bf
commit bb66699
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/server/src/internalClusterTest/java/org/opensearch/indices/IndicesRequestCacheIT.java b/server/src/internalClusterTest/java/org/opensearch/indices/IndicesRequestCacheIT.java
@@ -81,18 +81,14 @@ public static Collection<Object[]> parameters() {
             new Object[] { Settings.builder().put(CLUSTER_CONCURRENT_SEGMENT_SEARCH_SETTING.getKey(), true).build() }
         );
     }
-    /*
-    new Object[] { Settings.builder().put(FeatureFlags.PLUGGABLE_CACHE, "true").build() },
-            new Object[] { Settings.builder().put(FeatureFlags.PLUGGABLE_CACHE, "false").build() }
-     */
 
     @Override
     protected Settings featureFlagSettings() {
-        return Settings.builder()
-            .put(super.featureFlagSettings()).put(FeatureFlags.CONCURRENT_SEGMENT_SEARCH, "true")
-            .put(super.featureFlagSettings()).put(FeatureFlags.PLUGGABLE_CACHE, "true")
+        return Settings.builder().put(super.featureFlagSettings())
+            .put(FeatureFlags.CONCURRENT_SEGMENT_SEARCH, "true")
+            .put(FeatureFlags.PLUGGABLE_CACHE, "true")
             .build();
-    }
+    } // For now hardcode TC feature flag as true. Attempt to backport the changes allowing us to parameterize it
 
     // One of the primary purposes of the query cache is to cache aggs results
     public void testCacheAggs() throws Exception {