)]}'
{"quantum/plugins/openvswitch/agent/ovs_quantum_agent.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"e5dfb0cd5a4e66301e23d3c181905734e70b9d8e","unresolved":false,"context_lines":[{"line_number":808,"context_line":"        bridge_mappings\u003dbridge_mappings,"},{"line_number":809,"context_line":"        root_helper\u003dconfig.AGENT.root_helper,"},{"line_number":810,"context_line":"        polling_interval\u003dconfig.AGENT.polling_interval,"},{"line_number":811,"context_line":"        enable_tunneling\u003dconfig.OVS.enable_tunneling,"},{"line_number":812,"context_line":"        tunnel_type\u003dconfig.AGENT.tunnel_type,"},{"line_number":813,"context_line":"    )"},{"line_number":814,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Fj6I%3D","line":811,"updated":"2013-06-27 19:00:20.000000000","message":"Lines 815-818 should be enough to provide good config file backwards compatibility.  We don\u0027t have worry about the class init since this is internal code.","commit_id":"e53db058d6d09a8260e29c7373a6d73b5b14e982"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"e5dfb0cd5a4e66301e23d3c181905734e70b9d8e","unresolved":false,"context_lines":[{"line_number":813,"context_line":"    )"},{"line_number":814,"context_line":""},{"line_number":815,"context_line":"    # If enable_tunneling is TRUE, set tunnel_type to default to GRE"},{"line_number":816,"context_line":"    if kwargs[\u0027enable_tunneling\u0027] and not kwargs[\u0027tunnel_type\u0027]:"},{"line_number":817,"context_line":"        kwargs[\u0027tunnel_type\u0027] \u003d constants.TYPE_GRE"},{"line_number":818,"context_line":""},{"line_number":819,"context_line":"    if kwargs[\u0027tunnel_type\u0027] in constants.TUNNEL_NETWORK_TYPES:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Fj6A%3D","line":816,"updated":"2013-06-27 19:00:20.000000000","message":"Consider slight tweak:\n\nif config.OVS.enable_tunneling and not kwargs[\u0027tunnel_type\u0027]:","commit_id":"e53db058d6d09a8260e29c7373a6d73b5b14e982"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"unresolved":false,"context_lines":[{"line_number":813,"context_line":"    )"},{"line_number":814,"context_line":""},{"line_number":815,"context_line":"    # If enable_tunneling is TRUE, set tunnel_type to default to GRE"},{"line_number":816,"context_line":"    if kwargs[\u0027enable_tunneling\u0027] and not kwargs[\u0027tunnel_type\u0027]:"},{"line_number":817,"context_line":"        kwargs[\u0027tunnel_type\u0027] \u003d constants.TYPE_GRE"},{"line_number":818,"context_line":""},{"line_number":819,"context_line":"    if kwargs[\u0027tunnel_type\u0027] in constants.TUNNEL_NETWORK_TYPES:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Fj4k%3D","line":816,"in_reply_to":"AAAAPn%2F%2Fj6A%3D","updated":"2013-06-27 19:06:30.000000000","message":"Done","commit_id":"e53db058d6d09a8260e29c7373a6d73b5b14e982"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"3caddd40a5d7769e64a6ee87db07f8b0684e087d","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        if tunnel_type in constants.TUNNEL_NETWORK_TYPES:"},{"line_number":174,"context_line":"            self.enable_tunneling \u003d True"},{"line_number":175,"context_line":"        else:"},{"line_number":176,"context_line":"            self.enable_tunneling \u003d False"},{"line_number":177,"context_line":"        self.local_ip \u003d local_ip"},{"line_number":178,"context_line":"        self.tunnel_count \u003d 0"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FjYo%3D","line":175,"updated":"2013-06-27 21:41:08.000000000","message":"I think the problem is here.","commit_id":"92c6a745c224da96631f8c0f43d88e9f560cfcfa"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        if tunnel_type in constants.TUNNEL_NETWORK_TYPES:"},{"line_number":174,"context_line":"            self.enable_tunneling \u003d True"},{"line_number":175,"context_line":"        else:"},{"line_number":176,"context_line":"            self.enable_tunneling \u003d False"},{"line_number":177,"context_line":"        self.local_ip \u003d local_ip"},{"line_number":178,"context_line":"        self.tunnel_count \u003d 0"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FjW4%3D","line":175,"in_reply_to":"AAAAPn%2F%2FjYo%3D","updated":"2013-06-27 21:53:23.000000000","message":"I disagree, the cleaner solution is to address this when building the agent configuration map.","commit_id":"92c6a745c224da96631f8c0f43d88e9f560cfcfa"}]}