-
Notifications
You must be signed in to change notification settings - Fork 40k
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
docs/rkt: Document the different UX. #21221
Conversation
Labelling this PR as size/L |
GCE e2e test build/test passed for commit da8415c22e43abb41b4a7c698e98e88a75768274. |
GCE e2e test build/test passed for commit 2e7a9691455af5451c4f85964c2c0eec9e10a014. |
GCE e2e test build/test passed for commit aa011372b812425197b10dbbfcee04d4fabb71bc. |
|
||
### Doesn't support ENTRYPOINT + CMD feature | ||
|
||
To run a Docker image, rkt will transfer it into App Container Image (ACI) format first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/transfer/convert ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link to ACI format?
@yujuhong Addressed comments, ptla, thanks :) |
GCE e2e test build/test passed for commit 50fe8dce26e72acee321ba733941edc4dd1d633e. |
- 1000 | ||
``` | ||
|
||
All the three examples above are valid as they contains the full executable path, plus the arguments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/contains/contain
lgtm with some nits. /cc @dchen1107 |
@yujuhong Comments fixed. Thank you :) |
GCE e2e test build/test passed for commit b3ed1b6. |
Automatic merge from submit-queue |
Auto commit by PR queue bot
As for #20905 (comment)
cc @kubernetes/sig-node @jonboulle @sjpotter