github-actions[bot] opened a new pull request, #67097: URL: https://github.com/apache/airflow/pull/67097
* Fix max_active_runs lost during DAG serialisation when value equals schema default The serialisation optimisation from #55849 strips DAG fields that match their schema.json default. For max_active_runs, max_active_tasks, and max_consecutive_failed_dag_runs this is wrong because their runtime defaults come from airflow.cfg, not the schema. When a user explicitly sets max_active_runs=16 and the config has max_active_runs_per_dag=1, the value gets stripped and the dag table ends up with 1. Skip the schema-default exclusion for these three config-driven fields so they always survive serialisation. * fix: introduce DAG_DEFAULTS * chore: add test for catchup * fix: revert comment change, that wording was better * fix: preserve DAG fields whose defaults match the schema default * fix: update schema default check script * fix: address review comments on serialisation tests * fix: add disable_bundle_versioning case to parametrised serialisation test * fix: improve docstring for config-driven fields serialisation test * fix: use tuple for pytest.mark.parametrize first argument (cherry picked from commit 76eb2a0900239c0df30a5e286c113b88cf60af7c) Co-authored-by: Selman <[email protected]> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
