From 141299e3000bd690583e7601c2c119ce92c68638 Mon Sep 17 00:00:00 2001 From: Martin Polden Date: Tue, 27 Jun 2023 13:50:07 +0200 Subject: Allow patching wantToUpgradeFlavor --- .../main/java/com/yahoo/vespa/hosted/provision/restapi/NodePatcher.java | 1 + .../java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java | 2 +- .../vespa/hosted/provision/restapi/responses/node4-after-changes.json | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) (limited to 'node-repository') diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodePatcher.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodePatcher.java index 407961dc054..32bba9336a2 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodePatcher.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodePatcher.java @@ -223,6 +223,7 @@ public class NodePatcher { case WANT_TO_RETIRE: case WANT_TO_DEPROVISION: case WANT_TO_REBUILD: + case WANT_TO_UPGRADE_FLAVOR: // These needs to be handled as one, because certain combinations are not allowed. return node.withWantToRetire(asOptionalBoolean(root.field(WANT_TO_RETIRE)) .orElseGet(node.status()::wantToRetire), diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java index 0ef80cbe6f5..53b1a3bcb89 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java @@ -196,7 +196,7 @@ public class NodesV2ApiTest { Utf8.toBytes("{\"ipAddresses\": [\"127.0.0.1\",\"::1\"]}"), Request.Method.PATCH), "{\"message\":\"Updated host4.yahoo.com\"}"); assertResponse(new Request("http://localhost:8080/nodes/v2/node/host4.yahoo.com", - Utf8.toBytes("{\"wantToRetire\": true}"), Request.Method.PATCH), + Utf8.toBytes("{\"wantToRetire\": true, \"wantToUpgradeFlavor\": true}"), Request.Method.PATCH), "{\"message\":\"Updated host4.yahoo.com\"}"); assertResponse(new Request("http://localhost:8080/nodes/v2/node/host4.yahoo.com", Utf8.toBytes("{\"currentVespaVersion\": \"6.43.0\",\"currentDockerImage\": \"docker-registry.domain.tld:8080/dist/vespa:6.45.0\"}"), Request.Method.PATCH), diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/node4-after-changes.json b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/node4-after-changes.json index 8adb6eeb20c..a3589e760ad 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/node4-after-changes.json +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/node4-after-changes.json @@ -62,7 +62,7 @@ "preferToRetire": false, "wantToDeprovision": false, "wantToRebuild": false, - "wantToUpgradeFlavor": false, + "wantToUpgradeFlavor": true, "down": false, "history": [ { -- cgit v1.2.3