Checkmk git commits

checkmk-commits@lists.checkmk.com

1 participants
64139 discussions

Check_MK Git: check_mk: install_nagios.sh: new option -m

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 624876025abc06e26582a796f1b0d74d80a91092 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=624876025abc06… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu Dec 23 18:00:40 2010 +0100 install_nagios.sh: new option -m --- .bugs/85 | 45 +++++++++++++++++++++++++++++++++++++++++++++ scripts/install_nagios.sh | 16 +++++++++++----- 2 files changed, 56 insertions(+), 5 deletions(-) diff --git a/.bugs/85 b/.bugs/85 new file mode 100644 index 0000000..fe689d4 --- /dev/null +++ b/.bugs/85 @@ -0,0 +1,45 @@ +Title: Modulare Agenten +Component: core +Benefit: 6 +State: open +Cost: 5 +Date: 2010-12-23 08:40:16 +Class: feature + +Die Checks sollen den Code für die Agenten selbst mitbringen. +Die Agenten werden dann auf Wunsch dynamisch zusammengebaut. + +Eine Idee ist z.B., dass jeder Check sich selbst einklassifiziert +in essential/extended/full. In essential sind nur die +absolut wichtigen Checks enthalten. So kann man dann einen +Agenten einer bestimmten Ausbaustufe bauen. + +Umsetzung: + +Erstmal die bestehenden Agenten auseinanderpflücken und in +die Checks einbauen. Das gleiche dann auch für den Windows-Agenten +machen. Gemeinsamer kann auch hier evtl. nach win.include geschoben +werden, aber nicht der generelle Rumpfteil. + +Dann muss man in Check_MK einen Befehl einbauen, um die Agenten +zu erzeugen: cmk -A. Hier gibt es viele denkbare Aufrufvarianten: + +* Angabe des Betriebssystems (windows, linux, aix, hpux, freebsd, solaris). +* Angabe der Wichtigkeit (essential, extended, full). +* beides kombiniert + +Die fertigen Agenten gehen dann nach var/check_mk/agents. Im Falle von +Unix sind es die Shellskripten. Im Falle von Windows ist es der Quellcode, +der natürlich erst noch übersetzt werden muss. + +Im Buildsystem muss ich jetzt darauf achten, dass bei einem 'make' +auch immer alle Agenten gebaut werden. + +Der nächste Schritt wäre dann die Möglichkeit, per main.mk spezifische +Agenten zu bauen, in dem man irgendwie angibt, welche Checks (genauer: +Sektionen) rein sollen. Beispiel: + +# Agent 'hirn' ist wie essential, aber ohne fs, aber mit ntp +agents["hirn"] = [ '@essential', '!fs', 'ntp' ] +agents["hirn2"] = [ '@hirn', '!df', 'depp' ] + diff --git a/scripts/install_nagios.sh b/scripts/install_nagios.sh index bdebe80..a55dcd3 100644 --- a/scripts/install_nagios.sh +++ b/scripts/install_nagios.sh @@ -54,12 +54,12 @@ exec > >(tee $LOGFILE) 2>&1 set -e -NAGIOS_VERSION=3.2.2 +NAGIOS_VERSION=3.2.3 PLUGINS_VERSION=1.4.15 -RRDTOOL_VERSION=1.4.3 +RRDTOOL_VERSION=1.4.4 CHECK_MK_VERSION=1.1.9i3 PNP_VERSION=0.6.6 -NAGVIS_VERSION=1.5.1 +NAGVIS_VERSION=1.5.6 SOURCEFORGE_MIRROR=dfn NAGIOS_URL="http://downloads.sourceforge.net/project/nagios/nagios-3.x/nagios-$NAGIOS_V…" @@ -148,11 +148,16 @@ esac # Process command line options if [ $# -gt 0 ]; then - while getopts "s:y" options $OPTS; do + while getopts "s:ym" options $OPTS; do case $options in y) YES=1 ;; + m) + WITHOUT_MK=1 + NOTICE=" +Setting up without Check_MK!" + ;; s) SITE=$OPTARG ;; @@ -190,6 +195,7 @@ Check_MK on a freshly installed Linux system. It will: The output of this script is logged into $LOGFILE. No user interaction is neccesary. +$NOTICE EOF if [ -z "$YES" ]; then @@ -985,7 +991,7 @@ killall nagios || true /etc/init.d/nagios start activate_initd nagios || true -if [ "$CHECK_MK_VERSION" ] +if [ "$CHECK_MK_VERSION" -a -z "$WITHOUT_MK" ] then # ----------------------------------------------------------------------------- heading "Check_MK"

13 years, 9 months

Check_MK Git: check_mk: Livestatus: write timeperiod transitions to logfile

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 5245b99b018b592b30d8a4ff29b17fbd7c076596 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=5245b99b018b59… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Dec 22 16:27:08 2010 +0100 Livestatus: write timeperiod transitions to logfile --- .bugs/7 | 25 ++++++++++++++++++++++--- ChangeLog | 3 +++ livestatus/src/TimeperiodsCache.cc | 21 +++++++++++++++++++-- livestatus/src/TimeperiodsCache.h | 2 ++ livestatus/src/module.c | 26 ++++++++++++++------------ 5 files changed, 60 insertions(+), 17 deletions(-) diff --git a/.bugs/7 b/.bugs/7 index 030a6bf..8112cfd 100644 --- a/.bugs/7 +++ b/.bugs/7 @@ -1,10 +1,11 @@ Title: Timeperiod-Übergänge ins Nagios-Log schreiben Component: livestatus +State: done +Class: feature +Date: 2010-12-21 10:56:28 Benefit: 3 -State: open Cost: 1 -Date: 2010-12-21 10:56:28 -Class: feature +Fun: 0 Livestatus hält ja ohnehin einen Cache der Timeperiods. Immer wenn eine umschaltet, sollte ein Logeintrag gemacht werden. @@ -12,3 +13,21 @@ Auch initial beim Starten des Systems. Anhand dieser Einträge ist es später viel einfacher, die Verfügbarkeiten zu berechnen. Das sollte man recht bald programmieren, damit die Leute schon Gelegenheit haben, Daten zu sammeln. + +2010-12-22 16:13:20: changed state open -> done +Ist erledigt. Jetzt wird immer ein Eintrag +gemacht. Beim Starten von Nagios: + +[1293030598] TIMEPERIOD TRANSITION: 24X7;-1;1 +[1293030598] TIMEPERIOD TRANSITION: 24x7;-1;1 +[1293030598] TIMEPERIOD TRANSITION: hirni10;-1;1 +[1293030598] TIMEPERIOD TRANSITION: hirni11;-1;1 +[1293030598] TIMEPERIOD TRANSITION: hirni9;-1;0 +[1293030598] TIMEPERIOD TRANSITION: none;-1;0 +[1293030598] TIMEPERIOD TRANSITION: workhours;-1;1 + +Später dann: + +[1293030660] TIMEPERIOD TRANSITION: hirni10;1;0 + +Doku von Livestatus is auch bereits aktualisiert. diff --git a/ChangeLog b/ChangeLog index 5715a30..838aecf 100644 --- a/ChangeLog +++ b/ChangeLog @@ -44,6 +44,9 @@ * Allow more than one column for StatsGroupBy: * Do not use function is_contact_member_of_contactgroup anymore (get compatible with Nagios CVS) + * Livestatus: log timeperiod transitions (active <-> inactive) into Nagios + log file. This will enable us to create availability reports more simple + in future. 1.1.9i2: Checks & Agents: diff --git a/livestatus/src/TimeperiodsCache.cc b/livestatus/src/TimeperiodsCache.cc index 089c277..ed94aba 100644 --- a/livestatus/src/TimeperiodsCache.cc +++ b/livestatus/src/TimeperiodsCache.cc @@ -57,14 +57,24 @@ void TimeperiodsCache::update(time_t now) } _cache_time = minutes; - _cache.clear(); // Loop over all timeperiods and compute if we are // currently in timeperiod *tp = timeperiod_list; while (tp) { bool is_in = 0 == check_time_against_period(now, tp); - _cache.insert(std::make_pair(tp, is_in)); + + // check previous state and log transition if state has changed + _cache_t::iterator it = _cache.find(tp); + if (it == _cache.end()) { // first entry + logTransition(tp->name, -1, is_in ? 1 : 0); + _cache.insert(std::make_pair(tp, is_in)); + } + else if (it->second != is_in) { + logTransition(tp->name, it->second ? 1 : 0, is_in ? 1 : 0); + it->second = is_in; + } + tp = tp->next; } pthread_mutex_unlock(&_cache_lock); @@ -87,3 +97,10 @@ bool TimeperiodsCache::inTimeperiod(timeperiod *tp) return is_in; } + +void TimeperiodsCache::logTransition(char *name, int from, int to) +{ + char buffer[64]; + snprintf(buffer, sizeof(buffer), "TIMEPERIOD TRANSITION: %s;%d;%d", name, from, to); + write_to_all_logs(buffer, LOG_INFO); +} diff --git a/livestatus/src/TimeperiodsCache.h b/livestatus/src/TimeperiodsCache.h index e3d070e..5a3d2a4 100644 --- a/livestatus/src/TimeperiodsCache.h +++ b/livestatus/src/TimeperiodsCache.h @@ -40,6 +40,8 @@ public: ~TimeperiodsCache(); void update(time_t now); bool inTimeperiod(timeperiod *tp); +private: + void logTransition(char *name, int from, int to); }; #endif // _TimeperiodsCache_h diff --git a/livestatus/src/module.c b/livestatus/src/module.c index 87449bc..46c2e81 100644 --- a/livestatus/src/module.c +++ b/livestatus/src/module.c @@ -386,54 +386,56 @@ int broker_event(int event_type, void *data) int broker_process(int event_type, void *data) { struct nebstruct_process_struct *ps = (struct nebstruct_process_struct *)data; - if (ps->type == NEBTYPE_PROCESS_EVENTLOOPSTART) + if (ps->type == NEBTYPE_PROCESS_EVENTLOOPSTART) { + update_timeperiods_cache(time(0)); start_threads(); + } } int verify_event_broker_options() { int errors = 0; - if(!(event_broker_options & BROKER_PROGRAM_STATE)) { + if (!(event_broker_options & BROKER_PROGRAM_STATE)) { logger( LG_CRIT, "need BROKER_PROGRAM_STATE (%i) event_broker_option enabled to work.", BROKER_PROGRAM_STATE ); errors++; } - if(!(event_broker_options & BROKER_TIMED_EVENTS)) { + if (!(event_broker_options & BROKER_TIMED_EVENTS)) { logger( LG_CRIT, "need BROKER_TIMED_EVENTS (%i) event_broker_option enabled to work.", BROKER_TIMED_EVENTS ); errors++; } - if(!(event_broker_options & BROKER_SERVICE_CHECKS)) { + if (!(event_broker_options & BROKER_SERVICE_CHECKS)) { logger( LG_CRIT, "need BROKER_SERVICE_CHECKS (%i) event_broker_option enabled to work.", BROKER_SERVICE_CHECKS ); errors++; } - if(!(event_broker_options & BROKER_HOST_CHECKS)) { + if (!(event_broker_options & BROKER_HOST_CHECKS)) { logger( LG_CRIT, "need BROKER_HOST_CHECKS (%i) event_broker_option enabled to work.", BROKER_HOST_CHECKS ); errors++; } - if(!(event_broker_options & BROKER_LOGGED_DATA)) { + if (!(event_broker_options & BROKER_LOGGED_DATA)) { logger( LG_CRIT, "need BROKER_LOGGED_DATA (%i) event_broker_option enabled to work.", BROKER_LOGGED_DATA ); errors++; } - if(!(event_broker_options & BROKER_COMMENT_DATA)) { + if (!(event_broker_options & BROKER_COMMENT_DATA)) { logger( LG_CRIT, "need BROKER_COMMENT_DATA (%i) event_broker_option enabled to work.", BROKER_COMMENT_DATA ); errors++; } - if(!(event_broker_options & BROKER_DOWNTIME_DATA)) { + if (!(event_broker_options & BROKER_DOWNTIME_DATA)) { logger( LG_CRIT, "need BROKER_DOWNTIME_DATA (%i) event_broker_option enabled to work.", BROKER_DOWNTIME_DATA ); errors++; } - if(!(event_broker_options & BROKER_STATUS_DATA)) { + if (!(event_broker_options & BROKER_STATUS_DATA)) { logger( LG_CRIT, "need BROKER_STATUS_DATA (%i) event_broker_option enabled to work.", BROKER_STATUS_DATA ); errors++; } - if(!(event_broker_options & BROKER_ADAPTIVE_DATA)) { + if (!(event_broker_options & BROKER_ADAPTIVE_DATA)) { logger( LG_CRIT, "need BROKER_ADAPTIVE_DATA (%i) event_broker_option enabled to work.", BROKER_ADAPTIVE_DATA ); errors++; } - if(!(event_broker_options & BROKER_EXTERNALCOMMAND_DATA)) { + if (!(event_broker_options & BROKER_EXTERNALCOMMAND_DATA)) { logger( LG_CRIT, "need BROKER_EXTERNALCOMMAND_DATA (%i) event_broker_option enabled to work.", BROKER_EXTERNALCOMMAND_DATA ); errors++; } - if(!(event_broker_options & BROKER_STATECHANGE_DATA)) { + if (!(event_broker_options & BROKER_STATECHANGE_DATA)) { logger( LG_CRIT, "need BROKER_STATECHANGE_DATA (%i) event_broker_option enabled to work.", BROKER_STATECHANGE_DATA ); errors++; }

13 years, 9 months

Check_MK Git: check_mk: gb: output and sort after benefit and cost

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 94fc2fea96bdb7abb7e90bba32ddce4002164447 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=94fc2fea96bdb7… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Dec 22 15:37:47 2010 +0100 gb: output and sort after benefit and cost --- .bugs/78 | 2 +- gb | 106 +++++++++++++++++++++++++++++++++++++++++++++++++++++++------- 2 files changed, 95 insertions(+), 13 deletions(-) diff --git a/.bugs/78 b/.bugs/78 index 92ab8cc..84f410c 100644 --- a/.bugs/78 +++ b/.bugs/78 @@ -1,4 +1,4 @@ -Title: local-Struktur: Doppeltes importieren von config +Title: local-Struktur: doppeltes Importieren von config Component: multisite Benefit: 1 State: open diff --git a/gb b/gb index 47902c3..a4afeb7 100755 --- a/gb +++ b/gb @@ -15,10 +15,13 @@ if sys.stdout.isatty(): tty_bgyellow = '\033[43m' tty_bgblue = '\033[44m' tty_bgmagenta = '\033[45m' + tty_bgcyan = '\033[46m' tty_bgwhite = '\033[47m' tty_bold = '\033[1m' tty_underline = '\033[4m' tty_normal = '\033[0m' + def tty_colors(codes): + return '\033[%sm' % (';'.join([str(c) for c in codes])) else: tty_red = '' tty_green = '' @@ -29,10 +32,13 @@ else: tty_white = '' tty_bgblue = '' tty_bgmagenta = '' + tty_bgcyan = '' tty_bold = '' tty_underline = '' tty_normal = '' tty_ok = 'OK' + def tty_colors(c): + return "" grep_color = tty_bold + tty_green @@ -58,6 +64,7 @@ def load_config(): def load_bugs(): global g_bugs g_bugs = {} + check_modified() for entry in os.listdir("."): try: bugid = int(entry) @@ -68,6 +75,20 @@ def load_bugs(): except: continue +def check_modified(): + global g_modified + g_modified = set([]) + for line in os.popen("git status --porcelain"): + if line.startswith("M") and ".bugs/" in line: + try: + id = line.rsplit("/", 1)[-1].strip() + g_modified.add(int(id)) + except: + pass + +def bug_is_modified(bugid): + return bugid in g_modified + def load_bug(bugid): bug = { @@ -127,13 +148,13 @@ def usage(): where COMMAND is one of: - list [STATE] - list bugs - new - create a new bug - show [# #..] - show several bugs (or all open) - resolve ID - change a bugs state - delete #.. - delete bug(s) - grep WHAT... - grep in titles and descriptions - edit [#] - open bug # in editor (or newest bug) + list [-g] [-r] [STATE] - list bugs (-g: sort after gain, -r: reverse) + new - create a new bug + show [# #..] - show several bugs (or all open) + resolve ID - change a bugs state + delete #.. - delete bug(s) + grep WHAT... - grep in titles and descriptions + edit [#] - open bug # in editor (or newest bug) """) sys.exit(1) @@ -142,10 +163,43 @@ def bug_state(bug): state = bug["state"] return "%s%-7s%s" % (tty_bold + states.get(state, ""), state, tty_normal) +def num_color(n, colors, inverse): + if inverse: + b = 40 + else: + b = 30 + + c = colors[n-1] + return tty_colors([b + c, 1]) + +def bug_gain(bug): + cost = int(bug["cost"]) + benefit = int(bug["benefit"]) + gain = benefit - cost + if gain >= 2: + color = 2 + elif gain > 0: + color = 6 + elif gain <= -3: + color = 1 + elif gain <= -2: + color = 5 + elif gain < 0: + color = 3 + else: + color = 4 + return "%s%d%s%d%s" % ( + tty_colors([40 + color, 1]), cost, tty_colors([0, 30 + color, 1]), benefit, tty_normal) + def list_bug(bug): - sys.stdout.write("%s#%04d%s %-6s %-9s %-10s %-60s\n" % - (tty_bold, int(bug["id"]), tty_normal, bug_state(bug), - bug["class"], bug["component"], bug["title"])) + if bug_is_modified(bug["id"]): + bold = tty_bold + tty_cyan + "(*) " + else: + bold = "" + sys.stdout.write("#%04d %s %-6s %-9s %-10s %s%-60s%s\n" % + (int(bug["id"]), + bug_gain(bug), bug_state(bug), + bug["class"], bug["component"], bold, bug["title"], tty_normal)) def show_bug(bug): list_bug(bug) @@ -153,10 +207,26 @@ def show_bug(bug): def main_list(args): bugs = g_bugs.values() + # arguments are tags from state, component and class. Multiple values # in one class are orred. Multiple types are anded. filters = {} + def cmp_gain(a, b): + ga = int(a["benefit"]) - int(a["cost"]) + gb = int(b["benefit"]) - int(b["cost"]) + return ga - gb + + sort = None + reverse = False for a in args: + if a == '-g': + sort = cmp_gain + continue + + if a == '-r': + reverse = True + continue + hit = False for tp, values in [ ( "component", components ), ( "state", states ), ( "class", classes ), ]: @@ -172,7 +242,8 @@ def main_list(args): if not hit: bail_out("No such component, state or class: %s" % a) - # sort + # Filter + newbugs = [] for bug in bugs: skip = False for tp, entries in filters.items(): @@ -180,7 +251,18 @@ def main_list(args): skip = True break if not skip: - list_bug(bug) + newbugs.append(bug) + bugs = newbugs + + # Sort + if sort: + newbugs.sort(sort) + if reverse: + newbugs.reverse() + + # Output + for bug in bugs: + list_bug(bug) def main_show(args):

13 years, 9 months

Check_MK Git: check_mk: gb: fix implementation of grep

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 8a8a2d971f592b4f6bb66e7f9911a3bf63d67f01 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=8a8a2d971f592b… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Dec 22 15:50:37 2010 +0100 gb: fix implementation of grep --- gb | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/gb b/gb index a4afeb7..0d4de22 100755 --- a/gb +++ b/gb @@ -384,6 +384,7 @@ def main_grep(args): match = grep(bug["title"], kw) if match: bug["title"] = match + didmatch = True lines = bug["description"].split("\n") bodylines = "" for line in lines:

13 years, 9 months

Check_MK Git: check_mk: Livestatus: do not use is_contact_member_of_contactgroup anymore

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 2c7f7cd78cb14aa8ef3d3ca6f9ed5e051dd3e490 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=2c7f7cd78cb14a… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Dec 22 15:52:16 2010 +0100 Livestatus: do not use is_contact_member_of_contactgroup anymore --- .bugs/6 | 10 +++++++--- ChangeLog | 2 ++ livestatus/src/ContactgroupsMemberColumn.cc | 9 +++++++-- 3 files changed, 16 insertions(+), 5 deletions(-) diff --git a/.bugs/6 b/.bugs/6 index 08f6291..e6619fb 100644 --- a/.bugs/6 +++ b/.bugs/6 @@ -1,12 +1,16 @@ Title: is_contact_member_of_contactgroup selbst programmieren Component: livestatus +State: done +Class: todo +Date: 2010-12-21 10:55:19 Benefit: 5 -State: open Cost: 2 -Date: 2010-12-21 10:55:19 -Class: todo +Fun: 0 Diese Funktion ist im CVS schon weg und sehr leicht selbst zu programmieren. Sollte man bald mal machen, damit ein stabiles Livestatus schon bereit ist, wenn das neue Nagios kommt. + +2010-12-22 15:51:55: changed state open -> done +Ist umgesetzt. Sollte jetzt funktionieren. diff --git a/ChangeLog b/ChangeLog index f47fa0b..5715a30 100644 --- a/ChangeLog +++ b/ChangeLog @@ -42,6 +42,8 @@ column to work you need to specify the base directory of the PNP graphs with the module option pnp_path=, e.g. pnp_path=/omd/sites/wato/var/pnp4nagios/perfdata * Allow more than one column for StatsGroupBy: + * Do not use function is_contact_member_of_contactgroup anymore (get compatible + with Nagios CVS) 1.1.9i2: Checks & Agents: diff --git a/livestatus/src/ContactgroupsMemberColumn.cc b/livestatus/src/ContactgroupsMemberColumn.cc index d26b8ee..3811671 100644 --- a/livestatus/src/ContactgroupsMemberColumn.cc +++ b/livestatus/src/ContactgroupsMemberColumn.cc @@ -28,6 +28,11 @@ bool ContactgroupsMemberColumn::isNagiosMember(void *cg, void *ctc) { - return is_contact_member_of_contactgroup((contactgroup *)cg, (contact *)ctc); + contactsmember *mem; + for (mem = ((contactgroup *)cg)->members; mem != NULL; mem = mem->next) + { + if (mem->contact_ptr == ctc) + return true; + } + return false; } -

13 years, 9 months

Check_MK Git: check_mk: Finally resolved LIESMICH.zutun completely

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 1ae20590c002cd453c2106239279e4b9af4c0c9b URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=1ae20590c002cd… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Dec 22 11:27:49 2010 +0100 Finally resolved LIESMICH.zutun completely All bugs and ideas are now stored in .bugs/. Please use ./gb list for a complete list. ./gb shows a simple help of how to use gb alias "git bug" --- .bugs/40 | 2 +- .bugs/55 | 2 +- .bugs/70 | 20 +++++++++++++++++ .bugs/71 | 10 ++++++++ .bugs/72 | 11 +++++++++ .bugs/73 | 11 +++++++++ .bugs/74 | 24 ++++++++++++++++++++ .bugs/75 | 14 ++++++++++++ .bugs/76 | 14 ++++++++++++ .bugs/77 | 15 +++++++++++++ .bugs/78 | 19 ++++++++++++++++ .bugs/79 | 13 +++++++++++ .bugs/80 | 16 +++++++++++++ .bugs/81 | 14 ++++++++++++ .bugs/82 | 26 ++++++++++++++++++++++ .bugs/83 | 15 +++++++++++++ .bugs/84 | 18 +++++++++++++++ LIESMICH.zutun | 65 -------------------------------------------------------- 18 files changed, 242 insertions(+), 67 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=1ae20590c0…

13 years, 9 months

Check_MK Git: check_mk: Converted a lot of further bugs/ideas to .bugs

by git＠mathias-kettner.de

13 years, 9 months

Check_MK Git: check_mk: Converted further bugs to .bugs

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: d0c491ee4a770fb6119b40f790d1175532e381c7 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=d0c491ee4a770f… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Dec 21 20:55:26 2010 +0100 Converted further bugs to .bugs --- .bugs/29 | 10 ++++++++++ .bugs/30 | 15 +++++++++++++++ .bugs/31 | 10 ++++++++++ .bugs/32 | 13 +++++++++++++ .bugs/33 | 10 ++++++++++ .bugs/34 | 12 ++++++++++++ .bugs/35 | 11 +++++++++++ .bugs/36 | 13 +++++++++++++ LIESMICH.zutun | 38 -------------------------------------- 9 files changed, 94 insertions(+), 38 deletions(-) diff --git a/.bugs/29 b/.bugs/29 new file mode 100644 index 0000000..88ef4c1 --- /dev/null +++ b/.bugs/29 @@ -0,0 +1,10 @@ +Title: Link vom Detail-Host zum Summary-Host +Component: multisite +Benefit: 1 +State: open +Cost: 1 +Date: 2010-12-21 20:43:21 +Class: feature + +Umgekehrt ist der Link doch auch schon da. Evtl. muss man dafür ein +Host-Tag einbauen, dass analog zu REALHOST funktioniert. diff --git a/.bugs/30 b/.bugs/30 new file mode 100644 index 0000000..c22f4a9 --- /dev/null +++ b/.bugs/30 @@ -0,0 +1,15 @@ +Title: host_authorization Konfiguration für Multisite +Component: multisite +Benefit: 1 +State: open +Cost: 2 +Date: 2010-12-21 20:45:11 +Class: feature + +Livestatus: dritte Authorization-Regel: host_authorization: strict/loose. +Default ist strict: man muss explizit Kontakt sein, um den Host zu sehen. +Loose: wenn man für mindestens einen Service des Hosts Kontakt ist, kann +man ihn sehen. + +Ich frage mich allerdings, ob die anderen beiden Regeln wirklich funktionieren. +Das ist noch nicht ganz gesichert. diff --git a/.bugs/31 b/.bugs/31 new file mode 100644 index 0000000..788657f --- /dev/null +++ b/.bugs/31 @@ -0,0 +1,10 @@ +Title: Kommando für Custom Notification +Component: multisite +Benefit: 2 +State: open +Cost: 1 +Date: 2010-12-21 20:46:23 +Class: feature + +Beim normalen Nagios gibts das auch. Dazu Checkboxen für "force" +und für "broadcast". diff --git a/.bugs/32 b/.bugs/32 new file mode 100644 index 0000000..1507919 --- /dev/null +++ b/.bugs/32 @@ -0,0 +1,13 @@ +Title: modified_attributes bei Bedarf zurücksetzen +Component: multisite +Benefit: 1 +State: open +Cost: 3 +Date: 2010-12-21 20:48:09 +Class: nastiness + +Wenn man z.B. die active checks aus und wieder eingeschaltet hat, +dann erscheint der grüne Haken. Irgendwie müsste man die modified +Attributes zurücksetzen können. Evtl. hilft auch die Doku, dass +man das über die state retention konfiguriert. Das Kommando +zum Zurücksetzen heisst CHANGE_SVC_MODATTR;<host_name>;<service_description>;<value> diff --git a/.bugs/33 b/.bugs/33 new file mode 100644 index 0000000..cceb386 --- /dev/null +++ b/.bugs/33 @@ -0,0 +1,10 @@ +Title: Datei local.mk, die nicht gebackupt wird +Component: core +Benefit: 1 +State: open +Cost: 1 +Date: 2010-12-21 20:50:02 +Class: feature + +Analog zu final, kommt noch nach final und wird von backup/restore +nicht angefasst. diff --git a/.bugs/34 b/.bugs/34 new file mode 100644 index 0000000..fb7004d --- /dev/null +++ b/.bugs/34 @@ -0,0 +1,12 @@ +Title: Umstellen einer Datenquelle einer View möglich machen +Component: multisite +Benefit: 1 +State: open +Cost: 2 +Date: 2010-12-21 20:50:45 +Class: feature + +Bei einer bestehenden View muss man die Datenquelle umstellen +können, zumindest wenn die neue Datenquelle alle Spalten unterstützt, +die aktuell verwendet werden (und Filter). Man muss z.B. von hostgroups +auf hostgroups_merged umstellen können. Alles andere ist viel zu umständlich. diff --git a/.bugs/35 b/.bugs/35 new file mode 100644 index 0000000..2526ef6 --- /dev/null +++ b/.bugs/35 @@ -0,0 +1,11 @@ +Title: View-Editor: Spaltenzahl irgendwie dynamisch machen +Component: multisite +Benefit: 3 +State: open +Cost: 4 +Date: 2010-12-21 20:51:19 +Class: feature + +Das aktuelle Prinzip ist eh doof, weil man keine Spalten in der Mitte +einfügen kann. Das müsste man irgendwie über Javascript hinbekommen. +Auch dass man nicht auf 10 begrenzt ist. diff --git a/.bugs/36 b/.bugs/36 new file mode 100644 index 0000000..b5bc62c --- /dev/null +++ b/.bugs/36 @@ -0,0 +1,13 @@ +Title: Linux-NICs: Check kompatibel zu if/if64 machen +Component: core +Benefit: 2 +State: open +Cost: 2 +Date: 2010-12-21 20:52:24 +Class: cleanup + +Man könnte das doch evtl. kompatibel mit if machen. Und dann evtl auch die +Ausgabe noch kompatibel. Man monitort dann nur die verwendeten NICs. Und +man bau eine neue Sektion <<<nics>>>, welche die Ausgaben von ethtool +und /proc/net/dev kombiniert, damit man einen Check bauen kann, der +wirklich 100% kompatibel zu if ist (dann klappen auch die Network Wethermaps) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index bb4647e..5150e47 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -2,44 +2,6 @@ IDEEN & VERBESSERUNGEN -------------------------------------------------------------------------------- -Windows-Agent: Eventlogeinträge nach UTF-8 kodieren. - -Multisite: Ein Link vom Detail-Host zum Summary-Host (umgekehrt ist er doch -schon da, oder?) - -Livestatus: dritte Authorization-Regel: host_authorization: strict/loose. -Default ist strict: man muss explizit Kontakt sein, um den Host zu sehen. -Loose: wenn man für mindestens einen Service des Hosts Kontakt ist, kann -man ihn sehen. - -Kommando was dringend fehlt: Custom notification (mit force und broadcast) - -Neue Spalten für Views: * long_service_output, host_notes - -Multisite: Kann man - wenn man Aggregation verwendet - beim normalen -Host einen Link auf den Summary host machen und umgekehrt? Evtl. -hartkodiert in show_view(), falls die Variable host gesetzt ist? - -Multisite: icons: das Erkennen der modified_attributes -funktioniert nur halb: Wenn man den Ausgangswert wiederherstellt, -bleibt das Attribut trotzdem modified... -Kann man die modified attributes beim Nagios so einstellen, -dass diese *nicht* bei einem Restart gespeichert werden? -Sonst werden sie nie zurückgesetzt. Oder kann man das evtl. -mit einem Kommando machen? - - Ja => CHANGE_SVC_MODATTR;<host_name>;<service_description>;<value> - Eine Datei local.mk, die von backup/restore *nicht* beruecksichtigt wird. - -Multisite: Bei einer bestehenden View muss man die Datenquelle umstellen -können, zumindest wenn die neue Datenquelle alle Spalten unterstützt, -die aktuell verwendet werden (und Filter). Man muss z.B. von hostgroups -auf hostgroups_merged umstellen können. Alles andere ist viel zu umständlich. - -Edit view: die Spaltenzahl ist zu niedrig. - -Inventur von Linux-NICs: Nur verwendete Karten monitoren. - Doku Windows: C:\> lodctr /s:c:\counter.txt ==> Holt aus der Registry die Namen und Beschreibungen der

13 years, 9 months

Check_MK Git: check_mk: Converted further bugs to .bugs

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: e3b5d1a91675dc23a83d77c896d5fc436747b4f1 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=e3b5d1a91675dc… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Dec 21 16:35:17 2010 +0100 Converted further bugs to .bugs --- .bugs/18 | 13 ++++++++++ .bugs/19 | 12 +++++++++ .bugs/20 | 12 +++++++++ .bugs/21 | 10 +++++++ .bugs/22 | 18 +++++++++++++ .bugs/23 | 10 +++++++ .bugs/24 | 12 +++++++++ .bugs/25 | 9 +++++++ .bugs/26 | 11 ++++++++ .bugs/27 | 12 +++++++++ .bugs/28 | 11 ++++++++ .bugs/9 | 6 ++++ LIESMICH.zutun | 73 -------------------------------------------------------- gb | 73 ++++++++++++++++++++++++++++++++++++++++++++++++++----- 14 files changed, 202 insertions(+), 80 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=e3b5d1a916…

13 years, 9 months

Check_MK Git: check_mk: Converted further bugs from LIESMICH.zutun

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: ae21e60e545910a84dd317dd5012cd2e44a4076d URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=ae21e60e545910… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Dec 21 12:21:16 2010 +0100 Converted further bugs from LIESMICH.zutun --- .bugs/11 | 15 +++++++++++++++ .bugs/12 | 24 ++++++++++++++++++++++++ .bugs/13 | 13 +++++++++++++ .bugs/14 | 14 ++++++++++++++ .bugs/15 | 13 +++++++++++++ .bugs/16 | 22 ++++++++++++++++++++++ .bugs/17 | 15 +++++++++++++++ .bugs/config | 1 + LIESMICH.zutun | 46 ---------------------------------------------- gb | 2 +- 10 files changed, 118 insertions(+), 47 deletions(-) diff --git a/.bugs/11 b/.bugs/11 new file mode 100644 index 0000000..7a8c61d --- /dev/null +++ b/.bugs/11 @@ -0,0 +1,15 @@ +Title: Keine Fehlermeldung bei Connection closed +Component: wato +Benefit: 2 +State: open +Cost: 3 +Date: 2010-12-21 12:09:13 +Class: nastiness + +Wenn der Check_MK Agent einen nicht reinlässt (only_from gesetzt), +dann bekommen man nicht Connection refused, sondern eine +leere Ausgabe. Check_MK wertet das aber nicht als (richtiger) +Fehler. WATO zeigt bei der Inventur einfach keine Dienste an. +Der try-inventory sollte das aber als Fehler behandeln und +irgendwie anzeigen. Schließlich ist über die Tags ja festgelegt, +dass vom Agenten Daten kommen sollen. diff --git a/.bugs/12 b/.bugs/12 new file mode 100644 index 0000000..160ac9f --- /dev/null +++ b/.bugs/12 @@ -0,0 +1,24 @@ +Title: Sorry, Operator 4 for lists not implemented. +Component: livestatus +Benefit: 1 +State: open +Cost: 3 +Date: 2010-12-21 12:11:12 +Class: nastiness + +Wenn ich im Thruk im Suchfenster einfach Enter drücke, kommt folgende Query bei raus: +GET services +Columns: .. +Filter: description ~~ +Filter: groups >= +Filter: plugin_output ~~ +Filter: long_plugin_output ~~ +Filter: host_name ~~ +Filter: host_alias ~~ +Filter: host_groups >= +Filter: comments ~~ +Filter: downtimes ~~ +Or: 2 +Or: 8 +livestatus: Sorry, Operator 4 for lists not implemented. +Was ist eigentlich Operator 4? diff --git a/.bugs/13 b/.bugs/13 new file mode 100644 index 0000000..3f7a57c --- /dev/null +++ b/.bugs/13 @@ -0,0 +1,13 @@ +Title: Umlaute in SNMP-Daten und Servicedescription +Component: checks +Benefit: 1 +State: open +Cost: 4 +Date: 2010-12-21 12:12:02 +Class: bug + +Ein Drucker sendet per SNMP Latin1 +kodierte Wert, z.B. Resttonerbehälter. Daraus baut Check_MK eine +Servicedesription, die in nagios/conf.d/check_mk_objects.cfg ein +Latin1-Zeichen enthält. Hier sollte unbedingt UTF-8 verwendet +werden. Nur: woher kennen wir die Kodierung vom Drucker?? diff --git a/.bugs/14 b/.bugs/14 new file mode 100644 index 0000000..e4513b2 --- /dev/null +++ b/.bugs/14 @@ -0,0 +1,14 @@ +Title: UTF-8 Kodierung von Eurosymbol geht nicht +Component: livestatus +Benefit: 1 +State: open +Cost: 3 +Date: 2010-12-21 12:12:58 +Class: bug + +Livestatus: bestimmte UTF-8 Zeichen werden nicht korrekt kodiert, z.B. "€". +Um das umzustellen, muss man wahrscheinlich einfach UTF-8 generell +annehmen. + +Die aktuelle Implementierung erkennt halbautomatisch eine Mischung +aus Latin1 und UTF-8. Kann man bei Nagios einfach UTF-8 verlangen? diff --git a/.bugs/15 b/.bugs/15 new file mode 100644 index 0000000..9117096 --- /dev/null +++ b/.bugs/15 @@ -0,0 +1,13 @@ +Title: Counter-Wrap: bei service aggregation fehlen Checks +Component: core +Benefit: 1 +State: open +Cost: 3 +Date: 2010-12-21 12:14:07 +Class: nastiness + +Wenn ein Counter Wrap passiert und der check selbst die Exception +nicht abfängt, dann wird das Ergebnis nicht an Nagios weitergeleitet. +Der Service wird auch nicht in die aggregierten Services eines Summary Hosts +aufgenommen. Das kann dazu führen, dass die Anzahl der Services in einem +Summary-Host variabel ist. diff --git a/.bugs/16 b/.bugs/16 new file mode 100644 index 0000000..7dd601e --- /dev/null +++ b/.bugs/16 @@ -0,0 +1,22 @@ +Title: check_mk -II bei Clustern löscht die Checks nicht +Component: core +Benefit: 2 +State: open +Cost: 3 +Date: 2010-12-21 12:15:32 +Class: bug + +Wenn man ein check_mk -II macht, dann werden clustered +services nicht entfernt. Um das sauber hinzubekommen, muss man sicherstellen, +dass der Anwender check_mk -II mit *allen* nodes aufgerufen hat. Denn nur +so darf man die Checks auf dem Cluster entfernen. Sonst kann man nicht sicher +sein, dass Services wegfliegen, die eigentlich von einem anderem Node kommen. +Das müsste dann auch micht clustered_services_of funktionieren. Ich muss +einfach alle Cluster durchgehen, feststellen, ob jeweils alle deren Nodes +inventurisiert werden, und darf dann die betroffenen Services wegwerfen. + +Man könnte auch überlegen, dass man ein -I oder -II auf den Cluster-Namen +zulässt und dann einfach automatisch anstelle dessen die Nodes inventarisiert +und zusätzlich die Checks vom Cluster entfernt. Das könnte eine einfache +Lösung sein. So sorgt der User selbst dafür, dass der Cluster richtig +behandelt wird. diff --git a/.bugs/17 b/.bugs/17 new file mode 100644 index 0000000..6be03cb --- /dev/null +++ b/.bugs/17 @@ -0,0 +1,15 @@ +Title: Tabelle log bei multisite setup: Chronologie ist falsch +Component: multisite +Benefit: 2 +State: open +Cost: 3 +Date: 2010-12-21 12:17:45 +Class: bug + +Die Logereignisse von den verschiedenen Sites werden nicht +korrekt zusammengemischt. Auch das mit dem Limit wird dann +blöd interpretiert. Eigentlich braucht man noch ein nachgelagertes +Limit, dass Multisite selbst anwendet (auch für die Alert statistics). +Es muss so laufen: Bei einem Limit von 100 bekommt jede Site einzeln +das Limit von 100. Dann sortiert man. Und dann wendet man nochmal +das Limit von 100 an. diff --git a/.bugs/config b/.bugs/config index fa2ff10..f73a243 100644 --- a/.bugs/config +++ b/.bugs/config @@ -15,6 +15,7 @@ components = [ classes = [ "cleanup", + "nastiness", "bug", "todo", "phantasm", diff --git a/LIESMICH.zutun b/LIESMICH.zutun index b5b5064..1070810 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -1,49 +1,3 @@ -WATO: Wenn ein Agent einen rauswirft (leere Ausgabe, Connection closed), kommt im -WATO bei der Inventur keine sinnvolle Fehlermeldung. - -ich bin mir nicht sicher ob das nun ein Thruk Problem oder ein Livestatus Problem ist. Vermutlich beides :-) -Wenn ich im Thruk im Suchfenster einfach Enter drücke, kommt folgende Query bei raus: -GET services -Columns: .. -Filter: description ~~ -Filter: groups >= -Filter: plugin_output ~~ -Filter: long_plugin_output ~~ -Filter: host_name ~~ -Filter: host_alias ~~ -Filter: host_groups >= -Filter: comments ~~ -Filter: downtimes ~~ -Or: 2 -Or: 8 -livestatus: Sorry, Operator 4 for lists not implemented. - - -Noch ein Problem mit Umlauten: Ein Drucker sendet per SNMP Latin1 -kodierte Wert, z.B. Resttonerbehälter. Daraus baut Check_MK eine -Servicedesription, die in nagios/conf.d/check_mk_objects.cfg ein -Latin1-Zeichen enthält. Hier sollte unbedingt UTF-8 verwendet -werden. Nur: woher kennen wir die Kodierung vom Drucker?? - -Livestatus: bestimmte UTF-8 Zeichen werden nicht korrekt kodiert, z.B. "€". -Um das umzustellen, muss man wahrscheinlich einfach UTF-8 generell -annehmen. - -Wenn ein Counter Wrap passiert und der check selbst die Exception -nicht abfängt, dann wird das Ergebnis nicht an Nagios weitergeleitet. -Der Service wird auch nicht in die aggregierten Services eines Summary Hosts -aufgenommen. Das kann dazu führen, dass die Anzahl der Services in einem -Summary-Host variabel ist. - -Cluster-Inventur(1): Wenn man ein check_mk -II macht, dann werden clustered -services nicht entfernt. Um das sauber hinzubekommen, muss man sicherstellen, -dass der Anwender check_mk -II mit *allen* nodes aufgerufen hat. Denn nur -so darf man die Checks auf dem Cluster entfernen. Sonst kann man nicht sicher -sein, dass Services wegfliegen, die eigentlich von einem anderem Node kommen. -Das müsste dann auch micht clustered_services_of funktionieren. Ich muss -einfach alle Cluster durchgehen, feststellen, ob jeweils alle deren Nodes -inventurisiert werden, und darf dann die betroffenen Services wegwerfen. - Cluster-Inventur(2): Wenn ein Service auf der Node sitzt und man die Regeln zu Clustered services ändert, entfernt ein -II das nicht von der node. diff --git a/gb b/gb index 01f9721..559d48f 100755 --- a/gb +++ b/gb @@ -134,7 +134,7 @@ def bug_state(bug): return "%s%-7s%s" % (tty_bold + states.get(state, ""), state, tty_normal) def list_bug(bug): - sys.stdout.write("#%04d %-6s %-7s %-10s %-60s\n" % + sys.stdout.write("#%04d %-6s %-9s %-10s %-60s\n" % (int(bug["id"]), bug_state(bug), bug["class"], bug["component"], bug["title"])) def show_bug(bug):

13 years, 9 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits