Skip to content

Commit

Permalink
fix(ci): trigger e2e on keycloakify changes; don't wait for each imag…
Browse files Browse the repository at this point in the history
…e instead use timeout (#1175)

* fix(ci): add `keycloak` to paths that trigger E2E

* No need to wait for particular images, kubernetes will retry pulling images and backoff if not found

Given we build many docker images, we would otherwise wait for each - not just the 2 here.

Instead, use timeout to fail if the images never make it.
  • Loading branch information
corneliusroemer authored Feb 28, 2024
1 parent 0b13afb commit 2254a5b
Showing 1 changed file with 3 additions and 15 deletions.
18 changes: 3 additions & 15 deletions .github/workflows/e2e-k3d.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,11 @@ on:
workflow_dispatch:
push:
paths:
- "website/**"
- "backend/**"
- "keycloak/**"
- "kubernetes/**"
- "preprocessing/**"
- "website/**"
- "deploy.py"
- ".github/scripts/**"
- ".github/workflows/**"
Expand Down Expand Up @@ -80,26 +81,13 @@ jobs:
run: cd website && npx playwright install-deps
if: steps.playwright-cache.outputs.cache-hit == 'true'

- name: Wait for Backend Docker Image
uses: lewagon/[email protected]
with:
ref: ${{ github.sha }}
check-name: Build Backend Docker Image
repo-token: ${{ secrets.GITHUB_TOKEN }}

- name: Wait for Website Docker Image
uses: lewagon/[email protected]
with:
ref: ${{ github.sha }}
check-name: Build Website Docker Image
repo-token: ${{ secrets.GITHUB_TOKEN }}

- name: Deploy with helm
uses: WyriHaximus/github-action-helm3@v4
with:
exec: ./deploy.py --verbose helm --branch ${{ github.ref_name }} --sha ${{ github.sha }} --dockerconfigjson ${{ secrets.GHCR_DOCKER_CONFIG }}

- name: Wait for the pods to be ready
timeout-minutes: 10
run: ./.github/scripts/wait_for_pods_to_be_ready.py
- name: Sleep for 20 secs
run: sleep 20
Expand Down

0 comments on commit 2254a5b

Please sign in to comment.