From: Noah Watkins Date: Wed, 5 Dec 2018 22:04:48 +0000 (-0800) Subject: start_osds: use list instead of keys (re-introduce) X-Git-Tag: v3.2.4~2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=e57e2d98a19e0127ed0b6e7c1c932060be3c3021;p=ceph-ansible.git start_osds: use list instead of keys (re-introduce) the python3 fix merged by: https://github.com/ceph/ceph-ansible/pull/3346 was reintroduced a few days later by: https://github.com/ceph/ceph-ansible/commit/82a6b5adec4d72eb4b7219147f2225b7b2904460 and this patch fixes it again :) Signed-off-by: Noah Watkins (cherry picked from commit 3cf5fd2c3ee1fc342ac8dc3365ed82d863c7127e) --- diff --git a/roles/ceph-osd/tasks/start_osds.yml b/roles/ceph-osd/tasks/start_osds.yml index 1284525c6..0fd18853b 100644 --- a/roles/ceph-osd/tasks/start_osds.yml +++ b/roles/ceph-osd/tasks/start_osds.yml @@ -68,7 +68,7 @@ state: started enabled: yes daemon_reload: yes - with_items: "{{ devices if osd_scenario != 'lvm' and containerized_deployment else (ceph_osd_ids.stdout | from_json).keys() if osd_scenario == 'lvm' and not containerized_deployment else osd_ids_non_container.stdout_lines }}" + with_items: "{{ devices if osd_scenario != 'lvm' and containerized_deployment else ((ceph_osd_ids.stdout | from_json).keys() | list) if osd_scenario == 'lvm' and not containerized_deployment else osd_ids_non_container.stdout_lines }}" - name: ensure systemd service override directory exists file: