Skip to content

Commit e249920

Browse files
authored
Merge pull request #206 from bcgov/fix/dont-crash-on-queued-dag-status
fix: handle 'queued' status in DAG trigger script, update Dockerfile …
2 parents e69621a + a384962 commit e249920

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

dag-trigger/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from alpine:3.14
1+
from alpine:3.21
22

33
RUN apk add --no-cache --upgrade bash jq curl
44

dag-trigger/airflow-dag-trigger.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ while true; do
8989
echo "DAG succeeded"
9090
exit 0
9191
;;
92-
'running' )
92+
'running' | 'queued' )
9393
echo '...waiting 10 seconds'
9494
sleep 10
9595
;;

helm/cas-airflow-dag-trigger/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ apiVersion: v2
22
name: cas-airflow-dag-trigger
33
description: A Helm chart for triggering remote DAGs on an existing airflow instance
44
type: application
5-
version: 1.0.20 # Changing this requires updating the image tag in .github/workflows/release.yaml
5+
version: 1.0.21 # Changing this requires updating the image tag in .github/workflows/release.yaml

0 commit comments

Comments
 (0)