]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Change the default l3_agent_manager to L3NATAgent
authorAkihiro MOTOKI <motoki@da.jp.nec.com>
Sat, 2 Mar 2013 10:45:16 +0000 (19:45 +0900)
committerAkihiro MOTOKI <motoki@da.jp.nec.com>
Sun, 3 Mar 2013 07:58:10 +0000 (16:58 +0900)
This is a l3-agent version of 349aa3ef3563c019f4d0a98fc30117ae0a6cade3.

The combination of the plugin without agent extension support
and L3NATAgentWithStateReport results in an error:
"AttributeError: No such RPC function 'report_state'".

This patch changes l3_agent_manager to be L3NATAgent instead of
L3NATAgentWithStateReport since all plugins do not support
L3NATAgentWithStateReport and having this as the default breaks
all current deployments that upgrade source without changing
their config files.

Fixes bug 1139726

Change-Id: Ibb5c04c89bfad741ec43a372cbf97445f7b3d76c

etc/l3_agent.ini
quantum/agent/l3_agent.py

index 133576c3f37c5b48fd3ae175d3eae6a7f73c5f5c..c08fa37cb052cb269e65caba44d8f8d06d3e3b57 100644 (file)
@@ -2,6 +2,9 @@
 # Show debugging output in log (sets DEBUG log level output)
 # debug = True
 
+# The Quantum L3 Agent manager
+# l3_agent_manager = quantum.agent.l3_agent.L3NATAgent
+
 # L3 requires that an interface driver be set.  Choose the one that best
 # matches your plugin.
 
index c67ebe6a190220a59efa1eccb63bbb80fa0a48e6..bfaa1c415e51596d6ddce9192c6671082862f556 100644 (file)
@@ -144,7 +144,7 @@ class L3NATAgent(manager.Manager):
                    help=_("UUID of external network for routers implemented "
                           "by the agents.")),
         cfg.StrOpt('l3_agent_manager',
-                   default='quantum.agent.l3_agent.L3NATAgentWithStateReport',
+                   default='quantum.agent.l3_agent.L3NATAgent',
                    help=_("The Quantum L3 Agent manager.")),
     ]