jedcunningham commented on a change in pull request #18974:
URL: https://github.com/apache/airflow/pull/18974#discussion_r771079145
##########
File path: chart/values.schema.json
##########
@@ -711,6 +711,59 @@
}
]
},
+ "enableBuiltInSecretEnvVars": {
+ "description": "Uses built-in secret values set as environment
variables passed to Airflow. You should supply corresponding environment
variables as ``extraEnv`` variables if you disable them here.",
+ "type": "object",
+ "additionalProperties": false,
+ "x-docsSection": "Airflow",
+ "properties": {
+ "AIRFLOW__CORE__FERNET_KEY": {
+ "description": "Enable ``AIRFLOW__CORE__FERNET_KEY``
variable to be read from the Fernet key Secret (either derived from release
name ``Values.Release.Name``-fernet-key or from ``.Values.fernetKeySecretName``
if specified)",
+ "type": "boolean",
+ "default": true
+ },
+ "AIRFLOW__CORE__SQL_ALCHEMY_CONN": {
+ "description": "Enable ``AIRFLOW__CORE__SQL_ALCHEMY_CONN``
variable to be read from the Metadata Secret (from
``.Values.data.metadataSecretName``)",
+ "type": "boolean",
+ "default": true
+ },
+ "AIRFLOW_CONN_AIRFLOW_DB": {
+ "description": "Enable ``AIRFLOW_CONN_AIRFLOW_DB``
variable to be read from the Metadata Secret (from
``.Values.data.metadataSecretName``)",
+ "type": "boolean",
+ "default": true
+ },
+ "AIRFLOW__WEBSERVER__SECRET_KEY": {
+ "description": "Enable ``AIRFLOW__WEBSERVER__SECRET_KEY``
variable to be read from the Webserver Secret Key Secret (from
``.Values.webserverSecretKeySecretName``)",
+ "type": "boolean",
+ "default": true
+ },
+ "AIRFLOW__CELERY__CELERY_RESULT_BACKEND": {
+ "description": "Enable
``AIRFLOW__CELERY__CELERY_RESULT_BACKEND`` variable to be read from the Celery
Result Backend Secret (from ``.Values.data.resultBackendSecretName``) - Airflow
<1.10.0 variant",
+ "type": "boolean",
+ "default": true
+ },
+ "AIRFLOW__CELERY__RESULT_BACKEND": {
+ "description": "Enable ``AIRFLOW__CELERY__RESULT_BACKEND``
variable to be read from the Celery Result Backend Secret (from
``.Values.data.resultBackendSecretName``)",
+ "type": "boolean",
+ "default": true
+ },
+ "AIRFLOW__CELERY__BROKER_URL": {
+ "description": "Enable ``AIRFLOW__CELERY__BROKER_URL``
variable to be read from the Celery Result Backend Secret (from
``.Values.data.brokerUrlSecretName``)",
Review comment:
```suggestion
"description": "Enable ``AIRFLOW__CELERY__BROKER_URL``
variable to be read from the Celery Broker URL Secret (from
``.Values.data.brokerUrlSecretName``)",
```
##########
File path: chart/values.schema.json
##########
@@ -711,6 +711,59 @@
}
]
},
+ "enableBuiltInSecretEnvVars": {
+ "description": "Uses built-in secret values set as environment
variables passed to Airflow. You should supply corresponding environment
variables as ``extraEnv`` variables if you disable them here.",
+ "type": "object",
+ "additionalProperties": false,
+ "x-docsSection": "Airflow",
+ "properties": {
+ "AIRFLOW__CORE__FERNET_KEY": {
+ "description": "Enable ``AIRFLOW__CORE__FERNET_KEY``
variable to be read from the Fernet key Secret (either derived from release
name ``Values.Release.Name``-fernet-key or from ``.Values.fernetKeySecretName``
if specified)",
+ "type": "boolean",
+ "default": true
+ },
+ "AIRFLOW__CORE__SQL_ALCHEMY_CONN": {
+ "description": "Enable ``AIRFLOW__CORE__SQL_ALCHEMY_CONN``
variable to be read from the Metadata Secret (from
``.Values.data.metadataSecretName``)",
Review comment:
The rest of these also have similar behavior like fernet key - it
defaults to the release name + a fixed string, or uses the secret name provided
in `data` instead.
--
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]
