-
Notifications
You must be signed in to change notification settings - Fork 95
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
feat: add observability config #342
Conversation
Signed-off-by: wenfeng <[email protected]>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #342 +/- ##
==========================================
+ Coverage 62.72% 63.01% +0.29%
==========================================
Files 133 144 +11
Lines 12128 13103 +975
==========================================
+ Hits 7607 8257 +650
- Misses 3977 4263 +286
- Partials 544 583 +39
Continue to review full report at Codecov.
|
Signed-off-by: wenfeng <[email protected]>
the k8s.yaml need to add observability config |
Signed-off-by: wenfeng <[email protected]>
This PR was made for contributors, as for k8s.yml, my idea is we could add it after our observability sub-system is ready |
* feat: add observability config Signed-off-by: wenfeng <[email protected]> * fix lint Signed-off-by: wenfeng <[email protected]> * fix suggestions Signed-off-by: wenfeng <[email protected]> Signed-off-by: wenfeng <[email protected]>
Signed-off-by: wenfeng [email protected]
What problem does this PR solve?
Issue Number: close N/a
Problem Summary
add observability config
What is changed and how does it work?
Check List
Tests