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

2 files changed, 7 insertions, 12 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
diff --git a/tox.ini b/tox.ini
index 7dec61e51..fc032ff94 100644
--- a/tox.ini
+++ b/tox.ini
@@ -145,53 +145,49 @@ commands =
 [testenv:deploy-centos-binary]
 whitelist_externals = find
                       bash
-                      sudo
 setenv =
   DOCKER_BUILD_TEST=1
 commands =
   find . -type f -name "*.py[c|o]" -delete
   bash -c "if [ ! -d .testrepository ]; then testr init; fi"
   testr run test_build.DeployTestCentosBinary
-  sudo {toxinidir}/tools/deploy_aio.sh centos binary
+  {toxinidir}/tools/deploy_aio.sh centos binary
   {toxinidir}/tools/dump_info.sh
 
 [testenv:deploy-centos-source]
 whitelist_externals = find
                       bash
-                      sudo
 setenv =
   DOCKER_BUILD_TEST=1
 commands =
   find . -type f -name "*.py[c|o]" -delete
   bash -c "if [ ! -d .testrepository ]; then testr init; fi"
   testr run test_build.DeployTestCentosSource
-  sudo {toxinidir}/tools/deploy_aio.sh centos source
+  {toxinidir}/tools/deploy_aio.sh centos source
   {toxinidir}/tools/dump_info.sh
 
 [testenv:deploy-oraclelinux-binary]
 whitelist_externals = find
                       bash
-                      sudo
 setenv =
   DOCKER_BUILD_TEST=1
 commands =
   find . -type f -name "*.py[c|o]" -delete
   bash -c "if [ ! -d .testrepository ]; then testr init; fi"
   testr run test_build.DeployTestOracleLinuxBinary
-  sudo {toxinidir}/tools/deploy_aio.sh oraclelinux binary
+  {toxinidir}/tools/deploy_aio.sh oraclelinux binary
   {toxinidir}/tools/dump_info.sh
 
 [testenv:deploy-oraclelinux-source]
 whitelist_externals = find
                       bash
-                      sudo
 setenv =
   DOCKER_BUILD_TEST=1
 commands =
   find . -type f -name "*.py[c|o]" -delete
   bash -c "if [ ! -d .testrepository ]; then testr init; fi"
   testr run test_build.DeployTestOracleLinuxSource
-  sudo {toxinidir}/tools/deploy_aio.sh oraclelinux source
+  {toxinidir}/tools/deploy_aio.sh oraclelinux source
   {toxinidir}/tools/dump_info.sh
 
 [testenv:deploy-ubuntu-binary]
@@ -203,20 +199,19 @@ commands =
   find . -type f -name "*.py[c|o]" -delete
   bash -c "if [ ! -d .testrepository ]; then testr init; fi"
   testr run test_build.DeployTestUbuntuBinary
-  sudo {toxinidir}/tools/deploy_aio.sh ubuntu binary
+  {toxinidir}/tools/deploy_aio.sh ubuntu binary
   {toxinidir}/tools/dump_info.sh
 
 [testenv:deploy-ubuntu-source]
 whitelist_externals = find
                       bash
-                      sudo
 setenv =
   DOCKER_BUILD_TEST=1
 commands =
   find . -type f -name "*.py[c|o]" -delete
   bash -c "if [ ! -d .testrepository ]; then testr init; fi"
   testr run test_build.DeployTestUbuntuSource
-  sudo {toxinidir}/tools/deploy_aio.sh ubuntu source
+  {toxinidir}/tools/deploy_aio.sh ubuntu source
   {toxinidir}/tools/dump_info.sh
 
 [testenv:deploy-multinode-ubuntu-source]