Skip to content

travis: update image versions #1103

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

Merged
merged 5 commits into from
May 25, 2020
Merged

travis: update image versions #1103

merged 5 commits into from
May 25, 2020

Conversation

julienschmidt
Copy link
Member

@julienschmidt julienschmidt commented May 17, 2020

Description

Updates the images on Travis to latest versions.
For better comparability (and thus less hacks), it further changes the mysql_wait.sh script to use the mysql client inside the docker container if docker is running.

Checklist

  • Code compiles correctly
  • All tests passing

@julienschmidt julienschmidt added this to the v1.6.0 milestone May 17, 2020
@julienschmidt
Copy link
Member Author

julienschmidt commented May 17, 2020

Somehow the wait_mysql.sh waits forever with MySQL 8.0 on Xenial. Any idea what could be going wrong there @methane?

Job log: https://travis-ci.com/github/go-sql-driver/mysql/jobs/335790306

@julienschmidt
Copy link
Member Author

Found this very helpful hint, which we never implemented: #798 (comment)
Thanks for warning us already 2 years ago @arvenil 😇

@julienschmidt julienschmidt requested a review from methane May 24, 2020 18:43
@julienschmidt julienschmidt merged commit 8c3a2d9 into master May 25, 2020
@julienschmidt julienschmidt deleted the travis-update branch May 29, 2020 09:30
tz70s pushed a commit to tz70s/mysql that referenced this pull request Sep 5, 2020
* travis: update image versions

* travis: use mysql client inside the docker container for polling

* travis: force mysql client to connect via tcp

* travis: use root pass in mysql client

* travis: use empty root password
tz70s pushed a commit to tz70s/mysql that referenced this pull request Sep 5, 2020
* travis: update image versions

* travis: use mysql client inside the docker container for polling

* travis: force mysql client to connect via tcp

* travis: use root pass in mysql client

* travis: use empty root password
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