matthew/sonic-buildimage
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Lazy start log level thread

Browse Source
This commit is contained in:
Junchao-Mellanox 2023-12-29 11:39:08 +08:00 committed by GitHub
parent 64ac9db120
commit c027fefc17
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/sonic-py-common/sonic_py_common/logger.py
View File

@ -27,20 +27,20 @@ class Logger(object):
DEFAULT_LOG_OPTION = LOG_OPTION_NDELAY DEFAULT_LOG_OPTION = LOG_OPTION_NDELAY
def __init__(self, log_identifier=None, log_facility=DEFAULT_LOG_FACILITY, log_option=DEFAULT_LOG_OPTION, def __init__(self, log_identifier=None, log_facility=DEFAULT_LOG_FACILITY, log_option=DEFAULT_LOG_OPTION,
enable_set_log_level_on_fly=False): enable_set_log_level_on_fly=False, db_name=None):
if log_identifier is None: if log_identifier is None:
log_identifier = os.path.basename(sys.argv[0]) log_identifier = os.path.basename(sys.argv[0])
if db_name is None:
self.db_name = log_identifier
else:
self.db_name = db_name
self.enable_set_log_level_on_fly = enable_set_log_level_on_fly
# Initialize syslog # Initialize syslog
syslog.openlog(ident=log_identifier, logoption=log_option, facility=log_facility) syslog.openlog(ident=log_identifier, logoption=log_option, facility=log_facility)
self._logger = None self._logger = None
if enable_set_log_level_on_fly:
# Performance warning: linkToDbNative will potentially create a new thread.
# The thread listens to CONFIG DB for log level changes.
self.logger.linkToDbNative(log_identifier, 'NOTICE')
def __del__(self): def __del__(self):
syslog.closelog() syslog.closelog()
@ -64,6 +64,10 @@ class Logger(object):
SwssLogger.getInstance.return_value = instance SwssLogger.getInstance.return_value = instance
instance.getMinPrio.return_value = syslog.LOG_NOTICE instance.getMinPrio.return_value = syslog.LOG_NOTICE
self._logger = SwssLogger.getInstance() self._logger = SwssLogger.getInstance()
if self.enable_set_log_level_on_fly:
# Performance warning: linkToDbNative will potentially create a new thread.
# The thread listens to CONFIG DB for log level changes.
self._logger.linkToDbNative(self.db_name, 'NOTICE')
return self._logger return self._logger
# #