Skip to content

Commit a46961d

Browse files
committed
Fix Docker dashboard build
Without stripping ownership metadata from the downloaded `grpcurl` binary we'll end up blowing up at runtine
1 parent 846fc77 commit a46961d

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

deploy/docker/Dockerfile.dashboard

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,11 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
3131
esac \
3232
&& curl -fsSL -o /tmp/grpcurl.tgz \
3333
"https://github.com/fullstorydev/grpcurl/releases/download/v${GRPCURL_VERSION}/grpcurl_${GRPCURL_VERSION}_linux_${GRPCURL_ARCH}.tar.gz" \
34-
&& tar -xzf /tmp/grpcurl.tgz -C /usr/local/bin grpcurl \
34+
&& tar -xzf /tmp/grpcurl.tgz --no-same-owner -C /usr/local/bin grpcurl \
3535
&& chmod +x /usr/local/bin/grpcurl \
36-
&& rm /tmp/grpcurl.tgz
36+
&& rm /tmp/grpcurl.tgz \
37+
# Smoke-test the binary
38+
&& grpcurl -version
3739
WORKDIR /app
3840
COPY --from=deps /app/node_modules ./node_modules
3941
COPY dashboard/ ./

0 commit comments

Comments
 (0)