Skip to content

Commit f915440

Browse files
Artyopaanm
authored andcommitted
ci: refactor k8s-kind wait for image step
Using the wait-for-image action Also moving from /default to /test in ariane config so we don't have timeout because they'll be triggered once image build is done Signed-off-by: Antony Reynaud <antony.reynaud@isovalent.com>
1 parent f3d8004 commit f915440

4 files changed

Lines changed: 19 additions & 28 deletions

File tree

.github/ariane-config.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,7 @@ triggers:
1010
- tests-cifuzz.yaml
1111
- lint-ariane-config.yaml
1212
- conformance-kubespray.yaml
13-
- k8s-kind-network-e2e.yaml
1413
- lint-images-base.yaml
15-
- k8s-kind-network-policies-e2e.yaml
1614
# This trigger needs to be kept in sync with ariane-scheduled.yaml workflow
1715
/test\s*:
1816
workflows:
@@ -38,6 +36,8 @@ triggers:
3836
- tests-clustermesh-upgrade.yaml
3937
- tests-datapath-verifier.yaml
4038
- hubble-cli-integration-test.yaml
39+
- k8s-kind-network-e2e.yaml
40+
- k8s-kind-network-policies-e2e.yaml
4141
depends-on:
4242
- /build-images-dependency
4343
/ci-aks:

.github/workflows/conformance-k8s-network-policies.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ jobs:
8686
login-host: ${{ vars.DOCKER_READ_HOST }}
8787
login-username: ${{ vars.DOCKER_READ_USERNAME }}
8888
login-password: ${{ secrets.DOCKER_READ_PASSWORD }}
89+
auth-required: ${{ vars.DOCKER_AUTH_REQUIRED }}
8990

9091
- name: Create kind cluster
9192
uses: helm/kind-action@ef37e7f390d99f746eb8b610417061a60e82a6cc # v1.14.0

.github/workflows/k8s-kind-network-e2e.yaml

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -202,21 +202,16 @@ jobs:
202202
--docker-username="${{ vars.DOCKER_READ_USERNAME }}" \
203203
--docker-password="${{ secrets.DOCKER_READ_PASSWORD }}"
204204
205-
- name: Login to docker registry for image wait
206-
if: ${{ vars.DOCKER_AUTH_REQUIRED == 'true' }}
207-
uses: docker/login-action@650006c6eb7dba73a995cc03b0b2d7f5ca915bee # v4.2.0
208-
with:
209-
registry: ${{ vars.DOCKER_READ_HOST}}
210-
username: ${{ vars.DOCKER_READ_USERNAME}}
211-
password: ${{ secrets.DOCKER_READ_PASSWORD }}
212-
213205
- name: Wait for images to be available
214206
timeout-minutes: 30
215-
shell: bash
216-
run: |
217-
for image in cilium-ci operator-generic-ci hubble-relay-ci ; do
218-
until docker manifest inspect ${{ vars.DOCKER_READ_HOST }}/${{ vars.DOCKER_READ_ORG }}/$image:${{ steps.vars.outputs.sha }} &> /dev/null; do sleep 45s; done
219-
done
207+
uses: ./.github/actions/wait-for-images
208+
with:
209+
SHA: ${{ steps.vars.outputs.sha }}
210+
images: cilium-ci operator-generic-ci hubble-relay-ci
211+
login-host: ${{ vars.DOCKER_READ_HOST }}
212+
login-username: ${{ vars.DOCKER_READ_USERNAME }}
213+
login-password: ${{ secrets.DOCKER_READ_PASSWORD }}
214+
auth-required: ${{ vars.DOCKER_AUTH_REQUIRED }}
220215

221216
# Warning: since this is a privileged workflow, subsequent workflow job
222217
# steps must take care not to execute untrusted code.

.github/workflows/k8s-kind-network-policies-e2e.yaml

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -213,21 +213,16 @@ jobs:
213213
--docker-username="${{ vars.DOCKER_READ_USERNAME }}" \
214214
--docker-password="${{ secrets.DOCKER_READ_PASSWORD }}"
215215
216-
- name: Login to docker registry for image wait
217-
if: ${{ vars.DOCKER_AUTH_REQUIRED == 'true' }}
218-
uses: docker/login-action@650006c6eb7dba73a995cc03b0b2d7f5ca915bee # v4.2.0
219-
with:
220-
registry: ${{ vars.DOCKER_READ_HOST}}
221-
username: ${{ vars.DOCKER_READ_USERNAME}}
222-
password: ${{ secrets.DOCKER_READ_PASSWORD }}
223-
224216
- name: Wait for images to be available
225217
timeout-minutes: 30
226-
shell: bash
227-
run: |
228-
for image in cilium-ci operator-generic-ci hubble-relay-ci ; do
229-
until docker manifest inspect ${{ vars.DOCKER_READ_HOST }}/${{ vars.DOCKER_READ_ORG }}/$image:${{ steps.vars.outputs.sha }} &> /dev/null; do sleep 45s; done
230-
done
218+
uses: ./.github/actions/wait-for-images
219+
with:
220+
SHA: ${{ steps.vars.outputs.sha }}
221+
images: cilium-ci operator-generic-ci hubble-relay-ci
222+
login-host: ${{ vars.DOCKER_READ_HOST }}
223+
login-username: ${{ vars.DOCKER_READ_USERNAME }}
224+
login-password: ${{ secrets.DOCKER_READ_PASSWORD }}
225+
auth-required: ${{ vars.DOCKER_AUTH_REQUIRED }}
231226

232227
# Warning: since this is a privileged workflow, subsequent workflow job
233228
# steps must take care not to execute untrusted code.

0 commit comments

Comments
 (0)