From: Guillaume Abrioux Date: Thu, 17 May 2018 15:29:20 +0000 (+0200) Subject: take-over: fix bug when trying to override variable X-Git-Tag: v3.2.0beta1~132 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=415dc0a29b10b28cbd047fe28eb4dd38419ea5dc;p=ceph-ansible.git take-over: fix bug when trying to override variable A customer has been facing an issue when trying to override `monitor_interface` in inventory host file. In his use case, all nodes had the same interface for `monitor_interface` name except one. Therefore, they tried to override this variable for that node in the inventory host file but the take-over-existing-cluster playbook was failing when trying to generate the new ceph.conf file because of undefined variable. Typical error: ``` fatal: [srvcto103cnodep01]: FAILED! => {"failed": true, "msg": "'dict object' has no attribute u'ansible_bond0.15'"} ``` Including variables like this `include_vars: group_vars/all.yml` prevent us from overriding anything in inventory host file because it overwrites everything you would have defined in inventory. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1575915 Signed-off-by: Guillaume Abrioux --- diff --git a/infrastructure-playbooks/take-over-existing-cluster.yml b/infrastructure-playbooks/take-over-existing-cluster.yml index 0dcc1870f..0da650986 100644 --- a/infrastructure-playbooks/take-over-existing-cluster.yml +++ b/infrastructure-playbooks/take-over-existing-cluster.yml @@ -34,10 +34,10 @@ - iscsi-gw become: true - tasks: - - include_vars: roles/ceph-defaults/defaults/main.yml - - include_vars: group_vars/all.yml + roles: + - ceph-defaults + post_tasks: - name: get the name of the existing ceph cluster shell: | basename $(grep --exclude '*.bak' -R fsid /etc/ceph/ | egrep -o '^[^.]*' | head -n 1)