Set maxAttempts to 10 to workaround slower container startup #11
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since switching to aws-sdk-v3, we're seeing a lot of random errors like:
I thought this was a manifestation of jestjs/jest#2549 impacting the handling of retry errors due to the pervasive use of
instanceof
checks in aws-sdk-v3 similar to aws/aws-sdk-js-v3#3851. I also thought this might be related to jest fake timers interfering with aws-sdk-v3 connection timers. Eventually, I tried just increasing the number of retries and that seems to have resolved the errors. It doesn't seem worthwhile to try to find the "sweet spot" on this configuration parameter -- I imagine something less than 10 attempts would be sufficient -- but I'd rather this Just Work™ (which the value of10
seems to do).