diff options
author | jonmv <venstad@gmail.com> | 2023-06-02 16:23:35 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2023-06-02 16:23:35 +0200 |
commit | 3fa0461f0378b1d7566b8536f374f976e69bffa4 (patch) | |
tree | 717be3664ce484e8abb5dedecc641627cd1a2e7d /config-model-api | |
parent | a82b87143f8dcc1218a0ce5718fcf09c73c1485c (diff) |
Revert signature
Diffstat (limited to 'config-model-api')
3 files changed, 9 insertions, 9 deletions
diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentInstanceSpec.java b/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentInstanceSpec.java index 9ce1a48f226..a4be547fe70 100644 --- a/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentInstanceSpec.java +++ b/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentInstanceSpec.java @@ -277,9 +277,9 @@ public class DeploymentInstanceSpec extends DeploymentSpec.Steps { } /** Returns the host TTL to use for given environment and region, if any */ - public Optional<Duration> hostTTL(Environment environment, RegionName region) { + public Optional<Duration> hostTTL(Environment environment, Optional<RegionName> region) { return zones().stream() - .filter(zone -> zone.concerns(environment, Optional.of(region))) + .filter(zone -> zone.concerns(environment, region)) .findFirst() .flatMap(DeploymentSpec.DeclaredZone::hostTTL) .or(() -> hostTTL); diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentSpec.java b/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentSpec.java index 5e79ceb738f..f355a61fa8a 100644 --- a/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentSpec.java +++ b/config-model-api/src/main/java/com/yahoo/config/application/api/DeploymentSpec.java @@ -209,7 +209,7 @@ public class DeploymentSpec { * deploy to, e.g., test and staging zones, and want to avoid the delay of having to provision hosts. */ public Optional<Duration> hostTTL(InstanceName instance, Environment environment, RegionName region) { - return instance(instance).flatMap(spec -> spec.hostTTL(environment, region)) + return instance(instance).flatMap(spec -> spec.hostTTL(environment, Optional.of(region))) .or(this::hostTTL); } diff --git a/config-model-api/src/test/java/com/yahoo/config/application/api/DeploymentSpecWithoutInstanceTest.java b/config-model-api/src/test/java/com/yahoo/config/application/api/DeploymentSpecWithoutInstanceTest.java index 013c161ad9b..e5578723612 100644 --- a/config-model-api/src/test/java/com/yahoo/config/application/api/DeploymentSpecWithoutInstanceTest.java +++ b/config-model-api/src/test/java/com/yahoo/config/application/api/DeploymentSpecWithoutInstanceTest.java @@ -778,9 +778,9 @@ public class DeploymentSpecWithoutInstanceTest { DeploymentSpec spec = DeploymentSpec.fromXml(r); assertEquals(Optional.of(Duration.ofDays(1)), spec.hostTTL()); DeploymentInstanceSpec instance = spec.requireInstance("default"); - assertEquals(Optional.of(Duration.ofMinutes(1)), instance.hostTTL(prod, RegionName.from("us-east-1"))); - assertEquals(Optional.of(Duration.ofDays(1)), instance.hostTTL(prod, RegionName.from("us-west-1"))); - assertEquals(Optional.of(Duration.ofDays(1)), instance.hostTTL(test, RegionName.defaultName())); + assertEquals(Optional.of(Duration.ofMinutes(1)), instance.hostTTL(prod, Optional.of(RegionName.from("us-east-1")))); + assertEquals(Optional.of(Duration.ofDays(1)), instance.hostTTL(prod, Optional.of(RegionName.from("us-west-1")))); + assertEquals(Optional.of(Duration.ofDays(1)), instance.hostTTL(test, Optional.empty())); r = """ <deployment version='1.0' cloud-account='012345678912'> @@ -793,9 +793,9 @@ public class DeploymentSpecWithoutInstanceTest { spec = DeploymentSpec.fromXml(r); assertEquals(Optional.empty(), spec.hostTTL()); instance = spec.requireInstance("default"); - assertEquals(Optional.of(Duration.ofMinutes(1)), instance.hostTTL(prod, RegionName.from("us-east-1"))); - assertEquals(Optional.of(Duration.ofDays(1)), instance.hostTTL(prod, RegionName.from("us-west-1"))); - assertEquals(Optional.empty(), instance.hostTTL(test, RegionName.defaultName())); + assertEquals(Optional.of(Duration.ofMinutes(1)), instance.hostTTL(prod, Optional.of(RegionName.from("us-east-1")))); + assertEquals(Optional.of(Duration.ofDays(1)), instance.hostTTL(prod, Optional.of(RegionName.from("us-west-1")))); + assertEquals(Optional.empty(), instance.hostTTL(test, Optional.empty())); } private static Set<String> endpointRegions(String endpointId, DeploymentSpec spec) { |