)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"726f1f662c5d902cc1e610479d7262870b22e715","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":44,"id":"952c8f05_341b7bbe","updated":"2023-08-17 14:20:01.000000000","message":"recheck","commit_id":"fe263a6b59a156263272e4c403e41bd9a5ba9da1"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"2025b91fb74fa5d5e201a1b5c6ac92f3f0750a45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":51,"id":"20282de6_2ee8f586","updated":"2023-08-29 13:33:29.000000000","message":"recheck redfish job failure appears to be transient. Pxe-grub appears to be incompatability - opendev hold requested","commit_id":"03ff6c6fac5ba365c9a141f6e7bdb9ee52f0a720"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ef73e23bb3471f9749397e9beb91841c85c74f95","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":55,"id":"7c2d04ea_c845d27f","updated":"2023-09-05 18:50:54.000000000","message":"This is fine; I wish the docs were more pessimistic but I can understand why not :). At least for one cycle warning people away isn\u0027t so terrible.","commit_id":"01bd925cd2096a0d610a5773d9986824914b26ea"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"292ef2ef1553cb9a0b4d23b81641f8014d7f72ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":56,"id":"bbdcdec0_2fe713ee","updated":"2023-09-07 15:14:22.000000000","message":"recheck CI is happier now that we merged fixes.","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"557cd854ba23184a2d14acb4a2735d990f587a50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":56,"id":"22faff04_4dff14ae","updated":"2023-09-05 19:14:17.000000000","message":"this is essentially a +2 I\u0027m just waiting for the final V+1 version of this to put a final vote on it","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"9a993c569d0974d1bd6c970c7b8611babe4cddba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":59,"id":"1829bf87_d20ee623","updated":"2023-09-27 16:16:53.000000000","message":"recheck","commit_id":"f4ac8f56cf2f22893552e4e83fc79c95d0dc660a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f465efd18b6aafb6e898166843e8f4b78da11dbf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":60,"id":"c8302f98_c370bab1","updated":"2023-10-09 13:13:03.000000000","message":"Henceforth, all boops and bumps shall be referred to as \"moopsy\".\n\nMoopsy!","commit_id":"3f77091c6320e196c83ecd10b9a8efd9d437e9f2"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"25b36e1f506e9cd1a4eca12b372f5e322189471f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":60,"id":"20061d4a_b23ad992","updated":"2023-10-09 15:23:04.000000000","message":"Please fix nits here or in a follow-up.","commit_id":"3f77091c6320e196c83ecd10b9a8efd9d437e9f2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4b1f94fb8763e6a43b3e2c15b4032f3cf6b47479","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":60,"id":"e8611534_bfc94530","updated":"2023-10-10 13:11:03.000000000","message":"recheck","commit_id":"3f77091c6320e196c83ecd10b9a8efd9d437e9f2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1c0793a708a9ad154cdfc3cbddf1ff9fbe6e28f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":60,"id":"9711b7f0_d8a0677b","updated":"2023-10-10 00:47:47.000000000","message":"recheck streaming error from build","commit_id":"3f77091c6320e196c83ecd10b9a8efd9d437e9f2"}],"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":"2025b91fb74fa5d5e201a1b5c6ac92f3f0750a45","unresolved":false,"context_lines":[{"line_number":2073,"context_line":"    sudo ip link set dev $brbm_tap up"},{"line_number":2074,"context_line":"    sudo ip link set dev $ovs_tap up"},{"line_number":2075,"context_line":""},{"line_number":2076,"context_line":"    sudo ovs-vsctl -- --if-exists del-port $ovs_tap -- add-port br-int $ovs_tap tag\u003d$tag_id"},{"line_number":2077,"context_line":"    sudo ovs-vsctl -- --if-exists del-port $brbm_tap -- add-port $IRONIC_VM_NETWORK_BRIDGE $brbm_tap"},{"line_number":2078,"context_line":""},{"line_number":2079,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":44,"id":"dc7f187a_cef831fa","line":2076,"range":{"start_line":2076,"start_character":64,"end_line":2076,"end_character":70},"updated":"2023-08-29 13:33:29.000000000","message":"this needs to be br-ex for OVN","commit_id":"fe263a6b59a156263272e4c403e41bd9a5ba9da1"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"7bc08610c6d4a78b166607c5007adc8492460fd2","unresolved":true,"context_lines":[{"line_number":2246,"context_line":"            # Need to handle the json dict we get from the API (yeah, wut?!)"},{"line_number":2247,"context_line":"            # and transform that so jq can do the needful. We also can\u0027t store"},{"line_number":2248,"context_line":"            # it as a variable in ram, otherwise bash tries to escape it."},{"line_number":2249,"context_line":"            external_gw_v4\u003d$(cat $gw_info |tr \\\u0027 \\\" |sed s/True/true/ | jq \".external_fixed_ips[0].ip_address\" |sed s/\\\"//g )"},{"line_number":2250,"context_line":"            rm $gw_info"},{"line_number":2251,"context_line":"            sudo ip addr"},{"line_number":2252,"context_line":"            sudo ip link set dev br-ex up || true"}],"source_content_type":"application/x-shellscript","patch_set":46,"id":"04fc581f_80b86029","line":2249,"range":{"start_line":2249,"start_character":0,"end_line":2249,"end_character":3},"updated":"2023-08-22 09:00:48.000000000","message":"We can use json format and drop the gw_info tempfile + sed|tr stuff.\n\n  external_gw_v4\u003d$(openstack router show $Q_ROUTER_NAME -c external_gateway_info -f json | jq -r .external_gateway_info.external_fixed_ips[0].ip_address)","commit_id":"fed27acce8b49b8a17ca16c34a2039d97f8b886e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4edf55dce69b9b3fd764828506737c7db454b8f9","unresolved":false,"context_lines":[{"line_number":2246,"context_line":"            # Need to handle the json dict we get from the API (yeah, wut?!)"},{"line_number":2247,"context_line":"            # and transform that so jq can do the needful. We also can\u0027t store"},{"line_number":2248,"context_line":"            # it as a variable in ram, otherwise bash tries to escape it."},{"line_number":2249,"context_line":"            external_gw_v4\u003d$(cat $gw_info |tr \\\u0027 \\\" |sed s/True/true/ | jq \".external_fixed_ips[0].ip_address\" |sed s/\\\"//g )"},{"line_number":2250,"context_line":"            rm $gw_info"},{"line_number":2251,"context_line":"            sudo ip addr"},{"line_number":2252,"context_line":"            sudo ip link set dev br-ex up || true"}],"source_content_type":"application/x-shellscript","patch_set":46,"id":"e0f8b671_ee3ccf3a","line":2249,"range":{"start_line":2249,"start_character":0,"end_line":2249,"end_character":3},"in_reply_to":"04fc581f_80b86029","updated":"2023-08-29 20:44:44.000000000","message":"Done","commit_id":"fed27acce8b49b8a17ca16c34a2039d97f8b886e"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"7bc08610c6d4a78b166607c5007adc8492460fd2","unresolved":true,"context_lines":[{"line_number":2257,"context_line":"            # This is the default space fallback for all neutron networking,"},{"line_number":2258,"context_line":"            # since this is only if we create a dedicated provisioning"},{"line_number":2259,"context_line":"            # network in the job config."},{"line_number":2260,"context_line":"            sudo ip route add 10.1.0.0/20 via $external_gw_v4"},{"line_number":2261,"context_line":"        fi"},{"line_number":2262,"context_line":"    fi"},{"line_number":2263,"context_line":"    # Here is a good place to restart tcpdump to begin capturing packets."}],"source_content_type":"application/x-shellscript","patch_set":46,"id":"26d66218_b32a2c73","line":2260,"range":{"start_line":2260,"start_character":30,"end_line":2260,"end_character":41},"updated":"2023-08-22 09:00:48.000000000","message":"We can use IPV4_ADDRS_SAFE_TO_USE variable here?","commit_id":"fed27acce8b49b8a17ca16c34a2039d97f8b886e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4edf55dce69b9b3fd764828506737c7db454b8f9","unresolved":false,"context_lines":[{"line_number":2257,"context_line":"            # This is the default space fallback for all neutron networking,"},{"line_number":2258,"context_line":"            # since this is only if we create a dedicated provisioning"},{"line_number":2259,"context_line":"            # network in the job config."},{"line_number":2260,"context_line":"            sudo ip route add 10.1.0.0/20 via $external_gw_v4"},{"line_number":2261,"context_line":"        fi"},{"line_number":2262,"context_line":"    fi"},{"line_number":2263,"context_line":"    # Here is a good place to restart tcpdump to begin capturing packets."}],"source_content_type":"application/x-shellscript","patch_set":46,"id":"82d867a6_4c594747","line":2260,"range":{"start_line":2260,"start_character":30,"end_line":2260,"end_character":41},"in_reply_to":"26d66218_b32a2c73","updated":"2023-08-29 20:44:44.000000000","message":"Done","commit_id":"fed27acce8b49b8a17ca16c34a2039d97f8b886e"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"b50305141d3b68ce2e8529824828aa8bbceffb5e","unresolved":true,"context_lines":[{"line_number":2240,"context_line":"            openstack router set --disable-snat --external-gateway public $Q_ROUTER_NAME"},{"line_number":2241,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME"},{"line_number":2242,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME -c external_gateway_info -f value"},{"line_number":2243,"context_line":"            gw_info\u003d$(mktemp)"},{"line_number":2244,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME -c external_gateway_info -f value \u003e $gw_info"},{"line_number":2245,"context_line":"            # Need to handle the json dict we get from the API (yeah, wut?!)"},{"line_number":2246,"context_line":"            # and transform that so jq can do the needful. We also can\u0027t store"},{"line_number":2247,"context_line":"            # it as a variable in ram, otherwise bash tries to escape it."}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"a69edef1_bcd3a3d6","line":2244,"range":{"start_line":2243,"start_character":12,"end_line":2244,"end_character":114},"updated":"2023-08-30 10:25:54.000000000","message":"Can remove these, gw_info tempfile is not used.","commit_id":"68df6039b6114b84c3a00b28a51c1c43c087f0bd"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":false,"context_lines":[{"line_number":2240,"context_line":"            openstack router set --disable-snat --external-gateway public $Q_ROUTER_NAME"},{"line_number":2241,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME"},{"line_number":2242,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME -c external_gateway_info -f value"},{"line_number":2243,"context_line":"            gw_info\u003d$(mktemp)"},{"line_number":2244,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME -c external_gateway_info -f value \u003e $gw_info"},{"line_number":2245,"context_line":"            # Need to handle the json dict we get from the API (yeah, wut?!)"},{"line_number":2246,"context_line":"            # and transform that so jq can do the needful. We also can\u0027t store"},{"line_number":2247,"context_line":"            # it as a variable in ram, otherwise bash tries to escape it."}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"9fe6e497_1366f022","line":2244,"range":{"start_line":2243,"start_character":12,"end_line":2244,"end_character":114},"in_reply_to":"a69edef1_bcd3a3d6","updated":"2023-09-05 17:41:47.000000000","message":"Done","commit_id":"68df6039b6114b84c3a00b28a51c1c43c087f0bd"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"b50305141d3b68ce2e8529824828aa8bbceffb5e","unresolved":true,"context_lines":[{"line_number":2255,"context_line":"            # Transform to json jq can parse, which includes replacing Python"},{"line_number":2256,"context_line":"            # boolean strings."},{"line_number":2257,"context_line":"            external_gw_v4\u003d$(openstack router show $Q_ROUTER_NAME -c external_gateway_info -f json | jq -r .external_gateway_info.external_fixed_ips[0].ip_address)"},{"line_number":2258,"context_line":"            rm $gw_info"},{"line_number":2259,"context_line":"            sudo ip addr"},{"line_number":2260,"context_line":"            sudo ip link set dev br-ex up || true"},{"line_number":2261,"context_line":"            # This route is only used *if* we actually provision with a"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"155b5fc5_fc4cd573","line":2258,"range":{"start_line":2258,"start_character":12,"end_line":2258,"end_character":23},"updated":"2023-08-30 10:25:54.000000000","message":"Can remove.","commit_id":"68df6039b6114b84c3a00b28a51c1c43c087f0bd"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":false,"context_lines":[{"line_number":2255,"context_line":"            # Transform to json jq can parse, which includes replacing Python"},{"line_number":2256,"context_line":"            # boolean strings."},{"line_number":2257,"context_line":"            external_gw_v4\u003d$(openstack router show $Q_ROUTER_NAME -c external_gateway_info -f json | jq -r .external_gateway_info.external_fixed_ips[0].ip_address)"},{"line_number":2258,"context_line":"            rm $gw_info"},{"line_number":2259,"context_line":"            sudo ip addr"},{"line_number":2260,"context_line":"            sudo ip link set dev br-ex up || true"},{"line_number":2261,"context_line":"            # This route is only used *if* we actually provision with a"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"46842d88_c130c076","line":2258,"range":{"start_line":2258,"start_character":12,"end_line":2258,"end_character":23},"in_reply_to":"155b5fc5_fc4cd573","updated":"2023-09-05 17:41:47.000000000","message":"Done","commit_id":"68df6039b6114b84c3a00b28a51c1c43c087f0bd"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"b50305141d3b68ce2e8529824828aa8bbceffb5e","unresolved":true,"context_lines":[{"line_number":2263,"context_line":"            # happen. i.e. job specific config."},{"line_number":2264,"context_line":"            mtu\u003d$(($PUBLIC_BRIDGE_MTU - 30))"},{"line_number":2265,"context_line":"            v4mss\u003d$(($PUBLIC_BRIDGE_MTU - 40 ))"},{"line_number":2266,"context_line":"            # NOTE(TheJulia): v6mss should be -60 once it is supported."},{"line_number":2267,"context_line":"            # NOTE(TheJulia): The route commands below set *and* lock a"},{"line_number":2268,"context_line":"            # default MTU (disable PMTU discovery) and sets an Maximum Segment"},{"line_number":2269,"context_line":"            # Size to advertise for packets to be sent along the path. Normally"},{"line_number":2270,"context_line":"            # it is derived from the outbound interface MTU, which is wrong in"},{"line_number":2271,"context_line":"            # this scenario."},{"line_number":2272,"context_line":"            sudo ip route add $IRONIC_PROVISION_SUBNET_PREFIX via $external_gw_v4 mtu lock $mtu advmss $v4mss"},{"line_number":2273,"context_line":"            # This is the default space fallback for all neutron networking,"},{"line_number":2274,"context_line":"            # since this is only if we create a dedicated provisioning"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"32d3a4c9_5b18d80a","line":2271,"range":{"start_line":2266,"start_character":12,"end_line":2271,"end_character":28},"updated":"2023-08-30 10:25:54.000000000","message":"Do we want to mention LP#2032817 here?","commit_id":"68df6039b6114b84c3a00b28a51c1c43c087f0bd"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":false,"context_lines":[{"line_number":2263,"context_line":"            # happen. i.e. job specific config."},{"line_number":2264,"context_line":"            mtu\u003d$(($PUBLIC_BRIDGE_MTU - 30))"},{"line_number":2265,"context_line":"            v4mss\u003d$(($PUBLIC_BRIDGE_MTU - 40 ))"},{"line_number":2266,"context_line":"            # NOTE(TheJulia): v6mss should be -60 once it is supported."},{"line_number":2267,"context_line":"            # NOTE(TheJulia): The route commands below set *and* lock a"},{"line_number":2268,"context_line":"            # default MTU (disable PMTU discovery) and sets an Maximum Segment"},{"line_number":2269,"context_line":"            # Size to advertise for packets to be sent along the path. Normally"},{"line_number":2270,"context_line":"            # it is derived from the outbound interface MTU, which is wrong in"},{"line_number":2271,"context_line":"            # this scenario."},{"line_number":2272,"context_line":"            sudo ip route add $IRONIC_PROVISION_SUBNET_PREFIX via $external_gw_v4 mtu lock $mtu advmss $v4mss"},{"line_number":2273,"context_line":"            # This is the default space fallback for all neutron networking,"},{"line_number":2274,"context_line":"            # since this is only if we create a dedicated provisioning"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"e9ae3743_710af783","line":2271,"range":{"start_line":2266,"start_character":12,"end_line":2271,"end_character":28},"in_reply_to":"32d3a4c9_5b18d80a","updated":"2023-09-05 17:41:47.000000000","message":"*shrugs*\n\nThere are two issues, one we force the interface MTU too low for our multinode test because of VXLAN overhead. We could special case the hell out of it though. And then there is the underlying issue with the state of MTU support. Even then, if MTU is handled early on in OVN, our size downgrades are self imposed due tot he limit enforced due to vxlan. Actually, come to think about it, we might have conflicting settings between the two.","commit_id":"68df6039b6114b84c3a00b28a51c1c43c087f0bd"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":2263,"context_line":"            # happen. i.e. job specific config."},{"line_number":2264,"context_line":"            mtu\u003d$(($PUBLIC_BRIDGE_MTU - 30))"},{"line_number":2265,"context_line":"            v4mss\u003d$(($PUBLIC_BRIDGE_MTU - 40 ))"},{"line_number":2266,"context_line":"            # NOTE(TheJulia): v6mss should be -60 once it is supported."},{"line_number":2267,"context_line":"            # NOTE(TheJulia): The route commands below set *and* lock a"},{"line_number":2268,"context_line":"            # default MTU (disable PMTU discovery) and sets an Maximum Segment"},{"line_number":2269,"context_line":"            # Size to advertise for packets to be sent along the path. Normally"},{"line_number":2270,"context_line":"            # it is derived from the outbound interface MTU, which is wrong in"},{"line_number":2271,"context_line":"            # this scenario."},{"line_number":2272,"context_line":"            sudo ip route add $IRONIC_PROVISION_SUBNET_PREFIX via $external_gw_v4 mtu lock $mtu advmss $v4mss"},{"line_number":2273,"context_line":"            # This is the default space fallback for all neutron networking,"},{"line_number":2274,"context_line":"            # since this is only if we create a dedicated provisioning"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"2e720719_d302e231","line":2271,"range":{"start_line":2266,"start_character":12,"end_line":2271,"end_character":28},"in_reply_to":"32d3a4c9_5b18d80a","updated":"2023-09-01 20:50:43.000000000","message":"++","commit_id":"68df6039b6114b84c3a00b28a51c1c43c087f0bd"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":2059,"context_line":""},{"line_number":2060,"context_line":"        # NOTE(TheJulia): Show the network data to ease troubleshooting,"},{"line_number":2061,"context_line":"        # Normally, this will be the private network for devstack."},{"line_number":2062,"context_line":"        openstack --os-cloud $OS_CLOUD network show $ironic_net_id"},{"line_number":2063,"context_line":"        tag_id\u003d$(openstack --os-cloud $OS_CLOUD network show $ironic_net_id -c \"provider:segmentation_id\" -f value)"},{"line_number":2064,"context_line":"    fi"},{"line_number":2065,"context_line":"    local ovs_tap\u003dovs-tap"}],"source_content_type":"application/x-shellscript","patch_set":56,"id":"9790bb60_29f4e50f","line":2062,"updated":"2023-09-08 08:28:30.000000000","message":"This command seems redundant, you may rather want to use die_if_not_set on tag_id","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":2059,"context_line":""},{"line_number":2060,"context_line":"        # NOTE(TheJulia): Show the network data to ease troubleshooting,"},{"line_number":2061,"context_line":"        # Normally, this will be the private network for devstack."},{"line_number":2062,"context_line":"        openstack --os-cloud $OS_CLOUD network show $ironic_net_id"},{"line_number":2063,"context_line":"        tag_id\u003d$(openstack --os-cloud $OS_CLOUD network show $ironic_net_id -c \"provider:segmentation_id\" -f value)"},{"line_number":2064,"context_line":"    fi"},{"line_number":2065,"context_line":"    local ovs_tap\u003dovs-tap"}],"source_content_type":"application/x-shellscript","patch_set":56,"id":"6a423461_c2b9e07d","line":2062,"in_reply_to":"9790bb60_29f4e50f","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":2238,"context_line":"            sudo ip route replace $replace_range via $r_net_gateway"},{"line_number":2239,"context_line":"        else"},{"line_number":2240,"context_line":"            openstack router set --disable-snat --external-gateway public $Q_ROUTER_NAME"},{"line_number":2241,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME"},{"line_number":2242,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME -c external_gateway_info -f value"},{"line_number":2243,"context_line":"            # Need to handle the json dict we get from the API (yeah, wut?!)"},{"line_number":2244,"context_line":"            # and transform that so jq can do the needful. We also can\u0027t store"},{"line_number":2245,"context_line":"            # it as a variable in ram, otherwise bash tries to escape it."}],"source_content_type":"application/x-shellscript","patch_set":56,"id":"cd5eefa3_ba2c640b","line":2242,"range":{"start_line":2241,"start_character":1,"end_line":2242,"end_character":103},"updated":"2023-09-08 08:28:30.000000000","message":"One of these feels redundant (or both)","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":2238,"context_line":"            sudo ip route replace $replace_range via $r_net_gateway"},{"line_number":2239,"context_line":"        else"},{"line_number":2240,"context_line":"            openstack router set --disable-snat --external-gateway public $Q_ROUTER_NAME"},{"line_number":2241,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME"},{"line_number":2242,"context_line":"            openstack --os-cloud $OS_CLOUD router show $Q_ROUTER_NAME -c external_gateway_info -f value"},{"line_number":2243,"context_line":"            # Need to handle the json dict we get from the API (yeah, wut?!)"},{"line_number":2244,"context_line":"            # and transform that so jq can do the needful. We also can\u0027t store"},{"line_number":2245,"context_line":"            # it as a variable in ram, otherwise bash tries to escape it."}],"source_content_type":"application/x-shellscript","patch_set":56,"id":"33806e52_94fb92b6","line":2242,"range":{"start_line":2241,"start_character":1,"end_line":2242,"end_character":103},"in_reply_to":"cd5eefa3_ba2c640b","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":2285,"context_line":"    # Here is a good place to restart tcpdump to begin capturing packets."},{"line_number":2286,"context_line":"    # See: https://docs.openstack.org/devstack/latest/debugging.html"},{"line_number":2287,"context_line":"    stop_tcpdump"},{"line_number":2288,"context_line":"    start_tcpdump"},{"line_number":2289,"context_line":""},{"line_number":2290,"context_line":"    if [[ \"$IRONIC_IP_VERSION\" \u003d\u003d \"6\" ]]; then"},{"line_number":2291,"context_line":"        # route us back through the neutron router!"}],"source_content_type":"application/x-shellscript","patch_set":56,"id":"931ccfb4_73ce5e32","line":2288,"updated":"2023-09-08 08:28:30.000000000","message":"Is it intended to keep this on?","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":2285,"context_line":"    # Here is a good place to restart tcpdump to begin capturing packets."},{"line_number":2286,"context_line":"    # See: https://docs.openstack.org/devstack/latest/debugging.html"},{"line_number":2287,"context_line":"    stop_tcpdump"},{"line_number":2288,"context_line":"    start_tcpdump"},{"line_number":2289,"context_line":""},{"line_number":2290,"context_line":"    if [[ \"$IRONIC_IP_VERSION\" \u003d\u003d \"6\" ]]; then"},{"line_number":2291,"context_line":"        # route us back through the neutron router!"}],"source_content_type":"application/x-shellscript","patch_set":56,"id":"1ae6e7e3_f5d34724","line":2288,"in_reply_to":"931ccfb4_73ce5e32","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"}],"devstack/plugin.sh":[{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"be3308f7f0cc1f038f7269f613beabc34f5c8ff4","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"            if [[ \"$IRONIC_BAREMETAL_BASIC_OPS\" \u003d\u003d \"True\" \u0026\u0026 \"$IRONIC_IS_HARDWARE\" \u003d\u003d \"False\" ]]; then"},{"line_number":39,"context_line":"                echo_summary \"Precreating bridge: $IRONIC_VM_NETWORK_BRIDGE\""},{"line_number":40,"context_line":"                if [[ \"$Q_BUILD_OVS_FROM_GIT\" \u003d\u003d \"True\" ]]; then"},{"line_number":41,"context_line":"                    if [[ \"$Q_AGENT\" \u003d\u003d \"ovn\" ]]; then"},{"line_number":42,"context_line":"                        echo_summary \"Setting up OVN...\""},{"line_number":43,"context_line":"                        init_ovn"},{"line_number":44,"context_line":"                        start_ovn"},{"line_number":45,"context_line":"                    fi"},{"line_number":46,"context_line":"                else"},{"line_number":47,"context_line":"                    # NOTE(TheJulia): We are likely doing this to ensure"},{"line_number":48,"context_line":"                    # OVS is running."},{"line_number":49,"context_line":"                    echo_summary \"Installing OVS to pre-create bridge\""},{"line_number":50,"context_line":"                    install_package openvswitch-switch"},{"line_number":51,"context_line":"                fi"},{"line_number":52,"context_line":"                sudo ovs-vsctl -- --may-exist add-br $IRONIC_VM_NETWORK_BRIDGE"},{"line_number":53,"context_line":"            fi"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":56,"id":"cbe9d669_071c8b92","line":51,"range":{"start_line":40,"start_character":16,"end_line":51,"end_character":18},"updated":"2023-09-06 18:42:09.000000000","message":"Does build OVS from git really matter here? \nI think on CentOS Stream 9 we don\u0027t need to build from source? And in the future that will not be the case for ubuntu either.\n\nAlso the \"install_package\" breaks on CentOS because openvswitch-switch package is not a thing. \n\nSuggestion, if you do another follow up ...\n\n```\n                if [[ $Q_AGENT \u003d\u003d \"ovn\" ]]; then\n                    echo_summary \"Setting up OVN...\"\n                    init_ovn\n                    start_ovn\n                elif [[ \"$Q_BUILD_OVS_FROM_GIT\" !\u003d \"True\" ]]; then\n                    if is_ubuntu; then\n                        install_package openvswitch-switch\n                    else\n                        enable_service openvswitch\n                        restart_service openvswitch\n                    fi\n                fi\n```","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"20f80364115cb4fb46b9e0923c87d795de9546d8","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"            if [[ \"$IRONIC_BAREMETAL_BASIC_OPS\" \u003d\u003d \"True\" \u0026\u0026 \"$IRONIC_IS_HARDWARE\" \u003d\u003d \"False\" ]]; then"},{"line_number":39,"context_line":"                echo_summary \"Precreating bridge: $IRONIC_VM_NETWORK_BRIDGE\""},{"line_number":40,"context_line":"                if [[ \"$Q_BUILD_OVS_FROM_GIT\" \u003d\u003d \"True\" ]]; then"},{"line_number":41,"context_line":"                    if [[ \"$Q_AGENT\" \u003d\u003d \"ovn\" ]]; then"},{"line_number":42,"context_line":"                        echo_summary \"Setting up OVN...\""},{"line_number":43,"context_line":"                        init_ovn"},{"line_number":44,"context_line":"                        start_ovn"},{"line_number":45,"context_line":"                    fi"},{"line_number":46,"context_line":"                else"},{"line_number":47,"context_line":"                    # NOTE(TheJulia): We are likely doing this to ensure"},{"line_number":48,"context_line":"                    # OVS is running."},{"line_number":49,"context_line":"                    echo_summary \"Installing OVS to pre-create bridge\""},{"line_number":50,"context_line":"                    install_package openvswitch-switch"},{"line_number":51,"context_line":"                fi"},{"line_number":52,"context_line":"                sudo ovs-vsctl -- --may-exist add-br $IRONIC_VM_NETWORK_BRIDGE"},{"line_number":53,"context_line":"            fi"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":56,"id":"b548b9cc_736bdd9e","line":51,"range":{"start_line":40,"start_character":16,"end_line":51,"end_character":18},"in_reply_to":"0114f026_88d15414","updated":"2023-10-06 15:00:31.000000000","message":"I think we\u0027ve resolved this discussion. As such, marking resolved.","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"            if [[ \"$IRONIC_BAREMETAL_BASIC_OPS\" \u003d\u003d \"True\" \u0026\u0026 \"$IRONIC_IS_HARDWARE\" \u003d\u003d \"False\" ]]; then"},{"line_number":39,"context_line":"                echo_summary \"Precreating bridge: $IRONIC_VM_NETWORK_BRIDGE\""},{"line_number":40,"context_line":"                if [[ \"$Q_BUILD_OVS_FROM_GIT\" \u003d\u003d \"True\" ]]; then"},{"line_number":41,"context_line":"                    if [[ \"$Q_AGENT\" \u003d\u003d \"ovn\" ]]; then"},{"line_number":42,"context_line":"                        echo_summary \"Setting up OVN...\""},{"line_number":43,"context_line":"                        init_ovn"},{"line_number":44,"context_line":"                        start_ovn"},{"line_number":45,"context_line":"                    fi"},{"line_number":46,"context_line":"                else"},{"line_number":47,"context_line":"                    # NOTE(TheJulia): We are likely doing this to ensure"},{"line_number":48,"context_line":"                    # OVS is running."},{"line_number":49,"context_line":"                    echo_summary \"Installing OVS to pre-create bridge\""},{"line_number":50,"context_line":"                    install_package openvswitch-switch"},{"line_number":51,"context_line":"                fi"},{"line_number":52,"context_line":"                sudo ovs-vsctl -- --may-exist add-br $IRONIC_VM_NETWORK_BRIDGE"},{"line_number":53,"context_line":"            fi"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":56,"id":"5bf42b15_ea16052a","line":51,"range":{"start_line":40,"start_character":16,"end_line":51,"end_character":18},"in_reply_to":"4ecdbeee_4f3a2d05","updated":"2023-09-08 08:28:30.000000000","message":"So, was the intention to update this patch to add is_ubuntu?","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"            if [[ \"$IRONIC_BAREMETAL_BASIC_OPS\" \u003d\u003d \"True\" \u0026\u0026 \"$IRONIC_IS_HARDWARE\" \u003d\u003d \"False\" ]]; then"},{"line_number":39,"context_line":"                echo_summary \"Precreating bridge: $IRONIC_VM_NETWORK_BRIDGE\""},{"line_number":40,"context_line":"                if [[ \"$Q_BUILD_OVS_FROM_GIT\" \u003d\u003d \"True\" ]]; then"},{"line_number":41,"context_line":"                    if [[ \"$Q_AGENT\" \u003d\u003d \"ovn\" ]]; then"},{"line_number":42,"context_line":"                        echo_summary \"Setting up OVN...\""},{"line_number":43,"context_line":"                        init_ovn"},{"line_number":44,"context_line":"                        start_ovn"},{"line_number":45,"context_line":"                    fi"},{"line_number":46,"context_line":"                else"},{"line_number":47,"context_line":"                    # NOTE(TheJulia): We are likely doing this to ensure"},{"line_number":48,"context_line":"                    # OVS is running."},{"line_number":49,"context_line":"                    echo_summary \"Installing OVS to pre-create bridge\""},{"line_number":50,"context_line":"                    install_package openvswitch-switch"},{"line_number":51,"context_line":"                fi"},{"line_number":52,"context_line":"                sudo ovs-vsctl -- --may-exist add-br $IRONIC_VM_NETWORK_BRIDGE"},{"line_number":53,"context_line":"            fi"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":56,"id":"0114f026_88d15414","line":51,"range":{"start_line":40,"start_character":16,"end_line":51,"end_character":18},"in_reply_to":"5bf42b15_ea16052a","updated":"2023-09-13 22:37:22.000000000","message":"No, because if we\u0027re building from source in general on any platform, we would need to handle configure/init of OVN. Ubuntu just so happens to be the case where we need to do so for now.\n\nAlso, since it seems like we would need to continue to do the same for ipv6 testing of OVN in the future, it doesn\u0027t seem like we would want to constrain it further...","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"fa25120e6fecce397a69502188dd382eb1063c88","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"            if [[ \"$IRONIC_BAREMETAL_BASIC_OPS\" \u003d\u003d \"True\" \u0026\u0026 \"$IRONIC_IS_HARDWARE\" \u003d\u003d \"False\" ]]; then"},{"line_number":39,"context_line":"                echo_summary \"Precreating bridge: $IRONIC_VM_NETWORK_BRIDGE\""},{"line_number":40,"context_line":"                if [[ \"$Q_BUILD_OVS_FROM_GIT\" \u003d\u003d \"True\" ]]; then"},{"line_number":41,"context_line":"                    if [[ \"$Q_AGENT\" \u003d\u003d \"ovn\" ]]; then"},{"line_number":42,"context_line":"                        echo_summary \"Setting up OVN...\""},{"line_number":43,"context_line":"                        init_ovn"},{"line_number":44,"context_line":"                        start_ovn"},{"line_number":45,"context_line":"                    fi"},{"line_number":46,"context_line":"                else"},{"line_number":47,"context_line":"                    # NOTE(TheJulia): We are likely doing this to ensure"},{"line_number":48,"context_line":"                    # OVS is running."},{"line_number":49,"context_line":"                    echo_summary \"Installing OVS to pre-create bridge\""},{"line_number":50,"context_line":"                    install_package openvswitch-switch"},{"line_number":51,"context_line":"                fi"},{"line_number":52,"context_line":"                sudo ovs-vsctl -- --may-exist add-br $IRONIC_VM_NETWORK_BRIDGE"},{"line_number":53,"context_line":"            fi"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":56,"id":"4ecdbeee_4f3a2d05","line":51,"range":{"start_line":40,"start_character":16,"end_line":51,"end_character":18},"in_reply_to":"c16dc654_1003d3fa","updated":"2023-09-06 21:00:32.000000000","message":"ack, guarding with is_ubuntu is likely to be enough.\n\nafict on rpm platforms openvswitch is installed via bindep.\nother than this openvswitch-switch package install and a few other packages that changed names since fedora was working + the xinetd thing I was able to get it working on Stream 9 - i.e I got my reference pcap of IPv6+uefi booting with dhcp-agent and ipxe. \\o/","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d264a4a330f813735720bbbc07deb669986d3e33","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"            if [[ \"$IRONIC_BAREMETAL_BASIC_OPS\" \u003d\u003d \"True\" \u0026\u0026 \"$IRONIC_IS_HARDWARE\" \u003d\u003d \"False\" ]]; then"},{"line_number":39,"context_line":"                echo_summary \"Precreating bridge: $IRONIC_VM_NETWORK_BRIDGE\""},{"line_number":40,"context_line":"                if [[ \"$Q_BUILD_OVS_FROM_GIT\" \u003d\u003d \"True\" ]]; then"},{"line_number":41,"context_line":"                    if [[ \"$Q_AGENT\" \u003d\u003d \"ovn\" ]]; then"},{"line_number":42,"context_line":"                        echo_summary \"Setting up OVN...\""},{"line_number":43,"context_line":"                        init_ovn"},{"line_number":44,"context_line":"                        start_ovn"},{"line_number":45,"context_line":"                    fi"},{"line_number":46,"context_line":"                else"},{"line_number":47,"context_line":"                    # NOTE(TheJulia): We are likely doing this to ensure"},{"line_number":48,"context_line":"                    # OVS is running."},{"line_number":49,"context_line":"                    echo_summary \"Installing OVS to pre-create bridge\""},{"line_number":50,"context_line":"                    install_package openvswitch-switch"},{"line_number":51,"context_line":"                fi"},{"line_number":52,"context_line":"                sudo ovs-vsctl -- --may-exist add-br $IRONIC_VM_NETWORK_BRIDGE"},{"line_number":53,"context_line":"            fi"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":56,"id":"c16dc654_1003d3fa","line":51,"range":{"start_line":40,"start_character":16,"end_line":51,"end_character":18},"in_reply_to":"cbe9d669_071c8b92","updated":"2023-09-06 20:17:54.000000000","message":"For centos, it would have never worked to begin with, since we don\u0027t really support running devstack on centos. Which reminds me, we can remove fedora support now.\n\nAnyhow, I guarded by is_ubuntu so that we can ensure we keep the same functionality as was there before.","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"}],"doc/source/admin/ovn-networking.rst":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Overview"},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"OVN is largely considered an evolution of OVS, as a superset of capabilities"},{"line_number":11,"context_line":"which also offloads and shifts some of the configuration of networking away"},{"line_number":12,"context_line":"from configuration files, and towards a service modeled at serving a more"},{"line_number":13,"context_line":"scalable software defined networking experience. However, as with all newer"}],"source_content_type":"text/x-rst","patch_set":54,"id":"8e11a0c0_736ef732","line":10,"updated":"2023-09-01 20:50:43.000000000","message":"Maybe a specific statement at the top about how OVS is still the recommended route for Ironic installations? It\u0027s implied in a lot of places, but it might be valuable to say it explicitly.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Overview"},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"OVN is largely considered an evolution of OVS, as a superset of capabilities"},{"line_number":11,"context_line":"which also offloads and shifts some of the configuration of networking away"},{"line_number":12,"context_line":"from configuration files, and towards a service modeled at serving a more"},{"line_number":13,"context_line":"scalable software defined networking experience. However, as with all newer"}],"source_content_type":"text/x-rst","patch_set":54,"id":"dbd8f4ad_b9ffd446","line":10,"in_reply_to":"8e11a0c0_736ef732","updated":"2023-09-05 17:41:47.000000000","message":"Likely *not for long*, I\u0027ll update the language to hedge things as much as possible.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"As of the 2023.2 Release of Ironic, The Ironic project *can* confirm that"},{"line_number":41,"context_line":"OVN\u0027s DHCP server does work for PXE and iPXE operations when using **IPv4**,"},{"line_number":42,"context_line":"OVS version **3.11**, and OVN version **23.06.0**."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Support for IPv6 is presently pending changes to Neutron, as IPv6 requires"},{"line_number":45,"context_line":"additional configuration options and a different pattern of behavior, and"}],"source_content_type":"text/x-rst","patch_set":54,"id":"8c3c36e9_0035e962","line":42,"updated":"2023-09-01 20:50:43.000000000","message":"Just curious if anyone has tested this in the real world on real hardware (assuming yes?)","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"542aa76fa232e1f447bff6b20aba7bc782e06225","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"As of the 2023.2 Release of Ironic, The Ironic project *can* confirm that"},{"line_number":41,"context_line":"OVN\u0027s DHCP server does work for PXE and iPXE operations when using **IPv4**,"},{"line_number":42,"context_line":"OVS version **3.11**, and OVN version **23.06.0**."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Support for IPv6 is presently pending changes to Neutron, as IPv6 requires"},{"line_number":45,"context_line":"additional configuration options and a different pattern of behavior, and"}],"source_content_type":"text/x-rst","patch_set":54,"id":"e4f6e7c6_1ad18b6d","line":42,"in_reply_to":"0bf483fe_ec7de626","updated":"2023-10-06 15:03:35.000000000","message":"I believe this is resolved at this point since this was a question.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"As of the 2023.2 Release of Ironic, The Ironic project *can* confirm that"},{"line_number":41,"context_line":"OVN\u0027s DHCP server does work for PXE and iPXE operations when using **IPv4**,"},{"line_number":42,"context_line":"OVS version **3.11**, and OVN version **23.06.0**."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Support for IPv6 is presently pending changes to Neutron, as IPv6 requires"},{"line_number":45,"context_line":"additional configuration options and a different pattern of behavior, and"}],"source_content_type":"text/x-rst","patch_set":54,"id":"cd84a56b_d72ca245","line":42,"in_reply_to":"8c3c36e9_0035e962","updated":"2023-09-05 17:41:47.000000000","message":"Not yet, this is the first step in getting along that path.\n\nFWIW, I\u0027ve had customers try this with physical hardware but lack of a DHCP server capability in thier OVN for their baremetal. Also, OVN\u0027s release of these versions are only a few months old.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"9e5dac29246dbaf1c5742c686db2cb9372b9f764","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"As of the 2023.2 Release of Ironic, The Ironic project *can* confirm that"},{"line_number":41,"context_line":"OVN\u0027s DHCP server does work for PXE and iPXE operations when using **IPv4**,"},{"line_number":42,"context_line":"OVS version **3.11**, and OVN version **23.06.0**."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Support for IPv6 is presently pending changes to Neutron, as IPv6 requires"},{"line_number":45,"context_line":"additional configuration options and a different pattern of behavior, and"}],"source_content_type":"text/x-rst","patch_set":54,"id":"0bf483fe_ec7de626","line":42,"in_reply_to":"a51905d4_12c3a3fa","updated":"2023-09-05 19:01:31.000000000","message":"Given e2dk is the basis and reference for UEFI firmware, if we can get it to work in CI, there is a strong likelihood it will just work with hardware.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ef73e23bb3471f9749397e9beb91841c85c74f95","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"As of the 2023.2 Release of Ironic, The Ironic project *can* confirm that"},{"line_number":41,"context_line":"OVN\u0027s DHCP server does work for PXE and iPXE operations when using **IPv4**,"},{"line_number":42,"context_line":"OVS version **3.11**, and OVN version **23.06.0**."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Support for IPv6 is presently pending changes to Neutron, as IPv6 requires"},{"line_number":45,"context_line":"additional configuration options and a different pattern of behavior, and"}],"source_content_type":"text/x-rst","patch_set":54,"id":"a51905d4_12c3a3fa","line":42,"in_reply_to":"cd84a56b_d72ca245","updated":"2023-09-05 18:50:54.000000000","message":"It just makes me nervous that we confirm it can work when we mean (in CI) :)","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":46,"context_line":"thus has not been tested. Your advised to continue to us the"},{"line_number":47,"context_line":"``neutron-dhcp-agent`` if you need IPv6 at this time. Currently this support"},{"line_number":48,"context_line":"is being worked in Neutron"},{"line_number":49,"context_line":"`change 890683 \u003chttps://review.opendev.org/c/openstack/neutron/+/890683\u003e`_."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Maxmium Transmission Units"},{"line_number":52,"context_line":"--------------------------"}],"source_content_type":"text/x-rst","patch_set":54,"id":"f7ed4e6e_ff4345c0","line":49,"updated":"2023-09-01 20:50:43.000000000","message":"Might be better to link to a bug or spec if one exists?","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":false,"context_lines":[{"line_number":46,"context_line":"thus has not been tested. Your advised to continue to us the"},{"line_number":47,"context_line":"``neutron-dhcp-agent`` if you need IPv6 at this time. Currently this support"},{"line_number":48,"context_line":"is being worked in Neutron"},{"line_number":49,"context_line":"`change 890683 \u003chttps://review.opendev.org/c/openstack/neutron/+/890683\u003e`_."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Maxmium Transmission Units"},{"line_number":52,"context_line":"--------------------------"}],"source_content_type":"text/x-rst","patch_set":54,"id":"a5d2684c_216f830b","line":49,"in_reply_to":"f7ed4e6e_ff4345c0","updated":"2023-09-05 17:41:47.000000000","message":"Adding link to bug. Looks like Neutron is going to block this until the next release.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":1,"context_line":".. _ovn_networking:"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":"Use of OVN Networking"}],"source_content_type":"text/x-rst","patch_set":56,"id":"9befb6dd_17e1c663","line":1,"updated":"2023-09-08 08:28:30.000000000","message":"nit: not needed (you can always use :doc:`/admin/ovn-networking` instead)","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":1,"context_line":".. _ovn_networking:"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":"Use of OVN Networking"}],"source_content_type":"text/x-rst","patch_set":56,"id":"9cfa4301_316df225","line":1,"in_reply_to":"9befb6dd_17e1c663","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Overview"},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"OVN is largely considered an evolution of OVS. While the recommended that"},{"line_number":11,"context_line":"operators continue to utilize OVS with Ironic. OVN has an attractive a"},{"line_number":12,"context_line":"superset of capabilities and shifts some of the configuration of networking"},{"line_number":13,"context_line":"away from configuration files, and towards a service modeled at serving a more"}],"source_content_type":"text/x-rst","patch_set":56,"id":"d479f5aa_24a6ca64","line":10,"updated":"2023-09-08 08:28:30.000000000","message":"nit: \"it is recommended\"?","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Overview"},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"OVN is largely considered an evolution of OVS. While the recommended that"},{"line_number":11,"context_line":"operators continue to utilize OVS with Ironic. OVN has an attractive a"},{"line_number":12,"context_line":"superset of capabilities and shifts some of the configuration of networking"},{"line_number":13,"context_line":"away from configuration files, and towards a service modeled at serving a more"}],"source_content_type":"text/x-rst","patch_set":56,"id":"2d43388a_b7ed3030","line":10,"in_reply_to":"d479f5aa_24a6ca64","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"OVN is largely considered an evolution of OVS. While the recommended that"},{"line_number":11,"context_line":"operators continue to utilize OVS with Ironic. OVN has an attractive a"},{"line_number":12,"context_line":"superset of capabilities and shifts some of the configuration of networking"},{"line_number":13,"context_line":"away from configuration files, and towards a service modeled at serving a more"},{"line_number":14,"context_line":"scalable software defined networking experience. However, as with all newer"}],"source_content_type":"text/x-rst","patch_set":56,"id":"e9822ac8_a80f0694","line":11,"updated":"2023-09-08 08:28:30.000000000","message":"nit: the sentence feels incomplete. Replace the doc with a comma?","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"OVN is largely considered an evolution of OVS. While the recommended that"},{"line_number":11,"context_line":"operators continue to utilize OVS with Ironic. OVN has an attractive a"},{"line_number":12,"context_line":"superset of capabilities and shifts some of the configuration of networking"},{"line_number":13,"context_line":"away from configuration files, and towards a service modeled at serving a more"},{"line_number":14,"context_line":"scalable software defined networking experience. However, as with all newer"}],"source_content_type":"text/x-rst","patch_set":56,"id":"50c8571c_06587a53","line":11,"in_reply_to":"e9822ac8_a80f0694","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":53,"context_line":"Maxmium Transmission Units"},{"line_number":54,"context_line":"--------------------------"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"OVN presently lacks handling to fragment packets. We think..."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"But is it the Kernel networking and a lack of awareness?"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Truthfully we\u0027re not sure. In setting up CI jobs upstream, we discovered"},{"line_number":61,"context_line":"we had to be mindful of the virutal machine MTU, and we discovered that"},{"line_number":62,"context_line":"the segments might get chunked into sizes larger than what the was the"},{"line_number":63,"context_line":"configured MTU for the instance, resulting in the MTU being dropped."}],"source_content_type":"text/x-rst","patch_set":56,"id":"4b025ff1_60816786","line":60,"range":{"start_line":56,"start_character":0,"end_line":60,"end_character":26},"updated":"2023-09-08 08:28:30.000000000","message":"I\u0027d reword this part to be a bit less a stream-of-thoughts. What should the operators do about it? Maybe just drop it and lead with \"In setting up CI jobs...\"","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":53,"context_line":"Maxmium Transmission Units"},{"line_number":54,"context_line":"--------------------------"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"OVN presently lacks handling to fragment packets. We think..."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"But is it the Kernel networking and a lack of awareness?"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Truthfully we\u0027re not sure. In setting up CI jobs upstream, we discovered"},{"line_number":61,"context_line":"we had to be mindful of the virutal machine MTU, and we discovered that"},{"line_number":62,"context_line":"the segments might get chunked into sizes larger than what the was the"},{"line_number":63,"context_line":"configured MTU for the instance, resulting in the MTU being dropped."}],"source_content_type":"text/x-rst","patch_set":56,"id":"2818d127_a68dac37","line":60,"range":{"start_line":56,"start_character":0,"end_line":60,"end_character":26},"in_reply_to":"4b025ff1_60816786","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Because the NAT and Connection Tracking layer gets applied differently with"},{"line_number":82,"context_line":"OVN, as the router doesn\u0027t appear as a namespace or to the local OS kernel,"},{"line_number":83,"context_line":"you will will not be able to enable NAT translation for Bare Metal Networks"},{"line_number":84,"context_line":"under the direct management of OVN, that is if you don\u0027t have a separate"},{"line_number":85,"context_line":"TFTP service running from with-in that network."},{"line_number":86,"context_line":""}],"source_content_type":"text/x-rst","patch_set":56,"id":"a755fb71_ab2a2382","line":83,"updated":"2023-09-08 08:28:30.000000000","message":"nit: double \"will\"","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Because the NAT and Connection Tracking layer gets applied differently with"},{"line_number":82,"context_line":"OVN, as the router doesn\u0027t appear as a namespace or to the local OS kernel,"},{"line_number":83,"context_line":"you will will not be able to enable NAT translation for Bare Metal Networks"},{"line_number":84,"context_line":"under the direct management of OVN, that is if you don\u0027t have a separate"},{"line_number":85,"context_line":"TFTP service running from with-in that network."},{"line_number":86,"context_line":""}],"source_content_type":"text/x-rst","patch_set":56,"id":"b73aeccb_875e382a","line":83,"in_reply_to":"a755fb71_ab2a2382","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"This is a result of the kernel being unable to associate and handle return"},{"line_number":88,"context_line":"packet directly as part of the connection tracking layer. No direct work"},{"line_number":89,"context_line":"around for this is known, but generally Ironic encourages the use of Virtual"},{"line_number":90,"context_line":"Media where possible to sidestep this sort of issue and ensure a higher"},{"line_number":91,"context_line":"operational security posture for the deployment."},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":56,"id":"d27b7c9f_a1bc63f7","line":89,"updated":"2023-09-08 08:28:30.000000000","message":"nit: turn virtual media into a link to redfish","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"This is a result of the kernel being unable to associate and handle return"},{"line_number":88,"context_line":"packet directly as part of the connection tracking layer. No direct work"},{"line_number":89,"context_line":"around for this is known, but generally Ironic encourages the use of Virtual"},{"line_number":90,"context_line":"Media where possible to sidestep this sort of issue and ensure a higher"},{"line_number":91,"context_line":"operational security posture for the deployment."},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":56,"id":"c4530088_e2f64d7d","line":89,"in_reply_to":"d27b7c9f_a1bc63f7","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":111,"context_line":"incorrect based upon the DHCP interaction, and results in a belief"},{"line_number":112,"context_line":"that the TFTP server is locally attached."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"Example:"},{"line_number":115,"context_line":"Client: 10.1.0.13/28"},{"line_number":116,"context_line":"DHCP Server \u0026 Router: 10.1.0.1"},{"line_number":117,"context_line":"TFTP Server: 10.203.101.230"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Result:"},{"line_number":120,"context_line":"Grub2 believes it\u0027s default route is 10.0.0.0/8."}],"source_content_type":"text/x-rst","patch_set":56,"id":"53335a3a_ee133957","line":117,"range":{"start_line":114,"start_character":0,"end_line":117,"end_character":27},"updated":"2023-09-08 08:28:30.000000000","message":"This probably does not render the way you expect","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":111,"context_line":"incorrect based upon the DHCP interaction, and results in a belief"},{"line_number":112,"context_line":"that the TFTP server is locally attached."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"Example:"},{"line_number":115,"context_line":"Client: 10.1.0.13/28"},{"line_number":116,"context_line":"DHCP Server \u0026 Router: 10.1.0.1"},{"line_number":117,"context_line":"TFTP Server: 10.203.101.230"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Result:"},{"line_number":120,"context_line":"Grub2 believes it\u0027s default route is 10.0.0.0/8."}],"source_content_type":"text/x-rst","patch_set":56,"id":"d2bc9b7b_7fa3ba53","line":117,"range":{"start_line":114,"start_character":0,"end_line":117,"end_character":27},"in_reply_to":"53335a3a_ee133957","updated":"2023-09-13 22:37:22.000000000","message":"Done, rewrote it to be prose.","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"The networking node chassis must be configured to operate as a gateway."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"This can be confugred manually, but *should* (as far as Ironic is aware) be"},{"line_number":140,"context_line":"configured by Neutron and set on interfaces matching the bridge mappings. At"},{"line_number":141,"context_line":"least, it works that way in Devstack."},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":56,"id":"f197b57b_c7f8b226","line":139,"updated":"2023-09-08 08:28:30.000000000","message":"nit: configured","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"The networking node chassis must be configured to operate as a gateway."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"This can be confugred manually, but *should* (as far as Ironic is aware) be"},{"line_number":140,"context_line":"configured by Neutron and set on interfaces matching the bridge mappings. At"},{"line_number":141,"context_line":"least, it works that way in Devstack."},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":56,"id":"c2551823_7457dea9","line":139,"in_reply_to":"f197b57b_c7f8b226","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"25b36e1f506e9cd1a4eca12b372f5e322189471f","unresolved":true,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"OVN\u0027s handling of MTUs has been identified by OVN as being incomplete."},{"line_number":55,"context_line":"The reality is that it assumes the MTU is not further constained beyond"},{"line_number":56,"context_line":"the gateway, which sort of works in some caess for virtual machines, but"},{"line_number":57,"context_line":"might not be applicable with baremetal because your traffic may pass"},{"line_number":58,"context_line":"through lower, or higher MTUs."},{"line_number":59,"context_line":""}],"source_content_type":"text/x-rst","patch_set":60,"id":"8b4fdbc1_d0b5d4be","line":56,"updated":"2023-10-09 15:23:04.000000000","message":"nit: s/caess/cases/","commit_id":"3f77091c6320e196c83ecd10b9a8efd9d437e9f2"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"25b36e1f506e9cd1a4eca12b372f5e322189471f","unresolved":true,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Ideally, your environment should have consistent MTUs. If you cannot have"},{"line_number":61,"context_line":"consistent MTUs, we recommend clamping the MTU and Maximum Segment Size"},{"line_number":62,"context_line":"(MSS) using your front end router to ensure igress traffic is sized and"},{"line_number":63,"context_line":"fragmented appropriately. Egress traffic should inherent it\u0027s MTU size"},{"line_number":64,"context_line":"based upon the DHCP service configuration."},{"line_number":65,"context_line":""}],"source_content_type":"text/x-rst","patch_set":60,"id":"ed27e858_7ab6b368","line":62,"updated":"2023-10-09 15:23:04.000000000","message":"s/igress/ingress/","commit_id":"3f77091c6320e196c83ecd10b9a8efd9d437e9f2"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"25b36e1f506e9cd1a4eca12b372f5e322189471f","unresolved":true,"context_lines":[{"line_number":112,"context_line":"that the TFTP server is locally attached."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"For example, if a client is assigned ``10.1.0.13/28``, with a default"},{"line_number":115,"context_line":"gateway of ``10.1.0.1``, and a tftp-sever of ``10.203.101.230``,"},{"line_number":116,"context_line":"then grub2 believes it\u0027s default route is 10.0.0.0/8."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"This is being tracked as `bug 2033430 \u003chttps://bugs.launchpad.net/ironic/+bug/2033430\u003e`_"}],"source_content_type":"text/x-rst","patch_set":60,"id":"bebb22f9_3b55a278","line":115,"updated":"2023-10-09 15:23:04.000000000","message":"s/sever/server/","commit_id":"3f77091c6320e196c83ecd10b9a8efd9d437e9f2"}],"doc/source/install/configure-networking.rst":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"358bb5c4721d8a5f1646deca2ed008b2d2e83fa5","unresolved":true,"context_lines":[{"line_number":15,"context_line":"             is considered an advanced topic, and we encourage operators"},{"line_number":16,"context_line":"             interested in use of OVN to fully undestand it\u0027s capabilities"},{"line_number":17,"context_line":"             and state before attempting to utilize such a configuration."},{"line_number":18,"context_line":"             Pleaes see `OVN Networking \u003c../admin/ovn-networking\u003e`_ for"},{"line_number":19,"context_line":"             more details."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"It is recommended to use the baremetal ML2 mechanism driver and L2 agent for"}],"source_content_type":"text/x-rst","patch_set":56,"id":"909db70e_ed7057bb","line":18,"updated":"2023-09-08 08:28:30.000000000","message":"nit: Please\n\nnon-nit: :doc:`/admin/ovn-networking` (otherwise, you\u0027re missing .html)","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":false,"context_lines":[{"line_number":15,"context_line":"             is considered an advanced topic, and we encourage operators"},{"line_number":16,"context_line":"             interested in use of OVN to fully undestand it\u0027s capabilities"},{"line_number":17,"context_line":"             and state before attempting to utilize such a configuration."},{"line_number":18,"context_line":"             Pleaes see `OVN Networking \u003c../admin/ovn-networking\u003e`_ for"},{"line_number":19,"context_line":"             more details."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"It is recommended to use the baremetal ML2 mechanism driver and L2 agent for"}],"source_content_type":"text/x-rst","patch_set":56,"id":"9b6f69c9_cc7b1734","line":18,"in_reply_to":"909db70e_ed7057bb","updated":"2023-09-13 22:37:22.000000000","message":"Done","commit_id":"4744bac9221dc355adebacc2a372a550bcb6031a"}],"releasenotes/notes/ovn-support-6666dfa2e99e7ad4.yaml":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    `ovn documentation \u003chttps://docs.openstack.org/ironic/latest/admin/ovn-networking.html\u003e`_"},{"line_number":10,"context_line":"    as a result to help provide as much Ironic operator clarity as possible."},{"line_number":11,"context_line":"issues:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    Use of OVN may require disabling SNAT for provisioning with IPv4 when"},{"line_number":14,"context_line":"    using TFTP. This is due to the Linux Kernel, and how IP packet handling"},{"line_number":15,"context_line":"    occurs with OVN. No solution is known to this issue, and use of"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"7b1edf25_f0c44ca4","line":12,"updated":"2023-09-01 20:50:43.000000000","message":"I can do this during editing; but I suggest we make a single large release note about OVN support. I don\u0027t think it\u0027ll be helpful to operators to see these limitations as separate items in our notes.\n\nWhat do you think?","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    `ovn documentation \u003chttps://docs.openstack.org/ironic/latest/admin/ovn-networking.html\u003e`_"},{"line_number":10,"context_line":"    as a result to help provide as much Ironic operator clarity as possible."},{"line_number":11,"context_line":"issues:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    Use of OVN may require disabling SNAT for provisioning with IPv4 when"},{"line_number":14,"context_line":"    using TFTP. This is due to the Linux Kernel, and how IP packet handling"},{"line_number":15,"context_line":"    occurs with OVN. No solution is known to this issue, and use of"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"e122f75f_e5178168","line":12,"in_reply_to":"7b1edf25_f0c44ca4","updated":"2023-09-05 17:41:47.000000000","message":"I\u0027m not sure cobbling them together would grok well, given there are three distinct issues. It is a lot, and by combining I suspect people\u0027s eyes would glaze over.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"14e5bf5f880a4fa900d5c80bc97e621c4b633ee1","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    `ovn documentation \u003chttps://docs.openstack.org/ironic/latest/admin/ovn-networking.html\u003e`_"},{"line_number":10,"context_line":"    as a result to help provide as much Ironic operator clarity as possible."},{"line_number":11,"context_line":"issues:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    Use of OVN may require disabling SNAT for provisioning with IPv4 when"},{"line_number":14,"context_line":"    using TFTP. This is due to the Linux Kernel, and how IP packet handling"},{"line_number":15,"context_line":"    occurs with OVN. No solution is known to this issue, and use of"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"8935b4be_ab53d9bd","line":12,"in_reply_to":"e122f75f_e5178168","updated":"2023-10-06 15:03:06.000000000","message":"Since we are past release, This likely no longer matters as much. Each issue is distinct as well.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"}],"zuul.d/ironic-jobs.yaml":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":30,"context_line":"      tox_envlist: all"},{"line_number":31,"context_line":"      tempest_test_timeout: 2400"},{"line_number":32,"context_line":"      tempest_test_regex: ironic_tempest_plugin.tests.scenario"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"      tempest_concurrency: 1"},{"line_number":35,"context_line":"      devstack_localrc:"},{"line_number":36,"context_line":"        DEFAULT_INSTANCE_TYPE: baremetal"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"747b1f69_a42476f9","line":33,"updated":"2023-09-01 20:50:43.000000000","message":"unintentional?","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":false,"context_lines":[{"line_number":30,"context_line":"      tox_envlist: all"},{"line_number":31,"context_line":"      tempest_test_timeout: 2400"},{"line_number":32,"context_line":"      tempest_test_regex: ironic_tempest_plugin.tests.scenario"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"      tempest_concurrency: 1"},{"line_number":35,"context_line":"      devstack_localrc:"},{"line_number":36,"context_line":"        DEFAULT_INSTANCE_TYPE: baremetal"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"b672a2a6_fb815278","line":33,"in_reply_to":"747b1f69_a42476f9","updated":"2023-09-05 17:41:47.000000000","message":"yup, fixing!","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":366,"context_line":"        s-object: True"},{"line_number":367,"context_line":"        s-proxy: True"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":""},{"line_number":370,"context_line":"# NOTE(rpittau): the snmp job fails on ubuntu jammy so we pin it to"},{"line_number":371,"context_line":"# ubuntu focal for the time being."},{"line_number":372,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"2bd4c674_df164cb2","line":369,"updated":"2023-09-01 20:50:43.000000000","message":"accidental whitespace?","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":false,"context_lines":[{"line_number":366,"context_line":"        s-object: True"},{"line_number":367,"context_line":"        s-proxy: True"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":""},{"line_number":370,"context_line":"# NOTE(rpittau): the snmp job fails on ubuntu jammy so we pin it to"},{"line_number":371,"context_line":"# ubuntu focal for the time being."},{"line_number":372,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"5031bb5d_b700dd24","line":369,"in_reply_to":"2bd4c674_df164cb2","updated":"2023-09-05 17:41:47.000000000","message":"Done","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"89ff6eebe0d852c963196709f559343254ebcecd","unresolved":true,"context_lines":[{"line_number":392,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":393,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":394,"context_line":"        OVN_BRANCH: v23.06.0"},{"line_number":395,"context_line":"        OVS_BRANCH: v3.1.1"},{"line_number":396,"context_line":"      devstack_services:"},{"line_number":397,"context_line":"        q-agt: False"},{"line_number":398,"context_line":"        q-dhcp: False"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"b33c3fe2_76faffe5","line":395,"updated":"2023-09-01 20:50:43.000000000","message":"I\u0027m not sure if you want this PR to be merged yet, so sorry if this is early feedback... but do we have a path to not-version-locking this and/or building from git?","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fd87e5c2b205e27f9deec3ba4734c1884b7d5d90","unresolved":true,"context_lines":[{"line_number":392,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":393,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":394,"context_line":"        OVN_BRANCH: v23.06.0"},{"line_number":395,"context_line":"        OVS_BRANCH: v3.1.1"},{"line_number":396,"context_line":"      devstack_services:"},{"line_number":397,"context_line":"        q-agt: False"},{"line_number":398,"context_line":"        q-dhcp: False"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"5f0823c1_49a167c1","line":395,"in_reply_to":"07104c5b_b4ee4659","updated":"2023-09-13 22:37:22.000000000","message":"exactly!","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ef73e23bb3471f9749397e9beb91841c85c74f95","unresolved":true,"context_lines":[{"line_number":392,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":393,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":394,"context_line":"        OVN_BRANCH: v23.06.0"},{"line_number":395,"context_line":"        OVS_BRANCH: v3.1.1"},{"line_number":396,"context_line":"      devstack_services:"},{"line_number":397,"context_line":"        q-agt: False"},{"line_number":398,"context_line":"        q-dhcp: False"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"720aa726_a226490f","line":395,"in_reply_to":"29c4d22a_7ae9a7cd","updated":"2023-09-05 18:50:54.000000000","message":"This also reflects, to me, that we need to be MUCH less optimistic when we talk about how supported OVN is in the doc.\n\nWe work with OVN.\nOn IPv4 only.\nWith [list of exceptions]\nWith custom OVN\n\nThis may seem harsh (I don\u0027t want it to be), but I also think we have to be really careful about our operator-facing docs here. Right now, I\u0027m not sure it\u0027s possible for a non-contributing Ironic operator to run upstream Ironic w/OVN on bare metal. That means our docs should probably very clearly reflect that (or have detailed enough instructions to make it possible).","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"557cd854ba23184a2d14acb4a2735d990f587a50","unresolved":true,"context_lines":[{"line_number":392,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":393,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":394,"context_line":"        OVN_BRANCH: v23.06.0"},{"line_number":395,"context_line":"        OVS_BRANCH: v3.1.1"},{"line_number":396,"context_line":"      devstack_services:"},{"line_number":397,"context_line":"        q-agt: False"},{"line_number":398,"context_line":"        q-dhcp: False"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"07104c5b_b4ee4659","line":395,"in_reply_to":"2f895a05_431cb234","updated":"2023-09-05 19:14:17.000000000","message":"That nuance was decidedly *not* understood by me. That makes a lot more sense. So realistically (outside of CI job configuration), this is all about the DHCP server and less about the networking config (which is hellish in CI because of the missing OVN features + because CI is more complex).","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"9e5dac29246dbaf1c5742c686db2cb9372b9f764","unresolved":true,"context_lines":[{"line_number":392,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":393,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":394,"context_line":"        OVN_BRANCH: v23.06.0"},{"line_number":395,"context_line":"        OVS_BRANCH: v3.1.1"},{"line_number":396,"context_line":"      devstack_services:"},{"line_number":397,"context_line":"        q-agt: False"},{"line_number":398,"context_line":"        q-dhcp: False"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"2f895a05_431cb234","line":395,"in_reply_to":"720aa726_a226490f","updated":"2023-09-05 19:01:31.000000000","message":"So, it is harsh, and there is a huge giant nuance which I\u0027m unsure is understood here. People have been able to run Neutron + OVN + Neutron DHCP Agent for a while as a supported neutron configuration. The fundamental difference here is OVN\u0027s DHCP server instead of dnsmasq launched by neutron-dhcp-agent.\n\nAnd of course, that nuance is disjointed from any signaling which might occur from CI job configuration.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b42823ce1aca9b676829aa5f85fc5f90337ba0e8","unresolved":true,"context_lines":[{"line_number":392,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":393,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":394,"context_line":"        OVN_BRANCH: v23.06.0"},{"line_number":395,"context_line":"        OVS_BRANCH: v3.1.1"},{"line_number":396,"context_line":"      devstack_services:"},{"line_number":397,"context_line":"        q-agt: False"},{"line_number":398,"context_line":"        q-dhcp: False"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"29c4d22a_7ae9a7cd","line":395,"in_reply_to":"b33c3fe2_76faffe5","updated":"2023-09-05 17:41:47.000000000","message":"So a few things. This *does* built from GIT, but there is no \"latest\" option available for OVN. The versions are released in matching pairs for OVN and OVS.\n\nThe default is you build from git or you consume the host packages. Currently ubuntu ships out of date OVN, so we will need to build OVN for, for now.\n\nAdding notes, for now.","commit_id":"a65a12c7c50bb7d9bdf493608a01ea329f664ad0"}]}