)]}'
{"octavia/controller/worker/tasks/database_tasks.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"f5ae79ff5397cb7f863d218c2f5ce000da4dcd33","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        :returns: None"},{"line_number":137,"context_line":"        \"\"\""},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        LOG.debug(\"DB delete member for id: %s \" %"},{"line_number":140,"context_line":"                  member.id)"},{"line_number":141,"context_line":"        self.member_repo.delete(db_apis.get_session(), id\u003dmember.id)"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa80f949_560ad227","line":139,"updated":"2015-11-04 12:27:06.000000000","message":"I thought we were supposed to let the logging functions handle the string replacements? I\u0027m not sure if this warrants a -1 or not, probably only a minor issue.","commit_id":"28a94a071b74c6d44720f979224c0a9f1dd8b94d"},{"author":{"_account_id":8157,"name":"Bertrand LALLAU","email":"bertrand.lallau@gmail.com","username":"blallau"},"change_message_id":"1dc9315f7a5b2d69ae7f8085b9734b33e3d30e4d","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        :returns: None"},{"line_number":137,"context_line":"        \"\"\""},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        LOG.debug(\"DB delete member for id: %s \" %"},{"line_number":140,"context_line":"                  member.id)"},{"line_number":141,"context_line":"        self.member_repo.delete(db_apis.get_session(), id\u003dmember.id)"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"da85f559_e3a34dfe","line":139,"in_reply_to":"fa80f949_560ad227","updated":"2015-11-12 14:45:10.000000000","message":"Done","commit_id":"28a94a071b74c6d44720f979224c0a9f1dd8b94d"}],"octavia/tests/unit/controller/worker/flows/test_health_monitor_flows.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"f5ae79ff5397cb7f863d218c2f5ce000da4dcd33","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        self.assertIn(\u0027listener\u0027, health_mon_flow.requires)"},{"line_number":37,"context_line":"        self.assertIn(\u0027loadbalancer\u0027, health_mon_flow.requires)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        self.assertEqual(2, len(health_mon_flow.requires), 2)"},{"line_number":40,"context_line":"        self.assertEqual(0, len(health_mon_flow.provides), 0)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa80f949_113214f2","line":38,"updated":"2015-11-04 12:27:06.000000000","message":"Why are you overriding the messages here and nowhere else?","commit_id":"28a94a071b74c6d44720f979224c0a9f1dd8b94d"},{"author":{"_account_id":8157,"name":"Bertrand LALLAU","email":"bertrand.lallau@gmail.com","username":"blallau"},"change_message_id":"1dc9315f7a5b2d69ae7f8085b9734b33e3d30e4d","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        self.assertIn(\u0027listener\u0027, health_mon_flow.requires)"},{"line_number":37,"context_line":"        self.assertIn(\u0027loadbalancer\u0027, health_mon_flow.requires)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        self.assertEqual(2, len(health_mon_flow.requires), 2)"},{"line_number":40,"context_line":"        self.assertEqual(0, len(health_mon_flow.provides), 0)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"da85f559_03f531eb","line":38,"in_reply_to":"fa80f949_113214f2","updated":"2015-11-12 14:45:10.000000000","message":"good catch !!!","commit_id":"28a94a071b74c6d44720f979224c0a9f1dd8b94d"}],"octavia/tests/unit/controller/worker/tasks/test_amphora_driver_tasks.py":[{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"cacce48010443842ec3e568b8b414f9fa341f383","unresolved":false,"context_lines":[{"line_number":34,"context_line":"_load_balancer_mock \u003d mock.MagicMock()"},{"line_number":35,"context_line":"_load_balancer_mock.id \u003d LB_ID"},{"line_number":36,"context_line":"_vip_mock \u003d mock.MagicMock()"},{"line_number":37,"context_line":"_load_balancer_mock.vip \u003d _vip_mock"},{"line_number":38,"context_line":"_LB_mock \u003d mock.MagicMock()"},{"line_number":39,"context_line":"_amphorae_mock \u003d [_amphora_mock]"},{"line_number":40,"context_line":"_network_mock \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a8ffd7b_7155794e","line":37,"updated":"2015-11-25 17:40:44.000000000","message":"_load_balancer_mock.vip is defined but never used, i found that in the following code, we still use _vip_mock,  in that case, it might be good to remove \t_load_balancer_mock.vip \u003d _vip_mock or replace _vip_mock with _load_balancer_mock.vip.","commit_id":"0932d6e16f52aa5128461de6566359d0e62bb3dd"},{"author":{"_account_id":8157,"name":"Bertrand LALLAU","email":"bertrand.lallau@gmail.com","username":"blallau"},"change_message_id":"9a1ad38a11e8f9846b822255c2217375448509aa","unresolved":false,"context_lines":[{"line_number":34,"context_line":"_load_balancer_mock \u003d mock.MagicMock()"},{"line_number":35,"context_line":"_load_balancer_mock.id \u003d LB_ID"},{"line_number":36,"context_line":"_vip_mock \u003d mock.MagicMock()"},{"line_number":37,"context_line":"_load_balancer_mock.vip \u003d _vip_mock"},{"line_number":38,"context_line":"_LB_mock \u003d mock.MagicMock()"},{"line_number":39,"context_line":"_amphorae_mock \u003d [_amphora_mock]"},{"line_number":40,"context_line":"_network_mock \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a8ffd7b_72b1f2e0","line":37,"in_reply_to":"9a8ffd7b_7155794e","updated":"2015-11-25 20:32:02.000000000","message":"Done","commit_id":"0932d6e16f52aa5128461de6566359d0e62bb3dd"},{"author":{"_account_id":8157,"name":"Bertrand LALLAU","email":"bertrand.lallau@gmail.com","username":"blallau"},"change_message_id":"9a1ad38a11e8f9846b822255c2217375448509aa","unresolved":false,"context_lines":[{"line_number":34,"context_line":"_load_balancer_mock \u003d mock.MagicMock()"},{"line_number":35,"context_line":"_load_balancer_mock.id \u003d LB_ID"},{"line_number":36,"context_line":"_vip_mock \u003d mock.MagicMock()"},{"line_number":37,"context_line":"_load_balancer_mock.vip \u003d _vip_mock"},{"line_number":38,"context_line":"_LB_mock \u003d mock.MagicMock()"},{"line_number":39,"context_line":"_amphorae_mock \u003d [_amphora_mock]"},{"line_number":40,"context_line":"_network_mock \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a8ffd7b_f2f2c2d1","line":37,"in_reply_to":"9a8ffd7b_7155794e","updated":"2015-11-25 20:32:02.000000000","message":"Done","commit_id":"0932d6e16f52aa5128461de6566359d0e62bb3dd"}]}