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

Feature: github.com/streadway/amqp tracing #37

Open
axw opened this issue Mar 28, 2018 · 2 comments
Open

Feature: github.com/streadway/amqp tracing #37

axw opened this issue Mar 28, 2018 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@axw
Copy link
Member

axw commented Mar 28, 2018

Implement tracing for https://github.com/streadway/amqp.

This will probably have to be a collection of amqp.Channel method wrappers that:

  • add context.Context
  • create spans using said context
  • add headers (when we support distributed tracing)
  • call through to the underlying amqp.Channel
@axw axw added enhancement New feature or request help wanted Extra attention is needed labels Mar 28, 2018
@ekrucio
Copy link

ekrucio commented Nov 9, 2022

Is this issue still open? This missing feature is a deal-breaker for our migration to Elastic APM.
If it's open and noone is working on it, I would like to implement it.
@axw @graphaelli

@axw
Copy link
Member Author

axw commented Nov 14, 2022

@ekrucio nobody else is working on it. I see you have opened a PR, thank you - I'll take a look shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants