)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c4ccdf61e49eacffb8d59a35a49bfef3a4400002","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"50eb92c0_bcd15b4d","updated":"2026-01-23 10:33:49.000000000","message":"I\u0027m sorry but openstackclient isn\u0027t glanceclient. We don\u0027t want to inherit this broken behavior from OSC.","commit_id":"0d1578e4c031f09a0bcd4d23a5c156cbcd1df4ad"},{"author":{"_account_id":38765,"name":"Piotr Sipika","display_name":"Piotr Sipika","email":"psipika@bloomberg.net","username":"psipika"},"change_message_id":"227d33c09c482653236f58ad886abdd481fb0fde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8ec00f68_af53f923","in_reply_to":"50eb92c0_bcd15b4d","updated":"2026-01-23 19:32:11.000000000","message":"Not sure if I understand your comment. The functionality is broken in openstackclient, not glanceclient. It works in the latter.\nMy changeset was meant to mimic the latter for consistency throughout the codebase, but your suggested edit is simpler and more intuitive, so I\u0027ve accepted it.\nI\u0027ll remove the extra `import` of `strutils` on top as it\u0027s no longer needed.","commit_id":"0d1578e4c031f09a0bcd4d23a5c156cbcd1df4ad"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0388555a79c11ec2c480ef2ce7555a2ed2cfa490","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d9667977_78cb7168","updated":"2026-01-23 15:52:43.000000000","message":"You\u0027re going to need to fix some tests, it seems. Could you also mark comments as resolved once done, please and thanks 🙏","commit_id":"f093b422a726d35d77cc83e7c0790b8360c5ac5a"},{"author":{"_account_id":38765,"name":"Piotr Sipika","display_name":"Piotr Sipika","email":"psipika@bloomberg.net","username":"psipika"},"change_message_id":"56b726e42fcf9804e71fbc97dd121963504e09cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"44d45912_034306bf","in_reply_to":"b8e03816_4e85c170","updated":"2026-01-28 14:45:35.000000000","message":"Tests updated to support the functionality. Looks like build succeeded.\nThanks for your time and patience.","commit_id":"f093b422a726d35d77cc83e7c0790b8360c5ac5a"},{"author":{"_account_id":38765,"name":"Piotr Sipika","display_name":"Piotr Sipika","email":"psipika@bloomberg.net","username":"psipika"},"change_message_id":"227d33c09c482653236f58ad886abdd481fb0fde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b8e03816_4e85c170","in_reply_to":"d9667977_78cb7168","updated":"2026-01-23 19:32:11.000000000","message":"I believe I\u0027ve marked all previous comments as Resolved (checkbox on bottom-left of comment); please let me know if there\u0027s another spot where this needs to happen as I\u0027m unfamiliar with this UI.\nI\u0027ll be addressing test failures ASAP - thank you.","commit_id":"f093b422a726d35d77cc83e7c0790b8360c5ac5a"},{"author":{"_account_id":38765,"name":"Piotr Sipika","display_name":"Piotr Sipika","email":"psipika@bloomberg.net","username":"psipika"},"change_message_id":"3cd27116b122ba5742fc65ebb13edd7021159973","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6f248f12_46e7a5b9","updated":"2026-02-03 15:13:34.000000000","message":"@stephenfin@redhat.com I realize you\u0027ve got your review queue quite full, just wanted to bump this to bring to your attention. Thank you.","commit_id":"a2ea3777d48b216e441b330a16a0e4a54f917fe9"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e6856aaf10ef2505b9e4137aa264340cfa00b34a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"214f5ffd_93b85ee0","in_reply_to":"6f248f12_46e7a5b9","updated":"2026-02-10 14:59:03.000000000","message":"Apologies, I was out last week","commit_id":"a2ea3777d48b216e441b330a16a0e4a54f917fe9"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e6856aaf10ef2505b9e4137aa264340cfa00b34a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"814ce63d_27e12432","updated":"2026-02-10 14:59:03.000000000","message":"I\u0027ve just added a release note. Otherwise LGTM","commit_id":"0803fd2112a6ebb4de7e5f95a00b9bef3e1d86f2"}],"openstackclient/image/v2/image.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e96ecd02fa74f5cd20006cd38c4644a302410a36","unresolved":true,"context_lines":[{"line_number":1719,"context_line":"            ),"},{"line_number":1720,"context_line":"        )"},{"line_number":1721,"context_line":"        stores_group.add_argument("},{"line_number":1722,"context_line":"            \u0027--all-stores\u0027,"},{"line_number":1723,"context_line":"            type\u003dstrutils.bool_from_string,"},{"line_number":1724,"context_line":"            default\u003dNone,"},{"line_number":1725,"context_line":"            help\u003d_("},{"line_number":1726,"context_line":"                \"Make image available to all stores \""},{"line_number":1727,"context_line":"                \"(either \u0027--store\u0027 or \u0027--all-stores\u0027 required with the \""}],"source_content_type":"text/x-python","patch_set":1,"id":"e011e3ed_d8220c5a","line":1724,"range":{"start_line":1722,"start_character":27,"end_line":1724,"end_character":25},"updated":"2026-01-23 10:36:00.000000000","message":"Let\u0027s treat this as a proper boolean option instead. This should be safe given this is broken currently and has only been in one release.\n\n```suggestion\n            \u0027--all-stores\u0027,\n            action\u003d\u0027store_true\u0027,\n```","commit_id":"0d1578e4c031f09a0bcd4d23a5c156cbcd1df4ad"},{"author":{"_account_id":38765,"name":"Piotr Sipika","display_name":"Piotr Sipika","email":"psipika@bloomberg.net","username":"psipika"},"change_message_id":"227d33c09c482653236f58ad886abdd481fb0fde","unresolved":false,"context_lines":[{"line_number":1719,"context_line":"            ),"},{"line_number":1720,"context_line":"        )"},{"line_number":1721,"context_line":"        stores_group.add_argument("},{"line_number":1722,"context_line":"            \u0027--all-stores\u0027,"},{"line_number":1723,"context_line":"            type\u003dstrutils.bool_from_string,"},{"line_number":1724,"context_line":"            default\u003dNone,"},{"line_number":1725,"context_line":"            help\u003d_("},{"line_number":1726,"context_line":"                \"Make image available to all stores \""},{"line_number":1727,"context_line":"                \"(either \u0027--store\u0027 or \u0027--all-stores\u0027 required with the \""}],"source_content_type":"text/x-python","patch_set":1,"id":"d7d36bfa_e7232a02","line":1724,"range":{"start_line":1722,"start_character":27,"end_line":1724,"end_character":25},"in_reply_to":"e011e3ed_d8220c5a","updated":"2026-01-23 19:32:11.000000000","message":"Fix applied.","commit_id":"0d1578e4c031f09a0bcd4d23a5c156cbcd1df4ad"}]}