-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
azurerm_subnet.subnet: autorest:DoErrorUnlessStatusCode 429 PUT #7153
Comments
(And sure enough that gist has my |
Hi @colemickens! This is actually a confusing error message I think: HTTP 429 is supposed to represent "Too Many Requests" - implying there is rate limiting taking place on the ARM API. My guess is that we need to implement some kind of catch-all system of allowing 429 to be caught and cause the provider to sleep. Ideally the SDK would do this - I can add it to the Riviera SDK used by some resources, but probably not to the official one, so we will need to work around this in Terraform. |
Hm. I'm not sure that's the case. At this point in the plan, it's only had to provision a few resources (vnet, subnet, route table) and it when it chokes (about 50% of the time), it's always on this subnet resource. On the other hand, when I retry and it passes the subnet, it immediately sends a much larger number of requests (because within a couple seconds of passing the subnet, it will have created 20 NICs and 20 VMs) and I never get this error even when creating a relatively large number of VMs. Do you guys currently poll on the nsg creation operation? If you create the nsg and then the subnet too quickly, I think it will cause this error. |
+1 |
@stack72 Replying here to avoid 140 char limits. Based on the error message, it looks like we are adding a rule to the SG. The SG then goes into an "Updating" state. Before the state is fully back to "Ready", we are trying to assign the SG to the subnet. The Subnet modification fails because it references a resource that is in flux. So yes, I think that polling for completion of the SecurityGroupRulesClient CreateOrUpdate operation would be helpful. The latest client should be doing this sort of polling automatically. If it's not, that's a bug on us. |
Thanks for getting back to me So I have just opened a PR that will add some polling. We are using version P. |
Closed via #7307 |
Hi @stack72 I've built from master and I'm getting the following:
This looks like it might be related - have you seen it? |
Hi @Tasquith This looks like the resource is not waiting till it is finished. I need to revisit some of the code in this area Paul |
Just to let you know I'm getting same error than @Tasquith!
|
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. |
Most of the time when deploying this terraform file, I get a 429 on one of the subnet operations.
Looks like the subnet is created before the NSG is fully finished provisioning. Retrying always allows me to proceed.
Affected Resource(s)
Terraform Configuration Files
https://gist.githubusercontent.com/anonymous/0b0ffa731d79be1097127380479e2cff/raw/65bc650a553ea88a3d8aa907af1cd0df5e823fc7/azure.tf
Debug Output
[hopefully not necessary]
Standard Output
Azure Error
This is from the body of the 429 back from Azure:
The text was updated successfully, but these errors were encountered: