diff options
author | Valerij Fredriksen <freva@users.noreply.github.com> | 2023-05-10 09:49:42 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-10 09:49:42 +0200 |
commit | 5deb75d0ab5e39a9b3ca2d032cddd74cb27f7c7d (patch) | |
tree | 36fe0e0c832ddbe1bb3dc4c56aa7a3c2149c1111 /node-repository | |
parent | 32cc36e3af0b5e24fdcb27ece4e5920042a8c483 (diff) |
Revert "Unset cloud account in GCP"
Diffstat (limited to 'node-repository')
2 files changed, 4 insertions, 11 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java index 01fc103423c..478d49d67d4 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java @@ -5,8 +5,6 @@ import com.yahoo.collections.ListMap; import com.yahoo.component.Version; import com.yahoo.config.provision.ApplicationId; import com.yahoo.config.provision.ApplicationTransaction; -import com.yahoo.config.provision.CloudAccount; -import com.yahoo.config.provision.CloudName; import com.yahoo.config.provision.ClusterSpec; import com.yahoo.config.provision.Flavor; import com.yahoo.config.provision.NodeType; @@ -83,10 +81,8 @@ public class Nodes { Instant start = clock.instant(); int nodesWritten = performOn(list(), (node, mutex) -> { // TODO (valerijf): Remove after 8.162 -// if (node.cloudAccount().isUnspecified() && !zone.getCloud().account().isUnspecified()) -// node = node.with(zone.getCloud().account()); - if (zone.getCloud().name().equals(CloudName.GCP)) - node = node.with(CloudAccount.empty); + if (node.cloudAccount().isUnspecified() && !zone.getCloud().account().isUnspecified()) + node = node.with(zone.getCloud().account()); return write(node, mutex); }).size(); Instant end = clock.instant(); diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisioner.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisioner.java index 21d7f1908ab..9e449ab8772 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisioner.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisioner.java @@ -5,7 +5,6 @@ import com.yahoo.config.provision.ApplicationId; import com.yahoo.config.provision.ApplicationName; import com.yahoo.config.provision.ApplicationTransaction; import com.yahoo.config.provision.CloudAccount; -import com.yahoo.config.provision.CloudName; import com.yahoo.config.provision.ClusterSpec; import com.yahoo.config.provision.HostName; import com.yahoo.config.provision.NodeType; @@ -76,10 +75,8 @@ public class LoadBalancerProvisioner { try (var lock = db.lock(id.application())) { var loadBalancer = db.readLoadBalancer(id); loadBalancer.ifPresent(lb -> { -// if (!zoneAccount.isUnspecified() && lb.instance().isPresent() && lb.instance().get().cloudAccount().isUnspecified()) -// lb = lb.with(Optional.of(lb.instance().get().with(zoneAccount))); - if (nodeRepository.zone().getCloud().name().equals(CloudName.GCP)) - lb = lb.with(lb.instance().map(i -> i.with(CloudAccount.empty))); + if (!zoneAccount.isUnspecified() && lb.instance().isPresent() && lb.instance().get().cloudAccount().isUnspecified()) + lb = lb.with(Optional.of(lb.instance().get().with(zoneAccount))); db.writeLoadBalancer(lb, lb.state()); }); } |