jscheffl commented on code in PR #61039:
URL: https://github.com/apache/airflow/pull/61039#discussion_r3196728879
##########
chart/values.yaml:
##########
@@ -2309,6 +2309,39 @@ dagProcessor:
# classpath: "airflow.dag_processing.bundles.local.LocalDagBundle"
# kwargs: {}
+ # Per-bundle deployment option
+ # When enabled, creates a separate deployment for each bundle in
`dagBundleConfigList`
+ deployPerBundle:
Review Comment:
UPDATE: Airflow 2 support is dropped on main meanwhile - so discussion can
continue here.
As in the chart 2.0 we are allowed and plan to introduce breaking changes
this means we should make the deployment config in the best interest and we can
drop off legacy. Therefore still I'd opt to merge the different configu trees
into one such that deployment details can be defined directly on the bundle and
the switch to deploy per bundle would be a feature flag.
--
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]
