]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
config: use osd_memory_target value from ceph_conf_overrides if defined
authorGuillaume Abrioux <gabrioux@redhat.com>
Tue, 9 Aug 2022 20:44:36 +0000 (22:44 +0200)
committerGuillaume Abrioux <gabrioux@redhat.com>
Thu, 11 Aug 2022 09:10:24 +0000 (11:10 +0200)
otherwise it's impossible to override `osd_memory_target`
via `ceph_conf_overrides`.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2056675
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit f19dcb266ae6c6539e3ed613fdaa94fdde636b26)

roles/ceph-config/tasks/main.yml

index 968372310b4789ae328cc5d09cdb5ac3aa4c8329..ed76056242aa43ef4b37a56c29cedf045b4829bb 100644 (file)
       when:
         - devices | default([]) | length > 0
 
+    - name: set_fact _osd_memory_target
+      set_fact:
+        _osd_memory_target: "{{ item }}"
+      loop:
+        - "{{ ceph_conf_overrides.get('osd', {}).get('osd memory target', '') }}"
+        - "{{ ceph_conf_overrides.get('osd', {}).get('osd_memory_target', '') }}"
+      when:
+        - item
+        - item > osd_memory_target
+
     - name: set_fact _osd_memory_target
       set_fact:
         _osd_memory_target: "{{ ((ansible_facts['memtotal_mb'] * 1048576 * safety_factor | float) / num_osds | float) | int }}"
       when:
-        - not ceph_conf_overrides.get('osd', {}).get('osd_memory_target')
-        - not ceph_conf_overrides.get('osd', {}).get('osd memory target')
+        - _osd_memory_target is undefined
         - num_osds | default(0) | int > 0
         - ((ansible_facts['memtotal_mb'] * 1048576 * safety_factor | float) / num_osds | float) > osd_memory_target