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

Use takis again #706

Closed
wants to merge 2 commits into from
Closed

Use takis again #706

wants to merge 2 commits into from

Conversation

sushantdhiman
Copy link
Collaborator

@sushantdhiman sushantdhiman commented Jan 12, 2018

I noticed some builds are hanging forever in this pr. Reverted back to using takis again, which I think is more reliable.

If there is any reason for not using takis, please let me know

@sidorares
Copy link
Owner

takis is at tcp level, while wait-up script allows to wait until you can actually connect to database. Without that some test randomly fail if they start after docker is up but before server finished initialisation

@sidorares
Copy link
Owner

not sure why it was hanging, I did re-run once and it worked

@dougwilson
Copy link
Collaborator

I have been having Travis CI jobs hanging in various projects randomly for the past week. It may have been just something wrong with the Travis job itself (maybe some kind of infra problem?). I have to keep restarting jobs every so often because they'll hang forever.

@sushantdhiman
Copy link
Collaborator Author

Good point, we should use both takis and wait-up

@sidorares
Copy link
Owner

why both then?

@sushantdhiman
Copy link
Collaborator Author

Because build keep pinging mysql server even if its not listening

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants