)]}'
{"tempest/tests/image/v1/test_image_members.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"e4f147c5103cb554e942113f688d227d19a54df2","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from tempest import clients"},{"line_number":20,"context_line":"from tempest import exceptions"},{"line_number":21,"context_line":"import tempest.test"},{"line_number":22,"context_line":"from tempest.test import attr"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class ImageMembersTests(tempest.test.BaseTestCase):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F5%2F4%3D","line":22,"updated":"2013-03-07 03:26:54.000000000","message":"Some unused imports here?","commit_id":"8726e7114a6131f25f8b98cfb93e222ade402bc6"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7d2f2974b23b279da94a1600016f863f00447f86","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from tempest import clients"},{"line_number":20,"context_line":"from tempest import exceptions"},{"line_number":21,"context_line":"import tempest.test"},{"line_number":22,"context_line":"from tempest.test import attr"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class ImageMembersTests(tempest.test.BaseTestCase):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F47E%3D","line":22,"in_reply_to":"AAAAN3%2F%2F5%2F4%3D","updated":"2013-03-07 15:58:19.000000000","message":"Yeah, lazy copy and paste. Fixed now.","commit_id":"8726e7114a6131f25f8b98cfb93e222ade402bc6"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"e4f147c5103cb554e942113f688d227d19a54df2","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        name \u003d \u0027New Standard Image with Members\u0027"},{"line_number":47,"context_line":"        resp, image \u003d cls.client.create_image(name,"},{"line_number":48,"context_line":"                                              \u0027bare\u0027, \u0027raw\u0027,"},{"line_number":49,"context_line":"                                              is_public\u003dTrue, data\u003dimage_file)"},{"line_number":50,"context_line":"        image_id \u003d image[\u0027id\u0027]"},{"line_number":51,"context_line":"        cls.created_images.append(image_id)"},{"line_number":52,"context_line":"        return image_id"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F5%2Bk%3D","line":49,"updated":"2013-03-07 03:26:54.000000000","message":"Maybe should check status codes here?","commit_id":"8726e7114a6131f25f8b98cfb93e222ade402bc6"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7d2f2974b23b279da94a1600016f863f00447f86","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        name \u003d \u0027New Standard Image with Members\u0027"},{"line_number":47,"context_line":"        resp, image \u003d cls.client.create_image(name,"},{"line_number":48,"context_line":"                                              \u0027bare\u0027, \u0027raw\u0027,"},{"line_number":49,"context_line":"                                              is_public\u003dTrue, data\u003dimage_file)"},{"line_number":50,"context_line":"        image_id \u003d image[\u0027id\u0027]"},{"line_number":51,"context_line":"        cls.created_images.append(image_id)"},{"line_number":52,"context_line":"        return image_id"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F46w%3D","line":49,"in_reply_to":"AAAAN3%2F%2F5%2Bk%3D","updated":"2013-03-07 15:58:19.000000000","message":"I actually don\u0027t think that it is needed here. We\u0027re not testing create image so ensuring the proper response code from glance isn\u0027t needed. Also, if there is an error rest_client will raise the exception and this will fail.\n\nBut there is no harm to doing it I guess. So I\u0027ve added the check.","commit_id":"8726e7114a6131f25f8b98cfb93e222ade402bc6"}]}