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

support OCI helm charts #1429

Open
M0NsTeRRR opened this issue Nov 22, 2024 · 4 comments · May be fixed by #1430
Open

support OCI helm charts #1429

M0NsTeRRR opened this issue Nov 22, 2024 · 4 comments · May be fixed by #1430
Labels
chart:collector Issue related to opentelemetry-collector helm chart chart:demo Issues related to opentelemetry-demo helm chart chart:ebpf Issues related to opentelemetry-ebpf helm chart chart:kube-stack chart:operator Issue related to opentelemetry-operator helm chart enhancement New feature or request github_actions Pull requests that update GitHub Actions code

Comments

@M0NsTeRRR
Copy link

No description provided.

@M0NsTeRRR M0NsTeRRR linked a pull request Nov 22, 2024 that will close this issue
Open
@TylerHelmuth
Copy link
Member

@M0NsTeRRR please add details to this issue.

@M0NsTeRRR
Copy link
Author

M0NsTeRRR commented Nov 25, 2024 •
edited
Loading

I've added the link in the PR that fix the issue, but it would be cool if otel helm chart support OCI registries as highlighted in helm documentation https://helm.sh/docs/topics/registries/.

Some advantages:

  • We can proxy Helm charts through OCI registries like Harbor.
  • Standardization and operability by adhering to OCI standards.
  • Performance improvement, as you don't need to parse an entire YAML file to retrieve your TGZ file (though this has minimal impact for OTEL at the moment).

@TylerHelmuth
Copy link
Member

@M0NsTeRRR can you check if the prometheus charts support this

@M0NsTeRRR
Copy link
Author

M0NsTeRRR commented Nov 25, 2024 •
edited
Loading

@M0NsTeRRR can you check if the prometheus charts support this

yes they use ghcr.io as you can see here and the github action is here

@TylerHelmuth TylerHelmuth added enhancement New feature or request chart:collector Issue related to opentelemetry-collector helm chart chart:operator Issue related to opentelemetry-operator helm chart github_actions Pull requests that update GitHub Actions code chart:demo Issues related to opentelemetry-demo helm chart chart:ebpf Issues related to opentelemetry-ebpf helm chart chart:kube-stack labels Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
chart:collector Issue related to opentelemetry-collector helm chart chart:demo Issues related to opentelemetry-demo helm chart chart:ebpf Issues related to opentelemetry-ebpf helm chart chart:kube-stack chart:operator Issue related to opentelemetry-operator helm chart enhancement New feature or request github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(helm): publish OCI helm charts M0NsTeRRR/opentelemetry-helm-charts
2 participants
@TylerHelmuth @M0NsTeRRR