vincbeck commented on code in PR #32604:
URL: https://github.com/apache/airflow/pull/32604#discussion_r1269488082
##########
airflow/configuration.py:
##########
@@ -151,15 +151,31 @@ def _default_config_file_path(file_name: str) -> str:
return os.path.join(templates_dir, file_name)
-def retrieve_configuration_description() -> dict[str, dict[str, Any]]:
+def retrieve_configuration_description(
+ include_airflow: bool = True,
+ include_providers: bool = True,
+ selected_provider: str | None = None,
+) -> dict[str, dict[str, Any]]:
"""
Read Airflow configuration description from YAML file.
+ :param include_airflow: Include Airflow configs
+ :param include_providers: Include provider configs
+ :param selected_provider: If specified, include selected provider only
+ :param config_file_name: name of the file in "config_templates" directory
to read default config from
Review Comment:
```suggestion
```
##########
airflow/providers/celery/provider.yaml:
##########
@@ -60,3 +60,256 @@ sensors:
executors:
- airflow.providers.celery.executors.celery_executor.CeleryExecutor
-
airflow.providers.celery.executors.celery_kubernetes_executor.CeleryKubernetesExecutor
+
+config:
+ celery_kubernetes_executor:
+ description: |
+ This section only applies if you are using the
``CeleryKubernetesExecutor`` in
+ ``[core]`` section above
Review Comment:
Is "section above" still correct? It is now in a different config right?
Same comment for all occurrences of "section above"
--
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]
