search

aztfmod / rover

The rover is a docker container in charge of the deployment of the Terraform platform engineering for Azure
MIT License
173 stars 144 forks source link

Removed --subscription as not part of the az ad sp show command #180

Closed pmatthews05 closed 3 years ago

pmatthews05 commented 3 years ago

The command 'az ad sp show' does not have a --subscription parameter.

https://docs.microsoft.com/en-us/cli/azure/ad/sp?view=azure-cli-latest#az_ad_sp_show

When impersonate is set to true, as it hits the line, it fails. I have removed the subscription parameter to allow the code to flow without errors.

arnaudlh commented 3 years ago

thanks for your contribution @pmatthews05!