)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b5db537b022bc038a482327f38c1ad5d2f839d92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"779adb71_767ccf1c","updated":"2026-07-24 14:48:11.000000000","message":"Change in itself LGTM, but I\u0027d rather not add a flag that does nothing.","commit_id":"2feb67645b15200bc754f8b9d19ca3e2a3416a81"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"e83e92eb09283ffbe21696372c5c8490438f9fb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ba5d84fb_c224f97d","updated":"2026-07-23 13:16:08.000000000","message":"Unrelated failure: Server 5b56ceda-3732-402e-92d2-43893cda396d failed to reach ACTIVE status within 2000 seconds\nrecheck","commit_id":"2feb67645b15200bc754f8b9d19ca3e2a3416a81"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"9f4286c072512246679360bbde9d1fc8a3142ff8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8e0fec86_3488dfb7","updated":"2026-07-23 21:29:04.000000000","message":"recheck","commit_id":"2feb67645b15200bc754f8b9d19ca3e2a3416a81"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"3f949cb0081f709a259537fb894326eba2748c21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"464e16c3_7ef322d7","in_reply_to":"779adb71_767ccf1c","updated":"2026-07-30 10:03:36.000000000","message":"Makes sense, pushed complete chain now.","commit_id":"2feb67645b15200bc754f8b9d19ca3e2a3416a81"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"3f949cb0081f709a259537fb894326eba2748c21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"50ea8391_1ad8e9dc","updated":"2026-07-30 10:03:36.000000000","message":"Thank you Dmitry and Iury, pushed full set of patches.","commit_id":"4c91b9da66e90dc4152dd26c5e2cfb491f37bb7b"}],"ironic/tests/unit/drivers/modules/redfish/test_firmware.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"cdf003d4d5783340e0918158544ac5726a885209","unresolved":true,"context_lines":[{"line_number":2992,"context_line":"    @mock.patch.object(redfish_fw.RedfishFirmware, \u0027_execute_firmware_update\u0027,"},{"line_number":2993,"context_line":"                       autospec\u003dTrue)"},{"line_number":2994,"context_line":"    @mock.patch.object(redfish_utils, \u0027get_update_service\u0027, autospec\u003dTrue)"},{"line_number":2995,"context_line":"    def test_update_allow_grouping_reboots_true(self,"},{"line_number":2996,"context_line":"                                                mock_get_update_service,"},{"line_number":2997,"context_line":"                                                mock_execute_fw_update,"},{"line_number":2998,"context_line":"                                                mock_set_async_flags):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ee617fb3_f040f172","line":2995,"updated":"2026-07-24 01:13:13.000000000","message":"I\u0027m ok with this, but in the next part where you will add the logic to handle the batch procress, it will be good to have a test covering a case where you update multiple components that aren\u0027t bmc \u003d)","commit_id":"2feb67645b15200bc754f8b9d19ca3e2a3416a81"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"3f949cb0081f709a259537fb894326eba2748c21","unresolved":false,"context_lines":[{"line_number":2992,"context_line":"    @mock.patch.object(redfish_fw.RedfishFirmware, \u0027_execute_firmware_update\u0027,"},{"line_number":2993,"context_line":"                       autospec\u003dTrue)"},{"line_number":2994,"context_line":"    @mock.patch.object(redfish_utils, \u0027get_update_service\u0027, autospec\u003dTrue)"},{"line_number":2995,"context_line":"    def test_update_allow_grouping_reboots_true(self,"},{"line_number":2996,"context_line":"                                                mock_get_update_service,"},{"line_number":2997,"context_line":"                                                mock_execute_fw_update,"},{"line_number":2998,"context_line":"                                                mock_set_async_flags):"}],"source_content_type":"text/x-python","patch_set":1,"id":"6f8feb2b_485aa954","line":2995,"in_reply_to":"ee617fb3_f040f172","updated":"2026-07-30 10:03:36.000000000","message":"Thank you Iury. 999231 should cover this.","commit_id":"2feb67645b15200bc754f8b9d19ca3e2a3416a81"}]}
