Skip to content

feat(ci): push chart to ghcr and sign#205

Merged
squat merged 3 commits into
squat:mainfrom
aclerici38:oci-chart
May 7, 2026
Merged

feat(ci): push chart to ghcr and sign#205
squat merged 3 commits into
squat:mainfrom
aclerici38:oci-chart

Conversation

@aclerici38
Copy link
Copy Markdown
Contributor

Adds steps to push the new chart to ghcr as an oci artifact and sign it with cosign keyless.

OCI charts are more efficient and portable than traditional http repos, in addition the chart "living" next to the image in ghcr is nice ux for users. Cosign signing allows verifying the chart has been pushed by the ci here and not by a bad actor.

Use:
helm pull oci://ghcr.io/squat/charts/generic-device-plugin

cosign verify ghcr.io/squat/charts/generic-device-plugin  \
      --certificate-identity-regexp 'https://github.com/squat/generic-device-plugin/' \
      --certificate-oidc-issuer https://token.actions.githubusercontent.com

thanks!

@aclerici38
Copy link
Copy Markdown
Contributor Author

I bumped the chart version so the new steps would run, I'm not sure if you'd rather do that outside the PR or wait for an actual change to the chart

@squat
Copy link
Copy Markdown
Owner

squat commented May 7, 2026

Thanks @aclerici38! Question for you: if the chart repo is in OCR, should we remove the GitHub pages http repo to avoid confusion? Is it common for projects to have both? I didn't use helm myself so I'm not sure what's more common in the community.

Comment thread charts/generic-device-plugin/Chart.yaml
@aclerici38
Copy link
Copy Markdown
Contributor Author

should we remove the GitHub pages http repo to avoid confusion? Is it common for projects to have both?

It's definitely common for a chart to have both, though that's usually because the chart existed before OCI repos (adopted in 2022); I've seen a couple new charts just distribute through OCI.

IMO there's no reason to keep the http repo around, I'd just use OCI to distribute the chart. But I'm also not an expert, I only really use helm in my homelab so.. maybe someone else more experienced can chime in on it?

@squat squat enabled auto-merge May 7, 2026 16:30
@squat squat merged commit aacd9b5 into squat:main May 7, 2026
5 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.

2 participants