)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"b356137eabf6c76b939884038922ac18c947f05f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     wangxiyuan \u003cwangxiyuan@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-02-03 15:52:49 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix cache invalidation for application creadential"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The cache invalidation doesn\u0027t work for application creadential."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f93b717_23076e1b","line":7,"updated":"2018-02-03 22:52:00.000000000","message":"typo: credential","commit_id":"cc87abf22a0c5b83a50832ec9c83d71a4b8f8ead"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"b356137eabf6c76b939884038922ac18c947f05f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix cache invalidation for application creadential"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The cache invalidation doesn\u0027t work for application creadential."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This patch fixed it."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f93b717_e310a652","line":9,"updated":"2018-02-03 22:52:00.000000000","message":"typo: credential","commit_id":"cc87abf22a0c5b83a50832ec9c83d71a4b8f8ead"}],"keystone/tests/unit/application_credential/test_backends.py":[{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"820c1fe65cf2800e6f3554371aaeb8168e5f876f","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        app_cred \u003d self._new_app_cred_data(self.user_foo[\u0027id\u0027],"},{"line_number":155,"context_line":"                                           project_id\u003dself.tenant_bar[\u0027id\u0027])"},{"line_number":156,"context_line":"        self.app_cred_api.create_application_credential(app_cred)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        # cache the information"},{"line_number":159,"context_line":"        self.app_cred_api.get_application_credential(app_cred[\u0027id\u0027])"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f93b717_e00bbc28","line":157,"updated":"2018-02-03 22:49:03.000000000","message":"can you split this patch into 2? The first one with this test added, which will fail (so you use the `wip` decorator [1]). Then you add the fix to a follow up patch.\n\nin this way, we can have more confidence the patch actually fixes the issue.\n\n[1] https://github.com/openstack/keystone/blob/master/keystone/tests/unit/resource/test_backends.py#L439","commit_id":"cc87abf22a0c5b83a50832ec9c83d71a4b8f8ead"}]}