)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"e68e1df432e5cdf6fa57c42224698e5da6cf37c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1cb96ece_b4ef7c31","updated":"2026-08-26 13:24:52.000000000","message":"-1 for now, just to get some feedback, the logic looks good in general.","commit_id":"423cf7c92c340a0c3fa7543a1a081340f4d6aade"}],"ironic/drivers/modules/redfish/firmware.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"e68e1df432e5cdf6fa57c42224698e5da6cf37c8","unresolved":true,"context_lines":[{"line_number":908,"context_line":"            reboot_requested \u003d node.driver_internal_info.get("},{"line_number":909,"context_line":"                FIRMWARE_REBOOT_REQUESTED, False)"},{"line_number":910,"context_line":""},{"line_number":911,"context_line":"            vendor \u003d node.properties.get(\u0027vendor\u0027, \u0027\u0027)"},{"line_number":912,"context_line":"            is_dell \u003d vendor and \u0027Dell\u0027 in vendor.split()"},{"line_number":913,"context_line":""},{"line_number":914,"context_line":"            if is_dell and reboot_requested:"}],"source_content_type":"text/x-python","patch_set":3,"id":"0c5a4cd8_0d5207b9","line":911,"updated":"2026-08-26 13:24:52.000000000","message":"To me this could move this to a function in firmware_utils.py, wdyt?","commit_id":"423cf7c92c340a0c3fa7543a1a081340f4d6aade"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"e68e1df432e5cdf6fa57c42224698e5da6cf37c8","unresolved":true,"context_lines":[{"line_number":1317,"context_line":"            return drac_fw.check_scheduled_idrac_job(task, current_update)"},{"line_number":1318,"context_line":"        return None"},{"line_number":1319,"context_line":""},{"line_number":1320,"context_line":"    def _handle_post_reboot_verify(self, task, node, current_update,"},{"line_number":1321,"context_line":"                                   settings):"},{"line_number":1322,"context_line":"        \"\"\"Poll Dell LC job to verify firmware was applied after reboot."},{"line_number":1323,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"712d5cfd_2e189f79","line":1320,"updated":"2026-08-26 13:24:52.000000000","message":"This one I also think it could move to the firmware_utils.py, at least from L1320 till L1348 ? and the rest of the code we could move to logic on L1686?","commit_id":"423cf7c92c340a0c3fa7543a1a081340f4d6aade"}]}