Update to 1.0 provider, latest script extensions

Azure / terraform-azurerm-computegroup

Terraform Azure RM Compute Group Module

MIT License

10 stars 37 forks source link

Update to 1.0 provider, latest script extensions #11

Closed dtzar closed 6 years ago

dtzar commented 6 years ago

Updates to the latest azurerm 1.0 provider
Updates the Windows and Linux script extensions to the latest 2.x version see this repo . 1.5 is technically the latest 1.x version (up from existing 1.2).
Updates the cmd_extension variable to execute a safe echo hello since the empty string default fails. Fixes #10
Removes unused lb_port variable and references from it in the examples

dtzar commented 6 years ago

@VaijanathB @dcaro I've done a manual terraform apply and terraform destroy against the simple and jumpbox examples and they both pass without errors. There is a problem with the kitchen test from rake e2e which should be fixed.

dtzar commented 6 years ago

This should be good to merge now.

dtzar commented 6 years ago

@dcaro I fixed the lint problem so that passes. The e2e tests have a bug with the test framework - also per our office conversation a couple weeks ago and the comment above. I filed https://github.com/Azure/terraform-azurerm-computegroup/issues/15

dcaro commented 6 years ago

@dtzar is it possible to use resources instead of module for network and loadbalancer in the test fixtures?