-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 unit to execution time in telemetry events #2069
Add unit to execution time in telemetry events #2069
Conversation
I've just realized that the Chaos strategies are also reporting telemetry events.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2069 +/- ##
=======================================
Coverage 83.69% 83.69%
=======================================
Files 312 312
Lines 7114 7114
Branches 1054 1054
=======================================
Hits 5954 5954
Misses 789 789
Partials 371 371
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
I would say yes to both so there's parity in the documentation. |
Okay, I'll do that in a separate PR. |
Pull Request
The issue or feature being addressed
Follow up of #2060
Details on the issue fix or feature implementation
Confirm the following