Fixed issue #207 by Updated Nova’s Label (key/value pair) and Replica…

…s for consistency in values.yaml.
att-comdev · Apr 6, 2017 · f020ef6 · f020ef6
1 parent e111e8b
commit f020ef6
Show file tree

Hide file tree

Showing 13 changed files with 49 additions and 24 deletions.
diff --git a/nova/templates/daemonset-compute.yaml b/nova/templates/daemonset-compute.yaml
@@ -31,7 +31,7 @@ spec:
         ]'
     spec:
       nodeSelector:
-        {{ .Values.labels.compute_node_selector_key }}: {{ .Values.labels.compute_node_selector_value }}
+        {{ .Values.labels.agent.compute.node_selector_key }}: {{ .Values.labels.agent.compute.node_selector_value }}
       securityContext:
         runAsUser: 0
       hostNetwork: true

diff --git a/nova/templates/daemonset-libvirt.yaml b/nova/templates/daemonset-libvirt.yaml
@@ -31,7 +31,7 @@ spec:
         ]'
     spec:
       nodeSelector:
-        {{ .Values.labels.compute_node_selector_key }}: {{ .Values.labels.compute_node_selector_value }}
+        {{ .Values.labels.agent.libvirt.node_selector_key }}: {{ .Values.labels.agent.libvirt.node_selector_value }}
       securityContext:
         runAsUser: 0
       hostNetwork: true

diff --git a/nova/templates/deployment-api-metadata.yaml b/nova/templates/deployment-api-metadata.yaml
@@ -19,7 +19,7 @@ kind: Deployment
 metadata:
   name: nova-api-metadata
 spec:
-  replicas: {{ .Values.control_replicas }}
+  replicas: {{ .Values.replicas.api_metadata }}
   revisionHistoryLimit: {{ .Values.upgrades.revision_history }}
   strategy:
     type: {{ .Values.upgrades.pod_replacement_strategy }}
@@ -40,7 +40,7 @@ spec:
         ]'
     spec:
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.api_metadata.node_selector_key }}: {{ .Values.labels.api_metadata.node_selector_value }}
       containers:
         - name: nova-api
           image: {{ .Values.images.api }}

diff --git a/nova/templates/deployment-api-osapi.yaml b/nova/templates/deployment-api-osapi.yaml
@@ -19,7 +19,7 @@ kind: Deployment
 metadata:
   name: nova-api-osapi
 spec:
-  replicas: {{ .Values.control_replicas }}
+  replicas: {{ .Values.replicas.osapi }}
   revisionHistoryLimit: {{ .Values.upgrades.revision_history }}
   strategy:
     type: {{ .Values.upgrades.pod_replacement_strategy }}
@@ -40,7 +40,7 @@ spec:
         ]'
     spec:
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.osapi.node_selector_key }}: {{ .Values.labels.osapi.node_selector_value }}
       containers:
         - name: nova-osapi
           image: {{ .Values.images.api }}

diff --git a/nova/templates/deployment-conductor.yaml b/nova/templates/deployment-conductor.yaml
@@ -19,7 +19,7 @@ kind: Deployment
 metadata:
   name: nova-conductor
 spec:
-  replicas: {{ .Values.control_replicas }}
+  replicas: {{ .Values.replicas.conductor }}
   revisionHistoryLimit: {{ .Values.upgrades.revision_history }}
   strategy:
     type: {{ .Values.upgrades.pod_replacement_strategy }}
@@ -40,7 +40,7 @@ spec:
         ]'
     spec:
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.conductor.node_selector_key }}: {{ .Values.labels.conductor.node_selector_value }}
       containers:
         - name: nova-conductor
           image: {{ .Values.images.conductor }}

diff --git a/nova/templates/deployment-consoleauth.yaml b/nova/templates/deployment-consoleauth.yaml
@@ -19,7 +19,7 @@ kind: Deployment
 metadata:
   name: nova-consoleauth
 spec:
-  replicas: {{ .Values.control_replicas }}
+  replicas: {{ .Values.replicas.consoleauth }}
   revisionHistoryLimit: {{ .Values.upgrades.revision_history }}
   strategy:
     type: {{ .Values.upgrades.pod_replacement_strategy }}
@@ -40,7 +40,7 @@ spec:
         ]'
     spec:
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.consoleauth.node_selector_key }}: {{ .Values.labels.consoleauth.node_selector_value }}
       containers:
         - name: nova-consoleauth
           image: {{ .Values.images.consoleauth }}

diff --git a/nova/templates/deployment-scheduler.yaml b/nova/templates/deployment-scheduler.yaml
@@ -19,7 +19,7 @@ kind: Deployment
 metadata:
   name: nova-scheduler
 spec:
-  replicas: {{ .Values.control_replicas }}
+  replicas: {{ .Values.replicas.scheduler }}
   revisionHistoryLimit: {{ .Values.upgrades.revision_history }}
   strategy:
     type: {{ .Values.upgrades.pod_replacement_strategy }}
@@ -40,7 +40,7 @@ spec:
         ]'
     spec:
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.scheduler.node_selector_key }}: {{ .Values.labels.scheduler.node_selector_value }}
       containers:
         - name: nova-scheduler
           image: {{ .Values.images.scheduler }}

diff --git a/nova/templates/job-db-init.yaml b/nova/templates/job-db-init.yaml
@@ -28,7 +28,7 @@ spec:
     spec:
       restartPolicy: OnFailure
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.job.node_selector_key }}: {{ .Values.labels.job.node_selector_value }}
       containers:
         - name: nova-db-init
           image: {{ .Values.images.db_init }}

diff --git a/nova/templates/job-db-sync.yaml b/nova/templates/job-db-sync.yaml
@@ -28,7 +28,7 @@ spec:
     spec:
       restartPolicy: OnFailure
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.job.node_selector_key }}: {{ .Values.labels.job.node_selector_value }}
       containers:
         - name: nova-db-sync
           image: {{ .Values.images.db_sync }}

diff --git a/nova/templates/job-ks-endpoints.yaml b/nova/templates/job-ks-endpoints.yaml
@@ -29,7 +29,7 @@ spec:
     spec:
       restartPolicy: OnFailure
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.job.node_selector_key }}: {{ .Values.labels.job.node_selector_value }}
       containers:
 {{- range $key1, $osServiceType := tuple "compute" }}
 {{- range $key2, $osServiceEndPoint := tuple "admin" "internal" "public" }}

diff --git a/nova/templates/job-ks-service.yaml b/nova/templates/job-ks-service.yaml
@@ -29,7 +29,7 @@ spec:
     spec:
       restartPolicy: OnFailure
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.job.node_selector_key }}: {{ .Values.labels.job.node_selector_value }}
       containers:
 {{- range $key1, $osServiceType := tuple "compute" }}
         - name: {{ $osServiceType }}-ks-service-registration

diff --git a/nova/templates/job-ks-user.yaml b/nova/templates/job-ks-user.yaml
@@ -30,7 +30,7 @@ spec:
     spec:
       restartPolicy: OnFailure
       nodeSelector:
-        {{ .Values.labels.control_node_selector_key }}: {{ .Values.labels.control_node_selector_value }}
+        {{ .Values.labels.job.node_selector_key }}: {{ .Values.labels.job.node_selector_value }}
       containers:
         - name: nova-ks-user
           image: {{ .Values.images.ks_user }}

diff --git a/nova/values.yaml b/nova/values.yaml
@@ -18,13 +18,38 @@
 # name: value
 
 labels:
-  control_node_selector_key: openstack-control-plane
-  control_node_selector_value: enabled
-  compute_node_selector_key: openstack-compute-node
-  compute_node_selector_value: enabled
-
-control_replicas: 1
-compute_replicas: 1
+  agent:
+    compute:
+      node_selector_key: openstack-compute-node
+      node_selector_value: enabled
+    libvirt:
+      node_selector_key: openstack-compute-node  
+      node_selector_value: enabled
+  conductor:
+    node_selector_key: openstack-control-plane
+    node_selector_value: enabled
+  consoleauth:
+    node_selector_key: openstack-control-plane
+    node_selector_value: enabled
+  scheduler:
+    node_selector_key: openstack-control-plane
+    node_selector_value: enabled
+  osapi:
+    node_selector_key: openstack-control-plane
+    node_selector_value: enabled
+  api_metadata:
+    node_selector_key: openstack-control-plane
+    node_selector_value: enabled
+  job:
+    node_selector_key: openstack-control-plane
+    node_selector_value: enabled
+
+replicas:
+  api_metadata: 1
+  osapi: 1
+  conductor: 1
+  consoleauth: 1
+  scheduler: 1
 
 images:
   db_init: quay.io/stackanetes/stackanetes-kolla-toolbox:newton