)]}'
{"octavia/controller/worker/tasks/database_tasks.py":[{"author":{"_account_id":8157,"name":"Bertrand LALLAU","email":"bertrand.lallau@gmail.com","username":"blallau"},"change_message_id":"2d9c7fdcc7f58dcba34d47ad57d08f25a98ab88b","unresolved":false,"context_lines":[{"line_number":1020,"context_line":"        :param loadbalancer: The load balancer to disable health monitoring on"},{"line_number":1021,"context_line":"        \"\"\""},{"line_number":1022,"context_line":"        for amp in loadbalancer.amphorae:"},{"line_number":1023,"context_line":"            LOG.debug(\u0027Disabling health monitoring on amphora: %s\u0027, amp.id)"},{"line_number":1024,"context_line":"            try:"},{"line_number":1025,"context_line":"                self.amp_health_repo.delete(db_apis.get_session(),"},{"line_number":1026,"context_line":"                                            amphora_id\u003damp.id)"},{"line_number":1027,"context_line":"            except sqlalchemy.orm.exc.NoResultFound:"},{"line_number":1028,"context_line":"                LOG.debug(\u0027No existing amphora health record to delete \u0027"},{"line_number":1029,"context_line":"                          \u0027for amphora: %s, skipping.\u0027, amp.id)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a7e1126_9cb93fec","line":1029,"range":{"start_line":1023,"start_character":0,"end_line":1029,"end_character":63},"updated":"2015-12-17 11:56:34.000000000","message":"Code duplicated with DisableAmphoraHealthMonitoring class just above.\nCan we do better?","commit_id":"55c8c9d3f50030dc4b1ffdaea895495ada7cb70c"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"ed84a67d6510a8011a84ff404bc1313236cc4bc4","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        self.amp_health_repo \u003d repo.AmphoraHealthRepository()"},{"line_number":48,"context_line":"        super(BaseDatabaseTask, self).__init__(**kwargs)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def _delete_from_amp_heatlh(self, amphora_id):"},{"line_number":51,"context_line":"        \"\"\"Delete the amphora_health record for an amphora."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        :param amphora_id: The amphora id to delete"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_df42910e","line":50,"updated":"2016-01-15 15:57:43.000000000","message":"This method name is misspelled.","commit_id":"2720cb48546c24bd03c2e49fb45b74e89f0c440b"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"ed84a67d6510a8011a84ff404bc1313236cc4bc4","unresolved":false,"context_lines":[{"line_number":60,"context_line":"            LOG.debug(\u0027No existing amphora health record to delete \u0027"},{"line_number":61,"context_line":"                      \u0027for amphora: %s, skipping.\u0027, amphora_id)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _mark_amp_heatlh_busy(self, amphora_id):"},{"line_number":64,"context_line":"        \"\"\"Mark the amphora_health record busy for an amphora."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        :param amphora_id: The amphora id to mark busy"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_7f4e053c","line":63,"updated":"2016-01-15 15:57:43.000000000","message":"This method name is misspelled.","commit_id":"2720cb48546c24bd03c2e49fb45b74e89f0c440b"}],"octavia/tests/unit/controller/worker/tasks/test_database_tasks.py":[{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"8b3fba2290258dd1b2b25ea26e140c84b66fcc13","unresolved":false,"context_lines":[{"line_number":1055,"context_line":"        mock_amp_health_repo_delete.assert_called_once_with("},{"line_number":1056,"context_line":"            \u0027TEST\u0027, amphora_id\u003dAMP_ID)"},{"line_number":1057,"context_line":""},{"line_number":1058,"context_line":"    @mock.patch(\u0027octavia.db.repositories.AmphoraHealthRepository.delete\u0027)"},{"line_number":1059,"context_line":"    def test_disable_lb_amphorae_health_monitoring("},{"line_number":1060,"context_line":"            self,"},{"line_number":1061,"context_line":"            mock_amp_health_repo_delete,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a710552_004da63d","line":1058,"updated":"2015-12-16 12:25:45.000000000","message":"Shouldn\u0027t this be:\n\n@mock.patch(\u0027octavia.db.repositories.AmphoraLBHealthRepository.delete\u0027)","commit_id":"a65ba1cafc3a89c6679477806383f832b60d3983"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"485d7f593dd9aa88c61c07e8ae49747e7c9f871e","unresolved":false,"context_lines":[{"line_number":1055,"context_line":"        mock_amp_health_repo_delete.assert_called_once_with("},{"line_number":1056,"context_line":"            \u0027TEST\u0027, amphora_id\u003dAMP_ID)"},{"line_number":1057,"context_line":""},{"line_number":1058,"context_line":"    @mock.patch(\u0027octavia.db.repositories.AmphoraHealthRepository.delete\u0027)"},{"line_number":1059,"context_line":"    def test_disable_lb_amphorae_health_monitoring("},{"line_number":1060,"context_line":"            self,"},{"line_number":1061,"context_line":"            mock_amp_health_repo_delete,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a7e1126_ec032075","line":1058,"in_reply_to":"5a710552_004da63d","updated":"2015-12-17 02:07:15.000000000","message":"No\n\nhttps://github.com/openstack/octavia/blob/master/octavia/db/repositories.py#L450","commit_id":"a65ba1cafc3a89c6679477806383f832b60d3983"}]}