)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"870eac2821c8bca7a777545b831e8ac852dd9897","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1bb55ec3_2a0a470c","updated":"2026-04-07 17:30:14.000000000","message":"I can flip my vote on this if you all like and need something from a \"bug fix\" perspective ... but this feels like an opaquely-implemented state machine inside driver_internal_info ... can we just \n*actually* make it a state machine so it can be better documented and understood?\n\nThere\u0027s just a lot of complexity here without a lot of structure.","commit_id":"12729b676a10f513076c968e9e92dcc76633dfa9"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"0a0e541e321f0d9cc2777f456d35e90ca5b83573","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9a3fc981_3601a9c9","updated":"2026-04-02 14:54:51.000000000","message":"recheck - pep8 problem? locally is green","commit_id":"12729b676a10f513076c968e9e92dcc76633dfa9"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"13f413aa574803183fa3bde5603807634dca3b5c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d6964a26_eda5c712","in_reply_to":"1bb55ec3_2a0a470c","updated":"2026-04-09 01:45:14.000000000","message":"We do need something like a bug fix perspective, but I do agree we are going down a state machine, I\u0027ve also talked with Dmitry and he agree with you.\nSo I will think a bit on how to proceed on this approach","commit_id":"12729b676a10f513076c968e9e92dcc76633dfa9"}],"ironic/drivers/modules/redfish/firmware.py":[{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"1b71926252433f839f76f979e26e90609aff1fdd","unresolved":true,"context_lines":[{"line_number":729,"context_line":"                        # retrieve it, this is critical for PLDM-based NICs"},{"line_number":730,"context_line":"                        # which require OS to be fully booted before firmware"},{"line_number":731,"context_line":"                        # data is accessible"},{"line_number":732,"context_line":"                        if require_nic_data and net_adapters:"},{"line_number":733,"context_line":"                            nic_data_retrieved \u003d False"},{"line_number":734,"context_line":"                            for net_adp in net_adapters:"},{"line_number":735,"context_line":"                                if net_adp.controllers:"}],"source_content_type":"text/x-python","patch_set":3,"id":"94c167ae_fcbe9884","line":732,"range":{"start_line":732,"start_character":0,"end_line":732,"end_character":61},"updated":"2026-04-07 17:31:09.000000000","message":"nit: The nesting here, could it be simplified","commit_id":"12729b676a10f513076c968e9e92dcc76633dfa9"}]}