-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Filebeat: Elasticsearch server fileset: Add paths for Mac OSX and Windows #8891
Filebeat: Elasticsearch server fileset: Add paths for Mac OSX and Windows #8891
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Should we add a changelog for this?
For the OS X default path: Is that when installed with brew?
Sure, I'll add to this PR.
TBH, I don't know. It was the same path used in other |
jenkins, test this |
@ycombinator Can you follow up on this one after merging and see if the path is correct for OS X? |
Hey @radoondas In this PR I've set the Mac OSX logfile paths for the Looking at the commits on those other filesets, I see that you set the logfile paths on those. @ruflin and I were wondering how you chose |
34a8d7c
to
b046ffd
Compare
@ycombinator Thanks for fixing this. It looks like I did not check paths in modules's metricset. |
…dows (#8891) (#8937) Resolves #8887. This PR makes the `elasticsearch/server` fileset paths for Mac OSX and Windows consistent with other filesets' paths in the module. This also means that a user can now install Filebeat, enable the Elasticsearch module, and run Filebeat without any errors. (cherry picked from commit aec847a)
Resolves #8887.
This PR makes the
elasticsearch/server
fileset paths for Mac OSX and Windows consistent with other filesets' paths in the module. This also means that a user can now install Filebeat, enable the Elasticsearch module, and run Filebeat without any errors.