Skip to content
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

aws:Adding in retry to allow ec2/DescribeRouteTables consistency #6912

Closed
wants to merge 2 commits into from

Conversation

jrnt30
Copy link
Contributor

@jrnt30 jrnt30 commented May 28, 2016

Fixes #5335 - This adds in additional retry logic for the addition of the route. As it stands today, there have been several issues with the AWS API not returning the explicit route via ec2/DescribeRouteTables for some period of time.

The previous version just retried when the creation of the route was unsuccessful.

@catsby
Copy link
Contributor

catsby commented Jun 7, 2016

Do you know what kind of error we get when the situation you describe in #5335 happens, when a route is created but the follow up DescribeRoutes doesn't have it? I'm curious if this retry logic is better suited to be in findResourceRoute

@catsby catsby added the waiting-response An issue/pull request is waiting for a response from the community label Jun 7, 2016
@jrnt30
Copy link
Contributor Author

jrnt30 commented Jun 8, 2016

There was no error per se. It was that the response was empty, which ultimately led to the issue being generated that the resource didn't have an id (and would fail on subsequent runs due to an overlap of CIDR blocks).

I had considered the findResourceRoute however as it stands it would also potentially impact the read operation (since we are just retrying until we actually get a response back). In the missing case, that would incur the retry cost. I could however add in an additional param around the retry if that would be more desirable.

@jrnt30 jrnt30 changed the title GH-5335 - Adding in retry to allow AWS to catch up via API aws:Adding in retry to allow ec2/DescribeRouteTables consistency Jul 18, 2016
@jrnt30 jrnt30 closed this Jul 18, 2016
@jrnt30 jrnt30 deleted the GH-5335-Routetable-creation branch July 18, 2016 14:03
@ghost
Copy link

ghost commented Apr 24, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug provider/aws waiting-response An issue/pull request is waiting for a response from the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Intermittent AWS Eventual Consistency Issues on CircleCI?
3 participants