-
Notifications
You must be signed in to change notification settings - Fork 523
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
Add MSSQL Elastic Pool #2310
Add MSSQL Elastic Pool #2310
Conversation
d9da5a3
to
542c05d
Compare
542c05d
to
34d858a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work 🚀
@@ -0,0 +1,42 @@ | |||
package azure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this file - it doesn't look like we use it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 cheers, have deleted
…dundancy * Zone redundancy costs are additional, not instead of the existing compute cost, so this adds them as a separate cost component. * We don't need to show the extra data costs if there's no extra data usage. This only applies if provisioned. i thik we can get rid of the usage parameter for this but will do that separately.
e49428c
to
a4fd94e
Compare
This adds support for the
azurerm_mssql_elasticpool
resource and the deprecatedazurerm_sql_elasticpool
resource.It also fixes some issues with the existing
azurerm_mssql_database
resource:I think we can remove the
extra_data_storage_gb
usage param fromazurerm_mssql_database
since its value is always defined in Terraform. I haven't done that as part of this PR though.