-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
azurerm_frontdoor broken with recent Azure API update #8036
Comments
@WodansSon sorry for pinging you directly, as you've worked on Front Door recently I was wondering if you had any ideas/thoughts on the above? |
I have issues with Front Door too upon second application of Front Door. I get this error
The thing is that I don't change the custom domain at all. In my test case, I just changed backend pool name. Another individual who ran in to this issue updated tags. |
@bcline760 - correct, my I've had my first triage call with Microsoft for investigating it on the API side, I'll continue to provide updates here as I have them. |
Also #7498 was release that touched the Front Door. That was as recent as 2.20? Would that have any impact? I'm trying a test by using a provider version older than 2.20 to see if that change caused this. |
So, I've downgraded my provider to 2.19.0, had to mess with the state file a bit (backed it up first) but unfortunately the results are the same. This certainly appears to be an upstream change, not something specific to Terraform. |
@bcline760 I've updated the provider which has fixed my changes, PR here - #8046 it might solve your issues too |
Excellent. How quickly can this get merged? Front door is unusable right now. |
This has been released in version 2.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 = "~> 2.24.0"
}
# ... other configuration ... |
Still got the issue when trying to update AFD on azurerm version
|
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! |
Terraform (and AzureRM Provider) Version
Affected Resource(s)
azurerm_frontdoor
Terraform Configuration Files
Debug Output
Terraform Plan Output:
Azure CLI Output:
Expected Behavior
Terraform identifies that no changes are required
Actual Behavior
Azure API is responding with:
"id": "/subscriptions/0add5c8e-<snipped>/resourcegroups/frontdoor/providers/microsoft.network/frontdoors/bink-frontdoor/frontendendpoints/api-gb-bink-com"
instead of
"id": "/subscriptions/0add5c8e-<snipped>/resourcegroups/frontdoor/providers/Microsoft.Network/Frontdoors/bink-frontdoor/FrontendEndpoints/api-gb-bink-com"
Which terraform is identifying as different due to the lack of capitalisation.
Steps to Reproduce
Attempt to spin up a Front Door and Apply/Plan a second time
Important Factoids
Have raised a P1 with the Azure Support team to investigate as this is clearly a breaking API change.
The text was updated successfully, but these errors were encountered: