)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"e09ba47a2d47b79f9a21c017177edc89a92a0f8c","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for downloading image from preferred stores"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added quey parameter for store suggesetion while downloading the"},{"line_number":10,"context_line":"image to existing image-download command."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related: blueprint bp/download-from-specific-store"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"94377ebb_68348657","line":9,"range":{"start_line":9,"start_character":6,"end_line":9,"end_character":10},"updated":"2026-03-04 10:49:46.000000000","message":"*query ?","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"fc089336bed3fafca8b1a82330b49e6980ad6766","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for downloading image from preferred stores"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added quey parameter for store suggesetion while downloading the"},{"line_number":10,"context_line":"image to existing image-download command."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related: blueprint bp/download-from-specific-store"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"6781852b_9b938d94","line":9,"range":{"start_line":9,"start_character":31,"end_line":9,"end_character":42},"updated":"2026-02-27 20:34:38.000000000","message":"If you need another patchset: suggestion*","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"14a5b6e9901a27d13c0e279ddc88bbc074533814","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for downloading image from preferred stores"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added quey parameter for store suggesetion while downloading the"},{"line_number":10,"context_line":"image to existing image-download command."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related: blueprint bp/download-from-specific-store"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7dc16e1f_54a05a63","line":9,"range":{"start_line":9,"start_character":31,"end_line":9,"end_character":42},"in_reply_to":"6781852b_9b938d94","updated":"2026-04-02 06:18:52.000000000","message":"Acknowledged","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"14a5b6e9901a27d13c0e279ddc88bbc074533814","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for downloading image from preferred stores"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added quey parameter for store suggesetion while downloading the"},{"line_number":10,"context_line":"image to existing image-download command."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related: blueprint bp/download-from-specific-store"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"d6dd4f88_75b7ff7c","line":9,"range":{"start_line":9,"start_character":6,"end_line":9,"end_character":10},"in_reply_to":"94377ebb_68348657","updated":"2026-04-02 06:18:52.000000000","message":"Acknowledged","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"fc089336bed3fafca8b1a82330b49e6980ad6766","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"017c882d_8e828288","updated":"2026-02-27 20:34:38.000000000","message":"A couple of nitpicks if we need another patchset. I\u0027ll give Pranali a couple of days to review this before pushing.","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"e09ba47a2d47b79f9a21c017177edc89a92a0f8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"137e0923_380d5b4f","updated":"2026-03-04 10:49:46.000000000","message":"Just some extra nitpicks if planning for new patch set.","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"14a5b6e9901a27d13c0e279ddc88bbc074533814","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ab6a1da5_0d5717e8","updated":"2026-04-02 06:18:52.000000000","message":"Thank you for review!!","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"b0dc67f976fe4a61f7db9bd0421bd696250b4890","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2808397e_ca741e6d","updated":"2026-04-02 14:57:08.000000000","message":"My comments and Pranali\u0027s have been addressed, merging this.","commit_id":"b7274aa4d22b5ab6e90fc1a9e18d5c6fff95f1e6"}],"glanceclient/v2/shell.py":[{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"e09ba47a2d47b79f9a21c017177edc89a92a0f8c","unresolved":true,"context_lines":[{"line_number":692,"context_line":"        utils.exit(msg)"},{"line_number":693,"context_line":""},{"line_number":694,"context_line":"    prefer \u003d None"},{"line_number":695,"context_line":"    if hasattr(args, \u0027prefer\u0027) and args.prefer:"},{"line_number":696,"context_line":"        prefer \u003d [s.strip() for s in args.prefer.split(\u0027,\u0027) if s.strip()]"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"f52b410f_92b91ebe","line":695,"range":{"start_line":695,"start_character":35,"end_line":695,"end_character":46},"updated":"2026-03-04 10:49:46.000000000","message":"Isn\u0027t `if args.prefer:` enough here? since --prefer is defined via the @utils.arg decorator, args.prefer will always exist (defaulting to None) and hasattr is redundant.","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"14a5b6e9901a27d13c0e279ddc88bbc074533814","unresolved":false,"context_lines":[{"line_number":692,"context_line":"        utils.exit(msg)"},{"line_number":693,"context_line":""},{"line_number":694,"context_line":"    prefer \u003d None"},{"line_number":695,"context_line":"    if hasattr(args, \u0027prefer\u0027) and args.prefer:"},{"line_number":696,"context_line":"        prefer \u003d [s.strip() for s in args.prefer.split(\u0027,\u0027) if s.strip()]"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"6921b22e_5fe0665e","line":695,"range":{"start_line":695,"start_character":35,"end_line":695,"end_character":46},"in_reply_to":"f52b410f_92b91ebe","updated":"2026-04-02 06:18:52.000000000","message":"Acknowledged","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"fc089336bed3fafca8b1a82330b49e6980ad6766","unresolved":true,"context_lines":[{"line_number":693,"context_line":""},{"line_number":694,"context_line":"    prefer \u003d None"},{"line_number":695,"context_line":"    if hasattr(args, \u0027prefer\u0027) and args.prefer:"},{"line_number":696,"context_line":"        prefer \u003d [s.strip() for s in args.prefer.split(\u0027,\u0027) if s.strip()]"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"    try:"},{"line_number":699,"context_line":"        body \u003d gc.images.data(args.id,"}],"source_content_type":"text/x-python","patch_set":3,"id":"59a49d43_50818aaa","line":696,"range":{"start_line":696,"start_character":63,"end_line":696,"end_character":70},"updated":"2026-02-27 20:34:38.000000000","message":"Not a huge performance issue, but we call s.strip() twice. Maybe:\n\n    [store for store in [arg.strip() for arg in args.prefer.split(\u0027,\u0027)] if store]\n\nIf you need another patchset?","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"14a5b6e9901a27d13c0e279ddc88bbc074533814","unresolved":false,"context_lines":[{"line_number":693,"context_line":""},{"line_number":694,"context_line":"    prefer \u003d None"},{"line_number":695,"context_line":"    if hasattr(args, \u0027prefer\u0027) and args.prefer:"},{"line_number":696,"context_line":"        prefer \u003d [s.strip() for s in args.prefer.split(\u0027,\u0027) if s.strip()]"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"    try:"},{"line_number":699,"context_line":"        body \u003d gc.images.data(args.id,"}],"source_content_type":"text/x-python","patch_set":3,"id":"286f6790_9adddd39","line":696,"range":{"start_line":696,"start_character":63,"end_line":696,"end_character":70},"in_reply_to":"59a49d43_50818aaa","updated":"2026-04-02 06:18:52.000000000","message":"Acknowledged","commit_id":"bb4c1037887d3d7c1f8bbf7bc612cdde593bee8f"}]}