Skip to content
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

[Auditbeat] Cherry-pick #11667 to 7.0: Log error when Homebrew not installed #11948

Merged
merged 1 commit into from
Apr 30, 2019

Conversation

cwurm
Copy link
Contributor

@cwurm cwurm commented Apr 26, 2019

Cherry-pick of PR #11667 to 7.0 branch. Original message:

At the moment, if Auditbeat is run with the default configuration (where the package dataset is enabled) on a Mac where Homebrew is not installed it will fail to start.

This changes the behavior to only logging an error message:

2019-04-05T15:03:01.250+0100 ERROR [package] package/package.go:218 Failed to start package dataset on macOS. Error looking up /usr/local/Cellar - is Homebrew installed? Error: stat /usr/local/Cellar: no such file or directory

Change `package` dataset on macOS to logging an error when Homebrew is not installed instead of failing to start altogether. It will keep checking and start sending events as usual in case Homebrew is installed later.

(cherry picked from commit bbe4991)
@cwurm cwurm requested a review from a team as a code owner April 26, 2019 14:16
@cwurm cwurm changed the title Cherry-pick #11667 to 7.0: [Auditbeat] Log error when Homebrew not installed [Auditbeat] Cherry-pick #11667 to 7.0: Log error when Homebrew not installed Apr 26, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/secops

@cwurm cwurm requested a review from andrewkroh April 26, 2019 14:17
@cwurm cwurm merged commit d67ffc3 into elastic:7.0 Apr 30, 2019
@cwurm cwurm deleted the backport_11667_7.0 branch April 30, 2019 11:10
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.

3 participants