From: Boris Ranto <branto@redhat.com>
Date: Tue, 20 Nov 2018 10:46:08 +0000 (+0100)
Subject: defaults/facts: Use list instead of keys
X-Git-Tag: v4.0.0beta1~212
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=dfab42a21f4a401d828412a84c74848eea6e024b;p=ceph-ansible.git

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 <branto@redhat.com>
---

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: