Nir Soffer has posted comments on this change.

Change subject: virt-sparsify: add inplace virt-sparsify support
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/54427/1/lib/vdsm/rpc/Bridge.py
File lib/vdsm/rpc/Bridge.py:

Line 501:     'Volume_getInfo': {'ret': 'info'},
Line 502:     'Volume_getPath': {'ret': 'path'},
Line 503:     'Volume_getSize': {'ret': Volume_getsize_Ret},
Line 504:     'Volume_extendSize': {'ret': 'uuid'},
Line 505:     'Volume_sparsifyInplace': {},
You don't need to provide an entry for empty return value.
Line 506:     'Host_getAllTasks': {'ret': 'tasks'},
Line 507:     'Host_getJobs': {'ret': 'jobs'},
Line 508: }
Line 509: 


-- 
To view, visit https://gerrit.ovirt.org/54427
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie0a1269d71816a8ab25c4a00bfb483620cfefde8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to