summaryrefslogtreecommitdiffstats
path: root/config-model
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@gmail.com>2021-06-07 21:09:35 +0200
committerJon Bratseth <bratseth@gmail.com>2021-06-07 21:09:35 +0200
commit901d5aef064f250c43ebc11fbe95faa0a85ab802 (patch)
treebb591331506b326ba0af6cb40a536be26ed55baa /config-model
parentbcdd3565654fb7fe7bdcd17f6a5a5904088c8e82 (diff)
Only validate on hosted, and only if there are multiple nodes per group
Diffstat (limited to 'config-model')
-rw-r--r--config-model/src/main/java/com/yahoo/vespa/model/application/validation/first/RedundancyOnFirstDeploymentValidator.java4
1 files changed, 3 insertions, 1 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/first/RedundancyOnFirstDeploymentValidator.java b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/first/RedundancyOnFirstDeploymentValidator.java
index acffb0cf292..e6117299269 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/first/RedundancyOnFirstDeploymentValidator.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/first/RedundancyOnFirstDeploymentValidator.java
@@ -27,10 +27,12 @@ public class RedundancyOnFirstDeploymentValidator extends Validator {
@Override
public void validate(VespaModel model, DeployState deployState) {
+ if ( ! deployState.isHosted()) return;
if ( ! deployState.zone().environment().isProduction()) return;
for (ContentCluster cluster : model.getContentClusters().values()) {
- if (cluster.redundancy().finalRedundancy() == 1)
+ if (cluster.redundancy().finalRedundancy() == 1
+ && cluster.redundancy().totalNodes() > cluster.redundancy().groups())
deployState.validationOverrides().invalid(ValidationId.redundancyOne,
cluster + " has redundancy 1, which will cause it to lose data " +
"if a node fails. This requires an override on first deployment " +