[dev.icinga.com #1524] status totals are wrong when searching #630

Closed
icinga-migration opened this Issue May 16, 2011 · 11 comments

Projects

None yet

1 participant

@icinga-migration
Member

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

Created by Anonymous on 2011-05-16 11:36:35 +00:00

Assignee: ricardo
Status: Resolved (closed on 2012-04-13 19:02:49 +00:00)
Target Version: 1.7
Last Update: 2014-12-08 09:42:14 +00:00 (in Redmine)

Icinga Version: 1.10.0
OS Version: any

When a search is done, the totals in status.cgi are all wrong, all numbers are 0, regardless of what is searched for.

Attachments

Changesets

2011-12-16 21:45:50 +00:00 by ricardo 061f17a

classic ui: Fixed status totals are wrong when searching #1524

refs: #1524

Changed order of data processing. Status gets filtered and at the
same time the status totals are counted. In this way from now on
the numbers schould be correct.

2012-01-02 23:28:09 +00:00 by ricardo 6c8e32b

classic ui: Added multiple hosts/services to status.cgi GET #1981

refs: #1524
refs: #1981
refs: #2018
refs: #2173
refs: #2220

This is pretty much a rewrite of status.cgi.

some changes:
* you can define more then one object to display
** host=hostA&host=hostB&host=…
** same with host groups and service groups
* hostgroup=all now only shows which are in host groups
* host sorting is working now
* added 2 new cgi vars
** sortobject = determine if to sort host or service list (host_service_view)
** allunhandledproblems = to display ALL unhanded problems
* fixed and reorganized the links in status totals
* searching returns more then one host/service group if group name matches
* combined Host/Service view now properly supported
* code cleanup
* more little fixes and enhancements

2012-03-20 21:30:20 +00:00 by ricardo 4c10618

classic-ui: added service / host properties to links in status totals

refs: #1524

* now serviceprops and hostprops are added to links in status totals if they were set in the request
* let's see how this is doing

2012-04-05 23:26:56 +00:00 by ricardo 972e7a3

classi-ui: fixed performance issues and other problems in status.cgi

refs: #1524

* displaying of host and service group should be much faster now
  compared to the speed aftere the "big rewrite"
* fixed filtering for servicegroups
* fixed problem with wrong error message when filters are set

2012-04-13 18:48:07 +00:00 by ricardo 7b9ec5b

classic-ui: updated Changelog CHANGES section

refs: #1524
refs: #1981
refs: #2018
refs: #2117
refs: #2119
refs: #2145
refs: #2198
refs: #2343
refs: #2359
refs: #2373
refs: #2421

hope the "changes" section for classic ui isn't tooo long

Relations:

Member

Updated by mfriedrich on 2011-05-24 14:26:53 +00:00

  • Status changed from New to Feedback
  • Target Version changed from 1.4 to 1.5
Member

Updated by mfriedrich on 2011-07-16 11:39:36 +00:00

  • Target Version changed from 1.5 to 1.6

this is introduced when using host filters (precompiled regexes) and the asterisk for searching. search is returning only the first result by default, where the counting of the host and service totals is correct.

in order to fight the error, one might rewrite the search completely, and even if using regex, doing the counts for the status totals right in place (global variable instead of extra counting!).

this relates to #844

Member

Updated by mfriedrich on 2011-10-20 09:37:00 +00:00

  • Target Version deleted 1.6
Member

Updated by ricardo on 2011-11-05 16:40:14 +00:00

  • Estimated Hours deleted 2

Just wanted to know what these counters should represent. because sometimes they counted ALL hosts and services and sometimes just the ones which are displayed.

As we now have the TAC header I think it would make sense just to display the counts for the hosts and services which are displayed.

What do you think?

Member

Updated by mfriedrich on 2011-11-15 09:42:02 +00:00

basically, those counts do the following

  • add urls to different states
    • host states with servicestatustype=29 (up=2, down=4, unreachable=8, pending=1)
    • service states with hoststatustype=15 (ok=2, warning=4, unknown=8, critical=16, pending=1)
    • all problems
      • hosts states with servicestatustype=29 (down+unreachable+pending = 4+8 = 12)
      • service states with hoststatustype=15 (warning+unknown+critical = 4+8+16 = 28)
    • all types
      • hosts states with servicestatustype=29
      • service states with hoststatustype=15

the counts mostly did not happen correctly due to the navbarsearch filter not being honored when counting. having rewritten that, i think they would remain "self corrected" now by fixing #844

Member

Updated by mfriedrich on 2011-11-15 09:45:16 +00:00

  • File added classicui_status_counts_tacheader.png
  • File added classicui_status_counts_statusheader.png

ok, apparently they are not matching completely, see the differences on the unreachable count...

classicui_status_counts_tacheader.png

classicui_status_counts_statusheader.png

Member

Updated by ricardo on 2011-12-16 21:52:17 +00:00

  • Subject changed from When searching, totals in page is wrong to status totals are wrong when searching
  • Assigned to set to ricardo
  • Target Version set to 1.7
  • Done % changed from 0 to 90

in current rbartels/cgi

please test!!!

can't guarantee that all display_styles are displayed correctly, so please TEST!!!

Member

Updated by mfriedrich on 2012-04-03 13:28:41 +00:00

from my perspective this looks find for now. but still, that anag discussion thingy should match it too.

status totals should contain

  • the overall totals
  • the ones filtered by the view

as now implemented.

Member

Updated by ricardo on 2012-04-13 14:25:02 +00:00

  • File added statustotal_all_hostgroups.png

Hope people understand the way it works now.

For example: If you display all hostgroups, it only shows the status for all hostgroups, not for host's which aren't in any hostgroup. The same with servicegroups.

example for all hostgroups (4 hosts), filtered for services in unknown state
statustotal_all_hostgroups.png

Member

Updated by ricardo on 2012-04-13 19:02:49 +00:00

  • Status changed from Feedback to Resolved

Added note to changes in Changelog

Member

Updated by mfriedrich on 2014-12-08 09:42:14 +00:00

  • Project changed from 19 to Core, Classic UI, IDOUtils
  • Category changed from 54 to Classic UI
  • Icinga Version set to 1
  • OS Version set to any
@icinga-migration icinga-migration added this to the 1.7 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment