Data source/loadbalancer backend address pool/backend_ip_configurations to be computed

[marioanton]

• Adding compute to attribute backend_ip_configurations
• Minor lint issues

[leewilson86]

@tombuildsstuff

This PR is a fix for a bug my colleague found with the following PR which I raised a few days ago (#4605).

It fixes this error:

Resource 'data.azurerm_lb_backend_address_pool.beap' does not have attribute 'backend_ip_configurations' for variable 'data.azurerm_lb_backend_address_pool.beap.backend_ip_configurations'

[mbfrahry]

This looks good but we should remove the Optional: true tag as well

[mbfrahry]

Any attribute that isn't used to get the resource should remove the Required/Optional: true tag in favor of Computed: true

[mariobede]

Thanks, will sort that out.

[marioanton]

Changes committed and tested.

[mbfrahry]

LGTM! Thanks for addressing that

[ghost]

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