Copilot commented on code in PR #11659: URL: https://github.com/apache/gravitino/pull/11659#discussion_r3417709281
########## dev/docker/lance-rest-server/Dockerfile: ########## @@ -27,7 +27,8 @@ WORKDIR /opt/gravitino-lance-rest-server COPY --chmod=775 packages/gravitino-lance-rest-server /opt/gravitino-lance-rest-server -RUN chmod +x /opt/gravitino-lance-rest-server/bin/start-lance-rest-server.sh +RUN chmod 775 /opt/gravitino-lance-rest-server \ + && chmod +x /opt/gravitino-lance-rest-server/bin/start-lance-rest-server.sh Review Comment: In this image there is no user/group created for uid 1000 (unlike the other two Dockerfiles). If the container runs as uid=1000 with a non-root primary gid (common when using numeric USER), then `chmod 775` on a `root:root` directory still won’t allow writes, so the log-directory issue can persist. Add the same `useradd -u 1000 -g 0 ...` step (or explicitly set `USER 1000:0`) to guarantee the intended gid=0/group-writable 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]
