rgannu / burrow-graphite

A simple graphite exporter for gathering Kafka consumer group info from [burrow](https://github.com/linkedin/Burrow).
Apache License 2.0
1 stars 5 forks source link

Compilation Error (it couldn't be builded) #1

Open ujlbu4 opened 5 years ago

ujlbu4 commented 5 years ago

Hi, trying to build docker image, but it's not compiling:

Looks like late commit (https://github.com/rgannu/burrow-graphite/commit/7567da329961eefff9498c59cf7a1c1ed06bf278) broken it.

Full log:

docker-compose -f docker-compose.yml build   
Building burrow-graphite
Step 1/10 : FROM golang
latest: Pulling from library/golang
741437d97401: Pull complete
34d8874714d7: Pull complete
0a108aa26679: Pull complete
7f0334c36886: Pull complete
d35724ed4672: Pull complete
e75ac00960df: Pull complete
08686fc4280e: Pull complete
Digest: sha256:e967bad29bc5aee87532869894d38ab5c932ec7f67e6cfd6664d7de9a8f5bed3
Status: Downloaded newer image for golang:latest
 ---> 901414995ecd
Step 2/10 : COPY . /go/src/github.com/rgannu/burrow-graphite
 ---> 4e6cdbfee625
Step 3/10 : WORKDIR /go/src/github.com/rgannu/burrow-graphite
 ---> Running in 4f60a8191d7b
Removing intermediate container 4f60a8191d7b
 ---> e95e875ee8da
Step 4/10 : RUN go get ./
 ---> Running in f1ebfa095141
# github.com/rgannu/burrow-graphite/burrow_graphite
burrow_graphite/exporter.go:31:18: missing len argument to make([]graphite.Metric)
burrow_graphite/exporter.go:40:47: not enough arguments in call to graphite.NewMetric
    have (string, string)
    want (string, string, int64)
burrow_graphite/metrics.go:26:57: undefined: Metric
burrow_graphite/metrics.go:27:18: graphite.sendMetrics undefined (cannot refer to unexported field or method graphite.(*Graphite).sendMetrics)
ERROR: Service 'burrow-graphite' failed to build: The command '/bin/sh -c go get ./' returned a non-zero code: 2

Thanks

ericychoi commented 5 years ago

+1 here. I think it needs /vendor

mrh666 commented 5 years ago

Same issue: `root@mon:/tmp # git clone https://github.com/rgannu/burrow-graphite.git Cloning into 'burrow-graphite'... remote: Enumerating objects: 6, done. remote: Counting objects: 100% (6/6), done. remote: Compressing objects: 100% (6/6), done. remote: Total 24 (delta 0), reused 2 (delta 0), pack-reused 18 Unpacking objects: 100% (24/24), done. Checking connectivity... done.

root@mon:/tmp # cd burrow-graphite/

root@mon:/tmp/burrow-graphite # docker build . -t burrow-graphite:2019052801 Sending build context to Docker daemon 133.1kB Step 1/10 : FROM golang ---> 7ced090ee82e Step 2/10 : COPY . /go/src/github.com/rgannu/burrow-graphite ---> d840716fcaba Removing intermediate container 906231a20acd Step 3/10 : WORKDIR /go/src/github.com/rgannu/burrow-graphite ---> 4b8e8c6a81b2 Removing intermediate container 7071b16e5927 Step 4/10 : RUN go get ./ ---> Running in 728d971fa501

burrow_graphite/exporter.go:31:18: missing len argument to make([]graphite.Metric) burrow_graphite/exporter.go:40:47: not enough arguments in call to graphite.NewMetric have (string, string) want (string, string, int64) burrow_graphite/metrics.go:26:57: undefined: Metric burrow_graphite/metrics.go:27:18: graphite.sendMetrics undefined (cannot refer to unexported field or method graphite.(*Graphite).sendMetrics) The command '/bin/sh -c go get ./' returned a non-zero code: 2`

Is anyone figured out how to work it around?

aGallea commented 5 years ago

opened PR for compilation fixes