)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"50f4c8f224e789e23c7f4dad2390c0a5f4b40c76","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Pass relevant parameters to Token based authentication"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In case of token authentication is used pass relevant parameters"},{"line_number":10,"context_line":"to Token authenticator."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Co-Authored-By: Andrey Kurilin \u003candr.kurilin@gmail.com\u003e"},{"line_number":13,"context_line":"Change-Id: I9a04d89016a834fe96f1b77e91011f7fa4fdda51"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"5a3905b3_10e698e7","line":10,"updated":"2017-01-24 14:15:59.000000000","message":"I would love to mention the commit that caused the problem, I believe it was Ibb99fdafbf02b3e92f1a5d04d168151b3400b901","commit_id":"9940e3fe0e47ff5f2f6d05c9564d17fc19ca0f5c"}],"novaclient/client.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"8adfbdc00ad1dbff0a087eb697cb33d74e7e8d63","unresolved":false,"context_lines":[{"line_number":147,"context_line":"                                  project_name\u003dproject_name,"},{"line_number":148,"context_line":"                                  project_domain_id\u003dproject_domain_id,"},{"line_number":149,"context_line":"                                  project_domain_name\u003dproject_domain_name,"},{"line_number":150,"context_line":"                                  user_domain_id\u003duser_domain_id,"},{"line_number":151,"context_line":"                                  user_domain_name\u003duser_domain_name)"},{"line_number":152,"context_line":"        elif not auth:"},{"line_number":153,"context_line":"            auth \u003d identity.Password(username\u003dusername,"},{"line_number":154,"context_line":"                                     user_id\u003duser_id,"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a3905b3_9e98096c","line":151,"range":{"start_line":150,"start_character":34,"end_line":151,"end_character":67},"updated":"2017-01-23 17:37:22.000000000","message":"Generic Token plugin doesn\u0027t support these arguments.","commit_id":"3b3620b817e9afd60eb640c5f37cb6db5706abfc"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"50f4c8f224e789e23c7f4dad2390c0a5f4b40c76","unresolved":false,"context_lines":[{"line_number":139,"context_line":"                                  project_id\u003dproject_id,"},{"line_number":140,"context_line":"                                  project_name\u003dproject_name,"},{"line_number":141,"context_line":"                                  project_domain_id\u003dproject_domain_id,"},{"line_number":142,"context_line":"                                  project_domain_name\u003dproject_domain_name)"},{"line_number":143,"context_line":"        elif not auth:"},{"line_number":144,"context_line":"            auth \u003d identity.Password(username\u003dusername,"},{"line_number":145,"context_line":"                                     user_id\u003duser_id,"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a3905b3_10985869","line":142,"updated":"2017-01-24 14:15:59.000000000","message":"Looking at:\nhttps://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/generic/token.py\nand\nhttps://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/v3/token.py\nand\nhttps://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/v2.py#L152\n\nThis seems correct.","commit_id":"9940e3fe0e47ff5f2f6d05c9564d17fc19ca0f5c"}],"novaclient/tests/functional/test_auth.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"c8cc5d1640eae9bed9684cf3001db845909ae251","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        if not base.is_keystone_version_available(session, version):"},{"line_number":75,"context_line":"            self.skip(\"Identity API version 2.0 is not available.\")"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        self.nova(\"list\", identity_api_version\u003d\"2.0\")"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    def test_auth_with_token_v3(self):"},{"line_number":80,"context_line":"        session \u003d self.keystone.session"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a57fde8_9a734db1","line":77,"range":{"start_line":77,"start_character":13,"end_line":77,"end_character":17},"updated":"2017-01-12 20:04:47.000000000","message":"you should call nova_auth_with_token here","commit_id":"2b8493a5e949ec240643247feab9e92005a6409c"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"c8cc5d1640eae9bed9684cf3001db845909ae251","unresolved":false,"context_lines":[{"line_number":82,"context_line":"        if not base.is_keystone_version_available(session, version):"},{"line_number":83,"context_line":"            self.skip(\"Identity API version 3.0 is not available.\")"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        self.nova(\"list\", identity_api_version\u003d\"3\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a57fde8_da65d5e1","line":85,"range":{"start_line":85,"start_character":13,"end_line":85,"end_character":17},"updated":"2017-01-12 20:04:47.000000000","message":"same here.","commit_id":"2b8493a5e949ec240643247feab9e92005a6409c"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"9a2fc278d71ed6df86dccf8a35271f534084d160","unresolved":false,"context_lines":[{"line_number":45,"context_line":"                 \u0027--os-auth-url %s --os-endpoint-type publicURL\u0027 % ("},{"line_number":46,"context_line":"                     self.cli_clients.username,"},{"line_number":47,"context_line":"                     self.cli_clients.tenant_name,"},{"line_number":48,"context_line":"                     self.keystone.auth_token,"},{"line_number":49,"context_line":"                     url))"},{"line_number":50,"context_line":"        if self.cli_clients.insecure:"},{"line_number":51,"context_line":"            flags +\u003d \u0027 --insecure \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a3c09a3_d71a1d6e","line":48,"range":{"start_line":48,"start_character":21,"end_line":48,"end_character":45},"updated":"2017-01-13 11:46:13.000000000","message":"As far as I remember, obtaining a token from keystoneclient is not an easy task. Code below works for me when I last time tried to do it.\n\n    session \u003d self.client.client.session\n\n    auth_ref \u003d session.auth.plugin.get_access(session)\n    token \u003d auth_ref.auth_token","commit_id":"f3e04a1a070c78513a46add6714395cbf4bf90e8"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"58c97096172e9882a2834be0f139fb48dc878407","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    def _get_auth_token(self):"},{"line_number":86,"context_line":"        session \u003d self.client.client.session"},{"line_number":87,"context_line":"        auth_ref \u003d session.auth.plugin.get_access(session)"},{"line_number":88,"context_line":"        return auth_ref.auth_token"}],"source_content_type":"text/x-python","patch_set":8,"id":"5a3905b3_bc637a19","line":87,"range":{"start_line":87,"start_character":27,"end_line":87,"end_character":38},"updated":"2017-01-20 11:09:31.000000000","message":"just \"auth\" without \"plugin\"","commit_id":"23370a70ee4d54860281bb672c9549321c523a6a"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"8adfbdc00ad1dbff0a087eb697cb33d74e7e8d63","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        return tempest.lib.cli.base.execute("},{"line_number":31,"context_line":"            \"nova\", action, flags, cli_dir\u003dself.cli_clients.cli_dir)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def nova_auth_with_token(self, action, identity_api_version):"},{"line_number":34,"context_line":"        flags \u003d (\u0027--os-username %s --os-tenant-name %s --os-auth-token %s \u0027"},{"line_number":35,"context_line":"                 \u0027--os-auth-url %s --os-endpoint-type publicURL\u0027 % ("},{"line_number":36,"context_line":"                     self.cli_clients.username,"},{"line_number":37,"context_line":"                     self.cli_clients.tenant_name,"},{"line_number":38,"context_line":"                     self._get_auth_token(),"},{"line_number":39,"context_line":"                     self._get_url(identity_api_version)))"},{"line_number":40,"context_line":"        if self.cli_clients.insecure:"},{"line_number":41,"context_line":"            flags +\u003d \u0027 --insecure \u0027"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        return tempest.lib.cli.base.execute("},{"line_number":44,"context_line":"            \"nova\", action, flags, cli_dir\u003dself.cli_clients.cli_dir)"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    def test_auth_via_keystone_v2(self):"},{"line_number":47,"context_line":"        session \u003d self.keystone.session"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a3905b3_be006567","line":44,"range":{"start_line":33,"start_character":2,"end_line":44,"end_character":68},"updated":"2017-01-23 17:37:22.000000000","message":"Actually, the keystoneauth1 library takes care about parsing CLI arguments and creating keystone session, so this test doesn\u0027t check modified code","commit_id":"3b3620b817e9afd60eb640c5f37cb6db5706abfc"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"ae2d266193ee966161eb86b5e11a787e0d37fe5f","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                             project_name\u003dself.project_name, **kw)"},{"line_number":52,"context_line":"        nova.servers.list()"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        # NOTE(andreykurilin): token auth is completely broken in CLI"},{"line_number":55,"context_line":"        # flags \u003d (\u0027--os-username %s --os-tenant-name %s --os-auth-token %s \u0027"},{"line_number":56,"context_line":"        #         \u0027--os-auth-url %s --os-endpoint-type publicURL\u0027 % ("},{"line_number":57,"context_line":"        #             self.cli_clients.username,"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a3905b3_2ddfbb3a","line":54,"range":{"start_line":54,"start_character":31,"end_line":54,"end_character":69},"updated":"2017-01-24 13:56:01.000000000","message":"could you please open a bugreport for this or link existing one?","commit_id":"9940e3fe0e47ff5f2f6d05c9564d17fc19ca0f5c"},{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"c22a8a215d158387af8922bc348c7e486037615f","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                             project_name\u003dself.project_name, **kw)"},{"line_number":52,"context_line":"        nova.servers.list()"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        # NOTE(andreykurilin): token auth is completely broken in CLI"},{"line_number":55,"context_line":"        # flags \u003d (\u0027--os-username %s --os-tenant-name %s --os-auth-token %s \u0027"},{"line_number":56,"context_line":"        #         \u0027--os-auth-url %s --os-endpoint-type publicURL\u0027 % ("},{"line_number":57,"context_line":"        #             self.cli_clients.username,"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a3905b3_ed093396","line":54,"range":{"start_line":54,"start_character":31,"end_line":54,"end_character":69},"in_reply_to":"5a3905b3_2ddfbb3a","updated":"2017-01-24 13:58:13.000000000","message":"That patch is here:\n\n - [CLI] Fix token auth type\n - https://review.openstack.org/#/c/424256/\n\nThis is a pretty urgent fix, so I think we can pass on linking to it for now.","commit_id":"9940e3fe0e47ff5f2f6d05c9564d17fc19ca0f5c"}]}