diff options
author | Håkon Hallingstad <hakon@yahooinc.com> | 2022-02-04 16:25:10 +0100 |
---|---|---|
committer | Håkon Hallingstad <hakon@yahooinc.com> | 2022-02-04 16:25:10 +0100 |
commit | 0781f73dd5bffe1d805352214a9cc10a22ee4a59 (patch) | |
tree | 3b57178ab0ef59762fa7bf09e4eb61e6880a1122 | |
parent | bbbfcfa38c01142e8fd56b48fb08da30cd38383d (diff) |
Always create unique s3 bucket name
3 files changed, 3 insertions, 4 deletions
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/ArchiveService.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/ArchiveService.java index c939055ffa5..69eda662692 100644 --- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/ArchiveService.java +++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/ArchiveService.java @@ -15,7 +15,7 @@ import java.util.Set; */ public interface ArchiveService { - ArchiveBucket createArchiveBucketFor(ZoneId zoneId, boolean sharded); + ArchiveBucket createArchiveBucketFor(ZoneId zoneId); void updateBucketPolicy(ZoneId zoneId, ArchiveBucket bucket, Map<TenantName, String> authorizeIamRoleByTenantName); diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/MockArchiveService.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/MockArchiveService.java index 796ca46ae4e..ce7b56ad1f6 100644 --- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/MockArchiveService.java +++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/archive/MockArchiveService.java @@ -19,7 +19,7 @@ public class MockArchiveService implements ArchiveService { public Map<String, Set<String>> authorizedIamRolesForKey = new TreeMap<>(); @Override - public ArchiveBucket createArchiveBucketFor(ZoneId zoneId, boolean sharded) { + public ArchiveBucket createArchiveBucketFor(ZoneId zoneId) { return new ArchiveBucket("bucketName", "keyArn"); } diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/archive/CuratorArchiveBucketDb.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/archive/CuratorArchiveBucketDb.java index 3a232216d46..21914b87818 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/archive/CuratorArchiveBucketDb.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/archive/CuratorArchiveBucketDb.java @@ -72,8 +72,7 @@ public class CuratorArchiveBucketDb { } // We'll have to create a new bucket - var newBucket = archiveService.createArchiveBucketFor(zoneId, tenantsPerBucket().isPresent()) - .withTenant(tenant); + var newBucket = archiveService.createArchiveBucketFor(zoneId).withTenant(tenant); zoneBuckets.add(newBucket); curatorDb.writeArchiveBuckets(zoneId, zoneBuckets); updateArchiveUriCache(zoneId, zoneBuckets); |