)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"4a6a7e2ca4639eafcfc78ff518c9469e8f036ef5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9cde46dc_123dab9f","updated":"2026-09-25 14:37:47.000000000","message":"LGTM, I also checked if we allow setting it in config file, but maybe I am hallucinating because I dont think we ever did. Zuul just writes directly to OpenVswitch table and then agent reads from it.","commit_id":"e31f7a368d18023cc45a814b05ea41d734594798"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"0fdc16e0c5f2b112e7a8796ed259336d2d314927","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ad61219f_1f1f0db0","updated":"2026-09-25 14:28:29.000000000","message":"check experimental","commit_id":"e31f7a368d18023cc45a814b05ea41d734594798"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"dc2f44f26d7ed7b37560323ff0c32dcfc97977b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3935d66a_23db4812","updated":"2026-09-28 06:38:58.000000000","message":"I would add a couple of lines in the documentation about this change, saying that the FRR traffic will use the kernel routing","commit_id":"d9ca46858546f61eaba1bb97fd84ca1c1838db6f"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"65c267cbc5b94d0283b8b596e0ef6b3304a0ae21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aa404ca7_55815274","updated":"2026-09-29 08:17:19.000000000","message":"recheck\nfunctional failure is not related (some pkg discovery/mirroring problem as I see)","commit_id":"d9ca46858546f61eaba1bb97fd84ca1c1838db6f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a03417946a8253305b962727acc603c02474a2d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3116a407_be02fdd1","updated":"2026-09-28 21:55:30.000000000","message":"recheck again","commit_id":"d9ca46858546f61eaba1bb97fd84ca1c1838db6f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"115cf67e7b6dd3c571a8fc46823dbfb562358500","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d230f3aa_ebc2b0e5","updated":"2026-09-29 13:16:21.000000000","message":"recheck requirements patch merged now","commit_id":"d9ca46858546f61eaba1bb97fd84ca1c1838db6f"}],"neutron/agent/linux/svd.py":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"a2a7486195b5da9863b1045e67e1a0425517e857","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from neutron.privileged.agent.linux import svd as privileged_svd"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class SvdNoVxlanParent(Exception):"},{"line_number":25,"context_line":"    pass"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class SvdDeviceAlreadyExists(Exception):"}],"source_content_type":"text/x-python","patch_set":1,"id":"8fa34999_0ad59e9c","line":25,"range":{"start_line":24,"start_character":0,"end_line":25,"end_character":8},"updated":"2026-09-25 19:24:12.000000000","message":"Found this when rebasing my SVD change on top of yours. It would be best for this patch to remove it","commit_id":"e31f7a368d18023cc45a814b05ea41d734594798"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"c8ceed4353a827492c5ffa7fb5c69d0aca1492fe","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from neutron.privileged.agent.linux import svd as privileged_svd"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class SvdNoVxlanParent(Exception):"},{"line_number":25,"context_line":"    pass"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class SvdDeviceAlreadyExists(Exception):"}],"source_content_type":"text/x-python","patch_set":1,"id":"057c811f_b9b093ba","line":25,"range":{"start_line":24,"start_character":0,"end_line":25,"end_character":8},"in_reply_to":"8fa34999_0ad59e9c","updated":"2026-09-25 19:28:22.000000000","message":"Done","commit_id":"e31f7a368d18023cc45a814b05ea41d734594798"}],"neutron/agent/ovn/extensions/evpn/__init__.py":[{"author":{"_account_id":38298,"name":"Helen Chen","display_name":"Helen Chen","email":"ichen@redhat.com","username":"ingwherchen"},"change_message_id":"2b2d8694ab0ae35508f0ca81575e73d25cc8e0ac","unresolved":true,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f0d3e8f_c8146809","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"updated":"2026-09-25 18:24:05.000000000","message":"Will EVPN still work if OVS isn\u0027t configured with ovn-evpn-vxlan-ports?","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"dd4ebfc617b364f8dfded2965fd3171f6f8e4f98","unresolved":true,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"79cb0b6c_0d92d978","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"in_reply_to":"0a01bb69_e0ce0503","updated":"2026-09-25 19:22:23.000000000","message":"Can you elaborate what you mean? Are you saying we need to configure the parent for this vxlan device?","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"},{"author":{"_account_id":38298,"name":"Helen Chen","display_name":"Helen Chen","email":"ichen@redhat.com","username":"ingwherchen"},"change_message_id":"eef64888a7e593839e343e56300802c9a939d2cc","unresolved":false,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"5ca5ab3c_71c50ba1","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"in_reply_to":"414a4475_69863832","updated":"2026-09-25 20:03:04.000000000","message":"Acknowledged","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6794e4cee752550cf7be139dc228a6d978a6ef71","unresolved":true,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"b915e393_6ee42902","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"in_reply_to":"4d781631_356c78f2","updated":"2026-09-25 20:00:07.000000000","message":"Ah, I see. You\u0027re right but the vxlan_sys_4789 will still exist, it\u0027s created by OVS because OVN configures the vxlan interfaces on the br-int. We still need both `ovn-evpn-vxlan-local-ip` and `ovn-evpn-vxlan-ports` but that\u0027s for the ovn-controller.\n\nI don\u0027t think the agent extension should validate that as that is essentially an ovn-controller configuration. If the any of the required configurations is missing then it should be the ovn-controller that complains in its logs.\n\nThis patch just simply removes setting the vxlan_sys_4789 as the parent of svd vxlan but the configuration still applies and is needed for OVN.","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"},{"author":{"_account_id":38298,"name":"Helen Chen","display_name":"Helen Chen","email":"ichen@redhat.com","username":"ingwherchen"},"change_message_id":"96f3629303f1237d28a14215739de5b57edd1468","unresolved":true,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"4d781631_356c78f2","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"in_reply_to":"79cb0b6c_0d92d978","updated":"2026-09-25 19:47:58.000000000","message":"For the EVPN deployment to work, doesn\u0027t vxlan_sys_4789 still need to exist?  This means that both ovn-evpn-vxlan-local-ip and ovn-evpn-vxlan-ports must still be configured.  Even if it doesn\u0027t need to store the information, should the EVPN extension validate that the deployment has ovn-evpn-vxlan-local-ip and ovn-evpn-vxlan-ports configured?","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"1d6630b6ddb4516a033832fbcf24ec2370f9e7f6","unresolved":true,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"b4e87f3b_a71109ec","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"in_reply_to":"9f0d3e8f_c8146809","updated":"2026-09-25 18:32:30.000000000","message":"if we removed `ovn-evpn-vxlan-ports` from the local OVS db then ovn-controller should complain. But that\u0027s not what we\u0027re doing here. We only read that to be able to get the name of the vxlan parent","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"},{"author":{"_account_id":38298,"name":"Helen Chen","display_name":"Helen Chen","email":"ichen@redhat.com","username":"ingwherchen"},"change_message_id":"bd5d34632a15223fa41094f729208070eea445d4","unresolved":true,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"0a01bb69_e0ce0503","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"in_reply_to":"b4e87f3b_a71109ec","updated":"2026-09-25 18:54:13.000000000","message":"I just wonder if EVPN needs to validate that the parent vxlan is configured?  The parent vxlan configuration is still needed, even though EVPN no longer needs the parent vxlan\u0027s port information.","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"2f4d7acd3e72e31374b08481ca88146065de4b4c","unresolved":true,"context_lines":[{"line_number":68,"context_line":"            local_ip \u003d ext_ids[\u0027ovn-evpn-local-ip\u0027]"},{"line_number":69,"context_line":"            vxlan_port \u003d ext_ids[\u0027ovn-evpn-vxlan-ports\u0027]"},{"line_number":70,"context_line":"        except KeyError:"},{"line_number":71,"context_line":"            LOG.exception(\"Terminating: Must configure ovn-evpn-local-ip and \""},{"line_number":72,"context_line":"                          \"ovn-evpn-vxlan-ports\")"},{"line_number":73,"context_line":"            raise SystemExit(1)"},{"line_number":74,"context_line":"        vxlan_parent \u003d \u0027vxlan_sys_%s\u0027 % vxlan_port"},{"line_number":75,"context_line":"        mac \u003d net_lib.get_random_mac(CONF.base_mac.split(\u0027:\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"414a4475_69863832","side":"PARENT","line":72,"range":{"start_line":71,"start_character":72,"end_line":72,"end_character":47},"in_reply_to":"b915e393_6ee42902","updated":"2026-09-25 20:01:31.000000000","message":"I just tried and ovn-controller indeed complains after I removed the configuration\n```\n2026-09-25T20:00:48.498Z|00447|evpn_binding|WARN|Couldn\u0027t find EVPN tunnel for 4789 destination port.\n2026-09-25T20:00:48.498Z|00448|evpn_fdb|WARN|Couldn\u0027t find EVPN binding for fe:6b:bb:04:6d:86 MAC address.\n```","commit_id":"b1d0a8a53c087b99083be081a27886c7d894602b"}]}
