)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"1c0b922190eb4ad02a0c8ec54b26e53c2f8bc8f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"50938ca5_0bdf341d","updated":"2026-09-04 10:08:37.000000000","message":"Tested locally and it works as expected, runs after evpn.","commit_id":"a6110fabf33029301da4d340b9fa03a2fe1a6947"}],"neutron/services/pvlan/db_sync.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"58691c41d03a89e3f1cc2ac3c6dfe37b943ee470","unresolved":true,"context_lines":[{"line_number":48,"context_line":"    def do_sync(self):"},{"line_number":49,"context_line":"        LOG.debug(\u0027OVN-NB Sync PVLAN started @ %s\u0027, str(datetime.now()))"},{"line_number":50,"context_line":"        ctx \u003d context.get_admin_context()"},{"line_number":51,"context_line":"        self.sync_pvlan(ctx)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @staticmethod"},{"line_number":54,"context_line":"    def _define_drop_pvlan_acls():"}],"source_content_type":"text/x-python","patch_set":1,"id":"04392482_ec7d4dfc","line":51,"updated":"2026-09-08 07:34:33.000000000","message":"nit: do we need to have sync_pvlan() method addionally? Can\u0027t content of that method be simply moved here?","commit_id":"a6110fabf33029301da4d340b9fa03a2fe1a6947"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"b717e42ba108cb8549216efdf7a24ee24e66c8f1","unresolved":true,"context_lines":[{"line_number":48,"context_line":"    def do_sync(self):"},{"line_number":49,"context_line":"        LOG.debug(\u0027OVN-NB Sync PVLAN started @ %s\u0027, str(datetime.now()))"},{"line_number":50,"context_line":"        ctx \u003d context.get_admin_context()"},{"line_number":51,"context_line":"        self.sync_pvlan(ctx)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @staticmethod"},{"line_number":54,"context_line":"    def _define_drop_pvlan_acls():"}],"source_content_type":"text/x-python","patch_set":1,"id":"f285c11c_3f446038","line":51,"in_reply_to":"04392482_ec7d4dfc","updated":"2026-09-14 16:33:36.000000000","message":"Done!","commit_id":"a6110fabf33029301da4d340b9fa03a2fe1a6947"}],"neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovn_db_sync.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"58691c41d03a89e3f1cc2ac3c6dfe37b943ee470","unresolved":true,"context_lines":[{"line_number":2566,"context_line":"        self.assertNotIn(self.evpn_ls_name, self._get_ovn_evpn_ls_names())"},{"line_number":2567,"context_line":""},{"line_number":2568,"context_line":""},{"line_number":2569,"context_line":"class TestOvnNbSyncPVLAN(base.TestOVNFunctionalBase):"},{"line_number":2570,"context_line":"    \"\"\"Functional tests for PVLAN sync in ovn-db-sync.\"\"\""},{"line_number":2571,"context_line":""},{"line_number":2572,"context_line":"    _extension_drivers \u003d [\u0027port_security\u0027, \u0027qos\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"0ebe613e_174d9953","line":2569,"updated":"2026-09-08 07:34:33.000000000","message":"shouldn\u0027t we now move this test class to separate file `neutron/tests/functional/services/pvlan/test_db_sync.py`?","commit_id":"a6110fabf33029301da4d340b9fa03a2fe1a6947"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"b717e42ba108cb8549216efdf7a24ee24e66c8f1","unresolved":true,"context_lines":[{"line_number":2566,"context_line":"        self.assertNotIn(self.evpn_ls_name, self._get_ovn_evpn_ls_names())"},{"line_number":2567,"context_line":""},{"line_number":2568,"context_line":""},{"line_number":2569,"context_line":"class TestOvnNbSyncPVLAN(base.TestOVNFunctionalBase):"},{"line_number":2570,"context_line":"    \"\"\"Functional tests for PVLAN sync in ovn-db-sync.\"\"\""},{"line_number":2571,"context_line":""},{"line_number":2572,"context_line":"    _extension_drivers \u003d [\u0027port_security\u0027, \u0027qos\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"6bf94d34_7d3456d8","line":2569,"in_reply_to":"0ebe613e_174d9953","updated":"2026-09-14 16:33:36.000000000","message":"I did it this way to replicate how it was done for evpn https://review.opendev.org/c/openstack/neutron/+/999198","commit_id":"a6110fabf33029301da4d340b9fa03a2fe1a6947"}],"neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovn_db_sync.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"58691c41d03a89e3f1cc2ac3c6dfe37b943ee470","unresolved":true,"context_lines":[{"line_number":1646,"context_line":"        self.network_id \u003d network_id"},{"line_number":1647,"context_line":""},{"line_number":1648,"context_line":""},{"line_number":1649,"context_line":"class TestSyncPVLAN(test_mech_driver.OVNMechanismDriverTestCase):"},{"line_number":1650,"context_line":""},{"line_number":1651,"context_line":"    def setUp(self):"},{"line_number":1652,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"800df55c_6b5c95e2","line":1649,"updated":"2026-09-08 07:34:33.000000000","message":"similar question regarding this class too","commit_id":"a6110fabf33029301da4d340b9fa03a2fe1a6947"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"b717e42ba108cb8549216efdf7a24ee24e66c8f1","unresolved":true,"context_lines":[{"line_number":1646,"context_line":"        self.network_id \u003d network_id"},{"line_number":1647,"context_line":""},{"line_number":1648,"context_line":""},{"line_number":1649,"context_line":"class TestSyncPVLAN(test_mech_driver.OVNMechanismDriverTestCase):"},{"line_number":1650,"context_line":""},{"line_number":1651,"context_line":"    def setUp(self):"},{"line_number":1652,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"5b049eda_7131a657","line":1649,"in_reply_to":"800df55c_6b5c95e2","updated":"2026-09-14 16:33:36.000000000","message":"same reason as the one above. I can change both if needed!","commit_id":"a6110fabf33029301da4d340b9fa03a2fe1a6947"}]}