)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"bfd579d4c52ee088bef3d9e1e12cf863f3237ebb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c92193c3_5aa4f2bd","updated":"2025-06-30 09:44:23.000000000","message":"recheck - failed test metal3job integration failed due to connection unexpectedly closed while collecting logs","commit_id":"2be79d80961e25591fc9a46ca02f113af198cf3a"}],"ironic/api/controllers/v1/inspection_rule.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":"5a3efac47fdef786736a48519c40a3577da735d9","unresolved":true,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        if action.get(\u0027loop\u0027, []):"},{"line_number":52,"context_line":"            result[\u0027loop\u0027] \u003d action[\u0027loop\u0027]"},{"line_number":53,"context_line":"            result[\u0027multiple\u0027] \u003d action.get(\u0027multiple\u0027, \u0027any\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        converted_actions.append(result)"},{"line_number":56,"context_line":"    return converted_actions"}],"source_content_type":"text/x-python","patch_set":1,"id":"d7bc34b5_66af9f15","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":62},"updated":"2025-06-25 15:14:46.000000000","message":"`multiple` is an invalid field for actions.","commit_id":"8d2999c618662ba913d7f18b08056b28ec74cd44"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"47992412344972e9475307ece5fb15fdaf5223f7","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        if action.get(\u0027loop\u0027, []):"},{"line_number":52,"context_line":"            result[\u0027loop\u0027] \u003d action[\u0027loop\u0027]"},{"line_number":53,"context_line":"            result[\u0027multiple\u0027] \u003d action.get(\u0027multiple\u0027, \u0027any\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        converted_actions.append(result)"},{"line_number":56,"context_line":"    return converted_actions"}],"source_content_type":"text/x-python","patch_set":1,"id":"6ea277fb_7121f52d","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":62},"in_reply_to":"8f27d10c_678f3ab2","updated":"2025-06-25 15:32:22.000000000","message":"Acknowledged","commit_id":"8d2999c618662ba913d7f18b08056b28ec74cd44"},{"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":"38468a1e2bdce302a3856ab2e7df93dc26e391e3","unresolved":true,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        if action.get(\u0027loop\u0027, []):"},{"line_number":52,"context_line":"            result[\u0027loop\u0027] \u003d action[\u0027loop\u0027]"},{"line_number":53,"context_line":"            result[\u0027multiple\u0027] \u003d action.get(\u0027multiple\u0027, \u0027any\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        converted_actions.append(result)"},{"line_number":56,"context_line":"    return converted_actions"}],"source_content_type":"text/x-python","patch_set":1,"id":"8f27d10c_678f3ab2","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":62},"in_reply_to":"d7bc34b5_66af9f15","updated":"2025-06-25 15:17:09.000000000","message":"Although it seems it should, this migration does not recognize that field for an action.","commit_id":"8d2999c618662ba913d7f18b08056b28ec74cd44"},{"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":"aa33501ef86b831595895d158b663b2a0a2964b5","unresolved":true,"context_lines":[{"line_number":40,"context_line":"DEFAULT_RETURN_FIELDS \u003d [\u0027uuid\u0027, \u0027priority\u0027, \u0027phase\u0027, \u0027description\u0027]"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"def convert_actions(rpc_actions):"},{"line_number":44,"context_line":"    converted_actions \u003d []"},{"line_number":45,"context_line":"    for action in rpc_actions:"},{"line_number":46,"context_line":"        result \u003d {"}],"source_content_type":"text/x-python","patch_set":2,"id":"29a783f0_7c60d9a8","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":33},"updated":"2025-06-25 15:37:21.000000000","message":"nit: There\u0027s room to DRY `convert_actions` and `convert_conditions` into one single function instead.","commit_id":"697c4a4b3a6f54eb591acf601a52393c4cfeaff9"}]}
