Skip to content

Conversation

@shaloulcy
Copy link
Contributor

Signed-off-by: Eric Li [email protected]

Ⅰ. Describe what this PR did

restart_paused_container ci failed

FAIL: /go/src/github.com/alibaba/pouch/test/cli_restart_test.go:63: PouchRestartSuite.TestPouchRestartPausedContainer
/go/src/github.com/alibaba/pouch/test/cli_restart_test.go:68:
    command.PouchRun("pause", name).Assert(c, icmd.Success)
/go/src/github.com/alibaba/pouch/vendor/github.com/gotestyourself/gotestyourself/icmd/command.go:61:
    t.Fatalf("at %s:%d - %s\n", filepath.Base(file), line, err.Error())
... Error: at cli_restart_test.go:68 - 
Command:  /usr/local/bin/pouch pause TestPouchRestartPausedContainer
ExitCode: 1
Error:    exit status 1
Stdout:   
Stderr:   Error: failed to pause container TestPouchRestartPausedContainer: {"message":"container's status is not running: exited"}

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@pouchrobot pouchrobot added areas/test kind/bug This is bug report for project size/XS labels May 4, 2018
@codecov-io
Copy link

Codecov Report

Merging #1272 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1272   +/-   ##
=======================================
  Coverage   15.21%   15.21%           
=======================================
  Files         172      172           
  Lines       10735    10735           
=======================================
  Hits         1633     1633           
  Misses       8981     8981           
  Partials      121      121

@HusterWan
Copy link
Contributor

LGTM

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label May 4, 2018
@HusterWan HusterWan merged commit ebec621 into AliyunContainerService:master May 4, 2018
@HusterWan
Copy link
Contributor

@shaloulcy https://github.com/alibaba/pouch/blob/master/CONTRIBUTING.md

commit message format please check the CONTRIBUTING guidance, thanks a lot.

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

Labels

areas/test kind/bug This is bug report for project LGTM one maintainer or community participant agrees to merge the pull reuqest. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants