Bug 1291723

Summary: Old network SetupNetwork command doesn't update bridge opts and ethtool opts of custom properties in NetworkAttachment
Product: [oVirt] ovirt-engine Reporter: GenadiC <gcheresh>
Component: BLL.NetworkAssignee: Yevgeny Zaspitsky <yzaspits>
Status: CLOSED INSUFFICIENT_DATA QA Contact: Meni Yakove <myakove>
Severity: medium Docs Contact:
Priority: high    
Version: 3.6.0CC: bugs, danken, gcheresh, ylavi
Target Milestone: ovirt-3.6.5Flags: rule-engine: ovirt-3.6.z+
ylavi: blocker+
ylavi: planning_ack+
danken: devel_ack+
myakove: testing_ack+
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-02-07 11:42:49 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:
Attachments:
Description Flags
engine and vdsm logs none

Description GenadiC 2015-12-15 13:49:32 UTC
Description of problem:
We run automation test that uses the old setupNetwork and try to update the custom properties (bridge opts and ethtool opts).
After update succeeds checking on host shows the correct (updated) values, but  Rest doesn't show the correct values after update

Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. We run automation test to use old SN (test 13)
2. After updating the custom properties check the API (ethtool and bridge opts)
3.

Actual results:
The values in API weren't changed

Expected results:
After update of custom properties the values should change to the new values

Additional info:
Using new setupNetwork works fine

In order to run test13 of network_custom_properties feature you need to add to __init__ of the test to create network with bridge and ethtool opts

Comment 1 Meni Yakove 2015-12-15 19:27:33 UTC
Please add logs

Comment 2 GenadiC 2016-01-25 15:44:12 UTC
Created attachment 1118081 [details]
engine and vdsm logs

Comment 3 Yevgeny Zaspitsky 2016-02-04 09:35:00 UTC
Please add actual http calls you used (both for updating and for retrieving the info).

Comment 4 Dan Kenigsberg 2016-02-07 11:42:49 UTC
We fail to reproduce this issue; however we've identified two related bug 1304787 and bug 1304784 regarding custom properties and the old setup network command.

Comment 5 GenadiC 2016-02-22 12:28:20 UTC
Can't reproduce anymore