Skip to content

[v3-2-test] Skip test_schedule_tis_start_trigger pending #55068 backport decision#66309

Merged
potiuk merged 1 commit intoapache:v3-2-testfrom
potiuk:fix-66307-skip-start-trigger-test-v3-2
May 3, 2026
Merged

[v3-2-test] Skip test_schedule_tis_start_trigger pending #55068 backport decision#66309
potiuk merged 1 commit intoapache:v3-2-testfrom
potiuk:fix-66307-skip-start-trigger-test-v3-2

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented May 3, 2026

The test_schedule_tis_start_trigger test fails on v3-2-test because
the start_from_trigger deferred-at-schedule code path in
DagRun.schedule_tis is commented out as a TODO on this branch.

That path was re-enabled on main by #55068 (merged 2026-03-25), one
day after v3-2-test was cut. The PR is a 19-file feature change
that's too large to backport blindly into a release branch, so the
right call here is the conservative one: skip the test, link a
tracking issue at both the skip-reason and the disabled production-code
site, and leave the underlying decision (option 1: backport #55068,
option 2: formally drop start_from_trigger on 3.2) to the issue.

closes: nothing yet — see #66307 for the open decision

Failing CI run that surfaced this:
https://github.com/apache/airflow/actions/runs/25270258994/job/74097048671


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.7)

Generated-by: Claude Code (Opus 4.7) following the guidelines

…backport decision

The `start_from_trigger` deferred-at-schedule path is commented out
on v3-2-test (disabled in 91e1029 as a TODO) and was only
re-enabled on main by apache#55068, which landed one day after the v3-2
branch was cut. The test asserts the feature works and fails on
v3-2-test CI.

Skip the test on the 3.2 line and link the tracking issue from both
the test skip-reason and the disabled production-code site so the
follow-up isn't lost. Decision (backport apache#55068 or formally drop
the feature on 3.2) is tracked in apache#66307.
@potiuk potiuk requested review from XD-DENG and ashb as code owners May 3, 2026 15:23
@potiuk potiuk merged commit ee5e527 into apache:v3-2-test May 3, 2026
7 checks passed
@potiuk potiuk deleted the fix-66307-skip-start-trigger-test-v3-2 branch May 3, 2026 15:23
@potiuk potiuk restored the fix-66307-skip-start-trigger-test-v3-2 branch May 3, 2026 17:11
@potiuk potiuk deleted the fix-66307-skip-start-trigger-test-v3-2 branch May 3, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant