diff options
author | Jon Bratseth <bratseth@gmail.com> | 2021-06-07 21:09:35 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2021-06-07 21:09:35 +0200 |
commit | 901d5aef064f250c43ebc11fbe95faa0a85ab802 (patch) | |
tree | bb591331506b326ba0af6cb40a536be26ed55baa /config-model | |
parent | bcdd3565654fb7fe7bdcd17f6a5a5904088c8e82 (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.java | 4 |
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 " + |