x/group-based-policy
Code Issues Proposed changes

[aim] Fix retry issues and logging

Browse Source 
Pass create_if_absent=False to AIM's get_status() to hopefully reduce
transaction retries due to DBDuplicateEntry exceptions. This required
unpinning the version of AIM used, as well as a couple of fixes in the
AIM repo.

Change the RPC handlers to use Neutron's retry_db_errors decorator
rather than its own, so that DBDuplicateEntry exceptions are retried.

Avoid logging at error level when processing retriable exceptions.

Change-Id: I53740eea3cb7cacafceae589deec3b573ef6a68a
This commit is contained in:
Robert Kukura
2017-06-07 00:44:08 -04:00
parent 5b80125b21
commit 953e5d6ae5
8 changed files with 58 additions and 53 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gbpservice/neutron/plugins/ml2plus/patch_neutron.py
View File

@@ -199,9 +199,6 @@ from neutron.plugins.ml2 import models
from sqlalchemy.orm import exc
LOCK_NAME = 'ml2_db'
# REVISIT: This method gets decorated in Pike for removal in Queens. So this
# patching might need to be changed in Pike and removed in Queens.
def patched_get_locked_port_and_binding(session, port_id):