potiuk commented on pull request #18336:
URL: https://github.com/apache/airflow/pull/18336#issuecomment-922129873
I think relying on empty value is a hack.
Better solution would be to disable setting the values in helm chart .
Something like that should do (writing without checking so might need some
tweaking):
values.yaml:
```
data:
# If secret names are provided, use those secrets
useMetadataSecret: True
```
_helpers.yaml:
```
{{- if .Values.data.useMetadataSecret }}
- name: AIRFLOW__CORE__SQL_ALCHEMY_CONN
valueFrom:
secretKeyRef:
name: {{ template "airflow_metadata_secret" . }}
key: connection
- name: AIRFLOW_CONN_AIRFLOW_DB
valueFrom:
secretKeyRef:
name: {{ template "airflow_metadata_secret" . }}
key: connection
{{- end }}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
