openstack/openstack-ansible
Code Issues Proposed changes

Fix "configure container (other services)" when statement

Browse Source 
Configure container (other services) task should be executed when
cinder_volume uses other than lvm as backend (for example. rbd). But the
existing code completely skips both "Configure container
(cinder-volume)" and "Configure container (other services)" tasks. So
the cinder_volume container is not properly initialized when using rbd as
backend.

Change-Id: Ieb195e8a0cd3435fa3e872b609717aadc2ff41e8
Closes-Bug: #1780789
Closes-Bug: #1756005
This commit is contained in:
Chenjun Shen 2018-07-10 10:15:10 +02:00 committed by Jesse Pretorius (odyssey4me)
parent e859e19959
commit 92427a6531
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
playbooks/common-playbooks/cinder.yml
View File

@ -42,7 +42,7 @@
vars: vars:
network_address: "storage_address" network_address: "storage_address"
- name: Configure container (cinder-volume) - name: Configure container (cinder-volume) when lvm is in-use
include: "../common-tasks/os-{{ container_tech | default('lxc') }}-container-setup.yml" include: "../common-tasks/os-{{ container_tech | default('lxc') }}-container-setup.yml"
static: no static: no
vars: vars:
@ -62,7 +62,7 @@
static: no static: no
when: when:
- "not is_metal" - "not is_metal"
- "'cinder_volume' not in group_names" - "not ( 'cinder_volume' in group_names and cinder_backend_lvm_inuse | bool )"
- name: Configure log directories (on metal) - name: Configure log directories (on metal)
include: ../common-tasks/os-log-dir-setup.yml include: ../common-tasks/os-log-dir-setup.yml