----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17999/#review35517 -----------------------------------------------------------
Ship it! Ship It! - Girish Shilamkar On Feb. 25, 2014, 12:24 p.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17999/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2014, 12:24 p.m.) > > > Review request for cloudstack and Girish Shilamkar. > > > Bugs: CLOUDSTACK-5674 > https://issues.apache.org/jira/browse/CLOUDSTACK-5674 > > > Repository: cloudstack-git > > > Description > ------- > > Added few changes to fix test suite run and zone default value. > > > Diffs > ----- > > tools/marvin/marvin/asyncJobMgr.py dab85ca > tools/marvin/marvin/cloudstackConnection.py b8d9a38 > tools/marvin/marvin/cloudstackTestClient.py e8c6ab2 > tools/marvin/marvin/dbConnection.py 422fcfa > tools/marvin/marvin/deployDataCenter.py 862a1bf > tools/marvin/marvin/lib/utils.py 957807d > tools/marvin/marvin/marvinInit.py 5218d23 > tools/marvin/marvin/marvinLog.py 4161eca > tools/marvin/marvin/marvinPlugin.py f36bca8 > tools/marvin/marvin/sshClient.py 9b5bca2 > > Diff: https://reviews.apache.org/r/17999/diff/ > > > Testing > ------- > > Tested running with multiple parameters. > > > File Attachments > ---------------- > > Patch file for fix. > > https://reviews.apache.org/media/uploaded/files/2014/02/25/723e2875-3741-4c57-a70b-1af6dfd474cc__0001-Added-few-changes-for-CLOUDSTACK-5674.patch > > > Thanks, > > Santhosh Edukulla > >