kepi / IcingaChromedStatus

Extension for Chrome which shows status of Icinga or Nagios
https://chrome.google.com/extensions/detail/imaieifkljkcdepgaeommbdgihollphm
MIT License
32 stars 14 forks source link

Plugin doesn't handle paginated status.cgi #19

Closed wgj closed 11 years ago

wgj commented 11 years ago

Hello, Nagios 3.5.0

Icinga Status 0.1.2 ID: imaieifkljkcdepgaeommbdgihollphm

I have no regex in "Icinga Status – Options".

The issue is Incinga Status only shows 14/14 hosts, however I have 1,030 hosts according to status.cgi. I have noticed that the 14 hosts in Incinga Status just so happen to be all of the hosts on page 1 of status.cgi (and I don't remember status.cgi being paginated before this new Nagios release).

If I set Incinga Status' "URL to status.cgi" to "http://URL/nagios/cgi-bin/status.cgi?host=all&limit=0&limit=0" (this is the status.cgi option for "all" hosts and services), this forces the plugin to 'unknown error', and the plugin icon shows an orange 2, as in '2 unknown issues.

Edit:

Occasionally the plugin does show mostly correct information: The 'orange 2' was actually two issues in nagios, and sometimes instead of the boiler plate "unknown error" the popup shows the expected information, but it has the wrong number of hosts (782 hosts instead of 1030) and the correct number of services (3621).

kepi commented 11 years ago

HI, thanks for reporting. I'm aware of this issue and you can find quick fix in #12 for now. I will be releasing new version in couple of weeks I hope which will solve this better.

wgj commented 11 years ago

I am already using '&limit=0'. Please reread the issue. Incinga Status is missing around ~200 hosts.

kepi commented 11 years ago

I red it of course. Problem is you need to set this directly in code, it isn't possible to set it through options page.

Or did you try it in code also and it didn't work?

wgj commented 11 years ago

I made the fix in code, and it did not work. I still have about 200 out of 1000 hosts missing, while services has the correct amount (3000 out of 3000). In my amateur opinion, this doesn't seem to be the same 'first page results' issue.

For reference, setting it in code, and setting it through options both worked.