)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"c6b3143ce4d9772ffa2c717dbc3a728448794a01","unresolved":true,"context_lines":[{"line_number":10,"context_line":"creates and tears down VNI infrastructure on the VTEP node by"},{"line_number":11,"context_line":"shelling out to sudo ip and sudo vtysh. This mirrors the static"},{"line_number":12,"context_line":"setup done by the Zuul playbook but allows per-test VNI"},{"line_number":13,"context_line":"allocation, enabling parallel EVPN test execution and tests"},{"line_number":14,"context_line":"that require multiple VNIs."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"VNI allocation uses a shared file-based counter protected by an"},{"line_number":17,"context_line":"external lock (oslo.concurrency) so that parallel tempest"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"1770d1c7_a1c0c2e1","line":14,"range":{"start_line":13,"start_character":12,"end_line":14,"end_character":27},"updated":"2026-07-24 21:55:25.000000000","message":"I think this might be problematic in the current state. For example, if you have two concurrent tests using the same VNI then they might interfere. I also dont know what will happen if you have multiple tests accessing same frr service.\nWe can maybe solve in similar way as in neutron functional test.\nhttps://opendev.org/openstack/neutron/src/branch/master/neutron/tests/common/net_helpers.py#L1094\n\nFor each test we will start frr in a namespace, this will isolate each frr instance.","commit_id":"52a1ac0159d0a858c070798a9560723cf3b6b331"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"4bbda313ee72a43f168a7ce10fa75de55599aea3","unresolved":false,"context_lines":[{"line_number":10,"context_line":"creates and tears down VNI infrastructure on the VTEP node by"},{"line_number":11,"context_line":"shelling out to sudo ip and sudo vtysh. This mirrors the static"},{"line_number":12,"context_line":"setup done by the Zuul playbook but allows per-test VNI"},{"line_number":13,"context_line":"allocation, enabling parallel EVPN test execution and tests"},{"line_number":14,"context_line":"that require multiple VNIs."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"VNI allocation uses a shared file-based counter protected by an"},{"line_number":17,"context_line":"external lock (oslo.concurrency) so that parallel tempest"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9f11abfa_b9600ca6","line":14,"range":{"start_line":13,"start_character":12,"end_line":14,"end_character":27},"in_reply_to":"1770d1c7_a1c0c2e1","updated":"2026-07-31 19:53:34.000000000","message":"Never mind, I see now that `allocate_vni` will automatically hand out VNIs per for each test and make sure VNI are unique. The only downside of that is that test can not explicitly set which VNI they want to use, but that should be ok.","commit_id":"52a1ac0159d0a858c070798a9560723cf3b6b331"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"9bd181e108dd1f93d5822cef44218c1d13c93683","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"8d7915a2_bfd5ece4","updated":"2026-08-01 00:40:18.000000000","message":"recheck","commit_id":"71c1b1d5600b8d469730cc392dce6d9ed3ab246c"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"3e8ebe9b77412d0313efeced766b0d15e18cb996","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"147d1e5b_3054867c","updated":"2026-08-04 18:49:55.000000000","message":"recheck https://review.opendev.org/c/openstack/neutron-fwaas/+/999712 merged","commit_id":"5e704a88a58cdbb64baa1fb7f7b6654b71ae2af7"}],"neutron_tempest_plugin/scenario/test_evpn.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"93790fd3abe515752e9da8ffa94dae945d4f9f9a","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from tempest.lib.common.utils import data_utils"},{"line_number":17,"context_line":"from tempest.lib import decorators"},{"line_number":18,"context_line":"from tempest.lib import exceptions as lib_exc"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from neutron_tempest_plugin.common.evpn_provisioner import ("},{"line_number":21,"context_line":"    EVPNVNIProvisioner)"}],"source_content_type":"text/x-python","patch_set":12,"id":"be555e9c_6d9255be","line":18,"updated":"2026-08-01 05:22:13.000000000","message":"pep8: F401 \u0027tempest.lib.exceptions as lib_exc\u0027 imported but unused","commit_id":"6b417b83d3f1f105e957e784d15fe26f065ee70b"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"93790fd3abe515752e9da8ffa94dae945d4f9f9a","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from neutron_tempest_plugin.common.evpn_provisioner import ("},{"line_number":21,"context_line":"    EVPNVNIProvisioner)"},{"line_number":22,"context_line":"from neutron_tempest_plugin.common import ssh"},{"line_number":23,"context_line":"from neutron_tempest_plugin import config"},{"line_number":24,"context_line":"from neutron_tempest_plugin.scenario import base"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"037b41f8_9ed1d2e7","line":22,"updated":"2026-08-01 05:22:13.000000000","message":"pep8: F401 \u0027neutron_tempest_plugin.common.ssh\u0027 imported but unused","commit_id":"6b417b83d3f1f105e957e784d15fe26f065ee70b"}]}