)]}'
{"cinder/backup/chunkeddriver.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4b1464e8d693c5872d8ba17f451506bd2e5c74b5","unresolved":false,"context_lines":[{"line_number":796,"context_line":"        # Do a full restore first, then layer the incremental backups"},{"line_number":797,"context_line":"        # on top of it in order."},{"line_number":798,"context_line":"        is_base \u003d True"},{"line_number":799,"context_line":"        for backup1 in backup_list:"},{"line_number":800,"context_line":"            # ``volume_is_new`` lets the base restore skip writing all-zero"},{"line_number":801,"context_line":"            # chunks (a pristine target already reads as zero).  That is only"},{"line_number":802,"context_line":"            # safe for the *first* backup applied: once the base has written"}],"source_content_type":"text/x-python","patch_set":1,"id":"d9486b3f_05bf4459","line":799,"range":{"start_line":799,"start_character":12,"end_line":799,"end_character":19},"updated":"2026-09-10 17:30:42.000000000","message":"\"backup1\" isn\u0027t a great name here either, could maybe change that too.","commit_id":"8d8976094a0a05726e20f5a6ceba93e88a8423e7"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"e21dbaeadccd4b21a0050fc04415a9adaa193f82","unresolved":true,"context_lines":[{"line_number":791,"context_line":"            backup_list.append(prev_backup)"},{"line_number":792,"context_line":"            current_backup \u003d prev_backup"},{"line_number":793,"context_line":""},{"line_number":794,"context_line":"        backup_list.reverse()"},{"line_number":795,"context_line":""},{"line_number":796,"context_line":"        # Do a full restore first, then layer the incremental backups"},{"line_number":797,"context_line":"        # on top of it in order."}],"source_content_type":"text/x-python","patch_set":2,"id":"da1addfb_297554a1","line":794,"updated":"2026-09-17 03:26:16.000000000","message":"After this reverse() the full backup is first in backup_list, not last.\n\nThe comment at #782-783 still says last one in the list :-)","commit_id":"e38a8c044114afb6feb9f50177f90e0eb2d25003"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"415defdb7eb9c267089d230cc511ae0ea31f9792","unresolved":false,"context_lines":[{"line_number":791,"context_line":"            backup_list.append(prev_backup)"},{"line_number":792,"context_line":"            current_backup \u003d prev_backup"},{"line_number":793,"context_line":""},{"line_number":794,"context_line":"        backup_list.reverse()"},{"line_number":795,"context_line":""},{"line_number":796,"context_line":"        # Do a full restore first, then layer the incremental backups"},{"line_number":797,"context_line":"        # on top of it in order."}],"source_content_type":"text/x-python","patch_set":2,"id":"248b7f22_4f07555c","line":794,"in_reply_to":"da1addfb_297554a1","updated":"2026-09-17 12:25:59.000000000","message":"Good catch, thanks.","commit_id":"e38a8c044114afb6feb9f50177f90e0eb2d25003"}]}