provider/azurerm: Add example of VNET to VNET Peering #14338

anniehedgpeth · 2017-05-10T02:19:48Z

CI:

This example provisions and deploys the following resources into an Azure subscription:

Resource Group
2 VNETs w/1 subnet ea
VNET peering for each VNET

CI process:

terraform get, validate, plan to output, apply output, show
validate with azure CLI
terraform destroy

…t script, added mac specific deployment for local testing

…hmagnitude/terraform into topic-101-vm-from-user-image

* initial commit - 101-vm-from-user-image * added tfvars and info for README * added CI config; added sane defaults for variables; updated deployment script, added mac specific deployment for local testing * prep for PR

azure-vm-from-user-image

* initial commit * need to change lb_rule & nic * deploys locally * updated README * updated travis and deploy scripts for Hari's repo * renamed deploy script * clean up * prep for PR * updated readme * fixing conflict in .travis.yml

…topic-201-vnet-to-vnet-peering

…arijayms/terraform into topic-201-vnet-to-vnet-peering

tombuildsstuff

Hi @anniehedgpeth

Thanks for this PR - I've reviewed and left some comments inline, but it otherwise looks good :)

Thanks!

tombuildsstuff · 2017-05-10T21:21:19Z

examples/azure-cdn-with-storage-account/provider.tf.example

+#   client_id       = "REPLACE-WITH-YOUR-CLIENT-ID"
+#   client_secret   = "REPLACE-WITH-YOUR-CLIENT-SECRET"
+#   tenant_id       = "REPLACE-WITH-YOUR-TENANT-ID"
+# }


I believe this file can be removed since it's integrated into the main.tf?

It's gone now. :)

tombuildsstuff · 2017-05-10T21:21:43Z

examples/azure-cdn-with-storage-account/terraform.tfvars

+# hostname       = "myvm"
+# dns_name       = "mydnsname"
+# location       = "southcentralus"
+# admin_password = "T3rr@f0rmP@ssword"


I believe we can remove this file?

It's gone now.

tombuildsstuff · 2017-05-10T21:31:51Z

examples/azure-vnet-to-vnet-peering/README.md

+## outputs.tf
+This data is outputted when `terraform apply` is called, and can be queried using the `terraform output` command.
+
+## provider.tf


I believe this file can be removed since the contents are in the main.tf?

I changed the README; let me know what you think.

stack72 · 2017-05-11T16:40:00Z

LGTM - thanks for the followup work here :)

ghost · 2020-04-08T02:33:21Z

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.

anniehedgpeth and others added 30 commits April 17, 2017 19:07

initial commit - 101-vm-from-user-image

9f0d990

changed branch name

6f577a8

not deploying - storage problems

611d70a

provisions vm but image not properly prepared

fdff23a

storage not correct

0dac5ec

provisions properly

81e2559

changed main.tf to azuredeploy.tf

7dc8150

added tfvars and info for README

5c079fa

tfvars ignored and corrected file ext

c86ef6f

added CI config; added sane defaults for variables; updated deploymen…

534b236

…t script, added mac specific deployment for local testing

deploy.sh to be executable

0cf888b

executable deploy files

68f0003

added CI files; changed vars

c898510

merge upstream/master

9d1541c

prep for PR

5c8ecd8

removal of old folder

0267fca

prep for PR

f9bf6ba

wrong args for travis

cec0679

more PR prep

d78e8bd

updated README

6bf2df2

commented out variables in terraform.tfvars

b22cff0

Merge branch 'topic-101-vm-from-user-image' of https://github.com/10t…

09175de

…hmagnitude/terraform into topic-101-vm-from-user-image

Topic 101 vm from user image (#2)

0843f45

* initial commit - 101-vm-from-user-image * added tfvars and info for README * added CI config; added sane defaults for variables; updated deployment script, added mac specific deployment for local testing * prep for PR

added new template

49caea5

oops, left off master

2236828

Merge branch 'master' of github.com:harijayms/terraform

27167b5

merge upstream; prep for PR

5b8f7bb

prep for PR

7f8fe25

correct repository for destination

eecb2d2

renamed scripts to be more intuitive; added check for docker

92e341d

anniehedgpeth and others added 19 commits May 4, 2017 17:50

Merge branch 'master' into topic-101-vm-from-user-image

007ea39

Merge pull request #10 from 10thmagnitude/topic-101-vm-from-user-image

f6c13d4

azure-vm-from-user-image

removed old branch

1366980

vnet-to-vnet-peering initial commit

67ab1c9

azure-2-vms-loadbalancer-lbrules (#13)

6a1813e

* initial commit * need to change lb_rule & nic * deploys locally * updated README * updated travis and deploy scripts for Hari's repo * renamed deploy script * clean up * prep for PR * updated readme * fixing conflict in .travis.yml

merged with hari/master

ac20789

cleaned up commented out section

8dcf1b4

validate peers via cli

87c0c2b

add CI build tag

888b578

added graph

1cea884

added graph; deleted out.tfplan

fa57ec4

updating head

cba3a3a

Merge branch 'master' of https://github.com/harijayms/terraform into …

d3361ca

…topic-201-vnet-to-vnet-peering

Merge branch 'topic-201-vnet-to-vnet-peering' of https://github.com/h…

e6810cd

…arijayms/terraform into topic-201-vnet-to-vnet-peering

updating variable descriptions

297f544

added graph to README

ed38872

updated graph in README

d6a5f49

prepping for Hashicorp PR

9e6cffc

Prepping for Hashicorp PR

d2529b3

anniehedgpeth changed the title ~~provider/azurerm: Example of VNET to VNET Peering~~ provider/azurerm: Add example of VNET to VNET Peering May 10, 2017

radeksimko added documentation provider/azurerm labels May 10, 2017

tombuildsstuff suggested changes May 10, 2017

View reviewed changes

anniehedgpeth added 2 commits May 10, 2017 17:46

removing unneeded files

ceca834

changed README

b3e3e32

stack72 merged commit 5ed7f16 into hashicorp:master May 11, 2017

anniehedgpeth deleted the hashi-201-vnet-to-vnet-peering branch June 16, 2017 14:05

ghost locked and limited conversation to collaborators Apr 8, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

provider/azurerm: Add example of VNET to VNET Peering #14338

provider/azurerm: Add example of VNET to VNET Peering #14338

anniehedgpeth commented May 10, 2017

tombuildsstuff left a comment •

edited

Loading

tombuildsstuff May 10, 2017

anniehedgpeth May 10, 2017

tombuildsstuff May 10, 2017

anniehedgpeth May 10, 2017

tombuildsstuff May 10, 2017

anniehedgpeth May 10, 2017

stack72 commented May 11, 2017

ghost commented Apr 8, 2020

provider/azurerm: Add example of VNET to VNET Peering #14338

provider/azurerm: Add example of VNET to VNET Peering #14338

Conversation

anniehedgpeth commented May 10, 2017

CI:

tombuildsstuff left a comment • edited Loading

Choose a reason for hiding this comment

tombuildsstuff May 10, 2017

Choose a reason for hiding this comment

anniehedgpeth May 10, 2017

Choose a reason for hiding this comment

tombuildsstuff May 10, 2017

Choose a reason for hiding this comment

anniehedgpeth May 10, 2017

Choose a reason for hiding this comment

tombuildsstuff May 10, 2017

Choose a reason for hiding this comment

anniehedgpeth May 10, 2017

Choose a reason for hiding this comment

stack72 commented May 11, 2017

ghost commented Apr 8, 2020

tombuildsstuff left a comment •

edited

Loading