opentelemetry-helm-charts icon indicating copy to clipboard operation
opentelemetry-helm-charts copied to clipboard

Add CI scenarios for eBPF Chart

Open TylerHelmuth opened this issue 1 year ago • 6 comments

Currently the ebpf chart has no CI tests. We should add a test for the default values.yaml at least, but it would be best if we could have more comprehensive coverage.

TylerHelmuth avatar Nov 20 '23 16:11 TylerHelmuth

/cc @nicolastakashi

TylerHelmuth avatar Nov 20 '23 16:11 TylerHelmuth

Hi @TylerHelmuth I would like to have a try if this is still needed, thanks!

hyfj44255 avatar May 28 '24 05:05 hyfj44255

Hi @TylerHelmuth I would like to have a try if this is still needed, thanks!

Go for it @hyfj44255 🙏

nicolastakashi avatar May 28 '24 06:05 nicolastakashi

Hi @TylerHelmuth and @nicolastakashi sorry for the delay ,you mentioned that "Currently the ebpf chart has no CI tests",by CI tests do you mean the tests that are ran by CT(helm/chart-testing) like the test cases under charts/opentelemetry-demo/ci ?

hyfj44255 avatar Jul 16 '24 07:07 hyfj44255

Hey @hyfj44255, yes exactly like that

nicolastakashi avatar Jul 16 '24 18:07 nicolastakashi