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.

Sign up for GitHub

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

Jump to bottom

unit test failure: TestStartContainerTimeout #1352

Closed
sharanyad opened this issue Apr 18, 2018 · 2 comments
Closed

unit test failure: TestStartContainerTimeout #1352

sharanyad opened this issue Apr 18, 2018 · 2 comments
Assignees
@haikuoliu
Labels
kind/bug kind/testing
Milestone
1.18.0

Comments

@sharanyad
Copy link
Contributor

Summary

TestStartContainerTimeout fails intermittently

Expected Behavior

test passes

Observed Behavior

test fails intermittently

Environment Details

Appveyor

Supporting Log Snippets

--- FAIL: TestStartContainerTimeout (0.00s)
	<autogenerated>:1: missing call(s) to *mock_dockeriface.MockClient.StartContainerWithContext(is equal to id, is nil, is anything) c:/gopath/src/github.com/aws/amazon-ecs-agent/agent/dockerclient/dockerapi/docker_client_test.go:442
	<autogenerated>:1: aborting test due to missing call(s)
@sharanyad
Copy link
Contributor Author

Consider increasing the timeout value used in tests?

@haikuoliu
Copy link
Contributor

This is fixed in #1348

@haikuoliu haikuoliu added this to the 1.18.0 milestone May 16, 2018
@haikuoliu haikuoliu self-assigned this May 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@haikuoliu haikuoliu

Labels
kind/bug kind/testing
Projects
None yet
Milestone
1.18.0
Development

No branches or pull requests
3 participants
@adnxn @sharanyad @haikuoliu