Fix to include static settings in addition to dynamic repo metadata s…

…ettings during s3 plugin reload (#10452) (#10456) (cherry picked from commit 4fee40d) Signed-off-by: vikasvb90 <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Oct 9, 2023 · 6c04459 · 6c04459
1 parent dd8c51d
commit 6c04459
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/plugins/repository-s3/src/main/java/org/opensearch/repositories/s3/S3Repository.java b/plugins/repository-s3/src/main/java/org/opensearch/repositories/s3/S3Repository.java
@@ -460,9 +460,8 @@ public void reload(RepositoryMetadata newRepositoryMetadata) {
         readRepositoryMetadata();
 
         // Reload configs for S3RepositoryPlugin
-        final Map<String, S3ClientSettings> clientsSettings = S3ClientSettings.load(metadata.settings(), pluginConfigPath);
-        service.refreshAndClearCache(clientsSettings);
-        s3AsyncService.refreshAndClearCache(clientsSettings);
+        service.settings(metadata);
+        s3AsyncService.settings(metadata);
 
         // Reload configs for S3BlobStore
         BlobStore blobStore = getBlobStore();