]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
switch_to_container: refact wait for pg check
authorGuillaume Abrioux <gabrioux@redhat.com>
Fri, 15 May 2020 08:58:40 +0000 (10:58 +0200)
committerGuillaume Abrioux <gabrioux@redhat.com>
Fri, 22 May 2020 15:05:22 +0000 (17:05 +0200)
There is no need to make this check with several steps.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 8aed824f71e7f4ed53cd9e8e30964eff298c95cf)

infrastructure-playbooks/switch-from-non-containerized-to-containerized-ceph-daemons.yml

index b830f26a4eefeb7cdc6b1cbc1425cf05e73668e6..044891f0093b0a9f32206a5fbe4394f0eb169b18 100644 (file)
         name: ceph-osd
 
   post_tasks:
-    - name: get num_pgs
-      command: >
-        {{ container_binary }} exec ceph-mon-{{ hostvars[groups[mon_group_name][0]]['ansible_hostname'] }} ceph --cluster {{ cluster }} -s --format json
-      register: ceph_pgs
-      delegate_to: "{{ groups[mon_group_name][0] }}"
-
     - name: container - waiting for clean pgs...
       command: >
         {{ container_binary }} exec ceph-mon-{{ hostvars[groups[mon_group_name][0]]['ansible_hostname'] }} ceph --cluster {{ cluster }} -s --format json
       until: >
         (((ceph_health_post.stdout | from_json).pgmap.pgs_by_state | length) > 0)
         and
-        (((ceph_health_post.stdout | from_json).pgmap.pgs_by_state | selectattr('state_name', 'search', '^active\\+clean') | map(attribute='count') | list | sum) == (ceph_pgs.stdout | from_json).pgmap.num_pgs)
+        (((ceph_health_post.stdout | from_json).pgmap.pgs_by_state | selectattr('state_name', 'search', '^active\\+clean') | map(attribute='count') | list | sum) == (ceph_health_post.stdout | from_json).pgmap.num_pgs)
       delegate_to: "{{ groups[mon_group_name][0] }}"
       retries: "{{ health_osd_check_retries }}"
       delay: "{{ health_osd_check_delay }}"
-      when: (ceph_pgs.stdout | from_json).pgmap.num_pgs != 0
 
 
 - name: unset osd flags