)]}'
{"trove/taskmanager/models.py":[{"author":{"_account_id":6268,"name":"Robert Myers","email":"myer0052@gmail.com","username":"rmyers"},"change_message_id":"e73bb72af67157edacc4b8c36d75ef0286d91f4f","unresolved":false,"context_lines":[{"line_number":582,"context_line":"            else:"},{"line_number":583,"context_line":"                self.server.delete()"},{"line_number":584,"context_line":"        except Exception as ex:"},{"line_number":585,"context_line":"            LOG.error(_(\"Error during delete compute server %s\")"},{"line_number":586,"context_line":"                      % self.server.id)"},{"line_number":587,"context_line":"            LOG.exception(ex)"},{"line_number":588,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FatQ%3D","line":585,"updated":"2013-11-06 18:45:16.000000000","message":"This should be LOG.exception(\"error message\"). The exception method is a error log + traceback there is no need to pass the exception itself.","commit_id":"2954f128725b3ebbb678344199e9d3b3f3d880b2"},{"author":{"_account_id":6268,"name":"Robert Myers","email":"myer0052@gmail.com","username":"rmyers"},"change_message_id":"e73bb72af67157edacc4b8c36d75ef0286d91f4f","unresolved":false,"context_lines":[{"line_number":592,"context_line":"                dns_api \u003d create_dns_client(self.context)"},{"line_number":593,"context_line":"                dns_api.delete_instance_entry(instance_id\u003dself.db_info.id)"},{"line_number":594,"context_line":"        except Exception as ex:"},{"line_number":595,"context_line":"            LOG.error(_(\"Error during dns entry for instance %s \")"},{"line_number":596,"context_line":"                      % self.db_info.id)"},{"line_number":597,"context_line":"            LOG.exception(ex)"},{"line_number":598,"context_line":"            # Poll until the server is gone."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FatM%3D","line":595,"updated":"2013-11-06 18:45:16.000000000","message":"Same here","commit_id":"2954f128725b3ebbb678344199e9d3b3f3d880b2"},{"author":{"_account_id":6268,"name":"Robert Myers","email":"myer0052@gmail.com","username":"rmyers"},"change_message_id":"e73bb72af67157edacc4b8c36d75ef0286d91f4f","unresolved":false,"context_lines":[{"line_number":682,"context_line":"                      \"attached volume filesystem for volume %(vol_id)s of \""},{"line_number":683,"context_line":"                      \"instance %(id)s\") %"},{"line_number":684,"context_line":"                      {\u0027vol_id\u0027: self.volume_id, \u0027id\u0027: self.id})"},{"line_number":685,"context_line":"            LOG.exception(e)"},{"line_number":686,"context_line":"        finally:"},{"line_number":687,"context_line":"            self.update_db(task_status\u003dinst_models.InstanceTasks.NONE)"},{"line_number":688,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FatA%3D","line":685,"updated":"2013-11-06 18:45:16.000000000","message":"Same as above","commit_id":"2954f128725b3ebbb678344199e9d3b3f3d880b2"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"00cf4cb657640aa30e46bebf3ee9845466c28154","unresolved":false,"context_lines":[{"line_number":593,"context_line":"        except Exception as ex:"},{"line_number":594,"context_line":"            LOG.exception(_(\"Error during dns entry for instance %s \")"},{"line_number":595,"context_line":"                      % self.db_info.id)"},{"line_number":596,"context_line":"            "},{"line_number":597,"context_line":"            # Poll until the server is gone."},{"line_number":598,"context_line":"        def server_is_finished():"},{"line_number":599,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FJsQ%3D","line":596,"updated":"2013-11-13 18:07:18.000000000","message":"plz remove the excess spaces.","commit_id":"74b369b2d770f5541d6f3be14e3b1fc182d44efd"}]}