Fixes missing firewall rules for neutron_ovs_dpdk_agent service
authorTim Rozet <trozet@redhat.com>
Wed, 22 Mar 2017 14:52:03 +0000 (10:52 -0400)
committerTim Rozet <trozet@redhat.com>
Wed, 22 Mar 2017 14:52:03 +0000 (10:52 -0400)
Firewall config was being inherited by the dpdk service, however
since the firewall service name was the parent (neutron_ovs_agent)
and technically that service was not enabled - the rules were never
applied.  This modifies the service name as it is inherited using
map_replace.

Closes-Bug: 1674689

Change-Id: I6676205b8fc1fd578cb2435ad97fe577a9e81d95
Signed-off-by: Tim Rozet <trozet@redhat.com>
puppet/services/neutron-ovs-dpdk-agent.yaml
releasenotes/notes/fix-neutron-dpdk-firewall-436aee39a0d7ed65.yaml [new file with mode: 0644]

index e25bc49..2c7ab57 100644 (file)
@@ -69,7 +69,10 @@ outputs:
       service_name: neutron_ovs_dpdk_agent
       config_settings:
         map_merge:
-          - get_attr: [NeutronOvsAgent, role_data, config_settings]
+          - map_replace:
+            - get_attr: [NeutronOvsAgent, role_data, config_settings]
+            - keys:
+                tripleo.neutron_ovs_agent.firewall_rules: tripleo.neutron_ovs_dpdk_agent.firewall_rules
           - neutron::agents::ml2::ovs::enable_dpdk: true
             neutron::agents::ml2::ovs::datapath_type: {get_param: NeutronDatapathType}
             neutron::agents::ml2::ovs::vhostuser_socket_dir: {get_param: NeutronVhostuserSocketDir}
diff --git a/releasenotes/notes/fix-neutron-dpdk-firewall-436aee39a0d7ed65.yaml b/releasenotes/notes/fix-neutron-dpdk-firewall-436aee39a0d7ed65.yaml
new file mode 100644 (file)
index 0000000..bb18aed
--- /dev/null
@@ -0,0 +1,5 @@
+---
+fixes:
+  - Fixes firewall rules from neutron OVS agent not being
+    inherited correctly and applied in neutron OVS DPDK
+    template.