)]}'
{"ironic/drivers/modules/ilo/common.py":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"391cee3d45e4f8d8898bd44ea02403b287b264fb","unresolved":false,"context_lines":[{"line_number":781,"context_line":"    except UnicodeError:"},{"line_number":782,"context_line":"        actual_checksum \u003d fileutils.compute_file_checksum(image_location,"},{"line_number":783,"context_line":"                                                          algorithm\u003d\u0027md5\u0027)"},{"line_number":784,"context_line":"    except IOError as e:"},{"line_number":785,"context_line":"        LOG.error(\"Error opening file: %(file)s\", {\u0027file\u0027: image_location})"},{"line_number":786,"context_line":"        raise exception.ImageRefValidationFailed(image_href\u003dimage_location,"},{"line_number":787,"context_line":"                                                 reason\u003de)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fce034c_ee604235","line":784,"updated":"2019-04-15 07:59:23.000000000","message":"During stein cycle, we removed the utils.hash_file and used compute_file_checksum directly, I think we can just do the same, could you replace L780 with L782~783?\nProbably needs to update tests as well.","commit_id":"7352ba82b90739194f62929fd8f4328826cd8521"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"d6a333cf4d598aa957661e83d2266b7a7ae8b26b","unresolved":false,"context_lines":[{"line_number":781,"context_line":"    except UnicodeError:"},{"line_number":782,"context_line":"        actual_checksum \u003d fileutils.compute_file_checksum(image_location,"},{"line_number":783,"context_line":"                                                          algorithm\u003d\u0027md5\u0027)"},{"line_number":784,"context_line":"    except IOError as e:"},{"line_number":785,"context_line":"        LOG.error(\"Error opening file: %(file)s\", {\u0027file\u0027: image_location})"},{"line_number":786,"context_line":"        raise exception.ImageRefValidationFailed(image_href\u003dimage_location,"},{"line_number":787,"context_line":"                                                 reason\u003de)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fce034c_ea458b44","line":784,"in_reply_to":"3fce034c_ee604235","updated":"2019-04-15 10:02:58.000000000","message":"Done.\nBut no need to update the test cases as it is same for both.","commit_id":"7352ba82b90739194f62929fd8f4328826cd8521"}]}