Skip to content
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

provider/azurerm: Adding azurerm_virtual_machine resource #5514

Merged
merged 3 commits into from
Mar 17, 2016

Conversation

stack72
Copy link
Contributor

@stack72 stack72 commented Mar 8, 2016

Adds the azurerm_virtual_machine resource. We currently have tests for basic linux and windows machines. More tests will be added later. This is the initial work that will allow us to create machines :)

@avdhoot
Copy link

avdhoot commented Mar 17, 2016

Any progress? any ETA. Please guys. :(

@stack72 stack72 changed the title [WIP] provider/azurerm: Adding azurerm_virtual_machine resource provider/azurerm: Adding azurerm_virtual_machine resource Mar 17, 2016
@jen20
Copy link
Contributor

jen20 commented Mar 17, 2016

Acceptance test run:

make testacc TEST=./builtin/providers/azurerm TESTARGS="-run TestAccAzureRMVirtualMachine"
==> Checking that code complies with gofmt requirements...
/Users/James/Code/go/bin/stringer
go generate $(go list ./... | grep -v /vendor/)
TF_ACC=1 go test ./builtin/providers/azurerm -v -run TestAccAzureRMVirtualMachine -timeout 120m
=== RUN   TestAccAzureRMVirtualMachine_basicLinuxMachine
--- PASS: TestAccAzureRMVirtualMachine_basicLinuxMachine (592.77s)
=== RUN   TestAccAzureRMVirtualMachine_basicWindowsMachine
--- PASS: TestAccAzureRMVirtualMachine_basicWindowsMachine (851.18s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/azurerm    1443.968s

@jen20
Copy link
Contributor

jen20 commented Mar 17, 2016

This LGTM @stack72! I've rebased it and merged to master and will push shortly.

@jen20 jen20 merged commit 808ce45 into master Mar 17, 2016
@jen20
Copy link
Contributor

jen20 commented Mar 17, 2016

Landed in and: 808ce45, 4db3242, cb10904.

Thanks @stack72 - this is great work on this API!

@jen20 jen20 deleted the f-arm-virtual-machine branch March 17, 2016 15:41
@lmeyemezu
Copy link

Hi,
is Azure RM virtual machine yet available?
Regards

@stack72
Copy link
Contributor Author

stack72 commented Mar 18, 2016

Hi @lmeyemezu,

This has already been merged and will be made available in Terraform 0.6.14

Paul

@lmeyemezu
Copy link

Hi @stack72 ,
Thanks.

@ghost
Copy link

ghost commented Apr 27, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants