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

bugfix: specify both cmd and its args when create a container #1027

Merged
merged 1 commit into from
Apr 3, 2018

Conversation

YaoZengzeng
Copy link
Contributor

@YaoZengzeng YaoZengzeng commented Apr 2, 2018

Signed-off-by: YaoZengzeng [email protected]

Ⅰ. Describe what this PR did

Actually I forget to specify the args of the cmd executed in the container, so fill it in this PR.

Ⅱ. 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/M labels Apr 2, 2018
@YaoZengzeng YaoZengzeng force-pushed the projected branch 2 times, most recently from bb1d7b1 to 9982898 Compare April 2, 2018 04:58
@codecov-io
Copy link

codecov-io commented Apr 2, 2018

Codecov Report

Merging #1027 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1027   +/-   ##
=======================================
  Coverage   14.21%   14.21%           
=======================================
  Files         127      127           
  Lines        8474     8474           
=======================================
  Hits         1205     1205           
  Misses       7169     7169           
  Partials      100      100
Impacted Files Coverage Δ
daemon/mgr/cri.go 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ec48e96...9cd17c6. Read the comment docs.

@pouchrobot pouchrobot added size/XS and removed size/M labels Apr 2, 2018
@YaoZengzeng YaoZengzeng changed the title bugfix: specify both cmd and its args when create a container, don't return error if we can't get metadata from SandboxStore bugfix: specify both cmd and its args when create a container Apr 2, 2018
@allencloud
Copy link
Collaborator

Please provide more details about this pull request in pull request description. @YaoZengzeng
Thanks a lot.

@YaoZengzeng
Copy link
Contributor Author

@allencloud PTAL

@YaoZengzeng YaoZengzeng force-pushed the projected branch 3 times, most recently from 5adc900 to 9f714fd Compare April 2, 2018 07:00
@YaoZengzeng
Copy link
Contributor Author

With this PR, the node-e2e test result will be:

Ran 186 of 258 Specs in 1501.814 seconds
FAIL! -- 146 Passed | 40 Failed | 0 Pending | 72 Skipped

exciting 😄

@rudyfly
Copy link
Collaborator

rudyfly commented Apr 2, 2018

LGTM

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Apr 2, 2018
@allencloud allencloud merged commit 9f5f182 into AliyunContainerService:master Apr 3, 2018
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.

5 participants