Skip to content

Commit 974702f

Browse files
committed
Fix conformance release workflow
Problem: The job used in the conformance tests to wait for the release to exist wasn't working properly. It only saw jobs in progress, and not jobs that are pending, which is what we needed to wait for. Solution: Removed the action and replaced with a basic polling loop to ensure the NGF images exist before continuing with the tests.
1 parent 336ec98 commit 974702f

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

.github/workflows/conformance.yml

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@ on:
1414
schedule:
1515
- cron: "0 4 * * *" # run every day at 4am UTC
1616

17+
defaults:
18+
run:
19+
shell: bash
20+
1721
concurrency:
1822
group: ${{ github.ref_name }}-conformance
1923
cancel-in-progress: true
@@ -134,19 +138,16 @@ jobs:
134138

135139
- name: Wait for release to exist
136140
if: startsWith(github.ref, 'refs/tags/')
137-
uses: lewagon/wait-on-check-action@e106e5c43e8ca1edea6383a39a01c5ca495fd812 # v1.3.1
138-
with:
139-
ref: ${{ github.ref }}
140-
check-name: 'Build Image'
141-
repo-token: ${{ secrets.GITHUB_TOKEN }}
141+
run: |
142+
REF=${{ github.ref_name }}
143+
until docker pull ghcr.io/nginxinc/nginx-gateway-fabric:${REF#v}; do sleep 5; done
144+
until docker pull ghcr.io/nginxinc/nginx-gateway-fabric/nginx:${REF#v}; do sleep 5; done
142145
143146
- name: Setup conformance tests
144147
run: |
145148
ngf_prefix=$(echo ${{ steps.ngf-meta.outputs.tags }} | cut -d ":" -f 1)
146149
ngf_tag=$(echo ${{ steps.ngf-meta.outputs.tags }} | cut -d ":" -f 2)
147-
if [ ${{ github.event_name }} == "schedule" ]; then
148-
export GW_API_VERSION=main
149-
fi
150+
if [ ${{ github.event_name }} == "schedule" ]; then export GW_API_VERSION=main; fi
150151
make install-ngf-local-no-build NGF_PREFIX=${ngf_prefix} NGF_TAG=${ngf_tag}
151152
working-directory: ./conformance
152153

0 commit comments

Comments
 (0)