]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
Check if ceph_conf_overrides.global is defined before calling it 1301/head
authorGuillaume Abrioux <gabrioux@redhat.com>
Thu, 9 Feb 2017 09:54:49 +0000 (10:54 +0100)
committerGuillaume Abrioux <gabrioux@redhat.com>
Thu, 9 Feb 2017 19:28:58 +0000 (20:28 +0100)
Expand the fix in #1291 to all the playbook in order to get a full coverage.

Fix: #1294
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
roles/ceph-client/tasks/main.yml
roles/ceph-client/tasks/pre_requisite.yml
roles/ceph-mon/tasks/ceph_keys.yml

index b0ad7283da517a98df01e51c365a5cca5db026af..36ec8a765676242caab6fa0b3f32705dc5c2bd01 100644 (file)
@@ -3,4 +3,5 @@
 - include: create_users_keys.yml
   when:
     - user_config
+    - global_in_ceph_conf_overrides
     - ceph_conf_overrides.global.osd_pool_default_pg_num is defined
index 43abd45f7b21e80d3cda7c66e748609711927f87..788df130c5343b9a1c2c1de416d5c454561ca9c8 100644 (file)
@@ -7,3 +7,7 @@
     group: "ceph"
     mode: "0600"
   when: cephx
+
+- name: check if global key exists in ceph_conf_overrides
+  set_fact:
+    global_in_ceph_conf_overrides: "{{ 'global' in ceph_conf_overrides }}"
index 3986661a375c16973d1c8faf4a15aa83fd55d381..7908383569caeb5881377a37be51a232417d0175 100644 (file)
@@ -77,7 +77,9 @@
 
 - set_fact:
     osd_pool_default_pg_num: "{{ ceph_conf_overrides.global.osd_pool_default_pg_num }}"
-  when: ceph_conf_overrides.global.osd_pool_default_pg_num is defined
+  when:
+    - global_in_ceph_conf_overrides
+    - ceph_conf_overrides.global.osd_pool_default_pg_num is defined
 
 - name: test if rbd exists
   command: ceph --cluster {{ cluster }} osd pool stats rbd
 - include: rbd_pool_pgs.yml
   when:
     - rbd_pool_exist.rc == 0
+    - global_in_ceph_conf_overrides
     - ceph_conf_overrides.global.osd_pool_default_pg_num is defined
 
 - include: rbd_pool_size.yml
   when:
     - rbd_pool_exist.rc == 0
+    - global_in_ceph_conf_overrides
     - ceph_conf_overrides.global.osd_pool_default_size is defined
 
 - include: openstack_config.yml