Liron Ar has posted comments on this change.

Change subject: core, restapi: add GetVmOvfConfigurationQuery
......................................................................


Patch Set 7:

(1 comment)

....................................................
File 
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java
Line 149:                         new IdQueryParameters(guid));
Line 150: 
Line 151:         if (queryReturnValue.getSucceeded() && 
queryReturnValue.getReturnValue() != null) {
Line 152:             return SnapshotMapper.mapSnapshotConfiguration((String) 
queryReturnValue.getReturnValue(), ConfigurationType.OVF, model);
Line 153:         }
1. Done
2. I moved the test to BackendSnapshotsResourceTest, as now 
BackendSnapshotResource calls it's populate method.
3. Done
Line 154: 
Line 155:         return model;
Line 156:     }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I46340c4461b57a4c314fb50ca9c19ac5fd08a451
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Deepak C Shetty <deepa...@linux.vnet.ibm.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgot...@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