Thanks Mike.  Ya, I also did the same change locally and then did the
following to not track the hack in my branch.

git update-index --assume-unchanged
tools/marvin/marvin/cloudstackConnection.py

Thanks for submitting a patch for that.

Cheers,

Will




On Mon, Jun 3, 2013 at 1:05 PM, Mike Tutkowski <mike.tutkow...@solidfire.com
> wrote:

> In cloudstackConnection.py, I made the following change:
>
> -    def marvin_request(self, cmd, response_type=None, method='GET'):
>
> +    def marvin_request(self, cmd, response_type=None, method='GET',
> data=''):
>
>
> On Mon, Jun 3, 2013 at 11:03 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
> > I have fixed this in a patch I submitted last week.
> >
> > I'm not sure when it began, but I noticed it a long time ago and had just
> > sent out an e-mail then and corrected it in my sandbox.
> >
> > Let me see if I can find what I did to fix it.
> >
> >
> > On Mon, Jun 3, 2013 at 10:09 AM, Will Stevens <wstev...@cloudops.com
> >wrote:
> >
> >> Has anyone else experience this?  I just pulled in the master code into
> my
> >> branch and now I am getting this in my dev environment.
> >>
> >> [DEBUG] Executing command line: python
> >> ../marvin/marvin/deployDataCenter.py
> >> -i devcloud.cfg
> >> Traceback (most recent call last):
> >>   File "../marvin/marvin/deployDataCenter.py", line 517, in <module>
> >>     deploy.deploy()
> >>   File "../marvin/marvin/deployDataCenter.py", line 500, in deploy
> >>     self.loadCfg()
> >>   File "../marvin/marvin/deployDataCenter.py", line 451, in loadCfg
> >>     apiKey, securityKey = self.registerApiKey()
> >>   File "../marvin/marvin/deployDataCenter.py", line 390, in
> registerApiKey
> >>     listuserRes = self.testClient.getApiClient().listUsers(listuser)
> >>   File
> >>
> >>
> "/mnt/hgfs/palo_alto/incubator-cloudstack/tools/marvin/marvin/cloudstackAPI/cloudstackAPIClient.py",
> >> line 2385, in listUsers
> >>     response = self.connection.marvin_request(command, data=postdata,
> >> response_type=response)
> >> TypeError: marvin_request() got an unexpected keyword argument 'data'
> >>
> >> Thanks,
> >>
> >> ws
> >>
> >>
> >> On Mon, May 6, 2013 at 5:13 PM, Mike Tutkowski <
> >> mike.tutkow...@solidfire.com
> >> > wrote:
> >>
> >> > It looks like the marvin_request method in cloudstackConnection.py
> does
> >> not
> >> > have a parameter named 'data'.
> >> >
> >> > I changed the signature locally to the following and it works now:
> >> >
> >> > def marvin_request(self, cmd, response_type=None, method='GET',
> >> data=''):
> >> >
> >> >
> >> > On Mon, May 6, 2013 at 2:59 PM, Mike Tutkowski <
> >> > mike.tutkow...@solidfire.com
> >> > > wrote:
> >> >
> >> > > I don't have much Python experience, but it looks like we're trying
> to
> >> > > pass in a named parameter that doesn't exist on the receiving side.
> >> > >
> >> > > Perhaps I need to update a Python package?
> >> > >
> >> > >     def listUsers(self, command, postdata={}):
> >> > >
> >> > >         response = listUsersResponse()
> >> > >
> >> > >         response = self.connection.marvin_request(command,
> >> data=postdata,
> >> > > response_type=response)
> >> > >
> >> > >         return response
> >> > >
> >> > >
> >> > > On Mon, May 6, 2013 at 12:04 PM, Mike Tutkowski <
> >> > > mike.tutkow...@solidfire.com> wrote:
> >> > >
> >> > >> Hi,
> >> > >>
> >> > >> I just updated to the latest today and ran deployDataCenter.py to
> >> build
> >> > a
> >> > >> DevCloud2 environment.
> >> > >>
> >> > >> The script is having trouble. Any thoughts on this? Has this worked
> >> > >> recently for anyone else?
> >> > >>
> >> > >> Thanks!
> >> > >>
> >> > >> mtutkowski-LT:devcloud mtutkowski$ python
> >> > >> ../marvin/marvin/deployDataCenter.py -i devcloud.cfg
> >> > >> Traceback (most recent call last):
> >> > >>   File "../marvin/marvin/deployDataCenter.py", line 476, in
> <module>
> >> > >>     deploy.deploy()
> >> > >>   File "../marvin/marvin/deployDataCenter.py", line 459, in deploy
> >> > >>     self.loadCfg()
> >> > >>   File "../marvin/marvin/deployDataCenter.py", line 410, in loadCfg
> >> > >>     apiKey, securityKey = self.registerApiKey()
> >> > >>   File "../marvin/marvin/deployDataCenter.py", line 349, in
> >> > registerApiKey
> >> > >>     listuserRes =
> self.testClient.getApiClient().listUsers(listuser)
> >> > >>   File
> >> > >>
> >> >
> >>
> "/Users/mtutkowski/Documents/CloudStack/src/incubator-cloudstack/tools/marvin/marvin/cloudstackAPI/cloudstackAPIClient.py",
> >> > >> line 433, in listUsers
> >> > >>     response = self.connection.marvin_request(command,
> data=postdata,
> >> > >> response_type=response)
> >> > >> TypeError: marvin_request() got an unexpected keyword argument
> 'data'
> >> > >>
> >> > >> --
> >> > >> *Mike Tutkowski*
> >> > >> *Senior CloudStack Developer, SolidFire Inc.*
> >> > >> e: mike.tutkow...@solidfire.com
> >> > >> o: 303.746.7302
> >> > >> Advancing the way the world uses the cloud<
> >> > http://solidfire.com/solution/overview/?video=play>
> >> > >> *™*
> >> > >>
> >> > >
> >> > >
> >> > >
> >> > > --
> >> > > *Mike Tutkowski*
> >> > > *Senior CloudStack Developer, SolidFire Inc.*
> >> > > e: mike.tutkow...@solidfire.com
> >> > > o: 303.746.7302
> >> > > Advancing the way the world uses the cloud<
> >> > http://solidfire.com/solution/overview/?video=play>
> >> > > *™*
> >> > >
> >> >
> >> >
> >> >
> >> > --
> >> > *Mike Tutkowski*
> >> > *Senior CloudStack Developer, SolidFire Inc.*
> >> > e: mike.tutkow...@solidfire.com
> >> > o: 303.746.7302
> >> > Advancing the way the world uses the
> >> > cloud<http://solidfire.com/solution/overview/?video=play>
> >> > *™*
> >> >
> >>
> >
> >
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the cloud<
> http://solidfire.com/solution/overview/?video=play>
> > *™*
> >
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud<http://solidfire.com/solution/overview/?video=play>
> *™*
>

Reply via email to