Juan Hernandez has uploaded a new change for review.

Change subject: restapi: MAC pool elements must be optional
......................................................................

restapi: MAC pool elements must be optional

Currently the "allow_duplicates" element of the MAC pool type of the XML
schema is mandatory. This means that links to the MAC pools fail XML
schema validation. For example, a link like the following will fail
validation because it doesn't have the mandatory "allow_duplicates"
nested element:

  <mac_pool href="..." id="..."/>

In general all elements of XML complex types that may be used as links
need to be optional.

This patch changes the XML schema so that the "allow_duplicates" element
is optional.

Change-Id: Ie35fcae56df17619bdb0464699e7a154b4765d89
Bug-Url: https://bugzilla.redhat.com/1181711
Signed-off-by: Juan Hernandez <[email protected]>
---
M 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/81/36881/1

diff --git 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
index 85d76fd..732c1d8 100644
--- 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
+++ 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
@@ -1307,8 +1307,8 @@
     <xs:complexContent>
       <xs:extension base="BaseResource">
         <xs:sequence>
-          <xs:element name="allow_duplicates" type="xs:boolean" minOccurs="1"/>
-          <xs:element name="default_pool" type="xs:boolean" minOccurs="0"/>
+          <xs:element name="allow_duplicates" type="xs:boolean" minOccurs="0" 
maxOccurs="1"/>
+          <xs:element name="default_pool" type="xs:boolean" minOccurs="0" 
maxOccurs="1"/>
           <xs:element ref="ranges" minOccurs="0" maxOccurs="1"/>
         </xs:sequence>
       </xs:extension>


-- 
To view, visit http://gerrit.ovirt.org/36881
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie35fcae56df17619bdb0464699e7a154b4765d89
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to