Bug 1282441

Summary: [SR-IOV] - UI clusters less than 3.6 - 'passthrough' checkbox appears in vNIC profile dialog and 'pci-passthrough' type is part of the type list in the vNIC's dialog
Product: [oVirt] ovirt-engine Reporter: Michael Burman <mburman>
Component: BLL.NetworkAssignee: Dan Kenigsberg <danken>
Status: CLOSED CURRENTRELEASE QA Contact: Michael Burman <mburman>
Severity: medium Docs Contact:
Priority: medium    
Version: 3.6.0.2CC: bugs, danken, ylavi
Target Milestone: ovirt-3.6.2Flags: rule-engine: ovirt-3.6.z+
ylavi: planning_ack+
rule-engine: devel_ack+
rule-engine: testing_ack+
Target Release: 3.6.2.5   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-02-18 11:04:35 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:

Description Michael Burman 2015-11-16 12:28:18 UTC 
Description of problem:
[SR-IOV] - UI  clusters less than 3.6 - 'passthrough' checkbox appears in vNIC profile dialog and 'pci-passthrough' type is part of the type list in the vNIC's dialog.

- Compatibility version- UI clusters less than 3.6 - sriov checkboxes(passthrough) should be disabled in the vNIC profile dialog and shouldn't appear in the vNICs type list(pci-passthrouhg)? 

- In current version 3.6.0.3-0.1.el6 in clusters 3.5, 'passthrough' checkbox appears in vNIC profile dialog and 'pci-passthrough' type is part of the type list in the VM vNIC's dialog. This should be disabled or shouldn't be exist at all for clusters below 3.6.

Version-Release number of selected component (if applicable):
3.6.0.3-0.1.el6

How reproducible:
100
Comment 1 Red Hat Bugzilla Rules Engine 2015-11-16 13:40:37 UTC 
This bug is not marked for z-stream, yet the milestone is for a z-stream version, therefore the milestone has been reset.
Please set the correct milestone or add the z-stream flag.
Comment 2 Michael Burman 2016-01-11 13:17:32 UTC 
After discussing with DEV, we have 2 issues here:

1) Clusters less then 3.6 - Agreed that 'passthrough' checkbox should be disabled. 
2) 'pci-passthrough' type is part of the type list in a vNIC dialog in clusters less then 3.6 - Agreed that the current validation is enough -->
There is a validation for the 'passthrough' profile and type for cluster version less then 3.6 and approving such operation is not supported. Appropriate tool tip is appear to explain the reason. 

- Separate bug will be reported for such validations in REST API.
Comment 3 Michael Burman 2016-01-13 16:37:23 UTC 
Verified on - 3.6.2.5-0.1.el6