Skip to content

Run triggers inline with dag test - #34642

Merged
dstandish merged 3 commits into
apache:mainfrom
astronomer:run-triggers-inline-with-dag-test
Nov 27, 2023
Merged

Run triggers inline with dag test#34642
dstandish merged 3 commits into
apache:mainfrom
astronomer:run-triggers-inline-with-dag-test

Conversation

@dstandish

Copy link
Copy Markdown
Contributor

Avoids having to spin up the triggerer component, potentially nice debugging experience.

@dstandish
dstandishforce-pushed the run-triggers-inline-with-dag-test branch 2 times, most recently from 6a7d0be to 20ea487CompareSeptember 27, 2023 18:10
@dstandish
dstandish marked this pull request as ready for review September 27, 2023 18:11
@dstandish
dstandishforce-pushed the run-triggers-inline-with-dag-test branch 2 times, most recently from 321421a to 259f80fCompareNovember 9, 2023 06:22
No need to have trigger running -- will just run them async.
@dstandish
dstandishforce-pushed the run-triggers-inline-with-dag-test branch from 259f80f to c8b4a5eCompareNovember 13, 2023 17:21
Comment threadairflow/models/dag.py
@dstandish
dstandish requested a review from kaxilNovember 14, 2023 22:27
@dstandish
dstandish merged commit 7b37a78 into apache:mainNov 27, 2023
@dstandish
dstandish deleted the run-triggers-inline-with-dag-test branch November 27, 2023 14:48
@eladkaleladkal added this to the Airflow 2.8.0 milestone Nov 27, 2023
@eladkaleladkal added the type:bug-fix Changelog: Bug Fixes label Nov 27, 2023
ephraimbuddy pushed a commit that referenced this pull request Dec 5, 2023
No need to have trigger running -- will just run them async.
(cherry picked from commit 7b37a78)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:CLItype:bug-fixChangelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@dstandish@kaxil@eladkal@jedcunningham