Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for dagrun_failed in REST API #40600

Closed
1 of 2 tasks
reganbaum opened this issue Jul 4, 2024 · 1 comment
Closed
1 of 2 tasks

Add support for dagrun_failed in REST API #40600

reganbaum opened this issue Jul 4, 2024 · 1 comment
Labels
area:API Airflow's REST/HTTP API kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet

Comments

@reganbaum
Copy link

Description

Today, stopping the execution of a DAG run requires 2 REST API calls: one to identify non-completed tasks of a run, and another to mark those tasks as failed. This has the potential of running into race conditions, where tasks could complete between the two API calls, leading to tasks being marked as failed when they may have succeeded. Ideally, there is a single REST API that handles stopping the execution of a DAG run. The UI does this today by invoking the endpoint /dagrun_failed. The goal is to have a REST API endpoint available that stops a DAG run at /api/v1/dagrun_failed.

Use case/motivation

I would like to programmatically stop a running workflow, which actually stops the execution (not just changes the state in the database).

Related issues

#40594 (The dagrun_failed issue was originally combined with the dag_stats issue, but separated it out into its own issue for clearer tracking)

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@reganbaum reganbaum added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Jul 4, 2024
@dosubot dosubot bot added the area:API Airflow's REST/HTTP API label Jul 4, 2024
@potiuk
Copy link
Member

potiuk commented Jul 4, 2024

As explained in #40594 It's a wish but if you want to have your wish to materialize, you need to explain and participate in the discussions on improving airflow architecture happening here: https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-72+Task+Execution+Interface+aka+Task+SDK - and propose what kind of API and technology you would like to use there and why it is important. Until then it's not a "featiure" request, it's a discussion, because there is no clear way how this would work.

Converting it to a discussion.

@apache apache locked and limited conversation to collaborators Jul 4, 2024
@potiuk potiuk converted this issue into discussion #40602 Jul 4, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
area:API Airflow's REST/HTTP API kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet
Projects
None yet
Development

No branches or pull requests

2 participants