| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [multi-node-setup] Adapt to br-infra linux bridge https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/995778 | 03:27 |
|---|---|---|
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [multi-node-setup] Adapt to br-infra linux bridge vxlan https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/995916 | 03:45 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check zuul jobs migration patch vxlan https://review.opendev.org/c/openstack/neutron/+/995917 | 03:48 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [multi-node-setup] Adapt to br-infra linux bridge vxlan https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/995916 | 04:31 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [multi-node-setup] Adapt to br-infra linux bridge https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/995778 | 06:24 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check zuul jobs migration patch https://review.opendev.org/c/openstack/neutron/+/995252 | 06:25 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 08:20 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 08:21 |
| opendevreview | Merged openstack/neutron stable/2026.1: segments: Catch RP not found in ``_delete_nova_inventory`` https://review.opendev.org/c/openstack/neutron/+/995681 | 09:04 |
| opendevreview | Merged openstack/neutron stable/2025.2: segments: Catch RP not found in ``_delete_nova_inventory`` https://review.opendev.org/c/openstack/neutron/+/995682 | 09:04 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check zuul jobs migration patch https://review.opendev.org/c/openstack/neutron/+/995252 | 09:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Fix HAChassisGroupRouterEvent setting options.chassis on physnet routers https://review.opendev.org/c/openstack/neutron/+/995930 | 09:48 |
| luisfdez | I am backporting https://bugs.launchpad.net/neutron/+bug/2069718 to Caracal, and I find problems on multiqueue VMs. The tap device pre-created seems to not be multiqueue and when libvirt tries to use it for the VM it fails with "Invalid argument". Looking at os-vif commit, I'm a bit confused, it seems to state that netlink cannot set a multiqueue device, but it goes through pyroute2 that uses netlink? | 09:48 |
| luisfdez | ~sean-k-mooney ~ralonsoh am I'm missing anything fundamental for those mechanics to work? | 09:49 |
| ralonsoh | The os-vif patch (https://review.opendev.org/c/openstack/os-vif/+/971231) supports multiqueue | 09:53 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 09:54 |
| ralonsoh | so the problem is that it is trying to create a multiqueue port but the VM port is not? | 09:55 |
| luisfdez | I ported that, and I'm using it. But the devices created vi a "add" in pyroute2 are not marked as IFF_MULTI_QUEUE . | 09:55 |
| luisfdez | even if multi_queue is passed. | 09:56 |
| luisfdez | In that patch the "test_add_tap_with_multiqueue" says in comments """Note: The IFF_MULTI_QUEUE flag (0x0100) cannot be set via netlink. It is only set when a process opens /dev/net/tun with the IFF_MULTI_QUEUE flag in the ioctl. Libvirt sets this when attaching "" | 09:56 |
| luisfdez | I don't know what to expect from the tap device reated by pyroute2... I would expect it to be MQ enabled, but it is not... | 09:57 |
| ralonsoh | add debug logs in `create_tap` method | 09:58 |
| ralonsoh | just to know what values are you passing to this method | 09:58 |
| luisfdez | I see this log to pyroute "pyroute2 command add, arguments {'ifname': 'tap5133488f-3a', 'kind': 'tuntap', 'mode': 'tap', 'multi_queue': True} " | 09:58 |
| ralonsoh | so what is the output of `ip tuntap show <tap>` | 10:01 |
| luisfdez | tap7b4f2d2d-c8: tap pi persist filter | 10:04 |
| luisfdez | If I'm not mistaken, the call can be tested with "python3 -c "from pyroute2 import IPRoute; ip=IPRoute(); ip.link('add', ifname='taptest-mq', kind='tuntap', mode='tap', multi_queue=True); ip.close()" | 10:08 |
| ralonsoh | yes, I'm actually doing that locally | 10:10 |
| ralonsoh | >>> with iproute.IPRoute() as ip: | 10:10 |
| ralonsoh | ... ip.link('add', mode='tap', multi_queue=True, ifname='tap2', kind='tuntap') | 10:10 |
| ralonsoh | But the output is not the expected: | 10:11 |
| ralonsoh | # ip tuntap show tap2 | 10:11 |
| ralonsoh | tap2: tap pi persist filter | 10:11 |
| ralonsoh | instead of: | 10:11 |
| ralonsoh | tap1: tap multi_queue persist (created with `ip l` commands) | 10:11 |
| luisfdez | yup, I was tempted of patching the code and just call "ip tuntap add.." with multiqueu... but I wanted to check with you, in case I was missing some detail that would make the think work as it is. | 10:12 |
| ralonsoh | ooook, this is the correct call: | 10:12 |
| ralonsoh | >>> with iproute.IPRoute() as ip: | 10:12 |
| ralonsoh | ... ip.link('add', mode='tap', ifname='tap3', kind='tuntap', ifr={'multi_queue': True}) | 10:12 |
| ralonsoh | I'll open a LP bug and propose a patch now | 10:12 |
| luisfdez | awesome, will give it a try | 10:13 |
| luisfdez | ralonsoh: | 10:16 |
| luisfdez | awesome, it works | 10:16 |
| luisfdez | thanks | 10:16 |
| ralonsoh | cool | 10:16 |
| *** ykarel_ is now known as ykarel | 10:16 | |
| opendevreview | Rodolfo Alonso proposed openstack/os-vif master: ovs: Pass ``multi_queue`` inside ``ifr`` dict for tuntap devices https://review.opendev.org/c/openstack/os-vif/+/995933 | 10:27 |
| opendevreview | Rodolfo Alonso proposed openstack/os-vif master: ovs: Pass ``multi_queue`` inside ``ifr`` dict for tuntap devices https://review.opendev.org/c/openstack/os-vif/+/995933 | 10:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Fix ChassisAgentTypeChangeEvent ignoring events attribute https://review.opendev.org/c/openstack/neutron/+/995936 | 11:18 |
| sean-k-mooney | ralonsoh: looks striatforward enough | 11:23 |
| sean-k-mooney | i really otught we had multi queue tempest test | 11:24 |
| sean-k-mooney | but i guess thos either are nto ran or didnt have the os-vif tap creatrion enabled | 11:24 |
| sean-k-mooney | ralonsoh: +2 | 11:28 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: evpn: Mirror zebra-learned neighbors as Static_MAC_Binding https://review.opendev.org/c/openstack/neutron/+/995938 | 11:42 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 11:45 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 11:49 |
| opendevreview | Merged openstack/neutron master: evpn: Remove child_vxlan_port config option https://review.opendev.org/c/openstack/neutron/+/994820 | 12:05 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: evpn: Mirror zebra-learned neighbors as Static_MAC_Binding https://review.opendev.org/c/openstack/neutron/+/995938 | 12:09 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 12:10 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add server-side filtering support to PVLAN https://review.opendev.org/c/openstack/neutron/+/995712 | 12:30 |
| ralonsoh | sean-k-mooney, thanks! | 12:37 |
| sean-k-mooney | do you knwo if neutron as any tempet test for multi queue we coudl enabel in os-vif or nova | 12:41 |
| sean-k-mooney | that woudl be a nice followup if so just to catch this type of regression | 12:41 |
| sean-k-mooney | if not we can alwasy write one | 12:41 |
| ralonsoh | no that I'm aware | 12:41 |
| sean-k-mooney | ack | 12:41 |
| sean-k-mooney | it woudl not be hard to create a multiqueue flavor boot a cirros vsm and ssh in and check how many queue are there, to check the tap however woudl need whithbox | 12:42 |
| sean-k-mooney | ah https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/api/compute/test_multiqueue.py | 12:42 |
| sean-k-mooney | so that check it form the libvirt side | 12:43 |
| sean-k-mooney | we can see about enabling that going forward | 12:43 |
| ralonsoh | we can implement something similar tempest or n-t-p | 12:43 |
| sean-k-mooney | ya so tempest can just make sure the vm can boot with mulit queue requested | 12:44 |
| sean-k-mooney | which i think woudl have been enough in this case | 12:44 |
| sean-k-mooney | this exeicting test woudl assert tha tthe libvirt domain is using it | 12:44 |
| sean-k-mooney | we coudl also extended it to check the tap | 12:44 |
| sean-k-mooney | to make sure its really enabeld end ot end but i guess this just was never run with the config option set | 12:45 |
| ralonsoh | just as a heads-up, no drivers meeting today | 13:01 |
| lajoskatona | ralonsoh: thanks | 13:01 |
| slaweq | thx, have a nice weekend | 13:02 |
| ralonsoh | you too! | 13:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Fix ChassisAgentTypeChangeEvent ignoring events attribute https://review.opendev.org/c/openstack/neutron/+/995936 | 13:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Remove ``BaseEvent`` and fix ``matches()`` bypassing ``base_match()`` https://review.opendev.org/c/openstack/neutron/+/995945 | 13:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Remove ``BaseEvent`` and fix ``matches()`` bypassing ``base_match()`` https://review.opendev.org/c/openstack/neutron/+/995945 | 13:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: conf: Add missing ``min``/``max`` bounds to integer config options https://review.opendev.org/c/openstack/neutron/+/995507 | 13:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: conf: Add missing ``min`` bounds to timeout and interval options https://review.opendev.org/c/openstack/neutron/+/995508 | 13:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: conf: Add missing ``min`` bounds to count and size options https://review.opendev.org/c/openstack/neutron/+/995509 | 13:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: conf: Add missing ``min`` bounds to HA conntrackd options https://review.opendev.org/c/openstack/neutron/+/995510 | 13:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: conf: Add missing ``min`` bounds to BGP VRF ID options https://review.opendev.org/c/openstack/neutron/+/995511 | 13:10 |
| ralonsoh | bcafarel, hi! if you have less than 1 min: https://review.opendev.org/q/topic:%22bug/2158358%22 | 13:11 |
| ralonsoh | thanks! | 13:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add agnostic WSGI support for API worker election and identification https://review.opendev.org/c/openstack/neutron/+/995666 | 13:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add mod_wsgi support to ``get_api_worker_count()`` https://review.opendev.org/c/openstack/neutron/+/995800 | 13:32 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: evpn: Mirror zebra-learned neighbors as Static_MAC_Binding https://review.opendev.org/c/openstack/neutron/+/995938 | 13:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make ``get_start_time()`` server-agnostic https://review.opendev.org/c/openstack/neutron/+/995802 | 13:42 |
| opendevreview | Eduardo Olivares proposed openstack/neutron-tempest-plugin master: Add EVPN scenario test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/994308 | 13:42 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 13:43 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 13:59 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 14:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Remove ``BaseEvent`` and fix ``matches()`` bypassing ``base_match()`` https://review.opendev.org/c/openstack/neutron/+/995945 | 14:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Fix ``UpdateLRouterCommand`` ignoring ``if_exists`` https://review.opendev.org/c/openstack/neutron/+/995955 | 14:24 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add OVSDB IDL operations for Load_Balancer_Group https://review.opendev.org/c/openstack/ovn-octavia-provider/+/995959 | 14:33 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add OVSDB IDL operations for Load_Balancer_Group https://review.opendev.org/c/openstack/ovn-octavia-provider/+/995963 | 14:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: OVN maintenance log ``start_message`` should be a string https://review.opendev.org/c/openstack/neutron/+/995965 | 14:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Inherit ``SchemaAwarePeriodicsBase`` from ``abc.ABC`` https://review.opendev.org/c/openstack/neutron/+/995967 | 14:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Filter provider networks at DB level in ``check_fdb_aging_settings`` https://review.opendev.org/c/openstack/neutron/+/995967 | 15:01 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: trivial fix: Fix typo on PVLAN internals image https://review.opendev.org/c/openstack/neutron/+/995973 | 15:15 |
| opendevreview | Merged openstack/neutron stable/2025.2: ovn: Fix HA_Chassis_Group race condition in port create/update https://review.opendev.org/c/openstack/neutron/+/995450 | 16:19 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: ``FRROCIImage``: try multiple base image mirrors https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/992456 | 16:20 |
| opendevreview | Merged openstack/neutron master: Make ``ovs_create_tap`` True by default https://review.opendev.org/c/openstack/neutron/+/994752 | 16:52 |
| opendevreview | Merged openstack/neutron stable/2025.1: ovn: Fix HA_Chassis_Group race condition in port create/update https://review.opendev.org/c/openstack/neutron/+/995451 | 16:53 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: WIP: Delete onboard_network_subnets https://review.opendev.org/c/openstack/python-neutronclient/+/995993 | 17:42 |
| opendevreview | Eduardo Olivares proposed openstack/neutron-tempest-plugin master: Add EVPN scenario test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/994308 | 21:42 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!