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 docker command timeout #591

Merged
merged 2 commits into from
Oct 31, 2023
Merged

Increase docker command timeout #591

merged 2 commits into from
Oct 31, 2023

Conversation

davidfowl
Copy link
Member

Docker goes to into resource saver mode after 5 minutes of not running a container (by default). While in this mode, the commands we use for the docker check take quite some time (11+ seconds on my laptop). Increasing the timeout for 25 seconds instead.

Read more here:

https://docs.docker.com/desktop/use-desktop/resource-saver/

Docker goes to into resource saver mode after 5 minutes of not running a container (by default). While in this mode, the commands we use for the docker check take quite some time (11+ seconds on my laptop). Increasing the timeout for 25 seconds instead.

Read more here:

https://docs.docker.com/desktop/use-desktop/resource-saver/
@davidfowl davidfowl merged commit ceff200 into main Oct 31, 2023
4 checks passed
@davidfowl davidfowl deleted the davidfowl/increase-timeout branch October 31, 2023 20:42
@davidfowl
Copy link
Member Author

/backport to release/8.0-preview1

Copy link
Contributor

Started backporting to release/8.0-preview1: https://github.com/dotnet/aspire/actions/runs/6712252537

Copy link
Contributor

@davidfowl backporting to release/8.0-preview1 failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Patch format detection failed.
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

Copy link
Contributor

@davidfowl an error occurred while backporting to release/8.0-preview1, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.

@davidfowl
Copy link
Member Author

Why does this bot suck 😄

davidfowl added a commit that referenced this pull request Oct 31, 2023
* Increase docker command timeout

Docker goes to into resource saver mode after 5 minutes of not running a container (by default). While in this mode, the commands we use for the docker check take quite some time (11+ seconds on my laptop). Increasing the timeout for 25 seconds instead.

Read more here:

https://docs.docker.com/desktop/use-desktop/resource-saver/

* Update src/Aspire.Hosting/Dcp/DcpHostService.cs

Co-authored-by: James Newton-King <[email protected]>

---------

Co-authored-by: James Newton-King <[email protected]>
davidfowl added a commit that referenced this pull request Nov 1, 2023
* Increase docker command timeout

Docker goes to into resource saver mode after 5 minutes of not running a container (by default). While in this mode, the commands we use for the docker check take quite some time (11+ seconds on my laptop). Increasing the timeout for 25 seconds instead.

Read more here:

https://docs.docker.com/desktop/use-desktop/resource-saver/

* Update src/Aspire.Hosting/Dcp/DcpHostService.cs



---------

Co-authored-by: James Newton-King <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators Apr 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants