diff options
author | Valerij Fredriksen <freva@users.noreply.github.com> | 2022-01-07 13:58:37 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-01-07 13:58:37 +0100 |
commit | 24a7414eaabd7624de8ac8a20865e7a2b5b593d2 (patch) | |
tree | 6b1839b19b906ae802a4dc374d521e4c05a04e21 /node-repository | |
parent | 748941bedbb29f143bdd7a79439bdf4f0d6c16b1 (diff) | |
parent | f3a02ac4bf9482572cc764ddeb6e23cf1247daa3 (diff) |
Merge pull request #20684 from vespa-engine/freva/node-id
Use UUID for Linux container nodes
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/Node.java | 5 | ||||
-rw-r--r-- | node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-container1.json | 2 |
2 files changed, 4 insertions, 3 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/Node.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/Node.java index 4d63863a917..3db68a27234 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/Node.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/Node.java @@ -27,6 +27,7 @@ import java.util.List; import java.util.Objects; import java.util.Optional; import java.util.Set; +import java.util.UUID; import java.util.stream.Collectors; /** @@ -63,7 +64,7 @@ public final class Node implements Nodelike { /** Creates a node builder in the initial state (reserved) */ public static Node.Builder reserve(Set<String> ipAddresses, String hostname, String parentHostname, NodeResources resources, NodeType type) { - return new Node.Builder("fake-" + hostname, hostname, new Flavor(resources), State.reserved, type) + return new Node.Builder(UUID.randomUUID().toString(), hostname, new Flavor(resources), State.reserved, type) .ipConfig(IP.Config.ofEmptyPool(ipAddresses)) .parentHostname(parentHostname); } @@ -140,7 +141,7 @@ public final class Node implements Nodelike { * * - OpenStack: UUID * - AWS: Instance ID - * - Linux containers: fake-[hostname] + * - Linux containers: UUID */ public String id() { return id; } diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-container1.json b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-container1.json index c9784c7e610..b5efc69d6db 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-container1.json +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/docker-container1.json @@ -1,6 +1,6 @@ { "url": "http://localhost:8080/nodes/v2/node/test-node-pool-102-2", - "id": "fake-test-node-pool-102-2", + "id": "(ignore)", "state": "active", "type": "tenant", "hostname": "test-node-pool-102-2", |