diff options
author | gjoranv <gv@yahoo-inc.com> | 2017-02-18 04:02:06 +0100 |
---|---|---|
committer | gjoranv <gv@yahoo-inc.com> | 2017-02-19 01:39:43 +0100 |
commit | 805900f58e9428a0502f63e93181db009bf0bd07 (patch) | |
tree | 8c2f1702fd57ce966e86713791729bbae6cda2e3 /config-model/src | |
parent | 214266e06463cf9c23707facab513c027e9f550a (diff) |
Set flavor when a host is added
.. and also for an existing host if a given spec has a flavor,
and the host does not have a flavor yet.
Diffstat (limited to 'config-model/src')
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java b/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java index c2aeaa9d733..d653bae9d53 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java @@ -136,6 +136,7 @@ public class HostSystem extends AbstractConfigProducer<Host> { private HostResource addNewHost(HostSpec hostSpec) { Host host = new Host(this, hostSpec.hostname()); HostResource hostResource = new HostResource(host); + hostResource.setFlavor(hostSpec.flavor()); hostname2host.put(host.getHostName(), hostResource); Set<ClusterMembership> hostMemberships = new LinkedHashSet<>(); if (hostSpec.membership().isPresent()) @@ -159,7 +160,8 @@ public class HostSystem extends AbstractConfigProducer<Host> { // This is needed for single node host provisioner to work in unit tests for hosted vespa applications. HostResource hostResource = getExistingHost(host).orElseGet(() -> addNewHost(host)); retAllocatedHosts.put(hostResource, host.membership().orElse(null)); - hostResource.setFlavor(host.flavor()); + if (! hostResource.getFlavor().isPresent()) + hostResource.setFlavor(host.flavor()); } return retAllocatedHosts; } |