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

[17.03.x] vendor: update containerd to 977c511ed #31274

Merged
merged 1 commit into from Feb 23, 2017

Conversation

Projects
None yet
4 participants
@tonistiigi
Member

tonistiigi commented Feb 23, 2017

vendor: update containerd to 977c511ed
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>

@tonistiigi tonistiigi added this to the 17.03.0 milestone Feb 23, 2017

@vieux vieux self-assigned this Feb 23, 2017

@thaJeztah

This comment has been minimized.

Show comment
Hide comment
@thaJeztah

thaJeztah Feb 23, 2017

Member

one failing test on Windows;

00:55:10 ----------------------------------------------------------------------
00:55:10 FAIL: docker_api_logs_test.go:15: DockerSuite.TestLogsAPIWithStdout
00:55:10 
00:55:10 docker_api_logs_test.go:50:
00:55:10     c.Fatal("timeout waiting for logs to exit")
00:55:10 ... Error: timeout waiting for logs to exit
00:55:10 
00:55:11 
00:55:11 ----------------------------------------------------------------------
Member

thaJeztah commented Feb 23, 2017

one failing test on Windows;

00:55:10 ----------------------------------------------------------------------
00:55:10 FAIL: docker_api_logs_test.go:15: DockerSuite.TestLogsAPIWithStdout
00:55:10 
00:55:10 docker_api_logs_test.go:50:
00:55:10     c.Fatal("timeout waiting for logs to exit")
00:55:10 ... Error: timeout waiting for logs to exit
00:55:10 
00:55:11 
00:55:11 ----------------------------------------------------------------------
@vieux

This comment has been minimized.

Show comment
Hide comment
@vieux

vieux Feb 23, 2017

Collaborator

@tonistiigi @johnstep do you think it's related ?

Collaborator

vieux commented Feb 23, 2017

@tonistiigi @johnstep do you think it's related ?

@tonistiigi

This comment has been minimized.

Show comment
Hide comment
@tonistiigi

tonistiigi Feb 23, 2017

Member

@vieux doubt it, but I don't remember seeing this test fail before

Member

tonistiigi commented Feb 23, 2017

@vieux doubt it, but I don't remember seeing this test fail before

@vieux

This comment has been minimized.

Show comment
Hide comment
@vieux

vieux Feb 23, 2017

Collaborator

LGTM

Collaborator

vieux commented Feb 23, 2017

LGTM

@vieux vieux merged commit d06b03a into moby:17.03.x Feb 23, 2017

5 checks passed

dco-signed All commits are signed
experimental Jenkins build Docker-PRs-experimental 31050 has succeeded
Details
janky Jenkins build Docker-PRs 39662 has succeeded
Details
vendor Jenkins build Docker-PRs-vendor 2911 has succeeded
Details
windowsRS1 Jenkins build Docker-PRs-WoW-RS1 10744 has succeeded
Details

@thaJeztah thaJeztah added this to PRs in 17.03.2-maybe Feb 28, 2017

@thaJeztah thaJeztah removed this from PRs in 17.03.2-maybe Mar 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment