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/30 16:11:58 UTC

[GitHub] [airflow] trowdan commented on pull request #13852: check failed service before auto_remove it

trowdan commented on pull request #13852:
URL: https://github.com/apache/airflow/pull/13852#issuecomment-770235111


   > Can you also please add description @trowdan - with context - in the commit message. It's great to refer an inssue, but the commit messages have to be descriptive enough and provide enough context so that when you do `git log` you know what the change is about and why it was implmented. You can run `git log` and see examples of good descriptive messages there.
   
   Sure Jarek,
   I've squashed the commits and added a description. Let me now if it's ok now.
   Note that this fixes the https://github.com/apache/airflow/issues/13532 for DockerSwarmOperator, that is affected by the same issue. I'll try to take a look into DockerOperator and eventually open a new PR.


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