mirror of https://github.com/ceph/ceph-ansible.git
infra: use dedicated variables for balancer status
The balancer status is registered during the cephadm-adopt, rolling_update
and swith2container playbooks. But it is also used in the ceph-handler role
which is included in those playbooks too.
Even if the ceph-handler tasks are skipped for rolling_update and
switch2container, the balancer_status variable is erased with the skip task
result.
play1:
register: balancer_status
play2:
register: balancer_status <-- skipped
play3:
when: (balancer_status.stdout | from_json)['active'] | bool
This leads to issue like:
The conditional check '(balancer_status.stdout | from_json)['active'] | bool'
failed. The error was: Unexpected templating type error occurred on
({% if (balancer_status.stdout | from_json)['active'] | bool %} True
{% else %} False {% endif %}): expected string or buffer.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1982054
Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
(cherry picked from commit 386661699b
)
hotfix-bz2014304
v4.0.62.1
parent
8e939dc377
commit
91834f6c9f
|
@ -391,7 +391,7 @@
|
|||
|
||||
- name: get balancer module status
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer status -f json"
|
||||
register: balancer_status
|
||||
register: balancer_status_update
|
||||
changed_when: false
|
||||
|
||||
- name: set_fact pools_pgautoscaler_mode
|
||||
|
@ -402,7 +402,7 @@
|
|||
- name: disable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer off"
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
when: (balancer_status_update.stdout | from_json)['active'] | bool
|
||||
|
||||
- name: disable pg autoscale on pools
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} osd pool set {{ item.name }} pg_autoscale_mode off"
|
||||
|
@ -540,7 +540,7 @@
|
|||
- name: re-enable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer on"
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
when: (balancer_status_update.stdout | from_json)['active'] | bool
|
||||
|
||||
- name: set_fact container_exec_cmd_osd
|
||||
set_fact:
|
||||
|
|
|
@ -221,7 +221,7 @@
|
|||
|
||||
- name: get balancer module status
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer status -f json"
|
||||
register: balancer_status
|
||||
register: balancer_status_switch
|
||||
changed_when: false
|
||||
|
||||
- name: set_fact pools_pgautoscaler_mode
|
||||
|
@ -232,7 +232,7 @@
|
|||
- name: disable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer off"
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
when: (balancer_status_switch.stdout | from_json)['active'] | bool
|
||||
|
||||
- name: disable pg autoscale on pools
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} osd pool set {{ item.name }} pg_autoscale_mode off"
|
||||
|
@ -421,7 +421,7 @@
|
|||
- name: re-enable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer on"
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
when: (balancer_status_switch.stdout | from_json)['active'] | bool
|
||||
|
||||
|
||||
- name: switching from non-containerized to containerized ceph mds
|
||||
|
|
Loading…
Reference in New Issue