New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use lts-slim instead of lts-alpine #36
Conversation
@larshp Let me know when you want me to review! |
ready for review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too bad that Alpine is so slow, but the speedup is remarkable.
RUN apk add --no-cache git | ||
RUN apt-get -y update | ||
RUN apt-get -y upgrade | ||
RUN apt-get -y install git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely optional: I think there's also a command you can run to cleanup things a bit to reduce the size of the image slightly: https://github.com/exercism/prolog-test-runner/blob/main/Dockerfile#L8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added, f3b6c1f
there is something wrong with "npm link" in alpine or this setup somewhere, its beyond me, just needed an easy fix 😄
there is something strange happening in lts-alpine when running "npm link", which makes it take 10seconds
this changes to use the lts-slim docker image instead, which dramatically reduces runtime(9+ seconds reduction)