New Resource: azurerm_api_management_api_version_set

[metacpp]

This PR is to add azurerm_api_management_api_version_set resource to better support azurerm_api_management_api resource.

[tombuildsstuff]

hi @metacpp

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 these fields are likely to be Interpolated, unfortunately we're unable to use a CustomizeDiff here; as such we'll need to remove this and instead put it in the Create method for the moment

[metacpp]

While I can move it to CreateOrUpdate method, but I still don't quite understand the principle here. Do you know why the CustomDiff in AKS a is different than what you suggested here ?

[metacpp]

The PR has been updated and tests all passed:

[tombuildsstuff]

hi @metacpp

Thanks for pushing those commits to fix some of the comments - so that we can get this shipped in v1.24 (which will be going out shortly) I'm going to push a couple of commits to resolve the remaining issues, I hope you don't mind!

Thanks!

[tombuildsstuff]

Tests pass with the updated code:

[ghost]

This has been released in version 1.24.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 = "~> 1.24.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 👉 hashibot-feedback@hashicorp.com. Thanks!