feat: Add the capability to disable alert manager creation #20
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds a new
create_alert_managervariable to the AWS Managed Service for Prometheus (AMP) Terraform module, allowing users to conditionally disable Alert Manager creation. The variable defaults to true to maintain backward compatibility.Motivation and Context
Resolves Add option to disable Alert Manager creation in AWS Managed Prometheus module #19
Currently, the module always provisions an Alert Manager along with the AMP workspace
Users with existing external alerting systems or those who don't want Alert Manager are forced to create unnecessary resources
This adds extra overhead and confusion for users who don't need Alert Manager functionality
Breaking Changes
None. This change is fully backward compatible:
How Has This Been Tested?