aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2021-07-09 09:20:35 +0200
committerMartin Polden <mpolden@mpolden.no>2021-07-09 09:20:35 +0200
commit6c72070be4a3a834f479bb2463f3e26be15c4939 (patch)
treefe7cda949e22ab59a3189d7279e99875c3792fc4
parent7266eb91ecc273f821adf146ea0677bc539cdb15 (diff)
Reduce scheduled OS upgrade cooldown
The time it takes before a version is promoted to stable is already plenty conservative.
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeScheduler.java2
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java4
2 files changed, 3 insertions, 3 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeScheduler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeScheduler.java
index c2addbe9a67..d3b05922d26 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeScheduler.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeScheduler.java
@@ -111,7 +111,7 @@ public class OsUpgradeScheduler extends ControllerMaintainer {
/** The cool-down period that must pass before a stable version can be used */
private Duration cooldown() {
- return system.isCd() ? Duration.ZERO : Duration.ofDays(14);
+ return system.isCd() ? Duration.ZERO : Duration.ofDays(7);
}
}
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java
index d80a8ce1152..770d0a898fe 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java
@@ -116,7 +116,7 @@ public class OsUpgradeSchedulerTest {
tester.controller().osVersionTarget(cloud).get().osVersion().version());
// Enough time passes since promotion of stable release
- tester.clock().advance(Duration.ofDays(14).plus(Duration.ofSeconds(1)));
+ tester.clock().advance(Duration.ofDays(7).plus(Duration.ofSeconds(1)));
scheduler.maintain();
OsVersionTarget target0 = tester.controller().osVersionTarget(cloud).get();
assertEquals(version1, target0.osVersion().version());
@@ -139,7 +139,7 @@ public class OsUpgradeSchedulerTest {
// Enough time passes for stable version to be promoted. Nothing happens as stable is now before the manually
// triggered version
- tester.clock().advance(Duration.ofDays(14).plus(Duration.ofSeconds(1)));
+ tester.clock().advance(Duration.ofDays(7).plus(Duration.ofSeconds(1)));
scheduler.maintain();
assertEquals(version3, tester.controller().osVersionTarget(cloud).get().osVersion().version());
}