search

signalfx / splunk-otel-lambda

Splunk distribution of OpenTelemetry Lambda
Apache License 2.0
5 stars 4 forks source link

Replace nodejs propagator propagator-aws-xray with propagator-aws-xray-lambda #308

Closed technimad-splunk closed 3 weeks ago

technimad-splunk commented 3 weeks ago

According to the semantic conventions xray SHOULD be replaced with xray-lambda: https://github.com/open-telemetry/semantic-conventions/blob/main/docs/faas/aws-lambda.md#xray-lambda-propagator-configuration

This is a breaking change for current configurations. So care should be taken on how to communicate this change.

technimad-splunk commented 3 weeks ago

The xray-lambda propagator should only be used when only reporting spans to x-ray. So it doesn't apply to the implementation in this repo.