Skip to content

Bump go builder, build dependencies and runtime dependencies#69

Merged
mclasmeier merged 11 commits into
mainfrom
mc/go-builder
Mar 30, 2026
Merged

Bump go builder, build dependencies and runtime dependencies#69
mclasmeier merged 11 commits into
mainfrom
mc/go-builder

Conversation

@mclasmeier
Copy link
Copy Markdown
Collaborator

@mclasmeier mclasmeier commented Mar 20, 2026

Bumps

  • go builder (changing to go-toolset, which we also use for other images)
  • build dependencies
  • runtime dependencies in Dockerfile

Also removes the helm CLI from the image (which also comes with a critical CVE), thereby only supporting operator-based installs when using the image. I think this is alright, given that the nowadays hidden --helm option is only for specific dev needs until it is deprecated and removed completely.

Before:
image

After:
image

@mclasmeier mclasmeier changed the title Bump go builder Bump go builder, build dependencies and runtime dependencies Mar 24, 2026
Copy link
Copy Markdown
Collaborator

@GrimmiMeloni GrimmiMeloni left a comment

Choose a reason for hiding this comment

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

Overall looking good to me.
Question - is there any concern around the downloads we do here from google and github? I.e. shall we rather pin the downloads by SHA? (Or check against checksums?)
The recent wave of supply chain attacks makes me wonder if we can still trust a "known good" source to actually be known good all the time...

Comment thread Dockerfile
Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
Fix tar invocation
Copy link
Copy Markdown
Collaborator

@tommartensen tommartensen left a comment

Choose a reason for hiding this comment

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

LGTM

@mclasmeier mclasmeier merged commit f75bbc6 into main Mar 30, 2026
3 checks passed
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