]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
config: convert _osd_memory_target to int
authorGuillaume Abrioux <gabrioux@redhat.com>
Wed, 21 Nov 2018 13:38:25 +0000 (14:38 +0100)
committermergify[bot] <mergify[bot]@users.noreply.github.com>
Wed, 21 Nov 2018 14:33:27 +0000 (14:33 +0000)
ceph.conf doesn't accept float value.

Typical error seen:
```
$ sudo ceph daemon osd.2 config get osd_memory_target
Can't get admin socket path: unable to get conf option admin_socket for osd.2:
parse error setting 'osd_memory_target' to '7823740108,8' (strict_si_cast:
unit prefix not recognized)
```

This commit ensures the value inserted in ceph.conf will be an integer.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
roles/ceph-config/templates/ceph.conf.j2

index 826dbe65abd7d42bbee09319557aa1f6c27b9e9e..a8a54f8e8ca2c2f13f2554a06f62dd4c91cb1837 100644 (file)
@@ -90,11 +90,11 @@ filestore xattr use omap = true
 [osd]
 {% if is_hci and _num_osds > 0 %}
 {% if ansible_memtotal_mb * 1048576 * hci_safety_factor / _num_osds > osd_memory_target %} # hci_safety_factor is the safety factor for HCI deployments
-{% set _osd_memory_target = (ansible_memtotal_mb * 1048576 * hci_safety_factor / _num_osds) %}
+{% set _osd_memory_target = (ansible_memtotal_mb * 1048576 * hci_safety_factor / _num_osds) | int %}
 {% endif %}
 {% elif _num_osds > 0 %}
 {% if ansible_memtotal_mb * 1048576 * non_hci_safety_factor / _num_osds > osd_memory_target %} # non_hci_safety_factor is the safety factor for dedicated nodes
-{% set _osd_memory_target = (ansible_memtotal_mb * 1048576 * non_hci_safety_factor / _num_osds) %}
+{% set _osd_memory_target = (ansible_memtotal_mb * 1048576 * non_hci_safety_factor / _num_osds) | int %}
 {% endif %}
 {% endif %}
 osd memory target = {{ _osd_memory_target | default(osd_memory_target) }}