Skip to content

Commit

Permalink
fix: Updated output.tf and corrected comments in example
Browse files Browse the repository at this point in the history
  • Loading branch information
13archit committed Jun 1, 2023
1 parent e71e986 commit 5803462
Show file tree
Hide file tree
Showing 5 changed files with 17 additions and 17 deletions.
8 changes: 4 additions & 4 deletions _example/complete/example.tf
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ locals {
}

##-----------------------------------------------------------------------------
## Virtual Network module call.
## Virtual Network for which subnet will be created for private endpoint and vnet link will be created in private dns zone.
## Resource Group module call
## Resource group in which all resources will be deployed.
##-----------------------------------------------------------------------------
module "resource_group" {
source = "clouddrove/resource-group/azure"
Expand All @@ -21,8 +21,8 @@ module "resource_group" {
}

##-----------------------------------------------------------------------------
## Resource Group module call
## Resource group in which all resources will be deployed.
## Virtual Network module call.
## Virtual Network for which subnet will be created for private endpoint and vnet link will be created in private dns zone.
##-----------------------------------------------------------------------------
module "vnet" {
depends_on = [module.resource_group]
Expand Down
8 changes: 4 additions & 4 deletions _example/with_existing_dns_zone_in_diff_rg/example.tf
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ locals {
}

##-----------------------------------------------------------------------------
## Virtual Network module call.
## Virtual Network for which subnet will be created for private endpoint and vnet link will be created in private dns zone.
## Resource Group module call
## Resource group in which all resources will be deployed.
##-----------------------------------------------------------------------------
module "resource_group" {
source = "clouddrove/resource-group/azure"
Expand All @@ -21,8 +21,8 @@ module "resource_group" {
}

##-----------------------------------------------------------------------------
## Resource Group module call
## Resource group in which all resources will be deployed.
## Virtual Network module call.
## Virtual Network for which subnet will be created for private endpoint and vnet link will be created in private dns zone.
##-----------------------------------------------------------------------------
module "vnet" {
depends_on = [module.resource_group]
Expand Down
8 changes: 4 additions & 4 deletions _example/with_existing_dns_zone_in_diff_subs/example.tf
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ locals {
}

##-----------------------------------------------------------------------------
## Virtual Network module call.
## Virtual Network in which subnet will be created for private endpoint and for which vnet link will be created in private dns zone.
## Resource Group module call
## Resource group in which all resources will be deployed.
##-----------------------------------------------------------------------------
module "resource_group" {
source = "clouddrove/resource-group/azure"
Expand All @@ -22,8 +22,8 @@ module "resource_group" {
}

##-----------------------------------------------------------------------------
## Resource Group module call
## Resource group in which all resources will be deployed.
## Virtual Network module call.
## Virtual Network in which subnet will be created for private endpoint and for which vnet link will be created in private dns zone.
##-----------------------------------------------------------------------------
module "vnet" {
depends_on = [module.resource_group]
Expand Down
2 changes: 1 addition & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -284,7 +284,7 @@ resource "azurerm_private_dns_a_record" "arecord_diff-sub" {
resource "azurerm_monitor_diagnostic_setting" "acr-diag" {
count = var.enable_diagnostic && var.log_analytics_workspace_name != null || var.storage_account_name != null ? 1 : 0
name = lower("acr-${var.container_registry_config.name}-diag")
target_resource_id = azurerm_container_registry.main.*.id
target_resource_id = azurerm_container_registry.main[0].id
storage_account_id = var.storage_account_name != null ? var.storage_account_id : null
log_analytics_workspace_id = var.log_analytics_workspace_id

Expand Down
8 changes: 4 additions & 4 deletions output.tf
Original file line number Diff line number Diff line change
@@ -1,21 +1,21 @@
output "container_registry_id" {
description = "The ID of the Container Registry"
value = azurerm_container_registry.main.*.id
value = join("", azurerm_container_registry.main.*.id)
}

output "container_registry_login_server" {
description = "The URL that can be used to log into the container registry"
value = azurerm_container_registry.main.*.login_server
value = join("", azurerm_container_registry.main.*.login_server)
}

output "container_registry_admin_username" {
description = "The Username associated with the Container Registry Admin account - if the admin account is enabled."
value = var.admin_enabled == true ? azurerm_container_registry.main.*.admin_username : null
value = var.admin_enabled == true ? join("", azurerm_container_registry.main.*.admin_username) : null
}

output "container_registry_admin_password" {
description = "The Username associated with the Container Registry Admin account - if the admin account is enabled."
value = var.admin_enabled == true ? azurerm_container_registry.main.*.admin_password : null
value = var.admin_enabled == true ? join("", azurerm_container_registry.main.*.admin_password) : null
sensitive = true
}

Expand Down

0 comments on commit 5803462

Please sign in to comment.