azurerm_cosmosdb_sql_container doesn't call get throughput api when cosmos account is serverless

[ilmax]

Fixes #8175, this time for real :)

This is based on #8673 & #9187 /cc @yupwei68

Unfortunately I missed this one in 9187. This should now fix comsosdb SQL serverless.

make testacc TEST=./azurerm/internal/services/cosmos TESTARGS='-run=TestAccAzureRMCosmosDbSqlContainer' TESTTIMEOUT='80m'

=== RUN   TestAccAzureRMCosmosDbSqlContainer_basic
=== PAUSE TestAccAzureRMCosmosDbSqlContainer_basic
=== RUN   TestAccAzureRMCosmosDbSqlContainer_basic_serverless
=== PAUSE TestAccAzureRMCosmosDbSqlContainer_basic_serverless
=== RUN   TestAccAzureRMCosmosDbSqlContainer_complete
=== PAUSE TestAccAzureRMCosmosDbSqlContainer_complete
=== RUN   TestAccAzureRMCosmosDbSqlContainer_update
=== PAUSE TestAccAzureRMCosmosDbSqlContainer_update
=== RUN   TestAccAzureRMCosmosDbSqlContainer_autoscale
=== PAUSE TestAccAzureRMCosmosDbSqlContainer_autoscale
=== RUN   TestAccAzureRMCosmosDbSqlContainer_indexing_policy
=== PAUSE TestAccAzureRMCosmosDbSqlContainer_indexing_policy
=== CONT  TestAccAzureRMCosmosDbSqlContainer_basic
=== CONT  TestAccAzureRMCosmosDbSqlContainer_autoscale
=== CONT  TestAccAzureRMCosmosDbSqlContainer_complete
=== CONT  TestAccAzureRMCosmosDbSqlContainer_update
=== CONT  TestAccAzureRMCosmosDbSqlContainer_indexing_policy
=== CONT  TestAccAzureRMCosmosDbSqlContainer_basic_serverless
--- PASS: TestAccAzureRMCosmosDbSqlContainer_basic (1335.18s)
--- PASS: TestAccAzureRMCosmosDbSqlContainer_complete (1336.32s)
--- PASS: TestAccAzureRMCosmosDbSqlContainer_basic_serverless (1387.55s)
--- PASS: TestAccAzureRMCosmosDbSqlContainer_indexing_policy (1550.58s)
--- PASS: TestAccAzureRMCosmosDbSqlContainer_update (1564.60s)
--- PASS: TestAccAzureRMCosmosDbSqlContainer_autoscale (1662.43s)
PASS
ok      github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/cosmos      1662.540s

[ilmax]

/cc @mariussm FYI

[mariussm]

/cc @mariussm FYI

Thanks @ilmax ! :)

[jackofallops]

Thanks again @ilmax - This LGTM 👍

[jackofallops]

Tests look good:

[mariussm]

After this fix, I am seeing my sql_container objects created successfully, but on the next run, Terraform will destroy them and re-create them. This happens every time. Are you seeing the same issue @ilmax ?

[mariussm]

Seems the issue stems from having the following defined, that causes my replacement:

unique_key {
    paths = []
}

[mariussm]

Tested and works like a charm - awesome work @ilmax

[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!