Add enabled flag (#17)

* add enabled flag

Author: sarkis

main.tf

@@ -14,7 +14,7 @@ module "label" {
 }
 
 resource "aws_s3_bucket" "cache_bucket" {
-  count         = "${var.cache_enabled == "true" ? 1 : 0}"
+  count         = "${var.enabled == "true" && var.cache_enabled == "true" ? 1 : 0}"
   bucket        = "${local.cache_bucket_name_normalised}"
   acl           = "private"
   force_destroy = true
@@ -54,7 +54,7 @@ locals {
   cache_def = {
     "true" = [{
       type     = "S3"
-      location = "${var.cache_enabled == "true" ? aws_s3_bucket.cache_bucket.0.bucket : "none" }"
+      location = "${var.enabled == "true" && var.cache_enabled == "true" ? join("", aws_s3_bucket.cache_bucket.*.bucket) : "none" }"
     }]
 
     "false" = []
@@ -65,6 +65,7 @@ locals {
 }
 
 resource "aws_iam_role" "default" {
+  count              = "${var.enabled == "true" ? 1 : 0}"
   name               = "${module.label.id}"
   assume_role_policy = "${data.aws_iam_policy_document.role.json}"
 }
@@ -87,13 +88,14 @@ data "aws_iam_policy_document" "role" {
 }
 
 resource "aws_iam_policy" "default" {
+  count  = "${var.enabled == "true" ? 1 : 0}"
   name   = "${module.label.id}"
   path   = "/service-role/"
   policy = "${data.aws_iam_policy_document.permissions.json}"
 }
 
 resource "aws_iam_policy" "default_cache_bucket" {
-  count  = "${var.cache_enabled == "true" ? 1 : 0}"
+  count  = "${var.enabled == "true" && var.cache_enabled == "true" ? 1 : 0}"
   name   = "${module.label.id}-cache-bucket"
   path   = "/service-role/"
   policy = "${data.aws_iam_policy_document.permissions_cache_bucket.json}"
@@ -124,6 +126,8 @@ data "aws_iam_policy_document" "permissions" {
 }
 
 data "aws_iam_policy_document" "permissions_cache_bucket" {
+  count = "${var.enabled == "true" ? 1 : 0}"
+
   statement {
     sid = ""
 
@@ -141,17 +145,19 @@ data "aws_iam_policy_document" "permissions_cache_bucket" {
 }
 
 resource "aws_iam_role_policy_attachment" "default" {
+  count      = "${var.enabled == "true" ? 1 : 0}"
   policy_arn = "${aws_iam_policy.default.arn}"
   role       = "${aws_iam_role.default.id}"
 }
 
 resource "aws_iam_role_policy_attachment" "default_cache_bucket" {
-  count      = "${var.cache_enabled == "true" ? 1 : 0}"
+  count      = "${var.enabled == "true" && var.cache_enabled == "true" ? 1 : 0}"
   policy_arn = "${element(aws_iam_policy.default_cache_bucket.*.arn, count.index)}"
   role       = "${aws_iam_role.default.id}"
 }
 
 resource "aws_codebuild_project" "default" {
+  count        = "${var.enabled == "true" ? 1 : 0}"
   name         = "${module.label.id}"
   service_role = "${aws_iam_role.default.arn}"
 

outputs.tf

@@ -1,15 +1,15 @@
 output "project_name" {
-  value = "${aws_codebuild_project.default.name}"
+  value = "${join("", aws_codebuild_project.default.*.name)}"
 }
 
 output "project_id" {
-  value = "${aws_codebuild_project.default.id}"
+  value = "${join("", aws_codebuild_project.default.*.id)}"
 }
 
 output "role_arn" {
-  value = "${aws_iam_role.default.id}"
+  value = "${join("", aws_iam_role.default.*.id)}"
 }
 
 output "cache_bucket_name" {
-  value = "${var.cache_enabled == "true" ? aws_s3_bucket.cache_bucket.0.bucket : "UNSET" }"
+  value = "${var.enabled == "true" && var.cache_enabled == "true" ? join("", aws_s3_bucket.cache_bucket.*.bucket) : "UNSET" }"
 }

variables.tf

@@ -1,12 +1,15 @@
 variable "namespace" {
+  type    = "string"
   default = "global"
 }
 
 variable "stage" {
+  type    = "string"
   default = "default"
 }
 
 variable "name" {
+  type    = "string"
   default = "codebuild"
 }
 
@@ -21,31 +24,43 @@ variable "environment_variables" {
   description = "A list of maps, that contain both the key 'name' and the key 'value' to be used as additional environment variables for the build."
 }
 
+variable "enabled" {
+  type        = "string"
+  default     = "true"
+  description = "A boolean to enable/disable resource creation."
+}
+
 variable "cache_enabled" {
+  type        = "string"
   default     = "true"
   description = "If cache_enabled is true, create an S3 bucket for storing codebuild cache inside"
 }
 
 variable "cache_expiration_days" {
+  type        = "string"
   default     = "7"
   description = "How many days should the build cache be kept."
 }
 
 variable "cache_bucket_suffix_enabled" {
+  type        = "string"
   default     = "true"
   description = "The cache bucket generates a random 13 character string to generate a unique bucket name. If set to false it uses terraform-null-label's id value"
 }
 
 variable "build_image" {
+  type        = "string"
   default     = "aws/codebuild/docker:1.12.1"
   description = "Docker image for build environment, e.g. 'aws/codebuild/docker:1.12.1' or 'aws/codebuild/eb-nodejs-6.10.0-amazonlinux-64:4.0.0'. For more info: http://docs.aws.amazon.com/codebuild/latest/userguide/build-env-ref.html"
 }
 
 variable "build_compute_type" {
+  type    = "string"
   default = "BUILD_GENERAL1_SMALL"
 }
 
 variable "buildspec" {
+  type        = "string"
   default     = ""
   description = "Optional buildspec declaration to use for building the project"
 }
@@ -66,11 +81,13 @@ variable "tags" {
 }
 
 variable "privileged_mode" {
+  type        = "string"
   default     = "false"
   description = "(Optional) If set to true, enables running the Docker daemon inside a Docker container on the CodeBuild instance. Used when building Docker images"
 }
 
 variable "github_token" {
+  type        = "string"
   default     = ""
   description = "(Optional) GitHub auth token environment variable (`GITHUB_TOKEN`)"
 }