bugfixing in config & add quiet mode
authorAndrew Lorimer <andrew@charles.cortex>
Fri, 30 Aug 2019 12:58:30 +0000 (22:58 +1000)
committerAndrew Lorimer <andrew@charles.cortex>
Fri, 30 Aug 2019 12:58:30 +0000 (22:58 +1000)
logparse/config.py
logparse/formatting.py
logparse/interface.py
logparse/parsers/cron-journald.py
logparse/parsers/temperature.py
index c7455bd65323f639a11e516b2610a32a5eb22cdd..8b7d798ee936099905c2d99ede782b9da5450eab 100644 (file)
@@ -55,8 +55,9 @@ defaults = Configuration({
         'subject': 'logparse from $hostname$',
         'mailbin': '/usr/bin/mail',
     },
         'subject': 'logparse from $hostname$',
         'mailbin': '/usr/bin/mail',
     },
-    'rotate': 'n',
-    'verbose': 'n',
+    'rotate': False,
+    'verbose': False, 
+    'quiet': False,
     'hddtemp': {
         'drives': ['/dev/sda'],
         'host': '127.0.0.1',
     'hddtemp': {
         'drives': ['/dev/sda'],
         'host': '127.0.0.1',
@@ -79,7 +80,7 @@ defaults = Configuration({
     },
     'du': {
         'paths': ['/', '/etc', '/home'],
     },
     'du': {
         'paths': ['/', '/etc', '/home'],
-        'force-write': 'n',
+        'force-write': False,
     },
     'hostname-path': '/etc/hostname',
     'parsers': {},
     },
     'hostname-path': '/etc/hostname',
     'parsers': {},
index b303449acaa010b3a38b5d89a5557fa4b44cd457..ab6266efc5233840d3e1369b21c03ff3aa5adec9 100644 (file)
@@ -325,7 +325,7 @@ class Table(object):
 
     def draw_plain(self):
         output = tabulate(self._gen_list(), self.headers, tablefmt="fancy_grid" if self.borders else "plain", colalign=tuple(self._align_cols))
 
     def draw_plain(self):
         output = tabulate(self._gen_list(), self.headers, tablefmt="fancy_grid" if self.borders else "plain", colalign=tuple(self._align_cols))
-        return output
+        return output + "\n"*2
 
 
 class Table0(object):
 
 
 class Table0(object):
index 3c2b1a27351805ce6170f6b07082b4bc8ab5c980..c37ec9a946f64cf0ece2486d4a8a937b5f9633c6 100644 (file)
@@ -46,7 +46,8 @@ def main():
     argparser.add_argument('-l', '--logs', help='services to analyse', required=False)
     argparser.add_argument('-nl', '--ignore-logs', help='skip these services (takes precedence over -l)', required=False)
     argparser.add_argument('-es', '--embed-styles', help='make CSS rules inline rather than linking the file', required=False, default=False, action='store_true')
     argparser.add_argument('-l', '--logs', help='services to analyse', required=False)
     argparser.add_argument('-nl', '--ignore-logs', help='skip these services (takes precedence over -l)', required=False)
     argparser.add_argument('-es', '--embed-styles', help='make CSS rules inline rather than linking the file', required=False, default=False, action='store_true')
-    argparser.add_argument('-nh', '--plain', help='write/send plain text rather than HTML', required = False, default=False, action='store_true')
+    argparser.add_argument('-nh', '--plain', help='write/send plain text rather than HTML', required=False, default=False, action='store_true')
+    argparser.add_argument('-q', '--quiet', help='no output to stdout', required=False, default=False, action='store_true')
 
     # Load config
     if argparser.parse_args().config:
 
     # Load config
     if argparser.parse_args().config:
@@ -59,8 +60,10 @@ def main():
     logger = logging.getLogger(__name__)
     loghandler = logging.handlers.SysLogHandler(address = '/dev/log')
     loghandler.setFormatter(logging.Formatter(fmt='logparse.py[' + str(os.getpid()) + ']: %(message)s'))
     logger = logging.getLogger(__name__)
     loghandler = logging.handlers.SysLogHandler(address = '/dev/log')
     loghandler.setFormatter(logging.Formatter(fmt='logparse.py[' + str(os.getpid()) + ']: %(message)s'))
-    loghandler.setLevel(logging.WARNING)   # don't spam syslog with debug messages
-    if argparser.parse_args().verbose or (config.prefs['verbose'] == 'y' or config.prefs['verbose'] == 'yes'):
+    loghandler.setLevel(logging.INFO)   # don't spam syslog with debug messages
+    if argparser.parse_args().quiet or config.prefs['quiet']:
+        logging.basicConfig(level=logging.CRITICAL)
+    elif argparser.parse_args().verbose or config.prefs['verbose']:
         logging.basicConfig(level=logging.DEBUG)
         logger.debug("Verbose mode turned on")
     else:
         logging.basicConfig(level=logging.DEBUG)
         logger.debug("Verbose mode turned on")
     else:
@@ -139,6 +142,8 @@ def main():
             output.embed_css(prefs['css'])
         if (not os.path.isfile(dest_path)) and not (argparser.parse_args().overwrite or config['overwrite']):
             output.write(dest_path)
             output.embed_css(prefs['css'])
         if (not os.path.isfile(dest_path)) and not (argparser.parse_args().overwrite or config['overwrite']):
             output.write(dest_path)
+        elif logging.root.level == logging.CRITICAL:
+            pass
         else:
             logger.warning("Destination file already exists")
             if input("Would you like to overwrite {0}? (y/n) [n] ".format(dest_path)) == 'y':
         else:
             logger.warning("Destination file already exists")
             if input("Would you like to overwrite {0}? (y/n) [n] ".format(dest_path)) == 'y':
index 9df7e2bca244d8df28ffc430a5460c0ee79ddb4b..b085aeaba2c23a3d20c5f61f39f193cc6b99096a 100644 (file)
@@ -51,7 +51,7 @@ def parse_log():
 
     for usr, cmdlist in users.items():
         user_data = Data()
 
     for usr, cmdlist in users.items():
         user_data = Data()
-        user_data.subtitle = plural("cron sessions", len(cmdlist)) + " for " + usr
+        user_data.subtitle = plural("cron session", len(cmdlist)) + " for " + usr
         user_data.items = ("`{0}`".format(cmd) for cmd in cmdlist)
         user_data.orderbyfreq()
         user_data.truncl(config.prefs['maxcmd'])
         user_data.items = ("`{0}`".format(cmd) for cmd in cmdlist)
         user_data.orderbyfreq()
         user_data.truncl(config.prefs['maxcmd'])
index 9d08119b59755e23a2708ba32fa67f503c9eee68..d1d4058b0f88719544ee04c86f2ee80ea100a1a7 100644 (file)
@@ -110,7 +110,7 @@ def parse_log():
             section.append_data(temp_data)
 
     finally:
             section.append_data(temp_data)
 
     finally:
-        logger.info("Finished reading onboard temperatures")
+        logger.debug("Finished reading onboard temperatures")
         sensors.cleanup()
 
 
         sensors.cleanup()
 
 
@@ -141,7 +141,7 @@ def parse_log():
     logger.debug("Sum of temperatures: {}; Number of drives: {}; => Avg disk temp is {}".format(str(sumtemp), str(len(drives)), hddavg)) 
     hddtemp_data.subtitle += " (avg {0}{1}{2})".format(str(hddavg), DEG, CEL)
 
     logger.debug("Sum of temperatures: {}; Number of drives: {}; => Avg disk temp is {}".format(str(sumtemp), str(len(drives)), hddavg)) 
     hddtemp_data.subtitle += " (avg {0}{1}{2})".format(str(hddavg), DEG, CEL)
 
-    logger.info("Finished processing drive temperatures")
-
+    logger.debug("Finished processing drive temperatures")
     logger.info("Finished temp section")
     logger.info("Finished temp section")
+
     return section
     return section