Corrected Prefix Length

[johnwildes]

The maximum IP prefix length that can be reserved (according to docs.microsoft) is /28 or 16 usable addresses.

https://docs.microsoft.com/en-us/azure/virtual-network/public-ip-address-prefix#constraints

[mbfrahry]

This doc fix looks good but we'll want to update the validation for this attribute as well. The line you'll want to change is here https://github.com/terraform-providers/terraform-provider-azurerm/blob/master/azurerm/internal/services/network/resource_arm_public_ip_prefix.go#L62

[johnwildes]

@mbfrahry I updated the module, PR is here #5693

[mbfrahry]

Thanks for that @johnwildes. Can we keep the changes to a single pr though? It helps us have a single point of reference when looking at how/why a change was made. Also rebasing with master should allow the travis tests to pass.

[johnwildes]

@mbfrahry I think I understand what you're saying. I opened this PR for the documentation, and instead of updating the actual code in this PR as well, I opened a new PR. I'll remember this for the future and just submit one. I don't see a way to fix this from here.

[tombuildsstuff]

LGTM in conjunction with #5693 👍

dismissing since the other changes have been made in 5693

[tombuildsstuff]

The travis failures are unrelated (and have since been fixed in master) - this LGTM 👍

[ghost]

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