Skip to content
This repository was archived by the owner on May 8, 2023. It is now read-only.

Conversation

@coreyperkins
Copy link
Contributor

No description provided.

@pbolduc
Copy link
Owner

pbolduc commented Aug 2, 2016

As you are adding DSx_v2 size VMs, please add the other sizes, from Standard_D1_v2 to at least Standard_D5_v2. Also, since many people will use the template to set up something for testing, please leave the default instance size at DS2 (DS2_v2 is fine since it is the same cost). I do not want to burn through my MSDN credits using a VM size bigger than necessary.

If you fix the size of VMs, I will merge the pull request.

Also, I would like to see things more configurable,

  • parameter to configure Run Projections or not, can default to true.
  • parameter to use nginx or not. See issue Allow user to omit or change nginx version #5. It would also good to provide guidance in the read me on why people should be using nginx in this configuration.

@pbolduc
Copy link
Owner

pbolduc commented Oct 25, 2016

Sorry for the delay, I was on vacation (out of the country) for most of October. I was really busy before going on vacation and did not time to review your recent changes. I hope to get caught up this week and find time this weekend to get these changes merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants