diff options
author | Martin Polden <mpolden@mpolden.no> | 2022-04-12 10:00:45 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2022-04-12 10:34:21 +0200 |
commit | 589e3662f610811e0e54992c6cc0e30d05c8630b (patch) | |
tree | 1cb8d92fbbf1764352b061492db1384e98c7146a /node-repository/src/test/java/com/yahoo | |
parent | 61a9585f06313e42fb2b47fb02402a2924b6152d (diff) |
Only record osUpgraded event if current version was non-empty
Diffstat (limited to 'node-repository/src/test/java/com/yahoo')
3 files changed, 10 insertions, 11 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java index b8f6011d6bf..84270a54218 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRebooterTest.java @@ -16,6 +16,7 @@ import org.junit.Test; import java.time.Duration; import java.time.Instant; import java.util.List; +import java.util.Optional; import java.util.stream.Collectors; import static org.junit.Assert.assertEquals; @@ -60,7 +61,8 @@ public class NodeRebooterTest { // OS upgrade counts as reboot, so within 0x-1x there is no reboots tester.clock().advance(rebootInterval); rebooter.maintain(); - simulateReboot(nodeRepository); + scheduleOsUpgrade(nodeRepository); + simulateOsUpgrade(nodeRepository); assertReadyHosts(15, nodeRepository, 1L); // OS upgrade counts as reboot, but within 1x-2x reboots are scheduled again @@ -109,6 +111,8 @@ public class NodeRebooterTest { private void makeReadyHosts(int count, ProvisioningTester tester) { tester.makeReadyNodes(count, new NodeResources(64, 256, 1000, 10), NodeType.host, 10); + // Set initial OS version + tester.patchNodes(node -> node.type().isHost(), (node) -> node.with(node.status().withOsVersion(node.status().osVersion().withCurrent(Optional.of(Version.fromString("7.0")))))); } /** Set current reboot generation to the wanted reboot generation whenever it is larger (i.e record a reboot) */ @@ -122,7 +126,7 @@ public class NodeRebooterTest { /** Schedule OS upgrade for all host nodes */ private void scheduleOsUpgrade(NodeRepository nodeRepository) { - nodeRepository.osVersions().setTarget(NodeType.host, Version.fromString("7.0"), Duration.ZERO, false); + nodeRepository.osVersions().setTarget(NodeType.host, Version.fromString("7.1"), Duration.ZERO, false); } /** Simulate completion of an OS upgrade */ diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java index 44c02e49d13..af3b5dc69f2 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java @@ -351,11 +351,11 @@ public class NodeSerializerTest { assertFalse(serialized.status().osVersion().current().isPresent()); // Update OS version - serialized = serialized.withCurrentOsVersion(Version.fromString("7.1"), Instant.ofEpochMilli(123)) - // Another update for same version: - .withCurrentOsVersion(Version.fromString("7.1"), Instant.ofEpochMilli(456)); + serialized = serialized.withCurrentOsVersion(Version.fromString("7.1"), Instant.ofEpochMilli(42)) + .withCurrentOsVersion(Version.fromString("7.2"), Instant.ofEpochMilli(123)) + .withCurrentOsVersion(Version.fromString("7.2"), Instant.ofEpochMilli(456)); serialized = nodeSerializer.fromJson(State.provisioned, nodeSerializer.toJson(serialized)); - assertEquals(Version.fromString("7.1"), serialized.status().osVersion().current().get()); + assertEquals(Version.fromString("7.2"), serialized.status().osVersion().current().get()); var osUpgradedEvents = serialized.history().asList().stream() .filter(event -> event.type() == History.Event.Type.osUpgraded) .collect(Collectors.toList()); diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-node1-os-upgrade-complete.json b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-node1-os-upgrade-complete.json index 09b5575ea02..cb874896927 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-node1-os-upgrade-complete.json +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-node1-os-upgrade-complete.json @@ -55,11 +55,6 @@ "event": "activated", "at": 123, "agent": "application" - }, - { - "event": "osUpgraded", - "at": 123, - "agent": "system" } ], "ipAddresses": [ |