byu-oit / byu-oit/terraform-aws-postman-test-lambda
App Name length requirement is too short
- 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.