From 5999058d07deecc63a303e339cdf617f46595917 Mon Sep 17 00:00:00 2001 From: Adam Williamson Date: Tue, 10 Jul 2018 16:21:54 -0700 Subject: [PATCH] We should *not* check CURRREL for rolekit in _check ...because by this point in the upgrade test, the system is upgraded, and rolekit won't be there on F29+. Signed-off-by: Adam Williamson --- tests/role_deploy_domain_controller_check.pm | 2 -- 1 file changed, 2 deletions(-) diff --git a/tests/role_deploy_domain_controller_check.pm b/tests/role_deploy_domain_controller_check.pm index e5751572..d5d8d7a6 100644 --- a/tests/role_deploy_domain_controller_check.pm +++ b/tests/role_deploy_domain_controller_check.pm @@ -12,8 +12,6 @@ sub run { # always, for F29+ we decommission directly ourselves as rolekit # is deprecated my $version = get_var("VERSION"); - # for upgrade tests we need to check CURRREL not VERSION - $version = get_var("CURRREL") if (get_var("UPGRADE")); if ($version < 29 && $version ne 'Rawhide') { # check the role status, should be 'running' validate_script_output 'rolectl status domaincontroller/domain.local', sub { $_ =~ m/^running/ };