summaryrefslogtreecommitdiffstats
path: root/node-repository
diff options
context:
space:
mode:
authorjonmv <venstad@gmail.com>2023-11-07 13:29:16 +0100
committerjonmv <venstad@gmail.com>2023-11-07 13:29:16 +0100
commit9ef68c02ed778be5cb1c6301f7bfe3e17c1a5298 (patch)
treea70dd901c468d8306dd952217b7854f7a5a39a20 /node-repository
parent2fc631ac7e05e728b0bca098a4bbc2cec78c2522 (diff)
Can allocate child to reserved parent as well
Diffstat (limited to 'node-repository')
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java4
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java2
2 files changed, 3 insertions, 3 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java
index 9a4e47ed154..192d90e1da1 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java
@@ -783,9 +783,9 @@ public class Nodes {
if (host.allocation().map(alloc -> alloc.membership().retired()).orElse(false)) return false;
if (dynamicProvisioning)
- return EnumSet.of(Node.State.active, Node.State.ready, Node.State.provisioned).contains(host.state());
+ return EnumSet.of(State.provisioned, State.ready, State.reserved, State.active).contains(host.state());
else
- return host.state() == Node.State.active;
+ return host.state() == State.active;
}
public boolean suspended(Node node) {
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java
index b92d6fb6d18..7e7c78f82f4 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java
@@ -211,7 +211,7 @@ public class NodePrioritizer {
/**
* We may regret that a non-active node is allocated to a host and not offer it to the application
- * now, e.g if we want to retire the host.
+ * now, e.g., if we want to retire the host.
*
* @return true if we still want to allocate the given node to its parent
*/