)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a56a8c63a354af31b8d3796533cd536d29f3ed64","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"bgp: Don\u0027t allow VLAN networks"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The ovn-bgp service plugin registers to the before commit event"},{"line_number":10,"context_line":"and fails the request if a VLAN provider network is attempted to be"},{"line_number":11,"context_line":"create."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"d4a9a3de_f2528d59","line":9,"updated":"2026-05-18 13:28:54.000000000","message":"Hi Jakub, for me is not clear why VLAN is not supported with the BGP plugin. If this because how the compute bgp bridges are connected to the provider network? Can you give more detail?\n\nAlso a documentation appendix would be nice.","commit_id":"4fc78ccca871071a04e8d3d07d82cfd71ce68641"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"f515f21d87c5a289e0ad784ae133c10fe140ae98","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"bgp: Don\u0027t allow VLAN networks"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The ovn-bgp service plugin registers to the before commit event"},{"line_number":10,"context_line":"and fails the request if a VLAN provider network is attempted to be"},{"line_number":11,"context_line":"create."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"892343d8_6a029aee","line":9,"in_reply_to":"d4a9a3de_f2528d59","updated":"2026-05-19 13:25:03.000000000","message":"I put it to a followup patch along with basic documentaiton: https://review.opendev.org/c/openstack/neutron/+/989128","commit_id":"4fc78ccca871071a04e8d3d07d82cfd71ce68641"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"c4f15dd92451d8ef08a6b3865a6af27f158437b4","unresolved":true,"context_lines":[{"line_number":10,"context_line":"and fails the request if a VLAN provider network is attempted to be"},{"line_number":11,"context_line":"create."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Resolves-Bug: #2152203"},{"line_number":14,"context_line":"Assisted-By: claude-4.6-opus"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I57251da8e2661d984d29257ce2f81c72f29ff808"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"f2cebd75_748b27f3","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":8},"updated":"2026-05-19 13:44:33.000000000","message":"```suggestion\nCloses-Bug: #2152203\n```","commit_id":"1facf0d1f524f7711b28643fe3a285828b70d390"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6c31c8cbf50ccc6d42798631cd8486feec5deb02","unresolved":false,"context_lines":[{"line_number":10,"context_line":"and fails the request if a VLAN provider network is attempted to be"},{"line_number":11,"context_line":"create."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Resolves-Bug: #2152203"},{"line_number":14,"context_line":"Assisted-By: claude-4.6-opus"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I57251da8e2661d984d29257ce2f81c72f29ff808"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"68dc1756_f56f7c62","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":8},"in_reply_to":"f2cebd75_748b27f3","updated":"2026-05-19 14:46:41.000000000","message":"Fix applied.","commit_id":"1facf0d1f524f7711b28643fe3a285828b70d390"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"5b44a389db3769036b8ed110069b772ea60b53a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d8d089bc_5f2cf661","updated":"2026-05-20 08:46:22.000000000","message":"Rebased to be on top of the FT CI fix","commit_id":"3ea437e660ab0eea38a2a17f8e07d808a2b9d0b0"}],"neutron/tests/unit/services/bgp/test_plugin.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ee35670bc1ab7072194c721daffad9b713f204b8","unresolved":true,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def test_create_network_geneve_allowed(self):"},{"line_number":49,"context_line":"        res \u003d self._create_network("},{"line_number":50,"context_line":"            self.fmt, \u0027net-geneve\u0027, admin_state_up\u003dTrue, as_admin\u003dTrue)"},{"line_number":51,"context_line":"        self.assertEqual(webob.exc.HTTPCreated.code, res.status_int)"}],"source_content_type":"text/x-python","patch_set":1,"id":"62a81feb_cee2af47","line":50,"updated":"2026-05-11 22:58:21.000000000","message":"Is this really creating a Geneve network? self._create_network() doesn\u0027t pass it anywhere","commit_id":"f10d78f51b551a77ad21c62ca6c581a387d1d0e9"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"bc6da531862402f5fc5a0c6b9436cfe2763248fa","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def test_create_network_geneve_allowed(self):"},{"line_number":49,"context_line":"        res \u003d self._create_network("},{"line_number":50,"context_line":"            self.fmt, \u0027net-geneve\u0027, admin_state_up\u003dTrue, as_admin\u003dTrue)"},{"line_number":51,"context_line":"        self.assertEqual(webob.exc.HTTPCreated.code, res.status_int)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b5cccfd6_478d0c5b","line":50,"in_reply_to":"62a81feb_cee2af47","updated":"2026-05-15 21:35:07.000000000","message":"Good catch, better be explicit.","commit_id":"f10d78f51b551a77ad21c62ca6c581a387d1d0e9"}]}
