This is an automated email from the ASF dual-hosted git repository.

shahar1 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new a8e1bb052db Speed up breeze start-airflow by skipping a redundant CLI 
call (#73174)
a8e1bb052db is described below

commit a8e1bb052db47802878a682be6ff768b2cae2334
Author: Andrew Chang <[email protected]>
AuthorDate: Wed Sep 16 13:26:52 2026 +0800

    Speed up breeze start-airflow by skipping a redundant CLI call (#73174)
    
    check_environment.sh only needs the auth manager class path to decide
    whether to create FAB users. breeze always exports
    AIRFLOW__CORE__AUTH_MANAGER, and env vars are first in Airflow's config
    lookup, so the CLI would print that same value after a full Airflow
    startup. Read the env var directly and keep the CLI as a fallback.
---
 scripts/in_container/check_environment.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/scripts/in_container/check_environment.sh 
b/scripts/in_container/check_environment.sh
index e433e4b5bb4..0172bc94c7e 100755
--- a/scripts/in_container/check_environment.sh
+++ b/scripts/in_container/check_environment.sh
@@ -119,7 +119,11 @@ function startairflow_if_requested() {
             
AIRFLOW__DATABASE__LOAD_DEFAULT_CONNECTIONS=${LOAD_DEFAULT_CONNECTIONS} airflow 
db init
         fi
 
-        if airflow config get-value core auth_manager | grep -q 
"FabAuthManager"; then
+        local auth_manager="${AIRFLOW__CORE__AUTH_MANAGER:-}"
+        if [[ -z "${auth_manager}" ]]; then
+            auth_manager=$(airflow config get-value core auth_manager || true)
+        fi
+        if [[ "${auth_manager}" == *FabAuthManager* ]]; then
             airflow users create -u admin -p admin -f Thor -l Adminstra -r 
Admin -e [email protected] || true
 
             # Create all roles for testing if CREATE_ALL_ROLES is set

Reply via email to