)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"25d14b7c9c25440efbb50b52e5d4a6723c34f710","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"056f29bc_28844df5","updated":"2026-03-02 11:45:57.000000000","message":"Ah, interesting solution. Could you add a release note as described in https://docs.openstack.org/ironic/latest/contributor/faq.html#create-a-new-release-note please? This fix is important to highlight since it may result in a visible behavior change.","commit_id":"65e193596a5bb9ac082f838c6b42a0b7e4f2a4bd"},{"author":{"_account_id":38824,"name":"Léonard Suslian","display_name":"Léonard Suslian","email":"leonard@suslian.engineer","username":"synthe102"},"change_message_id":"a25a15b15bb4cccb4c697f3e9c678b63984c452d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1e723645_43e9266c","updated":"2026-03-03 17:41:58.000000000","message":"recheck","commit_id":"bb16b46c1ada67700968f43783b5140308999eb1"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"2221102ad44ab35caf743df9189098e2dd9c77bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d61f8808_4c02d17f","updated":"2026-03-06 08:41:28.000000000","message":"thanks, a comment inline","commit_id":"bb16b46c1ada67700968f43783b5140308999eb1"},{"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":"411c282d646449ce94c047985c7a6a4cb40894e5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b82e3ee2_3324cba6","updated":"2026-03-06 23:06:54.000000000","message":"recheck broken tox release today","commit_id":"01822fbe53191c3bfc4ad5cac467ed82c85eacef"},{"author":{"_account_id":38824,"name":"Léonard Suslian","display_name":"Léonard Suslian","email":"leonard@suslian.engineer","username":"synthe102"},"change_message_id":"992a8606bad0dce7addbab63143facbd11ab127e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fd86d49b_539c1d7e","in_reply_to":"b82e3ee2_3324cba6","updated":"2026-03-07 17:28:41.000000000","message":"Done","commit_id":"01822fbe53191c3bfc4ad5cac467ed82c85eacef"}],"ironic/drivers/modules/redfish/management.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9bafd661cbc335431d6e59cc34e8f7cd736c3d65","unresolved":false,"context_lines":[{"line_number":1244,"context_line":"        :param task: A task from TaskManager."},{"line_number":1245,"context_line":"        :raises: InvalidParameterValue if an invalid component, indicator"},{"line_number":1246,"context_line":"            or state is specified."},{"line_number":1247,"context_line":"        :raises: MissingParameterValue if a required parameter is missing"},{"line_number":1248,"context_line":"        :raises: RedfishError on driver-specific problems."},{"line_number":1249,"context_line":"        :returns: String representing the BMC reported Vendor or"},{"line_number":1250,"context_line":"                  Manufacturer, otherwise returns None."}],"source_content_type":"text/x-python","patch_set":8,"id":"05bf9626_6094661c","side":"PARENT","line":1247,"updated":"2026-03-09 13:26:31.000000000","message":"Please don\u0027t remove these: they can be raised by parse_driver_info that is called by get_system","commit_id":"6895838afbfc9951070bef84da870ce74d1001e6"}],"ironic/drivers/modules/redfish/utils.py":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"2221102ad44ab35caf743df9189098e2dd9c77bc","unresolved":true,"context_lines":[{"line_number":416,"context_line":"    \"\"\""},{"line_number":417,"context_line":"    try:"},{"line_number":418,"context_line":"        return _get_connection(node, lambda conn: conn.json.get(\u0027Vendor\u0027))"},{"line_number":419,"context_line":"    except Exception:"},{"line_number":420,"context_line":"        return None"},{"line_number":421,"context_line":""},{"line_number":422,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"0a1821a7_51a2cc8a","line":419,"updated":"2026-03-06 08:41:28.000000000","message":"this is not ideal, he bare except Exception silently discards all errors with no logging. Let\u0027s at least log the error before raising to help troubleshoooting.","commit_id":"bb16b46c1ada67700968f43783b5140308999eb1"},{"author":{"_account_id":38824,"name":"Léonard Suslian","display_name":"Léonard Suslian","email":"leonard@suslian.engineer","username":"synthe102"},"change_message_id":"5b2e2a1e1e6d81ab924980ec6e4eeeed20ce54bb","unresolved":false,"context_lines":[{"line_number":416,"context_line":"    \"\"\""},{"line_number":417,"context_line":"    try:"},{"line_number":418,"context_line":"        return _get_connection(node, lambda conn: conn.json.get(\u0027Vendor\u0027))"},{"line_number":419,"context_line":"    except Exception:"},{"line_number":420,"context_line":"        return None"},{"line_number":421,"context_line":""},{"line_number":422,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"a9bfa144_f65e9e5c","line":419,"in_reply_to":"0a1821a7_51a2cc8a","updated":"2026-03-06 16:59:23.000000000","message":"Thanks for the review. Added a Debug level log since its a best-effort lookup.","commit_id":"bb16b46c1ada67700968f43783b5140308999eb1"}]}
