Re: Minor changes to Marvin tests for Netscaler

2013-08-15 Thread Chiradeep Vittal
LGTM On 8/14/13 9:27 PM, "Sowmya Krishnan" wrote: >I am planning to modify some of the Netscaler tests >(test_netscaler_lb_algo, test_netscaler_lb_sticky, test_netscaler_lb) in >our integration suite to include adding of NS device in the setUpClass(). >And remove the device in tearDownClass(). C

RE: Minor changes to Marvin tests for Netscaler

2013-08-15 Thread Sowmya Krishnan
> -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Thursday, August 15, 2013 12:42 PM > To: dev@cloudstack.apache.org > Subject: Re: Minor changes to Marvin tests for Netscaler > > On Thu, Aug 15, 2013 at 04:27:19AM +, Sowmya Kri

Re: Minor changes to Marvin tests for Netscaler

2013-08-15 Thread Prasanna Santhanam
On Thu, Aug 15, 2013 at 04:27:19AM +, Sowmya Krishnan wrote: > I am planning to modify some of the Netscaler tests (test_netscaler_lb_algo, > test_netscaler_lb_sticky, test_netscaler_lb) in our integration suite to > include adding of NS device in the setUpClass(). And remove the device in >

Minor changes to Marvin tests for Netscaler

2013-08-14 Thread Sowmya Krishnan
I am planning to modify some of the Netscaler tests (test_netscaler_lb_algo, test_netscaler_lb_sticky, test_netscaler_lb) in our integration suite to include adding of NS device in the setUpClass(). And remove the device in tearDownClass(). Currently this is not consistently done across all test