Gilad Chaplik has posted comments on this change.

Change subject: restapi: NUMA feature restful API support
......................................................................


Patch Set 6:

(3 comments)

please verify.

there are some comments that weren't addressed.

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

Line 4174:   </xs:complexType>
Line 4175: 
Line 4176:   <xs:complexType name="VirtualNumaNode">
Line 4177:     <xs:complexContent>
Line 4178:       <xs:extension base="BaseDevice">
should be base=NumaNode
Line 4179:         <xs:sequence>
Line 4180:           <xs:element name="index" type="xs:int" minOccurs="0" 
maxOccurs="1"/>
Line 4181:           <xs:element name="memory" type="xs:long" minOccurs="0" 
maxOccurs="1"/>
Line 4182:           <xs:element name="numa_cpus" type="NumaCPUs" minOccurs="0" 
maxOccurs="1"/>


Line 4187:   </xs:complexType>
Line 4188: 
Line 4189:   <xs:complexType name="VirtualNumaNodes">
Line 4190:     <xs:complexContent>
Line 4191:       <xs:extension base="BaseDevices">
should be base=NumaNodes
Line 4192:         <xs:sequence>
Line 4193:           <xs:element ref="vm_numa_node" minOccurs="0" 
maxOccurs="unbounded">
Line 4194:             <xs:annotation>
Line 4195:               <xs:appinfo>


Line 4206:   
Line 4207:   <xs:element name="numa_node_pin" type="NumaNodePin"/>
Line 4208:   
Line 4209:   <xs:element name="numa_cpus" type="NumaCPUs"/>
Line 4210:   
remove trailing white space..
Line 4211:   <xs:element name="numa_cpu" type="NumaCPU"/>
Line 4212: 
Line 4213:   <xs:complexType name="NumaNodePins">
Line 4214:     <xs:sequence>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I72ed4b16c220decbf640f74c4aadffe423afc290
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jason Liao <chuan.l...@hp.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Jason Liao <chuan.l...@hp.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Xiaolei Shi <xiao-lei....@hp.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