-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Latency telemetry #1822
Latency telemetry #1822
Conversation
/retest |
/kind housekeeping |
/retest |
7c437d9
to
7ad1068
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: felixwang9817 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #1822 +/- ##
==========================================
- Coverage 84.84% 84.83% -0.02%
==========================================
Files 91 91
Lines 6924 7014 +90
==========================================
+ Hits 5875 5950 +75
- Misses 1049 1064 +15
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
Signed-off-by: Felix Wang <[email protected]>
8b32b2d
to
43b6de9
Compare
Closed due to #1983 |
What this PR does / why we need it: This PR adds latency telemetry to provider methods, online store methods, and offline store methods.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: