)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"f35d31cd1a58574c5ee226f9df3152a5bb6f96c1","unresolved":false,"context_lines":[{"line_number":10,"context_line":"_get_routers_can_schedule called from within"},{"line_number":11,"context_line":"_schedule_ha_routers_to_additional_agent when using:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"router_scheduler_driver \u003d neutron.scheduler.l3_agent_scheduler.AZLeastRoutersScheduler"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This was leading to problems with starting neutron-l3-agent on network nodes,"},{"line_number":16,"context_line":"causing HA routers to fail to start."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1641879"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"5a74a57a_7baa3772","line":15,"range":{"start_line":13,"start_character":0,"end_line":15,"end_character":77},"updated":"2016-11-22 22:50:06.000000000","message":"These two lines are too long. Please see section titled \u0027Summary of Git commit message structure\u0027 in https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"}],"neutron/scheduler/l3_agent_scheduler.py":[{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"e0a54b1b251b70137c60b8e66e0993f0a839803d","unresolved":false,"context_lines":[{"line_number":449,"context_line":"                target_routers.append(r)"},{"line_number":450,"context_line":""},{"line_number":451,"context_line":"        if not target_routers:"},{"line_number":452,"context_line":"            return []"},{"line_number":453,"context_line":""},{"line_number":454,"context_line":"        return super(AZLeastRoutersScheduler, self)._get_routers_can_schedule("},{"line_number":455,"context_line":"            plugin, context, target_routers, l3_agent)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_62227b03","line":452,"updated":"2016-11-21 12:56:07.000000000","message":"there are test cases for _get_routers_can_schedule in neutron.tests.unit.scheduler.test_l3_agent_scheduler, can you please add one in there that ensures that an empty list is returned when no routers are passed in (to easily trigger this conditional)?","commit_id":"0df00d661c34178b2f2de1cc362da01508fe78bb"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"71605c9852d8f70bc8075dbc8b306b861f0d579a","unresolved":false,"context_lines":[{"line_number":449,"context_line":"                target_routers.append(r)"},{"line_number":450,"context_line":""},{"line_number":451,"context_line":"        if not target_routers:"},{"line_number":452,"context_line":"            return []"},{"line_number":453,"context_line":""},{"line_number":454,"context_line":"        return super(AZLeastRoutersScheduler, self)._get_routers_can_schedule("},{"line_number":455,"context_line":"            plugin, context, target_routers, l3_agent)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_d3b400d6","line":452,"in_reply_to":"7a77a97e_62227b03","updated":"2016-11-23 06:14:16.000000000","message":"Done","commit_id":"0df00d661c34178b2f2de1cc362da01508fe78bb"}],"neutron/tests/unit/scheduler/test_l3_agent_scheduler.py":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"078cdc530d2cbbdc750b1fec10b7985902367662","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_32c41291","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"updated":"2016-11-22 15:51:09.000000000","message":"None -\u003e []","commit_id":"864291cc71caf679bb122c87a2e0ceb950f8a887"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"53a2592d2719a67908d097a46fdee73115ecaac4","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a74a57a_cf1078ab","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"in_reply_to":"5a74a57a_919fb44f","updated":"2016-11-25 06:02:38.000000000","message":"Done","commit_id":"864291cc71caf679bb122c87a2e0ceb950f8a887"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"0d822916f35af6edaca50d6e1c4f4a621f6b752c","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_d3e83d19","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"in_reply_to":"7a77a97e_32c41291","updated":"2016-11-23 06:14:26.000000000","message":"The problem only exists for AZLeastRoutersScheduler and this test therefore is unaffected by the change introduced by this commit. Note that the suggested change does not effect the end-result of this test.","commit_id":"864291cc71caf679bb122c87a2e0ceb950f8a887"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"bb26d89b3602a4b60600a4c884bcc78114e3ee79","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_1bd1f1fb","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"in_reply_to":"7a77a97e_32c41291","updated":"2016-11-22 16:30:59.000000000","message":"good catch!","commit_id":"864291cc71caf679bb122c87a2e0ceb950f8a887"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"02e49327d02367a291fb14bbfa2715f3aab4ae58","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a74a57a_919fb44f","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"in_reply_to":"7a77a97e_d3e83d19","updated":"2016-11-24 05:25:00.000000000","message":"Right, this doesn\u0027t affect the test but this implies that _get_routers_can_schedule() may return None. That\u0027s incorrect and can make a mistake as this bug. Therefore, I like we fix it in this patch.","commit_id":"864291cc71caf679bb122c87a2e0ceb950f8a887"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"078cdc530d2cbbdc750b1fec10b7985902367662","unresolved":false,"context_lines":[{"line_number":2129,"context_line":"        hosts \u003d set([a[\u0027host\u0027] for a in agents])"},{"line_number":2130,"context_line":"        self.assertEqual(expected_hosts, hosts)"},{"line_number":2131,"context_line":""},{"line_number":2132,"context_line":"    def test_az_scheduler_get_routers_can_schedule(self):"},{"line_number":2133,"context_line":"        result \u003d self.plugin.router_scheduler._get_routers_can_schedule("},{"line_number":2134,"context_line":"            self.plugin, mock.ANY, [], mock.ANY)"},{"line_number":2135,"context_line":"        self.assertEqual([], result)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_d5010806","line":2132,"range":{"start_line":2132,"start_character":8,"end_line":2132,"end_character":50},"updated":"2016-11-22 15:51:09.000000000","message":"This name doesn\u0027t mean the test case. I prefer test__get_routers_can_schedule_with_no_target_routers.","commit_id":"864291cc71caf679bb122c87a2e0ceb950f8a887"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"0d822916f35af6edaca50d6e1c4f4a621f6b752c","unresolved":false,"context_lines":[{"line_number":2129,"context_line":"        hosts \u003d set([a[\u0027host\u0027] for a in agents])"},{"line_number":2130,"context_line":"        self.assertEqual(expected_hosts, hosts)"},{"line_number":2131,"context_line":""},{"line_number":2132,"context_line":"    def test_az_scheduler_get_routers_can_schedule(self):"},{"line_number":2133,"context_line":"        result \u003d self.plugin.router_scheduler._get_routers_can_schedule("},{"line_number":2134,"context_line":"            self.plugin, mock.ANY, [], mock.ANY)"},{"line_number":2135,"context_line":"        self.assertEqual([], result)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_130f15e2","line":2132,"range":{"start_line":2132,"start_character":8,"end_line":2132,"end_character":50},"in_reply_to":"7a77a97e_d5010806","updated":"2016-11-23 06:14:26.000000000","message":"Done","commit_id":"864291cc71caf679bb122c87a2e0ceb950f8a887"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"588736103dfe867a136518fdc0072b8d54c41997","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a74a57a_8a9097ee","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"updated":"2016-11-23 02:47:03.000000000","message":"Do you have any reasons that my previous comment isn\u0027t reflected?","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"9de6860cd593b7a9f95e4dc609a5e8071da728bf","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a74a57a_b7d21b33","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"in_reply_to":"5a74a57a_8a9097ee","updated":"2016-11-23 06:16:25.000000000","message":"I am new at this, so my comment was still marked as \u0027Draft\u0027. That should be fixed now.","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"9ee4e25ae812e10af76572da5c8a9859b20d5a4e","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a74a57a_4c04b614","line":177,"range":{"start_line":177,"start_character":47,"end_line":177,"end_character":51},"in_reply_to":"5a74a57a_b7d21b33","updated":"2016-11-23 18:23:51.000000000","message":"@Pepijn,\n\nIn that case, welcome to Neutron! We appreciate your making the effort to start contributing to our project :-). When in doubt, please don\u0027t hesitate to ask questions here in Gerrit, in the mailing list or in freenode #openstack-neutron","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"df77a2301abbdae47775be83027f4aa6aff4f2e3","unresolved":false,"context_lines":[{"line_number":2129,"context_line":"        hosts \u003d set([a[\u0027host\u0027] for a in agents])"},{"line_number":2130,"context_line":"        self.assertEqual(expected_hosts, hosts)"},{"line_number":2131,"context_line":""},{"line_number":2132,"context_line":"    def test_az_scheduler_get_routers_can_schedule_with_no_target_routers(self):"},{"line_number":2133,"context_line":"        result \u003d self.plugin.router_scheduler._get_routers_can_schedule("},{"line_number":2134,"context_line":"            self.plugin, mock.ANY, [], mock.ANY)"},{"line_number":2135,"context_line":"        self.assertEqual([], result)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a74a57a_9b3633b3","line":2132,"range":{"start_line":2132,"start_character":79,"end_line":2132,"end_character":80},"updated":"2016-11-22 22:35:16.000000000","message":"Ooops, one too many","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"9de6860cd593b7a9f95e4dc609a5e8071da728bf","unresolved":false,"context_lines":[{"line_number":2129,"context_line":"        hosts \u003d set([a[\u0027host\u0027] for a in agents])"},{"line_number":2130,"context_line":"        self.assertEqual(expected_hosts, hosts)"},{"line_number":2131,"context_line":""},{"line_number":2132,"context_line":"    def test_az_scheduler_get_routers_can_schedule_with_no_target_routers(self):"},{"line_number":2133,"context_line":"        result \u003d self.plugin.router_scheduler._get_routers_can_schedule("},{"line_number":2134,"context_line":"            self.plugin, mock.ANY, [], mock.ANY)"},{"line_number":2135,"context_line":"        self.assertEqual([], result)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a74a57a_17e2ef03","line":2132,"range":{"start_line":2132,"start_character":79,"end_line":2132,"end_character":80},"in_reply_to":"5a74a57a_6ae5fba6","updated":"2016-11-23 06:16:25.000000000","message":"Done","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"9de6860cd593b7a9f95e4dc609a5e8071da728bf","unresolved":false,"context_lines":[{"line_number":2129,"context_line":"        hosts \u003d set([a[\u0027host\u0027] for a in agents])"},{"line_number":2130,"context_line":"        self.assertEqual(expected_hosts, hosts)"},{"line_number":2131,"context_line":""},{"line_number":2132,"context_line":"    def test_az_scheduler_get_routers_can_schedule_with_no_target_routers(self):"},{"line_number":2133,"context_line":"        result \u003d self.plugin.router_scheduler._get_routers_can_schedule("},{"line_number":2134,"context_line":"            self.plugin, mock.ANY, [], mock.ANY)"},{"line_number":2135,"context_line":"        self.assertEqual([], result)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a74a57a_f7dc9348","line":2132,"range":{"start_line":2132,"start_character":79,"end_line":2132,"end_character":80},"in_reply_to":"5a74a57a_9b3633b3","updated":"2016-11-23 06:16:25.000000000","message":"Done","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"588736103dfe867a136518fdc0072b8d54c41997","unresolved":false,"context_lines":[{"line_number":2129,"context_line":"        hosts \u003d set([a[\u0027host\u0027] for a in agents])"},{"line_number":2130,"context_line":"        self.assertEqual(expected_hosts, hosts)"},{"line_number":2131,"context_line":""},{"line_number":2132,"context_line":"    def test_az_scheduler_get_routers_can_schedule_with_no_target_routers(self):"},{"line_number":2133,"context_line":"        result \u003d self.plugin.router_scheduler._get_routers_can_schedule("},{"line_number":2134,"context_line":"            self.plugin, mock.ANY, [], mock.ANY)"},{"line_number":2135,"context_line":"        self.assertEqual([], result)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a74a57a_6ae5fba6","line":2132,"range":{"start_line":2132,"start_character":79,"end_line":2132,"end_character":80},"in_reply_to":"5a74a57a_9b3633b3","updated":"2016-11-23 02:47:03.000000000","message":"So I suggested \"test__get_routers_can_schedule_with_no_target_routers\".","commit_id":"29a0b6f88838f90b8c857e4bf0ce46ffaac127d6"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"99062aaf709829dc9eefbe173fe080bb5288da5e","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a74a57a_9606a6cc","line":177,"updated":"2016-11-24 21:13:47.000000000","message":"There are two definitions of _get_routers_can_schedule: https://github.com/openstack/neutron/blob/master/neutron/scheduler/l3_agent_scheduler.py#L118 and https://github.com/openstack/neutron/blob/master/neutron/scheduler/l3_agent_scheduler.py#L461. After your change, none of them can return None anymore. So this has to be changed to []","commit_id":"94e8a4727a87f9b6f8b4200d7b9e18e72dcdc3a4"},{"author":{"_account_id":24022,"name":"Pepijn Oomen","email":"pepijn@redpill-linpro.com","username":"pjoomen"},"change_message_id":"62d8c73d50c38ac52989b80a8bd83c30563dd058","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                    self.scheduler,"},{"line_number":175,"context_line":"                    \u0027_get_routers_can_schedule\u0027) as mock_target_routers:"},{"line_number":176,"context_line":"            mock_unscheduled_routers.return_value \u003d mock.ANY"},{"line_number":177,"context_line":"            mock_target_routers.return_value \u003d None"},{"line_number":178,"context_line":"            result \u003d self.scheduler.auto_schedule_routers("},{"line_number":179,"context_line":"                self.plugin, mock.ANY, mock.ANY, mock.ANY)"},{"line_number":180,"context_line":"        self.assertTrue(self.plugin.get_enabled_agent_on_host.called)"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a74a57a_ef27f40f","line":177,"in_reply_to":"5a74a57a_9606a6cc","updated":"2016-11-25 06:02:46.000000000","message":"Done","commit_id":"94e8a4727a87f9b6f8b4200d7b9e18e72dcdc3a4"}]}