Skip to content

Conversation

@Anumita
Copy link
Contributor

@Anumita Anumita commented May 10, 2021

Description
This PR is to port the support of creating and updating AKS-managed cluster with Azure RBAC from aks-preview extension. It adds a new argument --enable-azure-rbac to az aks create command and --enable-azure-rbac and --disable-azure-rbac to the az aks update command
Testing Guide

Create an AKS-managed cluster with Azure RBAC

az aks create -g MyResourceGroup -n MyManagedCluster --enable-aad --enable-azure-rbac

Update an AKS-managed Azure AD cluster

az aks update -g MyResourceGroup -n MyManagedCluster  --enable-azure-rbac
az aks update -g MyResourceGroup -n MyManagedCluster  --disable-azure-rbac

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

@Anumita Anumita changed the title az aks create and update azure-rbac [AKS] az aks create and update azure-rbac May 10, 2021
Copy link
Contributor

@weinong weinong left a comment

Choose a reason for hiding this comment

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

LGTM

@Anumita
Copy link
Contributor Author

Anumita commented May 12, 2021

@fengzhou-msft , could you help review and merge this PR? Review is done from the AKS side

@Anumita
Copy link
Contributor Author

Anumita commented May 12, 2021

@fengzhou-msft , review is done from our side and are good to merge. I have resolved your comments as well. Could you review as well?

@fengzhou-msft fengzhou-msft merged commit f0c9779 into Azure:dev May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants