Browse Source

Lower log level for the maintenance thread

Prior to this patch the maintenance thread was logging an warning when
any inconsistencies were found between the Neutron and OVN database
without any message saying that it's going to be fixed automatically.
That caused confusion between users.

This patch is changing the message to be less "scary" and lowering it's
log level as well because the inconsistencies are going to be fixed
without the need of any manual interaction from the administrator.

DEBUG was choosen over INFO for the message because usually INFO is
choosen to indicate that something has been completed already instead
of some on-going operation. That said, we are already logging an INFO
message at the end of the synchronization to inform that it has been
completed and how long it took.

Change-Id: I520f3f1f6bd61c2ec8d8ed29260c7ebec8f0184f
changes/22/566822/2
Lucas Alvares Gomes 4 years ago
parent
commit
7bc8164c38
  1. 5
      networking_ovn/common/maintenance.py

5
networking_ovn/common/maintenance.py

@ -224,7 +224,8 @@ class DBInconsistenciesPeriodics(object):
delete_inconsistencies = db_maint.get_deleted_resources()
if not any([create_update_inconsistencies, delete_inconsistencies]):
return
LOG.warning('Inconsistencies found in the database!')
LOG.debug('Maintenance task: Synchronizing Neutron '
'and OVN databases')
self._sync_timer.restart()
# Fix the create/update resources inconsistencies
@ -260,7 +261,7 @@ class DBInconsistenciesPeriodics(object):
'res_type': row.resource_type})
self._sync_timer.stop()
LOG.info('Maintenance thread synchronization finished '
LOG.info('Maintenance task synchronization finished '
'(took %.2f seconds)', self._sync_timer.elapsed())
def _create_lrouter_port(self, port):

Loading…
Cancel
Save