fix: Remove remaining default configuration.#14351
fix: Remove remaining default configuration.#14351mx-psi merged 15 commits intoopen-telemetry:mainfrom
Conversation
|
Please document this change better with all side effects and add a changelog item |
Merging this PR will not alter performance
|
…fix-otlp-default-balancer
Head branch was pushed to by a user without write access
|
fix missing test... |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #14351 +/- ##
==========================================
- Coverage 91.85% 91.82% -0.03%
==========================================
Files 677 677
Lines 42680 42679 -1
==========================================
- Hits 39203 39190 -13
- Misses 2423 2431 +8
- Partials 1054 1058 +4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Head branch was pushed to by a user without write access
|
How should I fix contrib-test? |
|
@tank-500m Would you be willing to create a follow-up PR to fix the errors? We will likely merge this PR first, update the version of the core libraries used in contrib, then update the expected error message in the failing tests. |
|
@evan-bradley |
It seems to be fixed now. |
…fix-otlp-default-balancer
|
Thank you for your contribution @tank-500m! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. |
Description
Removed the remaining default configuration.
OTLP exporter’s default load balancing is intended to be round_robin.
Link to tracking issue
Fixes #
#14090
Testing
Documentation