Bug 905831

Summary:	[webadmin] Webadmin allows 'run on specific host' to be checked even if no specific host is chosen
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Ido Begun <ibegun>
Component:	ovirt-engine-webadmin-portal	Assignee:	ofri <omasad>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Ido Begun <ibegun>
Severity:	unspecified	Docs Contact:
Priority:	medium
Version:	3.2.0	CC:	acathrow, bdagan, chetan, dfediuck, dyasny, ecohen, gchaplik, iheim, omasad, oramraz, Rhev-m-bugs, sgrinber, ykaul
Target Milestone:	---
Target Release:	3.2.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	sla
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.	Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	SLA	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	922807, 948448

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