osd: always run disk_list test

there is no need to have a condition on this task, this test should be
always run since the result will be interpreted later.

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

diff --git a/roles/ceph-osd/tasks/docker/start_docker_osd.yml b/roles/ceph-osd/tasks/docker/start_docker_osd.yml
index ec62f24eb5052c7460f40157b96064125a340d2e..4960b9ed848125e43482ebca3d40c7dcaa2952f8 100644 (file)
--- a/roles/ceph-osd/tasks/docker/start_docker_osd.yml
+++ b/roles/ceph-osd/tasks/docker/start_docker_osd.yml
@@ -15,8 +15,6 @@
   changed_when: false
   failed_when: false
   register: disk_list
-  when:
-    - ceph_release_num[ceph_release] < ceph_release_num.kraken
 
 - name: generate ceph osd docker run script
   become: true

diff --git a/roles/ceph-osd/templates/ceph-osd-run.sh.j2 b/roles/ceph-osd/templates/ceph-osd-run.sh.j2
index 724d4bfb9e91f7674a860fb7e9b6c4012295872f..11f8df85268a5ca228bb8450a8737e2dff351adb 100644 (file)
--- a/roles/ceph-osd/templates/ceph-osd-run.sh.j2
+++ b/roles/ceph-osd/templates/ceph-osd-run.sh.j2
@@ -5,7 +5,7 @@
 #############
 # FUNCTIONS #
 #############
-{% if disk_list.get("rc", 1) == 0 -%}
+{% if disk_list.get('rc') == 0 -%}
 function expose_partitions () {
 DOCKER_ENV=$(docker run --rm --name expose_partitions_${1} --privileged=true -v /dev/:/dev/ -v /etc/ceph:/etc/ceph -e CLUSTER={{ cluster }} -e OSD_DEVICE=/dev/${1} {{ ceph_docker_registry }}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }} disk_list)
   docker rm -f expose_partitions_${1}