Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #2186] statusmap resizing with exclude/include button #820

Closed
icinga-migration opened this issue Dec 12, 2011 · 6 comments
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Dec 12, 2011

This issue has been migrated from Redmine: https://dev.icinga.com/issues/2186

Created by mfriedrich on 2011-12-12 20:56:28 +00:00

Assignee: ricardo
Status: Resolved (closed on 2012-04-13 15:23:28 +00:00)
Target Version: 1.7
Last Update: 2014-12-08 09:32:53 +00:00 (in Redmine)


might be useful to be included in statusmap.

Really exclude/include hostgroups from statusmap when using corresponding button, thus making it smaller and clearer.
Hi, 

I'm currently using nagios3, and I would like to propose a patch to change the behaviour of the statusmap cgi. 
It may not interest everybody, but it's quite useful in the 
environment I use (hundred of monitored hosts). 

Actually, when users select groups to exclude/include, the whole map is still displayed, with the not-selected groups just hidden, which is not quite useful for me. 
The goal of this patch is to redraw the map with only the selected groups, as if they were the only one in the configuration. 

This way, the map displayed "looks bigger", and the view is more user-friendly than usual. 

Note: 
the patch attached was developed for nagios 3.0.6 (debian version) but was tested it on the 3.2 version successfully... 

Re-Note: 
the patch also change the background color of host having critical/warning services but still being up, from green to the 'yellow' warning color. 

http://exchange.nagios.org/directory/Patches/Nagios-Core/statusmap-resizing-with-exclude-2Finclude-buttton/details

please evaluate if possible for 1.7

Attachments

Changesets

2012-01-15 01:35:00 +00:00 by ricardo 0c9bd9a

classic ui: Added statusmap resizing with exclude/include button #2186

refs: #2186

simply added and seems working fine.
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 12, 2011

Updated by mfriedrich on 2011-12-12 20:59:07 +00:00

include all orig
include_all-orig.png

include_all patched
include_all-patch.png

include parents orig
include_parents-orig.png

include parents patched
include_parents-patch.png

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jan 15, 2012

Updated by ricardo on 2012-01-15 01:37:25 +00:00

  • Assigned to set to ricardo

Added to 'dev/cgis'

please test

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jan 20, 2012

Updated by mfriedrich on 2012-01-20 00:14:48 +00:00

works as expected and allows hostgroups to be included only. makes the view a bit more flattened.

we should credit the author, if we find out his name - 'Mat' is a bit too less information.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 3, 2012

Updated by mfriedrich on 2012-04-03 15:01:50 +00:00

  • Category set to 100
  • Done % changed from 0 to 100

for me it works. but i would really like to credit 'Mat'.

could you add that to the Changelog at least?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 13, 2012

Updated by ricardo on 2012-04-13 15:23:28 +00:00

  • Status changed from Feedback to Resolved

added to Changelog

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 8, 2014

Updated by mfriedrich on 2014-12-08 09:32:53 +00:00

  • Project changed from 19 to Core, Classic UI, IDOUtils
  • Category changed from 100 to Classic UI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.