azurerm_container_group - Add support of priority

[magodo]

Azure document of the priorty feature: https://learn.microsoft.com/en-us/azure/container-instances/container-instances-spot-containers-overview

Test

$ TF_ACC=1 go test -v -timeout=20h ./internal/services/containers -run="TestAccContainerGroup_priority"
=== RUN   TestAccContainerGroup_priority
=== PAUSE TestAccContainerGroup_priority
=== CONT  TestAccContainerGroup_priority
--- PASS: TestAccContainerGroup_priority (233.60s)
PASS
ok      github.com/hashicorp/terraform-provider-azurerm/internal/services/containers    233.614s

[katbyte]

Looks like we have a test failure"

------- Stdout: -------
=== RUN   TestAccContainerGroup_windowsComplete
=== PAUSE TestAccContainerGroup_windowsComplete
=== CONT  TestAccContainerGroup_windowsComplete
    testcase.go:113: Step 2/2 error running import: ImportStateVerify attributes not equivalent. Difference is shown below. The - symbol indicates attributes missing after import.
        
          map[string]string{
        - 	"ip_address": "20.253.106.52",
        + 	"ip_address": "",
          }
--- FAIL: TestAccContainerGroup_windowsComplete (406.01s)
FAIL

[magodo]

Hey @katbyte, I've locally run that test case with no problem:

💤  TF_ACC=1 go test -v -timeout=20h ./internal/services/containers -run="TestAccContainerGroup_windowsComplete"
=== RUN   TestAccContainerGroup_windowsComplete
=== PAUSE TestAccContainerGroup_windowsComplete
=== CONT  TestAccContainerGroup_windowsComplete
--- PASS: TestAccContainerGroup_windowsComplete (610.90s)
PASS
ok      github.com/hashicorp/terraform-provider-azurerm/internal/services/containers    610.915s

From your error above, I can't tell which property holds this map, also not from the schema definition. Is that something added up by some policy in the test sub you were running with?

From the code, I don't see how can my change affect that test case..

[catriona-m]

Thanks @magodo - I re-ran the tc tests and it looks like TestAccContainerGroup_windowsComplete is passing there now too 👍 . I did leave one minor query inline on the new test added. Thanks!

[catriona-m]

Thanks @magodo LGTM!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.