Update hpcloud/tail and fsnotify

[carlpett]

Fixes #5

This depends on a manual merge of fsnotify #205 in the vendor directory, which is pretty hacky. Asked for some attention on that PR.

[jryberg]

Was this never merged? Is it because it's not needed or something else?

// Johan

[carlpett]

Hi @jryberg,
This seems to have fallen of the radar a bit. I was hoping for the linked PR in fsnotify to be merged so this one would not be needed (it essentially "fakes" the merge, so we'd diverge from upstream).
However, that PR doesn't have much activity... Are you affected by #5?

[jryberg]

Yes, I'm affected. We have seen strange behaviour where it stops to read logs when the file is rotated in scenarios where two dockers share same file system and one of them rotates the log and the other are streaming the log.

[jryberg]

I have merged it in my own fork, just need to figure out how to build it =)

[carlpett]

Well, since the upstream merge doesn't seem to be happening, guess we'll merge it here, then.

[jryberg]

Thank you =)

Would you please include binaries as well?

[jryberg]

I managed to build it using go 1.12.5 and info from your Dockerfile.

However, one thing that puzzled me was I could not use the source code from the release. I had to use go get from github instead of the tar.gz.

I hope this can help someone =)

FROM golang:1.12 AS build

RUN go version && \
    go get -v -u github.com/kardianos/govendor && \
    go get -v -u github.com/carlpett/stream_exporter && \
    go install github.com/kardianos/govendor
WORKDIR /go/src/github.com/carlpett/stream_exporter
RUN govendor build +p

[carlpett]

Hm, I made a release (v0.2.2) which should have built binaries as well. Looks like one of the build tools (promu) has broken backwards compatibility.
I'm out travelling at the moment, but will try to fix it and make a "real" release.

[carlpett]

@jryberg Release now works!

[jryberg]

Thank you. I can also confirm that we had no hiccups with the latest version. Rotated logs works just fine now.

…

On Tue, May 21, 2019, 12:36 Calle Pettersson ***@***.***> wrote: @jryberg <https://github.com/jryberg> Release now works! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#8?email_source=notifications&email_token=AAGMOJ6JO3QBCIAFMMZ3SX3PWPGCZA5CNFSM4ESFZEDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV3PZBA#issuecomment-494337156>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAGMOJYAZK5SOECMLTQCCFTPWPGCZANCNFSM4ESFZEDA> .