byu-oit / byu-oit/terraform-aws-postman-test-lambda

App Name length requirement is too short

Open
#68 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
HCL
Stars
4
Forks
3
PR merge metrics
No merged PRs in 30d

Description

If my repository name is ces-adm-sv-byupw and I include that in my app name, then the postman module will complain about the name of the postman bucket being too long:

```
│ Error: expected length of bucket to be in the range (0 - 63), got ces-adm-sv-byupw-deploy-test-dev-postman-tests-***-logs

│ with module.app.module.postman_test_lambda.aws_s3_bucket.postman_bucket_logs[0],
│ on .terraform/modules/app.postman_test_lambda/main.tf line 49, in resource "aws_s3_bucket" "postman_bucket_logs":
│ 49: bucket = "${var.app_name}-postman-tests-${data.aws_caller_identity.current.account_id}-logs"

```

The app name shouldn't have to be so short. My suggestion would be to change the name of the bucket:

1. Remove the account id from the bucket name
2. Shorted `postman-tests` to `pm`

```terraform
bucket = "${var.app_name}-pm-logs"
```

On my part, I'm removing the `deploy-test` portion of the bucket name.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.