azurerm_api_management_subscription - Support allow_tracing property

[BlakeDickie]

Resolves #7968

• Added allow_tracing as a property on the azurerm_api_management_subscription resource.
  • Defaults to true to match the current state of the flag when subscriptions are created before this change.

[tombuildsstuff]

hey @BlakeDickie

Thanks for this PR - apologies for the delayed review here!

Taking a look through this is looking good - if we can add an acceptance test for this then this otherwise LGTM 👍

Thanks!

[tombuildsstuff]

could we add an acceptance covering setting, and enabling/disabling this field - we should be able to add this to the 'update' test to confirm this works

[BlakeDickie]

@tombuildsstuff No worries. I've added some acceptance tests for ensuring the new property defaults correctly and can be set to both true/false explicitly. It's my first time in this code base, so let me know if there's any tweaks that are needed.

[tombuildsstuff]

hey @BlakeDickie

Thanks for pushing those changes - I've taken a look through and if we can fix the one comment/issue then this otherwise LGTM 👍

Thanks!

[tombuildsstuff]

Tests pass:

[tombuildsstuff]

LGTM - thanks for pushing those changes @BlakeDickie 👍

[ghost]

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