infrastructure-playbooks: Get Ceph info in check mode

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 <bknecht@protonmail.ch>

diff --git a/infrastructure-playbooks/cephadm-adopt.yml b/infrastructure-playbooks/cephadm-adopt.yml
index 21dcca2cb468aea031008effa93f23ed46fff03a..ddc2993a3e9e5b70d356287536eb8b6a188b101a 100644 (file)
--- 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 6da698da1bac86511bc47a745957fac8f6c16fe2..baa7f0e9748dfd90fd9f5269bf42574e80846f4f 100644 (file)
--- a/infrastructure-playbooks/rolling_update.yml
+++ b/infrastructure-playbooks/rolling_update.yml
@@ -412,11 +412,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 862e4ffa84f6260da44da36165c8d361a6e26f67..df9b13098448a3ab6da394bec8a9e47030e59572 100644 (file)
--- 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