Skip to content

Commit

Permalink
fix(loadtester): copy binary correctly (#83)
Browse files Browse the repository at this point in the history
  • Loading branch information
sanposhiho authored May 2, 2023
1 parent b13f2a1 commit 682784e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion loadtester/client/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ RUN go build -o ./loadtester-client .

FROM alpine:latest

COPY --from=build-env /go/src/github.com/mercari/tortoise/loadtester/client /bin/loadtester-client
COPY --from=build-env /go/src/github.com/mercari/tortoise/loadtester/client/loadtester-client /bin/loadtester-client

RUN apk --no-cache add ca-certificates

Expand Down
4 changes: 3 additions & 1 deletion loadtester/server/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,14 @@ RUN go build -o ./loadtester-server .

FROM alpine:latest

COPY --from=build-env /go/src/github.com/mercari/tortoise/loadtester/server /bin/loadtester-server
COPY --from=build-env /go/src/github.com/mercari/tortoise/loadtester/server/loadtester-server /bin/loadtester-server

RUN apk --no-cache add ca-certificates

RUN addgroup -g 1001 app && adduser -D -G app -u 1001 app

RUN chmod +x /bin/loadtester-server

USER 1001

CMD ["/bin/loadtester-server"]

0 comments on commit 682784e

Please sign in to comment.