Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review32598 --- Ship it! Ship It! - Santhosh Edukulla On Jan. 14, 2014, 8:51 a.m

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review31717 --- Commit c108d7f0d0e1e1e746dd186e3153c2d8387da7e2 in branch refs/heads

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review31716 --- Ship it! Ship It! - Girish Shilamkar On Jan. 14, 2014, 8:51 a.m.

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/ --- (Updated Jan. 14, 2014, 8:51 a.m.) Review request for cloudstack, Girish Shilam

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review31116 --- Ship it! Ship It! - Santhosh Edukulla On Dec. 27, 2013, 11:49 a.

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/ --- (Updated Dec. 27, 2013, 11:49 a.m.) Review request for cloudstack, Girish Shila

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review30910 --- Ship it! Committed the first patch to 4.3 and master. - Girish Shi

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review30909 --- Commit 4d9b95e13d9c59009cf359ab3c869e2bb48ba62f in branch refs/heads

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review30908 --- Commit 029f4e36a0188520f46d7997df3e53ab1661fb63 in branch refs/heads

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review30862 --- Ship it! Ship It! - Santhosh Edukulla On Dec. 16, 2013, 10:01 a.

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/ --- (Updated Dec. 16, 2013, 10:01 a.m.) Review request for cloudstack, Girish Shila

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

> On Dec. 11, 2013, 6:04 p.m., Santhosh Edukulla wrote: > > test/integration/component/test_advancedsg_networks.py, line 1702 > > > > > > There is no clean up i believe in case of an exception? The important operatio

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/ --- (Updated Dec. 16, 2013, 9:45 a.m.) Review request for cloudstack, Girish Shilam

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

> On Dec. 12, 2013, 12:53 p.m., Santhosh Edukulla wrote: > > test/integration/component/test_advancedsg_networks.py, line 340 > > > > > > Ideally, deletion failure is also a failure assuming CS was not able to > > del

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review30255 --- test/integration/component/test_advancedsg_networks.py

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

> On Dec. 11, 2013, 6:04 p.m., Santhosh Edukulla wrote: > > test/integration/component/test_advancedsg_networks.py, line 376 > > > > > > Just verify the order of deletion. nwofferings or domain? If we have domain spec

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review30203 --- test/integration/component/test_advancedsg_networks.py

Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/ --- Review request for cloudstack, Girish Shilamkar, Santhosh Edukulla, and Srikante