)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"ca29d4382e515b357c82371475ed69aa3678fbf1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0f352db7_cc5654c9","updated":"2026-08-24 08:57:13.000000000","message":"Hi @noonedeadpunk@gmail.com, just following up on this patch. Could you please add Workflow +1 if it looks ready to merge? Thanks!","commit_id":"68b35a0cbd258a99724bf6be22695f2fe7105922"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"0a925f85c4c3eacef9ba43fca57f6d5e8550c28a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"743b5971_179ec06c","updated":"2026-08-21 10:09:50.000000000","message":"lgtm, thanks!","commit_id":"68b35a0cbd258a99724bf6be22695f2fe7105922"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f7109028bec542aed93be3e16b0c519d196753dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8ea6cbc5_e880c28f","in_reply_to":"0f352db7_cc5654c9","updated":"2026-08-24 15:06:39.000000000","message":"I want to wait for a couple of days before merging to give opportunity for other contributors to step in with reviews.","commit_id":"68b35a0cbd258a99724bf6be22695f2fe7105922"}],"freezer/job.py":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"5709dcf69210249ce81ae5e60691f002c4fa152c","unresolved":true,"context_lines":[{"line_number":287,"context_line":"                  \u0027cindernative_backup_az\u0027,"},{"line_number":288,"context_line":"                  \u0027cindernative_backup_container\u0027,"},{"line_number":289,"context_line":"                  \u0027nova_inst_id\u0027,"},{"line_number":290,"context_line":"                  \u0027nova_inst_name\u0027,"},{"line_number":291,"context_line":"                  \u0027cinder_vol_id\u0027,"},{"line_number":292,"context_line":"                  \u0027engine_name\u0027,"},{"line_number":293,"context_line":"                  ]"}],"source_content_type":"text/x-python","patch_set":1,"id":"2100b89f_6fde9a6a","line":290,"updated":"2026-08-20 13:56:52.000000000","message":"Are you sure `nova_inst_name` makes sense? As instance name can change, but it won\u0027t be reflected here.","commit_id":"06f28202aab922322f1526720355ef353ec04951"},{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"01380d2b69e26c97249afd6f376cd0967a085908","unresolved":true,"context_lines":[{"line_number":287,"context_line":"                  \u0027cindernative_backup_az\u0027,"},{"line_number":288,"context_line":"                  \u0027cindernative_backup_container\u0027,"},{"line_number":289,"context_line":"                  \u0027nova_inst_id\u0027,"},{"line_number":290,"context_line":"                  \u0027nova_inst_name\u0027,"},{"line_number":291,"context_line":"                  \u0027cinder_vol_id\u0027,"},{"line_number":292,"context_line":"                  \u0027engine_name\u0027,"},{"line_number":293,"context_line":"                  ]"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae67190_f1c0578a","line":290,"in_reply_to":"2100b89f_6fde9a6a","updated":"2026-08-20 16:05:38.000000000","message":"Dropped nova_inst_name — agreed that it\u0027s mutable and isn\u0027t used to locate the backup during restore, which relies on nova_inst_id. Kept the metadata limited to stable resource IDs.","commit_id":"06f28202aab922322f1526720355ef353ec04951"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"0a925f85c4c3eacef9ba43fca57f6d5e8550c28a","unresolved":false,"context_lines":[{"line_number":287,"context_line":"                  \u0027cindernative_backup_az\u0027,"},{"line_number":288,"context_line":"                  \u0027cindernative_backup_container\u0027,"},{"line_number":289,"context_line":"                  \u0027nova_inst_id\u0027,"},{"line_number":290,"context_line":"                  \u0027nova_inst_name\u0027,"},{"line_number":291,"context_line":"                  \u0027cinder_vol_id\u0027,"},{"line_number":292,"context_line":"                  \u0027engine_name\u0027,"},{"line_number":293,"context_line":"                  ]"}],"source_content_type":"text/x-python","patch_set":1,"id":"30331cee_3596a986","line":290,"in_reply_to":"dae67190_f1c0578a","updated":"2026-08-21 10:09:50.000000000","message":"Done","commit_id":"06f28202aab922322f1526720355ef353ec04951"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"5709dcf69210249ce81ae5e60691f002c4fa152c","unresolved":true,"context_lines":[{"line_number":288,"context_line":"                  \u0027cindernative_backup_container\u0027,"},{"line_number":289,"context_line":"                  \u0027nova_inst_id\u0027,"},{"line_number":290,"context_line":"                  \u0027nova_inst_name\u0027,"},{"line_number":291,"context_line":"                  \u0027cinder_vol_id\u0027,"},{"line_number":292,"context_line":"                  \u0027engine_name\u0027,"},{"line_number":293,"context_line":"                  ]"},{"line_number":294,"context_line":"        for field_name in fields:"}],"source_content_type":"text/x-python","patch_set":1,"id":"c7ce38b1_6157feed","line":291,"updated":"2026-08-20 13:56:52.000000000","message":"Should we also add `cinderbrick_vol_id`, and `glance_image_id`?","commit_id":"06f28202aab922322f1526720355ef353ec04951"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"0a925f85c4c3eacef9ba43fca57f6d5e8550c28a","unresolved":false,"context_lines":[{"line_number":288,"context_line":"                  \u0027cindernative_backup_container\u0027,"},{"line_number":289,"context_line":"                  \u0027nova_inst_id\u0027,"},{"line_number":290,"context_line":"                  \u0027nova_inst_name\u0027,"},{"line_number":291,"context_line":"                  \u0027cinder_vol_id\u0027,"},{"line_number":292,"context_line":"                  \u0027engine_name\u0027,"},{"line_number":293,"context_line":"                  ]"},{"line_number":294,"context_line":"        for field_name in fields:"}],"source_content_type":"text/x-python","patch_set":1,"id":"02320790_228c0ad6","line":291,"in_reply_to":"c2ef476d_c8a6fbdd","updated":"2026-08-21 10:09:50.000000000","message":"Done","commit_id":"06f28202aab922322f1526720355ef353ec04951"},{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"01380d2b69e26c97249afd6f376cd0967a085908","unresolved":true,"context_lines":[{"line_number":288,"context_line":"                  \u0027cindernative_backup_container\u0027,"},{"line_number":289,"context_line":"                  \u0027nova_inst_id\u0027,"},{"line_number":290,"context_line":"                  \u0027nova_inst_name\u0027,"},{"line_number":291,"context_line":"                  \u0027cinder_vol_id\u0027,"},{"line_number":292,"context_line":"                  \u0027engine_name\u0027,"},{"line_number":293,"context_line":"                  ]"},{"line_number":294,"context_line":"        for field_name in fields:"}],"source_content_type":"text/x-python","patch_set":1,"id":"c2ef476d_c8a6fbdd","line":291,"in_reply_to":"c7ce38b1_6157feed","updated":"2026-08-20 16:05:38.000000000","message":"Good point — added both glance_image_id and cinderbrick_vol_id so all resource-based modes (nova/cinder/cindernative/cinderbrick/glance) are self-describing, consistent with the rest. Also added a glance-mode metadata test.","commit_id":"06f28202aab922322f1526720355ef353ec04951"}]}
