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

Datadog exporter to use service_name from Resource #574

Closed
lzchen opened this issue Jul 9, 2021 · 10 comments
Closed

Datadog exporter to use service_name from Resource #574

lzchen opened this issue Jul 9, 2021 · 10 comments
Labels
bug Something isn't working

Comments

@lzchen
Copy link
Contributor

lzchen commented Jul 9, 2021

From this comment

@lzchen lzchen added the bug Something isn't working label Jul 9, 2021
@ericmustin
Copy link
Contributor

Please assign thanks! 😊

@github-actions
Copy link

github-actions bot commented Aug 9, 2021

This issue was marked stale due to lack of activity. It will be closed in 30 days.

@lzchen
Copy link
Contributor Author

lzchen commented Aug 10, 2021

@ericmustin
Are you still working on this?

@ericmustin
Copy link
Contributor

@lzchen ah, ty for the ping. I'll push something up here this week, should be quick. Lost track of it after the original pr sorry about that

@lzchen lzchen removed the backlog label Aug 10, 2021
@github-actions
Copy link

This issue was marked stale due to lack of activity. It will be closed in 30 days.

@lzchen
Copy link
Contributor Author

lzchen commented Sep 13, 2021

@ericmustin
Any plans on working on this?

@ericmustin
Copy link
Contributor

@lzchen Hey, sorry. I had some time scheduled for this but then I uh, quit my employer. So, unfort i have a bunch of $newjob stuff to do and this isnt a huge priority for me. @majorgreys Would probably be the right person here for all things Datadog moving forward, or at least, he can put you in touch with the appropriate ppl. fwiw i think there are plans to simply deprecate this exporter as the datadog-agent now accepts otlp natively (experimental), but i'll defer on that to @majorgreys

@lzchen
Copy link
Contributor Author

lzchen commented Sep 13, 2021

@ericmustin

Thanks for the reply. I'll unassign you for the time being.

@owais
Copy link
Contributor

owais commented Oct 10, 2021

@majorgreys @ericmustin if the plan is to deprecate it, should we open a ticket to track that? You might want to add a deprecation notice and release once more and after that remove the package from the repo.

@srikanthccv
Copy link
Member

See #900 and #1366

@srikanthccv srikanthccv closed this as not planned Won't fix, can't repro, duplicate, stale Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants