bugfixing in config & add quiet mode

diff --git a/logparse/config.py b/logparse/config.py
index c7455bd65323f639a11e516b2610a32a5eb22cdd..8b7d798ee936099905c2d99ede782b9da5450eab 100644 (file)
--- a/logparse/config.py
+++ b/logparse/config.py
@@ -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': {},

diff --git a/logparse/formatting.py b/logparse/formatting.py
index b303449acaa010b3a38b5d89a5557fa4b44cd457..ab6266efc5233840d3e1369b21c03ff3aa5adec9 100644 (file)
--- a/logparse/formatting.py
+++ b/logparse/formatting.py
@@ -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):

diff --git a/logparse/interface.py b/logparse/interface.py
index 3c2b1a27351805ce6170f6b07082b4bc8ab5c980..c37ec9a946f64cf0ece2486d4a8a937b5f9633c6 100644 (file)
--- a/logparse/interface.py
+++ b/logparse/interface.py
@@ -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':

diff --git a/logparse/parsers/cron-journald.py b/logparse/parsers/cron-journald.py
index 9df7e2bca244d8df28ffc430a5460c0ee79ddb4b..b085aeaba2c23a3d20c5f61f39f193cc6b99096a 100644 (file)
--- a/logparse/parsers/cron-journald.py
+++ b/logparse/parsers/cron-journald.py
@@ -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'])

diff --git a/logparse/parsers/temperature.py b/logparse/parsers/temperature.py
index 9d08119b59755e23a2708ba32fa67f503c9eee68..d1d4058b0f88719544ee04c86f2ee80ea100a1a7 100644 (file)
--- a/logparse/parsers/temperature.py
+++ b/logparse/parsers/temperature.py
@@ -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