fgalind1 opened a new pull request, #31066:
URL: https://github.com/apache/airflow/pull/31066
Some resources use `include "airflow.fullname" .` helper and some just use
`{{ .Release.Name }}`. fullname helper is a common good practice to name all
the different helm resources.
This allows to customize the resources and leverage the existing options
`fullnameOverride` or `nameOverride` and have consistency in all resources to
always use `include "airflow.fullname" .`
This makes it much easier when deploying airflow as a dependency on another
chart or in the same namespace, as we can clearly see airflow's chart name
"airflow" in the resources and/or allow to customize the prefix for all airflow
resources
The proposed helm chart was deployed in my local environment and all names
match properly between deployments, secrets, configmaps, etc.
--
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]
