From: Sébastien Han Date: Fri, 12 Oct 2018 16:32:40 +0000 (+0200) Subject: validate: only check device when they are devices X-Git-Tag: v5.0.0alpha1~416 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=2888c0825fe11176d3cd06fe3568fe340d6d6538;p=ceph-ansible.git validate: only check device when they are devices We only validate the devices that are passed if there is a list of devices to validate. Signed-off-by: Sébastien Han --- diff --git a/roles/ceph-validate/tasks/check_devices.yml b/roles/ceph-validate/tasks/check_devices.yml index 7e0803caf..884f11d45 100644 --- a/roles/ceph-validate/tasks/check_devices.yml +++ b/roles/ceph-validate/tasks/check_devices.yml @@ -24,19 +24,20 @@ register: dedicated_device_parted with_items: "{{ dedicated_devices }}" when: - - osd_scenario == 'non-collocated' + - dedicated_devices|default([]) | length > 0 - name: fail if one of the dedicated_device is not a device fail: msg: "{{ item }} is not a block special file!" with_items: "{{ dedicated_device_parted.results }}" when: - - osd_scenario == 'non-collocated' + - dedicated_devices|default([]) | length > 0 - item.failed - name: fail if number of dedicated_devices is not equal to number of devices fail: msg: "Number of dedicated_devices must be equal to number of devices. dedicated_devices: {{ dedicated_devices | length }}, devices: {{ devices | length }}" when: - - osd_scenario == 'non-collocated' - - dedicated_devices|length != devices|length + - dedicated_devices|default([]) | length > 0 + - devices | length > 0 + - dedicated_devices | length != devices | length \ No newline at end of file diff --git a/roles/ceph-validate/tasks/main.yml b/roles/ceph-validate/tasks/main.yml index 49761bd16..1c3ea4bfc 100644 --- a/roles/ceph-validate/tasks/main.yml +++ b/roles/ceph-validate/tasks/main.yml @@ -61,6 +61,7 @@ when: - osd_group_name in group_names - not osd_auto_discovery | default(False) + - devices|default([])|length > 0 - name: include check_eth_mon.yml include_tasks: check_eth_mon.yml