jbonofre commented on code in PR #185: URL: https://github.com/apache/polaris-tools/pull/185#discussion_r2936350209
########## console/docker/Dockerfile: ########## @@ -41,24 +43,31 @@ RUN npm run build # Production stage FROM registry.access.redhat.com/ubi9/nginx-126:9.7-1767846422 - +USER root +RUN groupadd --gid 10001 polaris && \ + useradd --uid 10000 --gid polaris polaris # Copy DISCLAIMER, LICENSE, NOTICE files COPY DISCLAIMER /DISCLAIMER COPY LICENSE-BUNDLE /LICENSE COPY NOTICE /NOTICE # Copy custom nginx configuration as template -COPY docker/nginx.conf /opt/app-root/etc/nginx.d/default.conf.template +COPY docker/nginx.conf /app/app-root/etc/nginx.d/default.conf.template Review Comment: By the way, I think that can break nginx as the nginx UBI9 image is expect the configuration file in this folder specifically. I would suggest to keep the initial path. -- 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]
