]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
playbook: remove duplicate facts
authorDimitri Savineau <dsavinea@redhat.com>
Tue, 15 Oct 2019 20:52:45 +0000 (16:52 -0400)
committerDimitri Savineau <savineau.dimitri@gmail.com>
Wed, 16 Oct 2019 14:41:20 +0000 (10:41 -0400)
The is_atomic and container_binary facts are already defined in the
ceph-facts role so we don't need to have dedicated tasks for that
before the ceph-facts role exectution.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
(cherry picked from commit 643b50bd4fe4459a1263562cf0e4ed81123390ba)

site-container.yml.sample
site.yml.sample

index 59efb62ac18675dfc4544d751001a4a44bbaeeef..b0cedb61691f3d33775dad9164a57a2aa499ddd2 100644 (file)
       run_once: true
       when: delegate_facts_host | bool
 
-    - name: check if it is atomic host
-      stat:
-        path: /run/ostree-booted
-      register: stat_ostree
-      tags: always
-
-    - name: set_fact is_atomic
-      set_fact:
-        is_atomic: '{{ stat_ostree.stat.exists }}'
-      tags: always
-
-    - name: check if podman binary is present
-      stat:
-        path: /usr/bin/podman
-      register: podman_binary
-
-    - name: set_fact container_binary
-      set_fact:
-        container_binary: "{{ 'podman' if (podman_binary.stat.exists and ansible_distribution == 'Fedora') or (ansible_os_family == 'RedHat' and ansible_distribution_major_version == '8') else 'docker' }}"
-
     - import_role:
         name: ceph-defaults
       tags: [with_pkg, fetch_container_image]
index d34c73948e815a3a2ea04c506c64af05c61be92f..2bedfe8fee5b6470c50c08beb6868b87d89c0bee 100644 (file)
       run_once: true
       when: delegate_facts_host | bool
 
-    - name: check if it is atomic host
-      stat:
-        path: /run/ostree-booted
-      register: stat_ostree
-      tags: always
-
-    - name: set_fact is_atomic
-      set_fact:
-        is_atomic: '{{ stat_ostree.stat.exists }}'
-      tags: always
-
   tasks:
     - import_role:
         name: ceph-defaults