Skip to content

Comments

fix: update throttling retry#14280

Merged
ShadowCat567 merged 6 commits intodevfrom
throttling
Oct 15, 2025
Merged

fix: update throttling retry#14280
ShadowCat567 merged 6 commits intodevfrom
throttling

Conversation

@ShadowCat567
Copy link
Contributor

@ShadowCat567 ShadowCat567 commented Oct 10, 2025

Description of changes

AWS SDK V3 no longer has retryable as a property on the errors that are thrown when throttling happens. This lead to the custom retry configuration we had set up in describeStacks not properly activating, which caused an increase in throttling errors with customers who had large/complicated applications.

Description of how you validated changes

Created a script that ran a large number (1000) of describeStacks operations and observed how it ran with and without the custom retry configuration set up.

Checklist

  • PR description included
  • yarn test passes
  • Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)
  • New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
  • Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ShadowCat567 ShadowCat567 changed the title fix: throttling fix: update throttling retry Oct 15, 2025
Copy link
Member

@svidgen svidgen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me. Thanks for digging on this!

@ShadowCat567 ShadowCat567 marked this pull request as ready for review October 15, 2025 20:30
@ShadowCat567 ShadowCat567 requested a review from a team as a code owner October 15, 2025 20:30
@ShadowCat567 ShadowCat567 merged commit f461d5f into dev Oct 15, 2025
5 checks passed
@ShadowCat567 ShadowCat567 deleted the throttling branch October 15, 2025 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants