Module: check_mk
Branch: master
Commit: 90b3eb190de3332565d161eacb865886f7d1030b
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=90b3eb190de333…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Tue Apr 17 16:55:20 2012 +0200
Updated bug entries #0691
---
.bugs/691 | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/.bugs/691 b/.bugs/691
index 0f37013..bc75964 100644
--- a/.bugs/691
+++ b/.bugs/691
@@ -1,9 +1,12 @@
Title: Remove/Secure eval() on html.var()
Component: wato
-State: open
+Class: nastiness
+State: done
Date: 2012-04-11 11:39:35
Targetversion: 1.2.0
-Class: nastiness
It is VERY insecure and VERY easy to do bad things at these places. They must be
secured/removed in some way.
+
+2012-04-17 16:55:16: changed state open -> done
+Has been fixed for direct reading from HTML parameters
Module: check_mk
Branch: master
Commit: 2b3ed0d9b49d3123a53f1560a1649df9d9e74ac4
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=2b3ed0d9b49d31…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Tue Apr 17 15:59:36 2012 +0200
Updated bug entries #0601, #0659, #0654
---
.bugs/601 | 7 +++++--
.bugs/654 | 7 +++++--
.bugs/659 | 7 +++++--
3 files changed, 15 insertions(+), 6 deletions(-)
diff --git a/.bugs/601 b/.bugs/601
index 6f86b24..f074e95 100644
--- a/.bugs/601
+++ b/.bugs/601
@@ -1,9 +1,9 @@
Title: Checkboxes in views and brower back in disharmony
Component: multisite
-State: open
+Class: bug
+State: works4me
Date: 2012-01-17 11:43:07
Targetversion: 1.2.0
-Class: bug
Tested on IE8, but might be browser independent.
The following steps produce the problem:
@@ -18,3 +18,6 @@ The following steps produce the problem:
5. Now again select a command.
-> Now the command will be applied to *all*
services instead of the one.
+
+2012-04-17 15:15:51: changed state open -> works4me
+Maybe already fixed. Tested in Firefox, IE7-9
diff --git a/.bugs/654 b/.bugs/654
index 770b51a..64cd990 100644
--- a/.bugs/654
+++ b/.bugs/654
@@ -1,10 +1,13 @@
Title: View editor mixes up views on editing
Component: multisite
-State: open
+Class: bug
+State: works4me
Date: 2012-02-15 08:48:38
Targetversion: future
-Class: bug
We had a strange problem. We had a view named "offline_locations" using the "All services"
table. Then we added a "offline_locations1" view which used the "All hosts" table.
BUt after saving the second view had the same options as the first view.
+
+2012-04-17 15:41:40: changed state open -> works4me
+Must have been a user problem. I can not reproduce such problems.
diff --git a/.bugs/659 b/.bugs/659
index 011f48a..238c7da 100644
--- a/.bugs/659
+++ b/.bugs/659
@@ -1,11 +1,14 @@
Title: Host validation error does invalidate the _transid of the current page
Component: wato
-State: open
+Class: bug
+State: works4me
Date: 2012-02-17 15:42:32
Targetversion: future
-Class: bug
If a host validation error occurs on saving a host, the user needs to return
to the folder an re-edit the host from start.
It seems that the transaction id of the current page gets invalid, and a
subsequent save on the same page wont work
+
+2012-04-17 15:22:17: changed state open -> works4me
+Seems that this has already been fixed. It works foor me over here
Module: check_mk
Branch: master
Commit: b75dabc039dfcc64fdffc114128c5e6f4d6a3eff
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=b75dabc039dfcc…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Tue Apr 17 16:13:24 2012 +0200
Fixed js eror in IE7 on WATO host edit page
---
.bugs/653 | 8 +++++-
web/htdocs/js/wato.js | 51 +++++++++++++++++++++++++-----------------------
2 files changed, 33 insertions(+), 26 deletions(-)
diff --git a/.bugs/653 b/.bugs/653
index 453180e..c09d1d4 100644
--- a/.bugs/653
+++ b/.bugs/653
@@ -1,10 +1,14 @@
Title: bulk inventory in omd shared mode
Component: wato
-State: open
+Class: bug
+State: done
Date: 2012-02-14 16:06:33
Targetversion: future
-Class: bug
At the moment the message about modifying the /etc/sudoers file is shown
on every cmk --automation problem during inventory. This is misleading.
The message should only occur when the entries are really missing in /etc/sudoers
+
+2012-04-17 16:00:21: changed state open -> done
+Has just been fixed. The sudo hint message is only shown when the sudo command fails "sudo:
+" in error message.
diff --git a/web/htdocs/js/wato.js b/web/htdocs/js/wato.js
index 42a9eb0..542d1d1 100644
--- a/web/htdocs/js/wato.js
+++ b/web/htdocs/js/wato.js
@@ -139,32 +139,35 @@ function wato_fix_visibility() {
}
var oTr = document.getElementById("attr_" + attrname);
- oTr.style.display = display;
-
- // Prepare current visibility information which is used
- // within the attribut validation in wato
- // Hidden attributes are not validated at all
- if ( !document.getElementById("attr_display_" + attrname) ){
- var newInput = document.createElement("input");
- newInput.name = "attr_display_" + attrname;
- newInput.id = "attr_display_" + attrname;
- newInput.type = "hidden";
- newInput.className = "text";
- oTr.appendChild(newInput);
- }
- var display_info = document.getElementById("attr_display_" + attrname)
- if ( display == "none" ) {
- display_info.value = "0";
- }else{
- display_info.value = "1";
- }
-
- // There is at least one item in this topic -> show it
- var topic = oTr.parentNode.parentNode.parentNode.id.substr(21);
- if( display == "" ){
- var index = hide_topics.indexOf(topic);
- if( index != -1 )
- delete hide_topics[index];
+ if(oTr) {
+ oTr.style.display = display;
+
+ // Prepare current visibility information which is used
+ // within the attribut validation in wato
+ // Hidden attributes are not validated at all
+ oAttrDisp = document.getElementById("attr_display_" + attrname);
+ if (!oAttrDisp){
+ var newInput = document.createElement("input");
+ newInput.name = "attr_display_" + attrname;
+ newInput.id = "attr_display_" + attrname;
+ newInput.type = "hidden";
+ newInput.className = "text";
+ oTr.appendChild(newInput);
+ }
+ if ( display == "none" ) {
+ oAttrDisp.value = "0";
+ } else {
+ oAttrDisp.value = "1";
+ }
+ oAttrDisp = null;
+
+ // There is at least one item in this topic -> show it
+ var topic = oTr.parentNode.parentNode.parentNode.id.substr(21);
+ if( display == "" ){
+ var index = hide_topics.indexOf(topic);
+ if( index != -1 )
+ delete hide_topics[index];
+ }
}
}
for (var item in volatile_topics){
Module: check_mk
Branch: master
Commit: 4d04510157dfde25bb5ba1c32a4978f1dcd74120
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=4d04510157dfde…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Tue Apr 17 16:50:54 2012 +0200
Do not put users in dummy contact group check_mk
---
.bugs/675 | 6 ++++--
ChangeLog | 2 ++
modules/check_mk.py | 16 +++++++++-------
3 files changed, 15 insertions(+), 9 deletions(-)
diff --git a/.bugs/675 b/.bugs/675
index ab69d9d..99bb7bb 100644
--- a/.bugs/675
+++ b/.bugs/675
@@ -1,9 +1,9 @@
Title: User is added to default contact group check_mk, but should not
Component: core
-State: open
+Class: bug
+State: done
Date: 2012-04-10 16:30:42
Targetversion: 1.2.0
-Class: bug
We did:
- Create a contact group "alle"
@@ -16,3 +16,5 @@ Happened:
- BUT: hh and omdadmin are members of check_mk automatically
(and should not)
- and thus hh receives notifications, but should not
+
+2012-04-17 16:50:38: changed state open -> done
diff --git a/ChangeLog b/ChangeLog
index d5444b3..2d46bf6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -5,6 +5,8 @@
* FIX: Automation try-inventory: Fixed problem on where checks which
produce equal service descriptions could lead to invalid inventory
results on cluster hosts.
+ * FIX: do not create contacts if they won't be assigned to any host
+ or service. Do *not* assign to dummy catch-all group "check_mk".
WATO:
* Added new permission "move hosts" to allow/deny moving of hosts in WATO
diff --git a/modules/check_mk.py b/modules/check_mk.py
index 535674d..3f03bd2 100755
--- a/modules/check_mk.py
+++ b/modules/check_mk.py
@@ -1877,6 +1877,15 @@ def create_nagios_config_contacts(outfile):
cnames.sort()
for cname in cnames:
contact = contacts[cname]
+ # If the contact is in no contact group or all of the contact groups
+ # of the contact have neither hosts nor services assigned - in other
+ # words if the contact is not assigned to any host or service, then
+ # we do not create this contact in Nagios. It's useless and wil produce
+ # warnings.
+ cgrs = [ cgr for cgr in contact.get("contactgroups", []) if cgr in contactgroups_to_define ]
+ if not cgrs:
+ continue
+
outfile.write("define contact {\n contact_name\t\t\t%s\n" % cname)
if "alias" in contact:
outfile.write(" alias\t\t\t\t%s\n" % contact["alias"].encode("utf-8"))
@@ -1891,13 +1900,6 @@ def create_nagios_config_contacts(outfile):
outfile.write(" %s_notification_period\t%s\n" % (what, contact.get("notification_period", "24X7")))
outfile.write(" %s_notification_commands\tcheck-mk-notify\n" % what)
- # Refer only to those contact groups that actually have objects assigned to. Otherwise
- # we will run into problems since Nagios does not allow contact groups that
- # are not assigned to at least one host or service?
- cgrs = [ cgr for cgr in contact.get("contactgroups", []) if cgr in contactgroups_to_define ]
- # Use dummy group if none is set. Nagios will not start otherwise
- if not cgrs:
- cgrs = [ "check_mk" ]
outfile.write(" contactgroups\t\t\t%s\n" % ", ".join(cgrs))
outfile.write("}\n\n")