Fix bug in snapshot update check for multiple v2 repo (opensearch-pro…

…ject#16379) Signed-off-by: Gaurav Bafna <[email protected]>
reta · Oct 18, 2024 · 9096aee · 9096aee
1 parent 0bded88
commit 9096aee
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/server/src/main/java/org/opensearch/repositories/RepositoriesService.java b/server/src/main/java/org/opensearch/repositories/RepositoriesService.java
@@ -81,6 +81,7 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Objects;
 import java.util.Set;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
@@ -729,7 +730,7 @@ public static void validateRepositoryMetadataSettings(
                         + " in the name as this delimiter is used to create pinning entity"
                 );
             }
-            if (repositoryWithShallowV2Exists(repositories)) {
+            if (repositoryWithShallowV2Exists(repositories, repositoryName)) {
                 throw new RepositoryException(
                     repositoryName,
                     "setting "
@@ -763,8 +764,13 @@ public static void validateRepositoryMetadataSettings(
         }
     }
 
-    private static boolean repositoryWithShallowV2Exists(Map<String, Repository> repositories) {
-        return repositories.values().stream().anyMatch(repo -> SHALLOW_SNAPSHOT_V2.get(repo.getMetadata().settings()));
+    private static boolean repositoryWithShallowV2Exists(Map<String, Repository> repositories, String repositoryName) {
+        return repositories.values()
+            .stream()
+            .anyMatch(
+                repository -> SHALLOW_SNAPSHOT_V2.get(repository.getMetadata().settings())
+                    && !Objects.equals(repository.getMetadata().name(), repositoryName)
+            );
     }
 
     private static boolean pinnedTimestampExistsWithDifferentRepository(

diff --git a/.../test/java/org/opensearch/repositories/blobstore/BlobStoreRepositoryRemoteIndexTests.java b/.../test/java/org/opensearch/repositories/blobstore/BlobStoreRepositoryRemoteIndexTests.java
@@ -431,6 +431,13 @@ public void testRepositoryCreationShallowV2() throws Exception {
             );
         }
 
+        // Modify repo-1 settings. This should go through
+        updateRepository(
+            client,
+            "test-repo-1",
+            Settings.builder().put(snapshotRepoSettings1).put("max_snapshot_bytes_per_sec", "10k").build()
+        );
+
         // Disable shallow snapshot V2 setting on test-repo-1
         updateRepository(
             client,