-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Terraform fails to delete default root cloudsql user #1138
Comments
I'll try to repro this. Did you try this more than once, and did it happen every time? Did both |
Hm, your example config won't load, because:
I'm going to continue trying to reproduce, but could you provide some more details about what steps reproduce this error? |
When I ran this config (modified so that |
The issue was with running two Thanks for your help! |
Confirming bug. Going to create 1 SQL instance and getting the same issue. Affected version: 0.11.3. |
Okay, it sounds like there is a bug here, but I'm still not clear on how I can repro it. Does it happen to you every time with the config as you've got it, @klausitto? Can you share that config? |
@ndmckinley, yep.
So, each time I ran this config - I got an error mentioned by OP. |
All righty, I'll take a look. Thanks! |
Sorry! Still no repro with:
(The provider's unversioned because it's built from
I copied that config exactly as posted. You could try attaching debug logs, and maybe I can work from there? |
I suppose I can try to update to terraform 0.11.3, in case the bug is introduced in the two patch versions between yours and mine, but terraform does semver, so I certainly hope not. :) |
@ndmckinley |
That would certainly be interesting! I'm testing that theory now. |
Sorry! Still no repro.
Can you post debug logs? Maybe we can see what's going wrong from there. |
I don't know how... but I switched back to 0.11.3 and everything works fine. But I swear... I tried multiple times (not concurrently) to spin up cloudsql - and it failed. So my user wasn't created and even error in CloudSQL log appeared like "Unexpected error". |
I'm going to chalk it up to intermittent server-side errors and close the issue. |
Yes I think this is due to server-issues. My team has a CI pipeline that runs two
Just as FYI @madamkiwi and I are on the same team. |
I'm sorry to hear you're having these issues! I've asked the Cloud SQL team to look into it, but I am going to leave this issue closed as it's not an issue with Terraform. |
Generate runtimeconfig IAM resources
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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
Terraform Version
terraform 0.11.3
provider 1.6.0
Affected Resource(s)
Please list the resources as a list, for example:
Terraform Configuration Files
https://gist.github.com/madamkiwi/e0aa06d6b4479cdaf8b1cbbcb4eeeb83#file-cloud-sql-tf
Debug Output
google cloud api logs: https://gist.github.com/madamkiwi/e0aa06d6b4479cdaf8b1cbbcb4eeeb83#file-google-cloud-api-log
terraform cli logs for run 1: https://gist.github.com/madamkiwi/e0aa06d6b4479cdaf8b1cbbcb4eeeb83#file-terraform-cli-error-1-log
terraform cli logs for run 2: https://gist.github.com/madamkiwi/e0aa06d6b4479cdaf8b1cbbcb4eeeb83#file-terraform-cli-error-2-log
Unfortunately we did not run this with debug log level turned on.
Panic Output
Expected Behavior
The DB instance stands up correctly without errors.
Actual Behavior
Two
terraform apply
s were triggered at the same time to create brand new resources and as a result it failed to bring up the db instance on both. The errors are in the links above.There may be a race condition between the two but we dont really think that possible given both invocations were creating totally different cloudsql instances.
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
Important Factoids
none
References
none
cc @kkallday
The text was updated successfully, but these errors were encountered: