You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Rajesh Battala <ra...@citrix.com> on 2013/04/30 05:25:14 UTC

Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10858/
-----------------------------------------------------------

Review request for cloudstack, Kishan Kavala, Murali Reddy, Vijay Venkatachalam, and Ram Ganesh.


Description
-------

This feature will introduce Netscaler as external LB provider in VPC.
As of now only 1 tier is support for external LB.
A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
Existing NetscalerElement is used and implements VpcProvider.
In VpcManager, Netscaler is added as one of the supported providers.
Netscaler will be dedicated to the vpc.


Diffs
-----

  api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
  plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 2bbdb04 
  server/src/com/cloud/network/NetworkServiceImpl.java 2dcb47d 
  server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
  server/src/com/cloud/network/vpc/VpcManagerImpl.java 224a680 
  server/test/com/cloud/vpc/ChildTestConfiguration.java PRE-CREATION 
  server/test/com/cloud/vpc/VpcManagerTest.java PRE-CREATION 
  server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
  server/test/resources/VpcContext.xml PRE-CREATION 

Diff: https://reviews.apache.org/r/10858/diff/


Testing
-------

Manual Testing:
==============
1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
2. Deletion of Vpc with the default offering with NS is deleted successfully.
3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
6. Verified Only one tier with netscaler as LB provider can be created. 
7. Verified deploying Instance in the tier is successful.
8. Verified a new nic got created with gateway ip from the tier cidr.
9. Verified deployed instance should get the ip from the specified tier cidr range.
10. Acquire public ip in the vpc.
11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
12. Deletion of LB rule is successful.
13. Modification of LB rule is successful
14. Creation of LB Health Check of TCP type is successful.
15. Deletion of LB Health Check of TCP type is successful.
16. Creation of LB Health Check of HTTP type is successful.
17. Deletion of LB Health Check of HTTP type is successful.
18. IpAssoc command is executed successful on Netscaler.
19. Deletion of tier will delete the tier and config on netscaler is cleared
20. Deletion of tier will mark the netscaler to be in free mode.


Unit Test:
===========
Created VpcManger tests and added few tests to createNetworkOfferingTest


Thanks,

Rajesh Battala


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

Posted by Rajesh Battala <ra...@citrix.com>.

> On May 6, 2013, 3:36 p.m., Murali Reddy wrote:
> > plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java, line 967
> > <https://reviews.apache.org/r/10858/diff/1/?file=286075#file286075line967>
> >
> >     implement and shutdown need to be implemented? how would VLAN be configured on NetScaler with out implementing network?

When a tier network (where LB is Netscaler)  is implemented, In Netscaler Element (implement and shutdown) methods will call which will take care of Netscaler in that tier network. (It works like implementing a guest network [ here it’s a guest network with vpc id and had cidr taken from the super cidr]) 


> On May 6, 2013, 3:36 p.m., Murali Reddy wrote:
> > plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java, line 980
> > <https://reviews.apache.org/r/10858/diff/1/?file=286075#file286075line980>
> >
> >     remove the TODO's, if there is no  need for method implementation and just return true

will remove them


> On May 6, 2013, 3:36 p.m., Murali Reddy wrote:
> > server/src/com/cloud/network/NetworkServiceImpl.java, line 1056
> > <https://reviews.apache.org/r/10858/diff/1/?file=286076#file286076line1056>
> >
> >     same should hold for other external devices as well. so please put generic comment (with out netscaler)

will make the comment to be generic


- Rajesh


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10858/#review20191
-----------------------------------------------------------


On April 30, 2013, 3:28 a.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated April 30, 2013, 3:28 a.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 2bbdb04 
>   server/src/com/cloud/network/NetworkServiceImpl.java 2dcb47d 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java 224a680 
>   server/test/com/cloud/vpc/ChildTestConfiguration.java PRE-CREATION 
>   server/test/com/cloud/vpc/VpcManagerTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
>   server/test/resources/VpcContext.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

Posted by Murali Reddy <mu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10858/#review20191
-----------------------------------------------------------



plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
<https://reviews.apache.org/r/10858/#comment41413>

    implement and shutdown need to be implemented? how would VLAN be configured on NetScaler with out implementing network?



plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
<https://reviews.apache.org/r/10858/#comment41414>

    remove the TODO's, if there is no  need for method implementation and just return true



server/src/com/cloud/network/NetworkServiceImpl.java
<https://reviews.apache.org/r/10858/#comment41422>

    same should hold for other external devices as well. so please put generic comment (with out netscaler)


- Murali Reddy


On April 30, 2013, 3:28 a.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated April 30, 2013, 3:28 a.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 2bbdb04 
>   server/src/com/cloud/network/NetworkServiceImpl.java 2dcb47d 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java 224a680 
>   server/test/com/cloud/vpc/ChildTestConfiguration.java PRE-CREATION 
>   server/test/com/cloud/vpc/VpcManagerTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
>   server/test/resources/VpcContext.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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/10858/#review22001
-----------------------------------------------------------


Commit 3e3e5830b45eeba16b3dacfb5475d53d8d2dee27 in branch refs/heads/master-6-17-stable from Jessica Wang
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3e3e583 ]

CLOUDSTACK-869: Add VPC dialog - add Public LB Provider dropdown, remove VPC Offering dropdown. When Public LB Provider is selected as Netscaler, pass "Default VPC offering with Netscaler" to createVPC API. When Public LB Provider is selected as VpcVirtualRouter, pass "Default VPC Offering" to createVPC API.


- ASF Subversion and Git Services


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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/10858/#review20382
-----------------------------------------------------------


Commit 2e2106002865cff8eb06b5c3a42bfd6f0f6848fb in branch refs/heads/internallb from Murali Reddy <mu...@gmail.com>
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=2e21060 ]

CLOUDSTACK-869 nTier Apps 2.0 : Support NetScalar as external LB provider


- ASF Subversion and Git Services


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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/10858/#review20374
-----------------------------------------------------------


Commit 2e2106002865cff8eb06b5c3a42bfd6f0f6848fb in branch refs/heads/master from Murali Reddy <mu...@gmail.com>
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=2e21060 ]

CLOUDSTACK-869 nTier Apps 2.0 : Support NetScalar as external LB provider


- ASF Subversion and Git Services


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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/10858/#review21571
-----------------------------------------------------------


Commit 5233e3216b11e69c7c7e051f0a6c3d0c6bf98803 in branch refs/heads/master from Pranav Saxena
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5233e32 ]

CLOUDSTACK-869:Netscaler support as an external LB provider


- ASF Subversion and Git Services


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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/10858/#review21567
-----------------------------------------------------------


Commit bcc5baa1636037baa8c5ffbd3bbf70df8af4d024 in branch refs/heads/master from Pranav Saxena
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=bcc5baa ]

CLOUDSTACK-869:Netscaler support as an external LB provider:front end


- ASF Subversion and Git Services


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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/10858/#review20385
-----------------------------------------------------------


Commit 2e2106002865cff8eb06b5c3a42bfd6f0f6848fb in branch refs/heads/ui-cisco-asa1000v-support from Murali Reddy <mu...@gmail.com>
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=2e21060 ]

CLOUDSTACK-869 nTier Apps 2.0 : Support NetScalar as external LB provider


- ASF Subversion and Git Services


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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/10858/#review21993
-----------------------------------------------------------


Commit a2c7d3a8a75b2cec266cef566b8828be7a1ebc72 in branch refs/heads/master from Jessica Wang
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a2c7d3a ]

CLOUDSTACK-869: Add VPC dialog - add Public LB Provider dropdown, remove VPC Offering dropdown. When Public LB Provider is selected as Netscaler, pass "Default VPC offering with Netscaler" to createVPC API. When Public LB Provider is selected as VpcVirtualRouter, pass "Default VPC Offering" to createVPC API.


- ASF Subversion and Git Services


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

Posted by Murali Reddy <mu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10858/#review20375
-----------------------------------------------------------

Ship it!


Commit: 2e2106002865cff8eb06b5c3a42bfd6f0f6848fb

- Murali Reddy


On May 8, 2013, 1:39 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10858/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:39 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.
> 
> 
> Description
> -------
> 
> This feature will introduce Netscaler as external LB provider in VPC.
> As of now only 1 tier is support for external LB.
> A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
> Existing NetscalerElement is used and implements VpcProvider.
> In VpcManager, Netscaler is added as one of the supported providers.
> Netscaler will be dedicated to the vpc.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
>   plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
>   server/pom.xml 808dd3e 
>   server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
>   server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
>   server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
>   server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
>   server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
> 
> Diff: https://reviews.apache.org/r/10858/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing:
> ==============
> 1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
> 2. Deletion of Vpc with the default offering with NS is deleted successfully.
> 3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
> 4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
> 5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
> 6. Verified Only one tier with netscaler as LB provider can be created. 
> 7. Verified deploying Instance in the tier is successful.
> 8. Verified a new nic got created with gateway ip from the tier cidr.
> 9. Verified deployed instance should get the ip from the specified tier cidr range.
> 10. Acquire public ip in the vpc.
> 11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
> 12. Deletion of LB rule is successful.
> 13. Modification of LB rule is successful
> 14. Creation of LB Health Check of TCP type is successful.
> 15. Deletion of LB Health Check of TCP type is successful.
> 16. Creation of LB Health Check of HTTP type is successful.
> 17. Deletion of LB Health Check of HTTP type is successful.
> 18. IpAssoc command is executed successful on Netscaler.
> 19. Deletion of tier will delete the tier and config on netscaler is cleared
> 20. Deletion of tier will mark the netscaler to be in free mode.
> 
> 
> Unit Test:
> ===========
> Created VpcManger tests and added few tests to createNetworkOfferingTest
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

Posted by Rajesh Battala <ra...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10858/
-----------------------------------------------------------

(Updated May 8, 2013, 1:39 p.m.)


Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.


Description
-------

This feature will introduce Netscaler as external LB provider in VPC.
As of now only 1 tier is support for external LB.
A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
Existing NetscalerElement is used and implements VpcProvider.
In VpcManager, Netscaler is added as one of the supported providers.
Netscaler will be dedicated to the vpc.


Diffs
-----

  api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
  plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
  server/pom.xml 808dd3e 
  server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
  server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
  server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
  server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
  server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 

Diff: https://reviews.apache.org/r/10858/diff/


Testing
-------

Manual Testing:
==============
1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
2. Deletion of Vpc with the default offering with NS is deleted successfully.
3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
6. Verified Only one tier with netscaler as LB provider can be created. 
7. Verified deploying Instance in the tier is successful.
8. Verified a new nic got created with gateway ip from the tier cidr.
9. Verified deployed instance should get the ip from the specified tier cidr range.
10. Acquire public ip in the vpc.
11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
12. Deletion of LB rule is successful.
13. Modification of LB rule is successful
14. Creation of LB Health Check of TCP type is successful.
15. Deletion of LB Health Check of TCP type is successful.
16. Creation of LB Health Check of HTTP type is successful.
17. Deletion of LB Health Check of HTTP type is successful.
18. IpAssoc command is executed successful on Netscaler.
19. Deletion of tier will delete the tier and config on netscaler is cleared
20. Deletion of tier will mark the netscaler to be in free mode.


Unit Test:
===========
Created VpcManger tests and added few tests to createNetworkOfferingTest


Thanks,

Rajesh Battala


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

Posted by Rajesh Battala <ra...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10858/
-----------------------------------------------------------

(Updated May 8, 2013, 1:38 p.m.)


Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.


Changes
-------

Incorporated the changes from the review comments.
All Unit tests are passing and rat is successful.


Description
-------

This feature will introduce Netscaler as external LB provider in VPC.
As of now only 1 tier is support for external LB.
A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
Existing NetscalerElement is used and implements VpcProvider.
In VpcManager, Netscaler is added as one of the supported providers.
Netscaler will be dedicated to the vpc.


Diffs (updated)
-----

  api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
  plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 7bd9c2e 
  server/pom.xml 808dd3e 
  server/src/com/cloud/network/NetworkServiceImpl.java 5e8be92 
  server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
  server/src/com/cloud/network/vpc/VpcManagerImpl.java a7f06e9 
  server/test/com/cloud/vpc/VpcTest.java PRE-CREATION 
  server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 

Diff: https://reviews.apache.org/r/10858/diff/


Testing
-------

Manual Testing:
==============
1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
2. Deletion of Vpc with the default offering with NS is deleted successfully.
3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
6. Verified Only one tier with netscaler as LB provider can be created. 
7. Verified deploying Instance in the tier is successful.
8. Verified a new nic got created with gateway ip from the tier cidr.
9. Verified deployed instance should get the ip from the specified tier cidr range.
10. Acquire public ip in the vpc.
11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
12. Deletion of LB rule is successful.
13. Modification of LB rule is successful
14. Creation of LB Health Check of TCP type is successful.
15. Deletion of LB Health Check of TCP type is successful.
16. Creation of LB Health Check of HTTP type is successful.
17. Deletion of LB Health Check of HTTP type is successful.
18. IpAssoc command is executed successful on Netscaler.
19. Deletion of tier will delete the tier and config on netscaler is cleared
20. Deletion of tier will mark the netscaler to be in free mode.


Unit Test:
===========
Created VpcManger tests and added few tests to createNetworkOfferingTest


Thanks,

Rajesh Battala


Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

Posted by Rajesh Battala <ra...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10858/
-----------------------------------------------------------

(Updated April 30, 2013, 3:28 a.m.)


Review request for cloudstack, Kishan Kavala, Murali Reddy, Alena Prokharchyk, Vijay Venkatachalam, and Ram Ganesh.


Changes
-------

added Alena 


Description
-------

This feature will introduce Netscaler as external LB provider in VPC.
As of now only 1 tier is support for external LB.
A new VPC offering will be created "Default VPC Offering with NS" with all the services provided by VPCVR and LB service with NetScaler.
Existing NetscalerElement is used and implements VpcProvider.
In VpcManager, Netscaler is added as one of the supported providers.
Netscaler will be dedicated to the vpc.


Diffs
-----

  api/src/com/cloud/network/vpc/VpcOffering.java 3961d0a 
  plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java 2bbdb04 
  server/src/com/cloud/network/NetworkServiceImpl.java 2dcb47d 
  server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java b1606db 
  server/src/com/cloud/network/vpc/VpcManagerImpl.java 224a680 
  server/test/com/cloud/vpc/ChildTestConfiguration.java PRE-CREATION 
  server/test/com/cloud/vpc/VpcManagerTest.java PRE-CREATION 
  server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java cbb6c00 
  server/test/resources/VpcContext.xml PRE-CREATION 

Diff: https://reviews.apache.org/r/10858/diff/


Testing
-------

Manual Testing:
==============
1. Creation of Vpc with the default offering with NS is created successfully. ( Enable Netscaler provider in network service providers)
2. Deletion of Vpc with the default offering with NS is deleted successfully.
3. Creation of new Vpc Network Offering with Netscaler as LB provider with dedicated mode is created successfully.
4. Creation of new Vpc Network Offering with Netscaler as LB provider with shared mode should throw exception.
5. Creation of tier (webtier) with the created Vpcnetscaler offering is created successfully.
6. Verified Only one tier with netscaler as LB provider can be created. 
7. Verified deploying Instance in the tier is successful.
8. Verified a new nic got created with gateway ip from the tier cidr.
9. Verified deployed instance should get the ip from the specified tier cidr range.
10. Acquire public ip in the vpc.
11. Verified creation on LB rule, is selecting only free dedicated Netscaler device and necessary configuration is created and LB rule is created on NS
12. Deletion of LB rule is successful.
13. Modification of LB rule is successful
14. Creation of LB Health Check of TCP type is successful.
15. Deletion of LB Health Check of TCP type is successful.
16. Creation of LB Health Check of HTTP type is successful.
17. Deletion of LB Health Check of HTTP type is successful.
18. IpAssoc command is executed successful on Netscaler.
19. Deletion of tier will delete the tier and config on netscaler is cleared
20. Deletion of tier will mark the netscaler to be in free mode.


Unit Test:
===========
Created VpcManger tests and added few tests to createNetworkOfferingTest


Thanks,

Rajesh Battala