Skip to content

Commit

Permalink
fix cron template render issue (#1944)
Browse files Browse the repository at this point in the history
* fixes cron template

* fix cron template unit test
  • Loading branch information
pgvishnuram committed Jul 28, 2023
1 parent d0652c8 commit 0116766
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/astronomer/templates/houston/houston-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -362,7 +362,7 @@ data:
This template is passed to the airflow chart as a string where the template is interpreted in that
context, including the airflow-chart's .Release.Name.
*/}}
schedule: "'{{"{{"}}- add 3 (regexFind \".$\" (adler32sum .Release.Name)) -}}-59/15 * * * *'"
schedule: {{`'{{- add 3 (regexFind ".$" (adler32sum .Release.Name)) -}}-59/15 * * * *'`}}
command:
- airflow-cleanup-pods
- --namespace={{`{{ .Release.Namespace }}`}}
Expand Down
2 changes: 1 addition & 1 deletion tests/chart_tests/test_houston_configmap.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ def common_test_cases(docs):

assert (
prod["deployments"]["helm"]["airflow"]["cleanup"]["schedule"]
== """'{{- add 3 (regexFind ".$" (adler32sum .Release.Name)) -}}-59/15 * * * *'"""
== '{{- add 3 (regexFind ".$" (adler32sum .Release.Name)) -}}-59/15 * * * *'
)

# validate yaml-embedded python
Expand Down

0 comments on commit 0116766

Please sign in to comment.