Do not print access_logs if disabled

dtan4 · Dec 20, 2016 · f73ebb8 · f73ebb8
1 parent 25278db
commit f73ebb8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/lib/terraforming/template/tf/alb.erb b/lib/terraforming/template/tf/alb.erb
@@ -10,12 +10,14 @@ resource "aws_alb" "<%= module_name_of(load_balancer) %>" {
 
     enable_deletion_protection = <%= load_balancer_attributes["deletion_protection.enabled"].to_s %>
 
+<%- if load_balancer_attributes["access_logs.s3.enabled"] == "true" -%>
     access_logs {
         bucket  = "<%= load_balancer_attributes["access_logs.s3.bucket"] %>"
         enabled = <%= load_balancer_attributes["access_logs.s3.enabled"] %>
         prefix  = "<%= load_balancer_attributes["access_logs.s3.prefix"] %>"
     }
 
+<%- end -%>
     tags {
 <% tags.each do |tag| -%>
         "<%= tag.key %>" = "<%= tag.value %>"

diff --git a/spec/lib/terraforming/resource/alb_spec.rb b/spec/lib/terraforming/resource/alb_spec.rb
@@ -128,12 +128,6 @@ module Resource
 
     enable_deletion_protection = true
 
-    access_logs {
-        bucket  = "my-elb-logs"
-        enabled = false
-        prefix  = "fuga"
-    }
-
     tags {
     }
 }