Added the required tests and merged to master -Koushik
> -----Original Message----- > From: Koushik Das [mailto:koushik....@citrix.com] > Sent: Friday, April 12, 2013 10:30 PM > To: dev@cloudstack.apache.org > Subject: RE: [MERGE] ASA 1000v as external firewall in isolated guest > networks > > > > > -----Original Message----- > > From: Chip Childers [mailto:chip.child...@sungard.com] > > Sent: Friday, April 12, 2013 7:27 PM > > To: dev@cloudstack.apache.org > > Subject: Re: [MERGE] ASA 1000v as external firewall in isolated guest > > networks > > > > On Fri, Apr 12, 2013 at 06:55:56PM +0530, Prasanna Santhanam wrote: > > > On Fri, Apr 12, 2013 at 09:20:39AM -0400, Chip Childers wrote: > > > > On Fri, Apr 12, 2013 at 06:39:29AM +0000, Koushik Das wrote: > > > > > Currently there are no functional tests, did e2e tests manually. > > > > > There are some external setup that is required which then needs > > > > > to be > > added to Cloudstack like Nexus 1000v, VNMC, ASA 1000v appliance > > details. I don't see any existing tests related to Vmware/Nexus stuff > > which I can reuse, so implementing all these steps would need some time. > > > > > > > > > > So wanted to check if it is ok to merge now and add the > > > > > functional tests > > subsequently? > > > > > > > > Before we merge it in, can we first discuss how we want to hand > > > > this sort of thing? It seems that at least a marvin test, not run > > > > by default, would be helpful here. Certainly documentation on how > > > > to setup an environment to support the test is also needed. > > > > > > > > Do other's have thoughts on this? > > > > > > +1 - I can spare some hardware to host virtual instances of the ASA > > > appliance. We do NetScaler VPXs in this fashion. But I'm not sure > > > what kind of pre-programming will be required on the ASA to run the > test. > > > So some documentation would certainly help. > > > > Koushik, > > > > I think that it will require working with Prasanna to get a Marvin > > test suite for the feature ready to go. I'd personally like to see > > that > > *prior* to the merge into master. We need to "up our game" on > > automated testing of new features, so that we stop digging ourselves > > deeper and deeper into a hole. > > Sure will work with Prasanna on this. > > Ideally I would like to write tests for network offering creation with VNMC as > provider and the newly added APIs for this feature and would like to reuse > existing code for testing the following features: > > - Vmware cluster setup with Nexus 1000v > - Network creation > - Vm deployment > - Acquiring public IP > - Static nat/pf rule creation > > Any help on the above would be appreciated. > > > > > > -chip