helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

ci: push charts to GHCR

Open onedr0p opened this issue 2 months ago • 8 comments

[!IMPORTANT] After this is merged and charts are released you might need to make the packages public from here

I also bumped some deps as well.

Fixes: #550

onedr0p avatar Sep 29 '25 21:09 onedr0p

cc @patrick-stephens

onedr0p avatar Sep 29 '25 21:09 onedr0p

I'm going to delegate to @stevehipwell for his expertise here plus I know he has some much improved CI he wants to push :)

patrick-stephens avatar Sep 30 '25 09:09 patrick-stephens

@patrick-stephens Ah, yes. I did just discover his comment here. However I think merging this is a good stop-gap until more work can be put into getting his action working here, or unless @stevehipwell wants to supersede this PR with his own using his action in the meantime.

onedr0p avatar Sep 30 '25 14:09 onedr0p

@onedr0p thanks for the PR. I'm planning on adding my PR soon, but this may be a good stopgap.

@patrick-stephens could you check if the OCI packages are public? Once they are I'll manually upload a chart version that we can test before making a decision on merging this change.

stevehipwell avatar Oct 08 '25 16:10 stevehipwell

No packages exist yet so they are not public, assuming we are pushing to ghcr.io/${{ github.repository }} which would map to ghcr.io/fluent/helm-charts.

We can change the package visibility easily enough, I'm not sure what it will default to initially and we may need to link it to this repo potentially - sometimes the action does that for you automatically.

patrick-stephens avatar Oct 09 '25 09:10 patrick-stephens

Bit surprised we have to update the actions, I'm guessing we should have a dependency PR for that somewhere.

It looks like Dependabot has been ignored so it's stopped checking, I've just manually re-triggered it.

stevehipwell avatar Oct 09 '25 09:10 stevehipwell

@stevehipwell what do you mean, it looks normal to me?

onedr0p avatar Oct 16 '25 15:10 onedr0p

/ping @patrick-stephens @stevehipwell

onedr0p avatar Oct 28 '25 11:10 onedr0p

/ping @patrick-stephens @stevehipwell

onedr0p avatar Nov 22 '25 11:11 onedr0p

It looks ok now but I know @stevehipwell was working on a few things

patrick-stephens avatar Nov 24 '25 15:11 patrick-stephens

@patrick-stephens

His comment:

@onedr0p thanks for the PR. I'm planning on adding my PR soon, but this may be a good stopgap.

Made it seem like this PR could go ahead, unless something changed since October that I am unaware of...

onedr0p avatar Nov 24 '25 15:11 onedr0p