)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"147af80f2ee4d0dca5a5c946fe1033eb082b4abe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5fa2648f_620bd9f0","updated":"2026-09-08 07:56:28.000000000","message":"recheck openstack-tox-docs","commit_id":"241570c09b8ca57b6832bba2164cd8cbd049d8b9"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"405855067c51a768a9a9752c212b9c76142ea500","unresolved":true,"context_lines":[{"line_number":1243,"context_line":"            # 559e50ac8a0849d6a43b4ddda844ad348583d2d3: if the port has no"},{"line_number":1244,"context_line":"            # subnets and this is a routed provider network with multiple"},{"line_number":1245,"context_line":"            # segments, an empty list is returned."},{"line_number":1246,"context_line":"            pass"},{"line_number":1247,"context_line":"        elif subnets:"},{"line_number":1248,"context_line":"            for segment in context.segments_to_bind:"},{"line_number":1249,"context_line":"                for subnet in subnets:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3836a958_2a2c0895","line":1246,"updated":"2026-09-11 09:30:54.000000000","message":"just a question: why not ``return allowed_binding_segments`` or ``return []`` here?","commit_id":"a2be88f6c229e4e50b72d953a903b227845711d3"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b9f043f852dd715a689686f1bc2d7973b271a11e","unresolved":true,"context_lines":[{"line_number":1243,"context_line":"            # 559e50ac8a0849d6a43b4ddda844ad348583d2d3: if the port has no"},{"line_number":1244,"context_line":"            # subnets and this is a routed provider network with multiple"},{"line_number":1245,"context_line":"            # segments, an empty list is returned."},{"line_number":1246,"context_line":"            pass"},{"line_number":1247,"context_line":"        elif subnets:"},{"line_number":1248,"context_line":"            for segment in context.segments_to_bind:"},{"line_number":1249,"context_line":"                for subnet in subnets:"}],"source_content_type":"text/x-python","patch_set":3,"id":"b079a9f1_dbd442eb","line":1246,"in_reply_to":"3836a958_2a2c0895","updated":"2026-09-11 09:37:49.000000000","message":"No reason at all. The logic is the same but is always better to fast exit. Your suggestion makes the code more explicit (and better).","commit_id":"a2be88f6c229e4e50b72d953a903b227845711d3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"0372ca47b593d226fc4b14fcd7f4b6b95a726473","unresolved":false,"context_lines":[{"line_number":1243,"context_line":"            # 559e50ac8a0849d6a43b4ddda844ad348583d2d3: if the port has no"},{"line_number":1244,"context_line":"            # subnets and this is a routed provider network with multiple"},{"line_number":1245,"context_line":"            # segments, an empty list is returned."},{"line_number":1246,"context_line":"            pass"},{"line_number":1247,"context_line":"        elif subnets:"},{"line_number":1248,"context_line":"            for segment in context.segments_to_bind:"},{"line_number":1249,"context_line":"                for subnet in subnets:"}],"source_content_type":"text/x-python","patch_set":3,"id":"2ec58378_43fefaf1","line":1246,"in_reply_to":"b079a9f1_dbd442eb","updated":"2026-09-11 09:49:33.000000000","message":"ok, thx for explanation. I was affraid that I am missing something :)","commit_id":"a2be88f6c229e4e50b72d953a903b227845711d3"}],"neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ed38e97b7ba422c4898bd70ff3abf661a24689a5","unresolved":false,"context_lines":[{"line_number":1660,"context_line":"            fake_segments[1][\u0027id\u0027], portbindings.VIF_TYPE_OVS, vif_details)"},{"line_number":1661,"context_line":""},{"line_number":1662,"context_line":"    def test_bind_port_vlan_subnet_without_segment_id(self):"},{"line_number":1663,"context_line":"        \"\"\"Bind to the first compatible segment when subnet has no segment_id.\"\"\""},{"line_number":1664,"context_line":"        segment_attrs_1 \u003d {\u0027network_type\u0027: \u0027vlan\u0027,"},{"line_number":1665,"context_line":"                           \u0027physical_network\u0027: \u0027fake-physnet\u0027,"},{"line_number":1666,"context_line":"                           \u0027segmentation_id\u0027: 23}"}],"source_content_type":"text/x-python","patch_set":1,"id":"719bc492_4b3f0873","line":1663,"updated":"2026-09-07 09:40:46.000000000","message":"pep8: E501 line too long (81 \u003e 79 characters)","commit_id":"2fd5d6649fee0eed517e0b4d3f01360071cd3ed6"}]}