)]}'
{"tempest/api/identity/admin/v3/test_roles.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"c17c33024368825b1c6cb404555a4aea6a633e72","unresolved":false,"context_lines":[{"line_number":48,"context_line":"                                    cls.role[\u0027id\u0027])"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"class RolesV3TestJSON(BaseRolesV3TestJSON):"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @classmethod"},{"line_number":54,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfdaf3ff_5050d6d4","line":51,"range":{"start_line":51,"start_character":6,"end_line":51,"end_character":21},"updated":"2019-01-17 03:03:36.000000000","message":"Why testcases in this class need force_tenant_isolation \u003d True?\nSeems we only need add check for CONF.identity_feature_group.immutable_user_source for those testcases that need to create user?","commit_id":"adc1609e74d100aff8fad16e09f5874e0adcfdc6"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"56e4db1da9c669b8881e4ee76e56824619fa4241","unresolved":false,"context_lines":[{"line_number":48,"context_line":"                                    cls.role[\u0027id\u0027])"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"class RolesV3TestJSON(BaseRolesV3TestJSON):"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @classmethod"},{"line_number":54,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfdaf3ff_60d2e202","line":51,"range":{"start_line":51,"start_character":6,"end_line":51,"end_character":21},"in_reply_to":"bfdaf3ff_5050d6d4","updated":"2019-01-17 13:56:11.000000000","message":"I see your point. I\u0027ll work on creating another patch and rebase this one on top of the new one. One for skipping tests that create users and one for setting force_tenant_isolation \u003d False\n\nI\u0027ll also do this for other open reviews that fall into this scenario.","commit_id":"adc1609e74d100aff8fad16e09f5874e0adcfdc6"}]}