feat: Add 'opentelemetry-exporter-otlp-json-http' package#5051
feat: Add 'opentelemetry-exporter-otlp-json-http' package#5051herin049 wants to merge 33 commits intoopen-telemetry:mainfrom
Conversation
MikeGoldsmith
left a comment
There was a problem hiding this comment.
This is massive at over 9k lines. I'm generally not in favour of such large PRs as you can't approve with confidence.
Saying that, I did notice a couple of things we should look at.
This PR will be dramatically smaller once the PR to add |
|
@herin049 is this ready for review or still stacked on another PR? |
|
I'm gonna move this to draft until #4996 merges so it's clear it has a dependency before we can do a final review. |
Description
Follow up to #4996 and finishes the implementation of the OTLP JSON exporter.
Fixes #1003.
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Tests have been added to the
opentelemetry-exporter-otlp-json-httpandopentelemetry-exporter-otlp-json-commonpackages, and the existing Docker tests have been completely rewritten to test gRPC, proto-HTTP and JSON-HTTP exporters with all signal types.Does This PR Require a Contrib Repo Change?
Checklist: