Merge pull request #22 from timhassett/master

Added optional name key for ListenerRules
theonestack · Mar 12, 2019 · 7ba356b · 7ba356b
2 parents f24ccfe + 8e79044
commit 7ba356b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/ecs-service.cfndsl.rb b/ecs-service.cfndsl.rb
@@ -251,7 +251,15 @@
           listener_conditions << { Field: "host-header", Values: hosts }
         end
 
-        ElasticLoadBalancingV2_ListenerRule("TargetRule#{rule['priority']}") do
+        if rule.key?("name")
+          rule_name = rule['name']
+        elsif rule['priority'].is_a? Integer
+          rule_name = "TargetRule#{rule['priority']}"
+        else
+          rule_name = "TargetRule#{index}"
+        end
+
+        ElasticLoadBalancingV2_ListenerRule(rule_name) do
           Actions [{ Type: "forward", TargetGroupArn: Ref('TaskTargetGroup') }]
           Conditions listener_conditions
           ListenerArn Ref("Listener")

diff --git a/ecs-service.config.yaml b/ecs-service.config.yaml
@@ -50,7 +50,8 @@ log_retention: 7
 #     path: /status
 #     code: 200
 #   rules:
-#     - path: /v2/*
+#     - name:  apiv2
+#       path: /v2/*
 #       host: api.*
 #       priority: 10
 #     - path: /api/v1/*