feat: Notifications submodule buckets ARN generation in alternate partitions (e.g. GovCloud)
tculp opened this issue · comments
Is your request related to a problem? Please describe.
The notifications submodule generates the bucket ARN with
locals {
bucket_arn = coalesce(var.bucket_arn, "arn:aws:s3:::${var.bucket}")
...
However, this does not work when the partition of the ARN is not aws
, e.g. in GovCloud.
Describe the solution you'd like.
The module should use the aws_partition data source to pull the current provider's partition.
Describe alternatives you've considered.
Specifying the bucket ARN directly.
This is strange because you then need to specify both the bucket
and bucket_arn
variables, since neither is computed from the other, and both are used in different resources.
I noticed after creating this feature request that the bucket_arn variable also exists, so I updated the request accordingly.
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.