)]}'
{"tempest/api/identity/admin/v3/test_domains.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"b7fb8d4d00b72804c5da8442226504c263e9b54e","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        self.client.assign_user_role_on_domain(domain[\u0027id\u0027], user_id,"},{"line_number":130,"context_line":"                                               role[\u0027id\u0027])"},{"line_number":131,"context_line":"        scopes \u003d self.non_admin_client.get_available_domain_scopes()"},{"line_number":132,"context_line":"        self.assertEqual(d_name, scopes[\u0027domains\u0027][0][\u0027name\u0027])"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"class DefaultDomainTestJSON(base.BaseIdentityV3AdminTest):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a68dd71_321cf676","line":132,"updated":"2016-01-21 11:46:26.000000000","message":"Could you also assert that the list scopes[\u0027domains\u0027] has exactly one item please.","commit_id":"e7665ea2ffd244fd7371e693c32371ae12544db8"},{"author":{"_account_id":18796,"name":"Alex Petrov","email":"apetrov@mirantis.com","username":"apetrov"},"change_message_id":"ed80d612d7ed96804bea7287b4d94c6662883fc5","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        self.client.assign_user_role_on_domain(domain[\u0027id\u0027], user_id,"},{"line_number":130,"context_line":"                                               role[\u0027id\u0027])"},{"line_number":131,"context_line":"        scopes \u003d self.non_admin_client.get_available_domain_scopes()"},{"line_number":132,"context_line":"        self.assertEqual(d_name, scopes[\u0027domains\u0027][0][\u0027name\u0027])"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"class DefaultDomainTestJSON(base.BaseIdentityV3AdminTest):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_f0dc4d1a","line":132,"in_reply_to":"9a68dd71_321cf676","updated":"2016-01-27 12:39:37.000000000","message":"Jordan, thanks for review. I have made the test more clear. Could you take a look again?","commit_id":"e7665ea2ffd244fd7371e693c32371ae12544db8"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"67375f7a316dd01d0f5e25ddde354bfcd170bb40","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        self.addCleanup(self._delete_domain, domain2[\u0027id\u0027])"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"        # Create a role"},{"line_number":128,"context_line":"        role_name \u003d data_utils.rand_name(name\u003d\u0027role\u0027)"},{"line_number":129,"context_line":"        role \u003d self.client.create_role(name\u003drole_name)[\u0027role\u0027]"},{"line_number":130,"context_line":"        self.addCleanup(self.client.delete_role, role[\u0027id\u0027])"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7a5de9d1_e3ff3ea8","line":128,"updated":"2016-01-29 04:03:30.000000000","message":"\u0027name\u003d\u0027 is not necessary.","commit_id":"ac6110967e34dc01fc9f37f12c264c905d2759de"},{"author":{"_account_id":18796,"name":"Alex Petrov","email":"apetrov@mirantis.com","username":"apetrov"},"change_message_id":"8e6c6192eb858a864a3286bdace2e2da83eed344","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        self.addCleanup(self._delete_domain, domain2[\u0027id\u0027])"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"        # Create a role"},{"line_number":128,"context_line":"        role_name \u003d data_utils.rand_name(name\u003d\u0027role\u0027)"},{"line_number":129,"context_line":"        role \u003d self.client.create_role(name\u003drole_name)[\u0027role\u0027]"},{"line_number":130,"context_line":"        self.addCleanup(self.client.delete_role, role[\u0027id\u0027])"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7a5de9d1_9cac1237","line":128,"in_reply_to":"7a5de9d1_e3ff3ea8","updated":"2016-02-02 13:01:53.000000000","message":"Done","commit_id":"ac6110967e34dc01fc9f37f12c264c905d2759de"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f6c78bb2b6d2ff040f773d4ddc81a8d1f275e613","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        self.addCleanup(self.client.delete_role, role[\u0027id\u0027])"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"        # Assign user role on domain #1"},{"line_number":133,"context_line":"        user_id \u003d self.non_admin_client.user_id"},{"line_number":134,"context_line":"        self.client.assign_user_role_on_domain(domain1[\u0027id\u0027], user_id,"},{"line_number":135,"context_line":"                                               role[\u0027id\u0027])"},{"line_number":136,"context_line":"        scopes \u003d self.non_admin_client.list_available_domain_scopes()"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"        # assert that the user is assigned with only domain #1"}],"source_content_type":"text/x-python","patch_set":8,"id":"5a5ae5dd_712e5afc","line":135,"range":{"start_line":133,"start_character":0,"end_line":135,"end_character":58},"updated":"2016-02-08 07:54:58.000000000","message":"I think this is probability here, as it assign domains on self.non_admin_client.user_id which can affect other tests using this user. Currently not but it can create issue.","commit_id":"ac66671aa98d49b7ad899cd7a780f4425ada1c1c"}],"tempest/services/identity/v3/json/identity_client.py":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"67375f7a316dd01d0f5e25ddde354bfcd170bb40","unresolved":false,"context_lines":[{"line_number":301,"context_line":"        self.expected_success(204, resp.status)"},{"line_number":302,"context_line":"        return service_client.ResponseBody(resp, body)"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    def get_available_domain_scopes(self):"},{"line_number":305,"context_line":"        \"\"\"Get available domain scopes  \"\"\""},{"line_number":306,"context_line":"        url \u003d \u0027auth/domains\u0027"},{"line_number":307,"context_line":"        resp, body \u003d self.get(url)"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a5de9d1_63f7cef5","line":304,"updated":"2016-01-29 04:03:30.000000000","message":"It should be \"list_available_domain_scopes\".","commit_id":"ac6110967e34dc01fc9f37f12c264c905d2759de"},{"author":{"_account_id":18796,"name":"Alex Petrov","email":"apetrov@mirantis.com","username":"apetrov"},"change_message_id":"ef8442d45b5c3bc6b258c7863fe3ff8255dffdd7","unresolved":false,"context_lines":[{"line_number":301,"context_line":"        self.expected_success(204, resp.status)"},{"line_number":302,"context_line":"        return service_client.ResponseBody(resp, body)"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    def get_available_domain_scopes(self):"},{"line_number":305,"context_line":"        \"\"\"Get available domain scopes  \"\"\""},{"line_number":306,"context_line":"        url \u003d \u0027auth/domains\u0027"},{"line_number":307,"context_line":"        resp, body \u003d self.get(url)"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a5de9d1_9c47321d","line":304,"in_reply_to":"7a5de9d1_63f7cef5","updated":"2016-02-02 12:50:11.000000000","message":"Done","commit_id":"ac6110967e34dc01fc9f37f12c264c905d2759de"}]}