claranet / terraform-azurerm-db-sql

Terraform module composition (feature) for Azure SQL Database (SQLServer based)
Apache License 2.0
14 stars 12 forks source link

fix: database default collation name #1

Closed kevin-fitlg closed 1 year ago

kevin-fitlg commented 1 year ago

The default value for variable databases_collation has a wrong casing. It is set to all uppercase letters and AzureRM API detects a diff when the collation is set via portal (like a restored database);

Even if you create a database via portal, it does not accept wrong casing.

image

image

Type of change

Changes proposed in this pull request

@claranet/fr-azure-reviewers

kevin-fitlg commented 1 year ago

That was quick :) Thanks team