-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(AzureRm) Adds a basic Azure Powershell module. #5910
base: master
Are you sure you want to change the base?
Conversation
I am still writing tests for this, but am curious as to whether people think it is preferable to do this as a separate module or whether I should try to integrate the functionality directly in the |
Removed Custom Deserializer for Subscription. Fixed Null Subscription Case
- Improved get_azurerm_context_info function
I think it might be a good idea to combine these, but I am not familiar with the distinction between the two clients. |
I have a WIP branch for that, but I feel like it might complicate tests and limit potential configurations more than I'd prefer. That being said, the tests for this are now complete and ready. |
Description
Adds a alternative version of the Azure module for use with Azure PowerShell, as they store their context files separately using different formats.
It still needs some tests to be written up for it.Motivation and Context
I use both the Azure PowerShell module and Azure CLI for my work, and the existing Azure module only supports Azure CLI. Originally I wanted to augment the existing module, I decided to make it a separate module instead as the context file structure is more complicated than what AzureCLI uses.
Screenshots (if appropriate):
How Has This Been Tested?
Checklist: