diff options
| author | Harald Musum <musum@verizonmedia.com> | 2021-02-08 15:37:15 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-02-08 15:37:15 +0100 |
| commit | 3cce2f1404a3055557898a3ed3ca9434f9b55f33 (patch) | |
| tree | 92327ff9ab72177f1d6ce5a21c8b559821fd2a56 | |
| parent | 71e7754a224dc97b16e2a04f3caa59ac762ddad3 (diff) | |
| parent | 2c515a94dd876464f0754950f1279d2dcd96bf11 (diff) | |
Merge pull request #16437 from vespa-engine/hmusum/allow-one-node-at-a-time-down-for-cluster-controller-cluster
Allow only one node down at a time for cluster controller clusters
| -rw-r--r-- | orchestrator/src/main/java/com/yahoo/vespa/orchestrator/policy/HostedVespaClusterPolicy.java | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/policy/HostedVespaClusterPolicy.java b/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/policy/HostedVespaClusterPolicy.java index d3e7afba25f..ff1c56f6b2f 100644 --- a/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/policy/HostedVespaClusterPolicy.java +++ b/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/policy/HostedVespaClusterPolicy.java @@ -56,8 +56,7 @@ public class HostedVespaClusterPolicy implements ClusterPolicy { } @Override - public void verifyGroupGoingDownPermanentlyIsFine(ClusterApi clusterApi) - throws HostStateChangeDeniedException { + public void verifyGroupGoingDownPermanentlyIsFine(ClusterApi clusterApi) throws HostStateChangeDeniedException { // This policy is similar to verifyGroupGoingDownIsFine, except that services being down in the group // is no excuse to allow suspension (like it is for verifyGroupGoingDownIsFine), since if we grant // suspension in this case they will permanently be down/removed. @@ -123,8 +122,7 @@ public class HostedVespaClusterPolicy implements ClusterPolicy { // I proxy host if (clusterApi.serviceType().equals(ServiceType.CLUSTER_CONTROLLER)) { - // All nodes have all state and we need to be able to remove the half that are retired on cluster migration - return ConcurrentSuspensionLimitForCluster.FIFTY_PERCENT; + return ConcurrentSuspensionLimitForCluster.ONE_NODE; } if (Set.of(ServiceType.STORAGE, ServiceType.SEARCH, ServiceType.DISTRIBUTOR, ServiceType.TRANSACTION_LOG_SERVER) @@ -164,8 +162,7 @@ public class HostedVespaClusterPolicy implements ClusterPolicy { } if (ServiceType.CLUSTER_CONTROLLER.equals(clusterApi.serviceType())) { - // All nodes have all state and we need to be able to remove the half that are retired on cluster migration - return ConcurrentSuspensionLimitForCluster.FIFTY_PERCENT; + return ConcurrentSuspensionLimitForCluster.ONE_NODE; } if (ServiceType.METRICS_PROXY.equals(clusterApi.serviceType())) { |