The Azure cli must be installed and cloud credentials must be manually entered and stored on the terraform host. This PR won't break anything but Azure just is not ready for prime-time. All said, this patch should not break our current ci.
How can the reviewer verify this PR?
run the Azure CI scripts
Any background context you want to provide?
As AWS does not support nested VMs, we were forced to run CI on metal instances. Azure supports nested virtualization so this may be a cheaper option
Screenshots or logs (if appropriate)
Questions:
Have you connected this PR to the issue it resolves?
yes
Does the documentation need an update?
probably
Does this add new Python dependencies?
no
Have you added unit or functional tests for this PR?
no
Does this patch update any configuration files?
no
What does this PR do?
Fixes #293 Adds option to run CI on Azure VMs.
Do you have any concerns with this PR?
The Azure cli must be installed and cloud credentials must be manually entered and stored on the terraform host. This PR won't break anything but Azure just is not ready for prime-time. All said, this patch should not break our current ci.
How can the reviewer verify this PR?
run the Azure CI scripts
Any background context you want to provide?
As AWS does not support nested VMs, we were forced to run CI on metal instances. Azure supports nested virtualization so this may be a cheaper option
Screenshots or logs (if appropriate)
Questions: