Skip to content
This repository has been archived by the owner on Dec 17, 2021. It is now read-only.

Remove now-broken hack and replace with proper solution #276

Merged
merged 1 commit into from
Sep 15, 2018

Conversation

jsf9k
Copy link
Collaborator

@jsf9k jsf9k commented Sep 14, 2018

Originally a hack was put in place to disable automatic Lambda retries because boto3 lacked support for this. Unfortunately this hack no longer works with the latest boto3 code because it touches internal variables that apparently have been removed or renamed. Fortunately, though, support for disabling automatic retries has been added.

Originally [a hack was put in place to disable automatic Lambda retries](boto/boto3#1104) because boto3 lacked support for this.  Unfortunately this hack no longer works with the latest boto3 code because it touches internal variables that apparently have been removed or renamed.  Fortunately, though, [support for disabling automatic retries has been added](boto/botocore#1260).
@jsf9k jsf9k requested a review from a team September 14, 2018 17:40
@jsf9k jsf9k changed the title Remove broken hack and replace with proper solution Remove now-broken hack and replace with proper solution Sep 14, 2018
@jsf9k jsf9k added the bug label Sep 14, 2018
@jsf9k
Copy link
Collaborator Author

jsf9k commented Sep 14, 2018

This is a pretty big deal, since domain-scan is broken right now if you do a fresh install or upgrade boto3 to the latest.

Copy link
Member

@IanLee1521 IanLee1521 left a comment

Choose a reason for hiding this comment

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

Looks great!

For those later looking for the reference, see: boto/boto3#1104 (comment)

@IanLee1521 IanLee1521 merged commit 02b6278 into master Sep 15, 2018
@IanLee1521 IanLee1521 deleted the bugfix/stop_retries branch September 15, 2018 23:49
@IanLee1521
Copy link
Member

See also: boto/botocore#1260

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants