Bug 1279375 - Reintroduce checks for online CPUs for cpu pinning
Summary: Reintroduce checks for online CPUs for cpu pinning
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backend.Core
Version: 3.6.0
Hardware: ppc64
OS: Linux
high
medium
Target Milestone: ovirt-3.6.3
: 3.6.3.2
Assignee: Roman Mohr
QA Contact: Artyom
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-11-09 10:17 UTC by Roman Mohr
Modified: 2016-03-11 07:22 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-03-11 07:22:50 UTC
oVirt Team: SLA
Embargoed:
rule-engine: ovirt-3.6.z+
rule-engine: exception+
mgoldboi: planning_ack+
rgolan: devel_ack+
mavital: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 46151 0 master MERGED scheduler: Add cpu pinning policy unit 2016-01-12 13:20:00 UTC
oVirt gerrit 52933 0 ovirt-engine-3.6 MERGED scheduler: Add cpu pinning policy unit 2016-02-07 13:21:20 UTC
oVirt gerrit 53207 0 ovirt-engine-3.6.3 MERGED scheduler: Add cpu pinning policy unit 2016-02-10 09:26:09 UTC

Description Roman Mohr 2015-11-09 10:17:27 UTC
Description of problem:
Since the problem of reporting online cpu cores on PPC seems to be fixed (see Bug 1148092) validation of cpu pinning should be reintroduced.

Version-Release number of selected component (if applicable):


How reproducible:
Add a VM and pin virtual cpu core 1 to non existent hot cpu thread 10000000.

Steps to Reproduce:
1. Create VM
2. Add a cpu pinning string like 1#1000000
3. Obviously the host cpu thread 1000000 does not exist, but no warning is issued and when you try to start the VM, vdsm fails with an unexpected error.

Actual results:
Currently only syntactic correctness of the cpu pinning string is guaranteed.
No cpu policy or configuration checks are hindering you from assigning virtual cpu cores to non existent host cpu threads or cores.

Expected results:

1) Do a soft check when configuring VMs and warn the user about possible invalid cpu pinnings.
2) Add a policy unit which checks the host before starting the VM there and remove it from the list if it does not provide the specified pinning requirements.

Additional info:

VDSM currently cashes the host capabilities (which include online CPUs). We need to update the capabilities when online CPU status change.

Comment 1 Red Hat Bugzilla Rules Engine 2015-11-09 10:27:33 UTC
This bug is not marked for z-stream, yet the milestone is for a z-stream version, therefore the milestone has been reset.
Please set the correct milestone or add the z-stream flag.

Comment 2 Red Hat Bugzilla Rules Engine 2015-11-10 08:34:36 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 3 Red Hat Bugzilla Rules Engine 2015-11-10 08:46:18 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 4 Artyom 2016-02-21 13:30:19 UTC
Verified on rhevm-backend-3.6.3.2-0.1.el6.noarch
Error while executing action: 

golden_env_mixed_virtio_1_0:
Cannot run VM. There is no host that satisfies current scheduling constraints. See below for details:
The host host_mixed_1 did not satisfy internal filter CpuPinning because missing online cpu core(s) 31.
The host host_mixed_2 did not satisfy internal filter PinToHost.


Note You need to log in before you can comment on or make changes to this bug.