Bug 1486547 - [UI] - 'Passthrough' should be disabled if the vNIC profile is attached and used by a VM
Summary: [UI] - 'Passthrough' should be disabled if the vNIC profile is attached and u...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Network
Version: 4.2.0
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ovirt-4.2.0
: ---
Assignee: Ales Musil
QA Contact: Michael Burman
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-08-30 06:34 UTC by Michael Burman
Modified: 2017-12-20 10:54 UTC (History)
1 user (show)

Fixed In Version: ovirt-engine-4.2.0-0.0.master.20170901193740.git7900511.el7.centos.noarch.rpm
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-12-20 10:54:27 UTC
oVirt Team: Network
Embargoed:
rule-engine: ovirt-4.2+
rule-engine: blocker+


Attachments (Terms of Use)
record (490.31 KB, application/x-gzip)
2017-08-30 06:34 UTC, Michael Burman
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 81202 0 master MERGED webadmin: Add check for the previous state of portMirroring and passthrough 2017-08-30 18:42:08 UTC

Description Michael Burman 2017-08-30 06:34:09 UTC
Created attachment 1319864 [details]
record

Description of problem:
[UI] - 'Passthrough' should be disabled if the vNIC profile is attached and used by a VM.

The 'passthrough' property shouldn't be editable if it is attached and used by a VM. 
There should be a tooltip as well saying that:
"'Passthrough' cannot be changed if the vNIC Profile is used by a VM"
No matter if the 'passthroguh' property is enabled or not, once used by VM it must be grayed out until detached from the VM. 

Version-Release number of selected component (if applicable):
4.2.0-0.0.master.20170828065003.git0619c76.el7.centos

How reproducible:
100%

Steps to Reproduce:
1. Create network without 'passthrough' vNIC profile
2. Create network with 'passthrough' vNIC profile
3. Add 2 vNICs to a VM with those profiles

Actual results:
In both cases the 'passthorugh' is editable although can't be really changed - 
If trying to approve such situation we get:
"Error while executing action: Cannot edit VM network interface profile. This VM network interface profile is used by virtual machines (5): VM1,"

Expected results:
If the vNIC is in use by a VM, the 'passthrough' property must be grayed out.

Comment 1 Red Hat Bugzilla Rules Engine 2017-08-30 07:25:53 UTC
This bug report has Keywords: Regression or TestBlocker.
Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP.

Comment 2 Michael Burman 2017-09-03 05:49:46 UTC
Verified on - 4.2.0-0.0.master.20170901193740.git7900511.el7.centos

Comment 3 Sandro Bonazzola 2017-12-20 10:54:27 UTC
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


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