From: Guillaume Abrioux <gabrioux@redhat.com>
Date: Tue, 10 Dec 2019 22:04:57 +0000 (+0100)
Subject: filestore-to-bluestore: ensure all dm are closed
X-Git-Tag: v6.0.0alpha1~165
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=8e6ef818a287e8bf139420142493843077ea3851;p=ceph-ansible.git

filestore-to-bluestore: ensure all dm are closed

This commit adds a task to ensure device mappers are well closed when
lvm batch scenario is used.
Otherwise, OSDs can't be redeployed given that devices that are rejected
by ceph-volume because they are locked.

Adding a condition `devices | default([]) | length > 0` to remove these
dm only when using lvm batch scenario.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
---

diff --git a/infrastructure-playbooks/filestore-to-bluestore.yml b/infrastructure-playbooks/filestore-to-bluestore.yml
index ebe9b17ba..00a0c52f4 100644
--- a/infrastructure-playbooks/filestore-to-bluestore.yml
+++ b/infrastructure-playbooks/filestore-to-bluestore.yml
@@ -175,6 +175,16 @@
             CEPH_CONTAINER_IMAGE: "{{ ceph_docker_registry + '/' + ceph_docker_image + ':' + ceph_docker_image_tag if containerized_deployment else None }}"
             CEPH_CONTAINER_BINARY: "{{ container_binary }}"
           loop: "{{ osd_fsid_list }}"
+
+        - name: ensure all dm are closed
+          command: dmsetup remove "{{ item['lv_path'] }}"
+          with_items: "{{ _lvm_list }}"
+          changed_when: false
+          failed_when: false
+          when:
+            - item['lv_path'] is defined
+              # Do not close mappers for non 'lvm batch' devices
+            - devices | default([]) | length > 0
       when: _lvm_list is defined
 
     - name: set_fact osd_ids