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/12/07 07:40:35 UTC

[GitHub] [airflow] ephraimbuddy edited a comment on pull request #18439: Check and run migration in commands if necessary

ephraimbuddy edited a comment on pull request #18439:
URL: https://github.com/apache/airflow/pull/18439#issuecomment-987643394


   Because we use init containers to run the migration, this change doesn't get hit when using the airflow chart. Only in a chart without a migration container will it get hit. 
   
   If we don't raise TimeoutError in the init containers, then this change is hit. However, it just stops for a few minutes to show the error message and the component will continue running.
   
   ~I think the migration container should not exit on error, we should have a way to restart it when it fails.~
   


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