generated from clowdhaus/terraform-aws-module-template
-
-
Notifications
You must be signed in to change notification settings - Fork 34
Closed
Description
Is your request related to a new offering from AWS?
No – the functionality already exists in the AWS provider. This is about adding flexibility in the Terraform module implementation.
Is your request related to a problem? Please describe.
Currently, the module always provisions an Alert Manager along with the AMP workspace.
In some cases, users already have external alerting systems or do not want to configure Alert Manager at all.
This results in unnecessary resources being created and managed, adding extra overhead and confusion.
Describe the solution you'd like.
Introduce a new input variable, for example:
variable "create_alert_manager" {
description = "Controls whether an Alert Manager definition is created along with the AMP workspace."
type = bool
default = true
}The module should only create the Alert Manager resources when create_alert_manager = true.
Describe alternatives you've considered.
- Forking the module and removing the Alert Manager resource (not ideal, since it loses upstream updates).
- Using count = 0 directly on the alert manager resource within the module (not possible without editing the module).
Additional context
This change would improve flexibility and allow users to adopt the module without being forced to create unnecessary resources.
I’d be happy to open a PR implementing this change.
Metadata
Metadata
Assignees
Labels
No labels