potiuk commented on PR #62318:
URL: https://github.com/apache/airflow/pull/62318#issuecomment-3993694109

   > To me it seems like the largest issue here is that the 
create_system_user_if_missing function in the entrypoint only does anything if 
the user doesn't exists in /etc/passwd which on Docker is fair as for this 
image the only user we'd be running as that could exist there is the default 
airflow. Could this function be modified to run an idempotent manner where it 
always attempts to adjust these values for the user?
   
   
   Yes. But also yoy have to make sure that that function continues to work in 
docker as well. Ideally if you want to change some behaviour beacause podman 
needs it, it should detect that you are in podman and act differently then 
-without modifying what happens in docker.
   
   Otherwise you are likely to make it works for podman but break for docker, 
which would be quite unfortunate, because almost everyone contributing uses 
docker.


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