)]}'
{"dragonflow/tests/fullstack/test_l2_responder.py":[{"author":{"_account_id":11343,"name":"Gal Sagie","email":"gal.sagie@gmail.com","username":"GalSagie"},"change_message_id":"c56732993f4d845b75fe2dbd39d006cb261d9833","unresolved":false,"context_lines":[{"line_number":94,"context_line":"                self.assertIsNone("},{"line_number":95,"context_line":"                    self._find_arp_responder_flow_by_ip(flows_delta, ip)"},{"line_number":96,"context_line":"                )"},{"line_number":97,"context_line":"            self.assertTrue(self._wait_for_flow_removal(flows_before, 30))"},{"line_number":98,"context_line":"        finally:"},{"line_number":99,"context_line":"            try:"},{"line_number":100,"context_line":"                vm.delete()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a57f1b5_a5af1893","line":97,"updated":"2016-02-11 10:01:47.000000000","message":"This is a bit racy test, i know its not related directly to your change but i notice it now.\nIf another test that effect the ARP table flows runs concurrently we might fail here.\n\nI think the correct thing to do is just to check the flow with that IP is removed.","commit_id":"8e42892bdf2758c310dbc0f4713a8e8ce599ba04"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"e39417f2c351e4ae799759a30eb8df0106b7172f","unresolved":false,"context_lines":[{"line_number":94,"context_line":"                self.assertIsNone("},{"line_number":95,"context_line":"                    self._find_arp_responder_flow_by_ip(flows_delta, ip)"},{"line_number":96,"context_line":"                )"},{"line_number":97,"context_line":"            self.assertTrue(self._wait_for_flow_removal(flows_before, 30))"},{"line_number":98,"context_line":"        finally:"},{"line_number":99,"context_line":"            try:"},{"line_number":100,"context_line":"                vm.delete()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a57f1b5_2573284f","line":97,"in_reply_to":"3a57f1b5_a5af1893","updated":"2016-02-11 10:03:52.000000000","message":"Done.\nThat\u0027s what this patch is supposed to do - if the _wait_for_flow_removal method fails, fallback to verifying our flow is removed.","commit_id":"8e42892bdf2758c310dbc0f4713a8e8ce599ba04"}]}