Re: Review Request 15922: Enhanced few features under Marvin

2013-12-11 Thread Santhosh Edukulla
> On Dec. 11, 2013, 8:57 a.m., Sebastien Goasguen wrote: > > tools/marvin/marvin/marvinInit.py, line 32 > > > > > > Pretty sure this is not PEP8 compliant I just ran the pep8 against this module and it worked to show n

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-11 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15922/#review30187 --- tools/marvin/marvin/deployDataCenter.py

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-09 Thread Will Stevens
gmail.com [williamstev...@gmail.com] on behalf > of Will Stevens [wstev...@cloudops.com] > >>> Sent: Saturday, December 07, 2013 12:24 AM > >>> To: Santhosh Edukulla > >>> Cc: dev@cloudstack.apache.org; Girish Shilamkar > >>> Subject: Re: Review

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-09 Thread Sebastien Goasguen
__ >>> From: williamstev...@gmail.com [williamstev...@gmail.com] on behalf of Will >>> Stevens [wstev...@cloudops.com] >>> Sent: Saturday, December 07, 2013 12:24 AM >>> To: Santhosh Edukulla >>> Cc: dev@cloudstack.apache.org

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-07 Thread Girish Shilamkar
er 07, 2013 12:24 AM >> To: Santhosh Edukulla >> Cc: dev@cloudstack.apache.org; Girish Shilamkar >> Subject: Re: Review Request 15922: Enhanced few features under Marvin >> >> If you looked at the steps I have done, you will see that I have done that... >> >>

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Girish Shilamkar
amstev...@gmail.com] on behalf of Will > Stevens [wstev...@cloudops.com] > Sent: Saturday, December 07, 2013 12:24 AM > To: Santhosh Edukulla > Cc: dev@cloudstack.apache.org; Girish Shilamkar > Subject: Re: Review Request 15922: Enhanced few features under Marvin > > If you l

RE: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Santhosh Edukulla
; Girish Shilamkar Subject: Re: Review Request 15922: Enhanced few features under Marvin If you looked at the steps I have done, you will see that I have done that... I have done a clean install in order to get the latest marvin. That didnt work so I tried grabbing a version from online. That didnt

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Will Stevens
> On Dec. 6, 2013, 6:35 a.m., Girish Shilamkar wrote: > > Committed to 4.2, 4.3 and master. > > Will Stevens wrote: > I am unable to run the following command on the 4.3 branch now: > > mvn -P developer -pl tools/devcloud -Ddeploysvr -X > > I get one of the following error:

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Will Stevens
> > Thereafter you can build devcloud. Let me know. > > > Regards, > Santhosh > > From: williamstev...@gmail.com [williamstev...@gmail.com] on behalf of > Will Stevens [wstev...@cloudops.com] > Sent: Friday, December 06, 2013 10:59

RE: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Santhosh Edukulla
stev...@gmail.com [williamstev...@gmail.com] on behalf of Will Stevens [wstev...@cloudops.com] Sent: Friday, December 06, 2013 10:59 PM To: Santhosh Edukulla Cc: dev@cloudstack.apache.org; Girish Shilamkar Subject: Re: Review Request 15922: Enhanced few features under Marvin Where do I get the latest pac

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Will Stevens
mber 06, 2013 4:14 PM > To: Girish Shilamkar > Cc: Santhosh Edukulla; Will Stevens; cloudstack > Subject: Re: Review Request 15922: Enhanced few features under Marvin > > > On Dec. 6, 2013, 6:35 a.m., Girish Shilamkar wrote: > > > Committed to 4.2, 4.3 and master. >

RE: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Santhosh Edukulla
From: Will Stevens [nore...@reviews.apache.org] on behalf of Will Stevens [wstev...@cloudops.com] Sent: Friday, December 06, 2013 4:14 PM To: Girish Shilamkar Cc: Santhosh Edukulla; Will Stevens; cloudstack Subject: Re: Review Request 15922: Enhanced few features under Marvin > On

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-06 Thread Will Stevens
> On Dec. 6, 2013, 6:35 a.m., Girish Shilamkar wrote: > > Committed to 4.2, 4.3 and master. I am unable to run the following command on the 4.3 branch now: mvn -P developer -pl tools/devcloud -Ddeploysvr -X I get one of the following error: [DEBUG] Executing command line: python ../marvin/marv

Re: Review Request 15922: Enhanced few features under Marvin

2013-12-05 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15922/#review29875 --- Ship it! Committed to 4.2, 4.3 and master. - Girish Shilamkar On