Omer Frenkel has posted comments on this change.
Change subject: restapi: cloud-init - rest api for start vm
......................................................................
Patch Set 15:
(6 comments)
....................................................
File
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
Line 2393: <xs:element name="dns" type="DNS"/>
Line 2394:
Line 2395: <xs:complexType name="DNS">
Line 2396: <xs:sequence>
Line 2397: <xs:element name="dns_servers">
basically DNS should have 2 lists of strings, servers and search-domains,
is it possible to do something simple like this:
<xs:element name="dns" type="DNS"/>
<xs:complexType name="DNS">
<xs:sequence>
<xs:element name="servers" type="xs:string" minOccurs="0"
maxOccurs="unbounded"/>
<xs:element name="search_domains" type="xs:string" minOccurs="0"
maxOccurs="unbounded"/>
</xs:sequence>
</xs:complexType>
??
Line 2398: <xs:complexType>
Line 2399: <xs:sequence>
Line 2400: <xs:element name="dns_server" type="xs:string"
minOccurs="0" maxOccurs="unbounded"/>
Line 2401: </xs:sequence>
Line 2421: </xs:complexType>
Line 2422:
Line 2423: <xs:element name="authorized_keys" type="AuthorizedKeys"/>
Line 2424:
Line 2425: <xs:complexType name="AuthorizedKeys">
the only difference i see is this part inside "VM" complex type:
<xs:annotation>
<xs:appinfo>
<jaxb:class name="VM"/>
</xs:appinfo>
</xs:annotation>
<xs:complexContent>
<xs:extension base="BaseResource">
...
is this what you mean?
Line 2426: <xs:sequence>
Line 2427: <xs:element ref="authorized_key" minOccurs="0"
maxOccurs="unbounded"/>
Line 2428: </xs:sequence>
Line 2429: </xs:complexType>
Line 2431: <xs:element name="cloud_init" type="CloudInit"/>
Line 2432:
Line 2433: <xs:complexType name="CloudInit">
Line 2434: <xs:sequence>
Line 2435: <xs:element name="host_name" type="xs:string" minOccurs="0"/>
this is the vm HOSTNAME
its not a reference to a host
Line 2436: <xs:element name="network" minOccurs="0">
Line 2437: <xs:complexType>
Line 2438: <xs:sequence>
Line 2439: <xs:element ref="nics" minOccurs="0"/>
Line 2436: <xs:element name="network" minOccurs="0">
Line 2437: <xs:complexType>
Line 2438: <xs:sequence>
Line 2439: <xs:element ref="nics" minOccurs="0"/>
Line 2440: <xs:element ref="dns" minOccurs="0"/>
when you configure your pc network, you can manually set the dns information,
it contains the dns servers and the search domains.
Line 2441: </xs:sequence>
Line 2442: </xs:complexType>
Line 2443: </xs:element> <!-- </network> -->
Line 2444: <xs:element ref="authorized_keys" minOccurs="0"/>
Line 2444: <xs:element ref="authorized_keys" minOccurs="0"/>
Line 2445: <xs:element name="regenerate_ssh_keys" type="xs:boolean"
minOccurs="0"/>
Line 2446: <xs:element name="timezone" type="xs:string" minOccurs="0"/>
Line 2447: <xs:element ref="users" minOccurs="0"/>
Line 2448: <xs:element ref="payload_file" minOccurs="0"
maxOccurs="unbounded"/>
will fix
Line 2449: </xs:sequence>
Line 2450: </xs:complexType>
Line 2451:
Line 2452: <xs:complexType name="VmPlacementPolicy">
....................................................
File
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
Line 292: cloud-init.regenerate_ssh_keys: 'xs:boolean',
Line 293: cloud-init.timezone: 'xs:string',
Line 294: cloud-init.users--COLLECTION: {users.password:
'xs:string',users.name: 'xs:string'},
Line 295: cloud-init.files--COLLECTION: {files.path: 'xs:string',
files.content: 'xs:string',
Line 296: files.encoding: 'xs:string'}}
ok will check and update
Line 297: description: start a virtual machine in the system identified
by the given id with the options specified in the request body
Line 298: urlparams: {}
Line 299: headers:
Line 300: Content-Type: {value: application/xml|json, required: true}
--
To view, visit http://gerrit.ovirt.org/15537
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6ad0bfeca23cf8d4b2887010081d63c258032611
Gerrit-PatchSet: 15
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadg...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Ori Liel <ol...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
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
