Skip to content

Commit

Permalink
update: do not gather facts on each play
Browse files Browse the repository at this point in the history
There's no benefit to gather facts again on each play in
rolling_update.yml

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 2c77d00)
  • Loading branch information
guits committed Jun 30, 2021
1 parent adfb9d3 commit 676aad9
Showing 1 changed file with 17 additions and 0 deletions.
17 changes: 17 additions & 0 deletions infrastructure-playbooks/rolling_update.yml
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,7 @@
hosts: "{{ mon_group_name|default('mons') }}"
serial: 1
become: True
gather_facts: false
tasks:
- name: upgrade ceph mon cluster
block:
Expand Down Expand Up @@ -305,6 +306,7 @@
- name: reset mon_host
hosts: "{{ mon_group_name|default('mons') }}"
become: True
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand All @@ -322,6 +324,7 @@
hosts: "{{ mon_group_name|default('mons') }}"
serial: 1
become: True
gather_facts: false
tasks:
- name: upgrade mgrs when no mgr group explicitly defined in inventory
when: groups.get(mgr_group_name, []) | length == 0
Expand Down Expand Up @@ -356,6 +359,7 @@
hosts: "{{ mgr_group_name|default('mgrs') }}"
serial: 1
become: True
gather_facts: false
tasks:
# The following task has a failed_when: false
# to handle the scenario where no mgr existed before the upgrade
Expand Down Expand Up @@ -389,6 +393,7 @@
- name: set osd flags
hosts: "{{ mon_group_name | default('mons') }}[0]"
become: True
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand Down Expand Up @@ -416,6 +421,7 @@
hosts: "{{ osd_group_name|default('osds') }}"
serial: 1
become: True
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand Down Expand Up @@ -489,6 +495,7 @@
- name: complete osd upgrade
hosts: "{{ mon_group_name|default('mons') }}[0]"
become: True
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand All @@ -511,6 +518,7 @@
- name: upgrade ceph mdss cluster, deactivate all rank > 0
hosts: "{{ mon_group_name | default('mons') }}[0]"
become: true
gather_facts: false
tasks:
- name: deactivate all mds rank > 0
when: groups.get(mds_group_name, []) | length > 0
Expand Down Expand Up @@ -611,6 +619,7 @@
upgrade_ceph_packages: True
hosts: active_mdss
become: true
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand Down Expand Up @@ -656,6 +665,7 @@
upgrade_ceph_packages: True
hosts: standby_mdss
become: True
gather_facts: false

tasks:
- import_role:
Expand Down Expand Up @@ -704,6 +714,7 @@
hosts: "{{ rgw_group_name|default('rgws') }}"
serial: 1
become: True
gather_facts: false
tasks:

- import_role:
Expand Down Expand Up @@ -747,6 +758,7 @@
hosts: "{{ rbdmirror_group_name|default('rbdmirrors') }}"
serial: 1
become: True
gather_facts: false
tasks:
- name: stop ceph rbd mirror
systemd:
Expand Down Expand Up @@ -779,6 +791,7 @@
hosts: "{{ nfs_group_name|default('nfss') }}"
serial: 1
become: True
gather_facts: false
tasks:
# failed_when: false is here so that if we upgrade
# from a version of ceph that does not have nfs-ganesha
Expand Down Expand Up @@ -828,6 +841,7 @@
- "{{ iscsi_gw_group_name|default('iscsigws') }}"
serial: 1
become: True
gather_facts: false
tasks:
# failed_when: false is here so that if we upgrade
# from a version of ceph that does not have iscsi gws
Expand Down Expand Up @@ -868,6 +882,7 @@
hosts: "{{ client_group_name|default('clients') }}"
serial: "{{ client_update_batch | default(20) }}"
become: True
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand Down Expand Up @@ -924,6 +939,7 @@
- name: complete upgrade
hosts: "{{ mon_group_name | default('mons') }}"
become: True
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand Down Expand Up @@ -1039,6 +1055,7 @@
- name: show ceph status
hosts: "{{ mon_group_name|default('mons') }}"
become: True
gather_facts: false
tasks:
- import_role:
name: ceph-defaults
Expand Down

0 comments on commit 676aad9

Please sign in to comment.