gatekeeper icon indicating copy to clipboard operation
gatekeeper copied to clipboard

feat: Add retry limit when fetching/listing expectations in ready tracker

Open David-Jaeyoon-Lee opened this issue 7 months ago • 2 comments

Describe the solution you'd like In order for #3308 to take affect, we need to add support for users to set a retry limit when fetching expectations. This involves creating a retryPredicate that sets the number of retries based on a user defined limit/count. Currently since the retryPredicate always retries, the crashOnFailureFetchingExpectations is moot and hidden (replaced with a constant false value).

Anything else you would like to add: In addition to adding support for retry limit, once this is implemented, we need to expose the crashOnFailureFetchingExpectations flag back to the user (delete constant false value and uncomment crashOnFailureFetchingExpectations flag) as well as change any logic that uses that flag as a constant (likely need to dereference the variable)

David-Jaeyoon-Lee avatar Jul 24 '24 19:07 David-Jaeyoon-Lee