Browse Source

rename node_removal_policies, add some comments and defaults

Tzu-Mainn Chen 7 years ago
parent
commit
2fa7c11256

+ 2 - 2
playbooks/provisioning/openstack/README.md

@@ -67,8 +67,8 @@ The `openstack_num_masters`, `openstack_num_infra` and
 `openstack_num_nodes` values specify the number of Master, Infra and
 App nodes to create.
 
-The `openstack_node_removal_policies` allows you to specify which App nodes to
-remove.
+The `openstack_nodes_to_remove` allows you to specify the numerical indexes
+of App nodes that should be removed; for example, ['0', '2'],
 
 The `openstack_flat_secgrp`, controls Neutron security groups creation for Heat
 stacks. Set it to true, if you experience issues with sec group rules

+ 1 - 1
playbooks/provisioning/openstack/provision-openstack.yml

@@ -23,7 +23,7 @@
     num_nodes: "{{ openstack_num_nodes }}"
     num_infra: "{{ openstack_num_infra }}"
     num_dns: "{{ openstack_num_dns | default(1) }}"
-    node_removal_policies: "{{ openstack_node_removal_policies | to_yaml }}"
+    nodes_to_remove: "{{ openstack_nodes_to_remove | default([]) |  to_yaml }}"
     master_volume_size: "{{ docker_volume_size }}"
     app_volume_size: "{{ docker_volume_size }}"
     infra_volume_size: "{{ docker_volume_size }}"

+ 2 - 1
playbooks/provisioning/openstack/sample-inventory/group_vars/all.yml

@@ -12,7 +12,8 @@ openstack_num_masters: 1
 openstack_num_infra: 1
 openstack_num_nodes: 2
 
-openstack_node_removal_policies: []
+# # Numerical index of nodes to remove
+# openstack_nodes_to_remove: []
 
 docker_volume_size: "15"
 

+ 1 - 1
roles/openstack-stack/defaults/main.yml

@@ -9,5 +9,5 @@ num_masters: 1
 num_nodes: 1
 num_dns: 1
 num_infra: 1
-node_removal_policies: []
+nodes_to_remove: []
 etcd_volume_size: 2

+ 1 - 1
roles/openstack-stack/templates/heat_stack.yaml.j2

@@ -621,7 +621,7 @@ resources:
     properties:
       count: {{ num_nodes }}
       removal_policies:
-      - resource_list: {{ node_removal_policies }}
+      - resource_list: {{ nodes_to_remove }}
       resource_def:
         type: server.yaml
         properties: