cri: fix integration test on cgroupsv2 system#6595
Conversation
|
Hi @zhsj. 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. |
|
Build succeeded.
|
|
Build succeeded.
|
|
Build succeeded.
|
TestUpdateContainerResources_MemoryLimit still uses old method to check memlimit in cgroup. Signed-off-by: Shengjing Zhu <[email protected]>
We have cgroups v2 testing happening in 2 places, both by running a nested VM running Fedora 35:
|
|
Build succeeded.
|
Yes, but only integration test(code in integration/client) has been run. CRI integration is not. |
|
/ok-to-test |
two cri integration buckets are available 1) kubernetes-sigs/cri-tools .. critest and 2) containerd/script/test/cri-integration.sh Don't think we've tried containerd/script integration yet with v2 enabled in fedora.. Also not sure how much of CGroupsV2 is being tested right now in the k8s prow driven node e2e bucket. They've had some features enable now so I would expect some buckets to be there. |
https://github.com/containerd/containerd/blob/main/.github/workflows/ci.yml#L397 Only on ubuntu-18.04. I'm not sure it should be added to vagrant based cgroupsv2 CI. It may become more flaky... |
nod it wasn't written to run in a vagrant vm.. vague memory of some restart tests that would/may require a different design.. |
Forwarded: containerd/containerd#6595 Gbp-Pq: Name 0007-cri-fix-integration-test-on-cgroupsv2-system.patch
TestUpdateContainerResources_MemoryLimit still uses old method to
check memlimit in cgroup.
BTW, should we enable cgroupsv2 integration test in CI?