diff options
3 files changed, 8 insertions, 15 deletions
diff --git a/config-provisioning/src/main/java/com/yahoo/config/provision/CloudAccount.java b/config-provisioning/src/main/java/com/yahoo/config/provision/CloudAccount.java index 91da7866603..9f6e9b37c0a 100644 --- a/config-provisioning/src/main/java/com/yahoo/config/provision/CloudAccount.java +++ b/config-provisioning/src/main/java/com/yahoo/config/provision/CloudAccount.java @@ -83,8 +83,6 @@ public class CloudAccount implements Comparable<CloudAccount> { return empty; if (META_BY_CLOUD.get("aws").matches(cloudAccount)) return new CloudAccount(cloudAccount, CloudName.AWS); - if (META_BY_CLOUD.get("gcp").matches(cloudAccount)) // TODO (freva): Remove July 2023 - return new CloudAccount(cloudAccount, CloudName.GCP); throw illegal(cloudAccount, "Must be on format '<cloud-name>:<account>' or 'default'"); } diff --git a/config-provisioning/src/test/java/com/yahoo/config/provision/CloudAccountTest.java b/config-provisioning/src/test/java/com/yahoo/config/provision/CloudAccountTest.java index 2a994ac607e..fafea04c9ec 100644 --- a/config-provisioning/src/test/java/com/yahoo/config/provision/CloudAccountTest.java +++ b/config-provisioning/src/test/java/com/yahoo/config/provision/CloudAccountTest.java @@ -32,17 +32,12 @@ class CloudAccountTest { @Test void gcp_accounts() { - CloudAccount oldFormat = CloudAccount.from("my-project"); - CloudAccount newFormat = CloudAccount.from("gcp:my-project"); - assertEquals(oldFormat, newFormat); - - for (CloudAccount account : List.of(oldFormat, newFormat)) { - assertFalse(account.isUnspecified()); - assertEquals(account, CloudAccount.from(account.value())); - assertEquals("my-project", account.account()); - assertEquals(CloudName.GCP, account.cloudName()); - assertEquals("gcp:my-project", account.value()); - } + CloudAccount account = CloudAccount.from("gcp:my-project"); + assertFalse(account.isUnspecified()); + assertEquals(account, CloudAccount.from(account.value())); + assertEquals("my-project", account.account()); + assertEquals(CloudName.GCP, account.cloudName()); + assertEquals("gcp:my-project", account.value()); } @Test diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/ArchiveBucketsSerializerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/ArchiveBucketsSerializerTest.java index 1d1b1124d22..2e308e926b0 100644 --- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/ArchiveBucketsSerializerTest.java +++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/ArchiveBucketsSerializerTest.java @@ -20,8 +20,8 @@ public class ArchiveBucketsSerializerTest { ArchiveBuckets archiveBuckets = new ArchiveBuckets( Set.of(new VespaManagedArchiveBucket("bucket1Name", "key1Arn").withTenants(Set.of(TenantName.from("t1"), TenantName.from("t2"))), new VespaManagedArchiveBucket("bucket2Name", "key2Arn").withTenant(TenantName.from("t3"))), - Set.of(new TenantManagedArchiveBucket("bucket3Name", CloudAccount.from("acct-1"), Instant.ofEpochMilli(1234)), - new TenantManagedArchiveBucket("bucket4Name", CloudAccount.from("acct-2"), Instant.ofEpochMilli(5678)))); + Set.of(new TenantManagedArchiveBucket("bucket3Name", CloudAccount.from("gcp:acct-1"), Instant.ofEpochMilli(1234)), + new TenantManagedArchiveBucket("bucket4Name", CloudAccount.from("gcp:acct-2"), Instant.ofEpochMilli(5678)))); assertEquals(archiveBuckets, ArchiveBucketsSerializer.fromSlime(ArchiveBucketsSerializer.toSlime(archiveBuckets))); } |