Skip to content

Commit

Permalink
update: move a set_fact
Browse files Browse the repository at this point in the history
ceph-facts roles makes decisions based on the fact `rolling_update` so
it must be called before we run this role.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2014304

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit bf36592)
  • Loading branch information
guits committed Oct 28, 2021
1 parent 45a1d63 commit 54d9643
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions infrastructure-playbooks/rolling_update.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,10 @@
run_once: true
when: delegate_facts_host | bool

- name: set_fact rolling_update
set_fact:
rolling_update: true

- import_role:
name: ceph-facts

Expand All @@ -105,9 +109,6 @@
- (containerized_deployment | bool) or (dashboard_enabled | bool)
- ceph_docker_registry_auth | bool

- name: set_fact rolling_update
set_fact:
rolling_update: true

- name: upgrade ceph mon cluster
tags: mons
Expand Down

0 comments on commit 54d9643

Please sign in to comment.