Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

Add support for Datadog APM tracing #189

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

goodspark
Copy link

Addresses #188

Ran weaver and DD agent locally (had to disable Xvfb, but that's mostly orthogonal to this) and hit the index and stats endpoints a couple times to verify traces were flowing to my Datadog account:

screenshot from 2019-02-10 14-46-50

@goodspark
Copy link
Author

@sanohin Ping

@sanohin
Copy link
Contributor

sanohin commented Apr 11, 2019

Hi, thank you for your contribution.
I'm not sure what Datadog is, I want to check whether we need it in the repository.
I am afraid of adding integrations with more services as this would lead to growing the codebase whenever we need to support a new one.

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

Successfully merging this pull request may close these issues.

2 participants