Miretpl commented on code in PR #61027:
URL: https://github.com/apache/airflow/pull/61027#discussion_r2726035760


##########
chart/newsfragments/61027.significant.rst:
##########
@@ -0,0 +1,4 @@
+Add ``dags.gitSync.version`` configuration
+
+The chart now uses ``dags.gitSync.version`` to determine what version of git 
sync is used. The ``images.gitSync.tag`` value does not influence this.
+The default ``dags.gitSync.version`` is ``4.4.2``. If you are using git sync 
v3, you must set ``dags.gitSync.version: 3.x.x`` explicitly.

Review Comment:
   If someone uses the 3rd version (I doubt that personally, but it is still 
supported), it will be a breaking change. Do we want to make this in that way 
(we are removing breaking change behaviour regarding the `workers.celery` 
section, so I think we should not introduce a different one in a different 
place)?



-- 
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]

Reply via email to