Skip to content

Conversation

@pierrejeambrun
Copy link
Member

Follow up of #53024

As mentioned by @jscheffl we should only fail early logs retrieval for Skipped task if this is an actual skip tasks (didn't run, i.e try_number is 0). If the skip occurs from a skip return code or anything else, logs should be available.

@pierrejeambrun pierrejeambrun added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Jul 9, 2025
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix, it works!

@jscheffl jscheffl merged commit b052f84 into apache:main Jul 9, 2025
59 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 9, 2025
(cherry picked from commit b052f84)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
@github-actions
Copy link

github-actions bot commented Jul 9, 2025

Backport successfully created: v3-0-test

Status Branch Result
v3-0-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jul 9, 2025
(cherry picked from commit b052f84)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
kaxil pushed a commit that referenced this pull request Jul 9, 2025
(cherry picked from commit b052f84)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
kaxil pushed a commit that referenced this pull request Jul 9, 2025
(cherry picked from commit b052f84)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
HsiuChuanHsu pushed a commit to HsiuChuanHsu/airflow that referenced this pull request Jul 10, 2025
stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:logging backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants