From c8348ab0d9b7e24cbc6769952a04c470095fa956 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Beno=C3=AEt=20Knecht?= Date: Mon, 26 Jul 2021 17:10:19 +0200 Subject: [PATCH] infrastructure-playbooks: Get Ceph info in check mode MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit In the `set osd flags` block, run the Ceph commands that gather information from the cluster (and don't make any changes to it) even when running in check mode. This allows the tasks that depend on the variables set by those tasks to succeed in check mode. Signed-off-by: Benoît Knecht (cherry picked from commit d7653dca95247e52c4a6821c1eec00748263082a) --- infrastructure-playbooks/cephadm-adopt.yml | 2 ++ infrastructure-playbooks/rolling_update.yml | 2 ++ ...h-from-non-containerized-to-containerized-ceph-daemons.yml | 4 +++- 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/infrastructure-playbooks/cephadm-adopt.yml b/infrastructure-playbooks/cephadm-adopt.yml index 21dcca2cb..ddc2993a3 100644 --- a/infrastructure-playbooks/cephadm-adopt.yml +++ b/infrastructure-playbooks/cephadm-adopt.yml @@ -449,6 +449,7 @@ run_once: true delegate_to: "{{ groups[mon_group_name][0] }}" changed_when: false + check_mode: false - name: get balancer module status command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer status -f json" @@ -456,6 +457,7 @@ run_once: true delegate_to: "{{ groups[mon_group_name][0] }}" changed_when: false + check_mode: false - name: set_fact pools_pgautoscaler_mode set_fact: diff --git a/infrastructure-playbooks/rolling_update.yml b/infrastructure-playbooks/rolling_update.yml index a1600a772..8644723e2 100644 --- a/infrastructure-playbooks/rolling_update.yml +++ b/infrastructure-playbooks/rolling_update.yml @@ -431,11 +431,13 @@ register: pool_list run_once: true changed_when: false + check_mode: false - name: get balancer module status command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer status -f json" register: balancer_status changed_when: false + check_mode: false - name: set_fact pools_pgautoscaler_mode set_fact: diff --git a/infrastructure-playbooks/switch-from-non-containerized-to-containerized-ceph-daemons.yml b/infrastructure-playbooks/switch-from-non-containerized-to-containerized-ceph-daemons.yml index 862e4ffa8..df9b13098 100644 --- a/infrastructure-playbooks/switch-from-non-containerized-to-containerized-ceph-daemons.yml +++ b/infrastructure-playbooks/switch-from-non-containerized-to-containerized-ceph-daemons.yml @@ -219,11 +219,13 @@ command: "{{ ceph_cmd }} --cluster {{ cluster }} osd dump -f json" register: pool_list changed_when: false + check_mode: false - name: get balancer module status command: "{{ ceph_cmd }} --cluster {{ cluster }} balancer status -f json" register: balancer_status changed_when: false + check_mode: false - name: set_fact pools_pgautoscaler_mode set_fact: @@ -749,4 +751,4 @@ # as in this case we know we want all owned by ceph user - name: set proper ownership on ceph directories command: "find /var/lib/ceph /etc/ceph -not -( -user {{ ceph_uid }} -or -group {{ ceph_uid }} -) -execdir chown {{ ceph_uid }}:{{ ceph_uid }} {} +" - changed_when: false \ No newline at end of file + changed_when: false -- 2.39.5