Azure / terraform-azurerm-avm-res-app-managedenvironment

AVM Terraform module for Container Apps Managed Environments
MIT License
5 stars 5 forks source link

[AVM Module Issue]: output from second plan are not 'clean' #37

Closed kewalaka closed 3 months ago

kewalaka commented 5 months ago

Check for previous/existing GitHub issues

Module specific issue

Issue Type?

Bug

(Optional) Module Version

0.1.2

(Optional) Correlation Id

No response

Description

when running terraform plan after a successful apply, some of the outputs need to be "changed"

     ~ output                    = jsonencode(
            {
              - properties = {
                  - customDomainConfiguration   = {
                      - certificatePassword        = null
                      - certificateValue           = null
                      - customDomainVerificationId = "REDACTED"
                      - dnsSuffix                  = null
                      - expirationDate             = null
                      - subjectName                = null
                      - thumbprint                 = null
                    }
                  - defaultDomain               = "REDACTED.australiaeast.azurecontainerapps.io"
                  - infrastructureResourceGroup = "REDACTED"
                  - peerAuthentication          = {
                      - mtls = {
                          - enabled = false
                        }
                    }
                  - staticIp                    = "10.1.1.150"
                  - vnetConfiguration           = {
                      - dockerBridgeCidr       = null
                      - infrastructureSubnetId = "/subscriptions/REDACTED/resourceGroups/REDACTED/providers/Microsoft.Network/virtualNetworks/REDACTED/subnets/REDACTED"
                      - internal               = true
                      - platformReservedCidr   = null
                      - platformReservedDnsIP  = null
                    }
                  - workloadProfiles            = [
                      - {
                          - name                = "Consumption"
                          - workloadProfileType = "Consumption"
                        },
                    ]
                }
            }

A plan after apply should report "no changes required"

kewalaka commented 5 months ago

Ref https://github.com/Azure/terraform-provider-azapi/issues/444#issuecomment-2033633513

A known issue the Azapi team are planning to address

kewalaka commented 3 months ago

fixed by #48