You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "dstandish (via GitHub)" <gi...@apache.org> on 2025/03/06 22:39:21 UTC

[PR] Enable disabling of dag bundle versioning [airflow]

dstandish opened a new pull request, #47473:
URL: https://github.com/apache/airflow/pull/47473

   closes https://github.com/apache/airflow/issues/47386
   
   ### Description
   
   We need a way for both individual dags and a whole instance to opt out of using versioned bundles.
   
   I think the most bang for the buck will be this approach:
   
   - config options, something like `use_bundle_versioning`
   - DAG kwarg that uses that config option as the default
   - [conditionally add bundle_version to the dagrun](https://github.com/apache/airflow/blob/e49d7964de28f06c4ca8e28719603bb644501fad/airflow/models/dag.py#L277)
   
   Basically, very similar interface wise to how we handle max_active_tasks.
   
   ### Use case/motivation
   
   This allows folks to opt in to running on the latest bundle version for each task - basically how Airflow 2 operates, even if they have a versioned bundle under the hood.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Enable disabling of dag bundle versioning [airflow]

Posted by "jedcunningham (via GitHub)" <gi...@apache.org>.
jedcunningham commented on code in PR #47473:
URL: https://github.com/apache/airflow/pull/47473#discussion_r1984202507


##########
tests/models/test_dag.py:
##########
@@ -120,9 +121,11 @@
 def clear_dags():
     clear_db_dags()
     clear_db_serialized_dags()
+    clear_db_dag_bundles()
     yield
     clear_db_dags()
     clear_db_serialized_dags()
+    clear_db_dag_bundles

Review Comment:
   ```suggestion
       clear_db_dag_bundles()
   ```



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Enable disabling of dag bundle versioning [airflow]

Posted by "dstandish (via GitHub)" <gi...@apache.org>.
dstandish merged PR #47473:
URL: https://github.com/apache/airflow/pull/47473


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org