-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[sandbox] refactor controller interface #7981
[sandbox] refactor controller interface #7981
Conversation
b3bf31a
to
91b951d
Compare
91b951d
to
65bfae2
Compare
ea1ec75
to
a1dc3ff
Compare
Update the sandbox controller interface to use local types rather than using the API types. Signed-off-by: Derek McGowan <[email protected]>
a1dc3ff
to
54d498e
Compare
Add options to sandbox controller interface. Update sandbox controller interface to fully utilize sandbox controller interface. Move grpc error conversion to service. Signed-off-by: Derek McGowan <[email protected]>
Signed-off-by: Derek McGowan <[email protected]>
54d498e
to
3431471
Compare
Signed-off-by: Derek McGowan <[email protected]>
f3b3659
to
b0e97c0
Compare
Finally figured out and fixed the issue with the cleanup error. Now it uses multierror rather than checking for a non-nil empty object to signal there was an issue cleaning up. Marking as ready to review. |
/test pull-containerd-sandboxed-node-e2e |
1 similar comment
/test pull-containerd-sandboxed-node-e2e |
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.
LGTM
Avoid using API types in containerd interface. Move local sandbox controller plugin under the plugins package.