Add doc for useManagedIdentity #3097
Conversation
|
|
richerlariviere
changed the title
richerlariviere
changed the title
richerlariviere
changed the title
|
/assign @brendandburns |
|
@CecileRobertMichon: adding LGTM is restricted to approvers and reviewers in OWNERS files. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CecileRobertMichon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
| |networkPlugin|no|Specifies the network plugin implementation for the cluster. Valid values are:<br>`"azure"` (default), which provides an Azure native networking experience <br>`"kubenet"` for k8s software networking implementation. <br> `"flannel"` for using CoreOS Flannel <br> `"cilium"` for using the default Cilium CNI IPAM | | ||
| |networkPolicy|no|Specifies the network policy enforcement tool for the cluster (currently Linux-only). Valid values are:<br>`calico` for Calico network policy.<br>`cilium` for cilium network policy (Lin).<br>See [network policy examples](../examples/networkpolicy) for more information| | ||
| |containerRuntime|no|The container runtime to use as a backend. The default is `docker`. The other options are `clear-containers` and `containerd`| | ||
| |addons|no|Configure various Kubernetes addons configuration (currently supported: tiller, kubernetes-dashboard). See `addons` configuration below| |
There was a problem hiding this comment.
Thanks for the reordering 👍
There was a problem hiding this comment.
No problem :) I think the rest of the file could be rearranged as well (I could do it later) but for now I just wanted to focus on the issue.
What this PR does / why we need it:
useManagedIdentityfield in clusterdefinition.md. The documentation matches the Managed Identity part of this document: https://github.com/Azure/acs-engine/blob/master/docs/kubernetes/features.mdWhich issue this PR fixes:
fixes #3075
Special notes for your reviewer:
This is my first contribution to acs-engine. I hope I got the contributing guidelines right :P.
If applicable: