DefaultProvisionService logs noisily for monitored service having state "N"

Description

Method DefaultProvisionService.doUpdate(MonitoredService) currently handles only statuses S (suspend polling), R (resume polling), and A (active). For a service with state N (not polled) it logs a WARN-level message Unhandled state: N. This state is not at all uncommon in the real world, so logging at this level about it is just useless noise. We should handle it equivalently to A, which is to say, just break without any logging or action.

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Jeff Gehlbach July 30, 2018 at 3:17 PM

Fixed

Details

Assignee

Reporter

Components

Sprint

Affects versions

Priority

PagerDuty

Created July 30, 2018 at 2:36 PM
Updated August 1, 2018 at 9:27 AM
Resolved August 1, 2018 at 9:27 AM