From 2f3d29395f0adc1ca656fde429d214c229ac81fa Mon Sep 17 00:00:00 2001 From: Erin Horning Date: Tue, 22 Aug 2023 09:09:37 -0600 Subject: [PATCH] remove validation --- modules/gcs/README.md | 14 +++++++------- modules/gcs/variables.tf | 9 --------- 2 files changed, 7 insertions(+), 16 deletions(-) diff --git a/modules/gcs/README.md b/modules/gcs/README.md index 7672f345..c2c2c93d 100644 --- a/modules/gcs/README.md +++ b/modules/gcs/README.md @@ -114,7 +114,7 @@ module "bucket" { | name | description | type | required | default | |---|---|:---:|:---:|:---:| | [name](variables.tf#L116) | Bucket name suffix. | string | ✓ | | -| [project_id](variables.tf#L179) | Bucket project id. | string | ✓ | | +| [project_id](variables.tf#L170) | Bucket project id. | string | ✓ | | | [cors](variables.tf#L17) | CORS configuration for the bucket. Defaults to null. | object({…}) | | null | | [encryption_key](variables.tf#L28) | KMS key that will be used for encryption. | string | | null | | [force_destroy](variables.tf#L34) | Optional map to set force destroy keyed by name, defaults to false. | bool | | false | @@ -125,12 +125,12 @@ module "bucket" { | [logging_config](variables.tf#L107) | Bucket logging configuration. | object({…}) | | null | | [notification_config](variables.tf#L121) | GCS Notification configuration. | object({…}) | | null | | [objects_to_upload](variables.tf#L135) | Objects to be uploaded to bucket. | map(object({…})) | | {} | -| [prefix](variables.tf#L169) | Optional prefix used to generate the bucket name. | string | | null | -| [retention_policy](variables.tf#L184) | Bucket retention policy. | object({…}) | | null | -| [storage_class](variables.tf#L193) | Bucket storage class. | string | | "MULTI_REGIONAL" | -| [uniform_bucket_level_access](variables.tf#L203) | Allow using object ACLs (false) or not (true, this is the recommended behavior) , defaults to true (which is the recommended practice, but not the behavior of storage API). | bool | | true | -| [versioning](variables.tf#L209) | Enable versioning, defaults to false. | bool | | false | -| [website](variables.tf#L215) | Bucket website. | object({…}) | | null | +| [prefix](variables.tf#L160) | Optional prefix used to generate the bucket name. | string | | null | +| [retention_policy](variables.tf#L175) | Bucket retention policy. | object({…}) | | null | +| [storage_class](variables.tf#L184) | Bucket storage class. | string | | "MULTI_REGIONAL" | +| [uniform_bucket_level_access](variables.tf#L194) | Allow using object ACLs (false) or not (true, this is the recommended behavior) , defaults to true (which is the recommended practice, but not the behavior of storage API). | bool | | true | +| [versioning](variables.tf#L200) | Enable versioning, defaults to false. | bool | | false | +| [website](variables.tf#L206) | Bucket website. | object({…}) | | null | ## Outputs diff --git a/modules/gcs/variables.tf b/modules/gcs/variables.tf index 1077c0f0..13c0d281 100644 --- a/modules/gcs/variables.tf +++ b/modules/gcs/variables.tf @@ -154,15 +154,6 @@ variable "objects_to_upload" { encryption_key = string })) })) - validation { - condition = alltrue([ - for k, v in var.objects_to_upload : - v.content != null - || - v.source != null - ]) - error_message = "Object must have either content or source defined." - } default = {} }