)]}'
{"keystone/tests/unit/protection/v3/test_trusts.py":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"87b7e04e08261ca3f5e3ee634f6004f7fd19fdb6","unresolved":false,"context_lines":[{"line_number":957,"context_line":"        self.user_id \u003d PROVIDERS.identity_api.create_user("},{"line_number":958,"context_line":"            domain_admin)[\u0027id\u0027]"},{"line_number":959,"context_line":"        PROVIDERS.assignment_api.create_grant("},{"line_number":960,"context_line":"            self.bootstrapper.admin_role_id, user_id\u003dself.user_id,"},{"line_number":961,"context_line":"            domain_id\u003dself.domain_id"},{"line_number":962,"context_line":"        )"},{"line_number":963,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_18cc75b9","line":960,"range":{"start_line":960,"start_character":12,"end_line":960,"end_character":43},"updated":"2019-08-19 19:59:43.000000000","message":"Domain user or domain admin? I thought we do make that distinction.","commit_id":"686ec6dda6dfa73e9634979231bcd9b2edbd1da8"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"1782d977ef8c7352d6d459ecd8db1d358e1a5a21","unresolved":false,"context_lines":[{"line_number":957,"context_line":"        self.user_id \u003d PROVIDERS.identity_api.create_user("},{"line_number":958,"context_line":"            domain_admin)[\u0027id\u0027]"},{"line_number":959,"context_line":"        PROVIDERS.assignment_api.create_grant("},{"line_number":960,"context_line":"            self.bootstrapper.admin_role_id, user_id\u003dself.user_id,"},{"line_number":961,"context_line":"            domain_id\u003dself.domain_id"},{"line_number":962,"context_line":"        )"},{"line_number":963,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_98f5c553","line":960,"range":{"start_line":960,"start_character":12,"end_line":960,"end_character":43},"in_reply_to":"7faddb67_18cc75b9","updated":"2019-08-19 20:34:49.000000000","message":"I had the same question. Using just the admin is consistent with how the other unit tests are set up, and this was Lance\u0027s justification for it: https://review.opendev.org/#/c/621017/7/keystone/tests/unit/protection/v3/test_registered_limits.py","commit_id":"686ec6dda6dfa73e9634979231bcd9b2edbd1da8"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"29cddbd9738abb218a90cc6806b2bb5a1f3367e9","unresolved":false,"context_lines":[{"line_number":957,"context_line":"        self.user_id \u003d PROVIDERS.identity_api.create_user("},{"line_number":958,"context_line":"            domain_admin)[\u0027id\u0027]"},{"line_number":959,"context_line":"        PROVIDERS.assignment_api.create_grant("},{"line_number":960,"context_line":"            self.bootstrapper.admin_role_id, user_id\u003dself.user_id,"},{"line_number":961,"context_line":"            domain_id\u003dself.domain_id"},{"line_number":962,"context_line":"        )"},{"line_number":963,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5faad753_bcd2a233","line":960,"range":{"start_line":960,"start_character":12,"end_line":960,"end_character":43},"in_reply_to":"7faddb67_8818101e","updated":"2019-09-06 21:10:45.000000000","message":"Yeah - I figured we should just do protection with a single role and default to admin. Instead of doing it with all three and spinning up even more tests.","commit_id":"686ec6dda6dfa73e9634979231bcd9b2edbd1da8"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"6104bdeb457af42ae7acb3cac1c34fc02b01e1d1","unresolved":false,"context_lines":[{"line_number":957,"context_line":"        self.user_id \u003d PROVIDERS.identity_api.create_user("},{"line_number":958,"context_line":"            domain_admin)[\u0027id\u0027]"},{"line_number":959,"context_line":"        PROVIDERS.assignment_api.create_grant("},{"line_number":960,"context_line":"            self.bootstrapper.admin_role_id, user_id\u003dself.user_id,"},{"line_number":961,"context_line":"            domain_id\u003dself.domain_id"},{"line_number":962,"context_line":"        )"},{"line_number":963,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_8818101e","line":960,"range":{"start_line":960,"start_character":12,"end_line":960,"end_character":43},"in_reply_to":"7faddb67_98f5c553","updated":"2019-08-19 23:33:07.000000000","message":"Yeah, looks like in practice, we don\u0027t have that distinction right now.","commit_id":"686ec6dda6dfa73e9634979231bcd9b2edbd1da8"}]}