Skip to content

Conversation

abel-von
Copy link
Contributor

@abel-von abel-von commented Mar 1, 2024

For sandboxed Task, we call the task api directly into the sandbox. For some kind of sandbox, we may have to update the resources of the sandbox before/after task creation or deletion, and it has to be done outside the sandbox, so we rely on the sandbox controller to update the resources of the sandbox.

@k8s-ci-robot
Copy link

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 /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions 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
Copy link
Contributor Author

abel-von commented Mar 1, 2024

needs rebase after #9736 #9853 #9882 #9883 #9884 #9903 #9904

@abel-von abel-von closed this Mar 1, 2024
@abel-von abel-von changed the title sandbox: add a new sandboxedTask type other than the shimTask sandbox: update sandbox resource before/after task creation/deletion Mar 1, 2024
@abel-von abel-von reopened this Mar 1, 2024
@abel-von abel-von mentioned this pull request Mar 1, 2024
19 tasks
@abel-von abel-von force-pushed the sandbox-task-0301 branch 4 times, most recently from b8115cd to 4dc8e31 Compare May 7, 2024 12:33
@abel-von abel-von force-pushed the sandbox-task-0301 branch from 4dc8e31 to 7fa479a Compare May 8, 2024 07:15
@abel-von abel-von force-pushed the sandbox-task-0301 branch from 7fa479a to 8fae650 Compare July 15, 2024 07:56
@abel-von abel-von force-pushed the sandbox-task-0301 branch from 8fae650 to 09004fd Compare July 15, 2024 08:01
@abel-von
Copy link
Contributor Author

/cc @mxpv @dmcgowan @fuweid

@k8s-ci-robot k8s-ci-robot requested review from dmcgowan, fuweid and mxpv July 15, 2024 08:11
@abel-von
Copy link
Contributor Author

Please take a look at the series of PRs including this PR and #9884 @9882, when you have time. I'd hope that maybe it can be merged into 2.1 if it is too rushed for 2.0. @fuweid @dmcgowan @mxpv

abel-von added 4 commits July 23, 2024 10:15
Signed-off-by: Abel Feng <fshb1988@gmail.com>
cri will call sandbox controller from the sandboxService, remove the
dependency of client.

Signed-off-by: Abel Feng <fshb1988@gmail.com>
there are two kinds of task, shimTask and sandboxedTask,
shimTask is the legacy implementation and compatible with the
podsandbox, the sandboxedTask is the task running in the sandbox created
by sandbox sontroller v2

Signed-off-by: Abel Feng <fshb1988@gmail.com>
Signed-off-by: Abel Feng <fshb1988@gmail.com>
@abel-von abel-von force-pushed the sandbox-task-0301 branch from 09004fd to eebcd60 Compare July 23, 2024 02:36
@abel-von
Copy link
Contributor Author

needs rebase after #9882 #9884

@k8s-ci-robot
Copy link

PR needs rebase.

Instructions 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-sigs/prow repository.

@AkihiroSuda AkihiroSuda added this to the 2.1 milestone Oct 21, 2024
Copy link

This PR is stale because it has been open 90 days with no activity. This PR will be closed in 7 days unless new comments are made or the stale label is removed.

@github-actions github-actions bot added the Stale label Jan 20, 2025
Copy link

This PR was closed because it has been stalled for 7 days with no activity.

@github-actions github-actions bot closed this Jan 27, 2025
@dmcgowan dmcgowan removed this from the 2.1 milestone Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants