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/08/02 08:46:39 UTC

[GitHub] [airflow] meryll opened a new issue, #25463: Unable to mark_tasks.set_state to SKIPPED

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

   ### Apache Airflow version
   
   2.3.3 (latest released)
   
   ### What happened
   
   Crash while trying to call set_state from module common.mark_tasks with state SKIPPED. 
   TaskInstanceState can be set to SKIPPED but while trying to set the same state to downstream dags the app crashes at DagRunState does not accept SKIPPED state. 
   
   ### What you think should happen instead
   
   Either:
   
   - allow DagRunState to be SKIPPED (more difficult)
   - while skipping tasks set downstream dags to state FAILED (easy)
   
   ### How to reproduce
   
   call common.mark_tasks.set_state with state SKIPPED
   
   ### Operating System
   
   macOs 12.5
   
   ### Versions of Apache Airflow Providers
   
   2.3.3
   
   ### Deployment
   
   Virtualenv installation
   
   ### Deployment details
   
   The error happens even when running airflow locally.
   
   ### 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] meryll commented on issue #25463: Unable to mark_tasks.set_state to SKIPPED

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

   Unfortunately we are calling this function from our custom airflow plugin. And currently, I am in the process of migrating it from airflow 2.2.3 to 2.3.3..
   Is there any other way to set airflow tasks to SKIPPED? 


-- 
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] boring-cyborg[bot] commented on issue #25463: Unable to mark_tasks.set_state to SKIPPED

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

   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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

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


[GitHub] [airflow] uranusjr closed issue #25463: Unable to mark_tasks.set_state to SKIPPED

Posted by GitBox <gi...@apache.org>.
uranusjr closed issue #25463: Unable to mark_tasks.set_state to SKIPPED
URL: https://github.com/apache/airflow/issues/25463


-- 
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] uranusjr commented on issue #25463: Unable to mark_tasks.set_state to SKIPPED

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

   The official way is to use the REST API. If you _must not_ send HTTP requests, [`set_state` on `TaskInstance`](https://airflow.apache.org/docs/apache-airflow/stable/_api/airflow/models/taskinstance/index.html#airflow.models.taskinstance.TaskInstance) is currently considered supported, although direct access to `airflow.models` is generally strongly discouraged.
   
   In the mean time, I’ll close this issue since we don’t support such use cases. Feel free to open a new issue if the REST API does not work as desired.


-- 
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] uranusjr commented on issue #25463: Unable to mark_tasks.set_state to SKIPPED

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

   That function is an internal helper that should not be called directly. Why are you doing it?


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