Trapd does not validate config against XSD

Description

Steps to reproduce:

1. Edit trapd-configuration.xml, changing snmp-trap-port to ssnmp-trap-port
2. Start OpenNMS
3. Run ss -planu sport = 162

Expected result: A UDP listener is present on port 162
Actual result: No UDP listener is present on port 162

There's a case to be made that Trapd should not even start up given the above typo in its configuration file. It follows that Trapd must not be validating its config against the XSD, which is also a bug unto itself.

Environment

https://mynms.opennms.com/Ticket/Display.html?id=5677

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Chandra Gorantla July 18, 2018 at 3:36 PM

This fix is now limited to meridian-2017.   Also since we don't want to change any configurations here,  snmp-port  needs to be specified as it is required field.  Changing description and  fix versions to reflect this change 

Chandra Gorantla July 11, 2018 at 10:08 PM

, I have added PR for meridian-2017. 

On meridian-2016 , TrapdConfiguration is auto-generated and uses castor to unmarshal xml, Do you need this fix in meridian-2016 as well ?

Chandra Gorantla July 11, 2018 at 9:57 PM
Edited

Fixed

Details

Assignee

Reporter

Components

Sprint

Affects versions

Priority

PagerDuty

Created June 29, 2018 at 4:29 PM
Updated July 31, 2018 at 6:58 PM
Resolved July 25, 2018 at 3:08 PM