]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-build.git/commit
ceph-ansible-pr-syntax-chec: add test_ceph_release_in_ceph_default 1104/head
authorSébastien Han <seb@redhat.com>
Wed, 22 Aug 2018 09:30:53 +0000 (11:30 +0200)
committerSébastien Han <seb@redhat.com>
Wed, 22 Aug 2018 10:04:32 +0000 (12:04 +0200)
commit1b96ccc9daa8e9eaa276a8f250dc38c90466c9c5
tree214d8e012ed9a0b15e0ccc6594ca375c71bf9073
parentbe210d45861c6e4da8b689578d4917a9886b61b0
ceph-ansible-pr-syntax-chec: add test_ceph_release_in_ceph_default
function

This new function searches for statements like '    -
ceph_release_num[ceph_release] ' in the ceph-defaults role. These must
never be used since the ceph_release is set by role running after
ceph-defaults.
However, the ceph-ansible ci passes this variable when running so it's
always defined, even before ceph-defaults runs. This is particularity
from our CI. So this is done, our Ci won't complain but general users
will end with an error like "'dict object' has no attribute" since the
variable does not exist at the time of the ceph-defaults play.

Signed-off-by: Sébastien Han <seb@redhat.com>
ceph-ansible-pr-syntax-check/build/build