Skip to content

Conversation

@ltangvald
Copy link
Collaborator

Daemonize lets us avoid a manual wait loop for the server to be ready, but
for 5.7 it causes a log redirection error when the image is started with -t,
so for now just revert 5.7 image to using the wait loop.

Daemonize lets us avoid a manual wait loop for the server to be ready, but
for 5.7 it causes a log redirection error when the image is started with -t,
so for now just revert 5.7 image to using the wait loop.
@tianon
Copy link
Member

tianon commented Oct 16, 2019

LGTM, but we do need to run ./update.sh and commit the copied result so that Travis tests the right thing 😅

@tianon tianon merged commit 75f81c8 into docker-library:master Oct 16, 2019
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Oct 16, 2019
Changes:

- docker-library/mysql@75f81c8: Merge pull request docker-library/mysql#601 from ltangvald/fixtty
- docker-library/mysql@6659750: Apply update.sh
- docker-library/mysql@4efd127: Revert to not using daemonize flag for temporary startup of 5.7 server
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.

2 participants