Skip to content

Increase the 30 seconds check #159

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

Closed
wants to merge 7 commits into from
Closed

Increase the 30 seconds check #159

wants to merge 7 commits into from

Conversation

shantanuo
Copy link

container fails to start when initializing the log file size takes more than 30 seconds to complete. Please increase this limit.

@yosifkit
Copy link
Member

yosifkit commented Apr 6, 2016

Perhaps we should just make it configurable through an environment variable passed in the docker run? I feel like most systems start quickly enough that we would rather fail fast and then allow a user to adjust if their system needs a little more time.

What do you think @ltangvald?

@ltangvald
Copy link
Collaborator

Yeah, passing it might be better. As you say it's usually enough, though I've also seen it fail on certain systems. Plus, 5.7 and 8.0 are slower to start up than the earlier versions.

@yosifkit
Copy link
Member

yosifkit commented Mar 4, 2020

Apologies for the delay 🙇‍♂️

Closing in favor of #646

@yosifkit yosifkit closed this Mar 4, 2020
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