Cleanup the firewall logic.

We added code in t-h-t to strip empty services from the service_names
list. (These are often the result of a service set to OS::Heat::None).

As such we can now drop this puppet reject statement.

Change-Id: Ie66f14f183de7e44a1f69af862f7d4be9a14c904

diff --git a/manifests/firewall.pp b/manifests/firewall.pp
index edcb5e7..3184cd3 100644 (file)
--- a/manifests/firewall.pp
+++ b/manifests/firewall.pp
@@ -102,7 +102,7 @@ class tripleo::firewall(
     #     dport: 999
     #     proto: udp
     #     action: accept
-    $service_names = reject(hiera('service_names', []), '^$')
+    $service_names = hiera('service_names', [])
     tripleo::firewall::service_rules { $service_names: }
   }