Subham-KRLX commented on PR #65758:
URL: https://github.com/apache/airflow/pull/65758#issuecomment-4363721100

   > > (...) breaks existing DAG setups when migrating to the Helm chart teams 
have to restructure their entire DAG paths just because of this hardcoded name
   > 
   > Which case of migration and case where re-structuring you mean?
   
   @jscheffl The main issue is that many teams have existing setups (especially 
in local docker-compose) where DAGs are mounted directly without an extra 
subdirectory level when migrating to the Helm chart the forced repo/ path 
breaks things like absolute SQL file lookups and Python package imports Since 
git-sync natively supports the link flag for this exact purpose exposing it 
allows users to maintain path consistency across their environments without 
needing to refactor their code just to fit the chart's hardcoded naming.
   
   


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