)]}'
{"nova/api/openstack/compute/image_metadata.py":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"21ce771557e4d1f703c4c46ccf3768487318bd8a","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        image[\u0027properties\u0027] \u003d metadata"},{"line_number":107,"context_line":"        try:"},{"line_number":108,"context_line":"            self.image_service.update(context, image_id, image, None)"},{"line_number":109,"context_line":"        except exception.ImageNotAuthorized as e:"},{"line_number":110,"context_line":"            raise exc.HTTPForbidden(explanation\u003de.format_message())"},{"line_number":111,"context_line":"        return dict(metadata\u003dmetadata)"},{"line_number":112,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FrGQ%3D","line":109,"updated":"2013-07-25 10:50:17.000000000","message":"Just wandering if this one can throw an \"ImageNotFound\" as well, and will nova API handle it correctly with a 404?\n\nKind of outside of the scope of this patch tho.","commit_id":"e34bc31362fd91a494ae5b24b60b67a105e47756"},{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"fac322171ccd93fd2f349c3809f8e2359f74f1f9","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        image[\u0027properties\u0027] \u003d metadata"},{"line_number":107,"context_line":"        try:"},{"line_number":108,"context_line":"            self.image_service.update(context, image_id, image, None)"},{"line_number":109,"context_line":"        except exception.ImageNotAuthorized as e:"},{"line_number":110,"context_line":"            raise exc.HTTPForbidden(explanation\u003de.format_message())"},{"line_number":111,"context_line":"        return dict(metadata\u003dmetadata)"},{"line_number":112,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2Fqzk%3D","line":109,"in_reply_to":"AAAAQn%2F%2Fq0M%3D","updated":"2013-07-25 13:06:52.000000000","message":"Agreed - thanks","commit_id":"e34bc31362fd91a494ae5b24b60b67a105e47756"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"493d1d77b5f660c0e045d37c3fe210d0cca2baf6","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        image[\u0027properties\u0027] \u003d metadata"},{"line_number":107,"context_line":"        try:"},{"line_number":108,"context_line":"            self.image_service.update(context, image_id, image, None)"},{"line_number":109,"context_line":"        except exception.ImageNotAuthorized as e:"},{"line_number":110,"context_line":"            raise exc.HTTPForbidden(explanation\u003de.format_message())"},{"line_number":111,"context_line":"        return dict(metadata\u003dmetadata)"},{"line_number":112,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2Fq0M%3D","line":109,"in_reply_to":"AAAAQn%2F%2FrGQ%3D","updated":"2013-07-25 13:03:45.000000000","message":"The self._get_image() call above should handle that case appropriately.  There could be a race condition leading to ImageNotFound being raised here, but the chances should be very small.  If it is an issue I would prefer to address it in another patch.","commit_id":"e34bc31362fd91a494ae5b24b60b67a105e47756"}]}