tags update for azurerm_container_group

[AliAllomani]

Fixes #9612

$ make acctests SERVICE='containers' TESTARGS='-run=TestAccAzureRMContainerGroup_linuxBasicTagsUpdate'
==> Checking that code complies with gofmt requirements...
==> Checking that Custom Timeouts are used...
==> Checking that acceptance test packages are used...
TF_ACC=1 go test -v ./azurerm/internal/services/containers/tests/ -run=TestAccAzureRMContainerGroup_linuxBasicTagsUpdate -timeout 180m -ldflags="-X=github.com/terraform-providers/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccAzureRMContainerGroup_linuxBasicTagsUpdate
=== PAUSE TestAccAzureRMContainerGroup_linuxBasicTagsUpdate
=== CONT  TestAccAzureRMContainerGroup_linuxBasicTagsUpdate
--- PASS: TestAccAzureRMContainerGroup_linuxBasicTagsUpdate (216.53s)
PASS
ok      github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/containers/tests    219.088s

[tombuildsstuff]

hey @AliAllomani

Thanks for this PR :)

I've taken a look through and left some comments inline but this is off to a good start, if we can fix up the comments (and the tests pass) then we should be able to get this merged 👍

Thanks!

[tombuildsstuff]

since we're performing an update here, we can remove this ID update (and Get retrieval) here, since the ID won't change during an update

[tombuildsstuff]

where this is an update, can we parse the ID rather than retrieving these from the state:

id, err := azure.ParseAzureResourceID(d.Id())
	if err != nil {
		return err
	}
	resGroup := id.ResourceGroup
	name := id.Path["containerGroups"]

[AliAllomani]

Thanks @tombuildsstuff, updated and tested the code as per your recommendations.

[tombuildsstuff]

LGTM - thanks for pushing those changes @AliAllomani

[tombuildsstuff]

Tests pass:

[ghost]

This has been released in version 2.44.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.44.0"
}
# ... other configuration ...

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!