]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
mds: move mds fs pools creation
authorGuillaume Abrioux <gabrioux@redhat.com>
Fri, 25 May 2018 00:39:01 +0000 (02:39 +0200)
committerGuillaume Abrioux <gabrioux@redhat.com>
Fri, 25 May 2018 09:16:56 +0000 (11:16 +0200)
When collocating mds on monitor node, the cephpfs will fail
because `docker_exec_cmd` is reset to `ceph-mds-monXX` which is
incorrect because we need to delegate the task on `ceph-mon-monXX`.
In addition, it wouldn't have worked since `ceph-mds-monXX` container
isn't started yet.

Moving the task earlier in the `ceph-mds` role will fix this issue.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1578086
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
roles/ceph-mds/tasks/main.yml

index adf971a106b686a4af5febb9f70199b62c50631a..d012751ea1257b7f0baff09c8ef22e5c3896945f 100644 (file)
@@ -1,4 +1,9 @@
 ---
+- name: include create_mds_filesystems.yml
+  include: create_mds_filesystems.yml
+  when:
+    - inventory_hostname == groups[mds_group_name] | first
+
 - name: set_fact docker_exec_cmd
   set_fact:
     docker_exec_cmd: "docker exec ceph-mds-{{ ansible_hostname }}"
@@ -8,11 +13,6 @@
 - name: include common.yml
   include: common.yml
 
-- name: include create_mds_filesystems.yml
-  include: create_mds_filesystems.yml
-  when:
-    - inventory_hostname == groups[mds_group_name] | first
-
 - name: non_containerized.yml
   include: non_containerized.yml
   when: not containerized_deployment