diff options
author | Martin Polden <mpolden@mpolden.no> | 2022-06-30 14:13:21 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2022-06-30 14:13:21 +0200 |
commit | 43782c8cdc64bd83f9a7380a484412e0f0833c1c (patch) | |
tree | f5c898367ce88ae5b68f608c7ebdbd938f4a7d14 /config-model/src/main/java/com | |
parent | 63d415951811b71dc94dec840673160f76e1a0a0 (diff) |
Deprecate combined cluster
Diffstat (limited to 'config-model/src/main/java/com')
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java index f971aa97e6a..2b9961232ea 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java @@ -732,9 +732,14 @@ public class ContainerModelBuilder extends ConfigModelBuilder<ContainerModel> { private List<ApplicationContainer> createNodes(ApplicationContainerCluster cluster, Element containerElement, Element nodesElement, ConfigModelContext context) { if (nodesElement.hasAttribute("type")) // internal use for hosted system infrastructure nodes return createNodesFromNodeType(cluster, nodesElement, context); - else if (nodesElement.hasAttribute("of")) // hosted node spec referencing a content cluster - return createNodesFromContentServiceReference(cluster, nodesElement, context); - else if (nodesElement.hasAttribute("count")) // regular, hosted node spec + else if (nodesElement.hasAttribute("of")) {// hosted node spec referencing a content cluster + // TODO: Remove support for combined clusters in Vespa 9 + List<ApplicationContainer> containers = createNodesFromContentServiceReference(cluster, nodesElement, context); + log.log(WARNING, "Declaring combined cluster with <nodes of=\"...\"> is deprecated without " + + "replacement, and the feature will be removed in Vespa 9. Use separate container and " + + "content clusters instead"); + return containers; + } else if (nodesElement.hasAttribute("count")) // regular, hosted node spec return createNodesFromNodeCount(cluster, containerElement, nodesElement, context); else if (cluster.isHostedVespa() && cluster.getZone().environment().isManuallyDeployed()) // default to 1 in manual zones return createNodesFromNodeCount(cluster, containerElement, nodesElement, context); |