generated from clowdhaus/terraform-aws-module-template
-
-
Notifications
You must be signed in to change notification settings - Fork 34
Closed
Labels
Description
Is your request related to a new offering from AWS?
Is this functionality available in the 5.100.0 & 6.16
- Yes ✅: aws_prometheus_workspace_configuration Since 5.100.0: https://github.com/hashicorp/terraform-provider-aws/blob/release/5.x/CHANGELOG.md#51000-june-12-2025
- Yes ✅: aws_prometheus_resource_policy Since 6.16: https://github.com/hashicorp/terraform-provider-aws/blob/main/CHANGELOG.md#6160-october-9-2025
Is your request related to a problem? Please describe.
I'd like to include support of aws_prometheus_workspace_configuration and aws_prometheus_resource_policy for example usecases:
aws_prometheus_workspace_configuration:
- Gives an ability to set metrics retention on workspace
- Limits per labels set
aws_prometheus_resource_policy:
- Adds ability to attach policy for cross-account access
Describe the solution you'd like.
- Adds variable to set retention as
retention_period_in_days, not required, no default value (to inherit AWS default, share your opinion) - Adds variable to allow user to pass
limits_per_label_setin a map, then creates multiple blocks by for_each + dynamic - Allows to attach a policy with a switch
attach_policybool, and a variablepolicyto attach custom user defined policy
Describe alternatives you've considered.
Not applicable
Additional context
I'm willing to open a PR for these changes, their size isn't big and not problematic to implement, I'm opening this FR first to hear maintainers opinion about these features