Azure / terraform-azure-container-apps

A Terraform module to deploy a container app in Azure
https://github.com/Azure/terraform-azure-container-apps
37 stars 24 forks source link

Improve `output identity_ids` #43

Closed lonegunmanb closed 9 months ago

lonegunmanb commented 9 months ago

Is there an existing issue for this?

Description

Now we have:

output "identity_ids" {
  description = "The identities of the Container App."
  value       = { for name, container in azurerm_container_app.container_app : name => container.identity }
}

Actually we've exported the whole container identity block, not just id. We should change the output's map value to a wrapped object type instead of export container.identity directly to avoid extra coupling between the provider's schema and the user's code.

New or Affected Resource(s)/Data Source(s)

output "identity_ids"

Potential Terraform Configuration

No response

References

No response