From 57f8a83020878b2862b6f30aabaceec4854cac22 Mon Sep 17 00:00:00 2001 From: Chuck SmileyRekiere Date: Thu, 27 Jan 2022 10:37:55 -0700 Subject: [PATCH] lf_logger_config_test.py : removed spacing from simple logger format, added filename lf_logger_config_test.py : updated to show example of using lf_logger_config.json one directy up lf_logger_config.json , lf_logger_config_example.json : updated with simple format Signed-off-by: Chuck SmileyRekiere --- py-scripts/lf_logger_config.json | 6 +++--- py-scripts/lf_logger_config.py | 5 ++++- py-scripts/sandbox/lf_logger_config_example.json | 2 +- py-scripts/sandbox/lf_logger_config_test.py | 2 +- 4 files changed, 9 insertions(+), 6 deletions(-) diff --git a/py-scripts/lf_logger_config.json b/py-scripts/lf_logger_config.json index b12a8c29..eb42e0d8 100644 --- a/py-scripts/lf_logger_config.json +++ b/py-scripts/lf_logger_config.json @@ -3,13 +3,13 @@ "disable_existing_loggers": false, "formatters": { "simple": { - "format": "%(created)-16f %(name)-16s %(levelname)-12s %(lineno)-6s %(funcName)-16s [%(module)s]: %(message)s" + "format": "%(created)f %(name)s %(levelname)-8s %(filename)s %(lineno)s %(funcName)s [%(module)s]: %(message)s" }, "extra": { - "format":"%(asctime)-16s %(name)-8s %(filename)-12s %(lineno)-6s %(funcName)-16s %(levelname)-8s [%(module)s]: %(message)s", + "format":"%(asctime)-16s %(name)-8s %(levelname)-8s %(filename)-12s %(lineno)-6s %(funcName)-16s [%(module)s]: %(message)s", "datefmt":"%m-%d %H:%M:%S"}, "extra2": { - "format":"%(created)f %(name)-8s %(filename)-12s %(lineno)-6s %(funcName)-16s %(levelname)-8s [%(module)s]: %(message)s", + "format":"%(created)f %(name)-8s %(levelname)-8s %(filename)-12s %(lineno)-6s %(funcName)-16s [%(module)s]: %(message)s", "datefmt":"%m-%d %H:%M:%S"} }, diff --git a/py-scripts/lf_logger_config.py b/py-scripts/lf_logger_config.py index f1e8e761..adc2552d 100644 --- a/py-scripts/lf_logger_config.py +++ b/py-scripts/lf_logger_config.py @@ -79,7 +79,10 @@ class lf_logger_config: # for now just configure the output formatting. Basic defaults # Change to level=logging.WARNING , also may limit some of the output logging.basicConfig(handlers=[logging.StreamHandler(stream=sys.stdout)], level=logging.INFO, - format='%(created)-16f %(name)-8s %(levelname)-12s %(lineno)-6s %(funcName)-30s [%(module)s]: %(message)s') + format='%(created)f %(name)s %(levelname)-8s %(filename)s %(lineno)s %(funcName)s [%(module)s]: %(message)s') + # Note: leave this for reference + # logging.basicConfig(handlers=[logging.StreamHandler(stream=sys.stdout)], level=logging.INFO, + # format='%(created)-16f %(name)-8s %(levelname)-12s %(lineno)-6s %(funcName)-30s [%(module)s]: %(message)s') # Note the propagate is tricky in the sence if not set correctly will create duplicate logs output, # setting to false logging.propagate = False diff --git a/py-scripts/sandbox/lf_logger_config_example.json b/py-scripts/sandbox/lf_logger_config_example.json index e2bd8614..2f9f1d05 100644 --- a/py-scripts/sandbox/lf_logger_config_example.json +++ b/py-scripts/sandbox/lf_logger_config_example.json @@ -3,7 +3,7 @@ "disable_existing_loggers": false, "formatters": { "simple": { - "format": "%(created)-16f %(name)-16s %(levelname)-12s %(lineno)-6s %(funcName)-16s [%(module)s]: %(message)s" + "format": "%(created)f %(name)s %(levelname)-8s %(filename)s %(lineno)s %(funcName)s [%(module)s]: %(message)s" }, "extra": { "format":"%(asctime)-16s %(name)-8s %(filename)-12s %(lineno)-6s %(funcName)-16s %(levelname)-8s [%(module)s]: %(message)s", diff --git a/py-scripts/sandbox/lf_logger_config_test.py b/py-scripts/sandbox/lf_logger_config_test.py index a279e6f4..06877f6d 100644 --- a/py-scripts/sandbox/lf_logger_config_test.py +++ b/py-scripts/sandbox/lf_logger_config_test.py @@ -63,7 +63,7 @@ def main(): # Example to read the json config # set the configuration file - logger_config.lf_logger_config_json = "lf_logger_config_example.json" + logger_config.lf_logger_config_json = "../lf_logger_config.json" logger_config.load_lf_logger_config() logger.info("An INFO message asctime from " + __name__)