Dan Kenigsberg has submitted this change and it was merged. Change subject: API: move diskSizeExtend logic from BindingXMLRPC ......................................................................
API: move diskSizeExtend logic from BindingXMLRPC In order to make cold extend disk usable with JSON-RPC, move logic verifying specified VM id from 'BindingXMLRPC -> diskSizeExtend' into 'API (VM class) -> diskSizeExtend' [since there's no other mediator for this logic to both xml/json rpc protocols]. Change-Id: I9014154f376609d8bd547b7c008954981785b551 Bug-Url: https://bugzilla.redhat.com/1178499 Signed-off-by: Daniel Erez <de...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/36590 Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Allon Mureinik <amure...@redhat.com> Reviewed-by: Tal Nisan <tni...@redhat.com> Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/36614 --- M vdsm/API.py M vdsm/rpc/BindingXMLRPC.py 2 files changed, 15 insertions(+), 15 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Daniel Erez: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36614 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9014154f376609d8bd547b7c008954981785b551 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches