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 <csher...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Ala Hino <ah...@redhat.com>
Gerrit-Reviewer: Amit Aviram <aavi...@redhat.com>
Gerrit-Reviewer: Candace Sheremeta <csher...@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: Idan Shaby <ish...@redhat.com>
Gerrit-Reviewer: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Tal Nisan <tni...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to