Merge pull request #2327 from adriang-90/fix/Dockerfiles

[fix] Refactored Dockerfiles

.changeset/fresh-dots-hear.md

+---
+'@eth-optimism/batch-submitter-service': patch
+'@eth-optimism/gas-oracle': patch
+'@eth-optimism/integration-tests': patch
+'@eth-optimism/l2geth': patch
+'@eth-optimism/hardhat-node': patch
+'@eth-optimism/contracts': patch
+'@eth-optimism/data-transport-layer': patch
+'@eth-optimism/message-relayer': patch
+---
+
+Refactored Dockerfiles

.changeset/mighty-news-jam.md

+---
+'@eth-optimism/batch-submitter-service': patch
+'@eth-optimism/gas-oracle': patch
+'@eth-optimism/integration-tests': patch
+'@eth-optimism/l2geth': patch
+'@eth-optimism/hardhat-node': patch
+'@eth-optimism/contracts': patch
+'@eth-optimism/data-transport-layer': patch
+'@eth-optimism/message-relayer': patch
+---
+
+Refactored Dockerfiles

ops/docker/Dockerfile.batch-submitter-service

@@ -15,5 +15,6 @@ FROM alpine:3.13
 RUN apk add --no-cache ca-certificates jq curl
 COPY --from=builder /go/batch-submitter/batch-submitter /usr/local/bin/
 
+WORKDIR /usr/local/bin
 COPY ./ops/scripts/batch-submitter.sh .
 ENTRYPOINT ["batch-submitter"]

ops/docker/Dockerfile.gas-oracle

@@ -2,7 +2,7 @@ FROM golang:1.15-alpine3.13 as builder
 
 RUN apk add --no-cache make gcc musl-dev linux-headers git jq bash
 
-ADD ./go/gas-oracle /gas-oracle
+COPY ./go/gas-oracle /gas-oracle
 RUN cd /gas-oracle && make gas-oracle
 
 FROM alpine:3.13
@@ -10,5 +10,6 @@ FROM alpine:3.13
 RUN apk add --no-cache ca-certificates jq curl
 COPY --from=builder /gas-oracle/gas-oracle /usr/local/bin/
 
+WORKDIR /usr/local/bin/
 COPY ./ops/scripts/gas-oracle.sh .
 ENTRYPOINT ["gas-oracle"]

ops/docker/Dockerfile.geth

@@ -15,6 +15,7 @@ FROM alpine:3.13
 RUN apk add --no-cache ca-certificates jq curl
 COPY --from=builder /go-ethereum/build/bin/geth /usr/local/bin/
 
+WORKDIR /usr/local/bin/
 EXPOSE 8545 8546 8547
 COPY ./ops/scripts/geth.sh .
 ENTRYPOINT ["geth"]

ops/docker/Dockerfile.l2geth-exporter

 FROM golang:1.16 as builder
 
-ADD ./go/l2geth-exporter /app/
+COPY ./go/l2geth-exporter /app/
 WORKDIR /app/
 RUN make build
 
-FROM alpine:latest
+FROM alpine:3.13
 RUN apk --no-cache add ca-certificates
 WORKDIR /root/
 COPY --from=builder /app/l2geth-exporter /usr/local/bin/

ops/docker/Dockerfile.op-exporter

 FROM golang:1.16 as builder
 
-ADD ./go/op-exporter /app/
+COPY ./go/op-exporter /app/
 WORKDIR /app/
 RUN make build
 
-FROM alpine:latest
+FROM alpine:3.13
 RUN apk --no-cache add ca-certificates
 WORKDIR /root/
 COPY --from=builder /app/op-exporter /usr/local/bin/

ops/docker/Dockerfile.packages

@@ -6,7 +6,12 @@
 # when used with typescript/hardhat: https://github.com/nomiclabs/hardhat/issues/1219
 FROM node:16.13-buster-slim as base
 
-RUN apt-get update -y && apt-get install -y git curl jq python3
+RUN apt-get update -y && apt-get install -y --no-install-recommends git \ 
+      curl \
+      jq \
+      python3 \
+      ca-certificates \
+  &&  rm -rf /var/lib/apt/lists/*
 
 # copy over the needed configs to run the dep installation
 # note: this approach can be a bit unhandy to maintain, but it allows
@@ -22,7 +27,7 @@ COPY packages/message-relayer/package.json ./packages/message-relayer/package.js
 COPY packages/replica-healthcheck/package.json ./packages/replica-healthcheck/package.json
 COPY integration-tests/package.json ./integration-tests/package.json
 
-RUN yarn install --frozen-lockfile
+RUN yarn install --frozen-lockfile && yarn cache clean
 
 COPY ./packages ./packages
 COPY ./integration-tests ./integration-tests

ops/docker/hardhat/Dockerfile

@@ -8,7 +8,7 @@ COPY [ \
 
 # install deps
 WORKDIR /hardhat
-RUN yarn install
+RUN yarn install && yarn cache clean
 
 # bring in dockerenv so that hardhat launches with host = 0.0.0.0 instead of 127.0.0.1
 # so that it's accessible from other boxes as well