> 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. > > > Sheng Yang wrote: > Got it. But still it should be a good idea to add it to generic mechanism > rather than implement it for each test case.
Sure, I will add an enhancement ticket and will update the base.py with VirtualMachine.exec_script_on_user_vm(). It will be help full in future test case dev. - Ashutosh ----------------------------------------------------------- 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 > >
