)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"c84189f91b336a1735910ce4b38bdf124b541459","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Adding constrain for NeutronOVSFirewallDriver in OVS-DPDK deployments"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For OVS-DPDK deployments, setting iptables as firewall driver"},{"line_number":10,"context_line":"is not suitable. Hence, added a constrain in compute node"},{"line_number":11,"context_line":"heat-templates to allow only native openvswitch"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8f788f5a3c08567e6af8871092fdeecd290ede2c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b38c4b7a_bd62352b","line":10,"range":{"start_line":10,"start_character":32,"end_line":10,"end_character":57},"updated":"2021-11-29 12:31:50.000000000","message":"I couldn\u0027t see how this add a constrain for compute nodes?","commit_id":"d9bdcfdc0a7ea43afde85673b7f661387b2fdba3"},{"author":{"_account_id":33688,"name":"Vijayalakshmi","display_name":"Viji Candappa","email":"vcandapp@redhat.com","username":"VijiC"},"change_message_id":"c8ffd0c5844ea441067ec3e898d5a8e15aac63f8","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Adding constrain for NeutronOVSFirewallDriver in OVS-DPDK deployments"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For OVS-DPDK deployments, setting iptables as firewall driver"},{"line_number":10,"context_line":"is not suitable. Hence, added a constrain in compute node"},{"line_number":11,"context_line":"heat-templates to allow only native openvswitch"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8f788f5a3c08567e6af8871092fdeecd290ede2c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"57d40cef_b2c17b04","line":10,"range":{"start_line":10,"start_character":32,"end_line":10,"end_character":57},"in_reply_to":"b38c4b7a_bd62352b","updated":"2021-11-29 12:58:01.000000000","message":"The recommended method to set firewall options via template is by adding this config in network-environment.yaml \nWhen overcloud is deloyed, this environment will be be referred and the constrains will be applied","commit_id":"d9bdcfdc0a7ea43afde85673b7f661387b2fdba3"}],"deployment/neutron/neutron-ovs-dpdk-agent-container-puppet.yaml":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"44101b0909aebc78881df10460d801c910c3517d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"      empty string will result in a default supported configuration."},{"line_number":44,"context_line":"    type: string"},{"line_number":45,"context_line":"    constraints:"},{"line_number":46,"context_line":"      - allowed_values: [\u0027noop\u0027, \u0027openvswitch\u0027]"},{"line_number":47,"context_line":"  # below parameters has to be set in neutron agent only for compute nodes."},{"line_number":48,"context_line":"  # as of now there is no other usecase for these parameters except dpdk."},{"line_number":49,"context_line":"  # should be moved to compute only ovs agent in case of any other usecases."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b7e3a86a_e775660a","line":46,"updated":"2021-12-14 18:08:40.000000000","message":"does \u0027\u0027 need to be in the allowed values here?","commit_id":"5e3ac51764bfe221a7d619dda54cd4fdb7935df6"}]}