)]}'
{"zun/image/driver.py":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"4898973d1bd45a933ac402bc469dd812e87bf1f7","unresolved":false,"context_lines":[{"line_number":139,"context_line":"    try:"},{"line_number":140,"context_line":"        image_driver.delete_image(context, img_id)"},{"line_number":141,"context_line":"    except Exception as e:"},{"line_number":142,"context_line":"        LOG.exception(\u0027Unknown exception occurred while deleting image: %s\u0027,"},{"line_number":143,"context_line":"                      six.text_type(e))"},{"line_number":144,"context_line":"        raise exception.ZunException(six.text_type(e))"},{"line_number":145,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_d813d2b7","line":142,"updated":"2017-10-31 15:27:51.000000000","message":"Could you please add the image id as well?\nCan be done in another patch also.","commit_id":"60e5f01e9ebee09c8d4e40885a6132163ee9928d"},{"author":{"_account_id":23055,"name":"miaohb","email":"miao.hongbao@zte.com.cn","username":"miaohb"},"change_message_id":"0f7c387f738988ddafcc3005cd1fed4d5fec55ec","unresolved":false,"context_lines":[{"line_number":139,"context_line":"    try:"},{"line_number":140,"context_line":"        image_driver.delete_image(context, img_id)"},{"line_number":141,"context_line":"    except Exception as e:"},{"line_number":142,"context_line":"        LOG.exception(\u0027Unknown exception occurred while deleting image: %s\u0027,"},{"line_number":143,"context_line":"                      six.text_type(e))"},{"line_number":144,"context_line":"        raise exception.ZunException(six.text_type(e))"},{"line_number":145,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_7eba7cc0","line":142,"in_reply_to":"3f4b6375_d813d2b7","updated":"2017-11-01 01:01:46.000000000","message":"Done","commit_id":"60e5f01e9ebee09c8d4e40885a6132163ee9928d"}],"zun/image/glance/driver.py":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"4898973d1bd45a933ac402bc469dd812e87bf1f7","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        LOG.debug(\u0027Delete an image %s in glance\u0027, img_id)"},{"line_number":158,"context_line":"        try:"},{"line_number":159,"context_line":"            return utils.delete_image(context, img_id)"},{"line_number":160,"context_line":"        except Exception as e:"},{"line_number":161,"context_line":"            raise exception.ZunException(six.text_type(e))"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_b8081691","line":160,"updated":"2017-10-31 15:27:51.000000000","message":"Could you add a statement to log the error?","commit_id":"60e5f01e9ebee09c8d4e40885a6132163ee9928d"},{"author":{"_account_id":23055,"name":"miaohb","email":"miao.hongbao@zte.com.cn","username":"miaohb"},"change_message_id":"0f7c387f738988ddafcc3005cd1fed4d5fec55ec","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        LOG.debug(\u0027Delete an image %s in glance\u0027, img_id)"},{"line_number":158,"context_line":"        try:"},{"line_number":159,"context_line":"            return utils.delete_image(context, img_id)"},{"line_number":160,"context_line":"        except Exception as e:"},{"line_number":161,"context_line":"            raise exception.ZunException(six.text_type(e))"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_ded97022","line":160,"in_reply_to":"3f4b6375_b8081691","updated":"2017-11-01 01:01:46.000000000","message":"Done","commit_id":"60e5f01e9ebee09c8d4e40885a6132163ee9928d"}],"zun/image/glance/utils.py":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"4898973d1bd45a933ac402bc469dd812e87bf1f7","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"def delete_image(context, img_id):"},{"line_number":101,"context_line":"    \"\"\"Delete an image.\"\"\""},{"line_number":102,"context_line":"    LOG.debug(\u0027Delete image %s\u0027, img_id)"},{"line_number":103,"context_line":"    glance \u003d create_glanceclient(context)"},{"line_number":104,"context_line":"    return glance.images.delete(img_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_b82db6fe","line":102,"range":{"start_line":102,"start_character":4,"end_line":102,"end_character":40},"updated":"2017-10-31 15:27:51.000000000","message":"This statement is redundant. It exists in glance/driver.py as well.","commit_id":"60e5f01e9ebee09c8d4e40885a6132163ee9928d"},{"author":{"_account_id":23055,"name":"miaohb","email":"miao.hongbao@zte.com.cn","username":"miaohb"},"change_message_id":"0f7c387f738988ddafcc3005cd1fed4d5fec55ec","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"def delete_image(context, img_id):"},{"line_number":101,"context_line":"    \"\"\"Delete an image.\"\"\""},{"line_number":102,"context_line":"    LOG.debug(\u0027Delete image %s\u0027, img_id)"},{"line_number":103,"context_line":"    glance \u003d create_glanceclient(context)"},{"line_number":104,"context_line":"    return glance.images.delete(img_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_3e3d24dd","line":102,"range":{"start_line":102,"start_character":4,"end_line":102,"end_character":40},"in_reply_to":"3f4b6375_b82db6fe","updated":"2017-11-01 01:01:46.000000000","message":"I think we\u0027d better keep it, because there will be some hard-coded implement like \"image_driver.delete_image\u0027.","commit_id":"60e5f01e9ebee09c8d4e40885a6132163ee9928d"}]}