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

Don't stop Metricbeat if aerospike server is down #6874

Merged
merged 1 commit into from Apr 17, 2018

Conversation

Projects
None yet
2 participants
@exekias
Copy link
Member

commented Apr 16, 2018

Ensure Aerospike module constructor doesn't return an error if it cannot
immediately connect to the server.

Don't stop Metricbeat if aerospike server is down
Ensure Aerospike module constructor doesn't return an error if it cannot
immediately connect to the server.

@exekias exekias force-pushed the exekias:fix-aerospike-constructor branch from 54d055d to 2545fc9 Apr 16, 2018

@exekias exekias added the review label Apr 16, 2018

@exekias

This comment has been minimized.

Copy link
Member Author

commented Apr 16, 2018

failing tests are not related

@ruflin ruflin merged commit fd15ea2 into elastic:master Apr 17, 2018

5 of 6 checks passed

beats-ci Build finished.
Details
CLA Commit author is a member of Elasticsearch
Details
Hound No violations found. Woof!
codecov/patch 60% of diff hit (within 100% threshold of 62.84%)
Details
codecov/project 62.89% (+0.04%) compared to e611c9d
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.