Skip to content

Commit 80b8391

Browse files
jnoninoEvgeny Luvsandugar
andauthored
Contribution from @luvs (#29)
* Add deployment_controller * Add default to task definition Co-authored-by: Evgeny Luvsandugar <[email protected]>
1 parent 2568b9d commit 80b8391

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

main.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,12 @@ resource "aws_ecs_service" "service" {
9696
field = lookup(ordered_placement_strategy.value, "field", null)
9797
}
9898
}
99+
dynamic "deployment_controller" {
100+
for_each = var.deployment_controller
101+
content {
102+
type = deployment_controller.value.type
103+
}
104+
}
99105
dynamic "placement_constraints" {
100106
for_each = var.placement_constraints
101107
content {

variables.tf

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,12 @@ variable "ordered_placement_strategy" {
6161
default = []
6262
}
6363

64+
variable "deployment_controller" {
65+
description = "(Optional) Deployment controller"
66+
type = list
67+
default = []
68+
}
69+
6470
variable "placement_constraints" {
6571
type = list(any)
6672
description = "(Optional) rules that are taken into consideration during task placement. Maximum number of placement_constraints is 10. This is a list of maps, where each map should contain \"type\" and \"expression\""
@@ -85,6 +91,7 @@ variable "service_registries" {
8591

8692
variable "task_definition_arn" {
8793
description = "(Required) The full ARN of the task definition that you want to run in your service."
94+
default = ""
8895
}
8996

9097
variable "force_new_deployment" {

0 commit comments

Comments
 (0)