You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Sowmya Krishnan <so...@citrix.com> on 2013/08/20 07:43:40 UTC
Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/
-----------------------------------------------------------
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
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by venkata swamy babu budumuru <ve...@citrix.com>.
> On Aug. 20, 2013, 6:30 a.m., venkata swamy babu budumuru wrote:
> > tools/marvin/marvin/integration/lib/common.py, line 62
> > <https://reviews.apache.org/r/13680/diff/1/?file=342462#file342462line62>
> >
> > IMO, we don't have to go for an additional library call for add_netscaler rather we can rely just on Netscaler.add in base.py. few reasons for saying this is as mentioned below.
> >
> > 1. Usually the automation environment is prepared out of a .cfg file where we can add the following to make a provider configured and enabled by default using something like below
> >
> > "providers": [
> >
> > { "broadcastdomainrange": "ZONE",
> >
> > "name": "Netscaler"
> > }
> > ]
> >
> >
> >
>
> Sowmya Krishnan wrote:
> The reason why I thought we need a separate library is because, adding NS involves the following: finding the physical network id, adding NS, finding the NS service providers, and enabling it if not enabled. In all, this is about 25 odd lines of code in 20 different set up classes across all tests. So I added a library.
>
> Regarding automation pre-configured environment which may include NS device as well, Here's the reason why I chose to add NS device in each test:
>
> Currently the tests assume that the NS device added by the test is the only one in the environment. We assert tests by doing an ssh into NS and testing if services are present. Now, if there are devices added externally unknown to the test or if there are more devices than those assumed by the test script, then the test potentially could fail looking at the wrong NS. That's the reason for adding NS independently in each test.
>
> Please let me know if any comments.
ok..that's fine.
- venkata swamy babu
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25340
-----------------------------------------------------------
On Aug. 20, 2013, 6:18 a.m., Sowmya Krishnan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13680/
> -----------------------------------------------------------
>
> (Updated Aug. 20, 2013, 6:18 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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by Sowmya Krishnan <so...@citrix.com>.
> On Aug. 20, 2013, 6:30 a.m., venkata swamy babu budumuru wrote:
> > test/integration/component/test_netscaler_lb.py, line 166
> > <https://reviews.apache.org/r/13680/diff/1/?file=342459#file342459line166>
> >
> > Since you have multiple test suites trying to add the same netscaler device, it will be good to keep this whole creation stuff in try block so that even if something fails then you will have your tearDown called.
> >
> > something like I mentioned here :
> >
> > try:
> >
> > add_netscaler()
> > NetworkOffering.create()
> > ServiceOffering.create()
> >
> > except:
> >
> > call tearDownClass()
> >
> > If we do it in the above way then if something goes wrong in network offering creation then our except block will still call the tearDown for removing netscaler device so that your rest of test suites will not fail during netscaler device addition.
Ok. Will do this for all tests
> On Aug. 20, 2013, 6:30 a.m., venkata swamy babu budumuru wrote:
> > tools/marvin/marvin/integration/lib/common.py, line 62
> > <https://reviews.apache.org/r/13680/diff/1/?file=342462#file342462line62>
> >
> > IMO, we don't have to go for an additional library call for add_netscaler rather we can rely just on Netscaler.add in base.py. few reasons for saying this is as mentioned below.
> >
> > 1. Usually the automation environment is prepared out of a .cfg file where we can add the following to make a provider configured and enabled by default using something like below
> >
> > "providers": [
> >
> > { "broadcastdomainrange": "ZONE",
> >
> > "name": "Netscaler"
> > }
> > ]
> >
> >
> >
The reason why I thought we need a separate library is because, adding NS involves the following: finding the physical network id, adding NS, finding the NS service providers, and enabling it if not enabled. In all, this is about 25 odd lines of code in 20 different set up classes across all tests. So I added a library.
Regarding automation pre-configured environment which may include NS device as well, Here's the reason why I chose to add NS device in each test:
Currently the tests assume that the NS device added by the test is the only one in the environment. We assert tests by doing an ssh into NS and testing if services are present. Now, if there are devices added externally unknown to the test or if there are more devices than those assumed by the test script, then the test potentially could fail looking at the wrong NS. That's the reason for adding NS independently in each test.
Please let me know if any comments.
- Sowmya
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25340
-----------------------------------------------------------
On Aug. 20, 2013, 6:18 a.m., Sowmya Krishnan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13680/
> -----------------------------------------------------------
>
> (Updated Aug. 20, 2013, 6:18 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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by venkata swamy babu budumuru <ve...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25340
-----------------------------------------------------------
test/integration/component/test_netscaler_lb.py
<https://reviews.apache.org/r/13680/#comment49726>
Since you have multiple test suites trying to add the same netscaler device, it will be good to keep this whole creation stuff in try block so that even if something fails then you will have your tearDown called.
something like I mentioned here :
try:
add_netscaler()
NetworkOffering.create()
ServiceOffering.create()
except:
call tearDownClass()
If we do it in the above way then if something goes wrong in network offering creation then our except block will still call the tearDown for removing netscaler device so that your rest of test suites will not fail during netscaler device addition.
tools/marvin/marvin/integration/lib/common.py
<https://reviews.apache.org/r/13680/#comment49725>
IMO, we don't have to go for an additional library call for add_netscaler rather we can rely just on Netscaler.add in base.py. few reasons for saying this is as mentioned below.
1. Usually the automation environment is prepared out of a .cfg file where we can add the following to make a provider configured and enabled by default using something like below
"providers": [
{ "broadcastdomainrange": "ZONE",
"name": "Netscaler"
}
]
- venkata swamy babu budumuru
On Aug. 20, 2013, 6:18 a.m., Sowmya Krishnan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13680/
> -----------------------------------------------------------
>
> (Updated Aug. 20, 2013, 6:18 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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
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 <ts...@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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25400
-----------------------------------------------------------
Commit e4b5d4d62e0e3c90407f426f3de1bd300b9c0359 in branch refs/heads/master from Sowmya Krishnan
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e4b5d4d ]
CLOUDSTACK-3927, CLOUDSTACK-3928: Fix NS scripts to add/remove NS device in setupClass/tearDown
Signed-off-by: Prasanna Santhanam <ts...@apache.org>
(cherry picked from commit 1c31ee22d40d77c10593d87b8237cd0489d192cc)
- 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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25399
-----------------------------------------------------------
Commit e4b5d4d62e0e3c90407f426f3de1bd300b9c0359 in branch refs/heads/master from Sowmya Krishnan
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e4b5d4d ]
CLOUDSTACK-3927, CLOUDSTACK-3928: Fix NS scripts to add/remove NS device in setupClass/tearDown
Signed-off-by: Prasanna Santhanam <ts...@apache.org>
(cherry picked from commit 1c31ee22d40d77c10593d87b8237cd0489d192cc)
- 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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25398
-----------------------------------------------------------
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 <ts...@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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by Prasanna Santhanam <ts...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25403
-----------------------------------------------------------
Ship it!
Ship It!
- Prasanna Santhanam
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
>
>
RE: Review Request 13680: CLOUDSTACK-3927, CLOUDSTACK-3928 Fix to
add/remove NS devices in setupClass/tearDownClass
Posted by Sowmya Krishnan <so...@citrix.com>.
Rayees, good point. But is it related to this fix? Probably we can have that as a separate patch? This patch is for handling NS device add and remove in setupClass/teardown for fixing failures in NS tests.
I would imagine not just Netscaler IP, but also the public and private interfaces happens to be hardcoded. We need to fix all of them. Can you open a separate bug for that?
> -----Original Message-----
> From: Rayees Namathponnan [mailto:noreply@reviews.apache.org] On Behalf
> Of Rayees Namathponnan
> Sent: Thursday, August 22, 2013 12:53 AM
> To: Prasanna Santhanam; Venkata SwamyBabu Budumuru
> Cc: Rayees Namathponnan; Sowmya Krishnan; cloudstack
> Subject: Re: Review Request 13680: CLOUDSTACK-3927, CLOUDSTACK-3928 Fix
> to add/remove NS devices in setupClass/tearDownClass
>
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13680/#review25378
> -----------------------------------------------------------
>
>
> There are hard coded IPs for netscaler devices; its very hard to manage during
> the runs; we should know how many netscaler devices required for entire
> regression suite and that need to added like below
>
> "netscaler": {
> 65 "ipaddress": 'NETSCALER_DEVICE_1',
> 66 "username": 'nsroot',
> 67 "password": 'nsroot',
>
> 75 },
>
> Its better to replace hardcodes IP from test with NETSCALER_DEVICE_1 ,
> NETSCALER_DEVICE_2 ..
>
>
>
> - Rayees Namathponnan
>
>
> 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
> >
> >
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by Rayees Namathponnan <ra...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/#review25378
-----------------------------------------------------------
There are hard coded IPs for netscaler devices; its very hard to manage during the runs; we should know how many netscaler devices required for entire regression suite and that need to added like below
"netscaler": {
65 "ipaddress": 'NETSCALER_DEVICE_1',
66 "username": 'nsroot',
67 "password": 'nsroot',
75 },
Its better to replace hardcodes IP from test with NETSCALER_DEVICE_1 , NETSCALER_DEVICE_2 ..
- Rayees Namathponnan
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
>
>
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by Sowmya Krishnan <so...@citrix.com>.
-----------------------------------------------------------
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
Re: Review Request 13680: CLOUDSTACK-3927,
CLOUDSTACK-3928 Fix to add/remove NS devices in setupClass/tearDownClass
Posted by Prasanna Santhanam <ts...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13680/
-----------------------------------------------------------
(Updated Aug. 20, 2013, 6:18 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