)]}'
{"ironic/common/glance_service/base_image_service.py":[{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"d6c02714731d4869eca5b1c0e8bbeadcb25dac32","unresolved":false,"context_lines":[{"line_number":194,"context_line":"        if not service_utils.is_image_active(image):"},{"line_number":195,"context_line":"            raise exception.ImageUnacceptable("},{"line_number":196,"context_line":"                image_id\u003dimage_id,"},{"line_number":197,"context_line":"                reason\u003d_(\"The image status requires to be active.\"))"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"        if not service_utils.is_image_available(self.context, image):"},{"line_number":200,"context_line":"            raise exception.ImageNotFound(image_id\u003dimage_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_b2aaa616","line":197,"updated":"2018-04-09 11:37:30.000000000","message":"Slightly unsure if this is the right place to put this check, but thinking about it more it seems the cleanest right now.","commit_id":"caf3dc204be99d85207b712dc6ab067ce29137cc"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"e81f2608cb9248bd4efceffcdacfafa3949c78e9","unresolved":false,"context_lines":[{"line_number":194,"context_line":"        if not service_utils.is_image_active(image):"},{"line_number":195,"context_line":"            raise exception.ImageUnacceptable("},{"line_number":196,"context_line":"                image_id\u003dimage_id,"},{"line_number":197,"context_line":"                reason\u003d_(\"The image status requires to be active.\"))"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"        if not service_utils.is_image_available(self.context, image):"},{"line_number":200,"context_line":"            raise exception.ImageNotFound(image_id\u003dimage_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_c6a51c49","line":197,"range":{"start_line":197,"start_character":42,"end_line":197,"end_character":51},"updated":"2018-04-09 14:25:13.000000000","message":"s/requires/is required/\ns/to be active/to be in an active state/\n\nWe could instead do it upon download, but we\u0027re not proxying the data through, so I think show fails us early and fast which is likely best.","commit_id":"caf3dc204be99d85207b712dc6ab067ce29137cc"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"50b129ae3870c74c699f88fc486a483b07c34c48","unresolved":false,"context_lines":[{"line_number":194,"context_line":"        if not service_utils.is_image_active(image):"},{"line_number":195,"context_line":"            raise exception.ImageUnacceptable("},{"line_number":196,"context_line":"                image_id\u003dimage_id,"},{"line_number":197,"context_line":"                reason\u003d_(\"The image status requires to be active.\"))"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"        if not service_utils.is_image_available(self.context, image):"},{"line_number":200,"context_line":"            raise exception.ImageNotFound(image_id\u003dimage_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_419b3665","line":197,"range":{"start_line":197,"start_character":42,"end_line":197,"end_character":51},"in_reply_to":"bf659307_c6a51c49","updated":"2018-04-09 14:30:22.000000000","message":"wrt wording, how about \u0027The status for the image must be \u0027active\u0027. It seems to me that it might be useful to indicate what the status *is*.","commit_id":"caf3dc204be99d85207b712dc6ab067ce29137cc"}]}