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

Initial support for dogstatsd events sent to statsd #1

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

Conversation

schahal
Copy link
Owner

@schahal schahal commented Jan 17, 2019

Related to influxdata#5224 - Add support for dogstatsd events

@arnoldyahad
Copy link

Hey!
Thanks for making this fork, i want to use it but im having trouble trying to "make" from your fork:

go build -ldflags " -X main.commit=ccfd9ca -X main.branch=master" ./cmd/telegraf
cmd/telegraf/telegraf.go:18:2: use of internal package not allowed
cmd/telegraf/telegraf.go:19:2: use of internal package not allowed
make[1]: *** [telegraf] Error 1
make: *** [all] Error 2

@schahal
Copy link
Owner Author

schahal commented Feb 22, 2019

@arnoldyahad make sure to try to clone this repo inside $GOPATH/src/github.com/influxdata... so:

$ cd $GOPATH/src/github.com/influxdata
$ git clone https://github.com/schahal/telegraf.git
...
$ git checkout feature/dogstatsd-events
...
$ make
dep ensure -vendor-only
go build -ldflags " -X main.commit=2d642c8c -X main.branch=feature/dogstatsd-events" ./cmd/telegraf
$ ls -l telegraf
-rwxr-xr-x  <redacted>  102212488 Feb 22 11:17 telegraf

As long as this fork is cloned in that path it should work (the packages in the code wont match the github URL of this fork)

Please note: you may have to install some packages manually first using go get (if it's not already done), not via dep. We have an open discussion (influxdata#5224) where in the datadog package (heavy!) creates conflicts with telegraf module dependencies, and the final version of this PR will remove that :)

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 this pull request may close these issues.

2 participants