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.

Sign up for GitHub

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

Jump to bottom

Feat: add option for tracing #17

Open

proost opened this issue Apr 11, 2023 · 1 comment · May be fixed by #18

Assignees

Contributor

proost commented Apr 11, 2023

Many APMs support trace for AWS services. Mostly APMs use session to monitor AWS services when using aws-sdk-v1
below is some examples,

v1: https://pkg.go.dev/github.com/newrelic/go-agent/v3/integrations/nrawssdk-v1#section-readme

v1: https://github.com/DataDog/dd-trace-go/blob/main/contrib/aws/aws-sdk-go/aws/aws.go

Splunk, Dynatrace

They integrates with Opentelemetry
In Opentelemetry, aws-sdk-v1 uses [aws-xray](https://github.com/aws/aws-xray-sdk-go) to trace.

so how about add a option to config for set tracing using session?

proost self-assigned this

Contributor

mingrammer commented Apr 13, 2023

Sounds good ideas. we can wrap the AWS session for both Kinesis and DynamoDB (for stastestore) clients.

proost linked a pull request

that will close this issue

feat: add session trace for kinesis & dynamoDB #18

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment