Programmer-RD-AI commented on PR #54448:
URL: https://github.com/apache/airflow/pull/54448#issuecomment-3194180960
I see your point, updating only the directory ownership without `-R`
probably works in most cases since file ownership mismatches are rare. That
said, I still think the `find … ! -user "${AIRFLOW_UID}"` approach is the safer
long-term solution.
The reason is: while changing just the top-level dirs may be “good enough”
90% of the time, it leaves edge cases where files deeper in the tree are still
owned by `root`, which can cause subtle permission issues later. With the
`find` approach we don’t pay the full cost of `chown -R`, but we still catch
those rare but possible scenarios.
In my opinion, a fix that *always* works is better than one that *usually*
works, especially for something like Airflow where people run it in very
different environments.
--
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]
