-
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: azurerm_compute_skus
#28382
base: main
Are you sure you want to change the base?
Conversation
56f95b9
to
b829cf2
Compare
data.azurerm_compute_skus
azurerm_compute_skus
azurerm_compute_skus
azurerm_compute_skus
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.
Thanks for this PR @tiwood! We no longer accept untyped data sources or resources in the provider. Before we proceed with a detailed review could I please ask you to rewrite this as a typed data source? The contributor guidelines on adding a new data source can help you with this.
ff2850b
to
80be6ed
Compare
@stephybun, thanks for the heads-up. I've made the requested changes. |
Community Note
Description
This adds a new data source to query available compute SKUs for a subscription.
This is helpful to check if the desired SKU is available before the apply phase, or to check if the SKU has the desired capabilities.
Example:
PR Checklist
For example: “
resource_name_here
- description of change e.g. adding propertynew_property_name_here
”Changes to existing Resource / Data Source
Testing
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
azurerm_compute_skus
- New data source to list available Azure Compute SKUs [New data source:azurerm_compute_skus
#28382]This is a (please select all that apply):
Related Issue(s)
Note
If this PR changes meaningfully during the course of review please update the title and description as required.