New resource: azurerm_cosmosdb_cassandra_table

[favoretti]

Fixes #10327

[favoretti]

$ TF_ACC=1 go test -v ./azurerm/internal/services/cosmos -timeout=1000m -run=TestAccCosmosDbCassandraTable_basic
=== RUN   TestAccCosmosDbCassandraTable_basic
=== PAUSE TestAccCosmosDbCassandraTable_basic
=== CONT  TestAccCosmosDbCassandraTable_basic
--- PASS: TestAccCosmosDbCassandraTable_basic (1056.29s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/cosmos	1057.545s

[tombuildsstuff]

hey @favoretti

Thanks for this PR - taking a look through on the whole this looks pretty good to me, my main question is whether it's worth parsing this information from a cassandra_keyspace_id field, rather than taking 3 separate strings for this information - since the Keyspace is likely to be provisioned at the same time - WDYT?

Thanks!

[favoretti]

$ TF_ACC=1 go test -v ./azurerm/internal/services/cosmos -timeout=1000m -run=TestAccCosmosDbCassandraTable_basic
=== RUN   TestAccCosmosDbCassandraTable_basic
=== PAUSE TestAccCosmosDbCassandraTable_basic
=== CONT  TestAccCosmosDbCassandraTable_basic
--- PASS: TestAccCosmosDbCassandraTable_basic (1055.85s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/cosmos	1057.009s

[djsly]

We were about to do it with azurerm_resource_group_template_deployment super excited to see this almost ready, @tombuildsstuff if you think this as a chance to be in the next release, please update the labels, I will halt some of my work! if not, azurerm_resource_group_template_deployment it will be :(

[favoretti]

@djsly 2.45 has been cut, so the best chance is it'll be in 2.46, which is next week. If I were to guess - this is good to merge.

[djsly]

Thansk a lot. I'll have high hope for the next release then !

[tombuildsstuff]

hey @favoretti

Thanks for pushing those changes - apologies for the delayed re-review here!

Taking a look through this mostly LGTM - however I believe we can also remove the resource_group_name field since this can be inferred from the Table ID, WDYT?

Thanks!

[tombuildsstuff]

LGTM - thanks for pushing those changes @favoretti

[favoretti]

$ TF_ACC=1 go test -v ./azurerm/internal/services/cosmos -timeout=1000m -run=TestAccCosmosDbCassandraTable_basic
=== RUN   TestAccCosmosDbCassandraTable_basic
=== PAUSE TestAccCosmosDbCassandraTable_basic
=== CONT  TestAccCosmosDbCassandraTable_basic
--- PASS: TestAccCosmosDbCassandraTable_basic (1051.26s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/cosmos	1052.527s

[ghost]

This has been released in version 2.46.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.46.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!