)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":26832,"name":"Stefan Hoffmann","email":"stefan.hoffmann@cloudandheat.com","username":"shoffmann"},"change_message_id":"bd44724408db9da89eb7f0a360f4fb8b19f9a0d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"31f9e5e3_4aaa6420","updated":"2025-02-26 12:06:14.000000000","message":"new approach for that at https://review.opendev.org/c/openstack/nova/+/942786\nabandoning this change.","commit_id":"a723af98e9b710eb9b25326e0681d689ce7e7a16"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1ac099b40e956ab894416a78121a8390c42ad8fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f4470e72_74b47c25","updated":"2024-07-19 12:24:57.000000000","message":"we had sever patches to correct the waiting for ovn.\n\nthjat is","commit_id":"a723af98e9b710eb9b25326e0681d689ce7e7a16"}],"nova/network/model.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1ac099b40e956ab894416a78121a8390c42ad8fe","unresolved":true,"context_lines":[{"line_number":493,"context_line":"        be sent by Neutron at \"plugtime\" - in other words, as soon as neutron"},{"line_number":494,"context_line":"        completes configuring the network backend."},{"line_number":495,"context_line":"        \"\"\""},{"line_number":496,"context_line":"        return self.is_hybrid_plug_enabled() or self.is_ovn_driver_used()"},{"line_number":497,"context_line":""},{"line_number":498,"context_line":"    def is_hybrid_plug_enabled(self):"},{"line_number":499,"context_line":"        return self[\u0027details\u0027].get(VIF_DETAILS_OVS_HYBRID_PLUG, False)"}],"source_content_type":"text/x-python","patch_set":1,"id":"16b4d2f5_2c375121","line":496,"updated":"2024-07-19 12:24:57.000000000","message":"ovn does not send plugtime event reliable\n\nwe have had multiple attpemt to try and wait for neutorn  and end up with \nhttps://github.com/openstack/nova/commit/0b0f40d1b308b29da537859b72080488560c23d4\nas the final solution.\n\nafter number other attpemts\nhttps://github.com/openstack/nova/commit/7a7a223602ca5aa0aca8f65a6ab143f1d8f8ec1b\nhttps://github.com/openstack/nova/commit/8b33ac064456482158b23c2a2d52f819ebb4c60e\nhttps://github.com/openstack/nova/commit/19f9b37721d9bc13bc1ed35a4f368b1d21b10a5b\n\n\nthis code is very very fragile and has a petty high change to cause regressions.\n\nwe need to be very careful not to regerss\nhttps://bugs.launchpad.net/neutron/+bug/1901707\nhttps://bugs.launchpad.net/neutron/+bug/1815989\n\nand the other work that was done to ensure tha tlive migration downtime is minimised\n\nas far as im awalre we elimiated the live migration downtime for ovn with \n\nhttps://review.opendev.org/c/openstack/neutron/+/828455\n\nwhich was heviley tested and backported to our downstream product as part of \nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d2104522\n\nim not saying this has not regressd\n\nbut the burden of proof for correctness is pretty high to modify this again given we have nto had report of operators having issue with this for 2 years.\n\nso before making any change to nova or neutron you should try and understand what has changed.\n\nthe behavior we have on master is the same behavior we have downstram in 17.1\nso i woudl like to understand how this regression was reintodufed on the neutorn/ovn side since we have not change the nova code in 4 years.","commit_id":"a723af98e9b710eb9b25326e0681d689ce7e7a16"}]}