Wednesday, 2021-09-29

opendevreviewliuyulong proposed openstack/neutron master: Add table for pps limitaion  https://review.opendev.org/c/openstack/neutron/+/80421303:50
opendevreviewliuyulong proposed openstack/neutron master: Reuse the qos constants from neuron-lib  https://review.opendev.org/c/openstack/neutron/+/80438003:50
opendevreviewliuyulong proposed openstack/neutron master: Make flow installation trunk size automatic adjustment  https://review.opendev.org/c/openstack/neutron/+/76507203:57
opendevreviewZhai Mengdong proposed openstack/neutron master: [OVN] Implement router gateway IP QoS  https://review.opendev.org/c/openstack/neutron/+/74901206:37
opendevreviewRodolfo Alonso proposed openstack/neutron master: Execute the quota reservation removal in an isolated DB txn  https://review.opendev.org/c/openstack/neutron/+/80998307:05
opendevreviewRodolfo Alonso proposed openstack/neutron stable/xena: Execute the quota reservation removal in an isolated DB txn  https://review.opendev.org/c/openstack/neutron/+/81112407:06
opendevreviewRodolfo Alonso proposed openstack/neutron master: [DVR] Check if SNAT iptables manager is initialized  https://review.opendev.org/c/openstack/neutron/+/81131807:11
jpichi all, how to restore OVN database when restoring neutron from mysqldump? is there a command?07:15
ralonsohjpic, that should be done by the sync workers07:15
ralonsohthose should sanitize the OVN DB according to the Neutron DB07:16
hjensaselvira: Hi, I want to try to reproduce https://bugs.launchpad.net/neutron/+bug/1944948, do you happen to have a good devstack configursation to set up neutron with ovn?07:25
elvirahjensas: sure!07:36
opendevreviewLajos Katona proposed openstack/neutron-vpnaas stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/neutron-vpnaas/+/80979507:45
lajoskatonaslaweq, ralonsoh, obondarev: Hi, Could you please check these release related patches if You have few minutes: https://review.opendev.org/q/project:openstack/networking-odl+status:open+-age:2w ?07:47
ralonsohlajoskatona, sure07:48
slaweqlajoskatona: sure07:48
obondarevlajoskatona, will do07:49
slaweqall done :)07:49
ralonsoh(I was late!)07:49
obondarevsame for me :)07:50
slaweqlajoskatona: FYI, some time ago we added to our docs https://docs.openstack.org/neutron/latest/contributor/policies/code-reviews.html#neutron-code-review-practices some agreement that such mechanical patches like e.g. https://review.opendev.org/c/openstack/networking-odl/+/809087 can be merged with one +2 :)07:51
lajoskatonaslaweq: ok, cool07:51
opendevreviewMerged openstack/networking-odl master: Drop install_venv  https://review.opendev.org/c/openstack/networking-odl/+/80990808:12
opendevreviewMerged openstack/networking-odl master: Update master for stable/xena  https://review.opendev.org/c/openstack/networking-odl/+/80908608:12
opendevreviewMerged openstack/networking-odl master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/networking-odl/+/80908708:23
opendevreviewYi Yang proposed openstack/neutron master: Use Openflow to implement DVR L3  https://review.opendev.org/c/openstack/neutron/+/80593008:32
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Sanitize profile column of ml2_port_bindings table in the DB  https://review.opendev.org/c/openstack/neutron/+/81141109:01
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Enable QoS minimum packet rate rule for OVS backend  https://review.opendev.org/c/openstack/neutron/+/80539109:02
opendevreviewMerged openstack/neutron stable/ussuri: Replace cirros 0.4.0 by 0.5.2 in ovn migration create-resources.sh.j2  https://review.opendev.org/c/openstack/neutron/+/81134309:06
opendevreviewMerged openstack/neutron stable/victoria: Replace cirros 0.4.0 by 0.5.2 in ovn migration create-resources.sh.j2  https://review.opendev.org/c/openstack/neutron/+/81134209:06
opendevreviewMerged openstack/neutron stable/wallaby: Replace cirros 0.4.0 by 0.5.2 in ovn migration create-resources.sh.j2  https://review.opendev.org/c/openstack/neutron/+/81134109:06
opendevreviewMerged openstack/neutron stable/xena: Replace cirros 0.4.0 by 0.5.2 in ovn migration create-resources.sh.j2  https://review.opendev.org/c/openstack/neutron/+/81134009:06
sebaralonsoh, could you have another look at the networksegments db constraints? I fixed the tests and added a release note to it. https://review.opendev.org/c/openstack/neutron/+/78871409:11
ralonsohseba, ok09:12
opendevreviewHemanth N proposed openstack/neutron stable/wallaby: [DVR] Set arp entries only for single IPs given as allowed addr pair  https://review.opendev.org/c/openstack/neutron/+/81149709:14
opendevreviewHemanth N proposed openstack/neutron stable/victoria: [DVR] Set arp entries only for single IPs given as allowed addr pair  https://review.opendev.org/c/openstack/neutron/+/81149809:14
sebatnx :)09:14
opendevreviewHemanth N proposed openstack/neutron stable/ussuri: [DVR] Set arp entries only for single IPs given as allowed addr pair  https://review.opendev.org/c/openstack/neutron/+/81149909:14
ralonsohseba, we are now in Yoga09:23
ralonsohyou need to move the alembic migration from Xena to Yoga09:24
sebawill do09:24
opendevreviewSebastian Lohff proposed openstack/neutron master: Fix dynamic segment allocation race condition  https://review.opendev.org/c/openstack/neutron/+/78871409:30
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Sanitize profile column of ml2_port_bindings table in the DB  https://review.opendev.org/c/openstack/neutron/+/81141109:54
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Enable QoS minimum packet rate rule for OVS backend  https://review.opendev.org/c/openstack/neutron/+/80539109:54
hjensaselvira: Hi, see my comment on https://review.opendev.org/c/openstack/neutron/+/811435 and the bug.10:11
elvirasure, hjensas. Thanks10:14
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Add API extension for QoS minimum pps rule  https://review.opendev.org/c/openstack/neutron/+/80304510:28
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: ovs-agent: Report pkt processing info in heartbeat  https://review.opendev.org/c/openstack/neutron/+/80044410:28
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Report CUSTOM_VNIC_TYPE_ traits on Neutron agent RP  https://review.opendev.org/c/openstack/neutron/+/80044510:28
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Report pkt processing capacity on Neutron agent RP  https://review.opendev.org/c/openstack/neutron/+/80044610:28
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Add port-resource-request-groups extension  https://review.opendev.org/c/openstack/neutron/+/80563710:30
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Sanitize profile column of ml2_port_bindings table in the DB  https://review.opendev.org/c/openstack/neutron/+/81141110:30
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron master: Enable QoS minimum packet rate rule for OVS backend  https://review.opendev.org/c/openstack/neutron/+/80539110:30
opendevreviewSlawek Kaplonski proposed openstack/neutron-lib master: Add oneline_string validators  https://review.opendev.org/c/openstack/neutron-lib/+/81170810:35
opendevreviewHarald JensÃ¥s proposed openstack/neutron master: [OVN] Allow IP allocation with different segments for OVN service ports  https://review.opendev.org/c/openstack/neutron/+/81171911:57
gmannlajoskatona: ralonsoh slaweq can you verify the neutron extensions for Xena in case i missed any https://review.opendev.org/c/openstack/devstack/+/81148512:32
lajoskatonagmann: sure12:32
lajoskatonagmann: I checked the list, and some extension/feature from it now yet fully working, no tempest tests for them, but better to keep them out from this list12:41
gmannlajoskatona: I think keep it enable should not harm as their test can be added later in tempest or neutron tempest plugin and they start running on stable/xena too. like trunk case. 12:45
gmannlajoskatona: or you think those will be risky to run on stable as per stability or so?12:45
gmannif so then we can remove for now and then add when  we add test, that is also fine12:45
lajoskatonagmann: on xena the features are not working12:46
gmannlajoskatona: i see. then we should remove. please highlight those and I will remove. 12:46
opendevreviewBernard Cafarelli proposed openstack/neutron stable/xena: [stable/xena] Drop -master jobs  https://review.opendev.org/c/openstack/neutron/+/81118712:49
gmannlajoskatona: also about if we want to enable trunk extensions so does testing on stable or not. please comment what neutron team thinks https://review.opendev.org/c/openstack/grenade/+/81151212:51
gmanni am ok with both option12:51
lajoskatonagmann: I listed all which should not be in the xena list, and thanks for taking care of it12:55
gmannlajoskatona: great, thanks a lot.12:55
ralonsohgmann, sure, let me check 12:56
opendevreviewRodolfo Alonso proposed openstack/neutron master: [DVR] Check if SNAT iptables manager is initialized  https://review.opendev.org/c/openstack/neutron/+/81131813:53
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: req: Bump Mako to 1.0.7  https://review.opendev.org/c/openstack/neutron-vpnaas/+/81173114:12
opendevreviewMerged openstack/neutron stable/xena: Revert "update subport status when trunk/subport create/delete is triggerred"  https://review.opendev.org/c/openstack/neutron/+/81097514:14
opendevreviewMerged openstack/neutron master: Replace "Inspector.from_engine()" with "sqlalchemy.inspect()"  https://review.opendev.org/c/openstack/neutron/+/80810314:14
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: req: Bump Mako to 1.0.7  https://review.opendev.org/c/openstack/neutron-vpnaas/+/81173114:50
spatelralonsoh Hi14:52
ralonsohspatel, hi14:53
spatelralonsoh I have question related DPDK performance. I have setup DPDK on one of my compute node to replace SRIOV 14:55
spatelI am seeing very poor performance, like standard virtio  14:56
spatellet me show you some numbers and the way i am doing load-testing14:56
opendevreviewMerged openstack/networking-bgpvpn master: Update master for stable/xena  https://review.opendev.org/c/openstack/networking-bgpvpn/+/80905115:00
opendevreviewMerged openstack/networking-bgpvpn master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/networking-bgpvpn/+/80905215:08
spatelralonsoh sorry for delay, here is the result - https://paste.opendev.org/show/809675/15:14
spatelI can only pump 150kpps packets after that i am seeing packet loss start15:15
spatelI have configure vm with CPU pinning and numa, my flavor setting - hw:cpu_policy='dedicated', hw:cpu_sockets='2', hw:cpu_threads='2', hw:mem_page_size='large'15:17
spatelin internet people saying with dpdk you can hit 10mpps rate but how ? 15:17
spateli have little confusion related hugepages.. how does DPDK use them i believe i did correct setting for hugepage so could you validate it?15:20
opendevreviewMerged openstack/networking-sfc master: Update master for stable/xena  https://review.opendev.org/c/openstack/networking-sfc/+/80970215:25
opendevreviewMerged openstack/networking-sfc master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/networking-sfc/+/80970315:35
spatelralonsoh do you have any idea about that15:43
ralonsohspatel, to be honest, I can't tell you, checking those logs, what is failing there15:45
spatelnothing failing but just poor transfer rate.. 15:46
ralonsohyes I know15:46
spateldo you think my VM is bottleneck here?15:46
ralonsohdid you have the FW enabled?15:46
spatelyou default security group 15:46
spateli can check log of contrack on compute node and see15:47
ralonsohanother question is15:47
ralonsohhow are you bechmarking the traffic?15:47
ralonsohfrom a VM to another host, right?15:48
spatelYes - From another host to VM15:48
ralonsohso the packets should cross a physical interface15:48
spatelPhysical server -----------> DPDK vm15:48
ralonsohhow did you configured this physical interface?15:49
spatelits single 10G nic port configure for dpdk 15:49
ralonsohare you using VLAN? VXLAN?15:49
spatelVLAN 15:49
spatelhttps://paste.opendev.org/show/809676/15:49
ralonsohok so you have your physical bridge connected to br-int and the DPDK port15:50
spatelYes15:51
spatelYou can see here - https://paste.opendev.org/show/809675/15:51
ralonsohI don't know if the performance issue is in the DPDK TAP port or the DPDK NIC port15:51
ralonsohyou can also check VM-to-VM15:51
ralonsohinside the same host15:51
spatelbr-provider attach with dpdk-0 port and then br-provider ----patch----br-int15:51
ralonsohthat will put aside the physical NIC15:52
spatelhmm that is good idea VM-to-VM on same host15:52
ralonsohcheck the FW (disable it)15:52
spatelBut if i run same load-test on SRIOV vm then result is much better 15:52
ralonsohcheck the packet size15:52
spatelI am using 64 and 150 packet size 15:52
spatelthat is what we have in production workload for UDP traffic 15:53
ralonsohas I said, check the FW then15:53
spatelI can see dpdk created this file that means its using hugepage - /dev/hugepages/rtemap_015:53
ralonsohI don't know if DPDK is properly configured or working fine15:53
spatelI am looking at iptables and not seeing any issue like connection table full etc.. 15:54
opendevreviewMerged openstack/networking-bagpipe master: Update master for stable/xena  https://review.opendev.org/c/openstack/networking-bagpipe/+/80905915:54
opendevreviewMerged openstack/networking-bagpipe master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/networking-bagpipe/+/80906215:54
spateli can remove port-security and give it a try 15:54
ralonsohno no, not iptables15:54
spatel?15:54
ralonsohyou can't use hybrid firewall15:54
spatelwhat FW are you talking about?15:54
spatelI am running OVN 15:54
ralonsohah, I didn't know that15:55
ralonsohI thought it was OVS15:55
spatelOVN + OVS-DPDK15:55
spateldoes that matter ?15:56
ralonsohit shouldn't 15:56
ralonsohI didn't test OVN+DPDK15:56
spatelhow do you test your DPDK deployment for performance (lets forget about OVN)15:57
spateli would like to know what people prefer to banchmark 15:58
ralonsohI used an IXIA traffic generator, with the different RFC tests implemented15:58
spateland what do you use in VM (testpmd?)15:59
ralonsoha dpdk traffic reflector15:59
spatelhmm but in real world it doesn't work like that right? 16:00
ralonsohinside the VM you need to use a DPDK application16:01
ralonsohe.g.: TREX with DPDK16:01
spatelI am seeing kernel context switching going about 200k when i run simple load-test16:01
spatelif i am running haproxy in VM then how does it use dpdk ?16:01
ralonsohyou need to create hugepages inside the VM16:02
ralonsohand assign the VM port to DPDK16:02
spatelmy workload is all audio processing which use udp stream and need very high packet rate.. 16:03
ralonsohand create DPDK compatible app16:03
spateli am creating VM with this flavor setting, doesn't that enough? hw:mem_page_size='large'16:03
ralonsohpackets inside the VM will use kernel space16:04
spatelyes i have noticed that 16:04
ralonsohthen there you have your performance stopper16:05
ralonsohwith DPDK you need everything, including the traffic inside the VM, being handled by DPDK16:05
ralonsohto have best performance16:05
spatelhmm 16:05
ralonsohin any case, I'm not an expert at all on this subject16:05
spatelhow does hugepage play role in dpdk ? 16:06
spatelyou assign hugepage to dpdk but then how does VM use that to copy packet? 16:06
ralonsohno idea16:07
spatelsome document saying you need to create hugepage and mount it and the tell libvirtd to use it but then what is this setting for in flavor hw:mem_page_size='large'16:07
opendevreviewBalazs Gibizer proposed openstack/neutron master: Enable min pps tempest tests  https://review.opendev.org/c/openstack/neutron/+/81174616:08
spateli didn't do any setting in libvirtd for hugepage 16:08
ralonsohyou have info here https://docs.openstack.org/nova/pike/admin/huge-pages.html16:08
spatelI did that - mount -t hugetlbfs hugetlbfs /dev/hugepages 16:10
spateldefault dpdk use this path - /dev/hugepages16:10
spatelThat is how i can see this - https://paste.opendev.org/show/809677/16:11
spatelcurious why VM directory is empty - /dev/hugepages/libvirt/qemu/1-instance-00000045/16:12
spatelis that normal or something wrong there?16:12
ralonsohtbh, no idea, this is Nova stuff16:13
spatelno problem.. so much moving piece so hard to understand :)16:14
spatelralonsoh do you know nuetron support bonding for SRIOV VF inside VM?16:20
spatelif that work then i can go with sriov and bond my vf inside vm16:20
ralonsoh1) neutron knows nothing about the VM network, this must be done inside the VM16:21
ralonsoh2) bonding two VFs from the same PF will provide no speed gain (just in case)16:21
spatelI have plan to replace nic with dual port 16:25
spatelso i will have two PF 16:25
spatelcurrently i am running very large cloud with all sriov compute and because of sriov nature i don't have bonding :(16:26
spatelif one of my TOR switch pop out then whole rack go down.. 16:26
spatelThat is why i am exploring alternative to use DPDK so i will have dual redundancy 16:27
spatelDoes neutron support providing dual VF from two different PF then i can do some magic in VM to bond them and my all problem will get resolved 16:28
spatelralonsoh ^16:28
ralonsohnow I don't remember how but you can specify the NIC to be used in a Neutron port16:29
ralonsohbut you should ask this in #nova16:29
opendevreviewBalazs Gibizer proposed openstack/neutron master: Enable min pps tempest tests  https://review.opendev.org/c/openstack/neutron/+/81174616:30
spatelralonsoh sure will ask there 16:31
spateli found this article and look like little complicated - https://www.redpill-linpro.com/techblog/2021/01/30/bonding-sriov-nics-with-openstack.html16:34
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP][HA] Do not add initial state change delay in HA router  https://review.opendev.org/c/openstack/neutron/+/81175116:37
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: req: Bump some requirements  https://review.opendev.org/c/openstack/neutron-vpnaas/+/81173117:20
opendevreviewMerged openstack/neutron master: Replace "tenant_id" with "project_id" in Quota engine  https://review.opendev.org/c/openstack/neutron/+/80584917:27
opendevreviewRodolfo Alonso proposed openstack/neutron master: Execute the quota reservation removal in an isolated DB txn  https://review.opendev.org/c/openstack/neutron/+/80998317:33
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: req: Bump some requirements  https://review.opendev.org/c/openstack/neutron-vpnaas/+/81173119:51

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!