pulumi / pulumi-azure-native

Azure Native Provider
Apache License 2.0
127 stars 34 forks source link

Frontdoor Documentation Issue #2131

Open BrianRosamilia opened 1 year ago

BrianRosamilia commented 1 year ago

File: themes/default/content/registry/packages/azure-native/api-docs/network/frontdoor/_index.md

In this example, setting the location like this

location: "westus",

Will result in this error from azure

Frontdoor location must be global

I also forgot that this Azure resource identifier is for, what has become, Front Door Classic 😭 Pulumi doesn't have an updated example for standard/premium. I completely forgot about this and when I got it to work, I was reminded by the message about migrating to standard/premium.

You may consider putting a warning about this because the name for standard/premium is really not obvious (cdn.Profile).

I managed to find a really great example in the Next.js organization if you need inspiration or if anyone else gets confused about this.

https://github.com/ShanonJackson/nextjs-azure/blob/7d60a1ef681f7ad44ecd04233fd6a835213ef0ab/infra/front-door/ui-backend.ts

(Really happy so far using Pulumi btw! Thanks)

squaremo commented 1 year ago

Thamks for this feedback @BrianRosamilia, appreciate it! Labeling with kind/bug, since the advice given re location: "westus" results in malfunctioning programs.

mikhailshilkov commented 1 year ago

@thomas11 I believe this comes from this line upstream. If you confirm, could you please open an issue upstream?

thomas11 commented 1 year ago

@mikhailshilkov great find. Confirmed and opened https://github.com/Azure/azure-rest-api-specs/issues/22554.