Allow cinder coordination backend to be configured

This is to allow operator to prevent enabling redis and/or
etcd from magically configuring cinder coordinator.

Note this change is backwards-compatible.

Change-Id: Ie10be55968e43e3b9cc347b1b58771c1f7b1b910
Related-Bug: #1840070
Signed-off-by: Radosław Piliszek <radoslaw.piliszek@gmail.com>
(cherry picked from commit 03b4c706fa)

ansible/group_vars/all.yml

@@ -815,6 +815,9 @@ cinder_volume_group: "cinder-volumes"
 cinder_iscsi_helper: "tgtadm"
 cinder_target_helper: "{{ cinder_iscsi_helper }}"
 
+# Valid options are [ '', redis, etcd ]
+cinder_coordination_backend: "{{ 'redis' if enable_redis|bool else 'etcd' if enable_etcd|bool else '' }}"
+
 # Valid options are [ nfs, swift, ceph ]
 cinder_backup_driver: "ceph"
 cinder_backup_share: ""

ansible/roles/cinder/templates/cinder.conf.j2

@@ -220,9 +220,9 @@ auth_endpoint = {{ keystone_internal_url }}
 {% endif %}
 
 [coordination]
-{% if enable_redis | bool %}
+{% if cinder_coordination_backend == 'redis' %}
 backend_url = {{ redis_connection_string }}
-{% elif enable_etcd | bool %}
+{% elif cinder_coordination_backend == 'etcd' %}
 # NOTE(jeffrey4l): python-etcd3 module do not support multi endpoint here.
 backend_url = etcd3://{{ hostvars[groups['etcd'][0]]['ansible_' + hostvars[groups['etcd'][0]]['api_interface']]['ipv4']['address'] }}:{{ etcd_client_port }}
 {% endif %}

etc/kolla/globals.yml

@@ -417,6 +417,8 @@ glance_enable_rolling_upgrade: "no"
 #cinder_backend_ceph: "{{ enable_ceph }}"
 #cinder_backend_vmwarevc_vmdk: "no"
 #cinder_volume_group: "cinder-volumes"
+# Valid options are [ '', redis, etcd ]
+#cinder_coordination_backend: "{{ 'redis' if enable_redis|bool else 'etcd' if enable_etcd|bool else '' }}"
 
 # Valid options are [ nfs, swift, ceph ]
 #cinder_backup_driver: "ceph"

releasenotes/notes/new-variable-cinder-coordination-backend-3426a4e1bd22cbbb.yaml

@@ -0,0 +1,6 @@
+---
+features:
+  - |
+    Cinder coordination backend can now be configured via
+    cinder_coordination_backend variable.
+    Coordination is optional and can now be set to either redis or etcd.