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

Published 20 hours ago • verified publisher icon dexidp

feat(helm): push chart to the OCI Registry

Open nlamirault opened this issue 1 year ago • 6 comments

Overview

What this PR does / why we need it

fix #128

Special notes for your reviewer

Checklist

  • [ ] Change log updated in Chart.yaml (see the contributing guide for details)
  • [ ] Chart version bumped in Chart.yaml (see the contributing guide for details)
  • [ ] Documentation regenerated by running make docs

nlamirault avatar Jul 18 '24 09:07 nlamirault

This is a good idea, but I personally do not want to use both the web and registry chart museum.

@sagikazarmark what do you think?

nabokihms avatar Jul 18 '24 13:07 nabokihms

I think we should shift towards using OCI. There is also an issue somewhere to move the chart back to the core repo to stay closer with the releases to the main project. I think we should tackle those two together.

sagikazarmark avatar Jul 18 '24 13:07 sagikazarmark

OCI is the new norm for Helm charts. What is holding this PR from being reviewed, approved and merged?

erikgb avatar Jan 13 '25 16:01 erikgb

What is holding this PR from being reviewed, approved and merged?

Physics: there are only 24 hours in a day.

sagikazarmark avatar Jan 13 '25 16:01 sagikazarmark

@nabokihms I think we can push to both registries for now and allow a transition period.

Once we move the chart back to the Dex repo, we can archive this one.

sagikazarmark avatar Jan 13 '25 16:01 sagikazarmark

@sagikazarmark anything else required?

Update: Ah maybe due to this intent:

  • https://github.com/dexidp/helm-charts/issues/159

mkilchhofer avatar Oct 20 '25 19:10 mkilchhofer