Skip to content

Conversation

rogercoll
Copy link
Contributor

Description

grpc/protobuf is not a valid configuration option for the otelconf protocol configuration, grpc should be used instead. See: https://github.com/open-telemetry/opentelemetry-go-contrib/blob/main/otelconf/v0.3.0/config.go#L30C26-L30C30

@rogercoll rogercoll requested a review from a team as a code owner September 12, 2025 15:56
@rogercoll
Copy link
Contributor Author

Skip changelog? 🙏

Copy link

codecov bot commented Sep 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.39%. Comparing base (2710536) to head (8271433).
⚠️ Report is 39 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13830      +/-   ##
==========================================
- Coverage   91.42%   91.39%   -0.03%     
==========================================
  Files         646      646              
  Lines       42605    42605              
==========================================
- Hits        38951    38940      -11     
- Misses       2833     2841       +8     
- Partials      821      824       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@songy23 songy23 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 12, 2025
@songy23 songy23 added the ready-to-merge Code review completed; ready to merge by maintainers label Sep 15, 2025
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Code review completed; ready to merge by maintainers Skip Changelog PRs that do not require a CHANGELOG.md entry Skip Contrib Tests Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants