CIDR variable cant be defaulted to an empty string

[egarbi]

Terraform Version

Terraform v0.11.2

Terraform Configuration Files

# AWS credentials and region
provider "aws" {
  region = "${var.region}"
}

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "1.9.1"
}

Expected Behavior

Plan should finish without errors

Actual Behavior

Error: module.vpc.aws_vpc.this: "cidr_block" must contain a valid CIDR, got error parsing: invalid CIDR address:

Steps to Reproduce

1. terraform init
2. terraform plan

[antonbabenko]

If you don't want to create VPC you should write like this:

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "1.9.1"

  create_vpc = false
}

if you want to create VPC you should specify CIDR like this:

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "1.9.1"

  cidr = "10.0.0.0/20"
}

[egarbi]

@antonbabenko I know how this module work, I was trying to pointing out a particular issue if I use the module without required values as indicated in registry's doc.
Maybe then default the create_vpc value to false. Don't know...

[antonbabenko]

@egarbi I could not leave this one as is, so please take a look at #93 :)

[egarbi]

Hi @antonbabenko sorry for the delay, this is better, the apply will exit with an error message and the user will be able to notice what is needed to change.

[antonbabenko]

v1.26.0 has been released. Thanks for your help!

[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.