Checkmk git commits

checkmk-commits@lists.checkmk.com

1 participants
64658 discussions

Check_MK Git: check_mk: availability: return empty list instead of None, if no items found

by Mathias Kettner

Module: check_mk Branch: master Commit: 72f6d3392d5b44804214617ed8501e7ad81c7624 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=72f6d3392d5b44… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Fri Oct 10 11:58:09 2014 +0200 availability: return empty list instead of None, if no items found --- web/plugins/views/availability.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/plugins/views/availability.py b/web/plugins/views/availability.py index 5c70cd3..af317a0 100644 --- a/web/plugins/views/availability.py +++ b/web/plugins/views/availability.py @@ -1182,7 +1182,7 @@ def render_availability_table(availability, from_time, until_time, range_title, if not availability: if not no_html: html.message(_("No matching hosts/services.")) - return # No objects + return [] # No objects grouping = avoptions["grouping"] fetch_data = []

10 years

Check_MK Git: check_mk: FIX for case when disk is removed from the device, thanks to Marcel Schulte

by Goetz Golla

Module: check_mk Branch: master Commit: e983bfcbb1b67d25f9fd2a06d59f59fd48c3a9ae URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=e983bfcbb1b67d… Author: Goetz Golla <gg(a)mathias-kettner.de> Date: Fri Oct 10 11:41:31 2014 +0200 FIX for case when disk is removed from the device, thanks to Marcel Schulte --- checks/diskstat.include | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/checks/diskstat.include b/checks/diskstat.include index a095b8a..6543833 100644 --- a/checks/diskstat.include +++ b/checks/diskstat.include @@ -234,7 +234,9 @@ def check_diskstat_generic(item, params, this_time, info, mode='sectors'): result = check_diskstat_line(this_time, item, params, line, mode) else: result = (2, "more than one device found in cluster") - return result + + if result != "": + return result return (3, "device missing")

10 years

Check_MK Git: check_mk: #1125 FIX EC actions couldn' t get saved when a EC rule had "Send monitoring notification" set

by Andreas Boesl

Module: check_mk Branch: master Commit: 6856f5fcb4584feb1363ac1f95ac6d9b4ce378a5 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=6856f5fcb4584f… Author: Andreas Boesl <ab(a)mathias-kettner.de> Date: Fri Oct 10 11:36:44 2014 +0200 #1125 FIX EC actions couldn't get saved when a EC rule had "Send monitoring notification" set This lead to the warning <tt>You are missing the action with the ID @NOTIFY which is still used in some rules</tt> --- .werks/1125 | 10 ++++++++++ ChangeLog | 1 + mkeventd/web/plugins/wato/mkeventd.py | 2 +- 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/.werks/1125 b/.werks/1125 new file mode 100644 index 0000000..05e6dc2 --- /dev/null +++ b/.werks/1125 @@ -0,0 +1,10 @@ +Title: EC actions couldn't get saved when a EC rule had "Send monitoring notification" set +Level: 1 +Component: ec +Compatible: compat +Version: 1.2.5i6 +Date: 1412933594 +Class: fix + +This lead to the warning +<tt>You are missing the action with the ID @NOTIFY which is still used in some rules</tt> diff --git a/ChangeLog b/ChangeLog index b38b8cc..3e77fe0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -157,6 +157,7 @@ * 1399 FIX: Fix left-over tac processes when showing Event Console history... * 1402 FIX: Fixed cased where counting events did not reach required count... * 1124 FIX: WATO EC configuration: no longer raises an exception when user has restricted WATO access... + * 1125 FIX: EC actions couldn't get saved when a EC rule had "Send monitoring notification" set... HW/SW-Inventory: * 0643 windows inventory: OS now contains the install date, reg_uninstall now contains the path... diff --git a/mkeventd/web/plugins/wato/mkeventd.py b/mkeventd/web/plugins/wato/mkeventd.py index 6ea1e41..81304c4 100644 --- a/mkeventd/web/plugins/wato/mkeventd.py +++ b/mkeventd/web/plugins/wato/mkeventd.py @@ -88,7 +88,7 @@ class ActionList(ListOf): rules = load_mkeventd_rules() for rule in rules: for action_id in rule.get("actions", []): - if action_id not in action_ids: + if action_id not in action_ids + ["@NOTIFY"]: raise MKUserError(varprefix, _("You are missing the action with the ID %s, " "which is still used in some rules.") % action_id)

10 years

Check_MK Git: check_mk: mkeventd: Can use the a sites local mongodb instance for history / archiving now

by Lars Michelsen

Module: check_mk Branch: master Commit: 98209c3ce9c8cb119d8eb648d73d392abd916f76 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=98209c3ce9c8cb… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Fri Oct 10 11:24:14 2014 +0200 mkeventd: Can use the a sites local mongodb instance for history / archiving now --- mkeventd/bin/mkeventd | 218 +++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 212 insertions(+), 6 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=98209c3ce9…

10 years

Check_MK Git: check_mk: forms: fixed exception when a validation error in a dict happens

by Mathias Kettner

Module: check_mk Branch: master Commit: 33c9deeb1be65fc4162e4381714fb46ed5d3156c URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=33c9deeb1be65f… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Fri Oct 10 09:22:07 2014 +0200 forms: fixed exception when a validation error in a dict happens --- web/htdocs/forms.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/web/htdocs/forms.py b/web/htdocs/forms.py index 972d57a..0588f19 100644 --- a/web/htdocs/forms.py +++ b/web/htdocs/forms.py @@ -89,8 +89,8 @@ def edit_dictionaries(dictionaries, value, focus=None, hover_help=True, try: new_value[keyname] = entries.from_html_vars(keyname) entries.validate_value(new_value[keyname], keyname) - except: - messages.append("%s: %s" % (entries.title(), e.message)) + except Exception, e: + messages.append("%s: %s" % (entries.title() or _("Properties"), e.message)) html.add_user_error(e.varname, e.message) if validate and not html.has_user_errors():

10 years

Check_MK Git: check_mk: Availability summary: add css for summary line

by Mathias Kettner

Module: check_mk Branch: master Commit: c128a8dce2b45a8c1cae3d408bb51bc86ac74f23 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=c128a8dce2b45a… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu Oct 9 18:36:11 2014 +0200 Availability summary: add css for summary line --- web/plugins/views/availability.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/web/plugins/views/availability.py b/web/plugins/views/availability.py index 55601df..5c70cd3 100644 --- a/web/plugins/views/availability.py +++ b/web/plugins/views/availability.py @@ -1427,7 +1427,7 @@ def render_availability_group(group_title, range_title, group_id, availability, if not "omit_buttons" in labelling and not no_html: table.cell("") if not "omit_host" in labelling: - table.cell("", _("Summary")) + table.cell("", _("Summary"), css="heading") if what == "service": table.cell("", "") @@ -1447,7 +1447,7 @@ def render_availability_group(group_title, range_title, group_id, availability, if number and av_levels and sid in [ "ok", "up" ]: css = "state%d" % check_av_levels(number, av_levels, considered_duration) - table.cell(sname, render_number(number, considered_duration), css="number " + css, help=help) + table.cell(sname, render_number(number, considered_duration), css="heading number " + css, help=help) os_aggrs, os_states = avoptions.get("outage_statistics", ([],[])) if sid in os_states: for aggr in os_aggrs:

10 years

Check_MK Git: check_mk: Small adaptions for computing BI availability

by Mathias Kettner

Module: check_mk Branch: master Commit: 77f96600af2047ce2f021146b94c4eaccf8321ec URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=77f96600af2047… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu Oct 9 17:47:28 2014 +0200 Small adaptions for computing BI availability --- web/plugins/views/availability.py | 24 ++++++++++++++++++------ 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/web/plugins/views/availability.py b/web/plugins/views/availability.py index bb17a82..55601df 100644 --- a/web/plugins/views/availability.py +++ b/web/plugins/views/availability.py @@ -852,7 +852,7 @@ def do_render_availability(rows, what, avoptions, timeline, timewarpcode, fetch= render_timeline(timeline_rows, from_time, until_time, total_duration, timeline, range_title, render_number, what, timewarpcode, avoptions, style="standalone") else: - fetch_data["table"] = render_availability_table(availability, from_time, until_time, range_title, + fetch_data["table"] = render_availability_table(availability, from_time, until_time, range_title, what, avoptions, render_number, fetch) if not fetch: @@ -1189,7 +1189,7 @@ def render_availability_table(availability, from_time, until_time, range_title, if not grouping: fetch_data.append((None, - render_availability_group(range_title, range_title, None, availability, from_time, + render_availability_group(range_title, range_title, None, availability, from_time, until_time, what, avoptions, render_number, fetch))) else: @@ -1216,7 +1216,7 @@ def render_availability_table(availability, from_time, until_time, range_title, # 3. Loop over all groups and render them for title, group_id in titled_groups: fetch_data.append((title, - render_availability_group(title, range_title, group_id, availability, + render_availability_group(title, range_title, group_id, availability, from_time, until_time, what, avoptions, render_number, fetch) )) @@ -1265,7 +1265,8 @@ def get_av_groups(availability, grouping): # When grouping is enabled, this function is called once for each group -def render_availability_group(group_title, range_title, group_id, availability, from_time, until_time, what, avoptions, render_number, fetch): +def render_availability_group(group_title, range_title, group_id, availability, + from_time, until_time, what, avoptions, render_number, fetch): # Filter out groups that we want to show this time group_availability = [] @@ -1339,8 +1340,12 @@ def render_availability_group(group_title, range_title, group_id, availability, host_url = "view.py?" + html.urlencode_vars([("view_name", "hoststatus"), ("site", site), ("host", host)]) if what == "bi": - bi_url = "view.py?" + html.urlencode_vars([("view_name", "aggr_single"), ("aggr_group", host), ("aggr_name", service)]) - table.cell(_("Aggregate"), '<a href="%s">%s</a>' % (bi_url, service)) + table.cell(_("Aggregate")) + if no_html: + html.write(service) + else: + bi_url = "view.py?" + html.urlencode_vars([("view_name", "aggr_single"), ("aggr_group", host), ("aggr_name", service)]) + html.write('<a href="%s">%s</a>' % (bi_url, service)) availability_columns = bi_availability_columns else: if not "omit_host" in labelling: @@ -1474,6 +1479,13 @@ def check_av_levels(number, av_levels, considered_duration): return 0 +def compute_bi_availability(avoptions, aggr_rows): + rows = [] + for aggr_row in aggr_rows: + these_rows, tree_state = get_bi_timeline(aggr_row["aggr_tree"], aggr_row["aggr_group"], avoptions, False) + rows += these_rows + return do_render_availability(rows, "bi", avoptions, timeline=False, timewarpcode=None, fetch=True) + # Render availability of a BI aggregate. This is currently # no view and does not support display options

10 years

Check_MK Git: check_mk: #1124 FIX WATO EC configuration: no longer raises an exception when user has restricted WATO access

by Andreas Boesl

Module: check_mk Branch: master Commit: 9dcfe7b5bd842fd821b375664e03ac6f2faf2b9f URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=9dcfe7b5bd842f… Author: Andreas Boesl <ab(a)mathias-kettner.de> Date: Thu Oct 9 16:38:57 2014 +0200 #1124 FIX WATO EC configuration: no longer raises an exception when user has restricted WATO access An exception happened, when a user had restricted WATO permissions, but was allowed to edit the WATO EC configuration. The following role settings caused this error: <tt>WATO - Check_MK's Web Administration Tool -> Read access to all modules.... -> No</tt> and <tt>Event Console -> Configuration of event rules -> Yes</tt> --- .werks/1124 | 14 ++++++++++++++ ChangeLog | 1 + web/htdocs/wato.py | 4 +++- 3 files changed, 18 insertions(+), 1 deletion(-) diff --git a/.werks/1124 b/.werks/1124 new file mode 100644 index 0000000..f27ae7c --- /dev/null +++ b/.werks/1124 @@ -0,0 +1,14 @@ +Title: WATO EC configuration: no longer raises an exception when user has restricted WATO access +Level: 1 +Component: ec +Compatible: compat +Version: 1.2.5i6 +Date: 1412864930 +Class: fix + +An exception happened, when a user had restricted WATO permissions, but was allowed to edit the WATO EC configuration. +The following role settings caused this error: + +<tt>WATO - Check_MK's Web Administration Tool -> Read access to all modules.... -> No</tt> +and +<tt>Event Console -> Configuration of event rules -> Yes</tt> diff --git a/ChangeLog b/ChangeLog index 5bc5c41..b38b8cc 100644 --- a/ChangeLog +++ b/ChangeLog @@ -156,6 +156,7 @@ * 1007 FIX: check_mkevents: fix case where events contain binary zeroes * 1399 FIX: Fix left-over tac processes when showing Event Console history... * 1402 FIX: Fixed cased where counting events did not reach required count... + * 1124 FIX: WATO EC configuration: no longer raises an exception when user has restricted WATO access... HW/SW-Inventory: * 0643 windows inventory: OS now contains the install date, reg_uninstall now contains the path... diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py index a4cf757..b0fea5a 100644 --- a/web/htdocs/wato.py +++ b/web/htdocs/wato.py @@ -214,7 +214,9 @@ def page_handler(): # Check general permission for this mode if modeperms != None and not config.may("wato.seeall"): for pname in modeperms: - config.need_permission("wato." + pname) + if '.' not in pname: + pname = "wato." + pname + config.need_permission(pname) # Do actions (might switch mode) action_message = None

10 years

Check_MK Git: check_mk: Fix styling of close symbol in filter lists

by Mathias Kettner

Module: check_mk Branch: master Commit: 92b1f506ab7eed1bc79a5a35d5d759c775ad363a URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=92b1f506ab7eed… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu Oct 9 16:16:37 2014 +0200 Fix styling of close symbol in filter lists --- web/htdocs/views.css | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/web/htdocs/views.css b/web/htdocs/views.css index ea89445..f0f1074 100644 --- a/web/htdocs/views.css +++ b/web/htdocs/views.css @@ -139,10 +139,11 @@ table.valuespec_listof td.vlof_content.filter { table.valuespec_listof td.vlof_buttons.filter { width: 1px; border-style: none; + vertical-align: top; } table.valuespec_listof td.vlof_buttons.filter img { position: relative; - top: -18px; + top: 2px; left: -28px; width: 16px; height: 16px;

10 years

Check_MK Git: check_mk: Fix default value of avoptions valuespec

by Mathias Kettner

Module: check_mk Branch: master Commit: 95948442d5f010d5fd143212231e38ad72af1e51 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=95948442d5f010… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu Oct 9 16:08:05 2014 +0200 Fix default value of avoptions valuespec --- web/htdocs/views.py | 4 +- web/plugins/views/availability.py | 89 ++++++++++++++++++++++--------------- 2 files changed, 56 insertions(+), 37 deletions(-) diff --git a/web/htdocs/views.py b/web/htdocs/views.py index e5df3ce..9ccf5b7 100644 --- a/web/htdocs/views.py +++ b/web/htdocs/views.py @@ -2191,7 +2191,9 @@ def group_value(row, group_painters): def get_painter_option(name): opt = multisite_painter_options[name] - if not config.may("general.painter_options"): + if "forced_value" in opt: + return opt["forced_value"] + elif not config.may("general.painter_options"): return opt['valuespec'].default_value() else: return opt.get("value", opt['valuespec'].default_value()) diff --git a/web/plugins/views/availability.py b/web/plugins/views/availability.py index 698d726..bb17a82 100644 --- a/web/plugins/views/availability.py +++ b/web/plugins/views/availability.py @@ -195,7 +195,7 @@ avoption_entries = [ False, Timerange( title = _("Time Range"), - default_value = 'm1', + default_value = 'd0', ) ), @@ -229,7 +229,8 @@ avoption_entries = [ ( "honor", _("Honor scheduled downtimes") ), ( "ignore", _("Ignore scheduled downtimes") ), ( "exclude", _("Exclude scheduled downtimes" ) ), - ] + ], + default_value = "honor", ) ), ( "exclude_ok", @@ -249,13 +250,19 @@ avoption_entries = [ columns = 2, elements = [ ( "flapping", - Checkbox(label = _("Consider periods of flapping states")), + Checkbox( + label = _("Consider periods of flapping states"), + default_value = True), ), ( "host_down", - Checkbox(label = _("Consider times where the host is down")), + Checkbox( + label = _("Consider times where the host is down"), + default_value = True), ), ( "unmonitored", - Checkbox(label = _("Include unmonitored time")), + Checkbox( + label = _("Include unmonitored time"), + default_value = True), ), ], optional_keys = False, @@ -278,7 +285,8 @@ avoption_entries = [ ( "warn", _("WARN") ), ( "crit", _("CRIT") ), ( "unknown", _("UNKNOWN") ), - ] + ], + default_value = "warn", ), ), ( "unknown", @@ -289,7 +297,8 @@ avoption_entries = [ ( "warn", _("WARN") ), ( "crit", _("CRIT") ), ( "unknown", _("UNKNOWN") ), - ] + ], + default_value = "unknown", ), ), ( "host_down", @@ -301,7 +310,8 @@ avoption_entries = [ ( "crit", _("CRIT") ), ( "unknown", _("UNKNOWN") ), ( "host_down", _("Host Down") ), - ] + ], + default_value = "host_down", ), ), ], @@ -380,7 +390,8 @@ avoption_entries = [ ( "honor", _("Base report only on service times") ), ( "ignore", _("Include both service and non-service times" ) ), ( "exclude", _("Base report only on non-service times" ) ), - ] + ], + default_value = "honor", ) ), @@ -394,7 +405,8 @@ avoption_entries = [ ( "honor", _("Distinguish times in and out of notification period") ), ( "exclude", _("Exclude times out of notification period" ) ), ( "ignore", _("Ignore notification period") ), - ] + ], + default_value = "ignore", ) ), @@ -409,7 +421,8 @@ avoption_entries = [ ( "host", _("By Host") ), ( "host_groups", _("By Host group") ), ( "service_groups", _("By Service group") ), - ] + ], + default_value = None, ) ), @@ -423,6 +436,7 @@ avoption_entries = [ ("yyyy-mm-dd hh:mm:ss", _("YYYY-MM-DD HH:MM:SS") ), ("epoch", _("Unix Timestamp (Epoch)") ), ], + default_value = "yyyy-mm-dd hh:mm:ss", ) ), ( "timeformat", @@ -440,19 +454,20 @@ avoption_entries = [ ("hours", _("Hours") ), ("hhmmss", _("HH:MM:SS") ), ], + default_value = "percentage_2", ) ), - # Short time intervals ( "short_intervals", "single", True, Integer( title = _("Short Time Intervals"), + label = _("Ignore intervals shorter or equal"), minvalue = 0, unit = _("sec"), - label = _("Ignore intervals shorter or equal"), + default_value = 0, ), ), @@ -476,7 +491,7 @@ avoption_entries = [ ( "sum", _("Display total sum (for % the average)") ), ( "average", _("Display average") ), ], - default_value = sum, + default_value = "sum", ) ), @@ -495,10 +510,10 @@ avoption_entries = [ False, Age( title = _("Query Time Limit"), - default_value = 30, - unit = _("sec"), help = _("Limit the execution time of the query, in order to " "avoid a hanging system."), + unit = _("sec"), + default_value = 30, ), ) ] @@ -511,48 +526,50 @@ def get_availability_options_from_url(): html.unplug() return avoptions - -def render_availability_options(): - if html.var("_reset") and html.check_transaction(): - config.save_user_file("avoptions", {}) - for varname in html.vars.keys(): - if varname.startswith("avo_"): - html.del_var(varname) - html.del_var("avoptions") - - avoptions = { +def get_default_avoptions(): + return { "range" : (time.time() - 86400, time.time()), + "rangespec" : "d0", + "labelling" : [], "downtimes" : { "include" : "honor", "exclude_ok" : False, }, - "notification_period" : "ignore", - "service_period" : "honor", "consider" : { "flapping" : True, "host_down" : True, "unmonitored" : True, }, - "dateformat" : "yyyy-mm-dd hh:mm:ss", - "timeformat" : "percentage_2", - "labelling" : [], - "rangespec" : "d0", "state_grouping" : { "warn" : "warn", "unknown" : "unknown", "host_down" : "host_down", }, + "av_levels" : None, "outage_statistics" : ([],[]), + "av_mode" : False, + "service_period" : "honor", + "notification_period" : "ignore", + "grouping" : None, + "dateformat" : "yyyy-mm-dd hh:mm:ss", + "timeformat" : "percentage_2", "short_intervals" : 0, "dont_merge" : False, - "show_timeline" : False, "summary" : "sum", - "av_levels" : None, - "av_mode" : False, - "grouping" : None, + "show_timeline" : False, "timelimit" : 30, } +def render_availability_options(): + if html.var("_reset") and html.check_transaction(): + config.save_user_file("avoptions", {}) + for varname in html.vars.keys(): + if varname.startswith("avo_"): + html.del_var(varname) + html.del_var("avoptions") + + avoptions = get_default_avoptions() + # Users of older versions might not have all keys set. The following # trick will merge their options with our default options. avoptions.update(config.load_user_file("avoptions", {}))

10 years

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits