-
Notifications
You must be signed in to change notification settings - Fork 149
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 Debian Slim as base image #148
Conversation
Thank you. Could I kindly ask you to rebase and squash? |
I rebased it now, but I am making these commits through the Github Web UI so I dont think I can squash them |
Dockerfile
Outdated
@@ -1,6 +1,6 @@ | |||
# syntax=docker/dockerfile:1.2 | |||
|
|||
ARG BASE_IMAGE=debian:latest | |||
ARG BASE_IMAGE=debian-slim:latest |
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.
This is wrong. That's why the integration test is failing.
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.
Fixed now
Pull request seems to work, but we really need to rebase and squash it. There are literary 5 changes but 10 commits and I cannot merge it because it says "This branch cannot be rebased due to conflicts". |
Replaced by #149 |
Should make the docker image a lot smaller