Incorrect link to task detail from task instance list

[Pad71]

Apache Airflow version

2.8.2

If "Other Airflow 2 version" selected, which one?

No response

What happened?

Clicking the filter button on a task in the task instance list (see picture) will use an incomplete link to the task instance detail (in the grid view)

wrong incomplete link (an example):
https://myairflow.net/dags/DW_TST1_EXTRACT_CONTROL/grid?root=CHK_ADS_D410_FILES&execution_date=2024-02-28T18:00:00+00:00&tab=graph&dag_run_id=scheduled__2024-02-28T18:00:00+00:00

The correct link should add "&task_id=task_instance_id" (ie. &task_id=CHK_ADS_D410_FILES)) to the link's end:
https://myairflow.net/dags/DW_TST1_EXTRACT_CONTROL/grid?root=CHK_ADS_D410_FILES&execution_date=2024-02-28T18:00:00+00:00&tab=graph&dag_run_id=scheduled__2024-02-28T18:00:00+00:00&task_id=CHK_ADS_D410_FILES

What you think should happen instead?

The correct Link should point to the task detail, not just filter it and stay on the dag run detail

How to reproduce

Click the "filter" link in the task instance list

Operating System

Kubernetes on Unix platform

Versions of Apache Airflow Providers

No response

Deployment

Official Apache Airflow Helm Chart

Deployment details

No response

Anything else?

No response

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[boring-cyborg]

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

[eladkal]

cc @bbovenzi

[Pad71]

Hi. @bbovenzi I looked at the code in the commit and it seems to me that the "root" part of the link is omitted in the url_root.

Old one:
url_root = url_for(
"Airflow.graph", dag_id=dag_id, root=task_id, execution_date=execution_date.isoformat()
)

New one:
url_root = url_for(
"Airflow.grid",
dag_id=dag_id,
task_id=task_id,
dag_run_id=run_id,
tab="graph",
map_index=attr.get("map_index", -1),
)

There is no root in the new one. I think, the "root" filtering (very helpful in large Dags) should remain, only "task_id" should be added.