You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/11/19 15:54:33 UTC

[GitHub] [airflow] dstandish edited a comment on issue #19688: Airflow does not load default connections

dstandish edited a comment on issue #19688:
URL: https://github.com/apache/airflow/issues/19688#issuecomment-974190683


   @potiuk hey i have an idea.
   
   We can create a `demo` section in config.
   
   Then:
   
   ```
   AIRFLOW__DEMO__LOAD_DEFAULT_CONNECTIONS
   AIRFLOW__DEMO__LOAD_EXAMPLE_DAGS
   ```
   
   And then maybe users would be more accepting of `demo` settings being disabled for certain scenarios.
   
   As it is, from user perspective, maybe since it's a "core" setting, there is the expectation that it should always work.
   
   But there is a negative to this idea because i'll bet sometimes people like example dags in the helm chart perhaps for testing or "demoing" reasons.
   
   On the other hand maybe by collecting these under a demo section, we could more cleanly enable / disable even in helm chart.
   
   It is maybe a bit odd to "elevate" these kinds of settings to the `core` section.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org