)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4e1fecf3405d109c20271ebc3e8568dec77ebc82","unresolved":true,"context_lines":[{"line_number":12,"context_line":"causing drivers meant to bind at lower levels to incorrectly bind"},{"line_number":13,"context_line":"at level 0."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rather than reverting the NetworkContext fix entirely [2], filter"},{"line_number":16,"context_line":"out dynamic segments at the single point where they must not appear:"},{"line_number":17,"context_line":"the initial call to ``_bind_port_level`` at level 0. Dynamic segments"},{"line_number":18,"context_line":"reach lower binding levels exclusively via ``continue_binding()``, so"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dd36a03d_b040d1c3","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":57},"updated":"2026-08-24 13:37:25.000000000","message":"Why https://review.opendev.org/c/openstack/neutron/+/994936 is not enough?","commit_id":"1e9cb2ab2d66049459be9e3e11a4653e4f83e56a"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"35d1a25f63b5c4e499c00a99fe196a2d828ab1e8","unresolved":true,"context_lines":[{"line_number":12,"context_line":"causing drivers meant to bind at lower levels to incorrectly bind"},{"line_number":13,"context_line":"at level 0."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rather than reverting the NetworkContext fix entirely [2], filter"},{"line_number":16,"context_line":"out dynamic segments at the single point where they must not appear:"},{"line_number":17,"context_line":"the initial call to ``_bind_port_level`` at level 0. Dynamic segments"},{"line_number":18,"context_line":"reach lower binding levels exclusively via ``continue_binding()``, so"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1b629631_cef60cb7","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":57},"in_reply_to":"dd36a03d_b040d1c3","updated":"2026-08-24 19:32:38.000000000","message":"If we do that revert, and backport it to stable/2026.1 - then we need to backport https://review.opendev.org/c/openstack/neutron/+/989141 to stable/2026.1. Otherwise we break unbind on stable/2026.1.\n\nPros with this fix:\n* all segments are in NetworkContext but filtered out at level 0 binding — both concerns solved?\n* _expand_segment always works from memory regardless of DB state\n* drivers at any lifecycle stage (bind, update, delete) always see a consistent segment view, independent of ordering or who releases what\n\nBut, I think Thomas suggestion on making the release of segment a neutron \"core\" responsibility is interesting, but needs discussion/research.\n\nI would be ok with https://review.opendev.org/c/openstack/neutron/+/994936 + backport https://review.opendev.org/c/openstack/neutron/+/989141. It is the minimal fix, and gives room to explore other improvements?","commit_id":"1e9cb2ab2d66049459be9e3e11a4653e4f83e56a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4e1fecf3405d109c20271ebc3e8568dec77ebc82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8ef64465_ac97cf7b","updated":"2026-08-24 13:37:25.000000000","message":"So far we have 3 approaches:\n* This patch\n* The revert: https://review.opendev.org/c/openstack/neutron/+/1000862\n* Other fix for the segment deletion: https://review.opendev.org/c/openstack/neutron/+/994936\n\nWe should decide which one is better","commit_id":"1e9cb2ab2d66049459be9e3e11a4653e4f83e56a"}]}