)]}'
{"devstack/ovs_setup.sh":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"2fa74f7012daacc232ce68a37b77d299dd8ff272","unresolved":false,"context_lines":[{"line_number":156,"context_line":"        _neutron_ovs_base_setup_bridge $INTEGRATION_BRIDGE"},{"line_number":157,"context_line":"        sudo ovs-vsctl --no-wait set bridge $INTEGRATION_BRIDGE fail-mode\u003dsecure other-config:disable-in-band\u003dtrue"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"        # setup external bridge if necessary"},{"line_number":160,"context_line":"        check_dnat\u003d$(echo $DF_APPS_LIST | grep \"DNATApp\")"},{"line_number":161,"context_line":"        if [[ \"$check_dnat\" !\u003d \"\" ]]; then"},{"line_number":162,"context_line":"            sudo ovs-vsctl add-br $PUBLIC_BRIDGE || true"},{"line_number":163,"context_line":"        fi"},{"line_number":164,"context_line":"    fi"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    cd $_pwd"}],"source_content_type":"text/x-sh","patch_set":8,"id":"bab6814e_0e8c207e","line":163,"range":{"start_line":159,"start_character":0,"end_line":163,"end_character":10},"updated":"2016-05-25 13:06:50.000000000","message":"Consider moving this to a different patch, so it can be merged faster. I think this is an important change that should be merged before the version tag.","commit_id":"c7895dc8c8f207776b7b30a30baf1c75e9d825e1"},{"author":{"_account_id":7805,"name":"Li Ma","email":"skywalker.nick@gmail.com","username":"Nick"},"change_message_id":"62fd951e91b2180d88a57972c671cc4547a7c635","unresolved":false,"context_lines":[{"line_number":156,"context_line":"        _neutron_ovs_base_setup_bridge $INTEGRATION_BRIDGE"},{"line_number":157,"context_line":"        sudo ovs-vsctl --no-wait set bridge $INTEGRATION_BRIDGE fail-mode\u003dsecure other-config:disable-in-band\u003dtrue"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"        # setup external bridge if necessary"},{"line_number":160,"context_line":"        check_dnat\u003d$(echo $DF_APPS_LIST | grep \"DNATApp\")"},{"line_number":161,"context_line":"        if [[ \"$check_dnat\" !\u003d \"\" ]]; then"},{"line_number":162,"context_line":"            sudo ovs-vsctl add-br $PUBLIC_BRIDGE || true"},{"line_number":163,"context_line":"        fi"},{"line_number":164,"context_line":"    fi"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    cd $_pwd"}],"source_content_type":"text/x-sh","patch_set":8,"id":"bab6814e_dbcc748b","line":163,"range":{"start_line":159,"start_character":0,"end_line":163,"end_character":10},"in_reply_to":"bab6814e_0e8c207e","updated":"2016-05-26 02:01:29.000000000","message":"When I tried to add the external bridge, I noticed some other problems related with integration bridge. So I fixed them together. You mean I need to separate them into three different patches? If so, I\u0027ll do it soon.","commit_id":"c7895dc8c8f207776b7b30a30baf1c75e9d825e1"}],"dragonflow/tests/fullstack/test_apps.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"2598b34a224db468f955ed11bb63b30047719d2d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"            test_utils.print_command([\u0027ip\u0027, \u0027addr\u0027])"},{"line_number":44,"context_line":"            test_utils.print_command([\u0027ovs-vsctl\u0027, \u0027show\u0027], True)"},{"line_number":45,"context_line":"            test_utils.print_command("},{"line_number":46,"context_line":"                [\u0027ovs-ofctl\u0027, \u0027show\u0027, self.conf.integration_bridge],"},{"line_number":47,"context_line":"                True"},{"line_number":48,"context_line":"            )"},{"line_number":49,"context_line":"            test_utils.print_command("}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_edeb3bda","line":46,"range":{"start_line":46,"start_character":43,"end_line":46,"end_character":47},"updated":"2016-05-23 04:14:21.000000000","message":"nit: Consider changing to self.integration_bridge.","commit_id":"61f7d7237084f057c5f4db7672f5cbe967cf93f0"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"2598b34a224db468f955ed11bb63b30047719d2d","unresolved":false,"context_lines":[{"line_number":47,"context_line":"                True"},{"line_number":48,"context_line":"            )"},{"line_number":49,"context_line":"            test_utils.print_command("},{"line_number":50,"context_line":"                [\u0027ovs-ofctl\u0027, \u0027dump-flows\u0027, self.conf.integration_bridge],"},{"line_number":51,"context_line":"                True"},{"line_number":52,"context_line":"            )"},{"line_number":53,"context_line":"            test_utils.print_command("}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_4dbd27d7","line":50,"range":{"start_line":50,"start_character":49,"end_line":50,"end_character":53},"updated":"2016-05-23 04:14:21.000000000","message":"nit: Same here.","commit_id":"61f7d7237084f057c5f4db7672f5cbe967cf93f0"}]}