Dennis Hoppe

@ramkumarcrk The following code will construct a tag which is used as name of the subnet. https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/master/main.tf#L391-L400 In my case I used the variables `name`, `database_subnets`, `private_subnets` and `public_subnets`.

@aidanmelen I appreciate your feedback. Unfortunately I do not have time for the next 2 weeks, but I saw that you had already created a pull request. 👍

The additional output `arn` is available since version 3.29.0 of the Terraform AWS provider. https://github.com/hashicorp/terraform-provider-aws/pull/13944#issuecomment-782201689 This means the version constraint needs to be modified and since this is an backwards...

I will have a look at these undeclared input variables later.

> I will have a look at these undeclared input variables later. Fixed it by excluding modules/_templates from pre-commit hook terraform_validate.

That depends on which operating system you are using. By default the mirror of the operating system is used as source. However, in the case of Debian or RedHat, you...

I wonder if we should check the version of Nginx somewhere before configuring port ranges. We already did that for some templates, based on the fact ```nginx_version```.

@ekohl I would like to merge this pull request. Any objections?

This also affects Debian and Ubuntu. Maybe we should migrate to Bintray, since they also provide packages for Erlang. ``` sudo tee /etc/apt/sources.list.d/bintray.rabbitmq.list

@madelaney I am confused. In version 1.7 the file ```slave.pp``` has only 299 lines. https://github.com/voxpupuli/puppet-jenkins/blob/v1.7.0/manifests/slave.pp It looks more like the master branch is being used. In this case the user...