aboutsummaryrefslogtreecommitdiffstats
path: root/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2021-08-18 12:58:22 +0200
committerMartin Polden <mpolden@mpolden.no>2021-08-18 13:00:25 +0200
commit3039216b76eb3255f28d50ed3e4a8d62fb8c604f (patch)
treeb004ff57dd8160549bcca87abf63c67e75bd45f6 /controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java
parent1ec01c1db964c6a0c3553a6e3c1eb8cd9700413d (diff)
Remove redundant handling of missing serialized field
Diffstat (limited to 'controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java')
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OsUpgradeSchedulerTest.java31
1 files changed, 0 insertions, 31 deletions
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 770d0a898fe..6e5e58ddf76 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
@@ -7,14 +7,12 @@ import com.yahoo.config.provision.zone.ZoneApi;
import com.yahoo.vespa.hosted.controller.ControllerTester;
import com.yahoo.vespa.hosted.controller.api.integration.deployment.StableOsVersion;
import com.yahoo.vespa.hosted.controller.integration.ZoneApiMock;
-import com.yahoo.vespa.hosted.controller.versions.OsVersion;
import com.yahoo.vespa.hosted.controller.versions.OsVersionTarget;
import org.junit.Test;
import java.time.Duration;
import java.time.Instant;
import java.util.List;
-import java.util.Set;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
@@ -67,35 +65,6 @@ public class OsUpgradeSchedulerTest {
assertEquals(version1, tester.controller().osVersionTarget(cloud).get().osVersion().version());
}
- @Test // TODO(mpolden): Remove this after 2021-09-01
- public void schedule_calendar_versioned_without_scheduled_time() {
- ControllerTester tester = new ControllerTester();
- OsUpgradeScheduler scheduler = new OsUpgradeScheduler(tester.controller(), Duration.ofDays(1));
- Instant t0 = Instant.parse("2021-01-23T07:00:00.00Z"); // Inside trigger period
- tester.clock().setInstant(t0);
-
- CloudName cloud = CloudName.from("cloud");
- ZoneApi zone = zone("prod.us-west-1", cloud);
- tester.zoneRegistry().setZones(zone).reprovisionToUpgradeOsIn(zone);
-
- // Initial run does nothing as the cloud does not have a target
- scheduler.maintain();
- assertTrue("No target set", tester.controller().osVersionTarget(cloud).isEmpty());
-
- // Target is set
- Version version0 = Version.fromString("7.0.0.20210123190005");
- // Simulate setting target without scheduledAt, to force parsing scheduled time from version number
- tester.curator().writeOsVersionTargets(Set.of(new OsVersionTarget(new OsVersion(version0, cloud),
- Duration.ofDays(1), Instant.EPOCH)));
-
- // Just over 45 days pass, and a new target replaces the expired one
- Version version1 = Version.fromString("7.0.0.20210302");
- tester.clock().advance(Duration.ofDays(45).plus(Duration.ofSeconds(1)));
- scheduler.maintain();
- assertEquals("New target set", version1,
- tester.controller().osVersionTarget(cloud).get().osVersion().version());
- }
-
@Test
public void schedule_stable_release() {
ControllerTester tester = new ControllerTester();