Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

[dev.icinga.com #2210] pnp-host-extension.xml from contrib/PNP_Integration #635

Closed
icinga-migration opened this issue Dec 21, 2011 · 9 comments

Comments

@icinga-migration
Copy link

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

Created by netscraper on 2011-12-21 09:22:58 +00:00

Assignee: jmosshammer
Status: Resolved (closed on 2012-01-26 16:18:09 +00:00)
Target Version: 1.6.2
Last Update: 2012-02-14 16:40:38 +00:00 (in Redmine)


Unfortunately I get an error when I clicking on the pnp picture in the host grid. strangely the services grid are works fine with the pnp-service-extension.xml from the same relese.

This ist the error report:

;---------------------------------------------------
;Icinga Interface Error Report
;---------------------------------------------------

;Header definitions
CreationTime = 'Mittwoch, 21. Dezember 2011 10:17:25'
URL = 'https://mon.company.ch/'
Platform = 'Win32'
User-Agent = 'Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; WOW64; Trident/5.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; .NET4.0E; MAFS)'

;The following errors occured
[Error 1]
Message = '"record" ist undefiniert'
File = 'https://mon.company.ch/modules/appkit/squishloader/javascript'
Line = '285'
Time = 'Mittwoch, 21. Dezember 2011 10:17:20'
Comment = 'No comment'

;EOF


Relations:

@icinga-migration
Copy link
Author

Updated by Frankstar on 2011-12-21 12:36:50 +00:00

witch version of icinga-web did you use ?
Tarball or git master ?

if git master, pls post the git log.

cd /path/to/icinga-web
git log -1

@icinga-migration
Copy link
Author

Updated by netscraper on 2011-12-21 13:01:54 +00:00

Frankstar wrote:

witch version of icinga-web did you use ?
Tarball or git master ?

if git master, pls post the git log.

[...]

it's the tarbal: icinga-web-1.6.1.tar.gz
MD5SUM: 79d8c4391bebc738bb2363fdddbf69b4

@icinga-migration
Copy link
Author

Updated by Frankstar on 2011-12-21 13:09:17 +00:00

  • Assigned to set to jmosshammer

2 same Bugs with the 1.6.1 tarball in 4 hours, smells like a bug :D

@icinga-migration
Copy link
Author

Updated by ossmon on 2011-12-28 07:45:46 +00:00

I downloaded the icinga-web 1.6.1 yesterday and i have the same error.

@icinga-migration
Copy link
Author

Updated by ossmon on 2011-12-28 08:09:12 +00:00

Hi, i have some new information for the developper. i found a difference between the two templates, for hosts and services.

grep boolean pnp-host-extension.xml
pnp4nagios_host_image_hover
pnp4nagios_host_image_hover
pnp4nagios_host_link
pnp4nagios_host_image_hover

grep boolean pnp-service-extension.xml
pnp4nagios_service_image_hover
pnp4nagios_service_image_hover
pnp4nagios_service_link
pnp4nagios_service_link

In the host template, the name of the parameter ist booleanCon d itionField
In the service template, the name of the parameter ist booleanCon f itionField

If i replace "booleanConditionField" with "booleanConfitionField" in the host template, then the pnp integration works again.

But the optionaly display of the pnp icon (with process_perf_data parameter set or not) doesn't work anymore.

@icinga-migration
Copy link
Author

Updated by tgelf on 2011-12-28 14:08:58 +00:00

I can confirm this, also using 1.6.1. It works for services as long as "booleanConditionField" is written wrong, and it works again for hosts as soon as you rename it to something wrong or just remove it. However I'd prefer to get it working the right way ;-)

@icinga-migration
Copy link
Author

Updated by mhein on 2012-01-20 11:02:19 +00:00

  • Target Version changed from 1.6.1 to 1.7

@icinga-migration
Copy link
Author

Updated by mhein on 2012-01-26 16:18:09 +00:00

  • Status changed from New to Resolved
  • Done % changed from 0 to 100

This is fixed now: https://git.icinga.org/?p=icinga-web.git;a=commit;h=6c0bd68cdaf8ddb537064e52f661cfe8a611f8b8

I didn't seen that bug.

@icinga-migration
Copy link
Author

Updated by mhein on 2012-02-14 16:40:38 +00:00

  • Target Version changed from 1.7 to 1.6.2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant