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/03/23 15:44:35 UTC

[GitHub] [airflow] akki opened a new issue #14959: Support all terminus task states for Docker Swarm Operator

akki opened a new issue #14959:
URL: https://github.com/apache/airflow/issues/14959


   **Apache Airflow version**: latest
   
   **What happened**:
   
   There are more terminus task states than the ones we currently check in Docker Swarm Operator. This makes the operator run infinitely when the service goes into these states.
    
   **What you expected to happen**:
   
   The operator should terminate.
   
   **How to reproduce it**:
   
   Run a Airflow task via the Docker Swarm operator and return failed status code from it.
   
   
   **Anything else we need to know**:
   
   So as a fix I have added the complete list of tasks from the Docker reference (https://docs.docker.com/engine/swarm/how-swarm-mode-works/swarm-task-states/)
   We would like to send this patch back upstream to Apache Airflow.
   


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

[GitHub] [airflow] eladkal commented on issue #14959: Support all terminus task states for Docker Swarm Operator

Posted by GitBox <gi...@apache.org>.
eladkal commented on issue #14959:
URL: https://github.com/apache/airflow/issues/14959#issuecomment-811987762


   commented on the 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

[GitHub] [airflow] eladkal closed issue #14959: Support all terminus task states for Docker Swarm Operator

Posted by GitBox <gi...@apache.org>.
eladkal closed issue #14959:
URL: https://github.com/apache/airflow/issues/14959


   


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

[GitHub] [airflow] boring-cyborg[bot] commented on issue #14959: Support all terminus task states for Docker Swarm Operator

Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #14959:
URL: https://github.com/apache/airflow/issues/14959#issuecomment-805010542


   Thanks for opening your first issue here! Be sure to follow the issue template!
   


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

[GitHub] [airflow] akki commented on issue #14959: Support all terminus task states for Docker Swarm Operator

Posted by GitBox <gi...@apache.org>.
akki commented on issue #14959:
URL: https://github.com/apache/airflow/issues/14959#issuecomment-811978094


   Hi @eladkal 
   
   Can you please advise how I can get [the fix for this issue](https://github.com/apache/airflow/pull/14960) merged?


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