-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Get-AzVmSize deprecation #27109
Get-AzVmSize deprecation #27109
Conversation
️✔️Az.Accounts
️✔️Az.Compute
️✔️Az.KeyVault
️✔️Az.Maintenance
️✔️Az.ManagedServiceIdentity
️✔️Az.Monitor
️✔️Az.Network
️✔️Az.OperationalInsights
️✔️Az.PrivateDns
️✔️Az.RecoveryServices
️✔️Az.Security
️✔️Az.Sql
️✔️Az.Ssh
️✔️Az.Storage
|
@grizzlytheodore by saying the api is deprecated, do you mean it cannot work at all or it is no longer under maintenance? |
@NoriZC the API still works but some of its output is outdated. |
Ok then it is recommended to deprecate this Cmdlet in next major release. Could you please add a breaking change announcement in this PR? Meanwhile an approval from Compute team memebers is needed as well. Thanks! |
@grizzlytheodore today is our code freeze date, please add a breaking change announcement in this PR, Meanwhile an approval from Compute team memebers is needed as well. If we can make it before tomorrow, we can catch the next release. |
This PR was labeled "needs-revision" because it has unresolved review comments or CI failures. |
Description
We dont want to really deprecate Get-AzVMSize, but redirect usage to Get-AzComputeResourceSku.
The REST API that Get-AzVMSize uses is deprecated: https://learn.microsoft.com/en-us/rest/api/compute/virtual-machine-sizes/list?view=rest-compute-2024-07-01&tabs=HTTP#code-try-0
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.md
and reviewed the following information:ChangeLog.md
file(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
.## Upcoming Release
header in the past tense.ChangeLog.md
if no new release is required, such as fixing test case only.