Skip to content

Commit

Permalink
Explicitly add ca-certificates to the release image (#258)
Browse files Browse the repository at this point in the history
It occurs to me that maybe the firehose issue we're seeing is a
lack of certs sitting inside the release image. If this commit
finds its way to the master branch this was the case.

Signed-off-by: Brian L. Troutwine <blt@postmates.com>
  • Loading branch information
blt authored May 1, 2017
1 parent 7434635 commit 9661301
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions docker/release/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,16 @@ FROM alpine:edge

RUN echo '@testing http://dl-cdn.alpinelinux.org/alpine/edge/testing' >> /etc/apk/repositories

RUN apk add --no-cache --update \
llvm-libunwind@testing \
openssl

RUN apk update \
&& apk upgrade --no-cache

RUN apk add --no-cache --update \
ca-certificates \
llvm-libunwind@testing \
openssl && \
update-ca-certificates && \
rm -rf /var/cache/apk/*

ARG DIST_ROOT="target/bindir"

COPY ${DIST_ROOT}/confd /etc/confd
Expand Down

0 comments on commit 9661301

Please sign in to comment.