-
Notifications
You must be signed in to change notification settings - Fork 8
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
Getting rate limit from Split API #106
Comments
I'll work on this and raise a PR @davidji99 |
Hmm, this is interesting. 20 resources at the default parallelization of 10 for terraform seems quite low for a rate limit. I'm curious, what happens if you set the In any event, let me raise this issue to my Split contacts. |
I did try adding Based on their documentation, when you receive a Here there is another provider which had the same issue: okta/terraform-provider-okta#186 Let me know what your contacts say about this issue. I'm planning to work on this by the end of this week, to have a PR and then we can decide if we merge or not :) |
Hi @davidji99, I just created a PR -> #111 |
The HTTP GET use case should be good enough since the rate limiting happens when the statefile is being refreshed in the plan phase. Only in the case of updating/recreating a very large Split environment with the provider we may hit the limit if the request happen in parallel. |
Hi there,
Terraform Version
Run
terraform -v
to show the version. If you are not running the latest version of Terraform, please upgrade because your issue may have already been fixed.HerokuX Provider Version
Run
terraform -v
to show core and any provider versions. A sample output could be:Affected Resource(s)
Please list the resources as a list, for example:
If this issue appears to affect multiple resources, it may be an issue with Terraform's core, so please mention this.
Terraform Configuration Files
Debug Output
Please provider a link to a GitHub Gist containing the complete debug output: https://www.terraform.io/docs/internals/debugging.html. Please do NOT paste the debug output in the issue; just paste a link to the Gist. Please MAKE SURE to mask any sensitive values.
TBD
Panic Output
If Terraform produced a panic, please provide a link to a GitHub Gist containing the output of the
crash.log
.Expected Behavior
What should have happened?
Split resources should be created
Actual Behavior
What actually happened?
We got a rate limit and we received a 429 from Split
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
Terraform apply with more than 20 resources (this number is not precise yet, as the provider doesn't provide actual numbers)
The text was updated successfully, but these errors were encountered: