From 6af52340ff1da075e0b03b089d24d197e33ece3f Mon Sep 17 00:00:00 2001 From: Gary Kotton Date: Thu, 29 Oct 2015 06:48:48 -0700 Subject: [PATCH] Fix usage of mutable object as default value Commit 0a300a2277a583fe28b00db2571982928c752554 added a bad parameter. Please see http://docs.python-guide.org/en/latest/writing/gotchas/ Change-Id: I0550c4c86774f8761287c596350f7461fec05fe4 --- neutron/tests/functional/agent/l2/base.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/neutron/tests/functional/agent/l2/base.py b/neutron/tests/functional/agent/l2/base.py index 9a29a72c3..39e511b11 100644 --- a/neutron/tests/functional/agent/l2/base.py +++ b/neutron/tests/functional/agent/l2/base.py @@ -120,7 +120,9 @@ class OVSAgentTestFramework(base.BaseOVSLinuxTestCase): agent.sg_agent = mock.Mock() return agent - def start_agent(self, agent, unplug_ports=[]): + def start_agent(self, agent, unplug_ports=None): + if unplug_ports is None: + unplug_ports = [] self.setup_agent_rpc_mocks(agent, unplug_ports) polling_manager = polling.InterfacePollingMinimizer() self.addCleanup(polling_manager.stop) -- 2.45.2