openstack/swift
Code Issues Proposed changes

auditors: Log and dump recon *before* sleeping

Browse Source 
No good reason to delay getting the info about a completed run out there.

Change-Id: I7d5c19304a5c5f83558e91624de356caaf0ab4d5
This commit is contained in:
Tim Burke
2021-04-24 22:21:33 -07:00
parent 4137326378
commit 9d6006f646
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
swift/common/db_auditor.py
View File

@@ -101,14 +101,14 @@ class DatabaseAuditor(Daemon):
self.logger.increment('errors') self.logger.increment('errors')
self.logger.exception(_('ERROR auditing')) self.logger.exception(_('ERROR auditing'))
elapsed = time.time() - begin elapsed = time.time() - begin
if elapsed < self.interval:
time.sleep(self.interval - elapsed)
self.logger.info( self.logger.info(
_('%(server_type)s audit pass completed: %(elapsed).02fs'), _('%(server_type)s audit pass completed: %(elapsed).02fs'),
{'elapsed': elapsed, 'server_type': self.server_type.title()}) {'elapsed': elapsed, 'server_type': self.server_type.title()})
dump_recon_cache({ dump_recon_cache({
'{}_auditor_pass_completed'.format(self.server_type): elapsed}, '{}_auditor_pass_completed'.format(self.server_type): elapsed},
self.rcache, self.logger) self.rcache, self.logger)
if elapsed < self.interval:
time.sleep(self.interval - elapsed)
def run_once(self, *args, **kwargs): def run_once(self, *args, **kwargs):
"""Run the database audit once.""" """Run the database audit once."""