diff options
author | Martin Polden <mpolden@mpolden.no> | 2021-03-18 13:14:54 +0100 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2021-03-18 13:14:54 +0100 |
commit | bc9cd68f49b3d51c238b68a0115442dc64f1e7b9 (patch) | |
tree | b5bd8f9e79d13d421d989e21526a93868a9ea39d /node-repository | |
parent | 4d0b336ec82a27c489957427273a0959c88edafd (diff) |
Non-tenant nodes are always within real limits
Diffstat (limited to 'node-repository')
2 files changed, 2 insertions, 1 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java index 5f4af786c77..e014b50a25f 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java @@ -123,7 +123,7 @@ class NodeAllocation { } } else if (! saturated() && hasCompatibleFlavor(candidate)) { - if (requestedNodes.type() == NodeType.tenant && ! nodeResourceLimits.isWithinRealLimits(candidate, cluster)) { + if (! nodeResourceLimits.isWithinRealLimits(candidate, cluster)) { ++rejectedDueToInsufficientRealResources; continue; } diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeResourceLimits.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeResourceLimits.java index ad03cafb4e3..4e74104c25b 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeResourceLimits.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeResourceLimits.java @@ -39,6 +39,7 @@ public class NodeResourceLimits { /** Returns whether the real resources we'll end up with on a given tenant node are within limits */ public boolean isWithinRealLimits(NodeCandidate candidateNode, ClusterSpec cluster) { + if (candidateNode.type() != NodeType.tenant) return true; // Resource limits only apply to tenant nodes return isWithinRealLimits(nodeRepository.resourcesCalculator().realResourcesOf(candidateNode, nodeRepository, cluster.isExclusive()), cluster.type()); } |