kacpermuda opened a new pull request, #75458:
URL: https://github.com/apache/airflow/pull/75458

   <!-- SPDX-License-Identifier: Apache-2.0
         https://www.apache.org/licenses/LICENSE-2.0 -->
   
   The log link of a task instance in a task loop doesn't lead to that 
iteration:
   
   - `TaskInstance.log_url` puts `region_index` where the UI expects a map 
index. For a task in a loop that isn't mapped, `region_index` is the iteration, 
so the link is `.../tasks/<task>/mapped/<iteration>`, a page that doesn't exist.
   - `RuntimeTaskInstance.log_url` uses only the public map index, so every 
iteration of a loop gets the same link.
   
   Both now build the link the UI builds itself (`getTaskInstanceLink` in 
`ui/src/utils/links.ts`): `/mapped/<map index>` only for a mapped task, plus 
`region_id` and `region_index` query parameters for a task instance in a 
dynamic region. Links of task instances outside a region don't change. On the 
stored task instance, telling a mapped task from a loop iteration needs its 
task; without it the link has the region parameters only, which the UI resolves 
the same way.
   
   These links reach users through failure emails, notifiers and OpenLineage 
events.
   
   related: #74752
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (Claude Code)
   
   Generated-by: Claude Code (Opus 5.5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   ---
   
   * Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
 for more information. Note: commit author/co-author name and email in commits 
become permanently public when merged.
   * For fundamental code changes, an Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
   * When adding dependency, check compliance with the [ASF 3rd Party License 
Policy](https://www.apache.org/legal/resolved.html#category-x).
   * For significant user-facing changes create newsfragment: 
`{pr_number}.significant.rst`, in 
[airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
 You can add this file in a follow-up commit after the PR is created so you 
know the PR number.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to