Skip to content

feat(helm): push chart to the OCI Registry#136

Open
nlamirault wants to merge 3 commits intodexidp:masterfrom
nlamirault:feat/helm-oci
Open

feat(helm): push chart to the OCI Registry#136
nlamirault wants to merge 3 commits intodexidp:masterfrom
nlamirault:feat/helm-oci

Conversation

@nlamirault
Copy link
Copy Markdown

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

Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
@nabokihms
Copy link
Copy Markdown
Member

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?

@sagikazarmark
Copy link
Copy Markdown
Member

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.

@erikgb
Copy link
Copy Markdown

erikgb commented Jan 13, 2025

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

@sagikazarmark
Copy link
Copy Markdown
Member

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

Physics: there are only 24 hours in a day.

Comment thread .github/workflows/release.yaml Outdated
@sagikazarmark
Copy link
Copy Markdown
Member

@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.

nlamirault and others added 2 commits September 15, 2025 14:34
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Co-authored-by: Márk Sági-Kazár <mark.sagikazar@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
@mkilchhofer
Copy link
Copy Markdown

mkilchhofer commented Oct 20, 2025

@sagikazarmark anything else required?


Update: Ah maybe due to this intent:

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.

Publish helm chart also as an OCI package (e.g. on GHCR registry)

5 participants