Skip to content

Updated base image to be from scratch instead of alpine #23

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

Conversation

nathanmcgarvey-modopayments
Copy link
Contributor

Added post-build check to guarantee statically linked binary
Updated builder container to use golang:1.20.5
Changed CMD to ENTRYPOINT as this is a single executable container with no arguments
Updated Makefile to setup a build multi-arch docker system if desired and TAG default to 1.9
Updated the docker buildx command to generate an sbom during build.
Added .gitignore

Added post-build check to guarantee statically linked binary
Updated builder container to use golang:1.20.5
Changed CMD to ENTRYPOINT as this is a single executable container with no arguments
Updated Makefile to setup a build multi-arch docker system if desired and TAG default to 1.9
Updated the docker buildx command to generate an sbom during build.
Added .gitignore
@stevehipwell
Copy link

@nathanmcgarvey-modopayments changing the base image has been sitting untouched on my to-do list for far too long, having looked over the code again it looks like scratch would work so this would be a big improvement. Do you want to update go.mod for Go v1.20? And what about adding Dependabot support while you're at it?

Copy link
Owner

@max-rocket-internet max-rocket-internet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, nice, thanks @nathanmcgarvey-modopayments 💖

@max-rocket-internet
Copy link
Owner

And what about adding Dependabot support while you're at it?

I've enabled it just now

@max-rocket-internet max-rocket-internet merged commit bbf4fd8 into max-rocket-internet:master Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants