Change in vdsm[master]: Validating number of required args in spmStart()
dougsland at redhat.com
dougsland at redhat.com
Wed Oct 24 03:18:33 UTC 2012
Douglas Schilling Landgraf has posted comments on this change.
Change subject: Validating number of required args in spmStart()
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File vdsm_cli/vdsClient.py
Line 713: return res['status']['code'], res['status']['message']
Line 714: return 0, ''
Line 715:
Line 716: def spmStart(self, args):
Line 717: validateArgTypes(args, [str, int, int, int, str, int, int], 5)
+1
Line 718: status = self.s.spmStart(*args)
Line 719: if status['status']['code']:
Line 720: return status['status']['code'], status['status']['message']
Line 721: return 0, status['uuid']
--
To view, visit http://gerrit.ovirt.org/8731
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iab0ddbd4b3f77fc2011e7b554a4800298a9cf39f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Amador Pahim <apahim at redhat.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Shu Ming <shuming at linux.vnet.ibm.com>
More information about the vdsm-patches
mailing list