)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"ffa96daf0bd147ac9335f722b2d02adee9ac059b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"12b4e714_a06ade0b","updated":"2024-10-24 16:13:37.000000000","message":"previous recheck didn\u0027t run?","commit_id":"827aff84748320829500e83459eb068c73534bf5"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"7178fa30c8f68ad0265d0e87d5b555de93709607","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1909e2b6_c41b12f7","updated":"2024-10-23 18:25:17.000000000","message":"recheck","commit_id":"827aff84748320829500e83459eb068c73534bf5"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"67972ad4a06f6a6214d045236cbb22ae01254e1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6dd9d3b0_a29eeee3","updated":"2024-10-24 16:13:26.000000000","message":"recheck","commit_id":"827aff84748320829500e83459eb068c73534bf5"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"efa6aa94ea2b9fa11b77cd43ef5bbcaaad1c20df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"476439fb_aa0af5f7","updated":"2024-10-23 18:25:40.000000000","message":"unit test error doesn\u0027t make sense, it is not related to the patch (also I have run locally and works fine)","commit_id":"827aff84748320829500e83459eb068c73534bf5"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"a62fd79d5c80c25fcf4b6f8099452866ad6987fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c8986128_c1d3d4e8","updated":"2024-10-25 14:29:56.000000000","message":"recheck","commit_id":"8b80383051e713abf0243142b38667c4b20b1abd"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8f32c055df125acd25328558b1cb43ec198dee62","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"86e0b33f_a9a51d58","updated":"2024-11-06 18:14:07.000000000","message":"Also, please add a release note, something about \u0027signature_verified\u0027 being a volume property that indicates that the volume was created from an image whose signature was verified, and this is not a property that applies to the image created when such a volume is uploaded as an image.","commit_id":"876e397af2da997c1450358139988ca681484ea3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"82ba07d668c47970ba4f4fddba0265e0e8a04004","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"85a8b456_f8f191f4","updated":"2024-11-06 15:41:50.000000000","message":"You need to update the help text for the related option; otherwise, LGTM.","commit_id":"876e397af2da997c1450358139988ca681484ea3"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"46171dab0df1544085394fc1f5f7cf76de7dbdd2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"14aa415b_e377001b","updated":"2024-10-28 09:49:13.000000000","message":"recheck","commit_id":"876e397af2da997c1450358139988ca681484ea3"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"87cf6e6c9588d11085725b9a7964f12ac67ae187","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dfd23882_536356ac","updated":"2024-11-07 14:24:23.000000000","message":"Looks good to me and has been reviewed by another core.","commit_id":"c65f43cb989f7e1ad5a8b999e6f3e266cddb36ee"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"45989ccbe0facfcc9d4b85c18c6e0894b532afbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d9399a06_6624ebde","updated":"2024-11-06 22:40:58.000000000","message":"Revision LGTM.","commit_id":"c65f43cb989f7e1ad5a8b999e6f3e266cddb36ee"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"59321edc9ba11378872f0454f06afd7af905dc64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"638a8344_d77c1cdf","updated":"2024-11-06 20:16:38.000000000","message":"thanks Brian for the review. Please take a look at the latest patchset when you have a minute. Thanks!","commit_id":"c65f43cb989f7e1ad5a8b999e6f3e266cddb36ee"}],"cinder/image/image_utils.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"82ba07d668c47970ba4f4fddba0265e0e8a04004","unresolved":true,"context_lines":[{"line_number":102,"context_line":"                     \u0027an operator has configured glance property protections \u0027"},{"line_number":103,"context_line":"                     \u0027to make some image properties read-only. Cinder will \u0027"},{"line_number":104,"context_line":"                     \u0027*always* filter out image metadata in the namespaces \u0027"},{"line_number":105,"context_line":"                     \u0027`os_glance` and `img_signature`; this configuration \u0027"},{"line_number":106,"context_line":"                     \u0027option allows operators to specify *additional* \u0027"},{"line_number":107,"context_line":"                     \u0027namespaces to be excluded.\u0027,"},{"line_number":108,"context_line":"                default\u003d[]),"}],"source_content_type":"text/x-python","patch_set":3,"id":"a68f6fad_867af352","line":105,"updated":"2024-11-06 15:41:50.000000000","message":"Need to add \u0027signature_verified\u0027 here.","commit_id":"876e397af2da997c1450358139988ca681484ea3"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"59321edc9ba11378872f0454f06afd7af905dc64","unresolved":false,"context_lines":[{"line_number":102,"context_line":"                     \u0027an operator has configured glance property protections \u0027"},{"line_number":103,"context_line":"                     \u0027to make some image properties read-only. Cinder will \u0027"},{"line_number":104,"context_line":"                     \u0027*always* filter out image metadata in the namespaces \u0027"},{"line_number":105,"context_line":"                     \u0027`os_glance` and `img_signature`; this configuration \u0027"},{"line_number":106,"context_line":"                     \u0027option allows operators to specify *additional* \u0027"},{"line_number":107,"context_line":"                     \u0027namespaces to be excluded.\u0027,"},{"line_number":108,"context_line":"                default\u003d[]),"}],"source_content_type":"text/x-python","patch_set":3,"id":"a6552575_f8036c81","line":105,"in_reply_to":"a68f6fad_867af352","updated":"2024-11-06 20:16:38.000000000","message":"Done","commit_id":"876e397af2da997c1450358139988ca681484ea3"}]}