Skip to content

Commit

Permalink
Merge pull request #1308 from aztfmod/AL-CI190822
Browse files Browse the repository at this point in the history
Update examples based on CI feedbacks
  • Loading branch information
arnaudlh authored Aug 19, 2022
2 parents ea0363d + df3fa77 commit 94d6251
Show file tree
Hide file tree
Showing 8 changed files with 38 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -74,11 +74,24 @@ diagnostics_definition = {
categories = {
log = [
#["Category name", "Diagnostics Enabled(true/false)", "Retention Enabled(true/false)", Retention_period]
["AutomaticTuning", true, false, 7],
["Blocks", true, false, 7],
["DatabaseWaitStatistics", true, false, 7],
["Deadlocks", true, false, 7],
["DevOpsOperationsAudit", true, false, 7],
["QueryStoreRuntimeStatistics", true, false, 7],
["QueryStoreWaitStatistics", true, false, 7],
["SQLInsights", true, false, 7],
["SQLSecurityAuditEvents", true, false, 7],
["Timeouts", true, false, 7],
["Errors", true, false, 7],

]
metric = [
#["Category name", "Diagnostics Enabled(true/false)", "Retention Enabled(true/false)", Retention_period]
["Basic", true, false, 7],
["InstanceAndAppAdvanced", true, false, 7],
["WorkloadManagement", true, false, 7],
]
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ mysql_flexible_server = {
name = "vks-flexible-testservers"
version = "8.0.21" #Possible values are 5.7, and 8.0.21
sku_name = "GP_Standard_D2ds_v4"

zone = 1
resource_group = {
key = "mysql_region1"
# lz_key = "" # Set the lz_key if the resource group is remote.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ mysql_flexible_server = {
name = "mysqltest-flexible-server"
version = "5.7" #Possible values are 5.7, and 8.0.21
sku_name = "GP_Standard_D2ds_v4"
zone = 1

keyvault = {
key = "mysql-re1" # (Required) when auto-generated administrator credentials needed.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,16 @@ diagnostics_definition = {
["AzureFirewallApplicationRule", true, false, 7],
["AzureFirewallNetworkRule", true, false, 7],
["AzureFirewallDnsProxy", true, false, 7],
["AZFWApplicationRule", true, false, 7],
["AZFWApplicationRuleAggregation", true, false, 7],
["AZFWDnsQuery", true, false, 7],
["AZFWFqdnResolveFailure", true, false, 7],
["AZFWIdpsSignature", true, false, 7],
["AZFWNatRule", true, false, 7],
["AZFWNatRuleAggregation", true, false, 7],
["AZFWNetworkRule", true, false, 7],
["AZFWNetworkRuleAggregation", true, false, 7],
["AZFWThreatIntel", true, false, 7],
]
metric = [
#["Category name", "Diagnostics Enabled(true/false)", "Retention Enabled(true/false)", Retention_period]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,16 @@ diagnostics_definition = {
["AzureFirewallApplicationRule", true, false, 7],
["AzureFirewallNetworkRule", true, false, 7],
["AzureFirewallDnsProxy", true, false, 7],
["AZFWApplicationRule", true, false, 7],
["AZFWApplicationRuleAggregation", true, false, 7],
["AZFWDnsQuery", true, false, 7],
["AZFWFqdnResolveFailure", true, false, 7],
["AZFWIdpsSignature", true, false, 7],
["AZFWNatRule", true, false, 7],
["AZFWNatRuleAggregation", true, false, 7],
["AZFWNetworkRule", true, false, 7],
["AZFWNetworkRuleAggregation", true, false, 7],
["AZFWThreatIntel", true, false, 7],
]
metric = [
#["Category name", "Diagnostics Enabled(true/false)", "Retention Enabled(true/false)", Retention_period]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ diagnostics_definition = {
metric = [
#["Category name", "Diagnostics Enabled(true/false)", "Retention Enabled(true/false)", Retention_period]
["Transaction", true, false, 7],
["Capacity", true, false, 7],
]
}
}
Expand Down
2 changes: 1 addition & 1 deletion modules/databases/mssql_database/database.tf
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ resource "azurerm_mssql_database" "mssqldb" {
create_mode = try(var.settings.create_mode, null)
creation_source_database_id = try(var.settings.creation_source_database_id, null)
elastic_pool_id = try(var.elastic_pool_id, null)
geo_backup_enabled = try(var.settings.geo_backup_enabled, false)
geo_backup_enabled = try(var.settings.geo_backup_enabled, null)
license_type = try(var.settings.license_type, null)
max_size_gb = try(var.settings.max_size_gb, null)
min_capacity = try(var.settings.min_capacity, null)
Expand Down
2 changes: 1 addition & 1 deletion modules/webapps/static_site/module.tf
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
resource "azurecaf_name" "static_site" {
name = var.name
resource_type = "azurerm_app_service" # FIXME: missing naming convention provider for 'azurerm_static_site' (https://github.com/aztfmod/terraform-provider-azurecaf/issues/178)
resource_type = "azurerm_static_site"
prefixes = var.global_settings.prefixes
random_length = var.global_settings.random_length
clean_input = true
Expand Down

0 comments on commit 94d6251

Please sign in to comment.