Juan Hernandez has posted comments on this change.

Change subject: core: Add selinux host info to VdsDynamic
......................................................................


Patch Set 5: Code-Review+1

(1 comment)

The RESTAPI part looks good.

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

Line 1515:   <xs:complexType name="SELinux">
Line 1516:       <xs:sequence>
Line 1517:         <xs:element name="mode" type="xs:string" minOccurs="0" 
maxOccurs="1" />
Line 1518:       </xs:sequence>
Line 1519:   </xs:complexType>
Please remember to fix indentation before merging: two spaces per level.
Line 1520: 
Line 1521:   <xs:element name="host" type="Host"/>
Line 1522: 
Line 1523:   <xs:element name="hosts" type="Hosts"/>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If472f68702b59280c721450d4db50dc27dc19a30
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <[email protected]>
Gerrit-Reviewer: Dima Kuznetsov <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: mooli tayer <[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

Reply via email to