[checkmk-commits] Check_MK Git: check_mk: logwatch.groups: Now also works in precompailed mode

Module: check_mk Branch: master Commit: e81955100519450fff37433dc9b4607d944250b3 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=e8195510051945… Author: Bastian Kuhn <bk(a)mathias-kettner.de> Date: Thu Jul 17 16:43:39 2014 +0200 logwatch.groups: Now also works in precompailed mode --- checks/logwatch | 33 +++++++++++++++++++++------------ 1 file changed, 21 insertions(+), 12 deletions(-) diff --git a/checks/logwatch b/checks/logwatch index 4d8c12a..84b34af 100644 --- a/checks/logwatch +++ b/checks/logwatch @@ -256,10 +256,15 @@ precompile_params['logwatch'] = logwatch_precompile # | | # '----------------------------------------------------------------------' -def logwatch_groups_of_logfile(filename): +def logwatch_group_precompile( hostname, item, _unused): + return logwatch_precompile( hostname, item, None ), host_extra_conf( hostname, logwatch_groups ) + +def logwatch_groups_of_logfile(filename, params=False): import fnmatch groups = [] - for line in host_extra_conf(g_hostname, logwatch_groups): + if not params: + params = host_extra_conf(g_hostname, logwatch_groups) + for line in params: for group_name, pattern in line: inclusion, exclusion = pattern if fnmatch.fnmatch(filename, inclusion) \ @@ -273,11 +278,10 @@ def inventory_logwatch_groups(info): inventory = [] for logfile in not_forwarded_logs: groups = logwatch_groups_of_logfile(logfile) - if groups: - for group in groups: - if group not in added_groups: - added_groups.append(group) - inventory.append((group, None )) + for group in groups: + if group not in added_groups: + added_groups.append(group) + inventory.append((group, None )) return inventory def check_logwatch_groups(item, params, info): @@ -295,7 +299,7 @@ def check_logwatch_groups(item, params, info): loglines.append(line) if line.startswith('[[['): logfile = line[3:-3] - if item in logwatch_groups_of_logfile(logfile): + if item in logwatch_groups_of_logfile(logfile, params[1]): found = True logfile_found = True else: @@ -310,10 +314,10 @@ check_info['logwatch.groups'] = { 'service_description': "LOG %s", 'group': 'logwatch', } -precompile_params['logwatch.groups'] = logwatch_precompile +precompile_params['logwatch.groups'] = logwatch_group_precompile #. -def check_logwatch_generic(item, params, loglines, found): +def check_logwatch_generic(item, params, loglines, found, groups=False): # Create directories, if neccessary try: logdir = logwatch_dir + "/" + g_hostname @@ -375,9 +379,14 @@ def check_logwatch_generic(item, params, loglines, found): # Get the patterns (either compile or reuse the precompiled ones) # Check_MK creates an empty string if the precompile function has # not been exectued yet. The precomile function creates an empty - # list when no rules/patterns are defined. + # list when no rules/patterns are defined. In case of the logwatch.groups + # checks, params are a tuple with the normal logwatch parameters on the first + # and the grouping patterns on the second position if params not in ('', None): - patterns = params # patterns already precompiled + if groups: + patterns = params[0] + else: + patterns = params # patterns already precompiled else: patterns = logwatch_precompile(g_hostname, item, None)