)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"2fc70d0bc90b7179e1c84a5892860777b2fe660b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2017-07-04 15:53:40 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improve with_nb_objects"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I647479dc35c8caf7773f99a9bb6a069ffe6ada41"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f1d235d_da9cba0f","line":8,"updated":"2017-07-04 17:07:08.000000000","message":"Please add a small description of the improvement","commit_id":"2240d81e9ccc3c803d48c03a34b078441cb9c409"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"d00af348ca334afdbdcb9017bfd59ada1c9e4416","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2017-07-04 15:53:40 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improve with_nb_objects"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I647479dc35c8caf7773f99a9bb6a069ffe6ada41"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f1d235d_6015c2e4","line":8,"in_reply_to":"3f1d235d_da9cba0f","updated":"2017-07-06 05:15:18.000000000","message":"Done","commit_id":"2240d81e9ccc3c803d48c03a34b078441cb9c409"}],"dragonflow/tests/unit/test_dnat_app.py":[{"author":{"_account_id":26131,"name":"Eyal Leshem","email":"eyal.leshem@toganetworks.com","username":"leyal"},"change_message_id":"8d5a55ab59aef27638fb4c2718c8936db72ba69f","unresolved":false,"context_lines":[{"line_number":113,"context_line":"                test_app_base.fake_ovs_port1)"},{"line_number":114,"context_line":"            mock_func.assert_not_called()"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    @utils.with_nb_objects(test_app_base.fake_floating_port)"},{"line_number":117,"context_line":"    def test_floatingip_ovs_port_removed(self):"},{"line_number":118,"context_line":"        self.controller.update(test_app_base.fake_local_port1)"},{"line_number":119,"context_line":"        self.controller.topology.ovs_port_updated(test_app_base.fake_ovs_port1)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_dc47014c","line":116,"updated":"2017-07-05 11:00:53.000000000","message":"Why do you need that object in the nb-DB ?","commit_id":"2240d81e9ccc3c803d48c03a34b078441cb9c409"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"2783b016932fb28321b790c85cabe31db6f5b90e","unresolved":false,"context_lines":[{"line_number":113,"context_line":"                test_app_base.fake_ovs_port1)"},{"line_number":114,"context_line":"            mock_func.assert_not_called()"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    @utils.with_nb_objects(test_app_base.fake_floating_port)"},{"line_number":117,"context_line":"    def test_floatingip_ovs_port_removed(self):"},{"line_number":118,"context_line":"        self.controller.update(test_app_base.fake_local_port1)"},{"line_number":119,"context_line":"        self.controller.topology.ovs_port_updated(test_app_base.fake_ovs_port1)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_e010b202","line":116,"in_reply_to":"3f1d235d_dc47014c","updated":"2017-07-06 05:13:36.000000000","message":"The update in line 120 causes it to be looked up.","commit_id":"2240d81e9ccc3c803d48c03a34b078441cb9c409"}],"dragonflow/tests/unit/test_topology.py":[{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"e6a0baa5d9bbbc02a6d12ad428bb41e2224b6aae","unresolved":false,"context_lines":[{"line_number":124,"context_line":"        # Another port online"},{"line_number":125,"context_line":"        with utils.with_nb_objects(test_app_base.fake_logic_switch1,"},{"line_number":126,"context_line":"                                   test_app_base.fake_local_port1,"},{"line_number":127,"context_line":"                                   test_app_base.fake_local_port2).context("},{"line_number":128,"context_line":"                                        self):"},{"line_number":129,"context_line":"            self.controller.update \u003d mock.Mock()"},{"line_number":130,"context_line":"            self.topology.ovs_port_updated(test_app_base.fake_ovs_port2)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_20690af8","line":127,"range":{"start_line":127,"start_character":67,"end_line":127,"end_character":75},"updated":"2017-07-06 05:36:22.000000000","message":"Why not implement __enter__/__exit__ if we\u0027re doing that.. I don\u0027t mind if you decide against.","commit_id":"afc2c80a43ac5b6fa3a76bbe8420c413481a9114"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"413aa69b59a3b13dd8be67096447f60276885e86","unresolved":false,"context_lines":[{"line_number":124,"context_line":"        # Another port online"},{"line_number":125,"context_line":"        with utils.with_nb_objects(test_app_base.fake_logic_switch1,"},{"line_number":126,"context_line":"                                   test_app_base.fake_local_port1,"},{"line_number":127,"context_line":"                                   test_app_base.fake_local_port2).context("},{"line_number":128,"context_line":"                                        self):"},{"line_number":129,"context_line":"            self.controller.update \u003d mock.Mock()"},{"line_number":130,"context_line":"            self.topology.ovs_port_updated(test_app_base.fake_ovs_port2)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_211ad716","line":127,"range":{"start_line":127,"start_character":67,"end_line":127,"end_character":75},"in_reply_to":"3f1d235d_20690af8","updated":"2017-07-09 06:39:07.000000000","message":"As a future improvement, that can be done. Since we don\u0027t *really* need to pass self - we can get nb_api from get_instance.","commit_id":"afc2c80a43ac5b6fa3a76bbe8420c413481a9114"}]}