> On Aug. 29, 2013, 4:33 a.m., Prasanna Santhanam wrote:
> > The NetScaler provider is not different from any other provider and the
> > test should not enable/disable it unless that's the test itself. The
> > deployment configuration would need to be done differently. The providers
> > required for a deployment should be enabled prior to the test run. This is
> > the same case as deploying a basic zone. We don't configure the security
> > group provider in the test, it just becomes part of the marvin deployment
> > configuration.
> >
> > eg: In the case of advanced zone with security groups
> > (setup/dev/advancedsg.cfg) you have the following section.
> > deployDataCenter.py configures and enables the provider for you.
> >
> > "providers": [
> > {
> > "broadcastdomainrange": "ZONE",
> > "name": "VirtualRouter"
> > },
> > {
> > "broadcastdomainrange": "ZONE",
> > "name": "SecurityGroupProvider"
> > }
> > ],
Prasanna - Can we enable NetScaler as external device without adding any device
in marvin config file ?
like
"providers": [
{
"broadcastdomainrange": "ZONE",
"name": "VirtualRouter"
},
{
"broadcastdomainrange": "ZONE",
"name": "SecurityGroupProvider"
}
{
"broadcastdomainrange": "ZONE",
"name": "Netscaler"
}
- Rayees
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13889/#review25707
-----------------------------------------------------------
On Aug. 28, 2013, 5:59 p.m., Sowmya Krishnan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13889/
> -----------------------------------------------------------
>
> (Updated Aug. 28, 2013, 5:59 p.m.)
>
>
> Review request for cloudstack, venkata swamy babu budumuru and Prasanna
> Santhanam.
>
>
> Bugs: CLOUDSTACK-4487
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> For a brand new set up, Netscaler service provider won't be present by
> defualt.
> Fixed the same in common.py.
>
>
> Diffs
> -----
>
> tools/marvin/marvin/integration/lib/common.py f27e87d
>
> Diff: https://reviews.apache.org/r/13889/diff/
>
>
> Testing
> -------
>
> Tested on both new MS and an existing one.
>
>
> Thanks,
>
> Sowmya Krishnan
>
>
