Subham-KRLX commented on PR #65758: URL: https://github.com/apache/airflow/pull/65758#issuecomment-4317497280
> I am not sure and following the bug ticket "why" thisis needed. In my view making this configurable adds un-needed complexity. > > I would dis-like to merge additional complexity if there is no described need. > > Side note: Plan mid-term is to remove gitsync completely from the chart and migrate to dag bundles. Adding new features to this inside the chart therefore is rather un-desired. > > @mortazakiani as you opened the bug ticket asking for the feature, is there any rationale or convincing argument why this complexity should be added? > I am not sure and following the bug ticket "why" thisis needed. In my view making this configurable adds un-needed complexity. > > I would dis-like to merge additional complexity if there is no described need. > > Side note: Plan mid-term is to remove gitsync completely from the chart and migrate to dag bundles. Adding new features to this inside the chart therefore is rather un-desired. > > @mortazakiani as you opened the bug ticket asking for the feature, is there any rationale or convincing argument why this complexity should be added? @jscheffl the hardcoded "repo" directory breaks existing DAG setups when migrating to the Helm chart teams have to restructure their entire DAG paths just because of this hardcoded name I know git sync is being deprecated but users need this working now It's just one config option that defaults to current behavior. -- 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]
