)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"63793a6b849535f8fc296a47c7b2472c95c40844","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2c81cd22_b824de9f","updated":"2025-11-17 11:12:00.000000000","message":"I don\u0027t believe this should be necessary. The `retriable_status_codes` parameter passed through to keystoneauth allows us to do this at the `Session` level. Is there any reason you couldn\u0027t just expand the set of status codes [used for ironic](https://opendev.org/openstack/openstacksdk/src/commit/eff7eb2d1710d5ca7de184b8e26eb7e8a2b21b8a/openstack/baremetal/v1/_common.py#L17-L23)?\n\nIf that\u0027s not a viable solution, could you explain why, and could you also explain why this should be an ironic-specific solution rather than a more generic feature of the proxy/resource layer in SDK or (better?) of keystoneauth?","commit_id":"69ad0a46c2e3b9d0af7e167503cb084b4ecd6e9c"},{"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":"390b1326e482ce3f6ed71ef8cb488b7b3f894e22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4f6e09d2_d5047b63","in_reply_to":"2c81cd22_b824de9f","updated":"2025-11-17 11:27:48.000000000","message":"Thanks a lot for the prompt review :D. \n\nThis is mostly work-in-progress (not tested yet). The idea for these additional retry coverage is that they do not have HTTP status codes since no HTTP response has occurred yet.\n\nAlso, this closes the gap left when the Nova driver moved from ironnicclient to the SDK.\n\nRelated Bugs: #2131095 and #2131094.","commit_id":"69ad0a46c2e3b9d0af7e167503cb084b4ecd6e9c"}]}