----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16154/#review30100 -----------------------------------------------------------
Commit 4ee3fdbde9e721021894e9f4a869fb89f5d786e7 in branch refs/heads/4.3 from Santhosh Edukulla [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=4ee3fdb ] CLOUDSTACK-5423: Fixed 5423 - ASF Subversion and Git Services On Dec. 10, 2013, 11:37 a.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16154/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2013, 11:37 a.m.) > > > Review request for cloudstack and Girish Shilamkar. > > > Bugs: CLOUDSTACK-5423 > https://issues.apache.org/jira/browse/CLOUDSTACK-5423 > > > Repository: cloudstack-git > > > Description > ------- > > 1. As part of recent changes to marvin, mentioned files requires some > changes. > 2. Some cleaning is as well required for these two files. Did clean up. > 3. Did cleanup in deployAndRun and added proper flow for ease of > understanding. > 4. Removed unwanted logging and used MarvinInit for initialization of Marvin. > 5. Removed few unwanted loops and redundant code. > 6. Renamed TestCaseExecuteEngine.py -> tcExecuteEngine.py inline with other > naming convention for modules in Marvin > 7. Added few codes and doc strings. > > > Diffs > ----- > > tools/marvin/marvin/TestCaseExecuteEngine.py f5af1fe > tools/marvin/marvin/configGenerator.py 3f98aca > tools/marvin/marvin/deployAndRun.py 8a758a1 > tools/marvin/marvin/tcExecuteEngine.py PRE-CREATION > > Diff: https://reviews.apache.org/r/16154/diff/ > > > Testing > ------- > > Tested DeployDC and Running Few test case runs with deployAndRun. > > > Thanks, > > Santhosh Edukulla > >
