)]}'
{"/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":"60c82b0f2fcb0fe11debb44ebeeca40789e8bb24","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a8665770_cf610c65","updated":"2025-02-24 16:17:35.000000000","message":"This should be backported IMO. That may require it to have a bug, I\u0027m not certain about nova policy.\n\nWe likely need a release note, too.","commit_id":"092752cac3f1821b2a72a2d255a4538b78ed543d"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"05ebf637384abf7f94e24ec8c18654fd78008823","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a2b42d9b_c9dfc394","in_reply_to":"a8665770_cf610c65","updated":"2025-02-25 16:26:46.000000000","message":"Done","commit_id":"092752cac3f1821b2a72a2d255a4538b78ed543d"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"05ebf637384abf7f94e24ec8c18654fd78008823","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fd1a7138_46d4dd96","updated":"2025-02-25 16:26:46.000000000","message":"Bug created and referenced. Release Note created.","commit_id":"4cb47b8c98559a49b7c684f83022ba60f1a7bfa5"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"7aba37ed0caa3ed5dde7fc6e466d741fd42c421a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b4b0d169_9c13cafe","updated":"2025-04-01 13:24:51.000000000","message":"Simple error reporting fix","commit_id":"37888e875f0cef8350991b6a9cb81da9dacc1bac"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"cc81eb291c670f83c4055c60530c40787dc5f7ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"756c8e27_51537ca5","updated":"2025-03-18 03:10:03.000000000","message":"recheck test timed out unrelated to ironic","commit_id":"37888e875f0cef8350991b6a9cb81da9dacc1bac"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"aad704c6e834773f1e7cba0b6c2dda1c3264e787","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8cf415a4_4bd8cc6b","updated":"2025-04-01 21:05:44.000000000","message":"recheck unrelated timeout in test","commit_id":"37888e875f0cef8350991b6a9cb81da9dacc1bac"}],"nova/virt/ironic/driver.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"049cc43ba3b1999bb49cfd7de6f8edb4997a8694","unresolved":true,"context_lines":[{"line_number":1226,"context_line":"                % {\u0027id\u0027: node.id,"},{"line_number":1227,"context_line":"                   \u0027deploy\u0027: deploy_msg,"},{"line_number":1228,"context_line":"                   \u0027power\u0027: power_msg,"},{"line_number":1229,"context_line":"                   \u0027storage\u0027: storage_msg})"},{"line_number":1230,"context_line":""},{"line_number":1231,"context_line":"        # Config drive"},{"line_number":1232,"context_line":"        configdrive_value \u003d None"}],"source_content_type":"text/x-python","patch_set":2,"id":"b3e4fb69_6bf389d4","line":1229,"updated":"2025-03-03 16:03:30.000000000","message":"so we chatted about this on irc a bit\n\nbefore the change to the sdk it used to pring something like this\n\n\"(deploy: power: some message storage: )\"\n\nnow its priting something like this\n\n\"(deploy: \u003copenstack.response 0xDEADBEEF\u003e power: \u003copenstack.response 0xBEEFDEAD\u003e storage: \u003copenstack.response 0xFFFFFFFF)\"\n\nand with this change its more like this\n\n\"(deploy: No Error power: some message storage: No Error)\"\n\nThis is appprently only logged today and not returned to the end user\n\nas such its ok to encode the underlying excpetion in the log.\n\nwe talked about perhaps only pringing the relevent validation error that happend instead of all 3 but that might break anyoen who is currently processing there logs.\n\n\nmy only real ask is to add some unit tests to just assert that the expected excption message is generated.\n\nthat will catch and future regressions.","commit_id":"4cb47b8c98559a49b7c684f83022ba60f1a7bfa5"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"4f4039974368cdf898da91d5bd1a3a2460e20ac8","unresolved":false,"context_lines":[{"line_number":1226,"context_line":"                % {\u0027id\u0027: node.id,"},{"line_number":1227,"context_line":"                   \u0027deploy\u0027: deploy_msg,"},{"line_number":1228,"context_line":"                   \u0027power\u0027: power_msg,"},{"line_number":1229,"context_line":"                   \u0027storage\u0027: storage_msg})"},{"line_number":1230,"context_line":""},{"line_number":1231,"context_line":"        # Config drive"},{"line_number":1232,"context_line":"        configdrive_value \u003d None"}],"source_content_type":"text/x-python","patch_set":2,"id":"47ac62a3_643902f9","line":1229,"in_reply_to":"b3e4fb69_6bf389d4","updated":"2025-03-17 22:47:18.000000000","message":"Done","commit_id":"4cb47b8c98559a49b7c684f83022ba60f1a7bfa5"}]}