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/01/10 20:05:23 UTC

[GitHub] [airflow] xinbinhuang edited a comment on pull request #13526: Make helm chart commands compatible with 1.10.14 image

xinbinhuang edited a comment on pull request #13526:
URL: https://github.com/apache/airflow/pull/13526#issuecomment-757535276


   > @xinbinhuang i am testing this with 1.10.14 and seeing bad termination behavior of celery workers. documented here: #13591
   > 
   > perhaps you have some insight?
   
   Now I am taking a second look at it. Actually, I don't think this PR is doing anything, because `["airflow", "scheduler"]` is essentially the same as `["bash", "-c", "exec airflow scheduler"]`.
   As for `["bash", "-c", "airflow webserver"]` is working, it's probably due to the fact that Docker does not signal the process properly which leaves it hanging, making it feels like a "graceful termination".
   
   I will comment further on the original issue


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