[dev.icinga.com #2473] add modified attributes row to extinfo.cgi showing diffs to original config #921
Comments
Updated by mfriedrich on 2012-03-28 16:52:27 +00:00 temp_host|servicestatus problem is that the cgis are not compiled with support to read that. requires a change in the statusdata structs the cgis require (breaks compatibility with other cores). from include/objects.h
needs to be added to structs in onclude/statusdata.h furthermore, xdata/xsddefault.c is required to read that information from status.dat as well. (writing already happens)
given that backend support, it should be easy to get the required information. |
Updated by mfriedrich on 2012-03-28 16:52:41 +00:00
|
Updated by mfriedrich on 2012-04-03 15:02:36 +00:00
|
Updated by mfriedrich on 2012-04-23 14:37:43 +00:00
the difficult part is to parse the modified_attributes bitmask into a readable string, which is also possible so far. the rest is adaption of the cgi reading as outlined above. |
Updated by mfriedrich on 2012-04-23 17:50:16 +00:00
currently in dev/cgis, please test! |
Updated by ricardo on 2012-04-24 15:02:14 +00:00 looks good and works for me. You can set this to "resolved" |
Updated by mfriedrich on 2012-04-24 15:06:00 +00:00
okido, thanks for the code beautify! |
Updated by mfriedrich on 2014-12-08 09:27:10 +00:00
|
This issue has been migrated from Redmine: https://dev.icinga.com/issues/2473
Created by mfriedrich on 2012-03-28 16:20:23 +00:00
Assignee: mfriedrich
Status: Resolved (closed on 2012-04-24 15:06:00 +00:00)
Target Version: 1.7
Last Update: 2014-12-08 09:27:10 +00:00 (in Redmine)
modified attributes are more prior (set via webgui) than the original config, so the core will use those and override settings from original config.
normally, you don't see that in the gui. you'll only see e.g. that notifications are disabled, then rush into the config, see that they are not, wonder why, look into retention.dat, see modified attributes.
a better way of doing this would be to actually show this within the classic ui itsself in extinfo.cgi
thruk already does that here - http://thruk.org/advanced.html
Attachments
Changesets
2012-04-23 17:43:23 +00:00 by mfriedrich 1538f2d
2012-04-24 06:59:49 +00:00 by mfriedrich 47b36d1
2012-04-24 14:55:59 +00:00 by ricardo b37cbf2
2012-04-24 21:56:46 +00:00 by ricardo 68c59b7
Relations:
The text was updated successfully, but these errors were encountered: