)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"5cbb52adcb3d29b67e898445f3927e04553935a9","unresolved":false,"context_lines":[{"line_number":11,"context_line":"plugged directly into br-ex, but allows l2 agent to do the wiring."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"There are known issues with setting it to br-ex (like wrong port"},{"line_number":14,"context_line":"statuses): If533cf7c4c379be78f5a15073accaff7f65973ab"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The only caveat to setting it to the default (\u0027\u0027) value is that in that"},{"line_number":17,"context_line":"case l2 agent should be configured with bridge mapping for physical"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a71b18c_c9f39071","line":14,"updated":"2016-12-07 09:19:10.000000000","message":"I\u0027ve also seen issues with non-flat networks too (vlan tagged), those don\u0027t get tagged/untagged","commit_id":"4439f8655c2de915840e0b7e1c0ad11215d24b78"}],"puppet/services/neutron-l3-compute-dvr.yaml":[{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"1dc7224c5b803cf0d6fb6750efcdd5149c3e5198","unresolved":false,"context_lines":[{"line_number":60,"context_line":"      config_settings:"},{"line_number":61,"context_line":"        map_merge:"},{"line_number":62,"context_line":"          - get_attr: [NeutronBase, role_data, config_settings]"},{"line_number":63,"context_line":"            neutron::agents::l3::agent_mode : \u0027dvr\u0027"},{"line_number":64,"context_line":"        - "},{"line_number":65,"context_line":"          if:"},{"line_number":66,"context_line":"          - external_network_bridge_empty"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a71b18c_746b1d86","line":63,"updated":"2016-12-07 15:35:00.000000000","message":"stick a hyphen on the beginning of this line and you should be good for the pep8 error.","commit_id":"2f132104d6ea7cb66ea4e64c600bcb8ebd0e4615"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c7f65e64979301773c8b7f4564dffdd7887067ba","unresolved":false,"context_lines":[{"line_number":60,"context_line":"      config_settings:"},{"line_number":61,"context_line":"        map_merge:"},{"line_number":62,"context_line":"          - get_attr: [NeutronBase, role_data, config_settings]"},{"line_number":63,"context_line":"            neutron::agents::l3::agent_mode : \u0027dvr\u0027"},{"line_number":64,"context_line":"        - "},{"line_number":65,"context_line":"          if:"},{"line_number":66,"context_line":"          - external_network_bridge_empty"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a71b18c_a478fc90","line":63,"in_reply_to":"3a71b18c_746b1d86","updated":"2016-12-08 11:33:18.000000000","message":"Done","commit_id":"2f132104d6ea7cb66ea4e64c600bcb8ebd0e4615"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c613a366f587a567177bbe65f24f987e3fbcdc3c","unresolved":false,"context_lines":[{"line_number":32,"context_line":"      path: /var/log/neutron/l3-agent.log"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"  # DEPRECATED: the following options are deprecated and are currently maintained"},{"line_number":35,"context_line":"  # for backwards compatibility. They will be removed in the Ocata cycle."},{"line_number":36,"context_line":"  NeutronExternalNetworkBridge:"},{"line_number":37,"context_line":"    description: Name of bridge used for external network traffic. Usually L2"},{"line_number":38,"context_line":"                 agent handles port wiring into external bridge, and hence the"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3a71b18c_1aa479a4","line":35,"updated":"2016-12-09 09:50:48.000000000","message":"or should it say Pike at this point?","commit_id":"5fd0f5c3b088d4bdd6262b9697d77cd40698e4bf"}],"puppet/services/neutron-l3.yaml":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"8993ea1a459e77fdaec247e0d682a6046d4263b9","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    type: string"},{"line_number":23,"context_line":"    default: \u0027\u0027"},{"line_number":24,"context_line":"  NeutronExternalNetworkBridge:"},{"line_number":25,"context_line":"    description: Name of bridge used for external network traffic."},{"line_number":26,"context_line":"    type: string"},{"line_number":27,"context_line":"    default: \u0027\u0027"},{"line_number":28,"context_line":"  NeutronL3AgentMode:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a71b18c_e9c84caa","line":25,"updated":"2016-12-07 09:09:07.000000000","message":"Maybe description should discourage setting it; and maybe it should be moved into deprecation options list.","commit_id":"4439f8655c2de915840e0b7e1c0ad11215d24b78"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"ed4df7b1fe8c6879ad24eb72cc50e2b9d7821d1a","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    type: string"},{"line_number":23,"context_line":"    default: \u0027\u0027"},{"line_number":24,"context_line":"  NeutronExternalNetworkBridge:"},{"line_number":25,"context_line":"    description: Name of bridge used for external network traffic."},{"line_number":26,"context_line":"    type: string"},{"line_number":27,"context_line":"    default: \u0027\u0027"},{"line_number":28,"context_line":"  NeutronL3AgentMode:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a71b18c_0ca7560b","line":25,"in_reply_to":"3a71b18c_e9c84caa","updated":"2016-12-07 09:24:53.000000000","message":"Definitely, I agree.","commit_id":"4439f8655c2de915840e0b7e1c0ad11215d24b78"},{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"1dc7224c5b803cf0d6fb6750efcdd5149c3e5198","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  NeutronExternalNetworkBridge:"},{"line_number":25,"context_line":"    description: Name of bridge used for external network traffic."},{"line_number":26,"context_line":"    type: string"},{"line_number":27,"context_line":"    default: \u0027\u0027"},{"line_number":28,"context_line":"  NeutronL3AgentMode:"},{"line_number":29,"context_line":"    description: |"},{"line_number":30,"context_line":"      Agent mode for L3 agent. Must be one of legacy or dvr_snat."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a71b18c_b42935f5","line":27,"updated":"2016-12-07 15:35:00.000000000","message":"Agree should be moved to deprecated parameter list.","commit_id":"2f132104d6ea7cb66ea4e64c600bcb8ebd0e4615"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c7f65e64979301773c8b7f4564dffdd7887067ba","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  NeutronExternalNetworkBridge:"},{"line_number":25,"context_line":"    description: Name of bridge used for external network traffic."},{"line_number":26,"context_line":"    type: string"},{"line_number":27,"context_line":"    default: \u0027\u0027"},{"line_number":28,"context_line":"  NeutronL3AgentMode:"},{"line_number":29,"context_line":"    description: |"},{"line_number":30,"context_line":"      Agent mode for L3 agent. Must be one of legacy or dvr_snat."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a71b18c_a4ed5c27","line":27,"in_reply_to":"3a71b18c_b42935f5","updated":"2016-12-08 11:33:18.000000000","message":"Done","commit_id":"2f132104d6ea7cb66ea4e64c600bcb8ebd0e4615"}]}