ceph
/
ceph-ansible
mirror of https://github.com/ceph/ceph-ansible.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Force osd pool min_size value to integer 

After b8d580b and e9e5d5a we could have either item.min_size or
osd_pool_default_min_size using string instead of int causing the
condition to be true when it's false.
As a result, the task could try to set the pool min_size value to
0 which leads to:

Error EINVAL: pool min_size must be between 1 and 1

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
pull/3686/head
Dimitri Savineau 2019-03-05 14:17:11 -05:00 committed by mergify[bot]
parent cb381b41fe
commit 4d32ecc980
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
roles/ceph-client/tasks/create_users_keys.yml
View File

@ -122,7 +122,7 @@
changed_when: false
when:
- pools | length > 0
- item.min_size | default(osd_pool_default_min_size) > ceph_osd_pool_default_min_size
- (item.min_size | default(osd_pool_default_min_size))|int > ceph_osd_pool_default_min_size
- name: assign application to pool(s)
command: "{{ docker_exec_cmd | default('') }} ceph --cluster {{ cluster }} osd pool application enable {{ item.name }} {{ item.application }}"

2
roles/ceph-mds/tasks/create_mds_filesystems.yml
View File

@ -22,7 +22,7 @@
delegate_to: "{{ groups[mon_group_name][0] }}"
changed_when: false
when:
- item.min_size | default(osd_pool_default_min_size) > ceph_osd_pool_default_min_size
- (item.min_size | default(osd_pool_default_min_size))|int > ceph_osd_pool_default_min_size
- name: check if ceph filesystem already exists
command: "{{ hostvars[groups[mon_group_name][0]]['docker_exec_cmd'] | default('') }} ceph --cluster {{ cluster }} fs get {{ cephfs }}"

2
roles/ceph-osd/tasks/openstack_config.yml
View File

@ -60,7 +60,7 @@
delegate_to: "{{ groups[mon_group_name][0] }}"
changed_when: false
when:
- item.min_size | default(osd_pool_default_min_size) > ceph_osd_pool_default_min_size
- (item.min_size | default(osd_pool_default_min_size))|int > ceph_osd_pool_default_min_size
- name: assign application to pool(s)
command: "{{ hostvars[groups[mon_group_name][0]]['docker_exec_cmd'] | default('') }} ceph --cluster {{ cluster }} osd pool application enable {{ item.name }} {{ item.application }}"