potiuk commented on PR #60270: URL: https://github.com/apache/airflow/pull/60270#issuecomment-3745459179
Question., If we really think it's a temporary solution only, I think it will add quite some pollution to config space - we will have two new config values that will become obsolete with the new solution - and they do not solve the concurrency overhead problem.. Why not simply make into a documentation for GitBundle impersonation: a) set group-writeable umask (002) b) make sure all your impersonated users have the same group as the user that runs DagFileProcessor That should (?) have the same effect as adding our code to manage the permissions, it would still require deployment level modifications, but at least it would not add new config params that will become useless in next version. Or maybe I missed something ? -- 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]
