kumaab commented on code in PR #660:
URL: https://github.com/apache/ranger/pull/660#discussion_r2415111969


##########
dev-support/ranger-docker/Dockerfile.ranger:
##########
@@ -23,13 +23,20 @@ ARG RANGER_VERSION
 ARG RANGER_DB_TYPE
 ARG TARGETARCH
 
+# Set Ranger environment variables
+ENV RANGER_HOME=/opt/ranger
+ENV RANGER_SCRIPTS=/home/ranger/scripts
+
 COPY ./dist/version                               /home/ranger/dist/
 COPY ./dist/ranger-${RANGER_VERSION}-admin.tar.gz /home/ranger/dist/
 
 COPY ./scripts/ranger.sh                                         
${RANGER_SCRIPTS}/
 COPY ./scripts/ranger-admin-install-${RANGER_DB_TYPE}.properties 
${RANGER_SCRIPTS}/ranger-admin-install.properties
 COPY ./scripts/create-ranger-services.py                         
${RANGER_SCRIPTS}/
 
+# Switch to root to create system directories
+USER root

Review Comment:
   User is already `root` at this point, so this line is redundant.



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