-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10702/#review19523
-----------------------------------------------------------

Ship it!


- Prasanna Santhanam


On April 22, 2013, 11:21 a.m., SrikanteswaraRao Talluri wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10702/
> -----------------------------------------------------------
> 
> (Updated April 22, 2013, 11:21 a.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Description
> -------
> 
> fix the string formatting error in cloudConnection.py
> 
> error seen:
> 
> Traceback (most recent call last):
>   File "/usr/local/bin/nosetests-2.7", line 9, in <module>
>     load_entry_point('nose==1.2.1', 'console_scripts', 'nosetests-2.7')()
>   File 
> "/usr/local/lib/python2.7/site-packages/nose-1.2.1-py2.7.egg/nose/core.py", 
> line 118, in __init__
>     **extra_args)
>   File "/usr/local/lib/python2.7/unittest/main.py", line 94, in __init__
>     self.parseArgs(argv)
>   File 
> "/usr/local/lib/python2.7/site-packages/nose-1.2.1-py2.7.egg/nose/core.py", 
> line 135, in parseArgs
>     self.config.configure(argv, doc=self.usage())
>   File 
> "/usr/local/lib/python2.7/site-packages/nose-1.2.1-py2.7.egg/nose/config.py", 
> line 338, in configure
>     self.plugins.configure(options, self)
>   File 
> "/usr/local/lib/python2.7/site-packages/nose-1.2.1-py2.7.egg/nose/plugins/manager.py",
>  line 284, in configure
>     cfg(options, config)
>   File 
> "/usr/local/lib/python2.7/site-packages/nose-1.2.1-py2.7.egg/nose/plugins/manager.py",
>  line 99, in __call__
>     return self.call(*arg, **kw)
>   File 
> "/usr/local/lib/python2.7/site-packages/nose-1.2.1-py2.7.egg/nose/plugins/manager.py",
>  line 167, in simple
>     result = meth(*arg, **kw)
>   File 
> "/usr/local/lib/python2.7/site-packages/marvin_nose-0.1.0-py2.7.egg/marvinPlugin.py",
>  line 42, in configure
>     deploy.loadCfg() if options.load else deploy.deploy()
>   File "/usr/local/lib/python2.7/site-packages/marvin/deployDataCenter.py", 
> line 452, in deploy
>     self.loadCfg()
>   File "/usr/local/lib/python2.7/site-packages/marvin/deployDataCenter.py", 
> line 409, in loadCfg
>     apiKey, securityKey = self.registerApiKey()
>   File "/usr/local/lib/python2.7/site-packages/marvin/deployDataCenter.py", 
> line 349, in registerApiKey
>     listuserRes = self.testClient.getApiClient().listUsers(listuser)
>   File 
> "/usr/local/lib/python2.7/site-packages/marvin/cloudstackAPI/cloudstackAPIClient.py",
>  line 436, in listUsers
>     response = self.connection.marvin_request(command, data=postdata, 
> response_type=response)
>   File 
> "/usr/local/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 
> 212, in marvin_request
>     cmdname, auth=False, payload=payload, data=data)
>   File 
> "/usr/local/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 
> 141, in request
>     (self.baseurl, c))
> TypeError: not all arguments converted during string formatting 
> 
> 
> This addresses bug CLOUDSTACK-2135.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackConnection.py 14b12e7 
> 
> Diff: https://reviews.apache.org/r/10702/diff/
> 
> 
> Testing
> -------
> 
> tested
> 
> 
> Thanks,
> 
> SrikanteswaraRao Talluri
> 
>

Reply via email to