-
Notifications
You must be signed in to change notification settings - Fork 9.7k
feat(go-vcr): include go-vcr
retryer override when configured via extra options
#44071
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…xtra options All generated service clients contain a functional option which overrides the default retryer with additional handling for `go-vcr`-specific errors. However, a handful of service clients include additional hand-written options which _also_ override the default retryer, typically addressing some nuance related to error codes unique to a given service. To ensure these service-local overrides do not overwrite the `go-vcr` handling, the extra options implementations have been modified to merge both sets of retryables.
Community GuidelinesThis comment is added to every new Pull Request to provide quick reference to how the Terraform AWS Provider is maintained. Please review the information below, and thank you for contributing to the community that keeps the provider thriving! 🚀 Voting for Prioritization
Pull Request Authors
|
7e7ae66
to
f831719
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀.
Warning This Issue has been closed, meaning that any additional comments are much easier for the maintainers to miss. Please assume that the maintainers will not see them. Ongoing conversations amongst community members are welcome, however, the issue will be locked after 30 days. Moving conversations to another venue, such as the AWS Provider forum, is recommended. If you have additional concerns, please open a new issue, referencing this one where needed. |
Description
All generated service clients contain a functional option which overrides the default retryer with additional handling for
go-vcr
-specific errors. However, a handful of service clients include additional hand-written options which also override the default retryer, typically addressing some nuance related to error codes unique to a given service. To ensure these service-local overrides do not overwrite thego-vcr
handling, the extra options implementations have been modified to merge both sets of retryables.Relations
Closes #42949
Relates #25602
Relates #42801
Output from Acceptance Testing
Example of a
go-vcr
failure which previously hung due to unintentional retries on the EC2 client. This now fails immediately once the firstgo-vcr
error is encountered (as desired).