)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28329,"name":"yangjianfeng","display_name":"JeffYang","email":"yjf1970231893@gmail.com","username":"yangjianfeng"},"change_message_id":"a933ce9bcb32c9cd88222601ac5c25f1fbc7eeba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"43bfc644_2c021c93","updated":"2025-09-22 11:24:13.000000000","message":"Hi Nova core team, When you have a chance, I would appreciate it if you could review this patch. I believe it helps resolve a long-standing issue.","commit_id":"bec0ba428bee3e7084a4c4e53a56e3531e1a8745"},{"author":{"_account_id":28329,"name":"yangjianfeng","display_name":"JeffYang","email":"yjf1970231893@gmail.com","username":"yangjianfeng"},"change_message_id":"c5cf93dfade42cce9acc1237d35c823e83421b22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bd849b0c_d1043f0e","updated":"2025-09-19 01:04:55.000000000","message":"recheck","commit_id":"bec0ba428bee3e7084a4c4e53a56e3531e1a8745"},{"author":{"_account_id":38925,"name":"Roman Hros","email":"roman.hros@dnation.cloud","username":"rhros"},"change_message_id":"d7f9a34db336dbc3c839eef841a5d46cfcd27928","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"aee313b0_efcdf9db","updated":"2026-07-31 13:27:34.000000000","message":"If I understand this patch correctly, you check for vnic_type and profile if it is the same, and if not reraise.\nI see 2 problems here:\n1. You cannot check vif_type or vif_details, and I am not sure but maybe it can influence later migration behavior.\n2. Profile can be the same, but doesn\u0027t have to be, e.g. see [2] in https://bugs.launchpad.net/neutron/+bug/1979072. I think it is based on the place, where rollback failed - before or after deleting \"migrating_to\" parameter.\n\nThat\u0027s why I decided to implement it differently in https://review.opendev.org/c/openstack/nova/+/999374.\nI am not reusing but calling delete + create. So new port has always correct attributes. Also I am not checking profile at all.","commit_id":"b8aba2e0ea23266518331887859001ee134c86c6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c0dbc20fead7b81f9fab3f224e7601333bfaf59e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"80c86980_bb035606","updated":"2026-07-31 13:54:32.000000000","message":"as noted on https://review.opendev.org/c/openstack/nova/+/999374 this shoudl not be a operator configable behvior.\n\ni think ultipmatly some varation on \nhttps://review.opendev.org/c/openstack/nova/+/653506 updated for master and with addtioanl testing will be the correct path forward.\n\nin the event that the destination portbingin exts i think the correct behivor will be ot detect if multiple port bidnign is in use and either create the destination protbinding uncondtionally or if multipel port bindign are not in use update the port as https://review.opendev.org/c/openstack/nova/+/653506 did.","commit_id":"b8aba2e0ea23266518331887859001ee134c86c6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2791c3c2328b254217b3033bc490c1ec96301b18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1b256974_345e04fd","in_reply_to":"80c86980_bb035606","updated":"2026-07-31 13:56:52.000000000","message":"https://review.opendev.org/c/openstack/nova/+/999374  is closer to the corret behvior it just does nto account for the fact that we could be using a single port binding instead of multipel if your using some out of tree neutron backend like cisco aci? or calico.","commit_id":"b8aba2e0ea23266518331887859001ee134c86c6"},{"author":{"_account_id":38925,"name":"Roman Hros","email":"roman.hros@dnation.cloud","username":"rhros"},"change_message_id":"a3a4e00d93a00945b487861ffc7cae0ea738591b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"42dbd51d_44dda31e","in_reply_to":"aee313b0_efcdf9db","updated":"2026-07-31 13:41:16.000000000","message":"Marked as unresolved.","commit_id":"b8aba2e0ea23266518331887859001ee134c86c6"}]}