Skip to content
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

Update nodes per node pool limit from 100 to 1000 #66691

Closed
wants to merge 1 commit into from
Closed

Update nodes per node pool limit from 100 to 1000 #66691

wants to merge 1 commit into from

Conversation

cedarkuo
Copy link

This issue show that AKS now allows for node pool of max 1000 nodes, but only VMSS document mention about this update. Therefore, I would like to update this information to AKS node pool limitation document.

Which description in service quotas and limits section
@PRMerger18
Copy link
Contributor

@cedarkuo : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Jak-MS Jak-MS added the aq-pr-triaged tracking label for the PR review team label Nov 25, 2020
@dlepow
Copy link
Contributor

dlepow commented Dec 2, 2020

@zr-msft or @mlearned - Could you please review this AKS PR? Many thanks

@dlepow dlepow removed their assignment Dec 2, 2020
@ktoliver ktoliver requested review from zr-msft and removed request for dlepow December 2, 2020 22:18
@cedarkuo
Copy link
Author

cedarkuo commented Dec 4, 2020

I just found out that there has strange sign in the branch name. I want to update this branch name to cedarkuo:update-aks-nodes-per-node-limit and send other PR for it, is that ok? Thanks.

@ktoliver
Copy link
Contributor

ktoliver commented Dec 4, 2020

I just found out that there has strange sign in the branch name. I want to update this branch name to cedarkuo:update-aks-nodes-per-node-limit and send other PR for it, is that ok? Thanks.

Hi @cedarkuo - If you want to change your branch name, you can close this pull request and open a new pull request in the new branch, OK?

@cedarkuo cedarkuo closed this Dec 5, 2020
@cedarkuo cedarkuo deleted the update-aks-–nodes-per-node-limit branch December 5, 2020 02:46
@cedarkuo
Copy link
Author

cedarkuo commented Dec 5, 2020

Sure, thanks for your advise. Here is a new pull request #67189

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants