apiri commented on a change in pull request #181: MINIFI-517: Added 
/opt/minifi/minifi-current symlink
URL: https://github.com/apache/nifi-minifi/pull/181#discussion_r365035634
 
 

 ##########
 File path: minifi-docker/dockermaven/Dockerfile
 ##########
 @@ -26,14 +26,15 @@ ARG MINIFI_VERSION
 ARG MINIFI_BINARY
 
 ENV MINIFI_BASE_DIR /opt/minifi
-ENV MINIFI_HOME $MINIFI_BASE_DIR/minifi-$MINIFI_VERSION
+ENV MINIFI_HOME $MINIFI_BASE_DIR/minifi-current
 
 # Setup MiNiFi user
 RUN addgroup -g $GID minifi || groupmod -n minifi `getent group $GID | cut -d: 
-f1`
 RUN adduser -S -H -G minifi minifi
-RUN mkdir -p $MINIFI_HOME
+RUN mkdir -p $MINIFI_BASE_DIR
 
 ADD $MINIFI_BINARY $MINIFI_BASE_DIR
+RUN ln -s $MINIFI_BASE_DIR/minifi-$MINIFI_VERSION $MINIFI_HOME
 RUN chown -R minifi:minifi $MINIFI_HOME
 
 Review comment:
   Believe this would also be problematic as mentioned on the other Dockerfile.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to