----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13680/#review25397 -----------------------------------------------------------
Commit 1c31ee22d40d77c10593d87b8237cd0489d192cc in branch refs/heads/4.2-forward from Sowmya Krishnan [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=1c31ee2 ] CLOUDSTACK-3927, CLOUDSTACK-3928: Fix NS scripts to add/remove NS device in setupClass/tearDown Signed-off-by: Prasanna Santhanam <t...@apache.org> - ASF Subversion and Git Services On Aug. 21, 2013, 10:47 a.m., Sowmya Krishnan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13680/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2013, 10:47 a.m.) > > > Review request for cloudstack, venkata swamy babu budumuru and Prasanna > Santhanam. > > > Bugs: CLOUDSTACK-3927 and CLOUDSTACK-3928 > > > Repository: cloudstack-git > > > Description > ------- > > Fix to add NS device in setupClass and remove in tearDown > Added a common method to perform these in common.add_netscaler() > Also fixes few account object issues in netscaler_configs. > > Patch will help in keeping each test suite independent. Otherwise we cannot > identify which NS an LB rule will be deployed in if there's more than one NS > device. > > > Diffs > ----- > > test/integration/component/test_netscaler_configs.py bcea254 > test/integration/component/test_netscaler_lb.py 3942f94 > test/integration/component/test_netscaler_lb_algo.py 477bd69 > test/integration/component/test_netscaler_lb_sticky.py 1edfd7b > tools/marvin/marvin/integration/lib/common.py 4f5acef > > Diff: https://reviews.apache.org/r/13680/diff/ > > > Testing > ------- > > Tested locallly > > > Thanks, > > Sowmya Krishnan > >
