-
Notifications
You must be signed in to change notification settings - Fork 195
CI failing due to hot plug CPUs #250
Comments
this patch is to ensure `nproc` is executed until the last vCPU has been connected fixes kata-containers#250 Signed-off-by: Julio Montes <[email protected]>
ensure `nproc` is executed until the last vCPU has been connected fixes kata-containers#250 Signed-off-by: Julio Montes <[email protected]>
ensure `nproc` is executed until the last vCPU has been connected fixes kata-containers#250 Signed-off-by: Julio Montes <[email protected]>
Seems like this issue is more complex than I thought, now that our kernel supports |
when the container is created, it is placed inside a CPU cgroup with the actual number of vCPUs, hence not all vCPU will be available for the containers when they are created. For more info see kata-containers#250. fixes kata-containers#252 Signed-off-by: Julio Montes <[email protected]>
I think we should revisit the user experience to calculate CPU numbers. E.g., the tests uses
which seems to require 3 vCPUs to me but we are actually plugging 4 vCPUs to the guest whereas libcontainer only represents 3 of them to the container process. |
@bergwolf we always start the containers with 1 vCPU (unless something different is specified in the configuration file), if take a look https://github.com/clearcontainers/runtime/blob/master/docs/cpu-constraints.md
|
once all vCPUs have been connected, the agent updates the cpuset cgroup of each container allowing them to access to these resources Depends-on: github.com/kata-containers/agent#220 fixes kata-containers#250 Signed-off-by: Julio Montes <[email protected]>
once all vCPUs have been connected, the agent updates the cpuset cgroup of each container allowing them to access to these resources fixes kata-containers#250 Signed-off-by: Julio Montes <[email protected]>
Summarizing 4 Failures:
[Fail] Hot plug CPUs container with CPU period and quota [It] should have 3 CPUs
/home/jenkins/jenkins_slave/workspace/kata-containers-tests-ubuntu-17-10-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:91
[Fail] Hot plug CPUs container with CPU period and quota [It] should have 4 CPUs
/home/jenkins/jenkins_slave/workspace/kata-containers-tests-ubuntu-17-10-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:91
[Fail] Hot plug CPUs container with CPU constraint [It] should have 3 CPUs
/home/jenkins/jenkins_slave/workspace/kata-containers-tests-ubuntu-17-10-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:108
[Fail] Hot plug CPUs container with CPU constraint [It] should have 3 CPUs
/home/jenkins/jenkins_slave/workspace/kata-containers-tests-ubuntu-17-10-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:108
Ran 98 of 159 Specs in 543.348 seconds
FAIL! -- 94 Passed | 4 Failed | 0 Pending | 61 Skipped --- FAIL: TestIntegration (567.07s)
FAIL
The text was updated successfully, but these errors were encountered: