)]}'
{"neutron/plugins/ml2/drivers/openvswitch/mech_driver/mech_openvswitch.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fa5eea9d198bc4d0e93bd334bce9da11ba3e1077","unresolved":false,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        def set_bridge_name_inner(bridge_name):"},{"line_number":124,"context_line":"            vif_details[portbindings.VIF_DETAILS_BRIDGE_NAME] \u003d bridge_name"},{"line_number":125,"context_line":"        bridge \u003d agent[\u0027configurations\u0027].get(\u0027integration_bridge\u0027, \u0027br-int\u0027)"},{"line_number":126,"context_line":"        vif_details[portbindings.VIF_DETAILS_BRIDGE_NAME] \u003d bridge"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        registry.publish(a_const.OVS_BRIDGE_NAME, events.BEFORE_READ,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_42a8808a","line":125,"range":{"start_line":125,"start_character":67,"end_line":125,"end_character":75},"updated":"2018-08-27 21:38:03.000000000","message":"Do we want to have a default here?  Don\u0027t seem to do it with other values, and I couldn\u0027t find br-int anywhere in the code except for the conf file default.","commit_id":"da4938d35a8f49a10249dad533a2314787da7fc6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"06f093afaf783cbe50dacc113695a7708433cd39","unresolved":false,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        def set_bridge_name_inner(bridge_name):"},{"line_number":124,"context_line":"            vif_details[portbindings.VIF_DETAILS_BRIDGE_NAME] \u003d bridge_name"},{"line_number":125,"context_line":"        bridge \u003d agent[\u0027configurations\u0027].get(\u0027integration_bridge\u0027, \u0027br-int\u0027)"},{"line_number":126,"context_line":"        vif_details[portbindings.VIF_DETAILS_BRIDGE_NAME] \u003d bridge"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        registry.publish(a_const.OVS_BRIDGE_NAME, events.BEFORE_READ,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_7f8e6a2e","line":125,"in_reply_to":"","updated":"2018-08-28 08:47:19.000000000","message":"My reasoning for setting the br-int default was effectively during an upgrade this might provide better behaviour as not all node would have to be upgraded for this code to take effect assuming the default bridge name was used. I can make this condtional instead","commit_id":"da4938d35a8f49a10249dad533a2314787da7fc6"}]}