-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
telegraf build fails on SmartOS (illumos) #2899
Comments
Looks like an issue with fsnotify, can you try updating it to the latest version and seeing if the issue remains? |
@bahamat Did you open an issue with fsnotify? If so can you link to it here? |
@danielnelson I didn't, but it looks like fsnotify/fsnotify#12 pretty much means that fsnotify isn't going to work any time soon. |
What do you think about this pull request? You wouldn't be able to use the tail or logparser inputs but it seems better than nothing. BTW I modified the makefile recently so to cross compile you would do:
|
That doesn't seem to work for me.
|
Can you double check that you have the pull request code checked out? I ask because I see this |
Ok, yeah, explicitly checking out the fix-solaris-build branch does seem to work for me. |
I merged the workaround for 1.4, if you are interested in tail/logparser or if you run into any other bugs, please open a new issue. Those plugins will, of course, need an update to fsnotify to fix. |
Bug report
Relevant telegraf.conf:
System info:
Telegraf version: v1.4 (according to CHANGELOG.md)
operating system: SmartOS base-64 2017Q1 image
Go: go version go1.8 solaris/amd64
Steps to reproduce:
Expected behavior:
Successful build
Actual behavior:
The text was updated successfully, but these errors were encountered: