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 2022/07/26 08:39:44 UTC

[GitHub] [airflow] uranusjr commented on a diff in pull request #24156: Create new databases from the ORM

uranusjr commented on code in PR #24156:
URL: https://github.com/apache/airflow/pull/24156#discussion_r929687216


##########
airflow/utils/db.py:
##########
@@ -1487,6 +1513,12 @@ def upgradedb(
     if errors_seen:
         exit(1)
 
+    if not to_revision and not _get_current_revision(session=session):
+        # Don't load default connections
+        os.environ['AIRFLOW__DATABASE__LOAD_DEFAULT_CONNECTIONS'] = 'False'

Review Comment:
   I wonder if this should be an argument to initdb ot something. Setting an environment variable here feels heavy-handed.



-- 
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