opensource/podman
1
0
Fork 0
mirror of https://github.com/containers/podman.git synced 2025-08-14 19:12:48 +08:00
Code Issues Packages Projects Releases Wiki Activity

CI test runner: upgrade tests rely on system tests

Browse Source 
Prevent future occurrences of #19894, by making upgrade tests
run any time there's a change to system tests. That's overly
broad: upgrade tests only rely on test/system/helpers.bash,
not test/system/anything-else. IMHO the cost of CI breaking
is higher than the cost of running unnecessary jobs.

Signed-off-by: Ed Santiago <santiago@redhat.com>
This commit is contained in:
Ed Santiago
2023-09-07 14:23:30 -06:00
parent c0d36b8ab8
commit afeab39d10
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
contrib/cirrus/runner.sh
View File

@ -80,7 +80,7 @@ function _run_sys() {
} }
function _run_upgrade_test() { function _run_upgrade_test() {
_bail_if_test_can_be_skipped test/upgrade _bail_if_test_can_be_skipped test/system test/upgrade
bats test/upgrade |& logformatter bats test/upgrade |& logformatter
} }
@ -408,7 +408,7 @@ _run_machine() {
} }
# Optimization: will exit if the only PR diffs are under docs/ or tests/ # Optimization: will exit if the only PR diffs are under docs/ or tests/
# with the exception of any given arguments. E.g., don't run e2e or upgrade # with the exception of any given arguments. E.g., don't run e2e or unit
# or bud tests if the only PR changes are in test/system. # or bud tests if the only PR changes are in test/system.
function _bail_if_test_can_be_skipped() { function _bail_if_test_can_be_skipped() {
local head base diffs local head base diffs