Improve logging for EventIpcManagerDefaultImpl

Description

Sometimes, it is important to know which events are being received by certain OpenNMS daemons, and also which events are being sent by the OpenNMS daemons.

Having the log level at DEBUG for the EventIpcManagerDefaultImpl class can give you that information without the need of enabling DEBUG for several daemons.

Unfortunately, the current implementation doesn't show enough information when "sending" an event, and shows too much information (and hard to consume) when "receiving" an event.

It would be nice to homologate the logging messages by re-implementing the "toString" method for all the "Event" classes, in order to show all the fields, ignoring the null ones and the empty collections.

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Alejandro Galue June 26, 2017 at 2:29 PM

Alejandro Galue June 21, 2017 at 2:09 PM

Here is how the log messages look like currently:

sending

receiving

Fixed

Details

Assignee

Reporter

Components

Affects versions

Priority

PagerDuty

Created June 21, 2017 at 2:03 PM
Updated June 29, 2017 at 1:36 PM
Resolved June 29, 2017 at 1:36 PM