Dan Kenigsberg has posted comments on this change.

Change subject: Correct vdsClient's help for the verb getVmsInfo
......................................................................


Patch Set 2: I would prefer that you didn't submit this

(2 inline comments)

....................................................
Commit Message
Line 5: CommitDate: 2012-08-29 13:37:23 +0800
Line 6: 
Line 7: Correct vdsClient's help for the verb getVmsInfo
Line 8: 
Line 9: https://bugzilla.redhat.com/show_bug.cgi?id=727791
please use

 Bug-Id: https://bugzilla.redhat.com/show_bug.cgi?id=727791
Line 10: 
Line 11: replace the help with the following:
Line 12: getVmsInfo <spUUID> [<sdUUID> [vmList](vmId1,vmId2,...)]
Line 13: 


....................................................
File vdsm_cli/vdsClient.py
Line 2222:                         )),
Line 2223:         'getVmsInfo': (serv.do_getVmsInfo,
Line 2224:                       ('<spUUID> [<sdUUID> 
[vmList](vmId1,vmId2,...)]',
Line 2225:                        'Return info of VMs from the pool or a backup 
domain '
Line 2226:                        'if it\'s sdUUID given. If vmList also given, 
only get '
it's->its

If vmList also given-> If vmList is also given
Line 2227:                        'info for these VMs.'
Line 2228:                        )),
Line 2229:         'getVmsList': (serv.do_getVmsList,
Line 2230:                       ('<spUUID> [sdUUID]',


--
To view, visit http://gerrit.ovirt.org/7560
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I152f28dea29f6ac6bb61c0597dbd3b596646c569
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhu Bo <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: ShaoHe Feng <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: Zhu Bo <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to