cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Prasanna Santhanam <...@apache.org>
Subject Re: easy bug to fix for new comer
Date Fri, 21 Jun 2013 06:07:47 GMT
On Thu, Jun 20, 2013 at 02:01:54PM +0000, Daan Hoogland wrote: > Btw Sebastien,
> > Newcomer as I am; how do I test the test-code tests?

The first step would be to see if you are able to build marvin 

$ mvn -Pdeveloper -pl :cloud-marvin clean install

You can run tests suitable for basic zone within devcloud if you have it setup.
See the section on "devcloud tests" in the marvin wiki. [0]

Then you can try and run the checkin tests [1] (which are broken at the
moment as a result of a merge) but I hope to fix that today.

Automated tests will be run on the test infra @
jenkins.buildacloud.org [2] so any issues in the formatting will show up
there as well.

[0] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Marvin+-+Testing+with+Python
[1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Marvin+-+Testing+with+Python#Marvin-TestingwithPython-
[2] http://jenkins.buildacloud.org/view/cloudstack-qa/job/test-smoke-matrix/


Thanks for the patches!

> 
> -----Original Message-----
> From: Daan Hoogland [mailto:DHoogland@schubergphilis.com] 
> Sent: donderdag 20 juni 2013 15:43
> To: 'dev@cloudstack.apache.org'
> Subject: RE: easy bug to fix for new comer
> 
> Sure,
> 
> One file at a time!
> 
> -----Original Message-----
> From: Sebastien Goasguen [mailto:runseb@gmail.com] 
> Sent: donderdag 20 juni 2013 14:35
> To: dev@cloudstack.apache.org
> Subject: easy bug to fix for new comer
> 
> Hi,
> 
> Here is an easy bug to fix for a newcomer to cloudstack:
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-3096
> 
> install pep8 on your machine
> git clone cloudstack repo
> go to the marvin directory
> 
> run pep8 like I show in the bug.
> 
> edit the python scripts to fix the errors
> 
> once pep8 is clean, git commit everything, create a patch and send to review board.
> 
> Any takers ?
> 
> -Sebastien

-- 
Prasanna.,

------------------------
Powered by BigRock.com


Mime
View raw message