Skip to content

Commit 4f7aaf9

Browse files
committed
CCM-9689: Fix lambda descriptions for variables and outputs
1 parent 85af54e commit 4f7aaf9

8 files changed

Lines changed: 35 additions & 5 deletions

File tree

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
output "name" {
2+
description = "Name of the Amplify branch"
23
value = aws_amplify_branch.main.branch_name
34
}

infrastructure/modules/aws-backup-source/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ variable "backup_copy_vault_account_id" {
9393
}
9494

9595
variable "backup_plan_config_s3" {
96-
description = "Configuration for backup plans"
96+
description = "Configuration for backup plans for s3"
9797
type = object({
9898
enable = bool
9999
selection_tag = string

infrastructure/modules/eventpub/outputs.tf

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
output "sns_topic" {
2+
description = "SNS Topic ARN and Name"
23
value = {
34
arn = aws_sns_topic.main.arn
45
name = aws_sns_topic.main.name
56
}
67
}
78

89
output "s3_bucket_event_cache" {
10+
description = "S3 Bucket ARN and Name for event cache"
911
value = var.enable_event_cache ? {
1012
arn = module.s3bucket_event_cache[0].arn
1113
bucket = module.s3bucket_event_cache[0].bucket
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,19 @@
11
output "key_arn" {
2+
description = "ARN of the KMS key"
23
value = aws_kms_key.main.arn
34
}
45

56
output "key_id" {
7+
description = "ID of the KMS key"
68
value = aws_kms_key.main.key_id
79
}
810

911
output "admin_policy_arn" {
12+
description = "ARN of the admin IAM policy"
1013
value = aws_iam_policy.admin.arn
1114
}
1215

1316
output "user_policy_arn" {
17+
description = "ARN of the user IAM policy"
1418
value = aws_iam_policy.user.arn
1519
}
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,39 @@
11
output "function_name" {
2+
description = "Name of the Lambda function"
23
value = aws_lambda_function.main.function_name
34
}
45

56
output "function_arn" {
7+
description = "ARN of the Lambda function"
68
value = aws_lambda_function.main.arn
79
}
810

911
output "function_invoke_arn" {
12+
description = "Invoke ARN of the Lambda function"
1013
value = aws_lambda_function.main.invoke_arn
1114
}
1215

1316
output "function_qualified_arn" {
17+
description = "Qualified ARN of the Lambda function, including version or alias"
1418
value = aws_lambda_function.main.qualified_arn
1519
}
1620

1721
output "function_env_vars" {
22+
description = "Environment variables for the Lambda function"
1823
value = length(var.lambda_env_vars) == 0 ? {} : aws_lambda_function.main.environment[0].variables
1924
}
2025

2126
output "iam_role_name" {
27+
description = "Name of the IAM role associated with the Lambda function"
2228
value = aws_iam_role.main.name
2329
}
2430

2531
output "iam_role_arn" {
32+
description = "ARN of the IAM role associated with the Lambda function"
2633
value = aws_iam_role.main.arn
2734
}
2835

2936
output "cloudwatch_log_group_name" {
37+
description = "Name of the CloudWatch Log Group for the Lambda function"
3038
value = aws_cloudwatch_log_group.main.name
3139
}

infrastructure/modules/lambda/variables.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -115,13 +115,13 @@ variable "lambda_env_vars" {
115115

116116
variable "function_code_base_path" {
117117
type = string
118-
description = "The path to the sourcecode directories needed for this lambda"
118+
description = "The base path to the sourcecode directories needed for this lambda"
119119
default = "./"
120120
}
121121

122122
variable "function_code_dir" {
123123
type = string
124-
description = "The path to the sourcecode directories needed for this lambda"
124+
description = "The directory for this lambda"
125125
}
126126

127127
variable "function_s3_bucket" {
@@ -185,7 +185,7 @@ variable "sns_destination_kms_key" {
185185

186186
variable "lambda_dlq_message_retention_seconds" {
187187
type = number
188-
description = "KMS Key ARN to be used for SNS Topic for on-failure Lambda invocation records"
188+
description = "The number of seconds to retain messages in the Lambda DLQ SQS queue"
189189
default = 86400
190190
}
191191

@@ -209,7 +209,7 @@ variable "enable_lambda_insights" {
209209

210210
variable "lambda_at_edge" {
211211
type = bool
212-
description = "Enable the lambda insights layer, this must be disabled for lambda@edge usage"
212+
description = "Whether this Lambda is a Lambda@Edge function"
213213
default = false
214214
}
215215

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,44 @@
11
output "acl" {
2+
description = "The ACL of the S3 bucket. If the object ownership is set to 'BucketOwnerEnforced', the ACL will be 'private'. Otherwise, it will reflect the ACL set in the aws_s3_bucket_acl resource."
23
value = var.object_ownership == "BucketOwnerEnforced" ? "private" : aws_s3_bucket_acl.main[0].acl
34
}
45

56
output "arn" {
7+
description = "The ARN of the S3 bucket"
68
value = aws_s3_bucket.main.arn
79
}
810

911
output "bucket" {
12+
description = "The name of the S3 bucket"
1013
value = aws_s3_bucket.main.bucket
1114
}
1215

1316
output "bucket_domain_name" {
17+
description = "The domain name of the S3 bucket"
1418
value = aws_s3_bucket.main.bucket_domain_name
1519
}
1620

1721
output "bucket_regional_domain_name" {
22+
description = "The regional domain name of the S3 bucket"
1823
value = aws_s3_bucket.main.bucket_regional_domain_name
1924
}
2025

2126
output "hosted_zone_id" {
27+
description = "The hosted zone ID of the S3 bucket"
2228
value = aws_s3_bucket.main.hosted_zone_id
2329
}
2430

2531
output "id" {
32+
description = "The ID of the S3 bucket"
2633
value = aws_s3_bucket.main.id
2734
}
2835

2936
output "policy" {
37+
description = "The policy of the S3 bucket"
3038
value = aws_s3_bucket_policy.main.policy
3139
}
3240

3341
output "region" {
42+
description = "The AWS region where the S3 bucket is located"
3443
value = aws_s3_bucket.main.region
3544
}
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,29 @@
11
output "sqs_queue_url" {
2+
description = "The URL of the SQS queue"
23
value = aws_sqs_queue.sqs_queue.id
34
}
45

56
output "sqs_queue_arn" {
7+
description = "The ARN of the SQS queue"
68
value = aws_sqs_queue.sqs_queue.arn
79
}
810

911
output "sqs_dlq_url" {
12+
description = "The URL of the SQS dead-letter queue"
1013
value = var.create_dlq ? aws_sqs_queue.deadletter_queue[0].id : null
1114
}
1215

1316
output "sqs_dlq_arn" {
17+
description = "The ARN of the SQS dead-letter queue"
1418
value = var.create_dlq ? aws_sqs_queue.deadletter_queue[0].arn : null
1519
}
1620

1721
output "sqs_queue_name" {
22+
description = "The name of the SQS queue"
1823
value = "${local.csi}-queue"
1924
}
2025

2126
output "sqs_dlq_name" {
27+
description = "The name of the SQS dead-letter queue"
2228
value = var.create_dlq ? aws_sqs_queue.deadletter_queue[0].name : null
2329
}

0 commit comments

Comments
 (0)