Skip to content

Commit

Permalink
Cirrus: Fix & remove GraphQL API tests
Browse files Browse the repository at this point in the history
These tests don't need to run for every PR, and the thing they're
testing never runs on any branch except `main`.  Stop them from
running on this branch.  In case someone decides to try and use them,
also fix the errant API call which triggered the original problem.

Ref: containers#17383

Signed-off-by: Chris Evich <[email protected]>
  • Loading branch information
cevich committed Feb 7, 2023
1 parent 56856b0 commit 590186e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion .github/actions/check_cirrus_cron/cron_failures.sh
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ confirm_gha_environment
mkdir -p ./artifacts
cat > ./artifacts/query_raw.json << "EOF"
query {
ownerRepository(platform: "LINUX", owner: "@@OWNER@@", name: "@@REPO@@") {
ownerRepository(platform: "github", owner: "@@OWNER@@", name: "@@REPO@@") {
cronSettings {
name
lastInvocationBuild {
Expand Down
3 changes: 0 additions & 3 deletions contrib/cirrus/prebuild.sh
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,6 @@ if [[ "${DISTRO_NV}" =~ fedora ]]; then

# Tests for lib.sh
showrun ${SCRIPT_BASE}/lib.sh.t

export PREBUILD=1
showrun bash ${CIRRUS_WORKING_DIR}/.github/actions/check_cirrus_cron/test.sh
fi

msg "Checking 3rd party network service connectivity"
Expand Down

0 comments on commit 590186e

Please sign in to comment.