Skip to content

Conversation

@ShiqianTao
Copy link
Contributor

@ShiqianTao ShiqianTao commented Aug 30, 2022

[AKS] Add option Windows2019, Windows2022 to --os-sku for az aks nodepool add

Migrate from Azure CLI extension aks-preview: Azure/azure-cli-extensions#4549

Related command

az aks nodepool add --os-type Windows --os-sku Windows2019
az aks nodepool add --os-type Windows --os-sku Windows2022

Description

Customers can use this parameter to select OSSKU when creating Windows node pools.

Testing Guide

azdev test --live test_aks_nodepool_add_with_ossku_windows2022

History Notes
N/A


This checklist is used to make sure that common guidelines for a pull request are followed.

@ghost ghost requested a review from wangzelin007 August 30, 2022 10:32
@ghost ghost assigned zhoxing-ms Aug 30, 2022
@ghost ghost added this to the Aug 2022 (2022-09-06) milestone Aug 30, 2022
@ghost ghost added the Auto-Assign Auto assign by bot label Aug 30, 2022
@ghost ghost requested a review from yonzhan August 30, 2022 10:32
@ghost ghost added the AKS az aks/acs/openshift label Aug 30, 2022
@ShiqianTao
Copy link
Contributor Author

Hi @AbelHu @FumingZhang , could you help to review this PR? Thank you.

Copy link
Member

@FumingZhang FumingZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yonzhan
Copy link
Collaborator

yonzhan commented Aug 30, 2022

AKS

@zhoxing-ms zhoxing-ms changed the title [AKS] Add option Windows2019, Windows2022 to --os-sku for az aks nodepool add [AKS] az aks nodepool add: Add option Windows2019, Windows2022 to --os-sku parameter Aug 30, 2022
@ShiqianTao
Copy link
Contributor Author

Hi @zhoxing-ms Could you help to merge this PR? Thank you.

@zhoxing-ms zhoxing-ms merged commit 4a30b95 into Azure:dev Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AKS az aks/acs/openshift Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants