)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2e3e48295b68e7010b0711f5bf58d84b9fe9d032","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6bae7126_01183d9b","updated":"2025-04-16 10:56:39.000000000","message":"bad artom, we should have caught this on the master review but we forgot to file an upstream bug tracker for this.\n\nfor the full context of the backprot the downstream bug was orgianlly filed in bugzilla https://bugzilla.redhat.com/show_bug.cgi?id\u003d2255277 and later moved to jira https://issues.redhat.com/browse/OSPRH-2764\n\nthe imporant context for stable reviewers is that with the old ordering, a slow cinder backend coudl delay activating the port bidning for many minutes causing extened network downtime.\n\nso we reordered the calls to call neutron first and defer calling cidner after the networking setup is fully commpleted.","commit_id":"1904b07833e90dce98c4a5c2f5c850b37a865f43"},{"author":{"_account_id":20733,"name":"Rajesh Tailor","email":"ratailor@redhat.com","username":"rajesht"},"change_message_id":"022156cf14e8c87fa75fb6ea015875de1be47ef5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aea08878_f1cff116","updated":"2025-04-16 12:34:20.000000000","message":"just a minor nit. otherwise LGTM.","commit_id":"1904b07833e90dce98c4a5c2f5c850b37a865f43"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"11772b0cfa3f955c77de24e50aecc778b5485cdc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a22d4404_4aae0da0","updated":"2024-09-23 12:49:14.000000000","message":"no conflicts, clean cherry-pick, lgtm","commit_id":"1904b07833e90dce98c4a5c2f5c850b37a865f43"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"12e20108a7a9dfb91a59e22bcb7169db6e4f41d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f103e9a6_31633d1a","in_reply_to":"6bae7126_01183d9b","updated":"2025-04-16 13:16:15.000000000","message":"Thanks Sean for the review \u0026 context, based on that this looks good to go to me. (clean cherry pick from stable/2024.1, which was master at merge time).","commit_id":"1904b07833e90dce98c4a5c2f5c850b37a865f43"}],"nova/compute/manager.py":[{"author":{"_account_id":20733,"name":"Rajesh Tailor","email":"ratailor@redhat.com","username":"rajesht"},"change_message_id":"022156cf14e8c87fa75fb6ea015875de1be47ef5","unresolved":true,"context_lines":[{"line_number":9328,"context_line":"        # _post_live_migration_remove_source_vol_connections() to clean up"},{"line_number":9329,"context_line":"        # source volume connections and prevent potential data leaks. We"},{"line_number":9330,"context_line":"        # therefore activate the port bindings in a try block, and, regardless"},{"line_number":9331,"context_line":"        # of any expcetions during that process, clean up volume connections in"},{"line_number":9332,"context_line":"        # a finally block."},{"line_number":9333,"context_line":"        try:"},{"line_number":9334,"context_line":"            # NOTE(artom) At this point in time we have not bound the ports to"}],"source_content_type":"text/x-python","patch_set":1,"id":"82bd6546_ca1cd5f9","line":9331,"range":{"start_line":9331,"start_character":17,"end_line":9331,"end_character":27},"updated":"2025-04-16 12:34:20.000000000","message":"s/expcetion/exceptions/","commit_id":"1904b07833e90dce98c4a5c2f5c850b37a865f43"}]}