Sandro Bonazzola has posted comments on this change. Change subject: jsonrpcvdscli: add storage verbs ......................................................................
Patch Set 1: Traceback (most recent call last): File "/usr/lib/python2.7/site-packages/otopi/context.py", line 131, in _executeMethod method['method']() File "/usr/share/ovirt-hosted-engine-setup/scripts/../plugins/ovirt-hosted-engine-setup/storage/storage.py", line 1283, in _misc self._createStorageDomain() File "/usr/share/ovirt-hosted-engine-setup/scripts/../plugins/ovirt-hosted-engine-setup/storage/storage.py", line 857, in _createStorageDomain raise RuntimeError(status['status']['message']) RuntimeError: Invalid parameter: 'sdUUID=1 jsonrpc.Executor/4::INFO::2016-03-25 16:54:55,132::logUtils::49::dispatcher::(wrapper) Run and protect: createStorageDomain(storageType=u'5d6d70e5-34ab-4654-8880-7297bab5b8e4', sdUUID=1, domainName=u'mididell.ho me:/home/sata', typeSpecificArg=u'hosted_storage', domClass=1, domVersion=3, options=None) jsonrpc.Executor/4::ERROR::2016-03-25 16:54:55,133::task::868::Storage.TaskManager.Task::(_setError) Task=`c56872b6-90fb-4859-bdec-3373f2355176`::Unexpected error Traceback (most recent call last): File "/usr/share/vdsm/storage/task.py", line 875, in _run return fn(*args, **kargs) File "/usr/lib/python2.7/site-packages/vdsm/logUtils.py", line 50, in wrapper res = f(*args, **kwargs) File "/usr/share/vdsm/storage/hsm.py", line 2641, in createStorageDomain misc.validateUUID(sdUUID, 'sdUUID') File "/usr/share/vdsm/storage/misc.py", line 430, in validateUUID raise se.InvalidParameterException(name, uuid) InvalidParameterException: Invalid parameter: 'sdUUID=1' jsonrpc.Executor/4::DEBUG::2016-03-25 16:54:55,133::task::887::Storage.TaskManager.Task::(_run) Task=`c56872b6-90fb-4859-bdec-3373f2355176`::Task._run: c56872b6-90fb-4859-bdec-3373f2355176 (u'5d6d70e5-34ab-4654- 8880-7297bab5b8e4', 1, u'mididell.home:/home/sata', u'hosted_storage', 1, 3) {} failed - stopping task Looks again that vdscli and jsonrpcvdscli use different arguments ordering. -- To view, visit https://gerrit.ovirt.org/55252 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib2066afcbc8e7cac6a9ed534b2bb4034ea0fd854 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches