Skip to content

Commit

Permalink
ceph-config: drop osd_memory_target from ceph_conf_overrides
Browse files Browse the repository at this point in the history
As it's always being set in ceph.conf template, it leads to having duplicated osd_memory_target keys in rendered ceph conf while defining one in ceph_conf_overrides.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
  • Loading branch information
clwluvw authored and guits committed Jun 1, 2023
1 parent 23a8bbc commit f90d5d8
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions roles/ceph-config/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,13 @@
- "{{ ceph_conf_overrides.get('osd', {}).get('osd_memory_target', '') }}"
when: item

- name: drop osd_memory_target from conf override
set_fact:
ceph_conf_overrides: "{{ ceph_conf_overrides | combine({'osd': {item: omit}}, recursive=true) }}"
loop:
- osd memory target
- 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 }}"
Expand Down

0 comments on commit f90d5d8

Please sign in to comment.