From: Bryan Stillwell Date: Thu, 25 Jun 2015 20:29:49 +0000 (-0600) Subject: Correct the logic for running 'ceph-disk prepare' X-Git-Tag: v1.0.0~201^2 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=e634315b5c7618711746e0ff6e9293252c8cf309;p=ceph-ansible.git Correct the logic for running 'ceph-disk prepare' The logic was broken here for repeated runs. We only want to run 'ceph-disk prepare' when the disk does not contain a ceph partition, is not a partition, and raw_multi_journal is set. Previously it would attempt to run 'ceph-disk prepare' when there was a ceph partition because the second half of the 'or' was still true since it isn't a partition. --- diff --git a/roles/ceph-osd/tasks/raw_multi_journal.yml b/roles/ceph-osd/tasks/raw_multi_journal.yml index bc07b5162..b63263750 100644 --- a/roles/ceph-osd/tasks/raw_multi_journal.yml +++ b/roles/ceph-osd/tasks/raw_multi_journal.yml @@ -12,7 +12,9 @@ # failed, this is why we check if the device is a partition too. - name: prepare OSD disk(s) command: "ceph-disk prepare {{ item.2 }} {{ item.3 }}" - when: (item.0.rc != 0 or item.1.rc != 0) and raw_multi_journal + when: item.0.rc != 0 and + item.1.rc != 0 and + raw_multi_journal ignore_errors: True with_together: - parted.results