From a479774026a7bc2fe066a8ff9b5ad5d10aa1cec8 Mon Sep 17 00:00:00 2001 From: Adam Williamson Date: Mon, 23 Jul 2018 14:08:37 -0700 Subject: [PATCH] Correct previous commit Should be if, not unless...damn exit codes. Signed-off-by: Adam Williamson --- tests/role_deploy_domain_controller.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/role_deploy_domain_controller.pm b/tests/role_deploy_domain_controller.pm index ab744b41..57c165ee 100644 --- a/tests/role_deploy_domain_controller.pm +++ b/tests/role_deploy_domain_controller.pm @@ -78,7 +78,7 @@ sub run { # so no rolekit. First install the necessary packages assert_script_run "dnf -y groupinstall freeipa-server", 600; # FIXME workaround RHBZ#1606541 until it's fixed - unless (script_run "ls -l /usr/sbin/setup-ds.pl") { + if (script_run "ls -l /usr/sbin/setup-ds.pl") { record_soft_failure "389-ds-base-legacy-tools missing - #1606541"; assert_script_run "dnf -y install 389-ds-base-legacy-tools", 180; }