search

hoonmin / influxdb-collectd-proxy

A very simple proxy between collectd and influxdb.
MIT License
72 stars 26 forks source link

Make does not work with influxdb v0.8.8 #25

Open joelgriffiths opened 9 years ago

joelgriffiths commented 9 years ago

I was able to get it built a few days ago, but am unable to build it now no matter what I do. Does anybody have any ideas? root@influxdb-02:~/influxdb-collectd-proxy/influxdb-collectd-proxy# GOPATH=:pwd go version 'go version go1.4.1 linux/amd64 root@influxdb-02:~/influxdb-collectd-proxy/influxdb-collectd-proxy# make GOPATH=:pwd go get github.com/paulhammond/gocollectd GOPATH=:pwd go get github.com/influxdb/influxdb/client GOPATH=:pwd go build -o bin/influxdb-collectd-proxy

_/root/influxdb-collectd-proxy/influxdb-collectd-proxy

./influxdb-collectd-proxy.go:107: undefined: client.ClientConfig ./influxdb-collectd-proxy.go:129: undefined: client.Series ./influxdb-collectd-proxy.go:138: client.WriteSeries undefined (type client.Client has no field or method WriteSeries) ./influxdb-collectd-proxy.go:144: undefined: client.Series ./influxdb-collectd-proxy.go:151: undefined: client.Series ./influxdb-collectd-proxy.go:156: undefined: client.Series ./influxdb-collectd-proxy.go:246: undefined: client.Series make: ** [build] Error 2

dblooman commented 9 years ago

Also experiencing this issue, any work around?

joelgriffiths commented 9 years ago

I believe influxdb/client has changed. They removed WriteSeries. I'm trying to find a release that isn't broken. If you find one, post it here so others can benefit.

yanfali commented 9 years ago

Can we use godeps and lock down the commit hash of the upstream project?

joelgriffiths commented 9 years ago

Actually, after it fails, or you do a 'make get'

Do this: cd src/github.com/influxdb/influxdb git checkout v0.8.7

make build

Then you're g2g.

yanfali commented 9 years ago

OK, pull request #26 may help with this. Let's see what @hoonmin thinks is the best solution.

aranw commented 9 years ago

Anyone got a fix for this? Trying to setup this proxy but am getting this error.

hoonmin commented 9 years ago

Oh, sorry I'm late... I think Godep is the answer and @yanfali 's PR #26 looks good for me. Thank you all!

yanfali commented 9 years ago

@hoonmin should you close this now?

ndinhphi commented 9 years ago

@joelgriffiths thank you so much, you saved my time!

MengchenTony commented 9 years ago

Hello Any support planned for the influxdb client v0.9.x ? I had to upgrade to influx database server 0.9 and cannot use the proxy anymore... When doing a 'go build' with influxdb client last version having the same compilation error as mentioned above.

./influxdb-collectd-proxy.go:107: undefined: client.ClientConfig ./influxdb-collectd-proxy.go:129: undefined: client.Series ./influxdb-collectd-proxy.go:138: client.WriteSeries undefined (type client.Client has no field or method WriteSeries) ./influxdb-collectd-proxy.go:144: undefined: client.Series ./influxdb-collectd-proxy.go:151: undefined: client.Series ./influxdb-collectd-proxy.go:156: undefined: client.Series ./influxdb-collectd-proxy.go:246: undefined: client.Series

Thanks for the help