)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c04606cdee62de6faeaa76fec04313ad081836d0","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Sean Mooney \u003cwork@seanmooney.info\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-04-10 13:01:56 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Require conformation to reset server state."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change updates the server set state command"},{"line_number":10,"context_line":"to require conformation before it is applied."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"adbe7839_945e4ff3","line":7,"range":{"start_line":7,"start_character":8,"end_line":7,"end_character":20},"updated":"2025-04-10 13:18:50.000000000","message":"confirmation","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c4301b31aa488f41045d355fc630b6fb627fde51","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sean Mooney \u003cwork@seanmooney.info\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-04-10 13:01:56 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Require conformation to reset server state."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change updates the server set state command"},{"line_number":10,"context_line":"to require conformation before it is applied."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fd97a086_9eb4f20c","line":7,"range":{"start_line":7,"start_character":8,"end_line":7,"end_character":20},"in_reply_to":"adbe7839_945e4ff3","updated":"2025-04-10 18:07:10.000000000","message":"Done","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c04606cdee62de6faeaa76fec04313ad081836d0","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Require conformation to reset server state."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change updates the server set state command"},{"line_number":10,"context_line":"to require conformation before it is applied."},{"line_number":11,"context_line":"The same pattern as project clean is used and"},{"line_number":12,"context_line":"a new --auto-approve flag is added to allow skipping"},{"line_number":13,"context_line":"the prompt."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0bf35830_5cf01e72","line":10,"range":{"start_line":10,"start_character":11,"end_line":10,"end_character":23},"updated":"2025-04-10 13:18:50.000000000","message":"confirmation","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c4301b31aa488f41045d355fc630b6fb627fde51","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Require conformation to reset server state."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change updates the server set state command"},{"line_number":10,"context_line":"to require conformation before it is applied."},{"line_number":11,"context_line":"The same pattern as project clean is used and"},{"line_number":12,"context_line":"a new --auto-approve flag is added to allow skipping"},{"line_number":13,"context_line":"the prompt."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8ab034af_15df72a7","line":10,"range":{"start_line":10,"start_character":11,"end_line":10,"end_character":23},"in_reply_to":"0bf35830_5cf01e72","updated":"2025-04-10 18:07:10.000000000","message":"Done","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c04606cdee62de6faeaa76fec04313ad081836d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1926a893_2bfaf102","updated":"2025-04-10 13:18:50.000000000","message":"Thanks, Sean. Couple of nits inline. Can we also get an `upgrade`-type release note too, please, since this is a \"breaking\" change of sorts.","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"47505524dfd79e1f6974b815052bfaed5022f12a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"649bff37_aad1e76d","in_reply_to":"1926a893_2bfaf102","updated":"2025-04-10 17:58:01.000000000","message":"ah missed this ill do that shortly","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c4301b31aa488f41045d355fc630b6fb627fde51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3643e366_f846b56c","in_reply_to":"649bff37_aad1e76d","updated":"2025-04-10 18:07:10.000000000","message":"Done","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"59d5bb710ca57fde5f3b9d73c43a6c4e1b5af0bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b6aa141e_dbb0d3e7","updated":"2025-04-11 10:54:13.000000000","message":"Fixed my own nits. LGTM now!","commit_id":"25cd1178b31a3d5582ab4ad77518fb63a856fd88"}],"openstackclient/compute/v2/server.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c04606cdee62de6faeaa76fec04313ad081836d0","unresolved":true,"context_lines":[{"line_number":4462,"context_line":"                \u0027(repeat option to set multiple properties)\u0027"},{"line_number":4463,"context_line":"            ),"},{"line_number":4464,"context_line":"        )"},{"line_number":4465,"context_line":""},{"line_number":4466,"context_line":"        parser.add_argument("},{"line_number":4467,"context_line":"            \u0027--auto-approve\u0027,"},{"line_number":4468,"context_line":"            action\u003d\u0027store_true\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"28afe26c_30e295f8","line":4465,"updated":"2025-04-10 13:18:50.000000000","message":"nit: drop the newlines before/after this","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"234900945dc8da06d21b8b71e006a496b1ba5ae8","unresolved":false,"context_lines":[{"line_number":4462,"context_line":"                \u0027(repeat option to set multiple properties)\u0027"},{"line_number":4463,"context_line":"            ),"},{"line_number":4464,"context_line":"        )"},{"line_number":4465,"context_line":""},{"line_number":4466,"context_line":"        parser.add_argument("},{"line_number":4467,"context_line":"            \u0027--auto-approve\u0027,"},{"line_number":4468,"context_line":"            action\u003d\u0027store_true\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"c6387606_5d8ab69a","line":4465,"in_reply_to":"28afe26c_30e295f8","updated":"2025-04-10 17:57:29.000000000","message":"Done","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c04606cdee62de6faeaa76fec04313ad081836d0","unresolved":true,"context_lines":[{"line_number":4476,"context_line":"            metavar\u003d\u0027\u003cstate\u003e\u0027,"},{"line_number":4477,"context_line":"            choices\u003d[\u0027active\u0027, \u0027error\u0027],"},{"line_number":4478,"context_line":"            help\u003d_("},{"line_number":4479,"context_line":"                \u0027New server state.\u0027"},{"line_number":4480,"context_line":"                \u0027If the server is in an error state this is almost never the \u0027"},{"line_number":4481,"context_line":"                \u0027correct command to run, prefer hard reboot where possible. \u0027"},{"line_number":4482,"context_line":"                \u0027**WARNING** Resetting the state is intented to workaround \u0027"},{"line_number":4483,"context_line":"                \u0027servers stuck in an intermediate state, such as deleting. \u0027"},{"line_number":4484,"context_line":"                \u0027If the server is in error due to a move operation, setting \u0027"},{"line_number":4485,"context_line":"                \u0027the state can result in instances that are no longer \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"2f447a86_7e870b33","line":4482,"range":{"start_line":4479,"start_character":35,"end_line":4482,"end_character":76},"updated":"2025-04-10 13:18:50.000000000","message":"```suggestion\n                \u0027New server state.\u0027\n                \u0027**WARNING** If the server is in an error state this is \u0027\n                \u0027almost never the correct command to run, prefer hard \u0027\n                \u0027reboot where possible. Resetting the state is intended \u0027\n                \u0027to workaround \u0027\n```","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"234900945dc8da06d21b8b71e006a496b1ba5ae8","unresolved":false,"context_lines":[{"line_number":4476,"context_line":"            metavar\u003d\u0027\u003cstate\u003e\u0027,"},{"line_number":4477,"context_line":"            choices\u003d[\u0027active\u0027, \u0027error\u0027],"},{"line_number":4478,"context_line":"            help\u003d_("},{"line_number":4479,"context_line":"                \u0027New server state.\u0027"},{"line_number":4480,"context_line":"                \u0027If the server is in an error state this is almost never the \u0027"},{"line_number":4481,"context_line":"                \u0027correct command to run, prefer hard reboot where possible. \u0027"},{"line_number":4482,"context_line":"                \u0027**WARNING** Resetting the state is intented to workaround \u0027"},{"line_number":4483,"context_line":"                \u0027servers stuck in an intermediate state, such as deleting. \u0027"},{"line_number":4484,"context_line":"                \u0027If the server is in error due to a move operation, setting \u0027"},{"line_number":4485,"context_line":"                \u0027the state can result in instances that are no longer \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"772a0366_4678cf89","line":4482,"range":{"start_line":4479,"start_character":35,"end_line":4482,"end_character":76},"in_reply_to":"2f447a86_7e870b33","updated":"2025-04-10 17:57:29.000000000","message":"Done","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c04606cdee62de6faeaa76fec04313ad081836d0","unresolved":true,"context_lines":[{"line_number":4584,"context_line":"        if parsed_args.state:"},{"line_number":4585,"context_line":"            if not parsed_args.auto_approve:"},{"line_number":4586,"context_line":"                if not self.ask_user_yesno("},{"line_number":4587,"context_line":"                    _("},{"line_number":4588,"context_line":"                        \"Resseting the server state can make it much harder \""},{"line_number":4589,"context_line":"                        \"to recover a server from an error state. If its in \""},{"line_number":4590,"context_line":"                        \"error due to a move operation this is likely not \""},{"line_number":4591,"context_line":"                        \"the correct approch to fix the problem. Are you sure\""},{"line_number":4592,"context_line":"                    )"},{"line_number":4593,"context_line":"                ):"},{"line_number":4594,"context_line":"                    return"}],"source_content_type":"text/x-python","patch_set":1,"id":"01a2f22b_f49ff08e","line":4591,"range":{"start_line":4587,"start_character":22,"end_line":4591,"end_character":78},"updated":"2025-04-10 13:18:50.000000000","message":"```suggestion\n                    _(\n                        \"Resetting the server state can make it much harder \"\n                        \"to recover a server from an error state. If the \"\n                        \"server is in error statue due to a failed move \"\n                        \"operation then this is likely not the correct \"\n                        \"approach to fix the problem. Do you wish to continue?\"\n```","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"234900945dc8da06d21b8b71e006a496b1ba5ae8","unresolved":false,"context_lines":[{"line_number":4584,"context_line":"        if parsed_args.state:"},{"line_number":4585,"context_line":"            if not parsed_args.auto_approve:"},{"line_number":4586,"context_line":"                if not self.ask_user_yesno("},{"line_number":4587,"context_line":"                    _("},{"line_number":4588,"context_line":"                        \"Resseting the server state can make it much harder \""},{"line_number":4589,"context_line":"                        \"to recover a server from an error state. If its in \""},{"line_number":4590,"context_line":"                        \"error due to a move operation this is likely not \""},{"line_number":4591,"context_line":"                        \"the correct approch to fix the problem. Are you sure\""},{"line_number":4592,"context_line":"                    )"},{"line_number":4593,"context_line":"                ):"},{"line_number":4594,"context_line":"                    return"}],"source_content_type":"text/x-python","patch_set":1,"id":"4ad8f55d_4f99e254","line":4591,"range":{"start_line":4587,"start_character":22,"end_line":4591,"end_character":78},"in_reply_to":"01a2f22b_f49ff08e","updated":"2025-04-10 17:57:29.000000000","message":"Done","commit_id":"f3fb6f0ad5110aefe3b593f2a9faf7d19cb0ee73"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3471b034ea2f5ab4fafedfff674ccd8e2946c686","unresolved":true,"context_lines":[{"line_number":4478,"context_line":"                \u0027**WARNING** If the server is in an error state this is \u0027"},{"line_number":4479,"context_line":"                \u0027almost never the correct command to run, \u0027"},{"line_number":4480,"context_line":"                \u0027prefer hard reboot where possible. \u0027"},{"line_number":4481,"context_line":"                \u0027Resetting the state is intented to workaround \u0027"},{"line_number":4482,"context_line":"                \u0027servers stuck in an intermediate state, such as deleting. \u0027"},{"line_number":4483,"context_line":"                \u0027If the server is in error due to a move operation, setting \u0027"},{"line_number":4484,"context_line":"                \u0027the state can result in instances that are no longer \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"561a44cc_5b63575a","line":4481,"range":{"start_line":4481,"start_character":40,"end_line":4481,"end_character":48},"updated":"2025-04-11 10:48:58.000000000","message":"```suggestion\n                \u0027Resetting the state is intended to workaround \u0027\n```","commit_id":"d835b29edd532a5886d8ff221d10157786b62dee"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6eeee7ee5aa5f72723ef03c293f2b19af6d187cb","unresolved":false,"context_lines":[{"line_number":4478,"context_line":"                \u0027**WARNING** If the server is in an error state this is \u0027"},{"line_number":4479,"context_line":"                \u0027almost never the correct command to run, \u0027"},{"line_number":4480,"context_line":"                \u0027prefer hard reboot where possible. \u0027"},{"line_number":4481,"context_line":"                \u0027Resetting the state is intented to workaround \u0027"},{"line_number":4482,"context_line":"                \u0027servers stuck in an intermediate state, such as deleting. \u0027"},{"line_number":4483,"context_line":"                \u0027If the server is in error due to a move operation, setting \u0027"},{"line_number":4484,"context_line":"                \u0027the state can result in instances that are no longer \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"967ed528_b646d4d5","line":4481,"range":{"start_line":4481,"start_character":40,"end_line":4481,"end_character":48},"in_reply_to":"561a44cc_5b63575a","updated":"2025-04-11 10:53:43.000000000","message":"Done","commit_id":"d835b29edd532a5886d8ff221d10157786b62dee"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3471b034ea2f5ab4fafedfff674ccd8e2946c686","unresolved":true,"context_lines":[{"line_number":4482,"context_line":"                \u0027servers stuck in an intermediate state, such as deleting. \u0027"},{"line_number":4483,"context_line":"                \u0027If the server is in error due to a move operation, setting \u0027"},{"line_number":4484,"context_line":"                \u0027the state can result in instances that are no longer \u0027"},{"line_number":4485,"context_line":"                \u0027usable and should be used with caution. (admin only)\u0027"},{"line_number":4486,"context_line":"            ),"},{"line_number":4487,"context_line":"        )"},{"line_number":4488,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"5c23dc40_76c43d1c","line":4485,"updated":"2025-04-11 10:48:58.000000000","message":"```suggestion\n                \u0027usable. (admin only)\u0027\n```","commit_id":"d835b29edd532a5886d8ff221d10157786b62dee"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6eeee7ee5aa5f72723ef03c293f2b19af6d187cb","unresolved":false,"context_lines":[{"line_number":4482,"context_line":"                \u0027servers stuck in an intermediate state, such as deleting. \u0027"},{"line_number":4483,"context_line":"                \u0027If the server is in error due to a move operation, setting \u0027"},{"line_number":4484,"context_line":"                \u0027the state can result in instances that are no longer \u0027"},{"line_number":4485,"context_line":"                \u0027usable and should be used with caution. (admin only)\u0027"},{"line_number":4486,"context_line":"            ),"},{"line_number":4487,"context_line":"        )"},{"line_number":4488,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"6fb189bf_ba56ee2e","line":4485,"in_reply_to":"5c23dc40_76c43d1c","updated":"2025-04-11 10:53:43.000000000","message":"Done","commit_id":"d835b29edd532a5886d8ff221d10157786b62dee"}]}
