)]}'
{"glance/async/flows/base_import.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"23a3472eb59edd5b167f037f645730e5830a4523","unresolved":false,"context_lines":[{"line_number":367,"context_line":"            log_msg \u003d _LE(\"Task ID %(task_id)s failed. Error: %(exc_type)s: \""},{"line_number":368,"context_line":"                          \"%(e)s\")"},{"line_number":369,"context_line":"            LOG.exception(log_msg, {\u0027exc_type\u0027: type(e),"},{"line_number":370,"context_line":"                                    \u0027e\u0027: encodeutils.exception_to_unicode(e),"},{"line_number":371,"context_line":"                                    \u0027task_id\u0027: task.task_id})"},{"line_number":372,"context_line":""},{"line_number":373,"context_line":"            err_msg \u003d _(\"Error: %(exc_type)s: %(e)s\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"5f4e5783_cba0cea8","line":370,"range":{"start_line":370,"start_character":41,"end_line":370,"end_character":76},"updated":"2017-10-10 17:57:02.000000000","message":"Exceptions shouldn\u0027t be formatted into the string passed into LOG.exception since LOG.exception will automatically print out whatever in scope exception there is. So this results in double logging of the exception details.\n\nThat part is existing code, but if you do another update, I would prefer to just see this removed.","commit_id":"6cb3a70ce556d2733cced2dd8880f1359caa35d1"}],"glance/scrubber.py":[{"author":{"_account_id":22448,"name":"Alexander Bashmakov","email":"abashmak@yahoo.com","username":"abashmak"},"change_message_id":"0f3656f743041fbb5869bf43eb691f05c41c7ff4","unresolved":false,"context_lines":[{"line_number":373,"context_line":"    def __init__(self, store_api):"},{"line_number":374,"context_line":"        LOG.info(_LI(\"Initializing scrubber with configuration: %s\"),"},{"line_number":375,"context_line":"                 {\u0027registry_host\u0027: CONF.registry_host,"},{"line_number":376,"context_line":"                                \u0027registry_port\u0027: CONF.registry_port})"},{"line_number":377,"context_line":""},{"line_number":378,"context_line":"        self.store_api \u003d store_api"},{"line_number":379,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"da6895a0_266b1a20","line":376,"range":{"start_line":376,"start_character":18,"end_line":376,"end_character":32},"updated":"2016-10-22 00:06:01.000000000","message":"sorry, one more nit: can you align \u0027registry_port\u0027 under \u0027registry_host\u0027 for visual indent.","commit_id":"920d864511f695eb5931ecb07e61c8ab096dedc0"}]}