)]}'
{"tempest/api/identity/base.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"6b4d548a88cf2353a964ec66c48817c6f1c9bcba","unresolved":false,"context_lines":[{"line_number":34,"context_line":"            skip_msg \u003d (\"%s skipped as Keystone admin endpoint was set.\\n\""},{"line_number":35,"context_line":"                        \"To test non-admin actions group/name [identity]/\""},{"line_number":36,"context_line":"                        \"endpoint_type should be set as \u0027publicURL\u0027.\""},{"line_number":37,"context_line":"                        % cls.__name__)"},{"line_number":38,"context_line":"        elif (not CONF.identity.endpoint_type.startswith(\u0027admin\u0027) and"},{"line_number":39,"context_line":"                \u0027admin\u0027 in cls.credentials):"},{"line_number":40,"context_line":"            skip_msg \u003d (\"%s skipped as Keystone non-admin endpoint was set.\\n\""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_fccc3463","line":37,"updated":"2015-07-10 12:43:51.000000000","message":"I think this assumption is incorrect. Depending on the deployment and where the tests are executed, it is possible to run non-admin tests against an admin endpoint, and/or admin tests against public end point.\n\nHere you are creating a situation where it\u0027s never possible to run both admin and non-admin test cases, which is certainly not what we want.","commit_id":"fb7a7845a0e5df58f50b8c934adfdeff88019c3d"},{"author":{"_account_id":9377,"name":"Sergey Slipushenko","email":"slipushenko@gmail.com","username":"sslypushenko"},"change_message_id":"b15770fc2b946c7453a978bf64d963ebea78700d","unresolved":false,"context_lines":[{"line_number":34,"context_line":"            skip_msg \u003d (\"%s skipped as Keystone admin endpoint was set.\\n\""},{"line_number":35,"context_line":"                        \"To test non-admin actions group/name [identity]/\""},{"line_number":36,"context_line":"                        \"endpoint_type should be set as \u0027publicURL\u0027.\""},{"line_number":37,"context_line":"                        % cls.__name__)"},{"line_number":38,"context_line":"        elif (not CONF.identity.endpoint_type.startswith(\u0027admin\u0027) and"},{"line_number":39,"context_line":"                \u0027admin\u0027 in cls.credentials):"},{"line_number":40,"context_line":"            skip_msg \u003d (\"%s skipped as Keystone non-admin endpoint was set.\\n\""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_68e05ecd","line":37,"in_reply_to":"ba3cc151_fccc3463","updated":"2015-07-10 13:46:30.000000000","message":"I\u0027ve already written my thoughts about it in comments to bug. In short:\n\nIf we run non-admin tests against admin endpoint we will get a failure (something like that - http://paste.openstack.org/show/361823/ ). It because non-admin identity client gets a token with primary credentials and tries to do some actions require admin rights. In public endpoint these actions don\u0027t require admin rights but in admin endpoint - they do.\n\nKeystone endpoint - it is something bigger than just URL, where service has been deployed. When you choose publicUrl instead of adminUrl you will get different rights. So, yes it is impossible to test both admin and public endpoints in same test run if we can specify only single endpoint type.","commit_id":"fb7a7845a0e5df58f50b8c934adfdeff88019c3d"}],"tempest/clients.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"021446858610e7e3705f754638a9827e29507a8d","unresolved":false,"context_lines":[{"line_number":351,"context_line":"        params_v2_public \u003d params.copy()"},{"line_number":352,"context_line":"        params_v2_public[\u0027endpoint_type\u0027] \u003d ("},{"line_number":353,"context_line":"            CONF.identity.v2_public_endpoint_type)"},{"line_number":354,"context_line":"        # Client uses non-admin endpoint type of Keystone API v2"},{"line_number":355,"context_line":"        self.identity_public_client \u003d IdentityClient(self.auth_provider,"},{"line_number":356,"context_line":"                                                     **params_v2_public)"},{"line_number":357,"context_line":"        params_v3 \u003d params.copy()"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a4dcd0f_fe351df7","line":354,"updated":"2015-08-03 14:12:55.000000000","message":"s/non-admin/public","commit_id":"deab6295cd946daefaa76e00d0df044f92b2d64f"},{"author":{"_account_id":16433,"name":"izadorozhna","email":"izadorozhna@mirantis.com","username":"izadorozhna"},"change_message_id":"e5bda4bca730bc205c6225c4c960c156a660909f","unresolved":false,"context_lines":[{"line_number":351,"context_line":"        params_v2_public \u003d params.copy()"},{"line_number":352,"context_line":"        params_v2_public[\u0027endpoint_type\u0027] \u003d ("},{"line_number":353,"context_line":"            CONF.identity.v2_public_endpoint_type)"},{"line_number":354,"context_line":"        # Client uses non-admin endpoint type of Keystone API v2"},{"line_number":355,"context_line":"        self.identity_public_client \u003d IdentityClient(self.auth_provider,"},{"line_number":356,"context_line":"                                                     **params_v2_public)"},{"line_number":357,"context_line":"        params_v3 \u003d params.copy()"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a4dcd0f_a40ea8e3","line":354,"in_reply_to":"1a4dcd0f_fe351df7","updated":"2015-08-03 14:24:32.000000000","message":"Done","commit_id":"deab6295cd946daefaa76e00d0df044f92b2d64f"}]}