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

Fix logstash output panic without hosts #2326

Merged
merged 1 commit into from
Aug 22, 2016

Conversation

urso
Copy link

@urso urso commented Aug 20, 2016

Resolves: #2325

Only logstash output was affected, due to wrong error variable being checked/returned thanks to scoping rules.

@urso urso added the review label Aug 20, 2016
- Fix logstash output configuration test if no hosts are configured
@ruflin
Copy link
Member

ruflin commented Aug 22, 2016

LGTM. I'm trying to fix LS build to make sure it all works before merging.

@ruflin
Copy link
Member

ruflin commented Aug 22, 2016

jenkins, retest it

1 similar comment
@ruflin
Copy link
Member

ruflin commented Aug 22, 2016

jenkins, retest it

@ruflin ruflin merged commit 0f104d0 into elastic:master Aug 22, 2016
@urso urso deleted the fix/2325-panic-no-hosts branch February 19, 2019 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants