Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nlamirault
Copy link

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

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)
3 participants