)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35929,"name":"Mahnoor Asghar","display_name":"Mahnoor Asghar","email":"masghar@redhat.com","username":"mahnoorasghar"},"change_message_id":"e64ba2800495c9fd52753fd6cf300e1b6c16a75f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7b0fb4fd_92c879bc","updated":"2024-10-25 12:08:04.000000000","message":"@Pavlov, perhaps adding the hashtag #ironic-week-prio will get you faster reviews.","commit_id":"80540bb52da3314d8371c9301616720731bd589d"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"996db31d933e1152dd6d286a10880e0474cf30af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"69b55548_d88bbcb1","updated":"2024-10-22 20:15:48.000000000","message":"@dtantsur, done","commit_id":"80540bb52da3314d8371c9301616720731bd589d"}],"ironic/drivers/modules/ipmitool.py":[{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"bade935a1e5bb8a407849aef3c94b9781a9d27d2","unresolved":true,"context_lines":[{"line_number":584,"context_line":"    :returns: True if the cmd_stderr contains a cipher suite error,"},{"line_number":585,"context_line":"        False otherwise."},{"line_number":586,"context_line":"    \"\"\""},{"line_number":587,"context_line":"    cs_errors \u003d ["},{"line_number":588,"context_line":"        \"Unsupported cipher suite ID\","},{"line_number":589,"context_line":"        \"Error in open session response message : no matching cipher suite\","},{"line_number":590,"context_line":"        \"Error in open session response message : invalid role\","}],"source_content_type":"text/x-python","patch_set":2,"id":"94e67ed5_27093313","line":587,"updated":"2024-10-22 20:21:50.000000000","message":"looking at this from the more general view, and knowing the absolute zoo of servers out there plus how strictly vendors adhere to standards :sarcasm:\n\nit may be beneficial to add some config option that would allow operators themselves expand the list of strings used here.\n\nsomething similar to CONF.ipmi.additional_retryable_ipmi_errors on L695 below, but specifically for errors that will trigger retry with another cipher suite version, not just retry with the same arguments.","commit_id":"80540bb52da3314d8371c9301616720731bd589d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"f8a530f76b897af86c2752c9f45b2d8bc7917865","unresolved":true,"context_lines":[{"line_number":584,"context_line":"    :returns: True if the cmd_stderr contains a cipher suite error,"},{"line_number":585,"context_line":"        False otherwise."},{"line_number":586,"context_line":"    \"\"\""},{"line_number":587,"context_line":"    cs_errors \u003d ["},{"line_number":588,"context_line":"        \"Unsupported cipher suite ID\","},{"line_number":589,"context_line":"        \"Error in open session response message : no matching cipher suite\","},{"line_number":590,"context_line":"        \"Error in open session response message : invalid role\","}],"source_content_type":"text/x-python","patch_set":2,"id":"a17e4eb5_f88266c1","line":587,"in_reply_to":"94e67ed5_27093313","updated":"2024-10-23 07:54:43.000000000","message":"Unfortunately, it\u0027s a good idea.","commit_id":"80540bb52da3314d8371c9301616720731bd589d"}]}
