config: followup on 8a5628b51

Add missing `--cluster {{ cluster }}` on task
`set osd_memory_target` in the main.yml file of the
ceph-config role.
Also it moves the task after ceph configuration file is actually written.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

diff --git a/roles/ceph-config/tasks/main.yml b/roles/ceph-config/tasks/main.yml
index 2655acad361c8902bff2d5063f52f1f3f1c573a0..9267199a031e04874ff11c2565346232902e1720 100644 (file)
--- a/roles/ceph-config/tasks/main.yml
+++ b/roles/ceph-config/tasks/main.yml
@@ -105,11 +105,6 @@
         - num_osds | default(0) | int > 0
         - ((ansible_facts['memtotal_mb'] * 1048576 * safety_factor | float) / num_osds | float) > osd_memory_target
 
-    - name: set osd_memory_target
-      command: "{{ ceph_cmd }} config set osd/host:{{ inventory_hostname }} osd_memory_target {{ _osd_memory_target | default(osd_memory_target) }}"
-      changed_when: false
-      delegate_to: "{{ groups[mon_group_name][0] }}"
-
 - name: create ceph conf directory
   file:
     path: "/etc/ceph"

diff --git a/roles/ceph-osd/tasks/main.yml b/roles/ceph-osd/tasks/main.yml
index c224ae6b64cf3028fc00cb924608ba8653c82e7d..3dc1e2117feb15799eed38d63b35435daf4cc259 100644 (file)
--- a/roles/ceph-osd/tasks/main.yml
+++ b/roles/ceph-osd/tasks/main.yml
@@ -112,4 +112,9 @@
     - not rolling_update | default(False) | bool
     - openstack_config | bool
     - inventory_hostname == groups[osd_group_name] | last
-  tags: wait_all_osds_up
\ No newline at end of file
+  tags: wait_all_osds_up
+
+- name: set osd_memory_target
+  command: "{{ ceph_cmd }} --cluster {{ cluster }} config set osd/host:{{ inventory_hostname }} osd_memory_target {{ _osd_memory_target | default(osd_memory_target) }}"
+  changed_when: false
+  delegate_to: "{{ groups[mon_group_name][0] }}"