)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"17ca605917376a7e480893a6319716e9c3af3e0e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     jolie \u003cguoshan@awcloud.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-01-05 16:56:46 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Not retry on deadlock Transactions in backend"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Keystone does not retry on deadlock Transactions [500 Error] in"},{"line_number":10,"context_line":"identity backend. Add wrapper for delete_user method."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba5201f7_11eb3814","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"updated":"2017-01-05 18:56:34.000000000","message":"Change this to \"Retry on deadlock Transactions in backend\"\n\nNot Retry pass the understanding that your patching is doing that (making it to not retry)","commit_id":"952a83b7cc61512f806ee4836f5bc4ded687a540"}],"keystone/tests/unit/test_v3_identity.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"21872cd77da193a8c9e400960c5354434bdf5432","unresolved":false,"context_lines":[{"line_number":549,"context_line":"        r \u003d self.credential_api.get_credential(credential2[\u0027id\u0027])"},{"line_number":550,"context_line":"        self.assertDictEqual(credential2, r)"},{"line_number":551,"context_line":""},{"line_number":552,"context_line":"    def test_retries_on_deadlock(self):"},{"line_number":553,"context_line":"        patcher \u003d mock.patch(\u0027sqlalchemy.orm.query.Query.delete\u0027,"},{"line_number":554,"context_line":"                             autospec\u003dTrue)"},{"line_number":555,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ba5201f7_09b4d67d","line":552,"range":{"start_line":552,"start_character":8,"end_line":552,"end_character":32},"updated":"2017-01-06 14:13:31.000000000","message":"Please make explicit what retries on deadlock\n\n    test_delete_user_retries_on_deadlock","commit_id":"fd6d9edc4d087e4d3b4070df5d99a2335cd188b8"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"21872cd77da193a8c9e400960c5354434bdf5432","unresolved":false,"context_lines":[{"line_number":572,"context_line":"        sql_delete_mock.side_effect \u003d side_effect"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"        try:"},{"line_number":575,"context_line":"            self.identity_api.delete_user(user_id\u003duuid.uuid4().hex)"},{"line_number":576,"context_line":"        finally:"},{"line_number":577,"context_line":"            if side_effect.patched:"},{"line_number":578,"context_line":"                patcher.stop()"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba5201f7_49817e8e","line":575,"range":{"start_line":575,"start_character":42,"end_line":575,"end_character":66},"updated":"2017-01-06 14:13:31.000000000","message":"you\u0027re passing an invalid user_id and this is making the test fail.\n\nthe delete_user calls get_user to make sure the user exists before deleting.\n\nkeystone.exception.UserNotFound: Could not find user: 787a7e321ec54f4ead2c8d26e24ed2ae","commit_id":"fd6d9edc4d087e4d3b4070df5d99a2335cd188b8"}]}