Michael Pasternak has posted comments on this change.

Change subject: restapi: cloud-init [6/6] - rest api for start vm
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
Line 223:           cloud-init.passwords.password: 'xs:string',
Line 224:           cloud-init.passwords.password.user: 'xs:string',
Line 225:           cloud-init.files.file.path: 'xs:string',
Line 226:           cloud-init.files.file.content: 'xs:string',
Line 227:           cloud-init.files.file.content.encoding: 'xs:string'}
since api doesn't have run-once context and to workaround #2 you can create 
another
signature for vm.start() here with payload & initialization + add description 
(run-once signature)

and remove  payload & initialization from the first one
Line 228:     urlparams: {}
Line 229:     headers:
Line 230:       Content-Type: {value: application/xml|json, required: true}
Line 231:       Correlation-Id: {value: 'any string', required: false}


-- 
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: 1
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: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to