Only create VPC if no VPC ID is passed

[bsuv]

terraform-aws-atlantis/main.tf

Line 18 in 508eb0c

module "vpc" {

Great work Anton!

Recently I also developed an Atlantis module to which I pass a vpc_id as a variable instead of a cidr_block to create a new one - under the assumption that it's more common for someone to deploy Atlantis in an existing VPC.

My proposal:

• have 2 input variables (cidr_block and vpc_id), both with dummy defaults.
• use count to determine whether a new VPC is needed, in which case the cidr_block would be used.

[antonbabenko]

@bsuv Thanks for opening this issue!

You are literally reading my mind. :) I am working on it right now. Expect a PR in a couple of hours with some more enhancements like this.

[antonbabenko]

The weather was so good today, so it took me a little bit longer to complete this.

v1.1.0 has been released and there is a possibility to reuse existing VPC and subnets.

[github-actions]

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.