Apply suggestions from code review

Signed-off-by: bowenlan-amzn <[email protected]>
opensearch-project · Feb 4, 2025 · f6a130a · f6a130a
1 parent ad2396b
commit f6a130a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/build.gradle b/build.gradle
@@ -586,7 +586,6 @@ integTest {
             excludeTestsMatching "org.opensearch.indexmanagement.indexstatemanagement.action.NotificationActionIT"
         }
         exclude 'org/opensearch/indexmanagement/indexstatemanagement/MetadataRegressionIT.class'
-        exclude 'org/opensearch/indexmanagement/indexstatemanagement/ConvertIndexToRemoteIT.class'
     }
 
     // TODO: raise issue in Core, this is because of the test framework
@@ -648,7 +647,6 @@ task integTestRemote(type: RestIntegTestTask) {
 
     // Snapshot action integration tests rely on node level setting path.repo which we can't set remotely
     exclude 'org/opensearch/indexmanagement/indexstatemanagement/action/SnapshotActionIT.class'
-    exclude 'org/opensearch/indexmanagement/indexstatemanagement/action/ConvertIndexToRemoteIT.class'
 }
 
 // === Set up BWC tests ===

diff --git a/...in/org/opensearch/indexmanagement/indexstatemanagement/step/restore/AttemptRestoreStep.kt b/...in/org/opensearch/indexmanagement/indexstatemanagement/step/restore/AttemptRestoreStep.kt
@@ -136,7 +136,7 @@ class AttemptRestoreStep(private val action: ConvertIndexToRemoteAction) : Step(
     ): String {
         val contextMap =
             managedIndexMetaData.convertToMap().filterKeys { key ->
-                key in AttemptSnapshotStep.validTopContextFields
+                key in validTopContextFields
             }
         val compiledValue =
             scriptService.compile(template, TemplateScript.CONTEXT)