Frank Kobzik has posted comments on this change.

Change subject: restapi: Allow enabling/disabling SSO
......................................................................


Patch Set 3:

(8 comments)

Thanks for valuable comments!

....................................................
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
Line 621:           <xs:element ref="reported_device_types" minOccurs="0"/>
Line 622:           <xs:element ref="ip_versions" minOccurs="0"/>
Line 623:           <xs:element ref="snapshot_statuses" minOccurs="0"/>
Line 624:           <xs:element ref="content_types" minOccurs="0" />
Line 625:           <xs:element ref="hook_states" minOccurs="0" />
I don't really get this, please see my new version.
Line 626:           <xs:element ref="stages" minOccurs="0" />
Line 627:         </xs:sequence>
Line 628:       </xs:extension>
Line 629:     </xs:complexContent>


Line 2298:     </xs:sequence>
Line 2299:     <xs:attribute name="type" type="xs:string"/>
Line 2300:   </xs:complexType>
Line 2301: 
Line 2302:   <!-- Single Sign-On Section -->
Done
Line 2303:   <xs:complexType name="Sso">
Line 2304:     <xs:sequence>
Line 2305:       <xs:element name="methods" type="SsoMethods" minOccurs="1" 
maxOccurs="1" />
Line 2306:     </xs:sequence>


Line 2304:     <xs:sequence>
Line 2305:       <xs:element name="methods" type="SsoMethods" minOccurs="1" 
maxOccurs="1" />
Line 2306:     </xs:sequence>
Line 2307:   </xs:complexType>
Line 2308: 
Done
Line 2309:   <xs:complexType name="SsoMethods">
Line 2310:     <xs:sequence>
Line 2311:       <xs:element name="method" type="SsoMethod" minOccurs="0" 
maxOccurs="unbounded" />
Line 2312:     </xs:sequence>


Line 2306:     </xs:sequence>
Line 2307:   </xs:complexType>
Line 2308: 
Line 2309:   <xs:complexType name="SsoMethods">
Line 2310:     <xs:sequence>
Done
Line 2311:       <xs:element name="method" type="SsoMethod" minOccurs="0" 
maxOccurs="unbounded" />
Line 2312:     </xs:sequence>
Line 2313:   </xs:complexType>
Line 2314: 


Line 2307:   </xs:complexType>
Line 2308: 
Line 2309:   <xs:complexType name="SsoMethods">
Line 2310:     <xs:sequence>
Line 2311:       <xs:element name="method" type="SsoMethod" minOccurs="0" 
maxOccurs="unbounded" />
Done
Line 2312:     </xs:sequence>
Line 2313:   </xs:complexType>
Line 2314: 
Line 2315:   <xs:simpleType name="SsoMethod">


Line 2315:   <xs:simpleType name="SsoMethod">
Line 2316:     <xs:restriction base="xs:string">
Line 2317:       <xs:enumeration value="GUEST_AGENT"/>
Line 2318:     </xs:restriction>
Line 2319:   </xs:simpleType>
Done
Line 2320: 
Line 2321:   <xs:complexType name="HighAvailability">
Line 2322:     <xs:sequence>
Line 2323:       <xs:element name="enabled" type="xs:boolean" minOccurs="0" 
maxOccurs="1"/>


Line 2526:           <xs:element name="creation_time" type="xs:dateTime" 
minOccurs="0"/>
Line 2527:           <xs:element name="origin" type="xs:string" minOccurs="0"/>
Line 2528:           <xs:element name="stateless" type="xs:boolean" 
minOccurs="0"/>
Line 2529:           <xs:element name="delete_protected" type="xs:boolean" 
minOccurs="0"/>
Line 2530:           <xs:element name="sso" type="Sso" minOccurs="0" 
maxOccurs="1"/>
Done
Line 2531:           <xs:element ref="console" minOccurs="0" maxOccurs="1"/>
Line 2532:           <xs:element name="timezone" type="xs:string" minOccurs="0" 
maxOccurs="1"/>
Line 2533:           <xs:element ref="domain" minOccurs="0" maxOccurs="1"/>
Line 2534:           <xs:element name="custom_properties" 
type="CustomProperties" minOccurs="0"/>


....................................................
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
Line 75:           vm.description: xs:string
Line 76:           vm.comment: xs:string
Line 77:           vm.stateless: xs:boolean
Line 78:           vm.delete_protected: xs:boolean
Line 79:           vm.sso.methods.method: --COLLECTION: {method: 'xs:string'}
Done
Line 80:           vm.console.enabled: xs:boolean
Line 81:           vm.cpu.topology.sockets: xs:int
Line 82:           vm.placement_policy.affinity: xs:string
Line 83:           vm.placement_policy.host.id|name: xs:string


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I25bf94646ea89684152e48f25ad604db6e59e86c
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-Reviewer: Ori Liel <[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