-
Notifications
You must be signed in to change notification settings - Fork 469
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
Add CI scenarios for eBPF Chart #964
Comments
/cc @nicolastakashi |
Hi @TylerHelmuth I would like to have a try if this is still needed, thanks! |
Go for it @hyfj44255 🙏 |
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 ? |
Hey @hyfj44255, yes exactly like that |
Signed-off-by: Yang, Robin <Robin.Yang@fmr.com>
Hi @nicolastakashi I created a pr #1310 would it be ok if you take a look when you got chance? thanks |
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.
The text was updated successfully, but these errors were encountered: