feat: use the Azure CLI as an auth method

[bacongobbler]

Signed-off-by: Matthew Fisher matt.fisher@microsoft.com

Reason for Change: This PR allows a user to authenticate using the Azure CLI 2.0 for development purposes. Marking this as a WIP as I haven't manually ran through with this; just experimenting with alternative auth methods.

Requirements:

• uses conventional commit messages
• includes documentation
• adds unit tests
• tested upgrade from previous version

Notes:

[acs-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bacongobbler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: tariq1890

If they are not already assigned, you can assign the PR to them by writing /assign @tariq1890 in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[bacongobbler]

I've gone through and manually tested this so I believe it's ready for review (once I rebase this).

To test, run az login, then interact with aks-engine as usual. No service principal is required for authentication, but you'll still need to create one for the cluster itself:

$ az login
$ az group create -n $RG_NAME -l $LOCATION
$ az ad sp create-for-rbac --role="Contributor" --scopes="/subscriptions/$SUBID/resourceGroups/$RG_NAME"
$ aks-engine deploy --auth-method cli ...

[CecileRobertMichon]

unit tests failed validate-dependencies

[bacongobbler]

d'oh. Thanks for catching that. Will fix

[CecileRobertMichon]

codecov is blocked because of our codecov breakage yesterday, if you rebase the PR it should run.

Is this ready for review?

[bacongobbler]

yes! I will go and rebase this now. Thanks for the heads up.

[bacongobbler]

to clarify: yes, this is ready for review.

[codecov]

Codecov Report

Merging #238 into master will decrease coverage by 0.05%.
The diff coverage is 48.48%.

@@            Coverage Diff             @@
##           master     #238      +/-   ##
==========================================
- Coverage   53.25%   53.19%   -0.06%     
==========================================
  Files          95       95              
  Lines       14246    14257      +11     
==========================================
- Hits         7586     7584       -2     
- Misses       5995     6008      +13     
  Partials      665      665

[CecileRobertMichon]

lgtm