Yair Zaslavsky has posted comments on this change.

Change subject: vdsbroker: introduce MergeVDSCommand
......................................................................


Patch Set 9: Code-Review+1

(1 comment)

http://gerrit.ovirt.org/#/c/26907/9/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java
File 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java:

Line 918:         return new ImageSizeReturnForXmlRpc(response);
Line 919:     }
Line 920: 
Line 921:     @Override
Line 922:     public StatusOnlyReturnForXmlRpc merge(String vmId, Map<String, 
String> drive,
Once again, please sync with Pitor after merged.
Line 923:             String baseVolUUID, String topVolUUID, String bandwidth, 
String jobUUID) {
Line 924:         JsonRpcRequest request =
Line 925:                 new RequestBuilder("VM.merge").withParameter("vmID", 
vmId)
Line 926:                         .withParameter("drive", drive)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I320d1ac7de50f4a24236c6d26af75a3483b8bc5e
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadg...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
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