Add cgroupV2 CPUQuotaPeriodUSec support

Open austinvazquez opened this issue 7 months ago • 5 comments

Addresses https://github.com/containerd/cgroups/issues/365

This change adds CPUQuotaPeriodUSec property for cgroupsv2. Heavily influenced by github.com/opencontainers/cgroups. Refactors a bit of the existing functions for testability.

Added some unit tests for validating the property is properly set.

May 19 '25 22:05 austinvazquez

Looks like might need to resolve CI running on Ubuntu 20 runners, but IDK if there are any tests we can add for this property?

May 19 '25 22:05 austinvazquez

I was able to add some unit tests for the property and CI is now only failing due to Ubuntu 20 runner no longer being available from GitHub.

May 20 '25 17:05 austinvazquez

Needs #368

May 24 '25 18:05 austinvazquez

I rebased this for the CI updates. It should be good to review now.

Jun 04 '25 15:06 austinvazquez

@containerd/maintainers Checking if we can merge this PR. Thank You

Jun 09 '25 20:06 Shubhranshu153

@containerd/maintainers Can we merge this one? Thank You

Jun 30 '25 20:06 Shubhranshu153