Nir Soffer has posted comments on this change.

Change subject: vdsm: added documentation for downloadImage method
......................................................................


Patch Set 4:

(1 comment)

https://gerrit.ovirt.org/#/c/37934/4/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.json:

Line 4657: #
Line 4658: # Download an image to a remote endpoint using the specified method
Line 4659: # and methodArgs.
Line 4660: #
Line 4661: # @methodArgs:       An ImageSharingMethodArgs record specifying the 
download method.
This is only one line, but you can separate it to another patch, since it is 
not related to having better online help in vdsClient.
Line 4662: #
Line 4663: # @storagepoolID:    The UUID of the Storage Pool associated with 
the Image
Line 4664: #
Line 4665: # @storagedomainID:  The UUID of the Storage Domain associated with 
the Image


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icb26a36734e2381473aee81614c924c3f285e86c
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Candace Sheremeta <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Ala Hino <[email protected]>
Gerrit-Reviewer: Amit Aviram <[email protected]>
Gerrit-Reviewer: Candace Sheremeta <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Greg Padgett <[email protected]>
Gerrit-Reviewer: Idan Shaby <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to