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

Filebeat: Add err check for config file read #3573

Merged
merged 1 commit into from Feb 10, 2017

Conversation

@breml
Copy link
Contributor

commented Feb 10, 2017

Fixes #3572

@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Feb 10, 2017

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run.

1 similar comment
@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Feb 10, 2017

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run.

@monicasarbu monicasarbu merged commit d0e20a9 into elastic:master Feb 10, 2017

3 checks passed

CLA Commit author has signed the CLA
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@breml breml deleted the breml:bug-err-check-additional-yaml branch Feb 13, 2017

@ruflin

This comment has been minimized.

Copy link
Collaborator

commented Feb 13, 2017

@breml Thanks for the fix.

@monicasarbu I think we should also have a changelog entry for this one?

monicasarbu added a commit to monicasarbu/beats that referenced this pull request Feb 13, 2017
monicasarbu added a commit to monicasarbu/beats that referenced this pull request Feb 13, 2017
@monicasarbu

This comment has been minimized.

Copy link
Contributor

commented Feb 13, 2017

I have opened a PR with the changelog changes: #3577

monicasarbu added a commit to monicasarbu/beats that referenced this pull request Feb 13, 2017
ruflin added a commit that referenced this pull request Feb 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.