caiok commented on a change in pull request #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#discussion_r127248048
########## File path: docker/4.4.0-alpine/Dockerfile ########## @@ -0,0 +1,59 @@ +FROM java:openjdk-8-jre-alpine +MAINTAINER Francesco Caliumi <francesco.cali...@gmail.com> + +# Install required packages +RUN apk add --no-cache \ + bash \ + su-exec + +ENV ZK_SERVERS= \ + BK_USER=bookkeeper \ + BK_PORT= \ + BK_BUILD_PORT=3181 \ + BOOKIE_OPTS="" \ + BK_JOURNAL_DIR=/data/journal \ + BK_LEDGER_DIR=/data/ledger \ + BK_INDEX_DIR=/data/index \ + BK_LEDGERS_PATH= + +# Add a user and make dirs +RUN set -x \ + && adduser -D "${BK_USER}" \ + && mkdir -p "${BK_JOURNAL_DIR}" "${BK_LEDGER_DIR}" "${BK_INDEX_DIR}" \ + && chown "$BK_USER:$BK_USER" "${BK_JOURNAL_DIR}" "${BK_LEDGER_DIR}" "${BK_INDEX_DIR}" + +ARG GPG_KEY=D0BC8D8A4E90A40AFDFC43B3E22A746A68E327C1 +ARG BK_VERSION=4.4.0 + +ENV DISTRO_NAME=bookkeeper-server-${BK_VERSION}-bin + +# Download Apache Bookkeeper, verify its PGP signature, untar and clean up +RUN set -x \ + && apk add --no-cache --virtual .build-deps \ + gnupg \ + wget \ + && mkdir -pv /opt \ + && cd /opt \ + && wget -q "https://archive.apache.org/dist/bookkeeper/bookkeeper-${BK_VERSION}/${DISTRO_NAME}.tar.gz" \ Review comment: The docker build process will be executed from docker hub (it will retrieve dockerfile and than make the build every night or so). For creating an image for the the latest commit (the "on build" image, like it's usually named) we need to publish that jars anyway. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services