Evaluate var.create variable to set count to 1 or 0

[jordiclariana]

In order to avoid hashicorp/terraform#18195

[antonbabenko]

I don't think this is necessary. Please try to run examples/disabled on your local setup.

[jordiclariana]

The examples work, and is not the point of the issue in terraform. The problem is when create parameter is a result of evaluating a ternary, like this: create = ${true ? true : false}. Then it will fail.

[jordiclariana]

From the examples/disabled, notice that I changed the create line:

  region = "eu-west-1"
}

#############################################################
# Data sources to get VPC and default security group details
#############################################################
data "aws_vpc" "default" {
  default = true
}

data "aws_security_group" "default" {
  name   = "default"
  vpc_id = "${data.aws_vpc.default.id}"
}

########################################################
# Security groups WILL NOT be created by these examples
########################################################
module "complete_sg_disabled" {
  source = "../../"

  create      = "${true ? false: false}"
  name        = "complete-sg"
  description = "Security group with all available arguments set (this is just an example)"
  vpc_id      = "${data.aws_vpc.default.id}"

  ingress_cidr_blocks = ["0.0.0.0/0"]
}

This produces this:

* module.complete_sg_disabled.aws_security_group.this: cannot parse "false" as an integer

[antonbabenko]

Jordi, thanks for this PR and for making this module a bit more forward-compatible with the upcoming version of Terraform! :)

[antonbabenko]

v2.1.0 has been released.

[github-actions]

I'm going to lock this pull request 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 related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.