Skip to content

Move [scheduler]dag_dir_list_interval to [dag_bundles]refresh_interval - #45722

Merged
ephraimbuddy merged 1 commit into
apache:mainfrom
astronomer:move_dag_dir_refresh_interval
Jan 17, 2025
Merged

ephraimbuddy merged 1 commit into
apache:mainfrom
astronomer:move_dag_dir_refresh_interval

Conversation

@jedcunningham

Copy link
Copy Markdown
Member

This config should be the default refresh_interval in AF3, so move its predecessor.

…rval`

This config should be the default refresh_interval in AF3, so move its
predecesor.
@jedcunningham jedcunningham added the full tests needed We need to run full set of tests for this PR to merge label Jan 16, 2025
@jedcunningham jedcunningham reopened this Jan 16, 2025
@ephraimbuddy
ephraimbuddy merged commit 6cde25f into apache:main Jan 17, 2025
@ephraimbuddy
ephraimbuddy deleted the move_dag_dir_refresh_interval branch January 17, 2025 08:38
@Lee-W

Lee-W commented Jan 17, 2025 •

Copy link
Copy Markdown
Member

I think we'll need to mark it as a breaking change and add a migration rule 🤔 I'll create PR for it

@Lee-W

Lee-W commented Jan 17, 2025

Copy link
Copy Markdown
Member

PR #45737 created

dauinh pushed a commit to dauinh/airflow that referenced this pull request Jan 24, 2025
…rval` (apache#45722)

This config should be the default refresh_interval in AF3, so move its
predecesor.
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
…rval` (apache#45722)

This config should be the default refresh_interval in AF3, so move its
predecesor.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dag-bundles AIP-66 area:scheduler full tests needed We need to run full set of tests for this PR to merge kind:documentation

Projects

Development

Successfully merging this pull request may close these issues.

3 participants