summaryrefslogtreecommitdiffstats
path: root/configserver/src
diff options
context:
space:
mode:
authorJon Marius Venstad <venstad@gmail.com>2021-03-05 14:57:39 +0100
committerJon Marius Venstad <venstad@gmail.com>2021-03-05 14:57:39 +0100
commitb7b0692a1d8d75977c53167582cadd5cde711b21 (patch)
tree1b247d881178459fe8262862b368b0426dcbb159 /configserver/src
parent9fc9ff3dbf177b13a7c8cd83217e0541471ea513 (diff)
Always use dedicated CCs when hosted, and remove migrator
Diffstat (limited to 'configserver/src')
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java10
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationCuratorDatabase.java4
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ModelContextImpl.java7
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ActivatedModelsBuilder.java3
-rw-r--r--configserver/src/test/java/com/yahoo/vespa/config/server/ModelContextImplTest.java3
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),