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.

Sign up for GitHub

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

Jump to bottom

Publish HelmCharts as OCI (Open Container Initative) format #74

Open
onedr0p opened this issue Mar 14, 2025 · 4 comments · May be fixed by #75
Open

Publish HelmCharts as OCI (Open Container Initative) format #74

onedr0p opened this issue Mar 14, 2025 · 4 comments · May be fixed by #75

Comments

@onedr0p
Copy link

onedr0p commented Mar 14, 2025

Description

Publish the helm charts as OCI, example to the intel docker hub, quay or GHCR?

Context

OCI is becoming more prevalent, example some of our dependencies like bitnami charts are also as OCI:
blog.bitnami.com/2023/04/httpsblog.bitnami.com202304bitnami-helm-charts-now-oci.html
In particular bitnami is deprecating the index.yaml and way and strongly suggest we migrate to OCI.

Possible Solution

My suggestion is to push as OCI as additional step (keeping existing way). If all goes well then we can also add some time frame and then start pushing new versions to OCI only. We need not disable the existing index.yaml as that would break users.
@mythi
Copy link
Contributor

mythi commented Mar 14, 2025

Publish the helm charts as OCI, example to the intel docker hub, quay or GHCR?

thanks for the issue. we have talked about this but it's not a priority and would first require a bit of prep work. would you mainly be interested in the device plugins stuff or some other helm charts as well?

@onedr0p
Copy link
Author

onedr0p commented Mar 14, 2025 •
edited
Loading

I understand, for me personally the charts intel-device-plugins-operator and intel-device-plugins-gpu would be nice.

@onedr0p onedr0p mentioned this issue Mar 17, 2025
Open
onedr0p added a commit to onedr0p/helm-charts that referenced this issue Mar 27, 2025
@onedr0p
ci: push Helm Chart to GHCR
0139de0 
>[!IMPORTANT]
> After this is merged and charts are released you will need to make the packages public from [here](https://github.com/orgs/intel/packages)

I also bumped some deps and replaced ACCESS_TOKEN secret with the builtin GITHUB_TOKEN

Fixes: intel#74
@onedr0p onedr0p linked a pull request Mar 27, 2025 that will close this issue
Open
@onedr0p
Copy link
Author

onedr0p commented Mar 27, 2025

@mythi I have added a PR here

#75

@onedr0p
Copy link
Author

onedr0p commented Mar 27, 2025 •
edited
Loading

I see what you mean by prep work, why are some charts folders being duplicated when a new chart version is made?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ci: push Helm Chart to GHCR onedr0p/helm-charts
2 participants
@onedr0p @mythi