Skip to content
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

AWS Lambda Semantic conventions spec does not provide mechanism for properly propagating context using SQS #380

Closed
rapphil opened this issue Dec 7, 2022 · 3 comments
Labels

Comments

@rapphil
Copy link
Member

rapphil commented Dec 7, 2022

Describe the bug
There exists OTel semantic conventions that define how to represent information related to spans representing the processing of SQS events in AWS Lambda.

As of now, there is no way to properly trace events that happens asynchronously in set of services that communicate using SQS messages. Every lambda invocation will create a new trace context (new trace id) for the SQS Event. If we have xray sampling enabled, that means that each hop in this set of services will have the probability of emitting a new trace or not emitting it. This creates a problem because you will not have the full path of events.

Customers could work around this limitation by using specific xray sampling rules for each hop in the chain of services, but this is not ideal and represents a mismatch with the behavior observed in synchronous http lambda invocations.

What did you expect to see?
We should provide alternatives for customers to use the parent context propagated in the message instead of creating a new trace context on every lambda invocation for an SQS Event.

Additional context

@rapphil rapphil changed the title AWS Lambda spec does not provide mechanism for properly propagating context using SQS AWS Lambda Semantic conventions spec does not provide mechanism for properly propagating context using SQS Dec 7, 2022
@rapphil
Copy link
Member Author

rapphil commented Dec 9, 2022

Right now we don't have a good solution that is originated from the spec/standardized. In fact, there is also this upstream issue open-telemetry/semantic-conventions#1187

@github-actions
Copy link

This issue is stale because it has been open 90 days with no activity. If you want to keep this issue open, please just leave a comment below and auto-close will be canceled

@github-actions github-actions bot added the stale label Mar 12, 2023
@github-actions
Copy link

This issue was closed because it has been marked as stale for 30 days with no activity.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Apr 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant