)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"41f312e56f4b1431119341f27d10e366e57bd551","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     JieLee \u003cJieLi2087@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-03 08:50:15 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug-1558888 reduce duplicate if statements in ovs_dvr_neutron_agent."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I7192e1c0447ea35649672caa771e5a9c6aa2636b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1af94dfe_64dbf24f","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":72},"updated":"2016-03-18 06:11:31.000000000","message":"Improve commit messages according with https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"b049ffad0f464e8d5a39d38f49aec8061dc7f076"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"41f312e56f4b1431119341f27d10e366e57bd551","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-01-03 08:50:15 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug-1558888 reduce duplicate if statements in ovs_dvr_neutron_agent."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I7192e1c0447ea35649672caa771e5a9c6aa2636b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1af94dfe_248a8a1b","line":8,"updated":"2016-03-18 06:11:31.000000000","message":"Closes-bug: #1558888","commit_id":"b049ffad0f464e8d5a39d38f49aec8061dc7f076"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"77a988dfce2a15b992b5422e49c509429ac43ff9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     JieLee \u003cJieLi2087@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-03 09:35:25 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"  Delete duplicated codes in in ovs_neutron_agent and ovs_dvr_neutron_agent."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"  Closes-bug: #1558888"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1af94dfe_f842edac","line":7,"updated":"2016-03-18 14:26:04.000000000","message":"nit: preceding blank spaces here and below","commit_id":"def94ef531391fc9510b5410f99725f0cf14e20f"},{"author":{"_account_id":11682,"name":"Ryan Moats","email":"rmoats@us.ibm.com","username":"regXboi"},"change_message_id":"3168031e277abb5449b1709a7da12cd11ba4227c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     JieLee \u003cJieLi2087@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-18 23:13:19 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Delete duplicated codes in in ovs_neutron_agent and ovs_dvr_neutron_agent."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-bug: #1558888"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_255cc0ec","line":7,"range":{"start_line":7,"start_character":24,"end_line":7,"end_character":27},"updated":"2016-03-22 02:39:18.000000000","message":"nit: duplicated word","commit_id":"bd8f1aa59ba630490a35d5b7af969bdfb81e09b0"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"f8bb29295984c5dc09cbc30100375416903ba711","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     JieLee \u003cJieLi2087@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-20 23:59:47 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Delete duplicated codes in ovs_neutron_agent and ovs_dvr_neutron_agent."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-bug: #1558888"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa0719c6_9f6cb736","line":7,"updated":"2016-03-23 09:01:20.000000000","message":"I think that the commit message does not well describe the situation. You are removing extra checks from ovs_dvr_neutron_agent that can be done in ovs_neutron_agent in one place.","commit_id":"46ad0596cee3bdf72d7c1675a97c226ab31c964c"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"603d90a3b012e93a0f39a931636b0595e531885d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     JieLee \u003cJieLi2087@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-24 17:39:43 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove extra checks in ovs_dvr_neutron_agent"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove the extra checks in ovs_dvr_neutron_agent that can be done in"},{"line_number":10,"context_line":"ovs_neutron_agent in one place."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da0c15f0_9ad4e020","line":7,"updated":"2016-03-28 13:28:20.000000000","message":"Would be helpful if there was ML2 prefix here","commit_id":"bb5cabb8ee852b03900fd2cc4ad7a1b91c0d41a1"}],"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"c5324a0fcfa227074b969f7a2098431de61a10e2","unresolved":false,"context_lines":[{"line_number":310,"context_line":"        if not self.in_distributed_mode():"},{"line_number":311,"context_line":"            LOG.debug(\"Not in distributed mode, ignoring invocation \""},{"line_number":312,"context_line":"                      \"of get_dvr_mac_address_list() \")"},{"line_number":313,"context_line":"            return"},{"line_number":314,"context_line":"        dvr_macs \u003d self.plugin_rpc.get_dvr_mac_address_list(self.context)"},{"line_number":315,"context_line":"        LOG.debug(\"L2 Agent DVR: Received these MACs: %r\", dvr_macs)"},{"line_number":316,"context_line":"        for mac in dvr_macs:"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa0719c6_71cdda5c","line":313,"updated":"2016-03-22 06:48:46.000000000","message":"why don\u0027t you remove this check as well?","commit_id":"46ad0596cee3bdf72d7c1675a97c226ab31c964c"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"f8bb29295984c5dc09cbc30100375416903ba711","unresolved":false,"context_lines":[{"line_number":310,"context_line":"        if not self.in_distributed_mode():"},{"line_number":311,"context_line":"            LOG.debug(\"Not in distributed mode, ignoring invocation \""},{"line_number":312,"context_line":"                      \"of get_dvr_mac_address_list() \")"},{"line_number":313,"context_line":"            return"},{"line_number":314,"context_line":"        dvr_macs \u003d self.plugin_rpc.get_dvr_mac_address_list(self.context)"},{"line_number":315,"context_line":"        LOG.debug(\"L2 Agent DVR: Received these MACs: %r\", dvr_macs)"},{"line_number":316,"context_line":"        for mac in dvr_macs:"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa0719c6_bf041b4a","line":313,"in_reply_to":"fa0719c6_71cdda5c","updated":"2016-03-23 09:01:20.000000000","message":"+1 This should be also removed.","commit_id":"46ad0596cee3bdf72d7c1675a97c226ab31c964c"}]}