Skip to content

Commit b24e2c6

Browse files
authored
Merge 6980120 into bf5d35d
2 parents bf5d35d + 6980120 commit b24e2c6

File tree

15 files changed

+37
-25
lines changed

15 files changed

+37
-25
lines changed

.github/workflows/release.yaml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ on:
44
push:
55
branches:
66
- main
7+
pull_request:
8+
branches: [develop, main]
9+
710

811
jobs:
912
release:
@@ -52,7 +55,7 @@ jobs:
5255
context: .
5356
dockerfile_path: Dockerfile
5457
github_token: ${{ secrets.GITHUB_TOKEN }}
55-
chart_version: 2.2.1
58+
chart_version: 3.0.0
5659
is_chart_release: true
5760
cas-airflow-dag-trigger-build:
5861
needs: [release]
@@ -65,5 +68,5 @@ jobs:
6568
context: dag-trigger
6669
dockerfile_path: dag-trigger/Dockerfile
6770
github_token: ${{ secrets.GITHUB_TOKEN }}
68-
chart_version: 1.2.1
71+
chart_version: 1.2.2
6972
is_chart_release: true

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM apache/airflow:2.10.5
1+
FROM apache/airflow:3.0.6
22

33
# Install the standard providers for Airflow 2 -- can be removed when we upgrade to Airflow 3
44
RUN pip install --no-cache-dir "apache-airflow==${AIRFLOW_VERSION}" apache-airflow-providers-standard

dag-trigger/airflow-dag-trigger.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ run_json=$(_curl "${auth_params[@]}" -X POST \
100100
"$dag_run_url" \
101101
-H 'Cache-Control: no-cache' \
102102
-H 'Content-Type: application/json' \
103-
-d "{\"conf\": $dag_config}")
103+
-d "{\"logical_date\": null, \"conf\": $dag_config}")
104104
dag_run_id=$(echo "$run_json" | jq -r .dag_run_id)
105105

106106
echo "Started dag run ID: $dag_run_id"

deploy.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ helm upgrade --install \
1818
helm upgrade --install --timeout 900s \
1919
--namespace "$AIRFLOW_NAMESPACE_PREFIX-$ENVIRONMENT" \
2020
-f ./helm/cas-airflow/values.yaml \
21-
--set-file airflow.webserver.webserverConfig=./helm/cas-airflow/webserver_config.py \
21+
--set-file airflow.apiServer.apiServerConfig=./helm/cas-airflow/webserver_config.py \
2222
-f "./helm/cas-airflow/values-$ENVIRONMENT.yaml" \
2323
--set namespaces.airflow="$AIRFLOW_NAMESPACE_PREFIX-$ENVIRONMENT" \
2424
--set namespaces.ggircs="$GGIRCS_NAMESPACE_PREFIX-$ENVIRONMENT" \

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.2.1 # Changing this requires updating the image tag in .github/workflows/release.yaml
5+
version: 1.2.2 # Changing this requires updating the image tag in .github/workflows/release.yaml

helm/cas-airflow-postgres-cluster/values.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,11 @@ postgresCluster:
2121
options: "SUPERUSER"
2222
databases:
2323
- postgres
24+
password:
25+
type: AlphaNumeric
2426
- name: airflow
27+
password:
28+
type: AlphaNumeric
2529
databases:
2630
- postgres
2731

helm/cas-airflow/Chart.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
apiVersion: v2
22
name: cas-airflow
33
type: application
4-
version: 2.2.1 # Changing this requires updating the image tag in .github/workflows/release.yaml
5-
appVersion: 2.10.5 # The airflow version
4+
version: 3.0.0 # Changing this requires updating the image tag in .github/workflows/release.yaml
5+
appVersion: 3.0.6 # The airflow version
66
description: Helm chart to deploy cas' flavour of airflow, compatible with OpenShift 4. This chart uses the vanilla airflow chart and adds cas' own templates and values.
77
icon: https://www.astronomer.io/static/airflowNewA.png
88
keywords:

helm/cas-airflow/templates/routes/route.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ spec:
88
host: {{ .Release.Name }}-{{ include "cas-airflow.namespaceSuffix" . }}.{{ .Values.route.clusterDomain }}
99
to:
1010
kind: Service
11-
name: {{ .Release.Name }}-webserver
11+
name: {{ .Release.Name }}-api-server
1212
weight: 100
1313
port:
14-
targetPort: airflow-ui
14+
targetPort: api-server
1515
tls:
1616
termination: edge
1717
insecureEdgeTerminationPolicy: Allow

helm/cas-airflow/templates/secrets/webserver.yaml renamed to helm/cas-airflow/templates/secrets/api.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
{{- $webserverSecret := (randAlphaNum 32) | b64enc | quote }}
1+
{{- $apiSecret := (randAlphaNum 32) | b64enc | quote }}
22

3-
{{- $secretName := printf "%s-%s" .Release.Name "webserver-secret" }}
3+
{{- $secretName := printf "%s-%s" .Release.Name "api-secret" }}
44
{{- $secret := (lookup "v1" "Secret" .Release.Namespace $secretName ) }}
55
{{- if $secret }}
6-
{{- $webserverSecret = index $secret.data "webserver-secret-key" }}
6+
{{- $apiSecret = index $secret.data "api-secret-key" }}
77
{{- end -}}
88

99

@@ -16,4 +16,4 @@ metadata:
1616
"helm.sh/hook": "pre-install,pre-upgrade"
1717
type: Opaque
1818
data:
19-
webserver-secret-key: {{ $webserverSecret }}
19+
api-secret-key: {{ $apiSecret }}

helm/cas-airflow/values-dev.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ airflow:
2626
persistentVolumeClaim:
2727
claimName: cas-airflow-dynamic-dags-pvc
2828

29-
webserver:
29+
apiServer:
3030
replicas: 2
3131
extraVolumeMounts:
3232
- name: gcs-credentials

0 commit comments

Comments
 (0)