> On July 26, 2013, 8:46 a.m., Sheng Yang wrote: > > test/integration/component/test_egress_fw_rules.py, line 166 > > <https://reviews.apache.org/r/12934/diff/2/?file=328044#file328044line166> > > > > Missing clean up for account and domain. > > Ashutosh Kelkar wrote: > No, cleanup for account and domain is in teamDownClass.
I didn't saw adding account or domain last time, but I found it now. I must be tired… > On July 26, 2013, 8:46 a.m., Sheng Yang wrote: > > test/integration/component/test_egress_fw_rules.py, line 183 > > <https://reviews.apache.org/r/12934/diff/2/?file=328044#file328044line183> > > > > Missing tearDown() > > Ashutosh Kelkar wrote: > No, teamDown is not missing. it's there on line 352. Yes. > On July 26, 2013, 8:46 a.m., Sheng Yang wrote: > > test/integration/component/test_egress_fw_rules.py, line 253 > > <https://reviews.apache.org/r/12934/diff/2/?file=328044#file328044line253> > > > > I think we already have alike mechanism for executing inside user VM? > > Check get_ssh_client. If it doesn't fit, we need some generic mechanism > > rather than one method for one test case. > > Ashutosh Kelkar wrote: > Yes, that method works when VM is created with mode=advance. Here we > don't want to create PF rule and setup public ip for a VM. we just want to > test the egress rule and policy. In this case we have to take hopes to > Hypervisor Host-> Router-> VM instance to reach out VM for checking the > public network access. > Got it. But still it should be a good idea to add it to generic mechanism rather than implement it for each test case. > On July 26, 2013, 8:46 a.m., Sheng Yang wrote: > > test/integration/component/test_egress_fw_rules.py, line 619 > > <https://reviews.apache.org/r/12934/diff/2/?file=328044#file328044line619> > > > > You need to wait a while (e.g. 60 seconds) for mgmt server complete the > > updating of RvR status. > > Ashutosh Kelkar wrote: > VirtualMachine.Create() waits till VM is up and VM comes up only after > Network is up leading RVR up. no need to put a wait here. No, the detection of RvR state is an separate thread, it won't change as soon as the VR is up. So you need to wait for a while. But in fact the VM's start up would likely take more than more minute, so it should be fine. - Sheng ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12934/#review23920 ----------------------------------------------------------- On July 26, 2013, 6:27 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12934/ > ----------------------------------------------------------- > > (Updated July 26, 2013, 6:27 a.m.) > > > Review request for cloudstack, Girish Shilamkar, Jayapal Reddy, and Prasanna > Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > Tests for egress firewall rules for advance zone. > > > Diffs > ----- > > test/integration/component/test_egress_fw_rules.py PRE-CREATION > > Diff: https://reviews.apache.org/r/12934/diff/ > > > Testing > ------- > > > Thanks, > > Ashutosh Kelkar > >