aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2018-07-03 11:10:30 +0200
committerGitHub <noreply@github.com>2018-07-03 11:10:30 +0200
commit610112d4be0016793c9bd3b79e73625e018afcff (patch)
treebfd7e0c5a4c8e275fba01e9f1c36f0cab342dafa
parentac35474116eeae036bab468a4314ed3d7295a6e6 (diff)
parent5ad6d3649b72d83955bcea9c5c911e2b85f45aab (diff)
Merge pull request #6322 from vespa-engine/hmusum/disable-loading-minimal-set-of-config-model-versions-for-prod
Disable loading minimal set of config model versions in prod
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ModelsBuilder.java7
1 files changed, 6 insertions, 1 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ModelsBuilder.java b/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ModelsBuilder.java
index 969a2d49586..090e7aac494 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ModelsBuilder.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/modelfactory/ModelsBuilder.java
@@ -8,6 +8,7 @@ import com.yahoo.config.model.api.ModelFactory;
import com.yahoo.config.provision.AllocatedHosts;
import com.yahoo.config.provision.ApplicationId;
import com.yahoo.config.provision.ApplicationLockException;
+import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.OutOfCapacityException;
import com.yahoo.config.provision.Version;
import com.yahoo.config.provision.Zone;
@@ -19,6 +20,7 @@ import com.yahoo.vespa.config.server.provision.StaticProvisioner;
import java.time.Instant;
import java.util.ArrayList;
+import java.util.Arrays;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;
@@ -140,7 +142,10 @@ public abstract class ModelsBuilder<MODELRESULT extends ModelResult> {
List<MODELRESULT> allApplicationVersions = new ArrayList<>();
allApplicationVersions.add(latestModelVersion);
- versions = keepThoseUsedOn(allocatedHosts.get(), versions);
+ // TODO: Enable for all zones
+ // Note: Cannot be enabled for prod zones yet, due to an issue with how AccessControlValidator works
+ if (Arrays.asList(Environment.dev, Environment.test, Environment.staging).contains(zone().environment()))
+ versions = keepThoseUsedOn(allocatedHosts.get(), versions);
// Make sure we build wanted version if we are building models for this major version
Version wantedVersion = Version.fromIntValues(wantedNodeVespaVersion.getMajor(), wantedNodeVespaVersion.getMinor(), wantedNodeVespaVersion.getMicro());