Use staging k8s.io/kubelet/cri/streaming package#9254
Conversation
|
Hi @adisky. 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. |
adisky
force-pushed
the
cri-streaming-from-k8s
branch
2 times, most recently
from
22a2277 to
ccd18e9
Compare
Use staging k8s.io/kubelet/cri/streaming package Signed-off-by: Aditi Sharma <[email protected]>
adisky
force-pushed
the
cri-streaming-from-k8s
branch
from
ccd18e9 to
03d81f5
Compare
|
Interesting (positive) impact on k8s dependency imports in the diffstat: Looks like we get to drop api and api-server from containerd. |
|
/ok-to-test |
…h upstream containerd/main update fork-external/main branch to upstream containerd/main at commit f90f80d Related work items: containerd#8736, containerd#8861, containerd#8924, containerd#8934, containerd#9027, containerd#9076, containerd#9104, containerd#9118, containerd#9141, containerd#9155, containerd#9177, containerd#9183, containerd#9184, containerd#9186, containerd#9187, containerd#9191, containerd#9200, containerd#9205, containerd#9211, containerd#9214, containerd#9215, containerd#9221, containerd#9223, containerd#9228, containerd#9231, containerd#9234, containerd#9242, containerd#9246, containerd#9247, containerd#9251, containerd#9253, containerd#9254, containerd#9255, containerd#9268
6 participants
cri streaming package from kubelet moved to k8 staging so that it can be used by container runtime
kubernetes/kubernetes#118253
This PR removes the containerd own package and use the K8s staging one