Change in vdsm[master]: vdsm: added documentation for downloadImage method

cshereme at redhat.com cshereme at redhat.com
Fri Mar 13 17:45:40 UTC 2015


Candace Sheremeta has posted comments on this change.

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


Patch Set 7:

(1 comment)

added quotes

https://gerrit.ovirt.org/#/c/37934/7/client/vdsClient.py
File client/vdsClient.py:

Line 2613:             'Returns: A task UUID',
Line 2614:             'Example:',
Line 2615:             'vdsClient -s 0 downloadImage \ ',
Line 2616:             '  {"url": "http://example.com/myimage", '
Line 2617:             '"headers": {"Foo": "Bar"}, "method": "http"} \ ',
> This is still not quoted.
Done
Line 2618:             '  spUUID sdUUID imgUUID'
Line 2619:         )),
Line 2620:         'prepareImage': (serv.prepareImage, (
Line 2621:             '<spUUID> <sdUUID> <imgUUID> [<volUUID>]',


-- 
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: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Candace Sheremeta <cshereme at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Candace Sheremeta <cshereme at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list