Add multi-master for Azure Cosmos DB accounts

[pietro]

The go SDK needs to be updated for this PR but I don't know if I should do it in this PR or in another. So, I only updated the package for CosmosDB.

[katbyte]

Hi @pietro,

Thanks for the contribution, we will update the SKD in another PR

[katbyte]

SDK upgrade #2163 needs to be merged first

[katbyte]

@pietro,

This LGTM, however i think you need to update the test to check for an additional geo_location as the test currently fails:

=== RUN   TestAccAzureRMCosmosDBAccount_multiMaster
--- FAIL: TestAccAzureRMCosmosDBAccount_multiMaster (1997.78s)
	testing.go:538: Step 0 error: Check failed: 1 error(s) occurred:
		
		* Check 1/2 error: Check 8/15 error: azurerm_cosmosdb_account.test: Attribute 'geo_location.#' expected "1", got "2"
FAIL

Look forward to getting this merged once the test passes and the SDK update is merged 🙂

[katbyte]

@pietro,

I hope you don't mind but i pushed some changes to fix the test so i could merge it today 🙂

[pietro]

@katbyte thanks for getting this merged!

[tombuildsstuff]

hi @pietro

Just to let you know that this has been released as a part of v1.18 of the AzureRM Provider (the full changelog is available here). You can upgrade to this by specifying the version in the provider block (as shown below) and then running terraform init -upgrade

provider "azurerm" {
  version = "=1.18.0"
}

Thanks!

[pietro]

thanks for the heads up @tombuildsstuff!

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