Skip to content

Commit

Permalink
azurerm_linux_web_app_slot - fix `container_registry_managed_ident…
Browse files Browse the repository at this point in the history
…ity_client_id` validation (#16149)
  • Loading branch information
jackofallops authored Mar 31, 2022
1 parent e4d2946 commit 5afbd0b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions internal/services/appservice/helpers/web_app_slot_schema.go
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import (
"github.com/Azure/azure-sdk-for-go/services/web/mgmt/2021-02-01/web"
"github.com/hashicorp/terraform-provider-azurerm/internal/sdk"
apimValidate "github.com/hashicorp/terraform-provider-azurerm/internal/services/apimanagement/validate"
msiValidate "github.com/hashicorp/terraform-provider-azurerm/internal/services/msi/validate"
"github.com/hashicorp/terraform-provider-azurerm/internal/tf/pluginsdk"
"github.com/hashicorp/terraform-provider-azurerm/internal/tf/validation"
"github.com/hashicorp/terraform-provider-azurerm/utils"
Expand Down Expand Up @@ -101,7 +100,7 @@ func SiteConfigSchemaLinuxWebAppSlot() *pluginsdk.Schema {
"container_registry_managed_identity_client_id": {
Type: pluginsdk.TypeString,
Optional: true,
ValidateFunc: msiValidate.UserAssignedIdentityID,
ValidateFunc: validation.IsUUID,
},

"default_documents": {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1158,7 +1158,7 @@ resource "azurerm_linux_web_app_slot" "test" {
}
container_registry_use_managed_identity = true
container_registry_managed_identity_client_id = azurerm_user_assigned_identity.test.id
container_registry_managed_identity_client_id = azurerm_user_assigned_identity.test.client_id
auto_swap_slot_name = "Production"
auto_heal_enabled = true
Expand Down

0 comments on commit 5afbd0b

Please sign in to comment.