From c34a12c8114ab1ddfd421530ed0ce372ca266cb0 Mon Sep 17 00:00:00 2001 From: Caleb Boylan Date: Fri, 3 Nov 2017 09:54:54 -0700 Subject: [PATCH] infra: fix take-over-existing-cluster.yml playbook The ansible inventory could have more than just ceph-ansible hosts, so we shouldnt use "hosts: all", also only grab one file when getting the ceph cluster name instead of failing when there is more than one file in /etc/ceph. Also fix location of the ceph.conf template (cherry picked from commit 41d10a2f6496c216eaad87112a0794e51204c578) --- .../take-over-existing-cluster.yml | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/infrastructure-playbooks/take-over-existing-cluster.yml b/infrastructure-playbooks/take-over-existing-cluster.yml index b4db02703..97a7cf884 100644 --- a/infrastructure-playbooks/take-over-existing-cluster.yml +++ b/infrastructure-playbooks/take-over-existing-cluster.yml @@ -16,20 +16,33 @@ vars_files: - roles/ceph-defaults/defaults/main.yml - group_vars/all.yml + - "host_vars/{{ ansible_hostname }}.yml" roles: - ceph-defaults - ceph-fetch-keys -- hosts: all +- hosts: + - mons + - agents + - osds + - mdss + - rgws + - nfss + - restapis + - rbdmirrors + - clients + - mgrs + - iscsi-gw become: true tasks: - include_vars: roles/ceph-defaults/defaults/main.yml - include_vars: group_vars/all.yml + - include_vars: "host_vars/{{ ansible_hostname }}.yml" - name: get the name of the existing ceph cluster shell: | - basename $(grep --exclude '*.bak' -R fsid /etc/ceph/ | egrep -o '^[^.]*') + basename $(grep --exclude '*.bak' -R fsid /etc/ceph/ | egrep -o '^[^.]*' | head -n 1) changed_when: false register: cluster_name @@ -51,7 +64,7 @@ - name: generate ceph configuration file action: config_template args: - src: "roles/ceph-common/templates/ceph.conf.j2" + src: "roles/ceph-config/templates/ceph.conf.j2" dest: "/etc/ceph/{{ cluster_name.stdout }}.conf" owner: "{{ ceph_conf_stat.stat.pw_name }}" group: "{{ ceph_conf_stat.stat.gr_name }}" -- 2.39.5