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

Question on boto3 traces #101

Open
berkowitzi opened this issue Aug 20, 2019 · 5 comments
Open

Question on boto3 traces #101

berkowitzi opened this issue Aug 20, 2019 · 5 comments

Comments

@berkowitzi
Copy link

cc @Jamim

We're finding that the boto3 patcher is not successfully patching our boto3 API calls and are wondering if you can discuss your approach and whether it has any shortcomings. Example snippets of boto3 calls that you would expect to be properly instrumented - particularly focusing on s3 usage would be much appreciated!

Apologies for the vague question, our investigation continues but we'd love any input you could provide.
Thanks!

@Jamim
Copy link
Contributor

Jamim commented Aug 21, 2019

Hello @berkowitzi,

I'll prepare some snippets tomorrow.
Also, feel free to ping me if I don't provide them within the next 24 hours.

Best regards!

@Jamim
Copy link
Contributor

Jamim commented Aug 23, 2019

Hi @berkowitzi,

Working on snippets for you, I found that the S3 client is implemented in a different way than the rest of boto3.
Unfortunately, I didn't notice this initially, so opentracing-instrumentation doesn't support S3 yet.

The good new is I already started implementing support for S3.
I'll let you know as soon as it is ready for testing.

Thanks for your patience!

@yurishkuro
Copy link
Contributor

I think this repo overall could've used examples/ dir to show how various integrations could be used.

@Jamim
Copy link
Contributor

Jamim commented Sep 2, 2019

Hey @berkowitzi,

I've created a demo that shows Boto 3 instrumentation:
https://github.com/Jamim/opentracing-boto3-demo

Let's say Thank you to @yurishkuro for quick acceptance of the improvements and releasing the updated version! 👏

Please let us know whether this issue might be closed.

Cheers!

@Jamim
Copy link
Contributor

Jamim commented Oct 9, 2019

Hi @berkowitzi,
Could you please provide any feedback?
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants