From dfab42a21f4a401d828412a84c74848eea6e024b Mon Sep 17 00:00:00 2001 From: Boris Ranto Date: Tue, 20 Nov 2018 11:46:08 +0100 Subject: [PATCH] defaults/facts: Use list instead of keys It is safer to use the list filter than the keys() method since the keys method does have some interoperability issues between python2 and python3 based ansible/jinja. Signed-off-by: Boris Ranto --- roles/ceph-defaults/tasks/facts.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/roles/ceph-defaults/tasks/facts.yml b/roles/ceph-defaults/tasks/facts.yml index c14f75797..548eeb634 100644 --- a/roles/ceph-defaults/tasks/facts.yml +++ b/roles/ceph-defaults/tasks/facts.yml @@ -228,7 +228,7 @@ - name: set_fact rgw_hostname set_fact: rgw_hostname: "{% set _value = ansible_hostname -%} - {% for key in ceph_current_status['servicemap']['services']['rgw']['daemons'].keys() -%} + {% for key in (ceph_current_status['servicemap']['services']['rgw']['daemons'] | list) -%} {% if key == ansible_fqdn -%} {% set _value = key -%} {% endif -%} @@ -247,4 +247,4 @@ import_tasks: set_radosgw_address.yml - name: populate service facts - service_facts: \ No newline at end of file + service_facts: -- 2.39.5