aboutsummaryrefslogtreecommitdiffstats
path: root/node-repository
diff options
context:
space:
mode:
authorValerij Fredriksen <valerijf@verizonmedia.com>2019-09-17 17:24:59 +0200
committerValerij Fredriksen <valerijf@verizonmedia.com>2019-09-17 17:35:22 +0200
commit5c88e152212abf2d2f641f6cc2868154c68c6271 (patch)
tree463ab5145edf4a26243cc4a92d04bfb82a3c1580 /node-repository
parent32cdfcf55580093233e1ebda304072e0bbdc7c39 (diff)
Flip default flag value
Diffstat (limited to 'node-repository')
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java6
-rw-r--r--node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTester.java1
2 files changed, 2 insertions, 5 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java
index ffbf311fc0f..5bc3703c11c 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java
@@ -6,7 +6,6 @@ import com.yahoo.config.provision.ClusterSpec;
import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.NodeResources;
import com.yahoo.config.provision.Zone;
-import com.yahoo.vespa.flags.BooleanFlag;
import com.yahoo.vespa.flags.FetchVector;
import com.yahoo.vespa.flags.FlagSource;
import com.yahoo.vespa.flags.Flags;
@@ -24,12 +23,10 @@ import java.util.Optional;
public class CapacityPolicies {
private final Zone zone;
- private final BooleanFlag useAdvertisedResourcesFlag;
private final JacksonFlag<com.yahoo.vespa.flags.custom.NodeResources> defaultResourcesFlag;
public CapacityPolicies(Zone zone, FlagSource flagSource) {
this.zone = zone;
- this.useAdvertisedResourcesFlag = Flags.USE_ADVERTISED_RESOURCES.bindTo(flagSource);
this.defaultResourcesFlag = Flags.DEFAULT_RESOURCES.bindTo(flagSource);
}
@@ -58,7 +55,8 @@ public class CapacityPolicies {
resources = resources.withDiskSpeed(NodeResources.DiskSpeed.any);
// Dev does not cap the cpu of containers since usage is spotty: Allocate just a small amount exclusively
- if (zone.environment() == Environment.dev && !useAdvertisedResourcesFlag.value())
+ // Do not cap in AWS as hosts are allocated on demand and 1-to-1, so the node can use the entire host
+ if (zone.environment() == Environment.dev && !zone.region().value().contains("aws-"))
resources = resources.withVcpu(0.1);
return resources;
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTester.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTester.java
index 203f922d132..dee32513457 100644
--- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTester.java
+++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTester.java
@@ -75,7 +75,6 @@ public class ProvisioningTester {
private final CapacityPolicies capacityPolicies;
private final ProvisionLogger provisionLogger;
private final LoadBalancerServiceMock loadBalancerService;
- private final InMemoryFlagSource flagSource = new InMemoryFlagSource();
private int nextHost = 0;
private int nextIP = 0;