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 2022/07/14 14:12:48 UTC

[GitHub] [airflow] frankcash opened a new issue, #25062: Task Group States defined for the UI aren't in the same order as the back-end python code

frankcash opened a new issue, #25062:
URL: https://github.com/apache/airflow/issues/25062

   ### Apache Airflow version
   
   2.3.0
   
   ### What happened
   
   The priority arrays defined in the following blocks of code do not set the same precedent:
   - https://github.com/apache/airflow/blob/main/airflow/www/static/js/graph.js#L601-L605
   - https://github.com/apache/airflow/blob/main/airflow/www/utils.py#L79-L95 
   
   Causing a potential mismatch when defining the state of the group
   
   ### What you think should happen instead
   
   _No response_
   
   ### How to reproduce
   
   _No response_
   
   ### Operating System
   
   Debian GNU/Linux 11 (bullseye)
   
   ### Versions of Apache Airflow Providers
   
   _No response_
   
   ### Deployment
   
   Astronomer
   
   ### Deployment details
   
   _No response_
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] pierrejeambrun commented on issue #25062: Task Group States defined for the UI aren't in the same order as the back-end python code

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

   Hello,
   
   For me this has been fixed in:
   https://github.com/apache/airflow/pull/26789
   
   Closing for now. Feel free to reopen if needed :)


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

[GitHub] [airflow] bbovenzi commented on issue #25062: Task Group States defined for the UI aren't in the same order as the back-end python code

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

   Yeah, we should remove the priority list in `graph.js` and send the value from utils to the graph [here](https://github.com/apache/airflow/blob/main/airflow/www/views.py#L2968)
   


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

[GitHub] [airflow] pierrejeambrun closed issue #25062: Task Group States defined for the UI aren't in the same order as the back-end python code

Posted by GitBox <gi...@apache.org>.
pierrejeambrun closed issue #25062: Task Group States defined for the UI aren't in the same order as the back-end python code
URL: https://github.com/apache/airflow/issues/25062


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