Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Update docs for Cluster Autoscaler on AKS Engine #5057

Merged
merged 3 commits into from
Nov 11, 2020
Merged

Conversation

abuccts
Copy link
Member

@abuccts abuccts commented Nov 5, 2020

Update docs for Cluster Autoscaler on AKS Engine.

Update docs for Cluster Autoscaler on AKS Engine.
@abuccts abuccts mentioned this pull request Nov 5, 2020
38 tasks
Update.
@coveralls
Copy link

coveralls commented Nov 5, 2020

Coverage Status

Coverage decreased (-0.1%) to 34.223% when pulling dad7aaf on xiongyf/ca-docs into 17c7949 on master.

@suiguoxin suiguoxin requested a review from hzy46 November 5, 2020 07:11
@hzy46
Copy link
Contributor

hzy46 commented Nov 6, 2020

@abuccts Would you like to add this doc to the openpai handbook? I think we can add a question like How to enable cluster autoscaler in https://openpai.readthedocs.io/en/latest/manual/cluster-admin/installation-faqs-and-troubleshooting.html, and let users refer to https://github.com/microsoft/pai/blob/master/contrib/aks-engine/readme.md.

@abuccts
Copy link
Member Author

abuccts commented Nov 6, 2020

Currently there's no way to enable CA in an existing cluster, you need to use aks and deploy with aks-engine from scratch, so this is a standalone part under contrib. It's not applicable for most users, so it might not be suitable for faq?

@hzy46
Copy link
Contributor

hzy46 commented Nov 9, 2020

Installation FAQ is not just FAQ for the quick-start installation. It could be any questions about the installation. I think it is OK to put this in the FAQ section and tell users who are interested to install the cluster from scratch and try this feature. We may not need to put the whole content. A link to /contrib/aks-engine is enough.

Update FAQ.
@abuccts abuccts merged commit 1e9580e into master Nov 11, 2020
@abuccts abuccts deleted the xiongyf/ca-docs branch November 12, 2020 02:24
@suiguoxin suiguoxin mentioned this pull request Nov 16, 2020
39 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants