)]}'
{"tempest/api/compute/admin/test_quotas.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"01918161d4594733d31db7dddb48656294ad3364","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    # TODO(afazekas): merge these test cases"},{"line_number":98,"context_line":"    @test.idempotent_id(\u0027ce9e0815-8091-4abd-8345-7fe5b85faa1d\u0027)"},{"line_number":99,"context_line":"    def test_get_updated_quotas(self):"},{"line_number":100,"context_line":"        # Verify that GET shows the updated quota set of project"},{"line_number":101,"context_line":"        project_name \u003d data_utils.rand_name(\u0027cpu_quota_tenant\u0027)"},{"line_number":102,"context_line":"        project_desc \u003d project_name + \u0027-desc\u0027"},{"line_number":103,"context_line":"        project \u003d self.identity_utils.create_project(name\u003dproject_name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_63076282","line":100,"updated":"2015-08-20 21:51:57.000000000","message":"it\u0027ll always be tenant to me","commit_id":"07e210d74f8d10e1b9c3a1e1699516d17ff20090"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"01918161d4594733d31db7dddb48656294ad3364","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    @test.idempotent_id(\u0027ce9e0815-8091-4abd-8345-7fe5b85faa1d\u0027)"},{"line_number":99,"context_line":"    def test_get_updated_quotas(self):"},{"line_number":100,"context_line":"        # Verify that GET shows the updated quota set of project"},{"line_number":101,"context_line":"        project_name \u003d data_utils.rand_name(\u0027cpu_quota_tenant\u0027)"},{"line_number":102,"context_line":"        project_desc \u003d project_name + \u0027-desc\u0027"},{"line_number":103,"context_line":"        project \u003d self.identity_utils.create_project(name\u003dproject_name,"},{"line_number":104,"context_line":"                                                     description\u003dproject_desc)"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_43001e66","line":101,"updated":"2015-08-20 21:51:57.000000000","message":"not going to chhange the rand-name prefix to project :)","commit_id":"07e210d74f8d10e1b9c3a1e1699516d17ff20090"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"2dd39a03c95dfbd419c955e5cf5d709df6557463","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    @test.idempotent_id(\u0027ce9e0815-8091-4abd-8345-7fe5b85faa1d\u0027)"},{"line_number":99,"context_line":"    def test_get_updated_quotas(self):"},{"line_number":100,"context_line":"        # Verify that GET shows the updated quota set of project"},{"line_number":101,"context_line":"        project_name \u003d data_utils.rand_name(\u0027cpu_quota_tenant\u0027)"},{"line_number":102,"context_line":"        project_desc \u003d project_name + \u0027-desc\u0027"},{"line_number":103,"context_line":"        project \u003d self.identity_utils.create_project(name\u003dproject_name,"},{"line_number":104,"context_line":"                                                     description\u003dproject_desc)"}],"source_content_type":"text/x-python","patch_set":6,"id":"da20952f_d8b33be8","line":101,"updated":"2015-08-27 22:50:48.000000000","message":"Heh, still tenant.\n\n(it\u0027ll always be tenant to me)","commit_id":"c4b820f5d1fdc1fa0c2a2eb17f2195433d991060"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"f2692f5f81ea630389f2294ec9bb14790de7f603","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    @test.idempotent_id(\u0027ce9e0815-8091-4abd-8345-7fe5b85faa1d\u0027)"},{"line_number":99,"context_line":"    def test_get_updated_quotas(self):"},{"line_number":100,"context_line":"        # Verify that GET shows the updated quota set of project"},{"line_number":101,"context_line":"        project_name \u003d data_utils.rand_name(\u0027cpu_quota_project\u0027)"},{"line_number":102,"context_line":"        project_desc \u003d project_name + \u0027-desc\u0027"},{"line_number":103,"context_line":"        project \u003d self.identity_utils.create_project(name\u003dproject_name,"},{"line_number":104,"context_line":"                                                     description\u003dproject_desc)"}],"source_content_type":"text/x-python","patch_set":7,"id":"da20952f_f840b137","line":101,"updated":"2015-09-02 01:18:23.000000000","message":"Hey, you changed it! :)","commit_id":"51663a018b5b622dfc07e1ca2f73c558d426c588"}],"tempest/api/network/test_routers.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"01918161d4594733d31db7dddb48656294ad3364","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @test.idempotent_id(\u0027e54dd3a3-4352-4921-b09d-44369ae17397\u0027)"},{"line_number":88,"context_line":"    def test_create_router_setting_project_id(self):"},{"line_number":89,"context_line":"        # Test creating router from admin user setting project_id."},{"line_number":90,"context_line":"        project \u003d data_utils.rand_name(\u0027test_tenant_\u0027)"},{"line_number":91,"context_line":"        description \u003d data_utils.rand_name(\u0027desc_\u0027)"},{"line_number":92,"context_line":"        project \u003d self.identity_utils.create_project(name\u003dproject,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_c31aeec5","line":89,"updated":"2015-08-20 21:51:57.000000000","message":"This should be an admin test, it should be in the admin test subdir. I guess fixing this is out of scope for this change, but it\u0027d be a quick followup patch if you wanted to fix this.","commit_id":"07e210d74f8d10e1b9c3a1e1699516d17ff20090"}],"tempest/common/cred_client.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"52d1e0b7690b254b04cbea6d34ef2b6ec70a3c10","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"da20952f_18e0d3cb","updated":"2015-09-02 22:40:17.000000000","message":"Maybe this should be named creds_client (for consistency with the class name)?","commit_id":"51663a018b5b622dfc07e1ca2f73c558d426c588"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"bcf63c880d330c7cf95d2609b989dadd21a67642","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"da20952f_af77499e","in_reply_to":"da20952f_18e0d3cb","updated":"2015-09-09 00:41:58.000000000","message":"Seems then to conflict with cred_provider.py in the same folder. Maybe the class should be CredClient? but then looks funny with get_creds_client.\n\nI would prefer to leave this up to the tempest team as a post review if possible as i\u0027m not sure how you want to organize all these terms and am happy with whatever you choose.","commit_id":"51663a018b5b622dfc07e1ca2f73c558d426c588"}],"tempest/test.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"52d1e0b7690b254b04cbea6d34ef2b6ec70a3c10","unresolved":false,"context_lines":[{"line_number":440,"context_line":"        This can be used for creating and tearing down projects in tests. It"},{"line_number":441,"context_line":"        should not be used for testing identity features."},{"line_number":442,"context_line":"        \"\"\""},{"line_number":443,"context_line":"        if CONF.identity.auth_version \u003d\u003d \u0027v2\u0027:"},{"line_number":444,"context_line":"            client \u003d self.os_admin.identity_client"},{"line_number":445,"context_line":"        else:"},{"line_number":446,"context_line":"            client \u003d self.os_admin.identity_v3_client"}],"source_content_type":"text/x-python","patch_set":7,"id":"da20952f_b898c76b","line":443,"updated":"2015-09-02 22:40:17.000000000","message":"You could try and use self.identity_version if defined, and fall back to CONF.identity.auth_version if not defined - so you could use this in identity tests as well. \n\nBut with the restriction from the comment this is fine.","commit_id":"51663a018b5b622dfc07e1ca2f73c558d426c588"}]}