Adding descriptions to the variables

modules/acm/variables.tf

@@ -4,21 +4,21 @@ variable "account_id" {
   
 }
 variable "assume_role"{
-    description = ""
+    description = "The name of the role used in your multi-account governance policy. By default AWSControlTowerExecution compatible with AWS Control Tower environments with or without AFT"
     type = string
     default ="AWSControlTowerExecution"
 }
 variable "environment"{
-    description = ""
+    description = "Environment for this deployment, the bucket and some random tags and strings will have it attached"
     type = string
     default = "dev"
 }
 variable "domain"{
-    description = ""
+    description = "FQDN for deploying this site"
     type = string
 }
 
 variable "zone_id"{
-    description = ""
+    description = "Zone ID for the ROUTE53 hosting the domain zone. This route 53 lives in the same account than the ACM and the CDN"
     type = string
 }
\ No newline at end of file

modules/cdn/outputs.tf

@@ -7,7 +7,7 @@ output "distribution_arn" {
 }
 
 output "cloudfront_origin_access_identity_iam_arns"{
-    description = "	The IAM arns of the origin access identities created"
+    description = "The IAM arns of the origin access identities created"
     value = module.cdn.cloudfront_origin_access_identity_iam_arns
 }
 output "cloudfront_origin_access_identity_ids"{

modules/cdn/variables.tf

@@ -4,27 +4,27 @@ variable "account_id" {
   
 }
 variable "assume_role"{
-    description = ""
+    description = "The name of the role used in your multi-account governance policy. By default AWSControlTowerExecution compatible with AWS Control Tower environments with or without AFT"
     type = string
     default ="AWSControlTowerExecution"
 }
 variable "environment"{
-    description = ""
+    description = "Environment for this deployment, the bucket and some random tags and strings will have it attached"
     type = string
     default = "dev"
 }
 variable "domain"{
-    description = ""
+    description = "FQDN for deploying this site"
     type = string
 }
 
 variable "bucket"{
-    description = ""
+    description = "Bucket name"
     type = string
 }
 
 variable "acm_certificate_arn" {
-    description = ""
+    description = "ACM certificate ARN for this domain"
     type = string
   
 }
\ No newline at end of file

modules/dns/variables.tf

@@ -4,13 +4,13 @@ variable "account_id" {
   
 }
 variable "environment"{
-    description = ""
+    description = "Environment for this deployment, the bucket and some random tags and strings will have it attached"
     type = string
     default = "dev"
 }
 variable "region" {
     type = string
-    description = "region"
+    description = "Region for the S3 related stuff. The CDN will be deployed on North Virginia"
 }
 variable "zone_id" {
     type = string
@@ -32,6 +32,5 @@ variable "domain" {
 
 variable "assume_role" {
     type = string
-    description = ""
-}
+    description = "The name of the role used in your multi-account governance policy. By default AWSControlTowerExecution compatible with AWS Control Tower environments with or without AFT"
 

modules/iam/variables.tf

 variable "force_destroy" {
-    description = ""
+    description = "Destroy protection for the s3 bucket. Put it to true in case you want to destroy the content of the bucket when this resource is deleted"
     type = bool
     default = false
 }
 
 variable "s3_bucket_id"{
-    description = ""
+    description = "S3 Bucket ID"
     type = string
 }
 
@@ -20,7 +20,7 @@ variable "identity_arn" {
 }
 variable "region" {
     type = string
-    description = "(optional) describe your variable"
+    description = "Region for the S3 related stuff. The CDN will be deployed on North Virginia"
 }
 
 variable "account_id" {
@@ -31,11 +31,11 @@ variable "account_id" {
 
 variable "assume_role" {
     type = string
-    description = ""
+    description = "The name of the role used in your multi-account governance policy. By default AWSControlTowerExecution compatible with AWS Control Tower environments with or without AFT"
 }
 
 variable "environment" {
     type = string
-    description = ""
+    description = "Environment for this deployment, the bucket and some random tags and strings will have it attached"
   
 }
\ No newline at end of file

modules/s3/variables.tf

 variable "force_destroy" {
-    description = ""
+    description = "Destroy protection for the s3 bucket. Put it to true in case you want to destroy the content of the bucket when this resource is deleted"
     type = bool
     default = false
 }
 
 variable "environment"{
-    description = ""
+    description = "Environment for this deployment, the bucket and some random tags and strings will have it attached"
     type = string
     default = "dev"
 }
 
 variable "region" {
     type = string
-    description = "(optional) describe your variable"
+    description = "Region for the S3 related stuff. The CDN will be deployed on North Virginia"
 }
 
 variable "account_id" {
@@ -27,5 +27,4 @@ variable "project_name" {
 
 variable "assume_role" {
     type = string
-    description = ""
-}
+    description = "The name of the role used in your multi-account governance policy. By default AWSControlTowerExecution compatible with AWS Control Tower environments with or without AFT"
\ No newline at end of file
\ No newline at end of file

outputs.tf

 output "cdn_account_id" {
     value = local.settings.cdn_account_id
-    description = "Setting value"
+    description = "Account where the CDN must be created"
 }
 output "s3_account_id" {
     value = local.settings.s3_account_id
-    description = "Setting value"
+    description = "Account where the S3 bucket must be created"
 }
 output "project_name" {
     value = local.settings.project_name
-    description = "Setting value"
+    description = "Name of the project to generate some random strings related to it"
 }
 output "region" {
     value = local.settings.region
-    description = "Setting value"
+    description = "Region for the S3 related stuff. The CDN will be deployed on North Virginia"
 }
 output "environment" {
     value = local.settings.environment
-    description = "Setting value"
+    description = "Environment for this deployment, the bucket and some random tags and strings will have it attached"
 }
 output "domain" {
     value = local.settings.domain
-    description = "Setting value"
+    description = "FQDN for deploying this site"
 }
 output "zone_id" {
     value = local.settings.zone_id
-    description = "Setting value"
+    description = "Zone ID for the ROUTE53 hosting the domain zone. This route 53 lives in the same account than the ACM and the CDN"
 }
 output "s3_force_destroy" {
     value = local.settings.s3_force_destroy
-    description = "Setting value"
+    description = "Destroy protection for the s3 bucket. Put it to true in case you want to destroy the content of the bucket when this resource is deleted"
 }
 output "assume_role" {
     value = local.settings.assume_role
-    description = "Setting value"
+    description = "The name of the role used in your multi-account governance policy. By default AWSControlTowerExecution compatible with AWS Control Tower environments with or without AFT"
 }
 output "cloudfront_origin_access_identity_iam_arns"{
-    description = "	The IAM arns of the origin access identities created"
+    description = "The IAM arns of the origin access identities created"
     value = module.distribution.cloudfront_origin_access_identity_iam_arns
 }
 output "cloudfront_origin_access_identity_ids"{
@@ -43,18 +43,18 @@ output "cloudfront_origin_access_identity_ids"{
     value = module.distribution.cloudfront_origin_access_identity_ids
 }
 output "cdn_endpoint" {
-    description = ""
+    description = "Endpoint generated by the CDN"
     value = module.distribution.cdn_endpoint
 }
 output "s3_bucket_bucket_domain_name" {
-    description = ""
+    description = "DNS for generated S3 Bucket"
     value = module.storage.s3_bucket_bucket_domain_name
 }
 output "acm_certificate_arn" {
-    description = ""
+    description = "ACM certificate ARN for this domain"
     value = module.certificate.acm_certificate_arn
 }
 output "s3_bucket_id" {
-    description = ""
+    description = "ID for generated S3 Bucket"
     value = module.storage.s3_bucket_id
 }
\ No newline at end of file