Skip to content
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

List Dag Run broken link to dag_run #40214

Closed
1 of 2 tasks
raphaelauv opened this issue Jun 13, 2024 · 8 comments
Closed
1 of 2 tasks

List Dag Run broken link to dag_run #40214

raphaelauv opened this issue Jun 13, 2024 · 8 comments
Assignees
Labels
affected_version:2.9 area:core area:UI Related to UI/UX. For Frontend Developers. good first issue kind:bug This is a clearly a bug

Comments

@raphaelauv
Copy link
Contributor

raphaelauv commented Jun 13, 2024

Apache Airflow version

2.9.2

What happened?

if I click on a dag_run on the list dag run webserver page

it always show the latest dag_run and not the dag_run I clicked on

How to reproduce

from airflow.operators.empty import EmptyOperator
from pendulum import today
from airflow import DAG

with DAG(
        dag_id='example',
        schedule_interval='0 0 * * *',
        start_date=today("UTC").add(days=-5)):
    EmptyOperator(task_id="nothing")
Screencast.from.13-06-2024.11_55_14.mp4

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@raphaelauv raphaelauv added area:core kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet labels Jun 13, 2024
@vatsrahul1001
Copy link
Collaborator

@raphaelauv I tried this and not able to reproduce this. Are you facing this for all of your DAG's?

@raphaelauv
Copy link
Contributor Author

raphaelauv commented Jun 18, 2024

yes in google chrome and firefox

from the page http://localhost:8090/dagrun/list/?_flt_3_dag_id=example

you must click on the link of the run_id column ( like in the screen record )

@vatsrahul1001
Copy link
Collaborator

@raphaelauv I am able to repo thus

@vatsrahul1001 vatsrahul1001 added area:UI Related to UI/UX. For Frontend Developers. good first issue and removed needs-triage label for new issues that we didn't triage yet pending-response labels Jun 18, 2024
@ayush3singh
Copy link

Hi, I would like to work on this issue, I have done some initial debugging. @potiuk can you please assign it to me.

@potiuk
Copy link
Member

potiuk commented Jun 21, 2024

Sure

@laoni3634
Copy link

When click on a dag_run in dag_run list page, the link is like below one. This link does not exist, so rediect to the latest one.
https://localhost/pipelines/dags/transition_execute/graph?dag_run_id=dag_run_id
When click on a grid in dag_detail page, the link is like below one. This link is correct.
https://localhost/pipelines/dags/transition_execute/grid?dag_run_id=dag_run_id

So I think the brief change is to change https://localhost/pipelines/dags/transition_execute/graph?dag_run_id=dag_run_id to
https://localhost/pipelines/dags/transition_execute/grid?dag_run_id=dag_run_id (just replace graph with grid) when click on a dag_run in dag_run list page.

@romsharon98
Copy link
Contributor

romsharon98 commented Aug 2, 2024

Couldn't reproduce it on the main branch.
It's already has fixed in 2.9.3 by @bbovenzi here #40241

@raphaelauv
Copy link
Contributor Author

@romsharon98 the issue was open against version 2.9.2

and it was fix by #40241 in version 2.9.3 ( that is part of the main branch )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected_version:2.9 area:core area:UI Related to UI/UX. For Frontend Developers. good first issue kind:bug This is a clearly a bug
Projects
None yet
Development

No branches or pull requests

7 participants