Frank Kobzik has posted comments on this change.
Change subject: restapi: Make adding (virtio-)console to virtual machines
optional
......................................................................
Patch Set 4: (5 inline comments)
Hi I put some comments - please take a look at AbstractBackendResource and
BackendVmPoolsResource ones. In the meantime I'm going to handle the tests.
Thanks.
....................................................
File
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
Line 1062: </xs:element>
Line 1063: <xs:element ref="transparent_hugepages" minOccurs="0"/>
Line 1064: </xs:sequence>
Line 1065: </xs:complexType>
Line 1066:
Done
Line 1067: <xs:complexType name="Console">
Line 1068: <xs:attribute name="enabled" type="xs:boolean"/>
Line 1069: </xs:complexType>
Line 1070:
Line 1944: <xs:element name="high_availability"
type="HighAvailability" minOccurs="0"/>
Line 1945: <xs:element name="display" type="Display" minOccurs="0"
maxOccurs="1"/>
Line 1946: <xs:element name="stateless" type="xs:boolean"
minOccurs="0"/>
Line 1947: <xs:element name="delete_protected" type="xs:boolean"
minOccurs="0"/>
Line 1948: <xs:element name="console" type="Console" minOccurs="0"/>
Done
Line 1949: <xs:element name="timezone" type="xs:string" minOccurs="0"
maxOccurs="1"/>
Line 1950: <xs:element ref="domain" minOccurs="0" maxOccurs="1"/>
Line 1951: <xs:element ref="usb" minOccurs="0" maxOccurs="1"/>
Line 1952: <xs:element name="tunnel_migration" type="xs:boolean"
minOccurs="0" maxOccurs="1"/>
Line 2138: <xs:element name="creation_time" type="xs:dateTime"
minOccurs="0"/>
Line 2139: <xs:element name="origin" type="xs:string" minOccurs="0"/>
Line 2140: <xs:element name="stateless" type="xs:boolean"
minOccurs="0"/>
Line 2141: <xs:element name="delete_protected" type="xs:boolean"
minOccurs="0"/>
Line 2142: <xs:element name="console" type="Console" minOccurs="0"/>
Done
Line 2143: <xs:element name="timezone" type="xs:string" minOccurs="0"
maxOccurs="1"/>
Line 2144: <xs:element ref="domain" minOccurs="0" maxOccurs="1"/>
Line 2145: <xs:element name="custom_properties"
type="CustomProperties" minOccurs="0"/>
Line 2146: <xs:element name="payloads" type="Payloads" minOccurs="0"/>
....................................................
File
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendResource.java
Line 418: return doGetEntity(entityType, query,
getQueryParams(queryParamsClass, id), id.toString());
Line 419: }
Line 420: }
Line 421:
Line 422: protected List<String> getConsoleDevicesForEntity(Guid id) {
Yes, please check my answer in PS3.
Line 423: return getEntity(List.class,
Line 424: VdcQueryType.GetConsoleDevices,
Line 425: new IdQueryParameters(id),
Line 426: "GetConsoleDevices", true);
....................................................
File
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java
Line 63:
Line 64: int size = pool.isSetSize() ? pool.getSize() : 1;
Line 65:
Line 66: AddVmPoolWithVmsParameters params = new
AddVmPoolWithVmsParameters(entity, vm, size, -1);
Line 67:
params.setConsoleEnabled(!getConsoleDevicesForEntity(template.getId()).isEmpty());
I put some in PS3 as well. Could you comment them?
Line 68:
Line 69: return performCreate(VdcActionType.AddVmPoolWithVms,
Line 70: params,
Line 71: new
QueryIdResolver<Guid>(VdcQueryType.GetVmPoolById,
--
To view, visit http://gerrit.ovirt.org/16454
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I733e4c4a9c7d8f5dbe68b6b26a49502b8a40ec9d
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
