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 2020/10/06 08:39:54 UTC

[GitHub] [airflow] potiuk commented on a change in pull request #11207: Breeze start-airflow command wasn't able to initialize the db in 1.10.x

potiuk commented on a change in pull request #11207:
URL: https://github.com/apache/airflow/pull/11207#discussion_r500103610



##########
File path: scripts/in_container/check_environment.sh
##########
@@ -107,8 +107,12 @@ function startairflow_if_requested() {
         . "$( dirname "${BASH_SOURCE[0]}" )/configure_environment.sh"
 
         # initialize db and create the admin user if it's a new run
-        airflow db init
-        airflow users create -u admin -p admin -f Thor -l Adminstra -r Admin -e dummy@dummy.email
+        if [[ ${RUN_AIRFLOW_1_10} == "true" ]]; then
+            airflow initdb
+        else
+            airflow db init
+            airflow users create -u admin -p admin -f Thor -l Adminstra -r Admin -e dummy@dummy.email

Review comment:
       Ther won't be indeed. But we are still able to install 1.10 in the ./breeze from 2.0 (for convenience) via `--install-airflow-version` . So we should add this option just for that case. I am going to do it shortly.




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

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