Checkmk git commits

checkmk-commits@lists.checkmk.com

1 participants
65243 discussions

Check_MK Git: check_mk: Makefile: ship missing .js files

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 66c9418cb97e266a6513d53f0f165a7779e086d2 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=66c9418cb97e26… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 19:43:46 2010 +0100 Makefile: ship missing .js files --- Makefile | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 09931eb..9ad879c 100644 --- a/Makefile +++ b/Makefile @@ -61,7 +61,7 @@ dist: mk-livestatus mkdir -p $(DISTNAME) tar czf $(DISTNAME)/checks.tar.gz $(TAROPTS) -C checks $$(cd checks ; ls) tar czf $(DISTNAME)/checkman.tar.gz $(TAROPTS) -C checkman $$(cd checkman ; ls) - tar czf $(DISTNAME)/web.tar.gz $(TAROPTS) -C web $$(cd web ; ls htdocs/*/*.{wav,jpg,png,gif,ico} htdocs/*.{py,css,js} plugins/*/*.py) + tar czf $(DISTNAME)/web.tar.gz $(TAROPTS) -C web $$(cd web ; ls htdocs/*/*.{wav,jpg,png,gif,ico,js} htdocs/*.{py,css} plugins/*/*.py) tar czf $(DISTNAME)/livestatus.tar.gz $(TAROPTS) -C livestatus $$(cd livestatus ; echo $(LIVESTATUS_SOURCES) ) tar czf $(DISTNAME)/pnp-templates.tar.gz $(TAROPTS) -C pnp-templates $$(cd pnp-templates ; ls *.php) tar czf $(DISTNAME)/pnp-rraconf.tar.gz $(TAROPTS) -C pnp-rraconf $$(cd pnp-rraconf ; ls *.rra.cfg README.rra)

13 years, 10 months

Check_MK Git: check_mk: printer_supply: handle case where no current available

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: c035d82729704e2b2e3cad430e92c736f7bd203d URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=c035d82729704e… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 17:13:08 2010 +0100 printer_supply: handle case where no current available --- ChangeLog | 1 + checks/printer_supply | 2 +- 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6df57d9..f47fa0b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -35,6 +35,7 @@ * Fixed problem with OnlyFrom: in Linux agent (df didn't work properly) * cups_queues: fixed plugin error due to invalid import of datetime, converted other checks from 'from datetime import...' to 'import datetime'. + * printer_supply: handle the case where the current value is missing Livestatus: * new column pnpgraph_present in table host and service. In order for this diff --git a/checks/printer_supply b/checks/printer_supply index a6df030..ed387fc 100644 --- a/checks/printer_supply +++ b/checks/printer_supply @@ -79,7 +79,7 @@ def check_printer_supply(item, params, info): for line in printer_supply_fix_infos(info): if line[0] == item: maxlevel = float(line[1]) - current = float(line[2]) + current = savefloat(line[2]) leftperc = 100.0 * current / maxlevel warn, crit = params # in percent infotext = "%.0f%% (levels at %.0f%% / %.0f%%)" % (leftperc, warn, crit)

13 years, 10 months

Check_MK Git: check_mk: Multisite: new view "Alert Statistics"

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 8027244d5931624a49a76748c8b3baca6eb040f1 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=8027244d593162… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 17:06:48 2010 +0100 Multisite: new view "Alert Statistics" --- ChangeLog | 3 ++ LIESMICH.zutun | 11 +++++-- web/htdocs/views.py | 27 +++++++++++----- web/plugins/views/builtin.py | 61 ++++++++++++++++++++++++++++++++++++- web/plugins/views/datasources.py | 9 +++++ web/plugins/views/painters.py | 35 +++++++++++++++++++++ web/plugins/views/sorters.py | 8 +++++ 7 files changed, 140 insertions(+), 14 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=8027244d59…

13 years, 10 months

Check_MK Git: check_mk: Livestatus: use Columns: instead of StatsGroupBy:

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 5be217725b7c213cc1a3c975b928a12027d4d23e URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=5be217725b7c21… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 16:12:00 2010 +0100 Livestatus: use Columns: instead of StatsGroupBy: StatsGroupBy: is now deprecated. Columns: does the same thing for you. --- ChangeLog | 3 +++ livestatus/src/Query.cc | 31 ++++++++----------------------- livestatus/src/Query.h | 3 --- 3 files changed, 11 insertions(+), 26 deletions(-) diff --git a/ChangeLog b/ChangeLog index 15fe292..ccef56e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -4,6 +4,9 @@ * Multisite: removed (undocumented) view parameters show_buttons and show_controls. Please use display_options instead. * Finally removed deprecated filesystem_levels. Please use check_parameters instead. + * Livestatus: The StatsGroupBy: header is still working but now deprecated. + Please simply use Columns: instead. If your query contains at least one Stats: + header than Columns: has the meaning of the old StatsGroupBy: header Core, Setup, etc.: * Create alias 'cmk' for check_mk in bin/ (easier typing) diff --git a/livestatus/src/Query.cc b/livestatus/src/Query.cc index c9a3542..877b765 100644 --- a/livestatus/src/Query.cc +++ b/livestatus/src/Query.cc @@ -177,12 +177,12 @@ Column *Query::createDummyColumn(const char *name) } - void Query::addColumn(Column *column) { _columns.push_back(column); } + bool Query::hasNoColumns() { return _columns.size() == 0 && !doStats(); @@ -419,23 +419,8 @@ void Query::parseAuthUserHeader(char *line) void Query::parseStatsGroupLine(char *line) { - if (!_table) - return; - - char *column_name; - while (column_name = next_field(&line)) { - Column *column = _table->column(column_name); - if (!column) { - _output->setError(RESPONSE_CODE_INVALID_HEADER, "StatsGroupBy: unknown column '%s'", column_name); - return; - } - _stats_group_columns.push_back(column); - } - - if (_stats_group_columns.size() == 0) { - _output->setError(RESPONSE_CODE_INVALID_HEADER, "StatsGroupBy: missing an argument"); - return; - } + logger(LOG_WARNING, "Warning: StatsGroupBy is deprecated. Please use Columns instead."); + parseColumnsLine(line); } @@ -645,7 +630,7 @@ void Query::start() // if we have no StatsGroupBy: column, we allocate one only row of Aggregators, // directly in _stats_aggregators. When grouping the rows of aggregators // will be created each time a new group is found. - if (_stats_group_columns.size() == 0) + if (_columns.size() == 0) { _stats_aggregators = new Aggregator *[_stats_columns.size()]; for (unsigned i=0; i<_stats_columns.size(); i++) @@ -691,7 +676,7 @@ bool Query::processDataset(void *data) Aggregator **aggr; // When doing grouped stats, we need to fetch/create a row // of aggregators for the current group - if (_stats_group_columns.size() > 0) { + if (_columns.size() > 0) { _stats_group_spec_t groupspec; computeStatsGroupSpec(groupspec, data); aggr = getStatsGroup(groupspec); @@ -732,7 +717,7 @@ bool Query::processDataset(void *data) void Query::finish() { // grouped stats - if (doStats() && _stats_group_columns.size() > 0) + if (doStats() && _columns.size() > 0) { // output values of all stats groups (output has been post poned until now) for (_stats_groups_t::iterator it = _stats_groups.begin(); @@ -984,8 +969,8 @@ Aggregator **Query::getStatsGroup(Query::_stats_group_spec_t &groupspec) void Query::computeStatsGroupSpec(Query::_stats_group_spec_t &groupspec, void *data) { - for (_stats_group_columns_t::iterator it = _stats_group_columns.begin(); - it != _stats_group_columns.end(); + for (_columns_t::iterator it = _columns.begin(); + it != _columns.end(); ++it) { Column *column = *it; diff --git a/livestatus/src/Query.h b/livestatus/src/Query.h index 5cc1955..1a85b63 100644 --- a/livestatus/src/Query.h +++ b/livestatus/src/Query.h @@ -87,9 +87,6 @@ class Query _stats_columns_t _stats_columns; // must also delete Aggregator **_stats_aggregators; - typedef vector <Column *> _stats_group_columns_t; - _stats_group_columns_t _stats_group_columns; - typedef vector<string> _stats_group_spec_t; typedef map<_stats_group_spec_t, Aggregator **> _stats_groups_t; _stats_groups_t _stats_groups;

13 years, 10 months

Check_MK Git: check_mk: Livestatus: StatsGroupBy: allows more than one columns

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 1822486a776386bff491ba50644d5b5e3effe9c4 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=1822486a776386… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 15:57:27 2010 +0100 Livestatus: StatsGroupBy: allows more than one columns --- ChangeLog | 1 + LIESMICH.zutun | 9 +--- livestatus/src/Query.cc | 110 ++++++++++++++++++++++++++++++---------------- livestatus/src/Query.h | 11 +++- 4 files changed, 82 insertions(+), 49 deletions(-) diff --git a/ChangeLog b/ChangeLog index d60c93d..15fe292 100644 --- a/ChangeLog +++ b/ChangeLog @@ -34,6 +34,7 @@ * new column pnpgraph_present in table host and service. In order for this 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: 1.1.9i2: Checks & Agents: diff --git a/LIESMICH.zutun b/LIESMICH.zutun index ecbbb5d..b749764 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -71,10 +71,6 @@ 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. -Livestatus: wenn man mehrere StatsGroupBy macht, wird der Speicher von den -alten nicht freigegeben (ist momentan eh nicht erlaubt. Fehlermeldung ausgeben). -In neuer Version sollte man irgendwann mehrere Group-Header erlauben. - Multisite: Table log: Hier wird nur der erste Site angezeigt, bzw. die Chronologie Reihenfolge stimmt nicht. Sortieren nach Sekunden plus Zeilennummer. @@ -462,12 +458,9 @@ mit dem Windows-Check erstellen (wo ein Graph eh fehlt). Multisite: Quicksearch evtl. case-insensitive machen? -Livestatus: StatsGroupBy: mehrere Spalten erlauben - Multisite: Die TOP-25 Alerts: Welche Dinge hosts/services hatten in einem Zeitraum die meisten Alerts. Das geht mit log und StatsGroupBy: host_name -service_description. Dazu muss ich allerdings zuerst eine Gruppierung nach -mehr als einem Feld einbauen. +service_description. Multisite: Filter, der Hosts zeigt, die entweder selbst Summary hosts sind oder keinen haben. diff --git a/livestatus/src/Query.cc b/livestatus/src/Query.cc index 6e7a130..c9a3542 100644 --- a/livestatus/src/Query.cc +++ b/livestatus/src/Query.cc @@ -62,8 +62,7 @@ Query::Query(InputBuffer *input, OutputBuffer *output, Table *table) : _output_format(OUTPUT_FORMAT_CSV), _limit(-1), _current_line(0), - _timezone_offset(0), - _stats_group_column(0) + _timezone_offset(0) { while (input->moreLines()) { @@ -151,6 +150,8 @@ Query::Query(InputBuffer *input, OutputBuffer *output, Table *table) : Query::~Query() { + return; + // delete dummy-columns for (_columns_t::iterator it = _dummy_columns.begin(); it != _dummy_columns.end(); @@ -418,22 +419,23 @@ void Query::parseAuthUserHeader(char *line) void Query::parseStatsGroupLine(char *line) { - if (!_table) - return; - - char *column_name = next_field(&line); - if (!column_name) { - _output->setError(RESPONSE_CODE_INVALID_HEADER, "StatsGroupBy: missing an argument"); - return; - } + if (!_table) + return; - Column *column = _table->column(column_name); - if (!column) { - _output->setError(RESPONSE_CODE_INVALID_HEADER, "StatsGroupBy: unknown column '%s'", column_name); - return; - } + char *column_name; + while (column_name = next_field(&line)) { + Column *column = _table->column(column_name); + if (!column) { + _output->setError(RESPONSE_CODE_INVALID_HEADER, "StatsGroupBy: unknown column '%s'", column_name); + return; + } + _stats_group_columns.push_back(column); + } - _stats_group_column = column; + if (_stats_group_columns.size() == 0) { + _output->setError(RESPONSE_CODE_INVALID_HEADER, "StatsGroupBy: missing an argument"); + return; + } } @@ -628,6 +630,7 @@ bool Query::doStats() return _stats_columns.size() > 0; } + void Query::start() { doWait(); @@ -642,7 +645,7 @@ void Query::start() // if we have no StatsGroupBy: column, we allocate one only row of Aggregators, // directly in _stats_aggregators. When grouping the rows of aggregators // will be created each time a new group is found. - if (!_stats_group_column) + if (_stats_group_columns.size() == 0) { _stats_aggregators = new Aggregator *[_stats_columns.size()]; for (unsigned i=0; i<_stats_columns.size(); i++) @@ -685,21 +688,22 @@ bool Query::processDataset(void *data) if (doStats()) { - Aggregator **aggr; - // When doing grouped stats, we need to fetch/create a row - // of aggregators for the current group - if (_stats_group_column) { - string groupname = _stats_group_column->valueAsString(data, this); - aggr = getStatsGroup(groupname); - } - else - aggr = _stats_aggregators; - - for (unsigned i=0; i<_stats_columns.size(); i++) - aggr[i]->consume(data, this); - - // No output is done while processing the data, we only - // collect stats. + Aggregator **aggr; + // When doing grouped stats, we need to fetch/create a row + // of aggregators for the current group + if (_stats_group_columns.size() > 0) { + _stats_group_spec_t groupspec; + computeStatsGroupSpec(groupspec, data); + aggr = getStatsGroup(groupspec); + } + else + aggr = _stats_aggregators; + + for (unsigned i=0; i<_stats_columns.size(); i++) + aggr[i]->consume(data, this); + + // No output is done while processing the data, we only + // collect stats. } else { @@ -728,16 +732,32 @@ bool Query::processDataset(void *data) void Query::finish() { // grouped stats - if (doStats() && _stats_group_column) + if (doStats() && _stats_group_columns.size() > 0) { + // output values of all stats groups (output has been post poned until now) for (_stats_groups_t::iterator it = _stats_groups.begin(); it != _stats_groups.end(); ++it) { + // dataset separator after first group if (it != _stats_groups.begin() && _output_format != OUTPUT_FORMAT_CSV) _output->addBuffer(",\n", 2); + outputDatasetBegin(); - outputString(it->first.c_str()); + + // output group columns first + _stats_group_spec_t groupspec = it->first; + bool first = true; + for (_stats_group_spec_t::iterator iit = groupspec.begin(); + iit != groupspec.end(); + ++iit) + { + if (!first) + outputFieldSeparator(); + else + first = false; + outputString((*iit).c_str()); + } Aggregator **aggr = it->second; for (unsigned i=0; i<_stats_columns.size(); i++) { @@ -818,7 +838,7 @@ void Query::outputInteger(int32_t value) void Query::outputInteger64(int64_t value) { char buf[32]; - int l = snprintf(buf, 32, "%lld", value); + int l = snprintf(buf, 32, "%lld", (long long int )value); _output->addBuffer(buf, l); } @@ -948,20 +968,32 @@ void Query::outputEndSublist() _output->addChar(']'); } -Aggregator **Query::getStatsGroup(string name) +Aggregator **Query::getStatsGroup(Query::_stats_group_spec_t &groupspec) { - _stats_groups_t::iterator it = _stats_groups.find(name); + _stats_groups_t::iterator it = _stats_groups.find(groupspec); if (it == _stats_groups.end()) { Aggregator **aggr = new Aggregator *[_stats_columns.size()]; for (unsigned i=0; i<_stats_columns.size(); i++) aggr[i] = _stats_columns[i]->createAggregator(); - _stats_groups.insert(make_pair(string(name), aggr)); + _stats_groups.insert(make_pair(groupspec, aggr)); return aggr; } else return it->second; } +void Query::computeStatsGroupSpec(Query::_stats_group_spec_t &groupspec, void *data) +{ + for (_stats_group_columns_t::iterator it = _stats_group_columns.begin(); + it != _stats_group_columns.end(); + ++it) + { + Column *column = *it; + groupspec.push_back(column->valueAsString(data, this)); + } +} + + void Query::doWait() { // If no wait condition and no trigger is set, @@ -1016,3 +1048,5 @@ void Query::doWait() } } while (!_wait_condition.accepts(_wait_object)); } + + diff --git a/livestatus/src/Query.h b/livestatus/src/Query.h index 49520e2..5cc1955 100644 --- a/livestatus/src/Query.h +++ b/livestatus/src/Query.h @@ -86,8 +86,12 @@ class Query typedef vector<StatsColumn *> _stats_columns_t; _stats_columns_t _stats_columns; // must also delete Aggregator **_stats_aggregators; - Column *_stats_group_column; - typedef map<string, Aggregator **> _stats_groups_t; + + typedef vector <Column *> _stats_group_columns_t; + _stats_group_columns_t _stats_group_columns; + + typedef vector<string> _stats_group_spec_t; + typedef map<_stats_group_spec_t, Aggregator **> _stats_groups_t; _stats_groups_t _stats_groups; public: @@ -128,7 +132,8 @@ public: private: bool doStats(); void doWait(); - Aggregator **getStatsGroup(string name); + Aggregator **getStatsGroup(_stats_group_spec_t &groupspec); + void computeStatsGroupSpec(_stats_group_spec_t &groupspec, void *data); Filter *createFilter(Column *column, int operator_id, char *value); void parseFilterLine(char *line, bool filter /* and not cond */); void parseStatsLine(char *line);

13 years, 10 months

Check_MK Git: check_mk: Multisite: new column host_tags

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 4e0d775041baeac2a41545172861349f07c6be4a URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=4e0d775041baea… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 13:32:08 2010 +0100 Multisite: new column host_tags --- ChangeLog | 3 ++- web/plugins/views/painters.py | 13 +++++++++++++ 2 files changed, 15 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index b657dd1..d60c93d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -21,8 +21,9 @@ * new HTML parameter display_options, which allows to switch off several parts of the output (e.g. the HTML header, external links, etc). * View hoststatus: show PNP graph of host (usually ping stats) - * Multisite: new tab "Display": here the user can choose time stamp + * new tab "Display": here the user can choose time stamp display format and PNP graph ranges + * new column "host_tags", showing the Check_MK host tags of a host Checks & Agents: * Fixed problem with OnlyFrom: in Linux agent (df didn't work properly) diff --git a/web/plugins/views/painters.py b/web/plugins/views/painters.py index fab7163..a60fea5 100644 --- a/web/plugins/views/painters.py +++ b/web/plugins/views/painters.py @@ -1000,6 +1000,19 @@ multisite_painters["host_custom_notes"] = { "paint" : paint_custom_notes } +def paint_host_tags(row): + for name, val in zip(row["host_custom_variable_names"], + row["host_custom_variable_values"]): + if name == "TAGS": + return "", val + return "","" + +multisite_painters["host_tags"] = { + "title" : "Host Tags (Check_MK)", + "short" : "Tags", + "columns" : [ "host_custom_variable_names", "host_custom_variable_values" ], + "paint" : paint_host_tags +} # _ _ _ # | | | | ___ ___| |_ __ _ _ __ ___ _ _ _ __ ___

13 years, 10 months

Check_MK Git: check_mk: Multisite: Add PNP graph to host detail view

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: de5b8ee14433561bf4c3a82eb466469b0ec26307 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=de5b8ee1443356… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 12:22:27 2010 +0100 Multisite: Add PNP graph to host detail view --- ChangeLog | 1 + web/plugins/views/builtin.py | 1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/ChangeLog b/ChangeLog index ca6645f..bc3a990 100644 --- a/ChangeLog +++ b/ChangeLog @@ -20,6 +20,7 @@ context buttons (cleans up button bar). * new HTML parameter display_options, which allows to switch off several parts of the output (e.g. the HTML header, external links, etc). + * View hoststatus: show PNP graph of host (usually ping stats) Checks & Agents: * Fixed problem with OnlyFrom: in Linux agent (df didn't work properly) diff --git a/web/plugins/views/builtin.py b/web/plugins/views/builtin.py index 99d0254..d660116 100644 --- a/web/plugins/views/builtin.py +++ b/web/plugins/views/builtin.py @@ -842,6 +842,7 @@ multisite_builtin_views = { ('num_services_unknown', 'host_unknown'), ('num_services_pending', 'host_pending'), ('host_custom_notes', None), + ('host_pnpgraph', None), ], 'public': True, 'show_filters': [],

13 years, 10 months

Check_MK Git: check_mk: Multisite: new feature: display options

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: a871bfbe45409bcbbe76e79dd7a7a3a6da0ed250 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=a871bfbe45409b… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon Dec 20 12:42:29 2010 +0100 Multisite: new feature: display options --- ChangeLog | 2 + LIESMICH.zutun | 9 ---- web/htdocs/config.py | 6 +++ web/htdocs/js/check_mk.js | 6 +- web/htdocs/views.py | 84 ++++++++++++++++++++++++++++++++++++---- web/plugins/views/painters.py | 58 ++++++++++++++++++++++++++-- 6 files changed, 140 insertions(+), 25 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=a871bfbe45…

13 years, 10 months

Check_MK Git: check_mk: Updated internal doku

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 48b7590c8958d66c16dac1e7e6717c8f412c4ca2 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=48b7590c8958d6… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Fri Dec 17 08:29:08 2010 +0100 Updated internal doku --- LIESMICH.zutun | 79 +++++++++++++++++++++++++------------------------------ 1 files changed, 36 insertions(+), 43 deletions(-) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index 3dac633..ec2d693 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -338,16 +338,16 @@ Auf diese Art funktioniert erstmal alles weiter. Trotzdem wird die Variable irgendwann planmäßig abgeschaltet. View options für Painter: Ein Painter hat eine Liste von Painter-Viewoptions. -Dazu gibt es ein dict: painter_view_options["pnpsource"] = { -( "PNP Timerange", "1", [("0":"4 hours"), ("1", "24 hours"), ... ] ) -Vor dem Anzeigen des Layouts ermittle ich alle benötigten View-Options. -Analog zu refersh mache ich Persisitierung und HTML-Variablen-Auswertung -und komme zu einem Ergebnis. Das speichere ich dann in view_option["pnpsource"]. -Dies kann der Painter einfach abfragen. Einsatz: PNP-Zeitraum, Darstellung -von Zeitstempeln (Delta/Absolute/Mixed). +Dazu gibt es ein dict: painter_view_options["pnpsource"] = { ( "PNP Timerange", +"1", [("0":"4 hours"), ("1", "24 hours"), ... ] ) Vor dem Anzeigen des +Layouts ermittle ich alle benötigten View-Options. Analog zu refersh +mache ich Persisitierung und HTML-Variablen-Auswertung und komme zu einem +Ergebnis. Das speichere ich dann in view_option["pnpsource"]. Dies kann der +Painter einfach abfragen. Einsatz: PNP-Zeitraum, Darstellung von Zeitstempeln +(Delta/Absolute/Mixed). -Multisite: Wenn man bei einer View einen Filter ausfüllt (user), dann -soll irgendwo ein Icon anzeigen, dass das Resultat gefiltert ist. +Multisite: Wenn man bei einer View einen Filter ausfüllt (user), dann soll +irgendwo ein Icon anzeigen, dass das Resultat gefiltert ist. Multisite: Hover: Wenn man über einen Painter geht, könnte man eine Hover aufmachen, der einen weiteren Painter anzeigt. Das macht man @@ -408,8 +408,8 @@ FreeBSD Agent: Die Sektionen/Checks ps und postfix_mailq sind jetzt schon kompatibel. Idee: Bei SNMP-Checks gibt es manchmal Daten, die sich dynamisch nicht ändern -(z.B. Interfacenamen, etc.). Man könnte diese Daten cachen, so dass sie nicht -jedes mal geholt werden, sondern z.B. nur alle 10 Minuten. +(z.B. Interfacenamen, etc.). Man könnte diese Daten cachen, so dass sie +nicht jedes mal geholt werden, sondern z.B. nur alle 10 Minuten. IDEE: Prefetching agent: Der agent soll die Daten schon berechnen kurz bevor sie abgefragt werden. Das ganze läuft so: Zunächst ermittelt der Agent, @@ -433,12 +433,12 @@ Livestatus: Spalte bei Services und Hosts, die das ausgeführte Kommando Multisite: Spalte, die das Kommando (die Befehlszeile) ausgibt. -Multisite: Man soll über eine Variable die URL für die rechte -Seite mitgeben können. So kann man direkt auf eine Unterseite -verlinken. Evtl. das sogar abrufbar über ein Icon :-) +Multisite: Man soll über eine Variable die URL für die rechte Seite mitgeben +können. So kann man direkt auf eine Unterseite verlinken. Evtl. das sogar +abrufbar über ein Icon :-) -Livestatus: Informationen ueber Eskalationen ausgeben (eigene Tabelle - oder Anreicherung von contacts, hosts, services) +Livestatus: Informationen ueber Eskalationen ausgeben (eigene Tabelle oder +Anreicherung von contacts, hosts, services) IPv6-Support in Check_MK: def lookup_ipaddress(hostname): @@ -459,41 +459,31 @@ def lookup_ipaddress(hostname): Ausserdem kommt check_icmp damit nicht klar. Hier muss dann auf check_ping ausgewichen werden. -Wenn eine IP-Adresse nicht aufgelöst werden kann, sollte stattdessen -eine (konfigurierbare) Dummyadresse verwendet werden. Sonst -scheitert check_mk -O, wenn ein DNS-Eintrag verschwindet. +Wenn eine IP-Adresse nicht aufgelöst werden kann, sollte stattdessen eine +(konfigurierbare) Dummyadresse verwendet werden. Sonst scheitert check_mk -O, +wenn ein DNS-Eintrag verschwindet. -Livestatus: neben custom_variable_names und custom_variable_values -waere noch custom_variables huebsch, welches beides kombiniert. +Livestatus: neben custom_variable_names und custom_variable_values waere +noch custom_variables huebsch, welches beides kombiniert. + +Disk IO read/write: Zu einem Check zusammenfassen. Einen gemeinsamen Graphen +mit dem Windows-Check erstellen (wo ein Graph eh fehlt). -Disk IO read/write: Zu einem Check zusammenfassen. Einen gemeinsamen -Graphen mit dem Windows-Check erstellen (wo ein Graph eh fehlt). - Multisite: Quicksearch evtl. case-insensitive machen? Livestatus: StatsGroupBy: mehrere Spalten erlauben -Multisite: Die TOP-25 Alerts: Welche Dinge hosts/services -hatten in einem Zeitraum die meisten Alerts. Das geht -mit log und StatsGroupBy: host_name service_description - -Multisite: Filter, der Hosts zeigt, die entweder selbst -Summary hosts sind oder keinen haben. +Multisite: Die TOP-25 Alerts: Welche Dinge hosts/services hatten in einem +Zeitraum die meisten Alerts. Das geht mit log und StatsGroupBy: host_name +service_description. Dazu muss ich allerdings zuerst eine Gruppierung nach +mehr als einem Feld einbauen. -Multisite: Könnte man nicht Aggregationen in Views einbauen. -Man würde dann in der View Regeln definieren, nach denen -Services und Hosts aggregiert werden können. Dazu müsste man -in den View-Editor einen neuen Abschnitt einbauen. Ist das -nicht ähnlich wie Gruppieren? Allerdings wäre es dann gut, -wenn man die Aggregationen irgendwo separat editieren kann -und in der View nur noch drauf verweisen. Sonst muss -man soviel Copy&Paste zwischen den Views machen... +Multisite: Filter, der Hosts zeigt, die entweder selbst Summary hosts sind +oder keinen haben. -Idee: Checks, die eigentlich keine Perfdaten liefern, könnten -über eine Konfiguration künstlich perfdaten bekommen, ala -status=0, status=1 etc. Das könnte man über eine Regel -konfigurierbar machen: -fake_perfdata = [ ... ] +Idee: Checks, die eigentlich keine Perfdaten liefern, könnten über eine +Konfiguration künstlich perfdaten bekommen, ala status=0, status=1 etc. Das +könnte man über eine Regel konfigurierbar machen: fake_perfdata = [ ... ] Idee: Inventurcheck könnte gleich die Checkergebnisse berechnen @@ -513,6 +503,9 @@ Snapins: die letzten 10 Notifikationen, die letzten 10 Alerts (evtl. umschaltbar per Tabs) Acknowledgements: Ankreuzung, ob persistent oder nicht +Hm. Komisch. Im Code sehe ich, dass ich für persistent eine 0 +setzen. Trotzdem bleibt das Acknowledgment über einen Nagios- +Neustart erhalten. Tactical Overview: Man könnte folgendes machen: Die Kästchen enthalten ja Links zu Views, welche die entsprechenden Probleme

13 years, 10 months

Check_MK Git: check_mk: finally removed filesystem_levels

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: fe3e337f3f8371ef19d6a70b139b56f24930ec21 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=fe3e337f3f8371… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Fri Dec 17 08:18:41 2010 +0100 finally removed filesystem_levels --- ChangeLog | 1 + LIESMICH.zutun | 7 +++---- checks/df | 1 - checks/df_netapp | 1 - checks/df_netapp32 | 1 - checks/vms_df | 1 - modules/check_mk.py | 26 ++++++-------------------- modules/check_mk_base.py | 7 ------- 8 files changed, 10 insertions(+), 35 deletions(-) diff --git a/ChangeLog b/ChangeLog index bf4873c..ca6645f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,6 +3,7 @@ * You need a current version of Livestatus for Multisite to work! * Multisite: removed (undocumented) view parameters show_buttons and show_controls. Please use display_options instead. + * Finally removed deprecated filesystem_levels. Please use check_parameters instead. Core, Setup, etc.: * Create alias 'cmk' for check_mk in bin/ (easier typing) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index 7bba7c7..3dac633 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -112,13 +112,12 @@ versuchen und wenn das nicht geht auf v1 umschalten. Falls das nicht geht, dann kann man eben kein SNMP machen. -------------------------------------------------------------------------------- -ab 1.1.11i1 +AUFRAUMEN bis 1.2.0: -filesystem_levels wirklich rausnehmen. Hinweis, dass check_parameters verwendet -werden soll. +Gemeinsamen Code von df-Checks in df.include auslagern (get_filesystem_levels, +filesystem_default_levels, etc.) -------------------------------------------------------------------------------- --------------------------------------------------------------------------------- UNSCHOENHEITEN -------------------------------------------------------------------------------- diff --git a/checks/df b/checks/df index e8f1190..a809e35 100644 --- a/checks/df +++ b/checks/df @@ -101,4 +101,3 @@ def inventory_df(checkname, info): return inventory check_info['df'] = (check_df, "fs_%s", 1, inventory_df) -precompile_params['df'] = precompile_filesystem_levels diff --git a/checks/df_netapp b/checks/df_netapp index 201a413..cad4a3f 100644 --- a/checks/df_netapp +++ b/checks/df_netapp @@ -68,4 +68,3 @@ def inventory_df_netapp(checkname, info): check_info['df_netapp'] = (check_df_netapp, "fs_%s", 1, inventory_df_netapp) snmp_info['df_netapp'] = ("1.3.6.1.4.1.789.1.5.4.1", [ 2, 29, 30 ] ) -precompile_params['df_netapp'] = precompile_filesystem_levels diff --git a/checks/df_netapp32 b/checks/df_netapp32 index 1a6ef8c..d933a2f 100644 --- a/checks/df_netapp32 +++ b/checks/df_netapp32 @@ -82,4 +82,3 @@ def inventory_df_netapp32(checkname, info): check_info['df_netapp32'] = (check_df_netapp32, "fs_%s", 1, inventory_df_netapp32) snmp_info['df_netapp32'] = ("1.3.6.1.4.1.789.1.5.4.1", [ 2, 3, 4 ] ) -precompile_params['df_netapp32'] = precompile_filesystem_levels diff --git a/checks/vms_df b/checks/vms_df index 9ecbc63..4caf99a 100644 --- a/checks/vms_df +++ b/checks/vms_df @@ -66,4 +66,3 @@ def check_vms_df(item, params, info): return (3, "Disk %s not found" % (item,)) check_info['vms_df'] = ( check_vms_df, "fs_%s", 1, inventory_vms_df ) -precompile_params['vms_df'] = precompile_filesystem_levels diff --git a/modules/check_mk.py b/modules/check_mk.py index 9598889..0066d6a 100755 --- a/modules/check_mk.py +++ b/modules/check_mk.py @@ -343,19 +343,6 @@ def no_inventory_possible(checkname, info): sys.stderr.write("Sorry. No inventory possible for check type %s.\n" % checkname) sys.exit(3) -def lookup_filesystem_levels(host, mountpoint): - levels = service_extra_conf(host, mountpoint, filesystem_levels) - # may return 0, 1 or more answers - if len(levels) == 0: - return filesystem_default_levels - else: - return levels[0] - -def precompile_filesystem_levels(host, item, params): - if params is filesystem_default_levels: - params = lookup_filesystem_levels(host, item) - return params - # +----------------------------------------------------------------------+ # | _ _ _ _ | @@ -1180,6 +1167,12 @@ def create_nagios_config(outfile = sys.stdout, hostnames = None): if summary_service_notification_periods != []: raise MKGeneralException("summary_service_notification_periods is not longer supported. Please use extra_summary_service_conf['notification_period'] instead.") + if filesystem_levels != []: + raise MKGeneralException("filesystem_levels is not longer supported.\n" + "Please use check_parameters instead.\n" + "Please refer to documentation:\n" + " --> http://mathias-kettner.de/checkmk_check_parameters.html\n") + output_conf_header(outfile) if hostnames == None: hostnames = all_hosts_untagged + all_active_clusters() @@ -1988,8 +1981,6 @@ opt_debug = False # make sure these names are defined (even if never needed) no_inventory_possible = None -precompile_filesystem_levels = None -filesystem_default_levels = None """) # Compile in all neccessary global variables @@ -3841,11 +3832,6 @@ for name in vars_after_config: errors += 1 # Special handling for certain deprecated variables -if filesystem_levels != []: - sys.stderr.write("WARNING: filesystem_levels is deprecated and will be removed\n" - "any decade now. Please use check_parameters instead! Details can be\n" - "found at http://mathias-kettner.de/checkmk_check_parameters.html.\n") - if type(snmp_communities) == dict: sys.stderr.write("ERROR: snmp_communities cannot be a dict any more.\n") errors += 1 diff --git a/modules/check_mk_base.py b/modules/check_mk_base.py index 2207999..7a296a4 100755 --- a/modules/check_mk_base.py +++ b/modules/check_mk_base.py @@ -956,13 +956,6 @@ def nodes_of(hostname): # the filesystem and the magic number. Since the size is only known at # check time this function's result cannot be precompiled. def get_filesystem_levels(host, mountpoint, size_gb, params): - # If no explicit levels are set, we take the configuration variable - # 'filesystem_levels' into account. This can *never* happen when - # we are running precompiled since the levels are then always - # explicit. - if params is filesystem_default_levels: - params = lookup_filesystem_levels(g_hostname, mountpoint) - # If no magic factor is given, we use the neutral factor 1.0 if len(params) < 3: params = (params[0], params[1], 1.0)

13 years, 10 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits