Skip to content

Make default as time.now() for TriggerDagRunOperator#48969

Merged
kaxil merged 6 commits into
apache:mainfrom
astronomer:triggerdagrunoperator-defaults-time-now
Apr 9, 2025
Merged

Make default as time.now() for TriggerDagRunOperator#48969
kaxil merged 6 commits into
apache:mainfrom
astronomer:triggerdagrunoperator-defaults-time-now

Conversation

@vatsrahul1001

Copy link
Copy Markdown
Contributor

closes: #48412
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@amoghrajesh amoghrajesh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sounds good but one comment

Comment thread providers/standard/src/airflow/providers/standard/operators/trigger_dagrun.py Outdated
@phanikumv
phanikumv requested a review from uranusjr April 9, 2025 08:11
Comment thread providers/standard/src/airflow/providers/standard/operators/trigger_dagrun.py Outdated
@kaxil
kaxil merged commit 57cda9e into apache:main Apr 9, 2025
@kaxil
kaxil deleted the triggerdagrunoperator-defaults-time-now branch April 9, 2025 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Manually triggered dags should have default logical_date and run_after of "right now"

4 participants