WIP: sandbox: add "sandboxedTask" in addition to shimTask#9574
WIP: sandbox: add "sandboxedTask" in addition to shimTask#9574abel-von wants to merge 7 commits intocontainerd:mainfrom
Conversation
|
Hi @abel-von. Thanks for your PR. I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
abel-von
force-pushed
the
sandbox-task-rebased
branch
from
baff001 to
15eba7d
Compare
and add some functions in client.Sandbox, and prefer using it to call sandbox apis in CRI plugins Signed-off-by: Abel Feng <[email protected]>
Signed-off-by: Abel Feng <[email protected]>
Signed-off-by: Abel Feng <[email protected]>
Signed-off-by: Abel Feng <[email protected]>
abel-von
force-pushed
the
sandbox-task-rebased
branch
from
849eced to
b34de4c
Compare
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
The codes has been splitted into several PRs, close this one. |
4 participants
If the sandbox provides task API itself, shim management is not required anymore, and shimTask is no longer suitable for the task type provided by the sandbox. So I'm adding a "sandboxedTask" type here, and separating the two types can effectively reduce code complexity.