-
Notifications
You must be signed in to change notification settings - Fork 950
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
bugfix: specify both cmd and its args when create a container #1027
Conversation
bb1d7b1
to
9982898
Compare
Codecov Report
@@ Coverage Diff @@
## master #1027 +/- ##
=======================================
Coverage 14.21% 14.21%
=======================================
Files 127 127
Lines 8474 8474
=======================================
Hits 1205 1205
Misses 7169 7169
Partials 100 100
Continue to review full report at Codecov.
|
Please provide more details about this pull request in pull request description. @YaoZengzeng |
@allencloud PTAL |
5adc900
to
9f714fd
Compare
Signed-off-by: YaoZengzeng <[email protected]>
With this PR, the node-e2e test result will be: Ran 186 of 258 Specs in 1501.814 seconds exciting 😄 |
LGTM |
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