Bug 811540 - Backend: Non 3.1 clusters Failed to add network via REST API if doesn't include VM network config
Summary: Backend: Non 3.1 clusters Failed to add network via REST API if doesn't inclu...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-core
Version: unspecified
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
: 3.1
Assignee: Roy Golan
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-04-11 11:21 UTC by Avi Tal
Modified: 2016-04-22 04:59 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-09 07:56:29 UTC
oVirt Team: ---

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Avi Tal 2012-04-11 11:21:04 UTC 
Failed to add network to non 3.1 cluster if rest element doesn't include VM network configurations.

POST:
<network id='...'>
<name>testrest1</name>
<data_center id='...'/>
<stp>false</stp>
<status><state>non_operational</state></status>
</network>

response:
<fault>
    <reason>Operation Failed</reason>
    <detail>[Non VM networks is not supported for this for this (compatibility) version of Data Center.]</detail>
</fault>

Solution:
Should have default value of True for VM networks.

Note: this CanDoAction has a typo of duplicate "for this" 
[Non VM networks is not supported for this for this (compatibility) version of Data Center.]
Comment 1 Yaniv Kaul 2012-04-15 11:52:29 UTC 
Did it work before adding the 'VM network' feature? If it did, it's a regression - please mark it as such.
Comment 2 Ori Liel 2012-04-16 07:22:28 UTC 
The parameter should either be mandatory in REST-API, but only if business logic justifies this, or it should get a default value in the Backend - IMO
Comment 3 Roy Golan 2012-04-16 08:17:41 UTC 
http://gerrit.ovirt.org/#change,3556
Comment 4 Itamar Heim 2012-08-09 07:56:29 UTC 
closing ON_QA bugs as oVirt 3.1 was released:
http://www.ovirt.org/get-ovirt/
Note You need to log in before you can comment on or make changes to this bug.