Module: check_mk
Branch: master
Commit: 1004b50236c3a64142b2764b9a81ecbce0f57a51
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=1004b50236c3a6…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Mon Jul 16 14:27:51 2018 +0200
6343 FIX Analyze parameters: Shows parameters of clustered services now
When the diagnose page to analyze the parameters of a service was opened for
a service that is assigned to a cluster host, it was not showing the
"Check origin and parameters" section. The effective parameters were not
shown.
Change-Id: Ia6fcc4a8956aac4f766394d39b3eb9b1fc5dc6ec
---
.werks/6343 | 14 ++++++++++++++
cmk_base/automations/check_mk.py | 13 ++++++++++++-
2 files changed, 26 insertions(+), 1 deletion(-)
diff --git a/.werks/6343 b/.werks/6343
new file mode 100644
index 0000000..da279a0
--- /dev/null
+++ b/.werks/6343
@@ -0,0 +1,14 @@
+Title: Analyze parameters: Shows parameters of clustered services now
+Level: 1
+Component: wato
+Class: fix
+Compatible: compat
+Edition: cre
+State: unknown
+Version: 1.6.0i1
+Date: 1531469273
+
+When the diagnose page to analyze the parameters of a service was opened for
+a service that is assigned to a cluster host, it was not showing the
+"Check origin and parameters" section. The effective parameters were not
+shown.
diff --git a/cmk_base/automations/check_mk.py b/cmk_base/automations/check_mk.py
index 96b061f..daa8b73 100644
--- a/cmk_base/automations/check_mk.py
+++ b/cmk_base/automations/check_mk.py
@@ -570,10 +570,21 @@ class AutomationAnalyseServices(Automation):
"parameters" : params,
}
+ # TODO: There is a lot of duplicated logic with discovery.py/check_table.py. Clean this
+ # whole function up.
+ if config.is_cluster(hostname):
+ autochecks = []
+ for node in config.nodes_of(hostname):
+ for check_plugin_name, item, paramstring in cmk_base.autochecks.read_autochecks_of(node):
+ descr = config.service_description(node, check_plugin_name, item)
+ if hostname == config.host_of_clustered_service(node, descr):
+ autochecks.append((check_plugin_name, item, paramstring))
+ else:
+ autochecks = cmk_base.autochecks.read_autochecks_of(hostname)
# 2. Load all autochecks of the host in question and try to find
# our service there
- for entry in cmk_base.autochecks.read_autochecks_of(hostname):
+ for entry in autochecks:
ct, item, params = entry # new format without host name
if (ct, item) not in table:
Module: check_mk
Branch: master
Commit: 4c8ec07b4b195c402dcccc2751ab6172f04947ec
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=4c8ec07b4b195c…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Fri Jul 13 14:18:48 2018 +0200
6347 FIX Add hover title to make long permission titles visible
Some permissions on the "Edit role" page have a title that is longer than the
title area which leads to stripped titles. To make them readable the stripped
titles now show the full text when hovering over it.
The texts may be improved in the future to be shorter.
Change-Id: I10393ffc705948e66068a33709731305cf414997
---
.werks/6347 | 15 +++++++++++++++
cmk/gui/forms.py | 2 +-
2 files changed, 16 insertions(+), 1 deletion(-)
diff --git a/.werks/6347 b/.werks/6347
new file mode 100644
index 0000000..5377064
--- /dev/null
+++ b/.werks/6347
@@ -0,0 +1,15 @@
+Title: Add hover title to make long permission titles visible
+Level: 1
+Component: wato
+Class: fix
+Compatible: compat
+Edition: cre
+State: unknown
+Version: 1.6.0i1
+Date: 1531484201
+
+Some permissions on the "Edit role" page have a title that is longer than the
+title area which leads to stripped titles. To make them readable the stripped
+titles now show the full text when hovering over it.
+
+The texts may be improved in the future to be shorter.
diff --git a/cmk/gui/forms.py b/cmk/gui/forms.py
index f963309..7d49b08 100644
--- a/cmk/gui/forms.py
+++ b/cmk/gui/forms.py
@@ -277,7 +277,7 @@ def section(title = None, checkbox = None, id = None, simple=False, hide = False
if legend:
html.open_td(class_=["legend", "simple" if simple else None])
if title:
- html.open_div(class_=["title", "withcheckbox" if checkbox else None])
+ html.open_div(class_=["title", "withcheckbox" if checkbox else None], title=title)
html.write(title)
html.span('.'*100, class_="dots")
html.close_div()