----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17702/#review34621 -----------------------------------------------------------
tools/marvin/marvin/integration/lib/base.py <https://reviews.apache.org/r/17702/#comment64807> One function should suffice i believe as such the internal implementation is not much varied. - Santhosh Edukulla On Feb. 14, 2014, 10:38 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17702/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2014, 10:38 a.m.) > > > Review request for cloudstack, Girish Shilamkar and Santhosh Edukulla. > > > Bugs: CLOUDSTACK-2232 > https://issues.apache.org/jira/browse/CLOUDSTACK-2232 > > > Repository: cloudstack-git > > > Description > ------- > > Adding automation test cases for Persistent Networks feature. > > > Diffs > ----- > > test/integration/component/test_persistent_networks.py f61ccaa > tools/marvin/marvin/config/config.cfg 7840b7c > tools/marvin/marvin/integration/lib/base.py aa23029 > tools/marvin/marvin/integration/lib/common.py e202391 > > Diff: https://reviews.apache.org/r/17702/diff/ > > > Testing > ------- > > > Thanks, > > Ashutosh Kelkar > >