)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6799,"name":"Nicholas Kuechler","email":"nkuechler@gmail.com","username":"nicholaskuechler"},"change_message_id":"6154dc9c45fe7f17e884705c10ffa0288c1ff0c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2527f214_93558c4d","updated":"2026-05-07 14:52:09.000000000","message":"Updated","commit_id":"e1befa66b74366c93360751ac144cf0bed9f7644"},{"author":{"_account_id":6799,"name":"Nicholas Kuechler","email":"nkuechler@gmail.com","username":"nicholaskuechler"},"change_message_id":"d5d35db5b0a93cd913b0f9eac2e60dc8abb8b391","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"86bf5df9_d2a1d8e0","updated":"2026-05-07 16:39:02.000000000","message":"recheck ironic-tempest-ovn-uefi-ipxe-ipv6","commit_id":"aa3f4349d6b75ccbdfdaa0c9f7cd16d643604d3a"},{"author":{"_account_id":6799,"name":"Nicholas Kuechler","email":"nkuechler@gmail.com","username":"nicholaskuechler"},"change_message_id":"f7ee3cb77d56f7e0f4879e99268f7c46cbd53d00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"12347b48_748f1063","updated":"2026-05-07 19:32:55.000000000","message":"recheck","commit_id":"e83bb1f15403037e994f47aab2cdc1735667ba74"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"238041486befe91de6494eb638f64b9d14369528","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4c967d07_eb74c452","updated":"2026-05-07 21:58:36.000000000","message":"recheck stop being flakey","commit_id":"e83bb1f15403037e994f47aab2cdc1735667ba74"}],"ironic/drivers/modules/redfish/management.py":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"31188f3e51d02cf6805dade6e3edf3531c2852e0","unresolved":true,"context_lines":[{"line_number":910,"context_line":"        # Get system identity from driver info"},{"line_number":911,"context_line":"        driver_info \u003d redfish_utils.parse_driver_info(node)"},{"line_number":912,"context_line":"        system_id \u003d driver_info[\u0027system_id\u0027]"},{"line_number":913,"context_line":"        system_identity \u003d system_id.split(\u0027/\u0027)[-1] if system_id else \u0027Unknown\u0027"},{"line_number":914,"context_line":""},{"line_number":915,"context_line":"        try:"},{"line_number":916,"context_line":"            drives \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"8a0046a9_30abb9a6","line":913,"updated":"2026-05-06 18:12:46.000000000","message":"I think this whole block can go away and we don\u0027t need to parse any of it. redfish_utils.parse_driver_info() is used to create the Sushy connection. Then the \u0027system_id\u0027 (if supplied and its not required to be) is used in the get_system() helper. The value of the \u0027system_id\u0027 field is supplied by the user as \u0027redfish_system_id\u0027 or None per the docs if its supplied its URL path to the ComputerSystem object you want to use. If its None then it defaults to the first ComputerSystem object. Well the ComputerSystem object is the \u0027system\u0027 parameter in this function. The value of all this then means its\n\n```suggestion\n        system_identity \u003d system.path.split(\u0027/\u0027)[-1]\n```\n\nAnd the driver_info \u003d redfish_utils.parse_driver_info(node) can just be deleted.\n\n```\n\u003e\u003e\u003e system.path\n\u0027/redfish/v1/Systems/System.Embedded.1\u0027\n\u003e\u003e\u003e system.path.split(\u0027/\u0027)[-1]\n\u0027System.Embedded.1\u0027\n```","commit_id":"390bb0af9bd5b721f1f4712d302852f378dd7476"},{"author":{"_account_id":6799,"name":"Nicholas Kuechler","email":"nkuechler@gmail.com","username":"nicholaskuechler"},"change_message_id":"6154dc9c45fe7f17e884705c10ffa0288c1ff0c2","unresolved":false,"context_lines":[{"line_number":910,"context_line":"        # Get system identity from driver info"},{"line_number":911,"context_line":"        driver_info \u003d redfish_utils.parse_driver_info(node)"},{"line_number":912,"context_line":"        system_id \u003d driver_info[\u0027system_id\u0027]"},{"line_number":913,"context_line":"        system_identity \u003d system_id.split(\u0027/\u0027)[-1] if system_id else \u0027Unknown\u0027"},{"line_number":914,"context_line":""},{"line_number":915,"context_line":"        try:"},{"line_number":916,"context_line":"            drives \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"daf6307f_b4c19912","line":913,"in_reply_to":"8a0046a9_30abb9a6","updated":"2026-05-07 14:52:09.000000000","message":"Acknowledged","commit_id":"390bb0af9bd5b721f1f4712d302852f378dd7476"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"49cbd436bdec85001f8ab8bd708a8157b00e9959","unresolved":true,"context_lines":[{"line_number":908,"context_line":"        storage_sensors \u003d {\u0027Drive\u0027: {}}"},{"line_number":909,"context_line":""},{"line_number":910,"context_line":"        # Get system identity from driver info"},{"line_number":911,"context_line":"        driver_info \u003d redfish_utils.parse_driver_info(node)"},{"line_number":912,"context_line":"        system_identity \u003d system.path.split(\u0027/\u0027)[-1]"},{"line_number":913,"context_line":""},{"line_number":914,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"00ea2b1d_67858fd6","line":911,"updated":"2026-05-07 16:37:45.000000000","message":"This line is no longer necessary and why the CI failed. delete it.","commit_id":"aa3f4349d6b75ccbdfdaa0c9f7cd16d643604d3a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"16f5dd5cb11771870217020afff53affa3c31798","unresolved":false,"context_lines":[{"line_number":908,"context_line":"        storage_sensors \u003d {\u0027Drive\u0027: {}}"},{"line_number":909,"context_line":""},{"line_number":910,"context_line":"        # Get system identity from driver info"},{"line_number":911,"context_line":"        driver_info \u003d redfish_utils.parse_driver_info(node)"},{"line_number":912,"context_line":"        system_identity \u003d system.path.split(\u0027/\u0027)[-1]"},{"line_number":913,"context_line":""},{"line_number":914,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"64da6f25_10580616","line":911,"in_reply_to":"00ea2b1d_67858fd6","updated":"2026-05-07 17:16:52.000000000","message":"Done","commit_id":"aa3f4349d6b75ccbdfdaa0c9f7cd16d643604d3a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"49cbd436bdec85001f8ab8bd708a8157b00e9959","unresolved":true,"context_lines":[{"line_number":954,"context_line":"        simple_storage_sensors \u003d {\u0027Drive\u0027: {}}"},{"line_number":955,"context_line":""},{"line_number":956,"context_line":"        # Get system identity from driver info"},{"line_number":957,"context_line":"        driver_info \u003d redfish_utils.parse_driver_info(node)"},{"line_number":958,"context_line":"        system_identity \u003d system.path.split(\u0027/\u0027)[-1]"},{"line_number":959,"context_line":""},{"line_number":960,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"d7e13db8_ace69e9e","line":957,"updated":"2026-05-07 16:37:45.000000000","message":"This line is no longer necessary and why the CI failed. delete it.","commit_id":"aa3f4349d6b75ccbdfdaa0c9f7cd16d643604d3a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"16f5dd5cb11771870217020afff53affa3c31798","unresolved":false,"context_lines":[{"line_number":954,"context_line":"        simple_storage_sensors \u003d {\u0027Drive\u0027: {}}"},{"line_number":955,"context_line":""},{"line_number":956,"context_line":"        # Get system identity from driver info"},{"line_number":957,"context_line":"        driver_info \u003d redfish_utils.parse_driver_info(node)"},{"line_number":958,"context_line":"        system_identity \u003d system.path.split(\u0027/\u0027)[-1]"},{"line_number":959,"context_line":""},{"line_number":960,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1512964d_31eb2a40","line":957,"in_reply_to":"d7e13db8_ace69e9e","updated":"2026-05-07 17:16:52.000000000","message":"Done","commit_id":"aa3f4349d6b75ccbdfdaa0c9f7cd16d643604d3a"}]}
