diff options
| author | Mohammed Naser <mnaser@vexxhost.com> | 2017-03-16 13:28:21 -0400 |
|---|---|---|
| committer | Mohammed Naser <mnaser@vexxhost.com> | 2017-03-18 14:13:53 -0400 |
| commit | 4c7a4099ca017512fdfdbd28ac47724abf900d09 (patch) | |
| tree | 3e355c0cb59c14c9f4dd08ff26244c5c681b6902 /tools | |
| parent | b96334db89fa1bf4f64a22b59a87acf1f5e98090 (diff) | |
| download | kolla-4c7a4099ca017512fdfdbd28ac47724abf900d09.tar.gz kolla-4c7a4099ca017512fdfdbd28ac47724abf900d09.zip | |
Remove global privilege escalation for deploy_aio.sh
At the moment, deploy_aio.sh is called with sudo which results
in the environment being wiped. This means that all Zuul
environment variables are removed and therefore when zuul-cloner
runs to checkout the branch of kolla-ansible, it will always
fallback to master as ZUUL_BRANCH is unset.
This patch removes the global usage of sudo in the tox.ini file
as there is existing usage of sudo in the bash script itself. This
will mean that we will only escalate privileges when needed rather
for the whole script and the environment variables should be passed
on properly resulting in the matching release of kolla-ansible being
checked out.
Closes-Bug: #1673756
Change-Id: I477a843e696be7136c020be6408afea01967879d
Diffstat (limited to 'tools')
| -rwxr-xr-x | tools/deploy_aio.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/deploy_aio.sh b/tools/deploy_aio.sh index bc3c032c9..d87e93ac7 100755 --- a/tools/deploy_aio.sh +++ b/tools/deploy_aio.sh @@ -24,5 +24,5 @@ pushd "${KOLLA_ANSIBLE_DIR}" sudo cp -a etc/kolla /etc/ # Generate passwords sudo tools/generate_passwords.py -./tools/deploy_aio.sh "$KOLLA_BASE" "$KOLLA_TYPE" +sudo ./tools/deploy_aio.sh "$KOLLA_BASE" "$KOLLA_TYPE" popd |