)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5d010731e2d3fb09e7ac6b4812d926e654b10bc3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Navid Pustchi \u003cnpustchi@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-07 17:33:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding authentication comaptibiliy for OpenStackClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In ksa, plugins loading is managed by ksa."},{"line_number":10,"context_line":"OpenStackClient token_endpoint plugin expects get_auth_ref[1],"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_27fdfa3a","line":7,"updated":"2016-03-07 17:42:54.000000000","message":"*compatibility","commit_id":"614c192a3c4ea9ad0c3e7681f60bb62aab98fd49"},{"author":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"change_message_id":"252d69ffaec25ab20e24445344ab66b5a916eac1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Navid Pustchi \u003cnpustchi@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-07 17:33:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding authentication comaptibiliy for OpenStackClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In ksa, plugins loading is managed by ksa."},{"line_number":10,"context_line":"OpenStackClient token_endpoint plugin expects get_auth_ref[1],"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_67eee28c","line":7,"range":{"start_line":7,"start_character":22,"end_line":7,"end_character":34},"updated":"2016-03-07 17:40:51.000000000","message":"compatibility","commit_id":"614c192a3c4ea9ad0c3e7681f60bb62aab98fd49"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5d010731e2d3fb09e7ac6b4812d926e654b10bc3","unresolved":false,"context_lines":[{"line_number":11,"context_line":"(a method consistent with ksa auth plugins) which is not implemented"},{"line_number":12,"context_line":"in ksa token_endpoint."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This change will not change overal ksa behavior, as it only makes"},{"line_number":15,"context_line":"OpenStackClient to be able to provide its token_endpoint plugin,"},{"line_number":16,"context_line":"necessary for authentication migration in osc from ksc to ksa[2]."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_87bb2e72","line":14,"updated":"2016-03-07 17:42:54.000000000","message":"*overall","commit_id":"614c192a3c4ea9ad0c3e7681f60bb62aab98fd49"},{"author":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"change_message_id":"252d69ffaec25ab20e24445344ab66b5a916eac1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"(a method consistent with ksa auth plugins) which is not implemented"},{"line_number":12,"context_line":"in ksa token_endpoint."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This change will not change overal ksa behavior, as it only makes"},{"line_number":15,"context_line":"OpenStackClient to be able to provide its token_endpoint plugin,"},{"line_number":16,"context_line":"necessary for authentication migration in osc from ksc to ksa[2]."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_a7e8ea7d","line":14,"range":{"start_line":14,"start_character":28,"end_line":14,"end_character":34},"updated":"2016-03-07 17:40:51.000000000","message":"overall","commit_id":"614c192a3c4ea9ad0c3e7681f60bb62aab98fd49"}],"keystoneauth1/token_endpoint.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"f20bd587352f05d40b51aeb45f6b794a15a17c16","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return self.endpoint"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def get_auth_ref(self, session, **kwargs):"},{"line_number":41,"context_line":"        \"\"\"Retun none for the auth refrence"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        this provides backward comaptibility"},{"line_number":44,"context_line":"        for OpenStackClient authentication"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_074c3ec6","line":41,"updated":"2016-03-07 17:39:04.000000000","message":"Spelling\n\nAlso, become familiar with PEP257 as you have a few violations here: https://www.python.org/dev/peps/pep-0257/","commit_id":"614c192a3c4ea9ad0c3e7681f60bb62aab98fd49"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5d010731e2d3fb09e7ac6b4812d926e654b10bc3","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return self.endpoint"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def get_auth_ref(self, session, **kwargs):"},{"line_number":41,"context_line":"        \"\"\"Retun none for the auth refrence"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        this provides backward comaptibility"},{"line_number":44,"context_line":"        for OpenStackClient authentication"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_476806f6","line":41,"in_reply_to":"5aef4532_074c3ec6","updated":"2016-03-07 17:42:54.000000000","message":"*None","commit_id":"614c192a3c4ea9ad0c3e7681f60bb62aab98fd49"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5d010731e2d3fb09e7ac6b4812d926e654b10bc3","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    def get_auth_ref(self, session, **kwargs):"},{"line_number":41,"context_line":"        \"\"\"Retun none for the auth refrence"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        this provides backward comaptibility"},{"line_number":44,"context_line":"        for OpenStackClient authentication"},{"line_number":45,"context_line":"        \"\"\""},{"line_number":46,"context_line":"        return None"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_87496e8e","line":43,"updated":"2016-03-07 17:42:54.000000000","message":"*compatibility\n \nCompatibility to what?","commit_id":"614c192a3c4ea9ad0c3e7681f60bb62aab98fd49"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"dbbd6111e9955ef3ad1739576c690df5595b92c3","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return self.endpoint"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def get_auth_ref(self, session, **kwargs):"},{"line_number":41,"context_line":"        \"\"\"Return the authentication reference of an auth plugin"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        :param session: A session object to be used for communication"},{"line_number":44,"context_line":"        :type session: keystoneauth1.session.session"}],"source_content_type":"text/x-python","patch_set":2,"id":"5aef4532_c3f9a1ca","line":41,"updated":"2016-03-07 20:29:48.000000000","message":"PEP257: One line summaries end with a period.","commit_id":"2ed6fb3217f2625941a5e2957e3f5d885aa91fa4"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"f70b68f4283f99c8e887fa6b57c23dff4f0c491d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        :param session: A session object to be used for communication"},{"line_number":44,"context_line":"        :type session: keystoneauth1.session.session"},{"line_number":45,"context_line":"        \"\"\""},{"line_number":46,"context_line":"        return None"}],"source_content_type":"text/x-python","patch_set":5,"id":"5aef4532_4f181085","line":46,"updated":"2016-03-07 21:58:54.000000000","message":"Could include an inline comment here that the token plugin does not have an auth ref, because it\u0027s a \"static\" authentication using a pre-existing token.","commit_id":"d119e787ed61b42b858e4e9d7b75fdf9775d7825"}]}