beyla
beyla copied to clipboard
Revert "Add metrics to measure latency of k8s informer (#1200)"
This reverts commit 6960d77c4043220eaafe617f52a2d6a9a805ef14.
These metrics are not accurate as they are using Pod creation time as reference. The idea was to track latency of informers but the approach is not correct.
Alternatively we could have a counter, which would track how many times the handlers of an informer are called, but I don't think that's very useful.
Codecov Report
Attention: Patch coverage is 53.33333% with 7 lines in your changes missing coverage. Please review.
Project coverage is 81.79%. Comparing base (
5cb7064) to head (7cc0dbf).
Additional details and impacted files
@@ Coverage Diff @@
## main #1214 +/- ##
==========================================
+ Coverage 81.76% 81.79% +0.03%
==========================================
Files 137 137
Lines 11509 11465 -44
==========================================
- Hits 9410 9378 -32
+ Misses 1562 1547 -15
- Partials 537 540 +3
| Flag | Coverage Δ | |
|---|---|---|
| integration-test | 60.91% <33.33%> (+0.49%) |
:arrow_up: |
| k8s-integration-test | 58.29% <53.33%> (-0.18%) |
:arrow_down: |
| oats-test | 36.28% <26.66%> (+0.13%) |
:arrow_up: |
| unittests | 53.10% <13.33%> (+0.15%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.