)]}'
{"neutron/db/agentschedulers_db.py":[{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"f46af10b6871e80a8b9a2d432f74d055e40a4223","unresolved":false,"context_lines":[{"line_number":220,"context_line":"        message\u003d\"This will be removed in the N cycle. \""},{"line_number":221,"context_line":"                \"Please use \u0027add_periodic_dhcp_agent_status_check\u0027 instead.\""},{"line_number":222,"context_line":"    )"},{"line_number":223,"context_line":"    def start_periodic_dhcp_agent_status_check(self):"},{"line_number":224,"context_line":"        if not cfg.CONF.allow_automatic_dhcp_failover:"},{"line_number":225,"context_line":"            LOG.info(_LI(\"Skipping periodic DHCP agent status check because \""},{"line_number":226,"context_line":"                         \"automatic network rescheduling is disabled.\"))"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_de548355","line":223,"updated":"2016-06-11 08:57:29.000000000","message":"I think the fix should be avoiding use this method. add_agent_status_check is only used here and start_periodic_l3_agent_status_check. And both of these methods are marked deprecated. If change to use the new method \"add_periodic_dhcp_agent_status_check\" I think line 229 will not be hit.","commit_id":"727b63aa819a82121d44490759988643a00bb8a9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"d06bdc50dbad30082e676abd6b7c1a844a489f77","unresolved":false,"context_lines":[{"line_number":227,"context_line":"            return"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        self.add_agent_status_check_worker("},{"line_number":230,"context_line":"            self.remove_networks_from_down_agents)"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"    def add_periodic_dhcp_agent_status_check(self):"},{"line_number":233,"context_line":"        if not cfg.CONF.allow_automatic_dhcp_failover:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_5644d3b5","line":230,"updated":"2016-06-13 12:26:50.000000000","message":"start_periodic_dhcp_agent_status_check L.223 itself is marked as deprecated too. How about dropping start_periodic_dhcp_agent_status_check too?","commit_id":"7bca52a20cb544fb901b0a3186aec6cad911ce95"}],"neutron/db/l3_agentschedulers_db.py":[{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"f46af10b6871e80a8b9a2d432f74d055e40a4223","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        message\u003d\"This will be removed in the N cycle. \""},{"line_number":84,"context_line":"                \"Please use \u0027add_periodic_l3_agent_status_check\u0027 instead.\""},{"line_number":85,"context_line":"    )"},{"line_number":86,"context_line":"    def start_periodic_l3_agent_status_check(self):"},{"line_number":87,"context_line":"        if not cfg.CONF.allow_automatic_l3agent_failover:"},{"line_number":88,"context_line":"            LOG.info(_LI(\"Skipping period L3 agent status check because \""},{"line_number":89,"context_line":"                         \"automatic router rescheduling is disabled.\"))"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_fe497fb8","line":86,"updated":"2016-06-11 08:57:29.000000000","message":"The same comment here.","commit_id":"727b63aa819a82121d44490759988643a00bb8a9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"d06bdc50dbad30082e676abd6b7c1a844a489f77","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    @debtcollector.removals.remove("},{"line_number":83,"context_line":"        message\u003d\"This will be removed in the N cycle. \""},{"line_number":84,"context_line":"                \"Please use \u0027add_periodic_l3_agent_status_check\u0027 instead.\""},{"line_number":85,"context_line":"    )"},{"line_number":86,"context_line":"    def start_periodic_l3_agent_status_check(self):"},{"line_number":87,"context_line":"        if not cfg.CONF.allow_automatic_l3agent_failover:"},{"line_number":88,"context_line":"            LOG.info(_LI(\"Skipping period L3 agent status check because \""}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_f6805fc1","line":85,"updated":"2016-06-13 12:26:50.000000000","message":"Same as for dhcp_agent scheduler.","commit_id":"7bca52a20cb544fb901b0a3186aec6cad911ce95"}]}