)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"552d5acf8d8bc06ca4ad11a3c51aa98ec47dcd1e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"24664ddf_84eaefc4","updated":"2024-11-21 11:55:47.000000000","message":"\u003e I\u0027m generally curious if we have coverage for this code path in integration tests; especially since a soft power off failure would likely be covered up by the fallback to a hard power off.\n\nI don\u0027t see any failures on the bifrost job, at least.","commit_id":"80c83d82f2e0754bbd26b166aeec7dd933d15145"},{"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":"634fdc25b11a6337fac7eb86729d1b424dc52af2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8fdcafdc_bf525de5","updated":"2024-11-20 18:07:33.000000000","message":"I\u0027m generally curious if we have coverage for this code path in integration tests; especially since a soft power off failure would likely be covered up by the fallback to a hard power off.","commit_id":"80c83d82f2e0754bbd26b166aeec7dd933d15145"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"2d1921095e25ff96c0afe748db8d350810f7c8e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"75237d72_921cff1a","updated":"2024-11-18 13:21:22.000000000","message":"metal3-integration and ironic-tempest-uefi-redfish-vmedia-4k seems unhappy, I will try to check the logs before adding the hashtag here","commit_id":"80c83d82f2e0754bbd26b166aeec7dd933d15145"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"968fac7a13006616969bf25241e0dc3219c95db1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"507b8be7_062d5128","updated":"2024-11-13 13:59:24.000000000","message":"recheck\n\nunstable unit tests","commit_id":"80c83d82f2e0754bbd26b166aeec7dd933d15145"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ae994d694495b4a24715184eaa3652502ae4698f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c71defbb_4a6ffbd4","updated":"2024-11-13 15:59:26.000000000","message":"recheck SSH timeout in the OVN job","commit_id":"80c83d82f2e0754bbd26b166aeec7dd933d15145"}],"ironic/drivers/modules/agent.py":[{"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":"634fdc25b11a6337fac7eb86729d1b424dc52af2","unresolved":true,"context_lines":[{"line_number":218,"context_line":"    @tenacity.retry(stop\u003dtenacity.stop_after_attempt(attempts),"},{"line_number":219,"context_line":"                    retry\u003d(tenacity.retry_if_result("},{"line_number":220,"context_line":"                        lambda state: state !\u003d states.POWER_OFF)"},{"line_number":221,"context_line":"                        | tenacity.retry_if_exception_type(Exception)),"},{"line_number":222,"context_line":"                    wait\u003dtenacity.wait_fixed(wait),"},{"line_number":223,"context_line":"                    reraise\u003dTrue)"},{"line_number":224,"context_line":"    def _wait_until_powered_off(task):"}],"source_content_type":"text/x-python","patch_set":1,"id":"deb31f8c_7b5c9834","line":221,"updated":"2024-11-20 18:07:33.000000000","message":"Maybe we should split out instead of being a lambda?","commit_id":"80c83d82f2e0754bbd26b166aeec7dd933d15145"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"552d5acf8d8bc06ca4ad11a3c51aa98ec47dcd1e","unresolved":true,"context_lines":[{"line_number":218,"context_line":"    @tenacity.retry(stop\u003dtenacity.stop_after_attempt(attempts),"},{"line_number":219,"context_line":"                    retry\u003d(tenacity.retry_if_result("},{"line_number":220,"context_line":"                        lambda state: state !\u003d states.POWER_OFF)"},{"line_number":221,"context_line":"                        | tenacity.retry_if_exception_type(Exception)),"},{"line_number":222,"context_line":"                    wait\u003dtenacity.wait_fixed(wait),"},{"line_number":223,"context_line":"                    reraise\u003dTrue)"},{"line_number":224,"context_line":"    def _wait_until_powered_off(task):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5cc7218b_414fe964","line":221,"in_reply_to":"deb31f8c_7b5c9834","updated":"2024-11-21 11:55:47.000000000","message":"Could be. My goal was just to move away the code as it was to reduce the complexity of the already complex function.","commit_id":"80c83d82f2e0754bbd26b166aeec7dd933d15145"}]}