venkatapgummadi commented on PR #13302: URL: https://github.com/apache/apisix/pull/13302#issuecomment-6425563125
Thanks @hanzhenfang, good catch. I’ve pushed 5ec20cf0 to address this. The initial retry interval now has a minimum of 1 second, and I’ve updated the documentation accordingly. The default remains 40 seconds. I added tests around `start_fetch` that check scheduled delays for failures and exceptions, the backoff cap, and reset after success. They also cover the fixed default interval and a cap below the initial interval. Schema tests verify that zero is rejected and one second is accepted in both configuration modes. The watch-tuning and Kubernetes schema suites pass in Docker: 43 assertions. I also verified that the new zero-interval regression test fails against the original schema and passes with this change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
