Bug 1954878
Summary: | [RFE] Auto Pinning Policy: improve tooltip description and policy names | ||
---|---|---|---|
Product: | Red Hat Enterprise Virtualization Manager | Reporter: | Germano Veit Michel <gveitmic> |
Component: | ovirt-engine | Assignee: | Liran Rotenberg <lrotenbe> |
Status: | CLOSED ERRATA | QA Contact: | Polina <pagranat> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | 4.4.6 | CC: | ahadas, emarcus, lrotenbe, michal.skrivanek, mtessun, sigbjorn.lie |
Target Milestone: | ovirt-4.4.7 | Keywords: | FutureFeature |
Target Release: | 4.4.7 | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | ovirt-engine-4.4.7.1 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2021-07-22 15:12:33 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | Virt | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1963681 |
Description
Germano Veit Michel
2021-04-29 00:39:43 UTC
Germano, this is what expected from the 'adjust' policy (it adjusts the CPU and NUMA topologies to the host). If you wish to preserve the current topologies, the 'existing' policy should be used (In reply to Arik from comment #1) > Germano, this is what expected from the 'adjust' policy (it adjusts the CPU > and NUMA topologies to the host). > If you wish to preserve the current topologies, the 'existing' policy should > be used does it need a (i) tooltip perhaps? Sounds like the Adjust policy did what it suppose to. (In reply to Michal Skrivanek from comment #2) > (In reply to Arik from comment #1) > > Germano, this is what expected from the 'adjust' policy (it adjusts the CPU > > and NUMA topologies to the host). > > If you wish to preserve the current topologies, the 'existing' policy should > > be used > > does it need a (i) tooltip perhaps? There is one. OK, thanks! I see now what 'Adjust' really means. However the fact that both the customer and I got this wrong means we have to improve it :) The current tooltip is: autoPinningLabelExplanation=VM pinned to a host can use the policies for automatic CPU and NUMA pinning. Existing keeps the current CPU topology. Adjust will also maximize the CPU topology according to the host. By default High Performance VM will be set with Existing policy. Assuming I got this right, may I suggest an improvement? "VM pinned to a host can use the policies for automatic CPU and NUMA pinning. 'Do Not Change' keeps the current vCPU and vNUMA node count, along with the pinning configuration. 'Existing' keeps the current vCPU and vNUMA node count of the VM and pins them to pCPU and pNUMA of the host. 'Adjust' resizes the vCPU and vNUMA counts of the Virtual Machine to match the ones of the Host, and also pins them to the Host resources. By default High Performance VM will be set with Existing policy, but if manual pinning configuration exists it will be kept instead. We could perhaps also rename them, maybe something like? 'Do not change' -> 'Existing' 'Existing' -> 'Pin' 'Adjust' -> 'Resize and Pin' Does this make more sense? (In reply to Arik from comment #1) > If you wish to preserve the current topologies, the 'existing' policy should be used It still changes the vNUMA count of the VM, but keeps the vCPU count... Is it a bug? (In reply to Germano Veit Michel from comment #4) > Assuming I got this right, may I suggest an improvement? > "VM pinned to a host can use the policies for automatic CPU and NUMA > pinning. 'Do Not Change' keeps the current vCPU and vNUMA node count, along > with the pinning configuration. 'Existing' keeps the current vCPU and vNUMA > node count of the VM and pins them to pCPU and pNUMA of the host. 'Adjust' > resizes the vCPU and vNUMA counts of the Virtual Machine to match the ones > of the Host, and also pins them to the Host resources. By default High > Performance VM will be set with Existing policy, but if manual pinning > configuration exists it will be kept instead. As for the very last part ("but if manual pinning configuration exists it will be kept instead"), I'd say that then the selected auto pinning policy should be the one that doesn't make any change (what is 'Do not change' today and 'Existing' in your suggestion below). > > We could perhaps also rename them, maybe something like? > 'Do not change' -> 'Existing' > 'Existing' -> 'Pin' > 'Adjust' -> 'Resize and Pin' > > Does this make more sense? I like the idea of changing 'Existing' to 'Pin' and 'Adjust' to 'Resize and Pin' I see why you propose 'Existing' for the first policy (as "use existing settings") but I think it's not that clear when it appears as the description of auto-pinning policy. Maybe something like 'Do nothing' or 'None' would be better? As discussed, opened RFE for 'Existing' to keep the vNUMA count: https://bugzilla.redhat.com/show_bug.cgi?id=1957526 Maybe instead of closing, let's use this BZ to track the changes in the tooltip and renaming, as we are discussing? (In reply to Arik from comment #10) > I like the idea of changing 'Existing' to 'Pin' and 'Adjust' to 'Resize and > Pin' > I see why you propose 'Existing' for the first policy (as "use existing > settings") but I think it's not that clear when it appears as the > description of auto-pinning policy. Maybe something like 'Do nothing' or > 'None' would be better? Yup, perhaps 'None' is even better. So we get 'None', 'Pin' and 'Resize and Pin'? And the tooltip like: "VM pinned to a host can use the policies for automatic CPU and NUMA pinning. 'None' keeps the current vCPU and vNUMA node count, along with the pinning configuration. 'Pin' keeps the current vCPU count of the VM and pins them to pCPU and pNUMA of the host, expanding the vNUMA count of the VM to match the one of the Host. 'Resize and Pin' resizes the vCPU and vNUMA counts of the Virtual Machine to match the ones of the Host, and also pins them to the Host resources. By default High Performance VM will be set with Existing policy, but if manual pinning configuration exists it will be kept instead. It will become more coherent once BZ1957526 is implemented, then 'Pin' will not resize anything (vNUMA). Although renaming the `Existing` to `Pin`, the tooltip won't show the `Pin` policy and the High Performance parts. They are currently blocked until BZ 1957551 is resolved. Verifying the bug on ovirt-engine-4.4.7.6-0.11.el8ev.noarch In this version we only have two options - 'None' or 'Resize and Pin'. Choice for 'Resize and Pin' brings the tooltip msg: CPU pinning topology will be lost The current configuration of the VM does not allow cpu pinning. The pinning topology will be lost when the VM is saved. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (Moderate: RHV Manager (ovirt-engine) security update [ovirt-4.4.7]), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2021:2865 |