-----------------------------------------------------------
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.
Changes
-------
Resolved comment#1. Now adding NS in try{} block
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 (updated)
-----
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
