openstack
/
deb-networking-ovn
Code Issues Proposed changes

Remove two log traces that are no longer useful 

This patch removes two traces that used to be useful during
development to detect when a process had the lock to handle a
notification. Since IDL will call notify() for every "update" rpc
method it receives from ovsdb-server, this trace can be quite frequent
and noisy.

In a normal tempest execution in gate it has around a 7% (~6500
occurences) of the total log entries.

Change-Id: I4d2a0be86f07b55d631bacf21d090635625a8ec6
This commit is contained in:
Daniel Alvarez 2017-06-21 17:55:19 +02:00
parent 3f6da12f13
commit b399986af2
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
networking_ovn/ovsdb/ovsdb_monitor.py
View File

@ -247,10 +247,7 @@ class OvnIdl(BaseOvnIdl):
# Do not handle the notification if the event lock is requested,
# but not granted by the ovsdb-server.
if (self.is_lock_contended and not self.has_lock):
LOG.debug("Don't have the event lock to handle the notify"
" events. Ignoring the event : %s", event)
return
LOG.debug("Have the event lock to handle the notify events")
self.notify_handler.notify(event, row, updates)
def post_connect(self):