Bug 905831 - [webadmin] Webadmin allows 'run on specific host' to be checked even if no specific host is chosen
Summary: [webadmin] Webadmin allows 'run on specific host' to be checked even if no sp...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: 3.2.0
Hardware: Unspecified
OS: Unspecified
medium
unspecified
Target Milestone: ---
: 3.2.0
Assignee: ofri
QA Contact: Ido Begun
URL:
Whiteboard: sla
Depends On:
Blocks: 922807 948448
TreeView+ depends on / blocked
 
Reported: 2013-01-30 09:27 UTC by Ido Begun
Modified: 2016-02-10 20:14 UTC (History)
13 users (show)

Fixed In Version: sf13.1
Doc Type: Release Note
Doc Text:
Previously, users could set the "Do Not Migrate" parameter on a virtual machine without defining a specific host to run the virtual machine, so the virtual machine could still be migrated. Now, when these conditions are met, the virtual machine's CPU will be pinned on the first host on which it runs. The virtual machine will not be migrated to other hosts.
Clone Of:
Environment:
Last Closed:
oVirt Team: SLA
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 12706 0 None None None Never
oVirt gerrit 13546 0 None None None Never

Description Ido Begun 2013-01-30 09:27:47 UTC 
Description of problem:
It's possible to check the "Run VM on the selected host (no migration allowed)" even if no specific host is chosen (i.e. the "Run on Any Host in Cluster" is checked).

Version-Release number of selected component (if applicable):
rhevm-3.2.0-5.el6ev

How reproducible:
100%

Steps to Reproduce:
1. Edit VM -> Host tab
2. Check the "Run on Any host in Cluster" radio box
3. Check the "Allow VM migration only upon Administrator specific request (system will not trigger automatic migration of this VM)" box then uncheck it

  
Actual results:
The "Run VM on the selected host (no migration allowed)" can be checked and no error is given when the OK button is clicked.

Expected results:
The "Run VM on the selected host (no migration allowed)" should be grayed out.

Additional info:
Comment 7 ofri 2013-03-27 13:17:17 UTC 
after discussion - the logic should be:
A = Pin to host
B = use host cpu

For RHEVM:
                              Any                 Selected
                              Host                   Host
-------------------------------------------------------------
Allow auto migration            
Allow Man. migration
Do not allow migration        B                      A+B



For oVirt:
                              Any                 Selected
                              Host                   Host
-------------------------------------------------------------
Allow auto migration          A                       A  
Allow Man. migration          A                       A
Do not allow migration       A+B                     A+B
Comment 8 ofri 2013-03-27 13:45:44 UTC 
Fixed:

after discussion - the logic should be:
A = cpu pinning
B = use host cpu

For RHEV:
                              Any                 Selected
                              Host                   Host
-------------------------------------------------------------
Allow auto migration                                    
Allow Man. migration                                   
Do not allow migration        B                      A+B



For oVirt:
                              Any                 Selected
                              Host                   Host
-------------------------------------------------------------
Allow auto migration                                  A  
Allow Man. migration                                  A
Do not allow migration        B                      A+B
Comment 9 ofri 2013-04-03 08:17:27 UTC 
Fixed:

A = cpu pinning
B = use host cpu
* = will be available on 3.2 - deprecated from 3.3 onward
** = will be available from 3.3 (currently blocked on Run-VM)

For RHEV:
                              Any                 Selected
                              Host                   Host
-------------------------------------------------------------
Allow auto migration                                    
Allow Man. migration           B*                    
Do not allow migration         B**                   A+B



For oVirt:
                              Any                 Selected
                              Host                   Host
-------------------------------------------------------------
Allow auto migration                                  A  
Allow Man. migration           B*                     A
Do not allow migration         B**                   A+B
Comment 10 Ido Begun 2013-04-14 11:59:17 UTC 
Tested on SF13.1:

UI behaves as it should according to Comment#9.
Comment 11 Itamar Heim 2013-06-11 08:48:09 UTC 
3.2 has been released
Comment 12 Itamar Heim 2013-06-11 08:48:14 UTC 
3.2 has been released
Comment 13 Itamar Heim 2013-06-11 08:48:22 UTC 
3.2 has been released
Comment 14 Itamar Heim 2013-06-11 08:53:21 UTC 
3.2 has been released
Comment 15 Itamar Heim 2013-06-11 09:24:01 UTC 
3.2 has been released
Note You need to log in before you can comment on or make changes to this bug.