From 8851b1039b6d4b5994a8e38df62a564f05be2d3a Mon Sep 17 00:00:00 2001 From: Guillaume Abrioux Date: Thu, 16 May 2019 18:16:28 +0200 Subject: [PATCH] ceph-ansible: give a common inventory for syntax-check let's provide a typical 'all_daemons' inventory host file for the syntax check process. Otherwise we can get failure because the site.yml file contains `hosts:` which try to resolve some `{{ groups }}` variables. Signed-off-by: Guillaume Abrioux --- ceph-ansible-pr-syntax-check/build/build | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ceph-ansible-pr-syntax-check/build/build b/ceph-ansible-pr-syntax-check/build/build index de0c256c..6d5df221 100644 --- a/ceph-ansible-pr-syntax-check/build/build +++ b/ceph-ansible-pr-syntax-check/build/build @@ -16,8 +16,8 @@ virtualenv $TEMPVENV # FUNCTIONS # ############# function syntax_check { - "$VENV"/ansible-playbook -i '127.0.0.1,' site.yml.sample --syntax-check --list-tasks -vv - "$VENV"/ansible-playbook -i '127.0.0.1,' site-docker.yml.sample --syntax-check --list-tasks -vv + "$VENV"/ansible-playbook -i "$WORKSPACE"/ceph-ansible/tests/functional/all_daemons/hosts site.yml.sample --syntax-check --list-tasks -vv + "$VENV"/ansible-playbook -i "$WORKSPACE"/ceph-ansible/tests/functional/all_daemons/hosts site-docker.yml.sample --syntax-check --list-tasks -vv # move roles and group_vars into the # infrastructure-playbooks directory for easier syntax checking @@ -26,7 +26,7 @@ function syntax_check { mv infrastructure-playbooks/group_vars/all.yml.sample infrastructure-playbooks/group_vars/all.yml export ANSIBLE_LIBRARY=$WORKSPACE/ceph-ansible/library - "$VENV"/ansible-playbook -i '127.0.0.1,' infrastructure-playbooks/*.yml --syntax-check --list-tasks -vv + "$VENV"/ansible-playbook -i "$WORKSPACE"/ceph-ansible/tests/functional/all_daemons/hosts infrastructure-playbooks/*.yml --syntax-check --list-tasks -vv } function ansible_lint { -- 2.39.5