)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"31ab66901fe9eed13fed87bed25fa228f057f4c8","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-02-12 17:28:33 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Separate projects client from identity V3 client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Partially implements blueprint consistent-service-method-names"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I236b2468de81539ae06937c2644c9840bea68b96"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"dae33548_0c561b29","line":8,"updated":"2016-02-15 13:18:36.000000000","message":"+ 1-2 line extra short description inside the commit message never harms, even if you have good ext links.","commit_id":"47a93ab4ccc729f7d56d8577bf1d439c402621d3"}],"tempest/clients.py":[{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"57ea7a0a93eda5f920349e3d98ff884df2850e01","unresolved":false,"context_lines":[{"line_number":115,"context_line":"from tempest.services.identity.v3.json.identity_client import IdentityV3Client"},{"line_number":116,"context_line":"from tempest.services.identity.v3.json.policies_client import \\"},{"line_number":117,"context_line":"    PoliciesClient as PoliciesV3Client"},{"line_number":118,"context_line":"from tempest.services.identity.v3.json.projects_client import ProjectsClient"},{"line_number":119,"context_line":"from tempest.services.identity.v3.json.regions_client import \\"},{"line_number":120,"context_line":"    RegionsClient as RegionsV3Client"},{"line_number":121,"context_line":"from tempest.services.identity.v3.json.services_client import \\"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_37ffdb35","line":118,"updated":"2016-02-11 10:29:52.000000000","message":"Shouldn\u0027t we call this ProjectsV3Client","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"2100ad8a46863add39e2fe85f3035c5edefe4012","unresolved":false,"context_lines":[{"line_number":115,"context_line":"from tempest.services.identity.v3.json.identity_client import IdentityV3Client"},{"line_number":116,"context_line":"from tempest.services.identity.v3.json.policies_client import \\"},{"line_number":117,"context_line":"    PoliciesClient as PoliciesV3Client"},{"line_number":118,"context_line":"from tempest.services.identity.v3.json.projects_client import ProjectsClient"},{"line_number":119,"context_line":"from tempest.services.identity.v3.json.regions_client import \\"},{"line_number":120,"context_line":"    RegionsClient as RegionsV3Client"},{"line_number":121,"context_line":"from tempest.services.identity.v3.json.services_client import \\"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_7f351af5","line":118,"in_reply_to":"3a57f1b5_37ffdb35","updated":"2016-02-11 15:17:47.000000000","message":"Hm, I don\u0027t think so. Anyway, let\u0027s listen to opinions of other reviewers :)","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"},{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"0d4559830dc71ed1037935e82bdaa272fcf32099","unresolved":false,"context_lines":[{"line_number":115,"context_line":"from tempest.services.identity.v3.json.identity_client import IdentityV3Client"},{"line_number":116,"context_line":"from tempest.services.identity.v3.json.policies_client import \\"},{"line_number":117,"context_line":"    PoliciesClient as PoliciesV3Client"},{"line_number":118,"context_line":"from tempest.services.identity.v3.json.projects_client import ProjectsClient"},{"line_number":119,"context_line":"from tempest.services.identity.v3.json.regions_client import \\"},{"line_number":120,"context_line":"    RegionsClient as RegionsV3Client"},{"line_number":121,"context_line":"from tempest.services.identity.v3.json.services_client import \\"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_62a261b0","line":118,"in_reply_to":"3a57f1b5_7f351af5","updated":"2016-02-11 15:23:29.000000000","message":"Well, I could live with it, as we\u0027re using TenantsClient for the V2 one, but let\u0027s wait for more opinions, I won\u0027t block the patch because of this ;)","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"},{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"57ea7a0a93eda5f920349e3d98ff884df2850e01","unresolved":false,"context_lines":[{"line_number":541,"context_line":"            self.auth_provider, **params_v3)"},{"line_number":542,"context_line":"        self.policies_client \u003d PoliciesV3Client(self.auth_provider,"},{"line_number":543,"context_line":"                                                **params_v3)"},{"line_number":544,"context_line":"        self.projects_client \u003d ProjectsClient(self.auth_provider, **params_v3)"},{"line_number":545,"context_line":"        self.regions_client \u003d RegionsV3Client(self.auth_provider, **params_v3)"},{"line_number":546,"context_line":"        self.credentials_client \u003d CredentialsV3Client(self.auth_provider,"},{"line_number":547,"context_line":"                                                      **params_v3)"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_178bbfc4","line":544,"updated":"2016-02-11 10:29:52.000000000","message":"ditto","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"}],"tempest/common/dynamic_creds.py":[{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"57ea7a0a93eda5f920349e3d98ff884df2850e01","unresolved":false,"context_lines":[{"line_number":94,"context_line":"                    os.subnets_client, os.ports_client,"},{"line_number":95,"context_line":"                    os.security_groups_client)"},{"line_number":96,"context_line":"        else:"},{"line_number":97,"context_line":"            return (os.identity_v3_client, os.projects_client, None, None,"},{"line_number":98,"context_line":"                    os.network_client, os.networks_client, os.subnets_client,"},{"line_number":99,"context_line":"                    os.ports_client, os.security_groups_client)"},{"line_number":100,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_17ddffdf","line":97,"updated":"2016-02-11 10:29:52.000000000","message":"Even if for v2 it\u0027s called tenants_client, I\u0027d name this projects_v3_client in order to be consistent","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"2100ad8a46863add39e2fe85f3035c5edefe4012","unresolved":false,"context_lines":[{"line_number":94,"context_line":"                    os.subnets_client, os.ports_client,"},{"line_number":95,"context_line":"                    os.security_groups_client)"},{"line_number":96,"context_line":"        else:"},{"line_number":97,"context_line":"            return (os.identity_v3_client, os.projects_client, None, None,"},{"line_number":98,"context_line":"                    os.network_client, os.networks_client, os.subnets_client,"},{"line_number":99,"context_line":"                    os.ports_client, os.security_groups_client)"},{"line_number":100,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_9f07b61b","line":97,"in_reply_to":"3a57f1b5_17ddffdf","updated":"2016-02-11 15:17:47.000000000","message":"See my comment in tempest/clients.py","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"}],"tempest/stress/driver.py":[{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"57ea7a0a93eda5f920349e3d98ff884df2850e01","unresolved":false,"context_lines":[{"line_number":151,"context_line":"                    users_client \u003d admin_manager.users_client"},{"line_number":152,"context_line":"                else:"},{"line_number":153,"context_line":"                    identity_client \u003d admin_manager.identity_v3_client"},{"line_number":154,"context_line":"                    projects_client \u003d admin_manager.projects_client"},{"line_number":155,"context_line":"                    roles_client \u003d None"},{"line_number":156,"context_line":"                    users_client \u003d None"},{"line_number":157,"context_line":"                domain \u003d (identity_client.auth_provider.credentials."}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_576e1702","line":154,"updated":"2016-02-11 10:29:52.000000000","message":"ditto about the renaming","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"2100ad8a46863add39e2fe85f3035c5edefe4012","unresolved":false,"context_lines":[{"line_number":151,"context_line":"                    users_client \u003d admin_manager.users_client"},{"line_number":152,"context_line":"                else:"},{"line_number":153,"context_line":"                    identity_client \u003d admin_manager.identity_v3_client"},{"line_number":154,"context_line":"                    projects_client \u003d admin_manager.projects_client"},{"line_number":155,"context_line":"                    roles_client \u003d None"},{"line_number":156,"context_line":"                    users_client \u003d None"},{"line_number":157,"context_line":"                domain \u003d (identity_client.auth_provider.credentials."}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_c219d57a","line":154,"in_reply_to":"3a57f1b5_576e1702","updated":"2016-02-11 15:17:47.000000000","message":"See my comment in tempest/clients.py","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"}],"tempest/test.py":[{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"57ea7a0a93eda5f920349e3d98ff884df2850e01","unresolved":false,"context_lines":[{"line_number":442,"context_line":"            users_client \u003d self.os_admin.users_client"},{"line_number":443,"context_line":"        else:"},{"line_number":444,"context_line":"            client \u003d self.os_admin.identity_v3_client"},{"line_number":445,"context_line":"            project_client \u003d self.os_adm.projects_client"},{"line_number":446,"context_line":"            roles_client \u003d None"},{"line_number":447,"context_line":"            users_client \u003d None"},{"line_number":448,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_b74e6b94","line":445,"updated":"2016-02-11 10:29:52.000000000","message":"ditto","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"2100ad8a46863add39e2fe85f3035c5edefe4012","unresolved":false,"context_lines":[{"line_number":442,"context_line":"            users_client \u003d self.os_admin.users_client"},{"line_number":443,"context_line":"        else:"},{"line_number":444,"context_line":"            client \u003d self.os_admin.identity_v3_client"},{"line_number":445,"context_line":"            project_client \u003d self.os_adm.projects_client"},{"line_number":446,"context_line":"            roles_client \u003d None"},{"line_number":447,"context_line":"            users_client \u003d None"},{"line_number":448,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_e23bb1cf","line":445,"in_reply_to":"3a57f1b5_b74e6b94","updated":"2016-02-11 15:17:47.000000000","message":"See my comment in tempest/clients.py","commit_id":"ce97431b905b9f4621e5b18c2ef5c22d51771578"}]}