Omer Frenkel has posted comments on this change.

Change subject: restapi: cloud-init - rest api for start vm
......................................................................


Patch Set 14:

(1 comment)

rebased and fixed according to Ori's request

....................................................
File 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
Line 903:             entity.setHostname(model.getHostName());
Line 904:         }
Line 905: 
Line 906:         if (model.isSetAuthorizedKeys() && 
!model.getAuthorizedKeys().getAuthorizedKey().isEmpty()) {
Line 907:             StringBuilder keys = new StringBuilder();
Done
Line 908:             for (AuthorizedKey authKey : 
model.getAuthorizedKeys().getAuthorizedKey()) {
Line 909:                 if ("root".equals(authKey.getUser().getName())) {
Line 910:                     if (keys.length() > 0) {
Line 911:                         keys.append("\n");


-- 
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: 14
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

Reply via email to