pkumar-singh commented on a change in pull request #2867:
URL: https://github.com/apache/bookkeeper/pull/2867#discussion_r739379031
##########
File path: tests/docker-images/statestore-image/Dockerfile
##########
@@ -46,8 +46,8 @@ RUN mkdir /opt/bookkeeper/conf
RUN mkdir /opt/bookkeeper/scripts
### -----Copy Jars------###
-ADD ./dist/server.tar.gz /opt/
-RUN mv /opt/server/lib/*.jar /opt/bookkeeper/lib/
+ADD ./dist/bookkeeper-server-*.*SNAPSHOT.tar.gz /opt/
Review comment:
Actually, as part of normal CI. It builds release version at location.
`bookkeeper-dist/server/build/distributions` is built every time, and name of
the tarball always contains SNAPSHOT. If you think a situation where name
would not contain SNAPSHOT. Sure we can remove. In fact it hardly matters
either way.
--
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]
