)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38762,"name":"Sahil Kumbhar","display_name":"sakumbha","email":"sakumbha@redhat.com","username":"sakumbha"},"change_message_id":"f94d899ff202c871bc26a360f8281cd2e1f18e0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"da7e6cfa_98998760","updated":"2026-03-12 10:53:46.000000000","message":"recheck","commit_id":"f10ed97906ac842a760f636fabf914a6e8c9c4cb"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"a1b2af7065eb15ae4428e511a37e1045a79be0e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cca135bd_6ab5c1b6","updated":"2026-05-06 13:54:37.000000000","message":"Looks good to me except for minor formatting issues","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"},{"author":{"_account_id":38762,"name":"Sahil Kumbhar","display_name":"sakumbha","email":"sakumbha@redhat.com","username":"sakumbha"},"change_message_id":"cc5e6f49f831e6b1d9a119b446c15c9d7ca3295d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b714ae3f_30650661","in_reply_to":"cca135bd_6ab5c1b6","updated":"2026-05-07 14:45:25.000000000","message":"Done fixed the formatting issues","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"360318c297f9106c55a9eefc21def7fc68226168","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"25936663_bc19d184","updated":"2026-05-07 14:26:20.000000000","message":"Look good to me! Please mark the opened comments as done if addressed already. Thank you !","commit_id":"edd516a5b88b7ccd6421bdcfe1f0c4198dfb3ede"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"929b4a60c476894c563b49abcd5069b3c4dc8c74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"69f55d46_e2f4ddc6","updated":"2026-05-07 14:09:02.000000000","message":"Looks good to me!!","commit_id":"edd516a5b88b7ccd6421bdcfe1f0c4198dfb3ede"},{"author":{"_account_id":38762,"name":"Sahil Kumbhar","display_name":"sakumbha","email":"sakumbha@redhat.com","username":"sakumbha"},"change_message_id":"cc5e6f49f831e6b1d9a119b446c15c9d7ca3295d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dc9864d2_972a9268","in_reply_to":"25936663_bc19d184","updated":"2026-05-07 14:45:25.000000000","message":"Done","commit_id":"edd516a5b88b7ccd6421bdcfe1f0c4198dfb3ede"}],"specs/2026.2/approved/glance_store/s3-location-without-credentials.rst":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"a1b2af7065eb15ae4428e511a37e1045a79be0e8","unresolved":true,"context_lines":[{"line_number":77,"context_line":"and used on the"},{"line_number":78,"context_line":"Glance side for lazy migration and URI-to-store prefix matching (e.g. the"},{"line_number":79,"context_line":"glance_store.stores entry name such as s3), not as an extra indirection"},{"line_number":80,"context_line":"for which key pair boto uses inside a single backend."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"Upgrade/migration for existing images:"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"987013af_1404fa74","line":80,"range":{"start_line":80,"start_character":10,"end_line":80,"end_character":13},"updated":"2026-05-06 13:54:37.000000000","message":"The paragraph is weirdly formatted, you might want to gqip in vim :)","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"},{"author":{"_account_id":38762,"name":"Sahil Kumbhar","display_name":"sakumbha","email":"sakumbha@redhat.com","username":"sakumbha"},"change_message_id":"cc5e6f49f831e6b1d9a119b446c15c9d7ca3295d","unresolved":false,"context_lines":[{"line_number":77,"context_line":"and used on the"},{"line_number":78,"context_line":"Glance side for lazy migration and URI-to-store prefix matching (e.g. the"},{"line_number":79,"context_line":"glance_store.stores entry name such as s3), not as an extra indirection"},{"line_number":80,"context_line":"for which key pair boto uses inside a single backend."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"Upgrade/migration for existing images:"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f61ab1d_1787b809","line":80,"range":{"start_line":80,"start_character":10,"end_line":80,"end_character":13},"in_reply_to":"987013af_1404fa74","updated":"2026-05-07 14:45:25.000000000","message":"Done","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"a1b2af7065eb15ae4428e511a37e1045a79be0e8","unresolved":true,"context_lines":[{"line_number":167,"context_line":"the URI omits credentials. Migration on service call adds one optional DB"},{"line_number":168,"context_line":"save per image when"},{"line_number":169,"context_line":"an image with a legacy S3 location is first accessed after upgrade; that"},{"line_number":170,"context_line":"matches existing update_store_in_locations behavior."},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"Other deployer impact"},{"line_number":173,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5482dd69_523e4045","line":170,"range":{"start_line":170,"start_character":17,"end_line":170,"end_character":42},"updated":"2026-05-06 13:54:37.000000000","message":"Ditto","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"},{"author":{"_account_id":38762,"name":"Sahil Kumbhar","display_name":"sakumbha","email":"sakumbha@redhat.com","username":"sakumbha"},"change_message_id":"cc5e6f49f831e6b1d9a119b446c15c9d7ca3295d","unresolved":false,"context_lines":[{"line_number":167,"context_line":"the URI omits credentials. Migration on service call adds one optional DB"},{"line_number":168,"context_line":"save per image when"},{"line_number":169,"context_line":"an image with a legacy S3 location is first accessed after upgrade; that"},{"line_number":170,"context_line":"matches existing update_store_in_locations behavior."},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"Other deployer impact"},{"line_number":173,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"33caa824_0f0b1a26","line":170,"range":{"start_line":170,"start_character":17,"end_line":170,"end_character":42},"in_reply_to":"5482dd69_523e4045","updated":"2026-05-07 14:45:25.000000000","message":"Acknowledged","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"a1b2af7065eb15ae4428e511a37e1045a79be0e8","unresolved":true,"context_lines":[{"line_number":238,"context_line":"migration-on-access: loading an image with legacy S3 location triggers"},{"line_number":239,"context_line":"rewrite to credential-free and save (single-store and multistore). No new"},{"line_number":240,"context_line":"tempest tests required;"},{"line_number":241,"context_line":"existing image API and S3 store tests remain valid."},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"Documentation Impact"},{"line_number":244,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"523723b7_2b4d2e4d","line":241,"range":{"start_line":241,"start_character":26,"end_line":241,"end_character":31},"updated":"2026-05-06 13:54:37.000000000","message":"Ditto","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"},{"author":{"_account_id":38762,"name":"Sahil Kumbhar","display_name":"sakumbha","email":"sakumbha@redhat.com","username":"sakumbha"},"change_message_id":"cc5e6f49f831e6b1d9a119b446c15c9d7ca3295d","unresolved":false,"context_lines":[{"line_number":238,"context_line":"migration-on-access: loading an image with legacy S3 location triggers"},{"line_number":239,"context_line":"rewrite to credential-free and save (single-store and multistore). No new"},{"line_number":240,"context_line":"tempest tests required;"},{"line_number":241,"context_line":"existing image API and S3 store tests remain valid."},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"Documentation Impact"},{"line_number":244,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"87204710_7f642b08","line":241,"range":{"start_line":241,"start_character":26,"end_line":241,"end_character":31},"in_reply_to":"523723b7_2b4d2e4d","updated":"2026-05-07 14:45:25.000000000","message":"Acknowledged","commit_id":"26b9bba4539426cc2859470fc42ec17c3360f7fd"}]}