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

Deploy Tool Hangs Indefinitely When Docker Hangs #786

Closed
ashovlin opened this issue Aug 24, 2023 · 3 comments
Closed

Deploy Tool Hangs Indefinitely When Docker Hangs #786

ashovlin opened this issue Aug 24, 2023 · 3 comments
Labels
bug This issue is a bug. p2 This is a standard priority issue queued

Comments

@ashovlin
Copy link
Member

ashovlin commented Aug 24, 2023

Describe the bug

I'm trying to deploy a new application, but the deploy tool is hanging indefinitely here:

Choose deployment option (recommended default: 1)
3

Name the Cloud Application to deploy your project to
--------------------------------------------------------------------------------
Enter the name of the new CloudFormationStack stack (default WebAppWithDockerFile):

Expected Behavior

Deployment to proceed

Current Behavior

Upon investigation, the docker info we're running is hanging indefinitely as well

await _commandLineWrapper.Run(
command,
streamOutputToInteractiveService: false,
onComplete: proc =>
{
processExitCode = proc.ExitCode;
containerType = RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ?
proc.StandardOut?.TrimEnd('\n') ??
throw new DockerInfoException(DeployToolErrorCode.FailedToCheckDockerInfo, "Failed to check if Docker is running in Windows or Linux container mode.") :
"linux";
});

Reproduction Steps

Unsure why my Docker installation is stuck, it seems like Docker can hang for multiple reasons: docker/cli#3433

It resolved itself once I did a factory reset of Docker Desktop.

Possible Solution

The deploy tool should be more resilient (set timeout on the child processes?) and display a helpful error message. For both docker info and node --version, I'd think we expect these to complete within a couple seconds.

Additional Information/Context

No response

Version used

1.14.6

Operating System and version

Windows 10

@ashovlin ashovlin added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Aug 24, 2023
@ashishdhingra ashishdhingra removed the needs-triage This issue or PR still needs to be triaged. label Aug 24, 2023
@ashishdhingra
Copy link
Contributor

ashishdhingra commented Aug 24, 2023

Needs review. Makes sense to have a timeout (might be configurable via parameter) for the child processes to return.

@ashishdhingra ashishdhingra added needs-review p2 This is a standard priority issue queued and removed needs-review labels Aug 24, 2023
@ashovlin
Copy link
Member Author

ashovlin commented Jul 8, 2024

Fixed in https://github.com/aws/aws-dotnet-deploy/releases/tag/1.21.13 via #837, we now timeout at a minute for the Docker and Node checks.

@ashovlin ashovlin closed this as completed Jul 8, 2024
Copy link

github-actions bot commented Jul 8, 2024

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. p2 This is a standard priority issue queued
Projects
None yet
Development

No branches or pull requests

2 participants