Skip to content

Commit

Permalink
Simplify Docker image build.
Browse files Browse the repository at this point in the history
In principle, the second command is not needed since coq-bot already depends on bot-components. Furthermore, since a few days, this second command seems to consistently trigger a timeout in our CI during the Docker build. So removing it could help fix this issue.
  • Loading branch information
Zimmi48 authored Sep 16, 2024
1 parent 850d39c commit 7ce706b
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions release.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ RUN opam install . --destdir /src/opam-install

# Store the dynamic dependencies of the server
RUN opam depext -ln coq-bot > /src/depexts-coq-bot
RUN opam depext -ln bot-components > /src/depexts-bot-components


FROM alpine:3.10 AS app
Expand All @@ -33,12 +32,10 @@ RUN apk update \
&& adduser coqbot -DG coqbot

COPY --from=builder /src/depexts-coq-bot depexts-coq-bot
COPY --from=builder /src/depexts-bot-components depexts-bot-components

# Install the required dynamic dependencies
RUN cat depexts-coq-bot | xargs apk --update add
RUN cat depexts-bot-components | xargs apk --update add

EXPOSE 8000

CMD ["./bot.exe"]
CMD ["./bot.exe"]

0 comments on commit 7ce706b

Please sign in to comment.