osd: fix bug in osd activation (container)

8a17433 introduced a regression for containerized deployments. When deploying with `osd_auto_discovery: false` the osds never got activated. This commit fixes it. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1947072 Signed-off-by: Guillaume Abrioux <[email protected]>
ceph · Apr 9, 2021 · 0b6560a · 0b6560a
1 parent b847398
commit 0b6560a
Showing 1 changed file with 24 additions and 27 deletions.
diff --git a/roles/ceph-osd/tasks/start_osds.yml b/roles/ceph-osd/tasks/start_osds.yml
@@ -31,33 +31,30 @@
           ls /var/lib/ceph/osd/ | sed 's/.*-//'
         register: activated_osds
 
-      - name: auto discovery specific tasks
-        when: osd_auto_discovery | bool
-        block:
-          - name: get prepared disk list
-            command: >
-              docker run --rm --net=host --privileged=true --pid=host --ipc=host --ulimit nofile=1024:4096 --cpu-quota=100000
-              -v /dev:/dev
-              -v /etc/localtime:/etc/localtime:ro
-              -v /var/lib/ceph:/var/lib/ceph:z,rshared
-              -v /etc/ceph:/etc/ceph:z
-              -v /var/run/ceph:/var/run/ceph:z
-              -v /var/run/udev/:/var/run/udev/
-              --entrypoint=ceph-disk
-              {{ ceph_docker_registry }}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}
-              list --format json
-            register: ceph_disk_list
-
-          - name: set_fact prepared_devices
-            set_fact:
-              prepared_devices: "{{ prepared_devices | default([]) | union([item.path]) }}"
-            with_items: "{{ ceph_disk_list.stdout | default('{}') | from_json }}"
-            when:
-              - ceph_disk_list.skipped is undefined
-              - item.path is defined
-              - item.partitions is defined
-              - (item.partitions | selectattr('type', 'match', 'data') | selectattr('state', 'defined') | map(attribute='state') | default([]) | list == ['prepared']) or
-                (item.partitions | selectattr('type', 'match', 'lockbox') | selectattr('state', 'defined') | map(attribute='state') | list == ['prepared'])
+      - name: get prepared disk list
+        command: >
+          docker run --rm --net=host --privileged=true --pid=host --ipc=host --ulimit nofile=1024:4096 --cpu-quota=100000
+          -v /dev:/dev
+          -v /etc/localtime:/etc/localtime:ro
+          -v /var/lib/ceph:/var/lib/ceph:z,rshared
+          -v /etc/ceph:/etc/ceph:z
+          -v /var/run/ceph:/var/run/ceph:z
+          -v /var/run/udev/:/var/run/udev/
+          --entrypoint=ceph-disk
+          {{ ceph_docker_registry }}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}
+          list --format json
+        register: ceph_disk_list
+
+      - name: set_fact prepared_devices
+        set_fact:
+          prepared_devices: "{{ prepared_devices | default([]) | union([item.path]) }}"
+        with_items: "{{ ceph_disk_list.stdout | default('{}') | from_json }}"
+        when:
+          - ceph_disk_list.skipped is undefined
+          - item.path is defined
+          - item.partitions is defined
+          - (item.partitions | selectattr('type', 'match', 'data') | selectattr('state', 'defined') | map(attribute='state') | default([]) | list == ['prepared']) or
+            (item.partitions | selectattr('type', 'match', 'lockbox') | selectattr('state', 'defined') | map(attribute='state') | list == ['prepared'])
 
       - name: activate containerized osd(s)
         shell: |