)]}'
{"/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":"5a829a943ab700b1d65f83e169e98a8b5cae9852","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aa1cdd8d_adbc858f","updated":"2026-06-10 17:40:00.000000000","message":"This is an operator-facing change and needs a release note.","commit_id":"21d4f8511829f52785eb333692eac3b56bf0bc65"}],"ironic/conf/types.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":"523e4ea25371a4e5329f801c7438699dbabf6acf","unresolved":true,"context_lines":[{"line_number":78,"context_line":"            raise ValueError("},{"line_number":79,"context_line":"                \u0027Value must be a valid kernel parameter string. Current value \u0027"},{"line_number":80,"context_line":"                \u0027is invalid. See \u0027"},{"line_number":81,"context_line":"                \u0027ironic.common.kernel_parameters.KERNEL_PARAMETER_GRAMMAR \u0027"},{"line_number":82,"context_line":"                \u0027for more information.\u0027)"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        return value"}],"source_content_type":"text/x-python","patch_set":2,"id":"05cd6f1e_8ef7afd8","line":81,"updated":"2026-06-10 17:29:46.000000000","message":"This needs to reference something other than our own code. Perhaps enumerate the allowed grammar here or generically refer to Ironic documentation?","commit_id":"21d4f8511829f52785eb333692eac3b56bf0bc65"},{"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":"f6aac1a9959f623e02edbdd2152df57fa67422dc","unresolved":true,"context_lines":[{"line_number":78,"context_line":"            raise ValueError("},{"line_number":79,"context_line":"                \u0027Value must be a valid kernel parameter string. Current value \u0027"},{"line_number":80,"context_line":"                \u0027is invalid. See \u0027"},{"line_number":81,"context_line":"                \u0027ironic.common.kernel_parameters.KERNEL_PARAMETER_GRAMMAR \u0027"},{"line_number":82,"context_line":"                \u0027for more information.\u0027)"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        return value"}],"source_content_type":"text/x-python","patch_set":2,"id":"e402a365_2bd325d1","line":81,"in_reply_to":"05cd6f1e_8ef7afd8","updated":"2026-06-15 12:39:02.000000000","message":"Common practice is to include the valid list in the error message.","commit_id":"21d4f8511829f52785eb333692eac3b56bf0bc65"}]}