Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1438658

Summary: [REST] - Create pass_through vnic profile failed
Product: [oVirt] ovirt-engine Reporter: Michael Burman <mburman>
Component: BLL.NetworkAssignee: Ales Musil <amusil>
Status: CLOSED CURRENTRELEASE QA Contact: Meni Yakove <myakove>
Severity: high Docs Contact:
Priority: high    
Version: futureCC: bugs, danken, obensass
Target Milestone: ovirt-4.2.0Keywords: Automation, Regression
Target Release: ---Flags: rule-engine: ovirt-4.2+
rule-engine: blocker+
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-08-30 08:20:39 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 2017-04-04 05:30:37 UTC
Description of problem:
[REST] - Create pass_through vnic profile failed.

On 4.2 create vnic profile with pass_through failed via rest api, but succeed via the UI.


POST network-ge-1.scl.lab.tlv.redhat.com/ovirt-engine/api/vnicprofiles

<vnic_profile>
    <name>C2_general_sriov_vnic1</name>
    <pass_through>
        <mode>enabled</mode>
    </pass_through>
    <network id="42f2b9c9-243c-4588-84c9-60a42adf8a2a"/>
</vnic_profile>


error:
<fault>
<detail>[Cannot add VM network interface profile. 'Port Mirroring,'Qos' and 'Network Filter' are not supported on passthrough profiles.]</detail>
<reason>Operation Failed</reason>
</fault>

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

How reproducible:
100

Steps to Reproduce:
1. Try to create a passthrough vNIC profile via rest

Actual results:
Failed

Expected results:
Should work

Comment 1 Red Hat Bugzilla Rules Engine 2017-04-05 07:38:20 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 Dan Kenigsberg 2017-08-06 11:55:06 UTC
I think that the original bug had to be solved by http://gerrit.ovirt.org/76093 but now Ori reports a new breakage

Comment 3 Ori Ben Sasson 2017-08-06 12:05:43 UTC
We get an error:
<fault>
<detail>VnicProfile [name] required for validateParameters</detail>
<reason>Incomplete parameters</reason>
</fault>

Comment 4 Ales Musil 2017-08-30 07:43:09 UTC
Hi Ori,
did you use the xml which is in the example? Because I can't reproduce your problem with the example xml.

Comment 5 Michael Burman 2017-08-30 08:20:39 UTC
(In reply to Ales Musil from comment #4)
> Hi Ori,
> did you use the xml which is in the example? Because I can't reproduce your
> problem with the example xml.

Hi Ales, 

I have tested latest master - 4.2.0-0.0.master.20170828065003.git0619c76.el7.centos and it is now working as expected.

POST - https://mburman-4-test-rpms.scl.lab.tlv.redhat.com/ovirt-engine/api/vnicprofiles

<vnic_profile>
    <name>C2_general_sriov_vnic2</name>
    <pass_through>
        <mode>enabled</mode>
    </pass_through>
    <network id="64b5a564-c102-4278-90ce-d54b143f30f7"/>
</vnic_profile>

<vnic_profilehref="/ovirt-engine/api/vnicprofiles/0e7cec3c-d6b4-47a3-ad26-7678ce0f8449"id="0e7cec3c-d6b4-47a3-ad26-7678ce0f8449">
<name>C2_general_sriov_vnic2</name>
<linkhref="/ovirt-engine/api/vnicprofiles/0e7cec3c-d6b4-47a3-ad26-7678ce0f8449/permissions"rel="permissions"/>
<migratable>false</migratable>
<pass_through>
<mode>enabled</mode>
</pass_through>
<port_mirroring>false</port_mirroring>
<networkhref="/ovirt-engine/api/networks/64b5a564-c102-4278-90ce-d54b143f30f7"id="64b5a564-c102-4278-90ce-d54b143f30f7"/>
</vnic_profile>