fix(components): count bytes received for OTLP gRPC exports#1249
fix(components): count bytes received for OTLP gRPC exports#1249tobz merged 1 commit intoDataDog:mainfrom
Conversation
|
Awesome. Thanks for your contribution. :) |
|
Thanks for the approval. The required dd-gitlab checks are still stuck in "Expected" on GitHub, so the PR can't merge yet. Could you please trigger or rerun the GitLab CI for this PR? |
Yeah, that's weird. I'll try giving those a kick to get things moving again. |
|
@gyubin02 Alright, got CI to run and things look good. Last step is just to sign our CLA. We don't have the CLA bot set up yet (yay, you're our first external contribution! 🎉) so I'm pasting it here manually. Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format. I have read the CLA Document and I hereby sign the CLA |
|
I have read the CLA Document and I hereby sign the CLA |
Summary
Fixes missing
component_bytes_received_totalaccounting for OTLP gRPC requests.The HTTP path was already incrementing
bytes_received, but the shared gRPC OTLP path was not. This change updatesthe gRPC metrics, logs, and traces
export()handlers to increment the counter using the serialized protobuf payloadsize.
Change Type
How did you test this PR?
component_bytes_received_totalisincremented.
cargo test -p saluki-components updates_bytes_receivedReferences
component_bytes_received_total. #1245