-
Notifications
You must be signed in to change notification settings - Fork 314
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
runtime-proxy: support cri v1 api #1052
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #1052 +/- ##
==========================================
+ Coverage 67.47% 67.51% +0.04%
==========================================
Files 251 252 +1
Lines 28435 28397 -38
==========================================
- Hits 19186 19173 -13
+ Misses 7924 7893 -31
- Partials 1325 1331 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
How to deal with community API changes, how does our version correspond to the community version, any suggestions? @FillZpp |
Signed-off-by: maao <maao420691301@gmail.com>
/hold cancel |
Referring to kubelet's approach(1.23), we need to design a compatibility solution, first detect whether v1 is supported, and fall back to v1alpha2 if it is not supported |
IMO,
Do you mean that the logic of runtime-proxy should be like this?
please tell me if i missed something.
|
Sorry, I have a little trouble understanding. I think cri server should be consistent with containerd/dockerd:
|
I will continue to work on this pr. @maaoBit |
/close |
@hormes: Closed this PR. In response to this:
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. |
Ⅰ. Describe what this PR does
add cri v1 api to koord-runtime-proxy
Ⅱ. Does this pull request fix one issue?
fix #1051
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
RuntimeResourceExecutor
move up to cri v1 apiRuntimeResourceExecutor
by converting v1alpha2 object to v1 objectV. Checklist
make test