From 780d571aed1b1278895e5b9e85b5209772d7ba16 Mon Sep 17 00:00:00 2001 From: Assaf Muller Date: Tue, 11 Aug 2015 14:20:07 -0400 Subject: [PATCH] Clean up test_dvr_router_rem_fips_on_restarted_agent * self._add_fip wasn't actually doing anything useful, it just throws off the reader by making him/her think that line is important when it isn't. * Added an assertion that checks that the FIP namespace exists before the reconfiguration of the router. Change-Id: Ifd27f20d9d8d11575212b95e3bbe534bfe2d3b8f --- neutron/tests/functional/agent/test_l3_agent.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/neutron/tests/functional/agent/test_l3_agent.py b/neutron/tests/functional/agent/test_l3_agent.py index ef2bd498ed8..18b8c3347e2 100644 --- a/neutron/tests/functional/agent/test_l3_agent.py +++ b/neutron/tests/functional/agent/test_l3_agent.py @@ -1276,8 +1276,8 @@ class TestDvrRouter(L3AgentTestFramework): self.agent.conf.agent_mode = 'dvr_snat' router_info = self.generate_dvr_router_info() router1 = self.manage_router(self.agent, router_info) - self._add_fip(router1, '192.168.111.12', self.agent.conf.host) fip_ns = router1.fip_ns.get_name() + self.assertTrue(self._namespace_exists(fip_ns)) restarted_agent = neutron_l3_agent.L3NATAgentWithStateReport( self.agent.host, self.agent.conf) router1.router[l3_constants.FLOATINGIP_KEY] = []