Page Sequence Monitor (PSM) doesn't appear to do retries

Description

A customer reported that retries don't seem to be working and verified myself in one of the unit tests by setting a "retry" parameter on any of the tests. Looks the retries were attempted to be implemented with the DefaultHttpMethodRetryHandler and perhaps that doesn't work as expected.

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Donald Desloge April 14, 2011 at 3:15 PM

You can now retry entire sequences with the following parameter.

1.8 commit dee7c6d5a3561ee62d870a4b9a7ece9b9bcf2833
1.9 commit e6eeffd56addf59d7cffd8188de5b045466b0dee

Benjamin Reed April 5, 2011 at 4:12 PM

This is trickier than it looks, pushing until 1.8.11 and 1.9.8 for now.

Donald Desloge April 4, 2011 at 5:11 PM

Reverted back to retry. UnitTests all had retries but the PollerConfig uses retry instead.

Donald Desloge April 4, 2011 at 4:58 PM

The retry only occurs when an IOException happens, this is the default behavior of the HttpClient. If it can't make a successful http method call it will retry.

Donald Desloge April 4, 2011 at 4:23 PM

The PageSequenceMonitor was checking for the wrong parameter for retries. Renamed retry to retries, this should be fixed now.

1.8 commit be2f0af68c71d4c0de731ebc11237242fa9778c2
1.9 commit 5cccebe67510bf8e3d5634f026478c0448d4a951

Fixed

Details

Assignee

Reporter

Labels

Components

Fix versions

Affects versions

Priority

PagerDuty

Created March 28, 2011 at 4:40 PM
Updated January 27, 2017 at 4:21 PM
Resolved April 14, 2011 at 3:15 PM