Juan Hernandez has posted comments on this change. Change subject: purgatory with sadistic rest architecture ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/37525/2/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd File backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd: Line 3808: </xs:extension> Line 3809: Line 3810: </xs:complexContent> Line 3811: Line 3812: </xs:complexType> You need element definitions, otherwise the server (the JAXBProvider class) won't be able to figure out what should be the name of the root of the XML document. So you need something like this: <!-- These two element declarations are the most important missing things: --> <xs:element name="unmanaged_networks" type="UnmanagedNetworks"/> <xs:element name="unmanaged_network" type="UnmanagedNetworks"/> <!-- This I just adjusted to make minOccurs explicit, the default is 1, not valid for this case: --> <xs:complexType name="UnmanagedNetworks"> <xs:complexContent> <xs:extension base="BaseResources"> <xs:sequence> <xs:element ref="unmanaged_network" minOccurs="0" maxOccurs="unbounded"> <xs:annotation> <xs:appinfo> <jaxb:property name="UnmanagedNetworks"/> </xs:appinfo> </xs:annotation> </xs:element> </xs:sequence> </xs:extension> </xs:complexContent> </xs:complexType> <!-- Same, make minOccurs explicit: --> <xs:complexType name="UnmanagedNetwork"> <xs:complexContent> <xs:extension base="BaseResource"> <xs:sequence> <xs:element name="unmanaged_network_name" type="xs:string" minOccurs="0" maxOccurs="1"/> <xs:element name="nic_name" type="xs:string" minOccurs="0" maxOccurs="1"/> <xs:element name="nic_id" type="xs:string" minOccurs="0" maxOccurs="1"/> </xs:sequence> </xs:extension> </xs:complexContent> </xs:complexType> Line 3813: Line 3814: <xs:complexType name="HostNics"> Line 3815: <xs:complexContent> Line 3816: <xs:extension base="BaseResources"> -- To view, visit http://gerrit.ovirt.org/37525 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idcf6e1292ce6f3a335051123be8c3389e8230bf0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
