-
Notifications
You must be signed in to change notification settings - Fork 30
♻️ tracing can be disabled on all services #6682
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
♻️ tracing can be disabled on all services #6682
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6682 +/- ##
==========================================
+ Coverage 87.66% 87.94% +0.27%
==========================================
Files 1527 1557 +30
Lines 60858 62790 +1932
Branches 2106 2106
==========================================
+ Hits 53352 55220 +1868
- Misses 7181 7245 +64
Partials 325 325
Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
services/autoscaling/tests/unit/test_modules_auto_scaling_computational.py
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes this was the step missing ... htx @GitHK
|
What do these changes do?
Expose the configuration to disable the opentelemetry to all the services that are currently using it.
Extra:
Related issue/s
How to test
Dev-ops checklist