)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"0b1278d4f588772085574f27be7312c2c88fa29b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1511fc61_f4cc12f2","updated":"2026-04-11 14:10:12.000000000","message":"Recheck","commit_id":"1697ff3e542053064a5e9a94bc7ba7f3cc6fef92"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"98d3930dfb93749a27437ad0736b37a68a80c174","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6a4ff543_77309f51","updated":"2026-04-13 06:30:51.000000000","message":"This broke ovn job https://zuul.openstack.org/builds?job_name\u003dironic-tempest-ovn-uefi-ipmi-pxe\u0026project\u003dopenstack%2Fneutron\u0026branch\u003dmaster\u0026skip\u003d0\n\n2026-04-12 01:51:28.990582 | controller | ++ /opt/stack/ironic/devstack/lib/ironic:configure_iptables:3661 :   pub_router_id\u003d51c8f74b-609a-4413-bd33-fd19c2c90341\n2026-04-12 01:51:28.994446 | controller | ++ /opt/stack/ironic/devstack/lib/ironic:configure_iptables:3662 :   [[ ! -z 51c8f74b-609a-4413-bd33-fd19c2c90341 ]]\n2026-04-12 01:51:28.998562 | controller | ++ /opt/stack/ironic/devstack/lib/ironic:configure_iptables:3663 :   sudo ip netns exec qrouter-51c8f74b-609a-4413-bd33-fd19c2c90341 iptables -A PREROUTING -t raw -p udp --dport 69 -j CT --helper tftp\n2026-04-12 01:51:29.011383 | controller | Cannot open network namespace \"qrouter-51c8f74b-609a-4413-bd33-fd19c2c90341\": No such file or directory\n2026-04-12 01:51:29.016653 | controller | + /opt/stack/ironic/devstack/lib/ironic:configure_iptables:1 :   exit_trap","commit_id":"1697ff3e542053064a5e9a94bc7ba7f3cc6fef92"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"981436dc7821c3b3121f83ebf6f9a59f6fa67a9f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d0825f65_1afd29d8","in_reply_to":"6a4ff543_77309f51","updated":"2026-04-13 09:48:21.000000000","message":"pushed https://review.opendev.org/c/openstack/ironic/+/984352","commit_id":"1697ff3e542053064a5e9a94bc7ba7f3cc6fef92"}],"devstack/lib/ironic":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"2c86f8e31428898155aff4453eb0d732787eedc3","unresolved":true,"context_lines":[{"line_number":3715,"context_line":""},{"line_number":3716,"context_line":"    # (rpittau) workaround to allow TFTP traffic on ubuntu bionic with conntrack helper disabled"},{"line_number":3717,"context_line":"    local qrouter"},{"line_number":3718,"context_line":"    qrouter\u003d$(sudo ip netns list | grep qrouter | awk \u0027{print $1;}\u0027)"},{"line_number":3719,"context_line":"    if [[ ! -z \"$qrouter\" ]]; then"},{"line_number":3720,"context_line":"        sudo ip netns exec $qrouter iptables -A PREROUTING -t raw -p udp --dport 69 -j CT --helper tftp"},{"line_number":3721,"context_line":"        sudo ip netns exec $qrouter ip6tables -A PREROUTING -t raw -p udp --dport 69 -j CT --helper tftp || true"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ee5a0f0a_5f45644d","side":"PARENT","line":3718,"updated":"2026-04-08 13:38:44.000000000","message":"ugh, yeah, this is OVS only.","commit_id":"45279830c450e4f7e6c92d39035b589a2b071abe"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"2c86f8e31428898155aff4453eb0d732787eedc3","unresolved":true,"context_lines":[{"line_number":3717,"context_line":"    local pub_router_id"},{"line_number":3718,"context_line":"    pub_router_id\u003d$(openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME -f value -c id)"},{"line_number":3719,"context_line":"    if [[ ! -z \"$pub_router_id\" ]]; then"},{"line_number":3720,"context_line":"        sudo ip netns exec qrouter-$pub_router_id iptables -A PREROUTING -t raw -p udp --dport 69 -j CT --helper tftp"},{"line_number":3721,"context_line":"        sudo ip netns exec qrouter-$pub_router_id ip6tables -A PREROUTING -t raw -p udp --dport 69 -j CT --helper tftp || true"},{"line_number":3722,"context_line":"    fi"},{"line_number":3723,"context_line":"}"},{"line_number":3724,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"5c3565ef_776cf451","line":3721,"range":{"start_line":3720,"start_character":117,"end_line":3721,"end_character":126},"updated":"2026-04-08 13:38:44.000000000","message":"still ovs-only, we likely need to add a note. There is no way this works with OVN + NAT.","commit_id":"1697ff3e542053064a5e9a94bc7ba7f3cc6fef92"}]}
