-
Notifications
You must be signed in to change notification settings - Fork 6
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
Added Otlp UDP Exporter for Lambda Support #129
Conversation
What is the format of trace/span id? It is neither W3C nor X-Ray format.
|
src/AWS.Distro.OpenTelemetry.AutoInstrumentation/OtlpUdpExporter.cs
Outdated
Show resolved
Hide resolved
src/AWS.Distro.OpenTelemetry.AutoInstrumentation/opentelemetry/proto/collector/README.md
Outdated
Show resolved
Hide resolved
It's in base64 and it's just for printing purposes. This is the ToString function by otel. That is from the raw otel span that is exported to X-Ray. In the X-Ray raw trace data, you can see it's in the correct format. When calling the sample app, the response trace id is |
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.
LGTM
Description of changes:
Added a new OTLP UDP exporter to export Activities in the form of OTLP spans over UDP. Most of the code is commented to explain how that was achieved but here is a summary:
opentelemetry/proto
as those files were copied directly from the proto repo and it builds properly after updating the project file. The main files to focus on arePlugin.cs
and the newOtlpUdpExporter.cs
file that has the bulk of the logic. I've added comments throughoutOtlpUdpExporter.cs
to explain what each function does at a high level.OTLP_TRACES_EXPORTER
to 'none'. This will be done in the lambda layer.AWS_LAMBDA_FUNCTION_NAME
and running the X-Ray Daemon locally to test E2E to X-Ray. You can also see that we maintained the resource attributes which are important info holding like sdk version:Below is the base64 string from the udp exporter:
Below is the raw trace data from the xray console:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.