_post_fail_hook: only do advisory package checks when appropriate

Same conditions as used in main.pm to load the tests in the
normal flow. It makes no sense to do this on non-update tests,
or on the non-matching support server case.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
Adam Williamson 2022-12-14 08:47:46 -08:00
parent 2023172c24
commit d012d31270
1 changed files with 8 additions and 4 deletions

View File

@ -159,10 +159,14 @@ sub post_fail_hook {
upload_logs "/var/tmp/imgbuild/program.log", failok => 1;
}
# For update tests, let's do the update package info log stuff,
# it may be useful for diagnosing the cause of the failure
advisory_get_installed_packages;
advisory_check_nonmatching_packages(fatal => 0);
if (get_var("ADVISORY_OR_TASK") && get_var("TEST") ne "install_default_update_netinst") {
unless (get_var("TEST") eq "support_server" && get_var("VERSION") ne get_var("CURRREL")) {
# For update tests, let's do the update package info log stuff,
# it may be useful for diagnosing the cause of the failure
advisory_get_installed_packages;
advisory_check_nonmatching_packages(fatal => 0);
}
}
}