SSLCertMonitor fails to initialise

Description

Wished to test the new SSL certificate monitor but after a restart on ONMS it is throwing the following error in eventd.log:

2013-09-23 17:37:23,060 WARN [Main] PollerConfigFactory: start: Failed to create monitor org.opennms.netmgt.poller.monitors.SSLCertMonitor for service SSLCert
org.opennms.netmgt.config.ConfigObjectRetrievalFailureException: Illegal access trying to instantiate monitor for service SSLCert with class-name org.opennms.netmgt.poller.monitors.SSLCertMonitor; nested exception is java.lang.IllegalAccessException: Class org.opennms.netmgt.config.DefaultServiceMonitorLocator can not access a member of class org.opennms.netmgt.poller.monitors.SSLCertMonitor with modifiers ""
at org.opennms.netmgt.config.DefaultServiceMonitorLocator.getServiceMonitor(DefaultServiceMonitorLocator.java:79)
at org.opennms.netmgt.config.PollerConfigManager.initializeServiceMonitors(PollerConfigManager.java:981)
at org.opennms.netmgt.config.PollerConfigManager.setUpInternalData(PollerConfigManager.java:120)
at org.opennms.netmgt.config.PollerConfigManager.<init>(PollerConfigManager.java:101)
at org.opennms.netmgt.config.PollerConfigFactory.<init>(PollerConfigFactory.java:90)
at org.opennms.netmgt.config.PollerConfigFactory.init(PollerConfigFactory.java:126)

Any ideas on how to resolve ?

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Seth Leger November 21, 2013 at 11:54 PM

Marking as fixed according to Ron's last comment.

Ron Roskens September 23, 2013 at 2:09 PM

I'd committed a fix for this on master back on Aug 30th. I've pull the change onto the 1.12 branch, so it should show up in the next release.

OpenNMS would initialize the class through reflection, and thats what the stack trace shows. Removing the constructors allows it to work, but makes it harder to run a specific type of unit test (testing for an expired certificate based on a past "current" time.)

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

PagerDuty

Created September 23, 2013 at 1:17 PM
Updated January 27, 2017 at 4:20 PM
Resolved November 21, 2013 at 11:54 PM