)]}'
{"tempest/api/image/base.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"5943457623df5d6385d40c67ae1f27533c421cc9","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        # data inside a dict that is keyed against \u0027image\u0027."},{"line_number":65,"context_line":"        if \u0027image\u0027 in image:"},{"line_number":66,"context_line":"            image \u003d image[\u0027image\u0027]"},{"line_number":67,"context_line":"        cls.created_images.append(image[\u0027id\u0027])"},{"line_number":68,"context_line":"        cls.addClassResourceCleanup(cls.client.wait_for_resource_deletion,"},{"line_number":69,"context_line":"                                    image[\u0027id\u0027])"},{"line_number":70,"context_line":"        cls.addClassResourceCleanup(test_utils.call_and_ignore_notfound_exc,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_3a15ac3d","line":67,"range":{"start_line":67,"start_character":8,"end_line":67,"end_character":46},"updated":"2017-11-06 21:48:18.000000000","message":"Seems like self.created_images is used in a few v1/v2 image tests. Do you think it\u0027s worth moving cls.created_images to the test themselves? Seems like most other places in Tempest just use this class array for resource_cleanup which is no longer done.","commit_id":"be8b810183fe595145d7e7ebd171c54e51f2c7bd"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"04d358f16c6f8122c3c38edc63705552c0166b44","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        # data inside a dict that is keyed against \u0027image\u0027."},{"line_number":65,"context_line":"        if \u0027image\u0027 in image:"},{"line_number":66,"context_line":"            image \u003d image[\u0027image\u0027]"},{"line_number":67,"context_line":"        cls.created_images.append(image[\u0027id\u0027])"},{"line_number":68,"context_line":"        cls.addClassResourceCleanup(cls.client.wait_for_resource_deletion,"},{"line_number":69,"context_line":"                                    image[\u0027id\u0027])"},{"line_number":70,"context_line":"        cls.addClassResourceCleanup(test_utils.call_and_ignore_notfound_exc,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_c61802d4","line":67,"range":{"start_line":67,"start_character":8,"end_line":67,"end_character":46},"in_reply_to":"3f4b6375_3a15ac3d","updated":"2017-11-07 00:58:33.000000000","message":"Yeah I\u0027ve thought about that, and because two files use it so I finally didn\u0027t change it:)","commit_id":"be8b810183fe595145d7e7ebd171c54e51f2c7bd"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"63dfbf9a0b6b4ade39663d7ede504750eec2df03","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        # data inside a dict that is keyed against \u0027image\u0027."},{"line_number":65,"context_line":"        if \u0027image\u0027 in image:"},{"line_number":66,"context_line":"            image \u003d image[\u0027image\u0027]"},{"line_number":67,"context_line":"        cls.created_images.append(image[\u0027id\u0027])"},{"line_number":68,"context_line":"        cls.addClassResourceCleanup(cls.client.wait_for_resource_deletion,"},{"line_number":69,"context_line":"                                    image[\u0027id\u0027])"},{"line_number":70,"context_line":"        cls.addClassResourceCleanup(test_utils.call_and_ignore_notfound_exc,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f485f77_c84dd89a","line":67,"range":{"start_line":67,"start_character":8,"end_line":67,"end_character":46},"in_reply_to":"3f4b6375_c61802d4","updated":"2017-11-17 03:25:47.000000000","message":"+1 of doing that but we can do in separate patch.","commit_id":"be8b810183fe595145d7e7ebd171c54e51f2c7bd"}]}