)]}'
{"tempest/api/volume/test_volumes_list.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3a505f8ef8a6adb27d732808944de793cb209a79","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        # TODO(zhufl): After microversion 3.48 service_uuid and shared_targets"},{"line_number":78,"context_line":"        # are added to the body of volume_detail, and microversion is not used"},{"line_number":79,"context_line":"        # in resource_setup for now, so we should create and get volume"},{"line_number":80,"context_line":"        # details in setUp in order to use the same microversion as in"},{"line_number":81,"context_line":"        # testcases till microversion support in resource_setup is fulfilled."},{"line_number":82,"context_line":"        if not hasattr(self, \u0027volume_list\u0027):"},{"line_number":83,"context_line":"            existing_volumes \u003d self.volumes_client.list_volumes()[\u0027volumes\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f6b1bfe_f5adf931","line":80,"range":{"start_line":80,"start_character":35,"end_line":80,"end_character":64},"updated":"2020-10-14 22:09:24.000000000","message":"but you are not using the 3.48 microversion.  and by default API run with 3.0 microversion.","commit_id":"2b69777acf4234006de4a89559afa352d0ff686e"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"1f611e93851c9cd3be1ff4430916bd9de6114fd1","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        # TODO(zhufl): After microversion 3.48 service_uuid and shared_targets"},{"line_number":78,"context_line":"        # are added to the body of volume_detail, and microversion is not used"},{"line_number":79,"context_line":"        # in resource_setup for now, so we should create and get volume"},{"line_number":80,"context_line":"        # details in setUp in order to use the same microversion as in"},{"line_number":81,"context_line":"        # testcases till microversion support in resource_setup is fulfilled."},{"line_number":82,"context_line":"        if not hasattr(self, \u0027volume_list\u0027):"},{"line_number":83,"context_line":"            existing_volumes \u003d self.volumes_client.list_volumes()[\u0027volumes\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f621f24_dc1c81c3","line":80,"range":{"start_line":80,"start_character":35,"end_line":80,"end_character":64},"in_reply_to":"1f621f24_54420f10","updated":"2020-11-18 01:55:17.000000000","message":"\"to move microversion selection from setUp to resource_setup in base class\",\n--that will be great. is there any patch on-going for it?","commit_id":"2b69777acf4234006de4a89559afa352d0ff686e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ca49e8b4a9bd9a05afa7e176b5f01839202b7b3e","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        # TODO(zhufl): After microversion 3.48 service_uuid and shared_targets"},{"line_number":78,"context_line":"        # are added to the body of volume_detail, and microversion is not used"},{"line_number":79,"context_line":"        # in resource_setup for now, so we should create and get volume"},{"line_number":80,"context_line":"        # details in setUp in order to use the same microversion as in"},{"line_number":81,"context_line":"        # testcases till microversion support in resource_setup is fulfilled."},{"line_number":82,"context_line":"        if not hasattr(self, \u0027volume_list\u0027):"},{"line_number":83,"context_line":"            existing_volumes \u003d self.volumes_client.list_volumes()[\u0027volumes\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"fffc6b78_0f74ef34","line":80,"range":{"start_line":80,"start_character":35,"end_line":80,"end_character":64},"in_reply_to":"1f621f24_dc1c81c3","updated":"2020-11-20 01:03:43.000000000","message":"not yet. give me some time I can push that maybe by Monday or so. Little busy for RBAC things this week/end.","commit_id":"2b69777acf4234006de4a89559afa352d0ff686e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a0ac0d4d050c0014efe49d06c18b2dc322ac30de","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        # TODO(zhufl): After microversion 3.48 service_uuid and shared_targets"},{"line_number":78,"context_line":"        # are added to the body of volume_detail, and microversion is not used"},{"line_number":79,"context_line":"        # in resource_setup for now, so we should create and get volume"},{"line_number":80,"context_line":"        # details in setUp in order to use the same microversion as in"},{"line_number":81,"context_line":"        # testcases till microversion support in resource_setup is fulfilled."},{"line_number":82,"context_line":"        if not hasattr(self, \u0027volume_list\u0027):"},{"line_number":83,"context_line":"            existing_volumes \u003d self.volumes_client.list_volumes()[\u0027volumes\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f621f24_54420f10","line":80,"range":{"start_line":80,"start_character":35,"end_line":80,"end_character":64},"in_reply_to":"7f6b1bfe_1dab7b89","updated":"2020-11-17 18:16:56.000000000","message":"I see your point now. we need to use the requested microvesion in all phases of tests. What we should do is to move microversion selection from setUp to resource_setup in base class.\n\nDownside of this solution is we end up list/show volume for every test instead of doing it once for all tests in this class.","commit_id":"2b69777acf4234006de4a89559afa352d0ff686e"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"26da80711a58afd9e1176cfa418e2e185f3da78d","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        # TODO(zhufl): After microversion 3.48 service_uuid and shared_targets"},{"line_number":78,"context_line":"        # are added to the body of volume_detail, and microversion is not used"},{"line_number":79,"context_line":"        # in resource_setup for now, so we should create and get volume"},{"line_number":80,"context_line":"        # details in setUp in order to use the same microversion as in"},{"line_number":81,"context_line":"        # testcases till microversion support in resource_setup is fulfilled."},{"line_number":82,"context_line":"        if not hasattr(self, \u0027volume_list\u0027):"},{"line_number":83,"context_line":"            existing_volumes \u003d self.volumes_client.list_volumes()[\u0027volumes\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f6b1bfe_1dab7b89","line":80,"range":{"start_line":80,"start_character":35,"end_line":80,"end_character":64},"in_reply_to":"7f6b1bfe_f5adf931","updated":"2020-10-19 01:58:51.000000000","message":"Yes by default we use volume 3.0, but if we set volume.min_microversion\u003d3.48 in tempest.conf, the testcase will fail unexpectedly.","commit_id":"2b69777acf4234006de4a89559afa352d0ff686e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"da4e202c25c5c3255ea6507ad3f0e500c1a54817","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        # TODO(zhufl): After microversion 3.48 service_uuid and shared_targets"},{"line_number":78,"context_line":"        # are added to the body of volume_detail, and microversion is not used"},{"line_number":79,"context_line":"        # in resource_setup for now, so we should create and get volume"},{"line_number":80,"context_line":"        # details in setUp in order to use the same microversion as in"},{"line_number":81,"context_line":"        # testcases till microversion support in resource_setup is fulfilled."},{"line_number":82,"context_line":"        if not hasattr(self, \u0027volume_list\u0027):"},{"line_number":83,"context_line":"            existing_volumes \u003d self.volumes_client.list_volumes()[\u0027volumes\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"d5550e56_7574d300","line":80,"range":{"start_line":80,"start_character":35,"end_line":80,"end_character":64},"in_reply_to":"fffc6b78_0f74ef34","updated":"2021-12-08 04:35:14.000000000","message":"sorry for late, fixing in https://review.opendev.org/c/openstack/tempest/+/820932","commit_id":"2b69777acf4234006de4a89559afa352d0ff686e"}]}