Module: check_mk
Branch: master
Commit: 04b537e9d01d3540dd8e831e82c1ac840a7e9366
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=04b537e9d01d35…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Tue Aug 23 15:34:25 2011 +0200
View editor: warn if no column has been added
---
.bugs/49 | 10 +++++++---
web/htdocs/views.py | 5 ++++-
2 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/.bugs/49 b/.bugs/49
index ab4e084..3f75c85 100644
--- a/.bugs/49
+++ b/.bugs/49
@@ -1,11 +1,15 @@
Title: View editor: Warnung, wenn keine Spalten ausgewählt
Component: multisite
+State: done
+Class: nastiness
+Date: 2010-12-21 21:31:11
Benefit: 1
-State: open
Cost: 1
-Date: 2010-12-21 21:31:11
-Class: nastiness
+Fun: 0
Wenn der Benutzer keine einzige Spalte ausgewält hat, darf
er die View nicht speichern können. Es gab schon User, die sich
gewundert haben, dass sie nix sehen.
+
+2011-08-23 15:33:19: changed state open -> done
+Jetzt gibts eine Fehlermeldung, wenn man keine Spalte hat.
diff --git a/web/htdocs/views.py b/web/htdocs/views.py
index 74be45a..1c4f392 100644
--- a/web/htdocs/views.py
+++ b/web/htdocs/views.py
@@ -496,7 +496,7 @@ def page_edit_view():
if oldname and oldname != view["name"] and (html.req.user, oldname) in html.multisite_views:
del html.multisite_views[(html.req.user, oldname)]
save_views(html.req.user)
- return page_message_and_forward("Your view has been saved.", "edit_views.py",
+ return page_message_and_forward(_("Your view has been saved."), "edit_views.py",
"<script type='text/javascript'>top.frames[0].location.reload();</script>\n")
except MKUserError, e:
@@ -965,6 +965,9 @@ def create_view():
else:
painternames.append((pname, viewname, tooltip))
+ if len(painternames) == 0:
+ raise MKUserError("col_1", _("Please add at least one column to your view."))
+
return {
"name" : name,
"owner" : html.req.user,
Module: check_mk
Branch: master
Commit: 97c583a3c81e63bb72cc68687489db479476e039
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=97c583a3c81e63…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Tue Aug 23 00:05:26 2011 +0200
Updated bug entries
---
.bugs/112 | 7 ++++---
.bugs/183 | 13 ++++++++++---
.bugs/208 | 5 +++++
.bugs/28 | 2 +-
4 files changed, 20 insertions(+), 7 deletions(-)
diff --git a/.bugs/112 b/.bugs/112
index dc0ba39..ca50fe4 100644
--- a/.bugs/112
+++ b/.bugs/112
@@ -1,10 +1,11 @@
Title: PNP-Links in Perf-O-Meter should be intelligent
Component: multisite
-Benefit: 1
State: open
-Cost: 1
-Date: 2011-01-28 09:34:33
Class: cleanup
+Date: 2011-01-28 09:34:33
+Benefit: 1
+Cost: 1
+Fun: 0
If there is no PNP database the perfometer should not contain a hlink
to PNP. We can use the same logic as in the intelligent PNP icon.
diff --git a/.bugs/183 b/.bugs/183
index 6647e82..eed8e9a 100644
--- a/.bugs/183
+++ b/.bugs/183
@@ -1,10 +1,10 @@
-Title: bluecoat_sensors pnp links/hover menus are broken
+Title: Services with + in their names break PNP links
Component: multisite
Benefit: 2
-State: open
+State: wontfix
Cost: 3
Date: 2011-03-22 15:25:58
-Class: cleanup
+Class: bug
The bluecoat sensors check creates checks with plus signs in the name, eg +12V bus voltage.
This seems to break the links/hovers to pnp. This might also be a problem in pnp.
@@ -12,3 +12,10 @@ This seems to break the links/hovers to pnp. This might also be a problem in pnp
The rrd files are named '+12V_bus_voltage.rrd'.
Need to be analyzed.
+
+2011-08-22 16:44:03: changed state open -> wontfix
+This seems to be a bug in PNP. Even in the PNP native
+GUI services with a + in the name do not work. I've
+contacted Jörg Linge...
+~
+
diff --git a/.bugs/208 b/.bugs/208
index 6b31dae..21a3a46 100644
--- a/.bugs/208
+++ b/.bugs/208
@@ -8,3 +8,8 @@ Class: cleanup
The article about filesystems should contain all information about
trends. Also hints to the various filesystem checks.
+
+Read also bug #208. And please check out, why the Demoserver
+show so nasty steps in the trend graphs...
+~
+
diff --git a/.bugs/28 b/.bugs/28
index 2049666..b783d48 100644
--- a/.bugs/28
+++ b/.bugs/28
@@ -4,7 +4,7 @@ Benefit: 2
State: open
Cost: 4
Date: 2010-12-21 16:41:20
-Class: feature
+Class: nastiness
Dazu müsste man aber erstmal rausfinden, wie man bei Windows
dies anstellt. Es hängt ja von der Landeseinstellung ab. Vielleicht
Module: check_mk
Branch: master
Commit: d619d506f39d20f6d2aa037243eed492213d7a86
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=d619d506f39d20…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Mon Aug 22 16:06:36 2011 +0200
Removed obsolete old check_mk_agent.hp
---
doc/treasures/check_mk_agent.hp | 128 ---------------------------------------
1 files changed, 0 insertions(+), 128 deletions(-)
diff --git a/doc/treasures/check_mk_agent.hp b/doc/treasures/check_mk_agent.hp
deleted file mode 100644
index 4567fd3..0000000
--- a/doc/treasures/check_mk_agent.hp
+++ /dev/null
@@ -1,128 +0,0 @@
-#!/bin/sh
-
-# This rudimentary agent for HP has been submitted by a
-# community member and is not part of Check_MK. It might
-# be more than nothing, though
-
-export MK_LIBDIR="/usr/lib/check_mk_agent"
-export MK_CONFDIR="/etc/check_mk_agent"
-export TOP_LOG=/tmp/top_info.log
-
-# All executables in PLUGINSDIR will simply be executed and their
-# ouput appended to the output of the agent. Plugins define their own
-# sections and must output headers with '<<<' and '>>>'
-PLUGINSDIR=$MK_LIBDIR/plugins
-
-# All executables in LOCALDIR will by executabled and their
-# output inserted into the section <<<local>>>. They must
-# *not* output a section header. A local extension may output
-# zero, one or several lines. Each line must be in the form
-# EXITCODE CHECK_NAME PERFDATA CHECK_OUTPUT
-# EXITCODE is 0, 1, 2 or 3
-# CHECK_NAME will be used as Nagios's service_descriptions and
-# must be unique for this host
-# PERFDATA is optional performance data in the standard Nagios
-# format for variable=value;warn;crit;min;max. You can send
-# only one variable currently. If you do not want to send
-# performance data output a single dash instead
-# CHECK_OUTPUT will be used by Nagios as the check's output
-# EXITCODE, CHECK_NAME and PERFDATA must not contain spaces
-# or tabs. CHECK_OUTPUT may contain spaces and tabs
-LOCALDIR=$MK_LIBDIR/local
-PATH=$PATH:/usr/bin:/usr/sbin
-VMLOGDIR=/tmp/vmstat.log
-
-echo '<<<check_mk>>>'
-echo Version: 1.1.0
-
-echo '<<<df>>>'
-df -kPl | sed 's/ / - /' | grep -v ^Filesystem
-
-echo '<<<ps>>>'
-UNIX95= ps -ex -o user,args
-
-# Daten holen
-#rm -f ${VMLOGDIR} 2>/dev/null
-#vmstat 1 5 > ${VMLOGDIR}
-
-# Speicherauslastung
-#echo '<<<mem>>>'
-#echo MemTotal: ${cat ${VMLOGDIR}|egrep -v "Time|kthr"|awk '{print $4}'|tail -1}
-#echo MemFree: ${cat ${VMLOGDIR}|egrep -v "Time|kthr"|awk '{print $5}'|tail -1}
-
-## Performancecounter Platten
-#echo '<<<iostat>>>'
-#date
-#/usr/bin/iostat
-
-# Network interfaces (Link, Autoneg, Speed)
-# This requires lanscan
-if which lanscan > /dev/null
-then
- echo '<<<netif>>>'
- for eth in $(lanscan -p|cut -d " " -f1)
- do
- echo $(lanadmin -g $eth|grep "Description"|cut -d " " -f23) $(lanadmin -x $eth|cut -d " " -f22,23) on $(lanadmin -g 0|grep "Operation Status"|cut -d " " -f12|cut -d "(" -f1|sed -e "s/up/yes/")
- done
-fi
-
-# Statistik der Netzwerkgeraete (Pakete, Kollisionen, etc)
-#echo '<<<netctr_hp>>>'
-# Genauen Zeitstempel einfuegen, da Counter von Zeit abhaengen
-#date
-#/usr/bin/netstat -iw|grep -v "Name"
-
-# CPU
-#echo '<<<cpu_hp>>>'
-#cat ${VMLOGDIR}|egrep -v "Time|kthr"|awk '{print $15}'|tail -1
-
-# You need statgrab to be installed. You'll find it
-# here: http://www.i-scream.org/libstatgrab/
-if statgrab > /tmp/statgrab.$$ 2>/dev/null
- then
- for s in proc cpu net load disk page general
- do
- echo "<<<statgrab_$s>>>"
- cat /tmp/statgrab.$$ | grep "^$s\." | cut -d. -f2-99 | sed 's/ *= */ /'
- done
-
- echo '<<<statgrab_mem>>>'
- cat /tmp/statgrab.$$ | egrep "^(swap|mem)\." | sed 's/ *= */ /'
-
- rm -f /tmp/statgrab.$$
-fi
-
-# PLUGIN
-if cd $PLUGINSDIR 2>/dev/null
-then
- for skript in $(ls)
- do
- if [ -x "$skript" ] ; then
- ./$skript
- fi
- done
-fi
-
-
-echo '<<<local>>>'
-if cd $LOCALDIR 2>/dev/null
-then
- for skript in $(ls)
- do
- if [ -x "$skript" ] ; then
- ./$skript
- fi
- done
-fi
-
-# MK's Remote Plugin Executor
-if [ -e "$MK_CONFDIR/mrpe.cfg" ]
-then
- echo '<<<mrpe>>>'
- grep -Ev '^[[:space:]]*($|#)' "$MK_CONFDIR/mrpe.cfg" | \
- while read descr cmdline
- do
- OUTPUT=$($cmdline)
- echo "$? $descr $OUTPUT"
- done
-fi