Skip to content

Commit

Permalink
consistently install bash + wget
Browse files Browse the repository at this point in the history
alpine had wget but no bash

ubuntu had bash but no wget

let's have both, since bash is handy for 'fly intercept' and wget is
used in k8s-topgun (which currently uses busybox, but now could be
switched to use mock)

Signed-off-by: Alex Suraci <[email protected]>
  • Loading branch information
vito committed Oct 22, 2020
1 parent 721befc commit 89aa687
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions dockerfiles/alpine/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,4 @@ RUN go build -o /assets/check ./cmd/check
FROM alpine:edge
COPY --from=builder assets/ /opt/resource/
RUN chmod +x /opt/resource/*
RUN apk --no-cache add bash wget
1 change: 1 addition & 0 deletions dockerfiles/ubuntu/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,4 @@ RUN go build -o /assets/check ./cmd/check
FROM ubuntu:bionic AS resource
COPY --from=builder assets/ /opt/resource/
RUN chmod +x /opt/resource/*
RUN apt-get update && apt-get -y -u install wget

0 comments on commit 89aa687

Please sign in to comment.