)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"0f56ccab8c0576724d61ab333e95750e90acebdf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2e454270_4de47b46","updated":"2025-02-25 15:50:57.000000000","message":"CI is currently busted (not your fault), but this patch looks good. Thanks!","commit_id":"e7d1f8e21114ea742201463b90742757cdaed8df"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"1637c3dee97decf7bca12db5777b534897de10ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"47d6a7be_bedc5eac","updated":"2025-02-27 17:33:48.000000000","message":"recheck last CI run was during glance breakages","commit_id":"e7d1f8e21114ea742201463b90742757cdaed8df"}],"ironic/common/glance_service/service_utils.py":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"414d3c3a6aa3f924f1f1750063711a8c4a63091f","unresolved":true,"context_lines":[{"line_number":128,"context_line":" "},{"line_number":129,"context_line":"    LOG.warning(\"Image access denied\")"},{"line_number":130,"context_line":"    LOG.warning(\"Reason: Owner mismatch or insufficient permissions\")"},{"line_number":131,"context_line":"    return False"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"def is_image_active(image):"},{"line_number":134,"context_line":"    \"\"\"Check the image status."}],"source_content_type":"text/x-python","patch_set":1,"id":"8c966c2c_1bbf572d","line":131,"updated":"2025-02-24 22:07:24.000000000","message":"A suggested alternate construction?\n\n\n```suggestion\n   ## Psuedocode\n   if not (context.project_id ..... ):\n       LOG.info(\"Access to {image} owned by {img_project} denied to requester {project}\", {\"image\": getattr(image, \u0027id\u0027, \u0027unknown\u0027), ge.get(id), \"img_project\": getattr(image, \u0027owner\u0027, \u0027unknown\u0027), \"project\": getattr(context, \u0027project_id\u0027, \u0027unknown\u0027)})\n       return False\n   else\n       return True\n```\n\nYou could also set the getattr() things above to variables and use them in the initial condition to simplify the code.\n\nPlease fix all pep8 issues as well.","commit_id":"b984666bce754e1fd45793333b002d567c5c9fb7"},{"author":{"_account_id":37707,"name":"Satoshi Shirosaka","display_name":"Satoshi S.","email":"satoss1108@gmail.com","username":"satoshi-sh"},"change_message_id":"4ae692647d7603b87c7c5e23d05c454e2346ac7d","unresolved":false,"context_lines":[{"line_number":128,"context_line":" "},{"line_number":129,"context_line":"    LOG.warning(\"Image access denied\")"},{"line_number":130,"context_line":"    LOG.warning(\"Reason: Owner mismatch or insufficient permissions\")"},{"line_number":131,"context_line":"    return False"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"def is_image_active(image):"},{"line_number":134,"context_line":"    \"\"\"Check the image status."}],"source_content_type":"text/x-python","patch_set":1,"id":"f7de5aaa_7890f9cc","line":131,"in_reply_to":"8c966c2c_1bbf572d","updated":"2025-02-25 01:33:14.000000000","message":"Acknowledged","commit_id":"b984666bce754e1fd45793333b002d567c5c9fb7"}]}