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.

Sign up for GitHub

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

Jump to bottom

provider/azurerm: Add example of CDN with Storage Account #14193

Merged
merged 49 commits into from
May 4, 2017
Merged

provider/azurerm: Add example of CDN with Storage Account #14193

merged 49 commits into from
May 4, 2017

Conversation

anniehedgpeth
Copy link
Contributor

@anniehedgpeth anniehedgpeth commented May 3, 2017 •
edited
Loading

CI: Build Status

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

  • Resource Group
  • Storage Account
  • CDN Profile
  • CDN Endpoint

CI process:

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

graph

anniehedgpeth and others added 30 commits April 17, 2017 19:07
@anniehedgpeth
initial commit - 101-vm-from-user-image
9f0d990
@anniehedgpeth
changed branch name
6f577a8
@anniehedgpeth
not deploying - storage problems
611d70a
@anniehedgpeth
provisions vm but image not properly prepared
fdff23a
@anniehedgpeth
storage not correct
0dac5ec
@anniehedgpeth
provisions properly
81e2559
@anniehedgpeth
changed main.tf to azuredeploy.tf
7dc8150
@anniehedgpeth
added tfvars and info for README
5c079fa
@anniehedgpeth
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
@anniehedgpeth
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
@anniehedgpeth
updated README
6bf2df2
commented out variables in terraform.tfvars
b22cff0
@anniehedgpeth
Merge branch 'topic-101-vm-from-user-image' of https://github.com/10t…
09175de 
…hmagnitude/terraform into topic-101-vm-from-user-image
@scottzilla
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
Scott Nowicki and others added 18 commits April 26, 2017 22:15
merge vm simple; vm from image
23627c8
@anniehedgpeth
initial commit
ac2e5c2
@anniehedgpeth
deploys locally
935c19a
@anniehedgpeth
updated deploy
66b7441
@anniehedgpeth
changed to allow http & https (like ARM tmplt)
50e8d8a
@anniehedgpeth
changed host_name & host_name variable desc
ace9b0b
@anniehedgpeth
added cdn branch; ci uses running vm to test
fd0231b
prep for PR
d8ef7b1
@anniehedgpeth
took out armviz button and minor README changes
25ef50c
@anniehedgpeth
changed host_name
63c494d
@anniehedgpeth
fixed merge conflicts
9a5a15a
@anniehedgpeth
changed host_name variable
d7e2acb
@anniehedgpeth
updating Hashicorp's changes to merged simple linux branch
dbaf8d1
@anniehedgpeth
updating files to merge w/master and prep for Hashicorp pr
b850cd5
@anniehedgpeth
reverted to Hashicorp's .travis.yml
26a4c33
@anniehedgpeth
terraform graph
83b16f0
@anniehedgpeth
removing pending example
9148c1c
@anniehedgpeth
Merge branch 'master' of https://github.com/hashicorp/terraform into …
97bcdfd 
…hashicorp-201-cdn-with-storage-account
@tombuildsstuff tombuildsstuff self-assigned this May 4, 2017
tombuildsstuff
tombuildsstuff approved these changes May 4, 2017
View reviewed changes
Copy link
Contributor

@tombuildsstuff tombuildsstuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @anniehedgpeth

Thanks for this PR :)

I've taken a look / run a terraform plan for the example and aside from this line this otherwise looks good to me :)

@tombuildsstuff tombuildsstuff added the waiting-response An issue/pull request is waiting for a response from the community label May 4, 2017
@tombuildsstuff tombuildsstuff removed the waiting-response An issue/pull request is waiting for a response from the community label May 4, 2017
@tombuildsstuff tombuildsstuff merged commit a9d5610 into hashicorp:master May 4, 2017
@anniehedgpeth
Copy link
Contributor Author

Thanks, @tombuildsstuff !!

@anniehedgpeth anniehedgpeth deleted the hashicorp-201-cdn-with-storage-account branch May 4, 2017 13:33
@ghost
Copy link

ghost commented Apr 13, 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 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tombuildsstuff tombuildsstuff tombuildsstuff approved these changes

Assignees

@tombuildsstuff tombuildsstuff

Labels
enhancement provider/azurerm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anniehedgpeth @tombuildsstuff @scottzilla