Skip to content

Conversation

@gtxistxgao
Copy link
Contributor

@gtxistxgao gtxistxgao commented Sep 26, 2020

Description

Support add Spot node pool as part of GA work item.

https://docs.microsoft.com/en-us/azure/aks/spot-node-pool

Testing Guide

az ask nodepool add -g resourceGroup --cluster-name clusterName -n spotPoolName --priority Spot --spot-max-price 88.88888

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


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

@gtxistxgao gtxistxgao requested a review from arrownj as a code owner September 26, 2020 02:11
@gtxistxgao gtxistxgao changed the title [AKS] Suppot add Spot node pool [AKS] Support add Spot node pool Sep 26, 2020
@arrownj arrownj self-assigned this Sep 27, 2020
@yungezz yungezz added the AKS az aks/acs/openshift label Sep 27, 2020
@yungezz yungezz added this to the S176 milestone Sep 27, 2020
@yungezz
Copy link
Member

yungezz commented Sep 27, 2020

hi @gtxistxgao which release you'd like this one to be in? From comments, this is GA item, seems has specific timeline

@arrownj
Copy link
Contributor

arrownj commented Sep 29, 2020

Hi @gtxistxgao , this is just a reminder that the CC date for next release is 9/30. Does this PR need to catch this release ?

@gtxistxgao
Copy link
Contributor Author

Hi @gtxistxgao , this is just a reminder that the CC date for next release is 9/30. Does this PR need to catch this release ?

It's not that urgent as RP's change not ready yet. Sorry for the confusion.

@gtxistxgao
Copy link
Contributor Author

@arrownj the RP change is in. Please merge this one so we can catch up next release.

@gtxistxgao
Copy link
Contributor Author

@arrownj the RP change is in. Please merge this one so we can catch up next release.

@arrownj ping... can you help merge it please.

@Azure Azure deleted a comment from yonzhan Oct 6, 2020
@yungezz yungezz merged commit 3c9b565 into Azure:dev Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AKS az aks/acs/openshift

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants