)]}'
{"/PATCHSET_LEVEL":[{"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":"33c3567a5b8be9a1149a4a65ea5e470d926ce012","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"db310c15_25b895be","updated":"2024-10-08 22:00:43.000000000","message":"I couldn\u0027t generate this bug locally, so, I might end up abandoning this later.","commit_id":"858926a27aab83e631d17738f4be7262c1a109ea"},{"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":"ff64c88214bb6947263bd562b5310f37c163a669","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"634f6ca7_e768c18f","updated":"2024-10-22 12:01:19.000000000","message":"Update:\n\nErrmmm, no matter the shape of ``args.uuid_or_name``, validation does not stop when a value is UUID-like, but not a valid UUID, so, we\u0027ll always check if it\u0027s instead a logical name. As for the related bug, maybe adding the extra exception catch may be all I need :facepalm:","commit_id":"b58cafc06f182ddb0c391bf542f848acf7b8e7cc"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"60038348f52fb79179ae17992cea3f7546696ec4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"720e5a42_63413ae6","updated":"2024-10-23 07:57:15.000000000","message":"Changing HTTP code from 5xx to a proper 4xx is normally considered backportable and does not require microversions.","commit_id":"c09525dd81b75884f6e2c2a19e3b04416e3ff08e"},{"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":"633ddac9cac33de99e9fc12f76e94bbda90b2c8b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"675130ed_5492eac1","updated":"2024-10-22 17:06:56.000000000","message":"This might make sense to backport? I\u0027m unsure given it\u0027s mildly api facing...","commit_id":"c09525dd81b75884f6e2c2a19e3b04416e3ff08e"},{"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":"448531c24dc3ae177e89a3cafb3025b57ed51d66","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"964d3fc3_e086db12","updated":"2024-10-28 10:29:08.000000000","message":"recheck - test_iso_9660 exited without returning a status.","commit_id":"c09525dd81b75884f6e2c2a19e3b04416e3ff08e"},{"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":"9cacb53328a92798bbc02211c245c33489682a31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d095411a_1ac40cdd","updated":"2024-10-23 19:41:37.000000000","message":"recheck .. ironic-tempest-bios-redfish-pxe POST_FAILURE (no logs)","commit_id":"c09525dd81b75884f6e2c2a19e3b04416e3ff08e"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"0ced9947616f25a93d29e3933340a87e0b3d6a5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"abb188e2_b85fd245","updated":"2024-10-24 13:28:54.000000000","message":"recheck different failure this time","commit_id":"c09525dd81b75884f6e2c2a19e3b04416e3ff08e"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"00aac0db5c2ac4292d1615c26d7fd817aff59e75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dd12f107_3d0f116d","updated":"2024-10-23 20:49:27.000000000","message":"recheck post failures are often transient","commit_id":"c09525dd81b75884f6e2c2a19e3b04416e3ff08e"}],"ironic/api/controllers/v1/node.py":[{"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":"69d237e30797ed331c3d87546dc69355f1cd9bbe","unresolved":true,"context_lines":[{"line_number":2353,"context_line":"        except exception.InvalidParameterValue as e:"},{"line_number":2354,"context_line":"            pecan.abort(http_client.BAD_REQUEST, e.args[0])"},{"line_number":2355,"context_line":"        except exception.InvalidUuidOrName as e:"},{"line_number":2356,"context_line":"            pecan.abort(http_client.BAD_REQUEST, e.args[0])"},{"line_number":2357,"context_line":"        if not remainder:"},{"line_number":2358,"context_line":"            return"},{"line_number":2359,"context_line":"        if ((remainder[0] \u003d\u003d \u0027portgroups\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"ce03ee0f_3bcbba78","line":2356,"updated":"2024-10-09 14:45:48.000000000","message":"So from looking at the code, this looks like an error that could happen. I\u0027m also suspicious this similar error exists in other things that use the *_ident pattern (where we take a UUID or Name) e.g. portgroup or runbook.","commit_id":"858926a27aab83e631d17738f4be7262c1a109ea"}]}
