support for azurestack

[Rakesh Kumar (rakku-ms)]

test against
azure: https://github.com/rakku-ms/get-keyvault-secrets/actions/runs/292243295
azurestack: https://github.com/rakku-ms/get-keyvault-secrets/actions/runs/292243294

[Rakesh Kumar (rakku-ms)]

bganapa sarathys Usha N (@N-Usha) Ashish Ranjan (@asranja) please review
garvitarai FYI

[Ashish Ranjan (asranja)]

Few suggestions:

1. We can use az cloud show to get the current cloud environment. With this, we can remove the environment parameter passed in the yml file. We can add a method in Authorizer getActiveCloud (we can decide on naming), which will return the current set cloud. This will then be available for all azure actions.
2. Can we add some common logic here in the authorizer so that it can be available to all azure actions.

Rakesh Kumar (@rakku-ms) Since we already added suffixes while registering AzureStack cloud. Can we not get them directly using this method.

[Rakesh Kumar (rakku-ms)]

Ashish Ranjan (@asranja) please review this PR in Authorizer code: microsoft/pipelines-appservice-lib#31

When is your team planning to update the azure-actions-webclient version?

"dependencies": {
"@actions/core": "1.1.3",
"azure-actions-webclient": "1.0.1"
},

[Rakesh Kumar (rakku-ms)]

Ashish Ranjan (@asranja) this repo is using the older version of Authorizer code which may take sometime to upgrade. So I have added the 'executeAzCliCommand' function in the keyvault code itself as a workaround for now. We don't need the additional parameter 'environment' now for this keyvault action. Please review and maybe create separate branch for AzureStack just like we did for login action.

[Rakesh Kumar (rakku-ms)]

Manish Narula (@narula0781), Chandra Mudili (@chandrasekharmudili) any update regarding this PR? are you planning to merge this to master or create a separate beta branch for AzureStack?

[Deepak Sattiraju (thesattiraju)]

Rakesh Kumar (@rakku-ms) could you update your branch and fix the merge conflicts?

[Rakesh Kumar (rakku-ms)]

DS-MS I fixed all merge conflicts with the master. Can you please merge this PR?

Successful runs:
AzureStack - https://github.com/rakku-ms/get-keyvault-secrets/runs/2197860526?check_suite_focus=true
AzureCloud - https://github.com/rakku-ms/get-keyvault-secrets/runs/2197860507?check_suite_focus=true