Juan Hernandez has posted comments on this change.

Change subject: restapi: Expose vm.auto_converge and vm.migrate_compressed 
properties
......................................................................


Patch Set 12: Code-Review+1

(1 comment)

Add a MigrationOptionsValidator class (or similar) and use it in VmValidator 
(and TemplateValidator) like we do with OsValiador, for example.

http://gerrit.ovirt.org/#/c/32133/12/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd:

Line 655:           <xs:element ref="selinux_modes" minOccurs="0" />
Line 656:           <xs:element ref="rng_sources" minOccurs="0" />
Line 657:           <xs:element ref="scheduling_policy_unit_types" 
minOccurs="0" />
Line 658:           <xs:element ref="qos_types" minOccurs="0" />
Line 659:           <xs:element ref="inheritable_booleans" minOccurs="0" />
Add explicit maxOccurs.
Line 660:         </xs:sequence>
Line 661:       </xs:extension>
Line 662:     </xs:complexContent>
Line 663:   </xs:complexType>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9161896d1709c7492201667aa2c5955c5e37efd0
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbe...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Martin Betak <mbe...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to