diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2021-03-05 14:57:39 +0100 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2021-03-05 14:57:39 +0100 |
commit | b7b0692a1d8d75977c53167582cadd5cde711b21 (patch) | |
tree | 1b247d881178459fe8262862b368b0426dcbb159 /configserver/src | |
parent | 9fc9ff3dbf177b13a7c8cd83217e0541471ea513 (diff) |
Always use dedicated CCs when hosted, and remove migrator
Diffstat (limited to 'configserver/src')
5 files changed, 3 insertions, 24 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java b/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java index 54ff693dbf5..98d0c32c5d5 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java @@ -1064,16 +1064,6 @@ public class ApplicationRepository implements com.yahoo.config.provision.Deploye @Override public Duration serverDeployTimeout() { return Duration.ofSeconds(configserverConfig.zookeeper().barrierTimeout()); } - @Override - public void setDedicatedClusterControllerCluster(ApplicationId id) { - requireDatabase(id).setDedicatedClusterControllerCluster(id); - } - - @Override - public boolean getDedicatedClusterControllerCluster(ApplicationId id) { - return requireDatabase(id).getDedicatedClusterControllerCluster(id); - } - private static void logConfigChangeActions(ConfigChangeActions actions, DeployLogger logger) { RestartActions restartActions = actions.getRestartActions(); if ( ! restartActions.isEmpty()) { diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationCuratorDatabase.java b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationCuratorDatabase.java index 4704958781e..97b96fa91a0 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationCuratorDatabase.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationCuratorDatabase.java @@ -111,10 +111,6 @@ public class ApplicationCuratorDatabase { : data.map(bytes -> Long.parseLong(Utf8.toString(bytes))); } - public boolean getDedicatedClusterControllerCluster(ApplicationId id) { - return curator.exists(dedicatedClusterControllerClusterPath(id)); - } - public void setDedicatedClusterControllerCluster(ApplicationId id) { curator.create(dedicatedClusterControllerClusterPath(id)); } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ModelContextImpl.java b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ModelContextImpl.java index 8e6d51a55df..5f89903bf07 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ModelContextImpl.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ModelContextImpl.java @@ -260,7 +260,6 @@ public class ModelContextImpl implements ModelContext { private final Quota quota; private final List<TenantSecretStore> tenantSecretStores; private final SecretStore secretStore; - private final boolean dedicatedClusterControllerCluster; private final String jvmGcOptions; @@ -276,8 +275,7 @@ public class ModelContextImpl implements ModelContext { Optional<ApplicationRoles> applicationRoles, Optional<Quota> maybeQuota, List<TenantSecretStore> tenantSecretStores, - SecretStore secretStore, - boolean dedicatedClusterControllerCluster) { + SecretStore secretStore) { this.featureFlags = new FeatureFlags(flagSource, applicationId); this.applicationId = applicationId; this.multitenant = configserverConfig.multitenant() || configserverConfig.hostedVespa() || Boolean.getBoolean("multitenant"); @@ -294,7 +292,6 @@ public class ModelContextImpl implements ModelContext { this.athenzDomain = athenzDomain; this.applicationRoles = applicationRoles; this.quota = maybeQuota.orElseGet(Quota::unlimited); - this.dedicatedClusterControllerCluster = dedicatedClusterControllerCluster; this.tenantSecretStores = tenantSecretStores; this.secretStore = secretStore; @@ -360,8 +357,6 @@ public class ModelContextImpl implements ModelContext { @Override public String jvmGCOptions() { return jvmGcOptions; } - @Override public boolean dedicatedClusterControllerCluster() { return hostedVespa && dedicatedClusterControllerCluster; } - private static <V> V flagValue(FlagSource source, ApplicationId appId, UnboundFlag<? extends V, ?, ?> flag) { return flag.bindTo(source) .with(FetchVector.Dimension.APPLICATION_ID, appId.serializedForm()) diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ActivatedModelsBuilder.java b/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ActivatedModelsBuilder.java index fd04d4b58fc..cfc47223b11 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ActivatedModelsBuilder.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ActivatedModelsBuilder.java @@ -164,8 +164,7 @@ public class ActivatedModelsBuilder extends ModelsBuilder<Application> { .readApplicationRoles(applicationId), zkClient.readQuota(), zkClient.readTenantSecretStores(), - secretStore, - zkClient.readDedicatedClusterControllerCluster()); + secretStore); } } diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/ModelContextImplTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/ModelContextImplTest.java index 0d066f748c8..7b9420b6b9e 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/ModelContextImplTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/ModelContextImplTest.java @@ -74,8 +74,7 @@ public class ModelContextImplTest { Optional.empty(), Optional.empty(), List.of(), - new SecretStoreProvider().get(), - false), + new SecretStoreProvider().get()), Optional.empty(), Optional.empty(), new Version(7), |