From c267740024f5280528dc22622f13fc8bf2194840 Mon Sep 17 00:00:00 2001 From: Paul Belanger Date: Tue, 18 Jul 2017 14:43:56 -0400 Subject: [PATCH] Remove check_sudo_usage logic from tox This doesn't make much sense in an ansible only world. There is my ways today to block sudo or check if sudo failed with ansible. No need to include our custom bash script for this check. If a job wants sudo, they can either not use the revoke-sudo role or write another job. Change-Id: I267541a813b5ff6e7301f50421af46a0db33304d Signed-off-by: Paul Belanger --- roles/tox/tasks/main.yaml | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/roles/tox/tasks/main.yaml b/roles/tox/tasks/main.yaml index ed855f63e..a29987211 100644 --- a/roles/tox/tasks/main.yaml +++ b/roles/tox/tasks/main.yaml @@ -51,19 +51,6 @@ fi } - function check_sudo_usage { - sudo $script_path/jenkins-sudo-grep.sh post - sudoresult=$? - - if [ $sudoresult -ne "0" ]; then - echo - echo "This test has failed because it attempted to execute commands" - echo "with sudo. See above for the exact commands used." - echo - exit 1 - fi - } - function check_oom { $script_path/jenkins-oom-grep.sh post oomresult=$? @@ -108,7 +95,6 @@ freeze_venv process_testr_artifacts - check_sudo_usage check_oom check_nose_html