Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit 2383676

Browse files
authored
Merge pull request #15 from geekcell/alarm-config
fix: alarm configuration showing up in every change set
2 parents 57c9e80 + a08619d commit 2383676

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ preconfigured solution for seamless scalability and high availability."
149149

150150
## Resources
151151

152-
- resource.aws_cloudwatch_log_group.main (main.tf#310)
152+
- resource.aws_cloudwatch_log_group.main (main.tf#313)
153153
- resource.aws_codedeploy_app.main (main.tf#231)
154154
- resource.aws_codedeploy_deployment_group.main (main.tf#238)
155155
- resource.aws_ecs_service.main (main.tf#56)

‎main.tf‎

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -268,10 +268,13 @@ resource "aws_codedeploy_deployment_group" "main" {
268268
events = var.codedeploy_auto_rollback_events
269269
}
270270

271-
alarm_configuration {
272-
enabled = length(var.codedeploy_cloudwatch_alarm_names) > 0 ? true : false
273-
alarms = var.codedeploy_cloudwatch_alarm_names
274-
ignore_poll_alarm_failure = var.codedeploy_ignore_poll_alarm_failure
271+
dynamic "alarm_configuration" {
272+
for_each = length(var.codedeploy_cloudwatch_alarm_names) > 0 ? [true] : []
273+
274+
content {
275+
alarms = var.codedeploy_cloudwatch_alarm_names
276+
ignore_poll_alarm_failure = var.codedeploy_ignore_poll_alarm_failure
277+
}
275278
}
276279

277280
load_balancer_info {

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /