diff options
author | Harald Musum <musum@verizonmedia.com> | 2019-05-28 10:27:09 +0200 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2019-05-28 10:27:09 +0200 |
commit | a15dedfcb24b3825710d93e18b7da7001500b4f6 (patch) | |
tree | d91c86140bd2244341bef55776ad18c8b175818e /node-admin | |
parent | da58e8e064f7fcbac13836a617100dac2504d607 (diff) |
No cpu capping in cd
Diffstat (limited to 'node-admin')
-rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java index 8c762736b8a..71710c93742 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java @@ -4,6 +4,8 @@ package com.yahoo.vespa.hosted.node.admin.nodeagent; import com.fasterxml.jackson.core.JsonProcessingException; import com.yahoo.config.provision.DockerImage; import com.yahoo.config.provision.Environment; +import com.yahoo.config.provision.SystemName; +import com.yahoo.config.provision.zone.ZoneId; import com.yahoo.log.LogLevel; import com.yahoo.vespa.flags.DoubleFlag; import com.yahoo.vespa.flags.FetchVector; @@ -361,7 +363,7 @@ public class NodeAgentImpl implements NodeAgent { } private ContainerResources getContainerResources(NodeAgentContext context) { - double cpuCap = context.zoneId().environment() == Environment.dev ? + double cpuCap = noCpuCap(context.zoneId()) ? 0 : context.node().getOwner() .map(NodeOwner::asApplicationId) @@ -372,6 +374,11 @@ public class NodeAgentImpl implements NodeAgent { return ContainerResources.from(cpuCap, context.node().getMinCpuCores(), context.node().getMinMainMemoryAvailableGb()); } + private boolean noCpuCap(ZoneId zoneId) { + return zoneId.environment() == Environment.dev + // TODO: Add other cd systems, not ideal when having just SystemName here + || zoneId.system() == SystemName.cd; + } private void scheduleDownLoadIfNeeded(NodeSpec node, Optional<Container> container) { if (node.getWantedDockerImage().equals(container.map(c -> c.image))) return; |