-
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
Update tier setting to use db-f1-micro #783
Conversation
The existing setting of "D0" caused an error for me when running `terraform apply`. This fixes the issue and allows the apply to proceed.
And thanks for the work on this integration btw! |
Ok, so there are a few things going on here.
So instead of this change, let's have two separate examples- one for a v1 database (tier D0), and another for a v2 database (tier db-f1-micro). And each one can set the region to something that's valid for that tier. We should probably also have more fleshed-out examples in general (instead of one that's just required fields), but we can do that another day if you don't feel like doing it now. How does this sound, @nathanleclaire? |
Sure, sounds good. You want me to update this PR to have the two examples you mention? |
If you don't mind, that would be great! |
The documentation was updated in #1066 to include two examples. One for 1st gen and one for the 2nd gen. Thanks for flagging this. |
Thanks @rosbo , I dropped the 🏀 on this one! |
Signed-off-by: Modular Magician <magic-modules@google.com>
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! |
The existing setting of "D0" caused an error for me when running
terraform apply
. This fixes the issue and allows the apply to proceed.ping @danawillow