)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31975,"name":"Olaf Seibert","email":"o.seibert@syseleven.de","username":"oseiberts11"},"change_message_id":"6e5d4d1178f39ea60250c53b28dd5af9fb30bc95","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4b9b3392_bd2a7e0f","updated":"2023-12-19 13:08:56.000000000","message":"In the patch given in bug https://bugs.launchpad.net/nova/+bug/1946752 the POSTCOPY flag is disabled in two locations. The fix here only does it in one of the two. It misses the later variable named `is_post_copy_enabled`.\n\nAre we sure that that is sufficient?\n\nAlso, using XOR to disable a flag isn\u0027t very clear. To be certain the flag isn\u0027t turned on here, you have to check that self._is_post_copy_enabled(migration_flags) checks that the exact same flag is already on. I would prefer migration_flags \u0026\u003d ~libvirt.VIR_MIGRATE_POSTCOPY.","commit_id":"989ee448906f4ca623f6b14ef8c7af8c8fa10e2f"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"faf54709ba5245667bbd8018ffb5b5f567e08314","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"46473bbe_87c9aa50","updated":"2023-12-19 10:40:40.000000000","message":"Yepp, this looks valid to backport, plus this is a clean cherry pick from master branch, where the patch has merged yesterday. LGTM.","commit_id":"989ee448906f4ca623f6b14ef8c7af8c8fa10e2f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2486e16d4586c5d146f7a3f2dd3d9d4c6ad5d078","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"08ad028a_b83e6111","updated":"2023-12-19 11:13:28.000000000","message":"yep looks good to me too. thanks for the backport","commit_id":"989ee448906f4ca623f6b14ef8c7af8c8fa10e2f"}]}