)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"718ab3dff58b8eb864ac00d8de34347c1669aff2","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This patch adds definition of the new API extension. That extension"},{"line_number":10,"context_line":"makes `router_id` attribute of Floating IP to be writable with POST and"},{"line_number":11,"context_line":"PUT requests and doing that will skip existing Neutron validation if"},{"line_number":12,"context_line":"Flaoting IP\u0027s network and tenant network have direct connection in any"},{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP."},{"line_number":15,"context_line":"Spec for that new feature was proposed in [1]."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ebfa5aab_ef08761e","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":8},"updated":"2026-08-05 12:56:12.000000000","message":"Floating","commit_id":"bb4333c18f9cdefd97520f8cd5a1f3dbd490d128"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"971e77a362e8c87b3eea2909b7658ec0a05129f5","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch adds definition of the new API extension. That extension"},{"line_number":10,"context_line":"makes `router_id` attribute of Floating IP to be writable with POST and"},{"line_number":11,"context_line":"PUT requests and doing that will skip existing Neutron validation if"},{"line_number":12,"context_line":"Flaoting IP\u0027s network and tenant network have direct connection in any"},{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP."},{"line_number":15,"context_line":"Spec for that new feature was proposed in [1]."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8c4cad42_cfc8a5e5","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":8},"in_reply_to":"ebfa5aab_ef08761e","updated":"2026-08-05 12:59:20.000000000","message":"Done","commit_id":"bb4333c18f9cdefd97520f8cd5a1f3dbd490d128"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"718ab3dff58b8eb864ac00d8de34347c1669aff2","unresolved":true,"context_lines":[{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP."},{"line_number":15,"context_line":"Spec for that new feature was proposed in [1]."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[1] https://review.opendev.org/c/openstack/neutron-specs/+/996269"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Related-bug: #2072505"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"96f1541c_d92a0605","line":16,"updated":"2026-08-05 12:56:12.000000000","message":"What Claude recommended:\n```\nInstead user will be able to tell explicitly which router will be\nused to do NAT for that Floating IP, bypassing the existing\nvalidation that requires a direct subnet connection between the\nfloating IP\u0027s external network and the tenant network through a\nrouter.\n```","commit_id":"bb4333c18f9cdefd97520f8cd5a1f3dbd490d128"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"971e77a362e8c87b3eea2909b7658ec0a05129f5","unresolved":false,"context_lines":[{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP."},{"line_number":15,"context_line":"Spec for that new feature was proposed in [1]."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[1] https://review.opendev.org/c/openstack/neutron-specs/+/996269"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Related-bug: #2072505"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6952e2f6_2822c4c7","line":16,"in_reply_to":"96f1541c_d92a0605","updated":"2026-08-05 12:59:20.000000000","message":"Done","commit_id":"bb4333c18f9cdefd97520f8cd5a1f3dbd490d128"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c16abeca54feb97ac3b616fa691285d19155ffc1","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This patch adds definition of the new API extension. That extension"},{"line_number":10,"context_line":"makes `router_id` attribute of Floating IP to be writable with POST and"},{"line_number":11,"context_line":"PUT requests and doing that will skip existing Neutron validation if"},{"line_number":12,"context_line":"Floating IP\u0027s network and tenant network have direct connection in any"},{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP, bypassing the existing"},{"line_number":15,"context_line":"validation that requires a direct subnet connection between the floating"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"006275b5_f0fc63ff","line":12,"range":{"start_line":12,"start_character":26,"end_line":12,"end_character":32},"updated":"2026-08-06 22:10:47.000000000","message":"s/project :)","commit_id":"653046f7ae6291102dc89129f0aba8f7a58690f3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"3f5fbbc267dea5575cb5d41dc85df64a70a35d52","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch adds definition of the new API extension. That extension"},{"line_number":10,"context_line":"makes `router_id` attribute of Floating IP to be writable with POST and"},{"line_number":11,"context_line":"PUT requests and doing that will skip existing Neutron validation if"},{"line_number":12,"context_line":"Floating IP\u0027s network and tenant network have direct connection in any"},{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP, bypassing the existing"},{"line_number":15,"context_line":"validation that requires a direct subnet connection between the floating"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"16a05c0c_8f4f1447","line":12,"range":{"start_line":12,"start_character":26,"end_line":12,"end_character":32},"in_reply_to":"006275b5_f0fc63ff","updated":"2026-08-07 07:15:03.000000000","message":"Done","commit_id":"653046f7ae6291102dc89129f0aba8f7a58690f3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c16abeca54feb97ac3b616fa691285d19155ffc1","unresolved":true,"context_lines":[{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP, bypassing the existing"},{"line_number":15,"context_line":"validation that requires a direct subnet connection between the floating"},{"line_number":16,"context_line":"IP\u0027s external network and the tenant network through a router."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"[1] https://review.opendev.org/c/openstack/neutron-specs/+/996269"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"aac22e54_1da60d4a","line":16,"range":{"start_line":16,"start_character":30,"end_line":16,"end_character":36},"updated":"2026-08-06 22:10:47.000000000","message":"same","commit_id":"653046f7ae6291102dc89129f0aba8f7a58690f3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"3f5fbbc267dea5575cb5d41dc85df64a70a35d52","unresolved":false,"context_lines":[{"line_number":13,"context_line":"router_id. Instead user will be able to tell explicitly which router"},{"line_number":14,"context_line":"will be used to do NAT for that Floating IP, bypassing the existing"},{"line_number":15,"context_line":"validation that requires a direct subnet connection between the floating"},{"line_number":16,"context_line":"IP\u0027s external network and the tenant network through a router."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"[1] https://review.opendev.org/c/openstack/neutron-specs/+/996269"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"8f552466_9fd571a4","line":16,"range":{"start_line":16,"start_character":30,"end_line":16,"end_character":36},"in_reply_to":"aac22e54_1da60d4a","updated":"2026-08-07 07:15:03.000000000","message":"Done","commit_id":"653046f7ae6291102dc89129f0aba8f7a58690f3"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"718ab3dff58b8eb864ac00d8de34347c1669aff2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"64ebd79d_01faabe4","updated":"2026-08-05 12:56:12.000000000","message":"minor nits, all good","commit_id":"bb4333c18f9cdefd97520f8cd5a1f3dbd490d128"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c16abeca54feb97ac3b616fa691285d19155ffc1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d6ae5452_b26682b0","updated":"2026-08-06 22:10:47.000000000","message":"just nits","commit_id":"653046f7ae6291102dc89129f0aba8f7a58690f3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2ca767d14d0a2360c85d998ff9d687da7f4c4749","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6e7fd6c6_f25181b0","updated":"2026-08-06 13:27:58.000000000","message":"recheck - unrelated issue with opendev.org","commit_id":"653046f7ae6291102dc89129f0aba8f7a58690f3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2c8a97b35071285aec6d7bf7ccfac3fbc313e9c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5e6f8749_7ca785a8","updated":"2026-08-07 15:10:24.000000000","message":"recheck unrelated test failure","commit_id":"976e82928d010d354486a5702d028af05adff0b1"}],"neutron_lib/api/definitions/base.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"718ab3dff58b8eb864ac00d8de34347c1669aff2","unresolved":true,"context_lines":[{"line_number":110,"context_line":"    \u0027floating-ip-port-forwarding-description\u0027,"},{"line_number":111,"context_line":"    \u0027floatingip-autodelete-internal\u0027,"},{"line_number":112,"context_line":"    \u0027floatingip-pools\u0027,"},{"line_number":113,"context_line":"    \u0027floating-ip-router-writable\u0027,"},{"line_number":114,"context_line":"    \u0027ip-substring-filtering\u0027,"},{"line_number":115,"context_line":"    \u0027l3-agent-scheduler-ha-chassis-priority\u0027,"},{"line_number":116,"context_line":"    \u0027l3-conntrack-helper\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3c84c04d_863e06b1","line":113,"range":{"start_line":113,"start_character":4,"end_line":113,"end_character":34},"updated":"2026-08-05 12:56:12.000000000","message":"nit: this should be right after `\u0027floating-ip-port-forwarding-description\u0027`","commit_id":"bb4333c18f9cdefd97520f8cd5a1f3dbd490d128"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"971e77a362e8c87b3eea2909b7658ec0a05129f5","unresolved":false,"context_lines":[{"line_number":110,"context_line":"    \u0027floating-ip-port-forwarding-description\u0027,"},{"line_number":111,"context_line":"    \u0027floatingip-autodelete-internal\u0027,"},{"line_number":112,"context_line":"    \u0027floatingip-pools\u0027,"},{"line_number":113,"context_line":"    \u0027floating-ip-router-writable\u0027,"},{"line_number":114,"context_line":"    \u0027ip-substring-filtering\u0027,"},{"line_number":115,"context_line":"    \u0027l3-agent-scheduler-ha-chassis-priority\u0027,"},{"line_number":116,"context_line":"    \u0027l3-conntrack-helper\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"054bec35_7cfdbd49","line":113,"range":{"start_line":113,"start_character":4,"end_line":113,"end_character":34},"in_reply_to":"3c84c04d_863e06b1","updated":"2026-08-05 12:59:20.000000000","message":"Done","commit_id":"bb4333c18f9cdefd97520f8cd5a1f3dbd490d128"}]}
