mirror of https://github.com/ceph/ceph-ansible.git
common: remove unnecessary run_once statements
pull/6746/head1303611
introduced tasks for disabling the pg_autoscaler on pools and the balancer but thoses tasks are already executed on the first monitor node so we don't need to add the run_once statement. Signed-off-by: Dimitri Savineau <dsavinea@redhat.com> (cherry picked from commit738fa9428a
)
parent
f9d60644ad
commit
06158c2ac5
|
@ -394,18 +394,15 @@
|
|||
- name: get balancer module status
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer status -f json"
|
||||
register: balancer_status
|
||||
run_once: true
|
||||
changed_when: false
|
||||
|
||||
- name: set_fact pools_pgautoscaler_mode
|
||||
set_fact:
|
||||
pools_pgautoscaler_mode: "{{ pools_pgautoscaler_mode | default([]) | union([{'name': item.pool_name, 'mode': item.pg_autoscale_mode}]) }}"
|
||||
run_once: true
|
||||
with_items: "{{ (pool_list.stdout | default('{}') | from_json)['pools'] }}"
|
||||
|
||||
- name: disable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer off"
|
||||
run_once: true
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
|
||||
|
@ -415,7 +412,6 @@
|
|||
cluster: "{{ cluster }}"
|
||||
pg_autoscale_mode: false
|
||||
with_items: "{{ pools_pgautoscaler_mode }}"
|
||||
run_once: true
|
||||
when:
|
||||
- pools_pgautoscaler_mode is defined
|
||||
- item.mode == 'on'
|
||||
|
@ -536,7 +532,6 @@
|
|||
name: "{{ item.name }}"
|
||||
cluster: "{{ cluster }}"
|
||||
pg_autoscale_mode: true
|
||||
run_once: true
|
||||
with_items: "{{ pools_pgautoscaler_mode }}"
|
||||
when:
|
||||
- pools_pgautoscaler_mode is defined
|
||||
|
@ -559,7 +554,6 @@
|
|||
|
||||
- name: re-enable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer on"
|
||||
run_once: true
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
|
||||
|
|
|
@ -218,24 +218,20 @@
|
|||
- name: get pool list
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} osd dump -f json"
|
||||
register: pool_list
|
||||
run_once: true
|
||||
changed_when: false
|
||||
|
||||
- name: get balancer module status
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer status -f json"
|
||||
register: balancer_status
|
||||
run_once: true
|
||||
changed_when: false
|
||||
|
||||
- name: set_fact pools_pgautoscaler_mode
|
||||
set_fact:
|
||||
pools_pgautoscaler_mode: "{{ pools_pgautoscaler_mode | default([]) | union([{'name': item.pool_name, 'mode': item.pg_autoscale_mode}]) }}"
|
||||
run_once: true
|
||||
with_items: "{{ (pool_list.stdout | default('{}') | from_json)['pools'] }}"
|
||||
|
||||
- name: disable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer off"
|
||||
run_once: true
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
|
||||
|
@ -245,7 +241,6 @@
|
|||
cluster: "{{ cluster }}"
|
||||
pg_autoscale_mode: false
|
||||
with_items: "{{ pools_pgautoscaler_mode }}"
|
||||
run_once: true
|
||||
when:
|
||||
- pools_pgautoscaler_mode is defined
|
||||
- item.mode == 'on'
|
||||
|
@ -410,7 +405,6 @@
|
|||
cluster: "{{ cluster }}"
|
||||
pg_autoscale_mode: true
|
||||
with_items: "{{ pools_pgautoscaler_mode }}"
|
||||
run_once: true
|
||||
when:
|
||||
- pools_pgautoscaler_mode is defined
|
||||
- item.mode == 'on'
|
||||
|
@ -432,7 +426,6 @@
|
|||
|
||||
- name: re-enable balancer
|
||||
command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer on"
|
||||
run_once: true
|
||||
changed_when: false
|
||||
when: (balancer_status.stdout | from_json)['active'] | bool
|
||||
|
||||
|
|
Loading…
Reference in New Issue