)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"3c79a42359e22349a4233ec8c51927d925fceaa5","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-09-23 16:49:17 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Rename v3_authenticate_token() to v3_create_token()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The name of this method has plagued me for years, so I figured I would"},{"line_number":10,"context_line":"finally propose a fix. The reason v3_authenticate_token() is a terrible"},{"line_number":11,"context_line":"name is that it implies that a token is being authenticated (in other"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_ad56f95b","line":8,"updated":"2015-09-23 16:57:50.000000000","message":"Although I realise this is part of a large set of fixes to tests, there is nothing in this commit message that says this is just changing test code.  When I first read it, my first impression was that you are change the Identity API.","commit_id":"3c3a81fdb1c34e81626fd5fab0272988440108d3"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"03f87b54b547dabe4de1b4062b1a1b0e1fe781d0","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Rename v3_authenticate_token() to v3_create_token()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The name of this method has plagued me for years, so I figured I would"},{"line_number":10,"context_line":"finally propose a fix. The reason v3_authenticate_token() is a terrible"},{"line_number":11,"context_line":"name is that it implies that a token is being authenticated (in other"},{"line_number":12,"context_line":"words: validated). As it turns out, we have another operation that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a1a9d01_27c6c19a","line":9,"updated":"2015-09-28 12:49:31.000000000","message":"as per my comment on the 1st patch, I\u0027d really like us to mention somewhere in the commit message that we are talking about test methods here - not the main code itself","commit_id":"eb4c928d1d31d030d43966e920072a8e26553478"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"8cfebfa676c54f4ed529ec59ff929c3eda0a71b2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Rename v3_authenticate_token() to v3_create_token()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The name of this method has plagued me for years, so I figured I would"},{"line_number":10,"context_line":"finally propose a fix. The reason v3_authenticate_token() is a terrible"},{"line_number":11,"context_line":"name is that it implies that a token is being authenticated (in other"},{"line_number":12,"context_line":"words: validated). As it turns out, we have another operation that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a1a9d01_b8971eeb","line":9,"in_reply_to":"9a1a9d01_27c6c19a","updated":"2015-09-29 12:06:29.000000000","message":"++","commit_id":"eb4c928d1d31d030d43966e920072a8e26553478"}],"keystone/tests/unit/test_v3_federation.py":[{"author":{"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},"change_message_id":"5b8eaa23b0908cf01aac6d39bd0d094020df33db","unresolved":false,"context_lines":[{"line_number":1981,"context_line":"            scoped_token, expected_status\u003dhttp_client.INTERNAL_SERVER_ERROR)"},{"line_number":1982,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":1983,"context_line":"        self.v3_create_token(scoped_token, expected_status\u003d500)"},{"line_number":1984,"context_line":"\u003e\u003e\u003e\u003e\u003e\u003e\u003e Rename v3_authenticate_token() to v3_create_token()"},{"line_number":1985,"context_line":""},{"line_number":1986,"context_line":"    def test_lists_with_missing_group_in_backend(self):"},{"line_number":1987,"context_line":"        \"\"\"Test a mapping that points to a group that does not exist"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_084281ae","line":1984,"updated":"2015-09-29 10:33:44.000000000","message":"forget to do these ?","commit_id":"eb4c928d1d31d030d43966e920072a8e26553478"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"f5d56d31b97d8966e997338b5bfc1f62aa6ae002","unresolved":false,"context_lines":[{"line_number":1981,"context_line":"            scoped_token, expected_status\u003dhttp_client.INTERNAL_SERVER_ERROR)"},{"line_number":1982,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":1983,"context_line":"        self.v3_create_token(scoped_token, expected_status\u003d500)"},{"line_number":1984,"context_line":"\u003e\u003e\u003e\u003e\u003e\u003e\u003e Rename v3_authenticate_token() to v3_create_token()"},{"line_number":1985,"context_line":""},{"line_number":1986,"context_line":"    def test_lists_with_missing_group_in_backend(self):"},{"line_number":1987,"context_line":"        \"\"\"Test a mapping that points to a group that does not exist"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_6436f753","line":1984,"updated":"2015-09-28 12:37:10.000000000","message":"looks like non-resolved conflict?","commit_id":"eb4c928d1d31d030d43966e920072a8e26553478"}]}