From cb562d27c8cbe70ce743a3ed401eea7dbae298ba Mon Sep 17 00:00:00 2001 From: Jon Bratseth Date: Fri, 6 Jan 2023 14:19:14 +0100 Subject: Exclusive containers in public --- .../java/com/yahoo/vespa/hosted/provision/NodeRepository.java | 1 + .../yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) (limited to 'node-repository') diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java index ae1ea6c4d71..ea22c768e90 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java @@ -205,6 +205,7 @@ public class NodeRepository extends AbstractComponent { */ public boolean exclusiveAllocation(ClusterSpec clusterSpec) { return clusterSpec.isExclusive() || + ( clusterSpec.type().isContainer() && zone.system().isPublic() && !zone.environment().isTest() ) || ( !zone().cloud().allowHostSharing() && !sharedHosts.value().isEnabled(clusterSpec.type().name())); } diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java index 4ce07d53ea9..98037c68ac8 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java @@ -215,13 +215,13 @@ public class AutoscalingTest { fixture.loader().applyCpuLoad(0.25f, 120); ClusterResources scaledResources = fixture.tester().assertResources("Scaling cpu up", - 3, 1, 5, 13.3, 66.1, + 4, 1, 4, 16.0, 40.8, fixture.autoscale()); fixture.deploy(Capacity.from(scaledResources)); fixture.deactivateRetired(Capacity.from(scaledResources)); fixture.loader().applyCpuLoad(0.1f, 120); fixture.tester().assertResources("Scaling down since cpu usage has gone down", - 3, 1, 2.5, 9.2, 61.1, + 3, 1, 4, 16, 30.6, fixture.autoscale()); } @@ -733,7 +733,7 @@ public class AutoscalingTest { fixture.loader().applyLoad(new Load(0.1, 0.1, 0.1), 100); fixture.tester().assertResources("With non-exclusive nodes, a better solution is " + "50% more nodes with half the cpu", - 3, 1, 1, 4, 100.0, + 3, 1, 1.1, 4, 100.0, fixture.autoscale()); fixture.tester().deploy(fixture.applicationId(), clusterSpec(true), fixture.capacity()); @@ -744,7 +744,7 @@ public class AutoscalingTest { } private ClusterSpec clusterSpec(boolean exclusive) { - return ClusterSpec.request(ClusterSpec.Type.container, + return ClusterSpec.request(ClusterSpec.Type.content, ClusterSpec.Id.from("test")).vespaVersion("8.1.2") .exclusive(exclusive) .build(); -- cgit v1.2.3