From cf4a430d0b1d7946fc65babc7841ea1e7c85cbee Mon Sep 17 00:00:00 2001 From: Guillaume Abrioux Date: Mon, 11 Jul 2022 16:07:13 +0200 Subject: [PATCH] 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 --- roles/ceph-config/tasks/main.yml | 5 ----- roles/ceph-osd/tasks/main.yml | 7 ++++++- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/roles/ceph-config/tasks/main.yml b/roles/ceph-config/tasks/main.yml index 2655acad3..9267199a0 100644 --- 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 c224ae6b6..3dc1e2117 100644 --- 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] }}"