potiuk commented on PR #36611:
URL: https://github.com/apache/airflow/pull/36611#issuecomment-1878585046

   BTW. By a quick look it is quite possible it does not work.
   
   The `aiflow-init` container runs `airflow version` which I think COULD in 
the past initialize the DB as side effect but this could have changed since. 
It's quite possible that `airflow version` does not initialize the DB now 
(because if it did, then it was a side-effect).
   
   If that's the case then the right solution will be to change this command to 
one that actually initializes the DB rather than changing the documentation. 
Because it means that the `airflow-init` service is simply quite badly defined 
now. 
   
   And it seems that our tests do not actually run `airflow-init` - so the 
right fix here will be in this case to change our tests to run the init, then 
verify if the database have been created and only then run the whole compose 
setup.
   
   
   


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

Reply via email to