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

increase timeout for docker start #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

better0fdead
Copy link

Increased timeout for docker start from 180s to 600s.
Have done it because sometimes 180s is not enough for github macOs runners.
E.g run, run.

Increased timeout for docker start from 180s to 600s.
Have done it because sometimes 180s is not enough for github macOS runners.
@better0fdead better0fdead force-pushed the better0fdead/increase-timeout branch from a313c98 to a657972 Compare June 20, 2023 16:21
@calm-mlin
Copy link

cc @00Kai0 @junchen1992 @khs1994

Would be nice to have this merged, I'm hitting the same issue.

@diegorondini
Copy link

Yep, same issue.
Also reported in:
#34
and:
#24

@limpbrains
Copy link

Please merge this, having same issue

@stephenwade
Copy link

You can use this fix now with
uses: better0fdead/actions-setup-docker@better0fdead/increase-timeout

SaeedRe pushed a commit to Kynetics/hara-ddiclient-dev that referenced this pull request Dec 27, 2023
The macOS build was failing time to time due to an issue described here:
docker-practice/actions-setup-docker#29
This commit uses the fix provided in the github issue.

Signed-off-by: Saeed Rezaee <[email protected]>
@anitacaron
Copy link

Many users would be happy with this merge 🙌🏼

@diegorondini
Copy link

Probably even 300 would be enough. Often it takes just a bit more than 180 seconds:

 + '[' 184 -gt 600 ']'
+ /Applications/Docker.app/Contents/Resources/bin/docker system info
+ ((  i++ == 0  ))
+ printf .
+ command -v docker
./usr/local/bin/docker
+ sleep 1
+ '[' 185 -gt 600 ']'
+ /Applications/Docker.app/Contents/Resources/bin/docker system info
+ ((  i++ == 0  ))
+ printf .
+ command -v docker
./usr/local/bin/docker
+ sleep 1
+ '[' 186 -gt 600 ']'
+ /Applications/Docker.app/Contents/Resources/bin/docker system info
+ echo '::notice::-- Docker is ready.Wait time is 186 s'
+ uname -a
Notice: -- Docker is ready.Wait time is 186 s

@daniel-wong-dfinity-org

Very yes.

@calm-mlin
Copy link

calm-mlin commented Mar 7, 2024

ping @khs1994 , I see you merged some PRs recently.

@anitacaron
Copy link

Probably even 300 would be enough. Often it takes just a bit more than 180 seconds:

In my case, it takes 421s, so around 600 would be better.

+ '[' 421 -gt 600 ']'
  + /Applications/Docker.app/Contents/Resources/bin/docker system info
  + echo '::notice::-- Docker is ready.Wait time is 421 s'
  + uname -a

@limpbrains
Copy link

I've switched to https://github.com/douglascamata/setup-docker-macos-action because of this issue. Works good for me

diegorondini pushed a commit to eclipse-hara/hara-ddiclient that referenced this pull request May 21, 2024
Switch to a different action to install docker on Mac OS, the current
one seems to be problematic.
https://github.com/marketplace/actions/setup-docker-on-macos

Useful references about the issue:
actions/runner-images#17
docker-practice/actions-setup-docker#29

Signed-off-by: Diego Rondini <[email protected]>
diegorondini pushed a commit to eclipse-hara/hara-ddiclient that referenced this pull request May 21, 2024
Switch to a different action to install docker on Mac OS, the current
one seems to be problematic.
https://github.com/marketplace/actions/setup-docker-on-macos

Useful references about the issue:
actions/runner-images#17
docker-practice/actions-setup-docker#29

Signed-off-by: Diego Rondini <[email protected]>
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.

7 participants