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

Consider moving this package to OTel-Go-Contrib #3

Closed
jmacd opened this issue Apr 8, 2020 · 2 comments · Fixed by open-telemetry/opentelemetry-go-contrib#22
Closed

Comments

@jmacd
Copy link

jmacd commented Apr 8, 2020

This is a nice library for the contrib repository:
https://github.com/open-telemetry/opentelemetry-go-contrib

@marwan-at-work
Copy link
Owner

marwan-at-work commented Apr 8, 2020

@jmacd I'd be happy to move it there, I have a couple of questions:

  1. How should the directory structure be? For example: <repo>/exporters/datadog?
  2. Should each exporter be its own sub-module (has its own go.mod file), or its own repository?
  3. I imagine github.com/open-telemetry/opentelemetry-go-contrib/datadog-exporter is pretty long for a module name, anyway we can make it shorter like using the vanity server that already exists for OT? Something like contrib.opentelemetry.io/go ?

@jmacd
Copy link
Author

jmacd commented Apr 17, 2020

I've already committed a exporters/metric/dogstatsd, which Chris is evaluating. You might add exporters/metric/datadog, I suggest.

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

Successfully merging a pull request may close this issue.

2 participants