)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"7bba476c76f7306c8fb1edb5f7466c2d1849895e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ca212eec_ce4170a7","updated":"2025-02-27 13:36:21.000000000","message":"recheck","commit_id":"78253d8e8cd7e4f14d2fd3eb2906f9befd0426b3"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4d0a0a3770de51199fbea113c4a7e475c58ccc36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a47ec741_1e08e1e0","updated":"2025-02-28 10:32:30.000000000","message":"This functionality (trunk with parent port external) is something that is not tested nor validated. According to [1], an external port:\n```\n              Represents a logical port which is external and not having\n              an OVS port in the integration bridge.\n              \u003ccode\u003eOVN\u003c/code\u003e will never receive any traffic from this port or\n              send any traffic to this port\n```\n\nCan you share in the LP bug, from your environment, the OVN and OVS relevant database tables? Is the tagged VLAN traffic accessing correctly to the parent (and external) port?\n\nUnless it is proved that this functionality works (that shouldn\u0027t), what it should be done in the Neutron code is to allow to use an external port as parent port.\n\n[1]https://github.com/ovn-org/ovn/blob/b7f6cc6b23fe6b46e730d219b543e62af8ccba41/ovn-nb.xml#L1055-L1068","commit_id":"7de92a9a52c6fdc82c02dd7d547df3d2d67c3dd5"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"9af3a34f92dd8894719c85435e0e095b6feb937a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"49ba81f4_1a50ce07","updated":"2025-02-28 06:18:27.000000000","message":"recheck neutron-ovs-tempest-multinode-full","commit_id":"7de92a9a52c6fdc82c02dd7d547df3d2d67c3dd5"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"78e04426d9b4d99d8eeb330bd4700f920ebaa8eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0f24b97c_c1c887ba","updated":"2025-02-28 10:39:02.000000000","message":"thanks for review Rodolfo.","commit_id":"7de92a9a52c6fdc82c02dd7d547df3d2d67c3dd5"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"07c25e63296d084409638957e2efeeda504d7c64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b527f64a_c4ed2eec","in_reply_to":"22217746_15e2f501","updated":"2025-02-28 10:42:06.000000000","message":"In order to make dhcp work for trunk subports, they also should be configured as `external` which is done only for vnic_type `baremetal` ports. I\u0027ve attached ovn outputs into the linked bug https://launchpad.net/bugs/2099675","commit_id":"7de92a9a52c6fdc82c02dd7d547df3d2d67c3dd5"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"78e04426d9b4d99d8eeb330bd4700f920ebaa8eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"22217746_15e2f501","in_reply_to":"a47ec741_1e08e1e0","updated":"2025-02-28 10:39:02.000000000","message":"this functionality with BM/external ports impelented here https://review.opendev.org/c/openstack/networking-generic-switch/+/928490. \nAnd tempest tests that cover this use-case added here https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/941013\nPlease find test results here https://1cc594fa7dea4a43967d-b4ffb63fd72a873cfc8fbd2b6e893a02.ssl.cf5.rackcdn.com/928490/31/check/ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode/43269bc/testr_results.html under ironic_tempest_plugin.tests.scenario.test_baremetal_multitenancy.BaremetalMultitenancy\n\nI\u0027ve discovered this bug while implementing configdrive since in the tests we\u0027ve assigned IP address manually always. https://review.opendev.org/c/openstack/nova/+/941227\nAnd current patch fixes the issue.","commit_id":"7de92a9a52c6fdc82c02dd7d547df3d2d67c3dd5"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"05f590326824c3981e2d5c1d5a1b1daf90fa7369","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"42d6f3c7_9db9c55c","in_reply_to":"b527f64a_c4ed2eec","updated":"2025-02-28 12:49:20.000000000","message":"@ralonsoh@redhat.com I\u0027m sorry the provided links are for OVS deployment, what kind of outpus you would like to see for ovn?\n\novn-nbctl --db\u003dtcp:127.0.0.1:6641 --no-leader-only  show\novn-nbctl --db\u003dtcp:127.0.0.1:6641 --no-leader-only find Logical_Router_Port\novn-nbctl --db\u003dtcp:127.0.0.1:6641 --no-leader-only find Logical_Switch_Port\n\n?","commit_id":"7de92a9a52c6fdc82c02dd7d547df3d2d67c3dd5"}],"neutron/services/trunk/drivers/ovn/trunk_driver.py":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"8bfcec8f39877c9d47fc75da6778d7f40c9734f1","unresolved":true,"context_lines":[{"line_number":142,"context_line":"                port_obj.PortBinding.update_object("},{"line_number":143,"context_line":"                    context,"},{"line_number":144,"context_line":"                    {\u0027profile\u0027: binding.profile,"},{"line_number":145,"context_line":"                     \u0027vif_type\u0027: portbindings.VIF_TYPE_UNBOUND},"},{"line_number":146,"context_line":"                    port_id\u003dsubport.port_id,"},{"line_number":147,"context_line":"                    host\u003dbinding.host)"},{"line_number":148,"context_line":"                port_obj.PortBindingLevel.delete_objects("}],"source_content_type":"text/x-python","patch_set":1,"id":"556154be_8587645b","line":145,"updated":"2025-02-26 17:56:26.000000000","message":"probably need to unset vnic_type as well","commit_id":"2ee7130e924266d90ce2ddb0c33759e8ed5ff88b"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"e1f5ba497958bb31ef5d936edeb4fc7081429764","unresolved":false,"context_lines":[{"line_number":142,"context_line":"                port_obj.PortBinding.update_object("},{"line_number":143,"context_line":"                    context,"},{"line_number":144,"context_line":"                    {\u0027profile\u0027: binding.profile,"},{"line_number":145,"context_line":"                     \u0027vif_type\u0027: portbindings.VIF_TYPE_UNBOUND},"},{"line_number":146,"context_line":"                    port_id\u003dsubport.port_id,"},{"line_number":147,"context_line":"                    host\u003dbinding.host)"},{"line_number":148,"context_line":"                port_obj.PortBindingLevel.delete_objects("}],"source_content_type":"text/x-python","patch_set":1,"id":"9ebe4306_8fc284eb","line":145,"in_reply_to":"556154be_8587645b","updated":"2025-02-27 09:35:56.000000000","message":"Done","commit_id":"2ee7130e924266d90ce2ddb0c33759e8ed5ff88b"}],"neutron/services/trunk/plugin.py":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"8bfcec8f39877c9d47fc75da6778d7f40c9734f1","unresolved":true,"context_lines":[{"line_number":476,"context_line":"                subport_obj \u003d self.core_plugin.get_port("},{"line_number":477,"context_line":"                    context, subport[\u0027port_id\u0027])"},{"line_number":478,"context_line":"                subport_obj_update \u003d {\u0027port\u0027: {"},{"line_number":479,"context_line":"                                      portbindings.VNIC_TYPE: new_vnic_type}}"},{"line_number":480,"context_line":"                self.core_plugin.update_port(context, subport[\"port_id\"],"},{"line_number":481,"context_line":"                                             subport_obj_update)"},{"line_number":482,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"f06254e6_6a28fd6d","line":479,"updated":"2025-02-26 17:56:26.000000000","message":"probably need to set vif_type as well.","commit_id":"2ee7130e924266d90ce2ddb0c33759e8ed5ff88b"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"7bba476c76f7306c8fb1edb5f7466c2d1849895e","unresolved":false,"context_lines":[{"line_number":476,"context_line":"                subport_obj \u003d self.core_plugin.get_port("},{"line_number":477,"context_line":"                    context, subport[\u0027port_id\u0027])"},{"line_number":478,"context_line":"                subport_obj_update \u003d {\u0027port\u0027: {"},{"line_number":479,"context_line":"                                      portbindings.VNIC_TYPE: new_vnic_type}}"},{"line_number":480,"context_line":"                self.core_plugin.update_port(context, subport[\"port_id\"],"},{"line_number":481,"context_line":"                                             subport_obj_update)"},{"line_number":482,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"83dc263a_bc6aed67","line":479,"in_reply_to":"f06254e6_6a28fd6d","updated":"2025-02-27 13:36:21.000000000","message":"Done","commit_id":"2ee7130e924266d90ce2ddb0c33759e8ed5ff88b"}]}