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

Wait for registry to exist on tests #4862

Merged
merged 1 commit into from Aug 10, 2017

Conversation

Projects
None yet
3 participants
@ruflin
Copy link
Collaborator

commented Aug 10, 2017

Quite a few of our tests fail because the registry does not exist yet when checked. This PR changes the get_registry function to wait until the registry exists and then return it. This should make some of the tests more stable.

Wait for registry to exist on tests
Quite a few of our tests fail because the registry does not exist yet when checked. This PR changes the get_registry function to wait until the registry exists and then return it. This should make some of the tests more stable.
@tsg

tsg approved these changes Aug 10, 2017

Copy link
Collaborator

left a comment

LGTM.

@kvch

kvch approved these changes Aug 10, 2017

@kvch kvch merged commit cba395a into elastic:master Aug 10, 2017

6 checks passed

CLA Commit author has signed the CLA
Details
beats-ci Build finished.
Details
codecov/patch Coverage not affected when comparing a1031e8...3d5d7d5
Details
codecov/project 62.47% (+4.15%) compared to a1031e8
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

ramon-garcia added a commit to ramon-garcia/beats that referenced this pull request Dec 5, 2017

Wait for registry to exist on tests (elastic#4862)
Quite a few of our tests fail because the registry does not exist yet when checked. This PR changes the get_registry function to wait until the registry exists and then return it. This should make some of the tests more stable.

athom added a commit to athom/beats that referenced this pull request Jan 25, 2018

Wait for registry to exist on tests (elastic#4862)
Quite a few of our tests fail because the registry does not exist yet when checked. This PR changes the get_registry function to wait until the registry exists and then return it. This should make some of the tests more stable.
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.