Docker fixes and cleanups

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

diff --git a/roles/ceph-mds/tasks/docker/main.yml b/roles/ceph-mds/tasks/docker/main.yml
index 8750a825993b710d41d52c02dcd1ca69547694dc..c867db8d3e07886fb8595d86b6db94af1577cd63 100644 (file)
--- a/roles/ceph-mds/tasks/docker/main.yml
+++ b/roles/ceph-mds/tasks/docker/main.yml
@@ -9,8 +9,8 @@
   when: ceph_health.rc != 0
 
 - include: pre_requisite.yml
-- include: fetch_configs.yml
 - include: dirs_permissions.yml
+- include: fetch_configs.yml
 
 - include: selinux.yml
   when: ansible_os_family == 'RedHat'

diff --git a/roles/ceph-mon/tasks/docker/main.yml b/roles/ceph-mon/tasks/docker/main.yml
index 960581b7e8d93c88a20ffdcfc19c742ba9d9dfa0..b6b668dd3b846e7425d9cb0d829a8bd75cf6ea11 100644 (file)
--- a/roles/ceph-mon/tasks/docker/main.yml
+++ b/roles/ceph-mon/tasks/docker/main.yml
@@ -11,7 +11,7 @@
 
 - name: set fact for using Atomic host
   set_fact:
-      is_atomic='{{ stat_ostree.stat.exists }}'
+      is_atomic: '{{ stat_ostree.stat.exists }}'
 
 - include: checks.yml
   when:
@@ -20,6 +20,8 @@
 
 - include: pre_requisite.yml
 
+- include: dirs_permissions.yml
+
 # let the first mon create configs and keyrings
 - include: create_configs.yml
   when:
@@ -29,8 +31,6 @@
 - include: fetch_configs.yml
   when: not mon_containerized_deployment_with_kv
 
-- include: dirs_permissions.yml
-
 - include: selinux.yml
   when: ansible_os_family == 'RedHat'
 

diff --git a/roles/ceph-osd/tasks/docker/main.yml b/roles/ceph-osd/tasks/docker/main.yml
index 9c959cca49a81117a6736b44df56e74d6357873e..c934b5a0133d0f6d56223b58de6f91f641d7dee9 100644 (file)
--- a/roles/ceph-osd/tasks/docker/main.yml
+++ b/roles/ceph-osd/tasks/docker/main.yml
@@ -16,7 +16,7 @@
 
 - name: set fact for using Atomic host
   set_fact:
-      is_atomic='{{ stat_ostree.stat.exists }}'
+      is_atomic: '{{ stat_ostree.stat.exists }}'
 
 - include: pre_requisite.yml
 

diff --git a/roles/ceph-restapi/tasks/docker/dirs_permissions.yml b/roles/ceph-restapi/tasks/docker/dirs_permissions.yml
index 6b02dbc21000e7b471d71d548afe69e7dd23c31d..c1425caea7ffe59aedcce48c0ec06ef2d1a4a0b3 100644 (file)
--- a/roles/ceph-restapi/tasks/docker/dirs_permissions.yml
+++ b/roles/ceph-restapi/tasks/docker/dirs_permissions.yml
@@ -29,12 +29,12 @@
   file:
     path: "{{ item }}"
     state: directory
-    owner: ceph
-    group: ceph
+    owner: "64045"
+    group: "64045"
     mode: "0755"
   with_items:
     - /etc/ceph/
     - /var/lib/ceph/bootstrap-osd
     - /var/lib/ceph/bootstrap-mds
     - /var/lib/ceph/bootstrap-rgw
-  when: not after_hamer
+  when: after_hamer

diff --git a/roles/ceph-restapi/tasks/docker/main.yml b/roles/ceph-restapi/tasks/docker/main.yml
index b54b12b5a257b6144fe04ec1647b178fa0be7049..6b1488474ed60bf54cabacba033a89fecaf3e11e 100644 (file)
--- a/roles/ceph-restapi/tasks/docker/main.yml
+++ b/roles/ceph-restapi/tasks/docker/main.yml
@@ -1,5 +1,5 @@
 ---
 - include: pre_requisite.yml
-- include: fetch_configs.yml
 - include: dirs_permissions.yml
+- include: fetch_configs.yml
 - include: start_docker_restapi.yml