From 0ec618ec8875ecc523c21842ef56a71d9f5e9347 Mon Sep 17 00:00:00 2001
From: Alex Schultz <aschultz@redhat.com>
Date: Fri, 6 Oct 2017 15:04:35 -0600
Subject: Fix standalone ControllerOpenstack vars

As we've moved to more dynamic generation of variables, the correct
variable names are *ControllerOpenstack* not *Controller* for the
example standalone environment.

Change-Id: Iaa39de9d8794a856e76cc9995d046484632cf604
Closes-Bug: #1721877
(cherry picked from commit 536d1c4af59dc22164666be5cb1826115fdfdeb9)
---
 environments/composable-roles/standalone.yaml | 12 ++++++------
 sample-env-generator/composable-roles.yaml    | 25 ++++++++++++++++++++-----
 2 files changed, 26 insertions(+), 11 deletions(-)

diff --git a/environments/composable-roles/standalone.yaml b/environments/composable-roles/standalone.yaml
index 3305c9ed..c12d72de 100644
--- a/environments/composable-roles/standalone.yaml
+++ b/environments/composable-roles/standalone.yaml
@@ -30,13 +30,13 @@ parameter_defaults:
   # Type: string
   ComputeHostnameFormat: '%stackname%-novacompute-%index%'
 
-  # Number of Controller nodes to deploy
+  # Number of ControllerOpenstack nodes
   # Type: number
-  ControllerCount: 3
+  ControllerOpenstackCount: 3
 
-  # Format for Controller node hostnames Note %index% is translated into the index of the node, e.g 0/1/2 etc and %stackname% is replaced with the stack name e.g overcloud
+  # Format for ControllerOpenstack node hostnames Note %index% is translated into the index of the node, e.g 0/1/2 etc and %stackname% is replaced with the stack name e.g overcloud
   # Type: string
-  ControllerHostnameFormat: '%stackname%-controller-%index%'
+  ControllerOpenstackHostnameFormat: '%stackname%-controller-%index%'
 
   # Number of Database nodes
   # Type: number
@@ -66,9 +66,9 @@ parameter_defaults:
   # Type: string
   OvercloudComputeFlavor: compute
 
-  # Name of the flavor for Controller nodes
+  # Name of the flavor for ControllerOpenstack nodes
   # Type: string
-  OvercloudControllerFlavor: control
+  OvercloudControllerOpenstackFlavor: control
 
   # Name of the flavor for Database nodes
   # Type: string
diff --git a/sample-env-generator/composable-roles.yaml b/sample-env-generator/composable-roles.yaml
index 91d6060f..2c929a4b 100644
--- a/sample-env-generator/composable-roles.yaml
+++ b/sample-env-generator/composable-roles.yaml
@@ -89,10 +89,8 @@ environments:
     files:
       overcloud.yaml:
         parameters:
-          - ControllerHostnameFormat
           - ComputeHostnameFormat
           - CephStorageHostnameFormat
-          - ControllerCount
           - ComputeCount
           - CephStorageCount
       puppet/services/time/ntp.yaml:
@@ -100,19 +98,21 @@ environments:
           - NtpServer
       sample-env-generator/composable-roles.yaml:
         parameters:
+          - ControllerOpenstackHostnameFormat
           - DnsServers
+          - ControllerOpenstackCount
           - DatabaseCount
           - MessagingCount
           - NetworkerCount
-          - OvercloudControllerFlavor
+          - OvercloudControllerOpenstackFlavor
           - OvercloudComputeFlavor
           - OvercloudCephStorageFlavor
           - OvercloudDatabaseFlavor
           - OvercloudMessagingFlavor
           - OvercloudNetworkerFlavor
     sample_values:
-      ControllerCount: 3
-      OvercloudControllerFlavor: control
+      ControllerOpenstackCount: 3
+      OvercloudControllerOpenstackFlavor: control
       ComputeCount: 1
       OvercloudComputeFlavor: compute
       CephStorageCount: 1
@@ -135,6 +135,10 @@ parameters:
     description: DNS servers to use for the Overcloud
     type: comma_delimited_list
   # Dynamic vars based on roles
+  ControllerOpenstackCount:
+    default: 0
+    description: Number of ControllerOpenstack nodes
+    type: number
   DatabaseCount:
     default: 0
     description: Number of Database nodes
@@ -147,10 +151,21 @@ parameters:
     default: 0
     description: Number of Networker nodes
     type: number
+  ControllerOpenstackHostnameFormat:
+    type: string
+    description: >
+      Format for ControllerOpenstack node hostnames
+      Note %index% is translated into the index of the node, e.g 0/1/2 etc
+      and %stackname% is replaced with the stack name e.g overcloud
+    default: "%stackname%-controller-%index%"
   OvercloudControllerFlavor:
     default: control
     description: Name of the flavor for Controller nodes
     type: string
+  OvercloudControllerOpenstackFlavor:
+    default: control
+    description: Name of the flavor for ControllerOpenstack nodes
+    type: string
   OvercloudComputeFlavor:
     default: compute
     description: Name of the flavor for Compute nodes
-- 
cgit 1.2.3-korg