aboutsummaryrefslogtreecommitdiffstats
path: root/node-repository
diff options
context:
space:
mode:
authorValerij Fredriksen <freva@users.noreply.github.com>2023-05-10 09:49:42 +0200
committerGitHub <noreply@github.com>2023-05-10 09:49:42 +0200
commit5deb75d0ab5e39a9b3ca2d032cddd74cb27f7c7d (patch)
tree36fe0e0c832ddbe1bb3dc4c56aa7a3c2149c1111 /node-repository
parent32cc36e3af0b5e24fdcb27ece4e5920042a8c483 (diff)
Revert "Unset cloud account in GCP"
Diffstat (limited to 'node-repository')
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/node/Nodes.java8
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisioner.java7
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());
});
}