walterddr commented on a change in pull request #8400:
URL: https://github.com/apache/pinot/pull/8400#discussion_r834608644
##########
File path: docker/images/pinot/Dockerfile
##########
@@ -71,7 +71,7 @@ ENV JAVA_OPTS="-Xms4G -Xmx4G -Dpinot.admin.system.exit=false"
VOLUME ["${PINOT_HOME}/configs", "${PINOT_HOME}/data"]
RUN apt-get update && \
- apt-get install -y --no-install-recommends vim less wget curl git python
&& \
+ apt-get install -y --no-install-recommends vim less wget curl git python
sysstat procps && \
Review comment:
@xiangfu0 ^ this is the actual runtime image correct?
checking there's one more `apt-get install` above in line 35 which I think
that is for the image running the maven build
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]