Skip to content

Fix macOS default log path for elasticsearch module based on brew paths#8939

Merged
radoondas merged 2 commits intoelastic:masterfrom
radoondas:es-module-osx-paths-fix
Nov 7, 2018
Merged

Fix macOS default log path for elasticsearch module based on brew paths#8939
radoondas merged 2 commits intoelastic:masterfrom
radoondas:es-module-osx-paths-fix

Conversation

@radoondas
Copy link
Contributor

MacOS default logs paths were not correct and we decided to have them equal to homebrew installation.

@radoondas radoondas requested a review from ycombinator November 6, 2018 07:48
@ycombinator
Copy link
Contributor

jenkins, test this

Copy link
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending green CI.

@ycombinator
Copy link
Contributor

jenkins, test this

4 similar comments
@ycombinator
Copy link
Contributor

jenkins, test this

@ycombinator
Copy link
Contributor

jenkins, test this

@ycombinator
Copy link
Contributor

jenkins, test this

@ycombinator
Copy link
Contributor

jenkins, test this

@radoondas radoondas merged commit 3f07231 into elastic:master Nov 7, 2018
radoondas added a commit to radoondas/beats that referenced this pull request Nov 7, 2018
…hs (elastic#8939)

* Fix macOS default log path for elasticsearch module based on homebrew paths.

* Add changelog record for macOS default path fix.

(cherry picked from commit 3f07231)
radoondas added a commit that referenced this pull request Nov 8, 2018
…hs (#8939) (#8978)

* Fix macOS default log path for elasticsearch module based on homebrew paths.

* Add changelog record for macOS default path fix.

(cherry picked from commit 3f07231)
@radoondas radoondas deleted the es-module-osx-paths-fix branch December 2, 2018 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants