-
Notifications
You must be signed in to change notification settings - Fork 44
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
chore: Add Dockerfile #517
Conversation
It is now a draft PR because before merging it'd be best to build and run it on a few different platforms and to receive feedback from more experienced Dockernauts. |
Benchmark ResultsSummary
✅ See Better Results...
❌ See Worse Results...
✨ See Unchanged Results...
🐋 See Full Results...
|
I feel unqualified to properly review this, but will note that I have looked at it. |
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.
Thanks for taking the time to write a Dockerfile =)
Made some suggestions, but my own dockerfile writing ability is a bit hazy at the moment.
RUN make build | ||
|
||
# Run stage | ||
FROM gcr.io/distroless/base-debian11 |
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.
praise: I've read about the distroless approach but never used it. Awesome to see it 🥳
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.
I second that :)
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.
LGTM!
That's a good starting point for our Dockerfile.
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.
LGTM! Thanks for getting our first step of Docker
in the door 😄.
RELEVANT ISSUE(S)
Resolves #516
DESCRIPTION
Introducing a Dockerfile.
We use a multistage approach which doesn't include the build environment in the resulting container.
This depends on https://github.com/GoogleContainerTools/distroless which we may not want to do. What we gain from this is a very small container.
I'd prefer using the new and more general terminology Containerfile but Dockerfile is very established, and DefraDB already brings a lot of new stuff.
Question: Should it live at the top-level or in a sub-directory?
I think this should be tested a bit more manually.
HOW HAS THIS BEEN TESTED?
DOCKER_BUILDKIT=1
on and offCHECKLIST:
ENVIRONMENT / OS THIS WAS TESTED ON?
Please specify which of the following was this tested on (remove or add your own):