]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Catch up RPC context fixes on NEC OpenFlow plugin
authorAkihiro MOTOKI <motoki@da.jp.nec.com>
Sun, 30 Dec 2012 20:58:36 +0000 (05:58 +0900)
committerAkihiro MOTOKI <motoki@da.jp.nec.com>
Tue, 15 Jan 2013 14:42:50 +0000 (23:42 +0900)
Fixes bug 1099179

Change-Id: Ib61081c3aaed8770906dd57874c45358b0418c3f

quantum/plugins/nec/agent/nec_quantum_agent.py
quantum/plugins/nec/nec_plugin.py

index 7e557fd4e7f78f6ef26c86ee5742a27dbceae653..8c8786b4e204e645cf68a653ae7f3a5f7a4107fd 100755 (executable)
@@ -28,7 +28,7 @@ import time
 from quantum.agent.linux import ovs_lib
 from quantum.common import config as logging_config
 from quantum.common import topics
-from quantum.openstack.common import context
+from quantum import context
 from quantum.openstack.common import log as logging
 from quantum.openstack.common import rpc
 from quantum.plugins.nec.common import config
@@ -54,8 +54,7 @@ class NECQuantumAgent(object):
         self.datapath_id = "0x%s" % self.int_br.get_datapath_id()
 
         # RPC network init
-        self.context = context.RequestContext('quantum', 'quantum',
-                                              is_admin=False)
+        self.context = context.get_admin_context_without_session()
         self.conn = rpc.create_connection(new=True)
 
     def update_ports(self, port_added=[], port_removed=[]):
index 3c82b52a13f606e098a95a3a00f22a3fc5bf418a..734f86385a1e5119dcc59210e992de021d7a25ed 100644 (file)
 # @author: Ryota MIBU
 
 from quantum.common import topics
+from quantum.common import rpc as q_rpc
 from quantum import context
 from quantum.db import dhcp_rpc_base
 from quantum.db import l3_db
 from quantum.openstack.common import log as logging
 from quantum.openstack.common import rpc
-from quantum.openstack.common.rpc import dispatcher
 from quantum.plugins.nec.common import config
 from quantum.plugins.nec.common import exceptions as nexc
 from quantum.plugins.nec.db import api as ndb
@@ -495,7 +495,6 @@ class NECPluginV2RPCCallbacks(dhcp_rpc_base.DhcpRpcCallbackMixin):
 
     def __init__(self, plugin):
         self.plugin = plugin
-        self.admin_context = context.get_admin_context()
 
     def create_rpc_dispatcher(self):
         '''Get the rpc dispatcher for this manager.
@@ -503,7 +502,7 @@ class NECPluginV2RPCCallbacks(dhcp_rpc_base.DhcpRpcCallbackMixin):
         If a manager would like to set an rpc API version, or support more than
         one class as the target of rpc messages, override this method.
         '''
-        return dispatcher.RpcDispatcher([self])
+        return q_rpc.PluginRpcDispatcher([self])
 
     def update_ports(self, rpc_context, **kwargs):
         """Update ports' information and activate/deavtivate them.
@@ -521,15 +520,15 @@ class NECPluginV2RPCCallbacks(dhcp_rpc_base.DhcpRpcCallbackMixin):
         datapath_id = kwargs['datapath_id']
         for p in kwargs.get('port_added', []):
             id = p['id']
-            port = self.plugin.get_port(self.admin_context, id)
+            port = self.plugin.get_port(rpc_context, id)
             if port and ndb.get_portinfo(id):
                 ndb.del_portinfo(id)
-                self.plugin.deactivate_port(self.admin_context, port)
+                self.plugin.deactivate_port(rpc_context, port)
             ndb.add_portinfo(id, datapath_id, p['port_no'],
                              mac=p.get('mac', ''))
-            self.plugin.activate_port_if_ready(self.admin_context, port)
+            self.plugin.activate_port_if_ready(rpc_context, port)
         for id in kwargs.get('port_removed', []):
-            port = self.plugin.get_port(self.admin_context, id)
+            port = self.plugin.get_port(rpc_context, id)
             if port and ndb.get_portinfo(id):
                 ndb.del_portinfo(id)
-                self.plugin.deactivate_port(self.admin_context, port)
+                self.plugin.deactivate_port(rpc_context, port)