potiuk commented on PR #62318:
URL: https://github.com/apache/airflow/pull/62318#issuecomment-4033419161
> @potiuk! I've updated the implementation to address the feedback:
>
> ✅ Fixed the chgrp/chmod commands to retain both ${AIRFLOW_USER_HOME_DIR}
and ${AIRFLOW_HOME} ✅ Added set_pythonpath_for_arbitrary_user() function that
explicitly sets PYTHONPATH and PATH for arbitrary user execution ✅ Synced the
inline heredoc with the standalone entrypoint script The changes allow the
production image to run with arbitrary UIDs on Podman while maintaining Docker
compatibility.
No . the issues wer enot addressed. You are not detecting podman and the way
it was passed also you are - for some reson, not explained changing the chown
into chgrp.
Are you sure you know what you are doing ? Or are you blindly followig what
AI generates for you? Can you explain in short paragraph what your PR is doing
now?
--
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]
