From 19068659c79e56ddbca0a19c6cf631e8271feac8 Mon Sep 17 00:00:00 2001 From: Guillaume Abrioux Date: Tue, 10 Dec 2019 23:04:57 +0100 Subject: [PATCH] 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 (cherry picked from commit 8e6ef818a287e8bf139420142493843077ea3851) --- infrastructure-playbooks/filestore-to-bluestore.yml | 10 ++++++++++ 1 file changed, 10 insertions(+) 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