Bug 1394555

Summary: [UI] 'passthough' should be disabled in Create/edit vNIC profile dialog for external network
Product: [oVirt] ovirt-engine Reporter: Mor <mkalfon>
Component: BLL.NetworkAssignee: Marcin Mirecki <mmirecki>
Status: CLOSED CURRENTRELEASE QA Contact: Meni Yakove <myakove>
Severity: medium Docs Contact:
Priority: low    
Version: 4.1.0CC: bugs, danken
Target Milestone: ovirt-4.2.0Flags: rule-engine: ovirt-4.2+
rule-engine: planning_ack+
rule-engine: devel_ack+
rule-engine: testing_ack+
Target Release: 4.2.0   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-12-20 11:06:47 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1366899    
Attachments:
Description Flags
fixed window none

Description Mor 2016-11-13 13:25:48 UTC 
Description of problem:
1. UI window of creating/editing vNIC profile, which is attached to external network, has logic problem. When checking and unchecking the "Passthrough" checkbox on this window, greyed-out option "Port Mirroring" becomes available for user selection. 

2. All other unsupported options of external networks in that window, should be greyed-out.

Version-Release number of selected component (if applicable):
oVirt Engine Version: 4.1.0-0.0.master.20161112231308.git672bd31.el7.centos

How reproducible:
100%

Steps to Reproduce:
1. Create a new external network by selecting name and external provider.
2. vNIC profile attached to that network should be created automatically.
3. Edit the vNIC profile.
4. Check/Uncheck the "passthrough" checkbox.

Actual results:
"Port-Mirroring" option become available for selection.


Expected results:
"Port-Mirroring" should stay greyed out.

Additional info:
Comment 1 Red Hat Bugzilla Rules Engine 2016-11-13 13:25:54 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 2 Marcin Mirecki 2017-05-26 13:03:35 UTC 
Title of bug says: 'passthough' should be disabled
Bug says: "Port-Mirroring" should stay greyed out.

I'm disabling both passthrough and portMirroring for external networks.
Comment 3 Mor 2017-06-05 10:56:46 UTC 
Verified on: 4.2.0-0.0.master.20170531203202.git1bf6667.el7.centos.
Comment 4 Mor 2017-06-05 10:58:02 UTC 
Created attachment 1285016 [details]
fixed window

Port Mirror is greyed out
Comment 5 Mor 2017-06-05 10:59:07 UTC 
(In reply to Mor from comment #4)
> Created attachment 1285016 [details]
> fixed window
> 
> Port Mirror is greyed out

* Passthrough
Comment 6 Sandro Bonazzola 2017-12-20 11:06:47 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.