diff --git a/install_modules.sh b/install_modules.sh index c2aace3e4..76e70720e 100755 --- a/install_modules.sh +++ b/install_modules.sh @@ -52,17 +52,3 @@ else fi puppet module list - -if type "apt-get" 2>/dev/null; then - # apt-get update needs to be run to be able (later) to install - # ubuntu-cloud-keyring package with puppet - /usr/bin/apt-get update -else - # disable SElinux - # something is python-cffi is preventing Nova & Keystone to - # correctly run when SElinux is enforced. See bug: - # https://bugzilla.redhat.com/show_bug.cgi?id=1249685 - # We use || true because if selinux is Disabled the following - # command would fail - /usr/sbin/setenforce 0 || true -fi diff --git a/run_tests.sh b/run_tests.sh index 431bb4e32..79db88be4 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -19,6 +19,7 @@ if [ $(id -u) != 0 ]; then fi $SUDO ./install_modules.sh + # TODO(pabelanger): Move this into tools/install_tempest.sh and add logic so we # can clone tempest outside of the gate. Also, tempest should be sandboxed into # the local directory but works needs to be added into puppet to properly find @@ -51,14 +52,15 @@ function uses_debs { } if uses_debs; then - sudo apt-get install -y dstat + $SUDO apt-get update + $SUDO apt-get install -y dstat elif is_fedora; then - sudo yum install -y dstat + $SUDO yum install -y dstat fi # use dstat to monitor system activity during integration testing if type "dstat" 2>/dev/null; then - $SUDO dstat -tcmndrylpg --top-cpu-adv --top-io-adv --nocolor | sudo tee --append /var/log/dstat.log > /dev/null & + $SUDO dstat -tcmndrylpg --top-cpu-adv --top-io-adv --nocolor | $SUDO tee --append /var/log/dstat.log > /dev/null & fi # Run puppet and assert something changes.