)]}'
{"tempest/test.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5f55da2c904bf14ad679e5608f2f327d6c2ae0c1","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                if hasattr(etype(), \u0027with_traceback\u0027):"},{"line_number":269,"context_line":"                    raise etype(value).with_traceback(trace)"},{"line_number":270,"context_line":"                else:"},{"line_number":271,"context_line":"                    raise etype(value)"},{"line_number":272,"context_line":"            finally:"},{"line_number":273,"context_line":"                del trace  # to avoid circular refs"},{"line_number":274,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9abe1de1_576acc1b","line":271,"updated":"2015-04-27 08:42:45.000000000","message":"It does not preserve the behaviour on python 2.","commit_id":"795c14afaa21065f48cf4c23dfb81a26af2b7be1"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"4d987921d60dd194bd57fe0818ea5c44f1b9d59d","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                if hasattr(etype(), \u0027with_traceback\u0027):"},{"line_number":269,"context_line":"                    raise etype(value).with_traceback(trace)"},{"line_number":270,"context_line":"                else:"},{"line_number":271,"context_line":"                    raise etype(value)"},{"line_number":272,"context_line":"            finally:"},{"line_number":273,"context_line":"                del trace  # to avoid circular refs"},{"line_number":274,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7ad32947_9f9d1f5c","line":271,"in_reply_to":"9abe1de1_576acc1b","updated":"2015-04-27 14:34:58.000000000","message":"Done","commit_id":"795c14afaa21065f48cf4c23dfb81a26af2b7be1"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5f55da2c904bf14ad679e5608f2f327d6c2ae0c1","unresolved":false,"context_lines":[{"line_number":309,"context_line":"                if hasattr(etype(), \u0027with_traceback\u0027):"},{"line_number":310,"context_line":"                    raise etype(value).with_traceback(trace)"},{"line_number":311,"context_line":"                else:"},{"line_number":312,"context_line":"                    raise etype(value)"},{"line_number":313,"context_line":"            finally:"},{"line_number":314,"context_line":"                del trace  # to avoid circular refs"},{"line_number":315,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9abe1de1_177454fb","line":312,"updated":"2015-04-27 08:42:45.000000000","message":"It does not preserve the behaviour on python 2.","commit_id":"795c14afaa21065f48cf4c23dfb81a26af2b7be1"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"4d987921d60dd194bd57fe0818ea5c44f1b9d59d","unresolved":false,"context_lines":[{"line_number":309,"context_line":"                if hasattr(etype(), \u0027with_traceback\u0027):"},{"line_number":310,"context_line":"                    raise etype(value).with_traceback(trace)"},{"line_number":311,"context_line":"                else:"},{"line_number":312,"context_line":"                    raise etype(value)"},{"line_number":313,"context_line":"            finally:"},{"line_number":314,"context_line":"                del trace  # to avoid circular refs"},{"line_number":315,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7ad32947_5fa0a790","line":312,"in_reply_to":"9abe1de1_177454fb","updated":"2015-04-27 14:34:58.000000000","message":"Done","commit_id":"795c14afaa21065f48cf4c23dfb81a26af2b7be1"}],"tempest/thirdparty/boto/test.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5f55da2c904bf14ad679e5608f2f327d6c2ae0c1","unresolved":false,"context_lines":[{"line_number":255,"context_line":"        except exception.BotoServerError as exc:"},{"line_number":256,"context_line":"            error_msg \u003d excMatcher.match(exc)"},{"line_number":257,"context_line":"            if error_msg is not None:"},{"line_number":258,"context_line":"                raise self.failureException(error_msg)"},{"line_number":259,"context_line":"        else:"},{"line_number":260,"context_line":"            raise self.failureException(\"BotoServerError not raised\")"},{"line_number":261,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9abe1de1_37e518a8","line":258,"updated":"2015-04-27 08:42:45.000000000","message":"Would be better to preserve the original trace here as well.","commit_id":"795c14afaa21065f48cf4c23dfb81a26af2b7be1"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"4d987921d60dd194bd57fe0818ea5c44f1b9d59d","unresolved":false,"context_lines":[{"line_number":255,"context_line":"        except exception.BotoServerError as exc:"},{"line_number":256,"context_line":"            error_msg \u003d excMatcher.match(exc)"},{"line_number":257,"context_line":"            if error_msg is not None:"},{"line_number":258,"context_line":"                raise self.failureException(error_msg)"},{"line_number":259,"context_line":"        else:"},{"line_number":260,"context_line":"            raise self.failureException(\"BotoServerError not raised\")"},{"line_number":261,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7ad32947_ff8e5326","line":258,"in_reply_to":"9abe1de1_37e518a8","updated":"2015-04-27 14:34:58.000000000","message":"Sure, but I think that might be a separate patch. I think we should just concentrate on enabling python3 support here instead of making improvements.","commit_id":"795c14afaa21065f48cf4c23dfb81a26af2b7be1"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"55620367d2703a692e63ee1816763d6f00b1a718","unresolved":false,"context_lines":[{"line_number":255,"context_line":"        except exception.BotoServerError as exc:"},{"line_number":256,"context_line":"            error_msg \u003d excMatcher.match(exc)"},{"line_number":257,"context_line":"            if error_msg is not None:"},{"line_number":258,"context_line":"                raise self.failureException(error_msg)"},{"line_number":259,"context_line":"        else:"},{"line_number":260,"context_line":"            raise self.failureException(\"BotoServerError not raised\")"},{"line_number":261,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7ad32947_5f09c730","line":258,"updated":"2015-04-27 14:56:31.000000000","message":"It would be better as:\n  six.reraise(self.failureException, *sys.exc_info()[1:2])\nor\n  six.reraise(self.failureException, error_msg, sys.exc_info()[2])\n\nNOTE: Do not assign sys.exc_info[2] to a local variable!\nIf you do so it must be deleted in the finally.\n\nThis is not what the original code does, but it supposed to do this.","commit_id":"8e5f96f199455b3207dc5ab7aaad1cc764f4a983"}]}