openstack
/
deb-networking-cisco
Code Issues Proposed changes

Add oslo db retry decorator to the RPC handlers 

Just as the L3 agent callbacks use a retry decorator to handle
DB deadlocks due to evenlet yields the cisco L3 cfg agent should
employ this decorator for operations analogous to those in the
L3 agent callbacks.

This patch makes code changes for this.

Closes-bug: #1551664
(cherry picked from commit c9c1d9024a52c39e33b7b79e116ea951bfa1ccd4)

Change-Id: I0efec8be384190ea3ca15c7475e19ba122d60856
This commit is contained in:
Bob Melander 2015-11-06 15:29:15 +01:00
parent cd714f4173
commit e7645aeac7
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
networking_cisco/plugins/cisco/l3/rpc/l3_router_cfg_agent_rpc_cb.py
View File

@ -21,6 +21,7 @@ from neutron.common import constants
from neutron.common import exceptions
from neutron.common import utils
from neutron import context as neutron_context
from neutron.db import api as db_api
from neutron.extensions import l3
from neutron.extensions import portbindings
from neutron import manager
@ -48,6 +49,7 @@ class L3RouterCfgRpcCallback(object):
return self._plugin
# version 1.0 API
@db_api.retry_db_errors
def cfg_sync_routers(self, context, host, router_ids=None,
hosting_device_ids=None):
"""Sync routers according to filters to a specific Cisco cfg agent.
@ -75,6 +77,7 @@ class L3RouterCfgRpcCallback(object):
return routers
# version 1.2 API
@db_api.retry_db_errors
def cfg_sync_all_hosted_routers(self, context, host):
adm_context = neutron_context.get_admin_context()
try: