switch_to_containers: fix umount ceph partitions

When a container is already running on a non containerized node then the
umount ceph partition task is skipped.
This is due to the container ps command which always returns 0 even if
the filter matches nothing.

We should run the umount task when:
1/ the container command is failing (not installed) : rc != 0
2/ the container command reports running ceph-osd containers : rc == 0

Also we should not fail on the ceph directory listing.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1616159

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
(cherry picked from commit 39cfe0aa65)
pull/4816/head v3.2.37
Dimitri Savineau 2019-11-27 11:27:09 -05:00 committed by Guillaume Abrioux
parent cbfa01f697
commit b08ac9cd44
1 changed files with 5 additions and 5 deletions

View File

@ -260,7 +260,7 @@
- name: check if containerized osds are already running
command: >
docker ps --filter='name=ceph-osd'
docker ps -q --filter='name=ceph-osd'
changed_when: false
failed_when: false
register: osd_running
@ -270,16 +270,16 @@
find /var/lib/ceph/osd {% if dmcrypt | bool %}/var/lib/ceph/osd-lockbox{% endif %} -maxdepth 1 -mindepth 1 -type d
register: osd_dirs
changed_when: false
failed_when: false
- name: unmount all the osd directories
command: >
umount {{ item }}
changed_when: false
failed_when: false
with_items:
- "{{ osd_dirs.stdout_lines }}"
when:
- osd_running.rc != 0
with_items: "{{ osd_dirs.stdout_lines }}"
when: osd_running.rc != 0 or osd_running.stdout_lines | length == 0
roles:
- ceph-defaults