diff options
| author | Ed Santiago <santiago@redhat.com> | 2021-11-08 13:44:03 -0700 |
|---|---|---|
| committer | Ed Santiago <santiago@redhat.com> | 2021-11-08 14:23:55 -0700 |
| commit | 2ed31f9f1d1cfe72fbe3871607dca4e681f64a05 (patch) | |
| tree | 5a6f181824f59a7baf8f278cb725448b717f061f /contrib/cirrus/pr-should-include-tests | |
| parent | 3a31ac50daa819e2dec686d2bdfc2e92c974e28e (diff) | |
| download | podman-2ed31f9f1d1cfe72fbe3871607dca4e681f64a05.tar.gz podman-2ed31f9f1d1cfe72fbe3871607dca4e681f64a05.tar.bz2 podman-2ed31f9f1d1cfe72fbe3871607dca4e681f64a05.zip | |
Minor test tweaks
- remove 'NO TESTS NEEDED' as a valid bypass string. Henceforth
only 'NO NEW TESTS NEEDED' will work.
- add a debugging aid for #11871, in which bodhi tests time out
in nslookup.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'contrib/cirrus/pr-should-include-tests')
| -rwxr-xr-x | contrib/cirrus/pr-should-include-tests | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/contrib/cirrus/pr-should-include-tests b/contrib/cirrus/pr-should-include-tests index 4b6329311..8103df41d 100755 --- a/contrib/cirrus/pr-should-include-tests +++ b/contrib/cirrus/pr-should-include-tests @@ -12,9 +12,6 @@ fi if [[ "${CIRRUS_CHANGE_MESSAGE}" =~ NO.NEW.TESTS.NEEDED ]]; then exit 0 fi -if [[ "${CIRRUS_CHANGE_MESSAGE}" =~ NO.TESTS.NEEDED ]]; then - exit 0 -fi # HEAD should be good enough, but the CIRRUS envariable allows us to test head=${CIRRUS_CHANGE_IN_REPO:-HEAD} @@ -52,14 +49,11 @@ if [[ -z "$filtered_changes" ]]; then exit 0 fi -# One last chance: perhaps the developer included the magic '[NO (NEW) TESTS NEEDED]' +# One last chance: perhaps the developer included the magic '[NO NEW TESTS NEEDED]' # string in an amended commit. if git log --format=%B ${base}..${head} | fgrep '[NO NEW TESTS NEEDED]'; then exit 0 fi -if git log --format=%B ${base}..${head} | fgrep '[NO TESTS NEEDED]'; then - exit 0 -fi cat <<EOF $(basename $0): PR does not include changes in the 'tests' directory |