Skip to content

Commit

Permalink
ISSUE apache#418: bump docker to use bk - 4.5.0
Browse files Browse the repository at this point in the history
Descriptions of the changes in this PR:
-  bump the docker image to use 4.5.0 binary, update pgp_key and bk version.
-  delete yum install for md5sum and sha1sum, since they are already in coreutils, which is default installed; and left them here will report error: 'No Match for argument: md5sum/sha1sum', while 'yum remove' in docker build.

have tested by running make run-dice.

Author: zhaijack <zhaijia03@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Francesco Caliumi <None>

This closes apache#436 from zhaijack/issue_418, closes apache#418
  • Loading branch information
zhaijack authored and sijie committed Aug 11, 2017
1 parent caba6b1 commit 7395408
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions docker/Dockerfile
Expand Up @@ -20,9 +20,9 @@
FROM centos:7
MAINTAINER Apache BookKeeper <dev@bookkeeper.apache.org>

ARG BK_VERSION=4.4.0
ARG BK_VERSION=4.5.0
ARG DISTRO_NAME=bookkeeper-server-${BK_VERSION}-bin
ARG GPG_KEY=B3D56514
ARG GPG_KEY=FD74402C

ENV BOOKIE_PORT=3181
EXPOSE $BOOKIE_PORT
Expand All @@ -31,7 +31,7 @@ ENV BK_USER=bookkeeper
# Download Apache Bookkeeper, untar and clean up
RUN set -x \
&& adduser "${BK_USER}" \
&& yum install -y java-1.8.0-openjdk-headless wget bash python md5sum sha1sum sudo \
&& yum install -y java-1.8.0-openjdk-headless wget bash python sudo \
&& mkdir -pv /opt \
&& cd /opt \
&& wget -q "https://archive.apache.org/dist/bookkeeper/bookkeeper-${BK_VERSION}/${DISTRO_NAME}.tar.gz" \
Expand All @@ -45,7 +45,7 @@ RUN set -x \
&& tar -xzf "$DISTRO_NAME.tar.gz" \
&& mv bookkeeper-server-${BK_VERSION}/ /opt/bookkeeper/ \
&& rm -rf "$DISTRO_NAME.tar.gz" "$DISTRO_NAME.tar.gz.asc" "$DISTRO_NAME.tar.gz.md5" "$DISTRO_NAME.tar.gz.sha1" \
&& yum remove -y wget md5sum sha1sum \
&& yum remove -y wget \
&& yum clean all

WORKDIR /opt/bookkeeper
Expand Down

0 comments on commit 7395408

Please sign in to comment.