opentelemetry-helm-charts
opentelemetry-helm-charts copied to clipboard
open-telemetry
[opentelemetry-ebpf-instrumentation] add new helm chart
opened a PR https://github.com/open-telemetry/opentelemetry-helm-charts/pull/1704
Is this different than https://github.com/open-telemetry/opentelemetry-helm-charts/tree/main/charts/opentelemetry-ebpf?
You'll need to recruit a SIG member from that repository to help maintain this chart.
hey @TylerHelmuth thanks for the response opentelemetry-ebpf helm chart refers to opentelemetry-network helm this repo is for opentelemetry-ebpf-instrumentation those are different repos even though the naming is similar
not super sure what you mean by SIG memory from that repo, can you elaborate?
How do those two repos interact? Would they every be installed together?
How do those two repos interact? Would they every be installed together?
@TylerHelmuth from the Beyla Due Diligence document:
Based on talks between Jonathan Perry and Nikola Grcevski … The conclusion is that the Beyla donation will supersede this project in the long run. OTel will recommend switching to the Beyla donation once feature parity is reached with the current network monitoring.
Hmmm. That makes me think it should be a feature in the existing chart.
I endorse this helm chart creation.
The previous opentelemetry-ebpf helm chart belongs ot the opentelemetry-network project, which at the moment has 0% feature overlap with opentelemetry-ebpf-instrumentation (both can provide network metrics but none of them follow the same specification).
@mariomac as a maintainer of opentelemetry-ebpf-instrumentation can you help with this chart?
@TylerHelmuth sure! @ittai-coralogix is regularly joining the opentelemetry-ebpf-instrumentation weekly SIG meetings and we are in touch about this issue.
Ok. To start, please review the current open PR for all thing related to your application. I'll take a look next week at some of the admin stuff we need charts in this repo to abide by.
To ensure this new chart is managed consistently with the rest of the charts in this repository, I’d recommend implementing the following or filing follow-up issues to track them:
- Update the root README.md to include a reference to the new chart.
- Integrate the chart with the Makefile for consistent tooling support.
- Add at least one example and ensure it’s is rendered via `make generate-examples', this helps catch breaking changes early.
