summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHÃ¥kon Hallingstad <hakon.hallingstad@gmail.com>2021-02-26 13:47:32 +0100
committerGitHub <noreply@github.com>2021-02-26 13:47:32 +0100
commiteafe3ab39c44628f3a584fbd5b895d5f9f5a1438 (patch)
treed6a0ee2655bf8d9438233b33b04ba42117e14763
parentfc352a2caebf17e4c300766054124a441a8be28a (diff)
Revert "Revert "Enable group suspension by default [run-systemtest]""
-rw-r--r--flags/src/main/java/com/yahoo/vespa/flags/Flags.java2
-rw-r--r--orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ClusterApiImplTest.java4
2 files changed, 3 insertions, 3 deletions
diff --git a/flags/src/main/java/com/yahoo/vespa/flags/Flags.java b/flags/src/main/java/com/yahoo/vespa/flags/Flags.java
index 1026af7f823..e6b7189455b 100644
--- a/flags/src/main/java/com/yahoo/vespa/flags/Flags.java
+++ b/flags/src/main/java/com/yahoo/vespa/flags/Flags.java
@@ -200,7 +200,7 @@ public class Flags {
APPLICATION_ID);
public static final UnboundBooleanFlag GROUP_SUSPENSION = defineFeatureFlag(
- "group-suspension", false,
+ "group-suspension", true,
List.of("hakon"), "2021-01-22", "2021-03-22",
"Allow all content nodes in a hierarchical group to suspend at the same time",
"Takes effect on the next suspension request to the Orchestrator.",
diff --git a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ClusterApiImplTest.java b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ClusterApiImplTest.java
index bc621bfc6aa..412a08a8305 100644
--- a/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ClusterApiImplTest.java
+++ b/orchestrator/src/test/java/com/yahoo/vespa/orchestrator/model/ClusterApiImplTest.java
@@ -114,8 +114,8 @@ public class ClusterApiImplTest {
} catch (HostStateChangeDeniedException e) {
assertThat(e.getMessage(),
containsString("Changing the state of cfg1 would violate enough-services-up: " +
- "Suspension of service with type 'configserver' would increase from 33% to 66%, " +
- "over the limit of 10%. Services down on resumed hosts: [1 missing config server]."));
+ "Suspension of service with type 'configserver' not allowed: 33% are suspended already. " +
+ "Services down on resumed hosts: [1 missing config server]."));
}
flagSource.withBooleanFlag(Flags.GROUP_SUSPENSION.id(), true);