-
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
New Data Source/Resource: azurerm_cdn_frontdoor_profile
#17061
Conversation
5d0b53c
to
915e1ce
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Just a quick test fix
internal/services/cdn/cdn_frontdoor_profile_data_source_test.go
Outdated
Show resolved
Hide resolved
Splitting out from #16671 Co-authored-by: Jeffrey Cline <[email protected]>
Co-authored-by: Jeffrey Cline <[email protected]>
Co-authored-by: Matthew Frahry <[email protected]>
bed7de0
to
0f66652
Compare
This functionality has been released in v3.9.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Having spent a decent chunk of time trying to review #16671, I've come to the conclusion that it's too big to review in it's current form - so I'll be pulling this apart and fixing it up at the same time.
This PR splits out the
azurerm_cdn_frontdoor_profile
Resource and adds a matching Data Source, since this is likely to be commonly references. The primary difference between this resource in this PR and in #16671 is thatsku_name
is now required andcdn_frontdoor_id
(which is a UUID) is now namedresource_guid
(which matches the naming of this field in other resources).Other PR's will build on this - as such this includes all of the SDK clients for simplicity - however those'll follow once this is in.