Tuesday, 2024-12-10

opendevreviewliuyulong proposed openstack/neutron-tempest-plugin master: Re-enable the router test_migration cases  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93631801:46
opendevreviewliuyulong proposed openstack/neutron master: Add basical functionalities for metadata path extension  https://review.opendev.org/c/openstack/neutron/+/88153501:57
opendevreviewliuyulong proposed openstack/neutron master: Add metadata path extension openflows  https://review.opendev.org/c/openstack/neutron/+/88809701:57
opendevreviewliuyulong proposed openstack/neutron master: Fullstack case for metadata path  https://review.opendev.org/c/openstack/neutron/+/88809801:57
opendevreviewliuyulong proposed openstack/neutron master: Add devstack plugin to enable ovs metadata_path  https://review.opendev.org/c/openstack/neutron/+/92858601:57
opendevreviewyatin proposed openstack/neutron master: [DNM] wsgi switch higher workers  https://review.opendev.org/c/openstack/neutron/+/93627208:15
opendevreviewMerged openstack/neutron-fwaas master: Drop extra space from nflog-prefix  https://review.opendev.org/c/openstack/neutron-fwaas/+/93736108:37
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Set always the GW LRP "gateway_mtu" option  https://review.opendev.org/c/openstack/neutron/+/93702608:37
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Optimization in the router port options generation  https://review.opendev.org/c/openstack/neutron/+/93702508:38
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Optimization in the router port options generation  https://review.opendev.org/c/openstack/neutron/+/93702508:39
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Set always the GW LRP "gateway_mtu" option  https://review.opendev.org/c/openstack/neutron/+/93702608:39
opendevreviewliuyulong proposed openstack/neutron-tempest-plugin master: Re-enable the router test_migration cases  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93631809:12
*** ralonsoh_ is now known as ralonsoh09:21
opendevreviewliuyulong proposed openstack/neutron-tempest-plugin master: Skip nested snat validation for DVR  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93744309:29
opendevreviewFernando Royo proposed openstack/ovsdbapp master: [DNM] Adding some logs to debug txn commit issue  https://review.opendev.org/c/openstack/ovsdbapp/+/93744509:42
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93695209:46
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93695209:48
opendevreviewKien Nguyen proposed openstack/neutron master: Use the correct input for OVN agent deletion  https://review.opendev.org/c/openstack/neutron/+/93698811:33
opendevreviewFernando Royo proposed openstack/ovsdbapp master: [DNM] Adding some logs to debug txn commit issue  https://review.opendev.org/c/openstack/ovsdbapp/+/93744511:39
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93695211:58
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93695211:58
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93695212:04
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with 4 workers  https://review.opendev.org/c/openstack/neutron/+/93642912:20
opendevreviewMerged x/whitebox-neutron-tempest-plugin master: Log packets captured by tcpdump on the nodes in case of test failure  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/93723713:24
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Dec 10 14:00:17 2024 UTC and is due to finish in 60 minutes.  The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'networking'14:00
haleybPing list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh14:00
mlavalle\o14:00
lajoskatonao/14:00
obondarevo/14:00
ralonsohhello14:00
ykarelo/14:00
s3rj1khi all14:00
frickler\o14:00
jlibosvao/14:00
haleybok, let's get started14:01
haleyb#announcements14:01
slaweqo/14:01
haleyb#link https://releases.openstack.org/epoxy/schedule.html14:01
haleybWe are currently in week R-1614:01
haleybEpoxy-2 milestone R-12 (week of Jan 6th)14:01
haleybReminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers14:02
haleybi see one item on the wiki for this week, unfortunately i might not be able to attend14:03
haleybit actually doesn't look like an RFE anyways14:03
haleybso maybe no meeting is required14:04
lajoskatona+1, no meeting is good meeting :-)14:04
opendevreviewDong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93745714:04
rubasovlate o/14:04
haleybi will ping him after meeting for status14:05
lajoskatonaespecially on Friday afternoon :-)14:05
haleybthe other thing i wanted to be aware of is the holiday break, a lot of people (me included) are taking time off the next few weeks14:06
* mlavalle will be off starting tomorrow14:06
haleybi was going to cancel the meeting for the 24th14:07
haleybbut with E-2 coming up shortly after, i'm not sure if we will have things to discuss on 12/3114:07
slaweqI won't be there 24th nor 31st :)14:08
haleyblike mlavalle, i will also be taking a break, but will be working a bit to make sure the wheels don't fall off14:08
lajoskatonaI also will be on PTO 14:09
haleybi will put my days off on the wiki, but essentially from the 18th to the 8th i will be off, completely unavailable the first week of January14:09
mlavallefirst time in 40 years that I take such a long break14:09
haleybmlavalle: you deserve it, and enjoy you're time off14:10
opendevreviewDong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93745714:10
lajoskatonamlavalle: enjoy it :-)14:10
haleybso should i also cancel the meeting on the 31st?14:11
slaweq++14:11
ykarel++14:11
mlavalle++14:11
ihrachysyes! some locations / culture consider NY eve as much a holiday as Christmas14:11
lajoskatona+114:11
haleybok, done. Meeting next week on the 17th, then next on Jan 7th14:12
haleybI will still be on the beach on the 7th, can someone volunteer to lead that one? I'm back on the 8th14:12
mlavalleyes I can14:13
haleybmlavalle: thanks. and that might be a busy week being E-2 with releases being proposed14:13
haleybi will send an email to the list regarding the meetings in case someone is not here14:14
haleyblast announcement i have is my weekly reminder14:15
haleybLet's continue to use the priorities dashboard for patches in the "ready to merge" state (weekly reminder)14:15
haleybany other announcements?14:16
haleyb#topic bugs14:16
haleybmlavalle was the deputy this week, his report is at14:17
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/3HIN2NODJM6S3BF3LKXQW6BMHSNQPKWP/14:17
haleybi see most have owners and/or patches14:17
haleybthis one does not14:18
haleyb#link https://bugs.launchpad.net/neutron/+bug/209121114:18
mlavalleyeah that one I marked as incomplete, but submitter responded, so it needs follow up14:19
haleybit's on yoga/zed14:19
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93695214:19
haleybfirst step would be to reproduce on master branch, not sure if anyone has cycles or a setup running ml2/ovs14:21
lajoskatonahaleyb: I can check, I try to spare some time for it14:21
haleyblajoskatona: ack thanks. this also could be a deployment issue, that was using ansible14:22
slaweqisn't it normal what is in this bug /14:22
slaweqwhen you run tcpdump your nic is in promiscous mode, right?14:23
slaweqor am I wrong?14:23
slaweqit would be bad if there would be packets from some other network14:23
slaweqbut from the same one I think it is kind of ok, no?14:23
lajoskatonapossible, I have to check14:24
ralonsohin OVS? TAP ports should receive VM traffic only14:24
haleybslaweq: it could be normal, all those things you describe seem valid, i guess with port security i would not have expected14:24
haleybit is a flat provider network14:25
slaweqbut port security is not filtering incoming packets based on the dst mac14:25
slaweqIIRC14:25
ralonsohwe do, we accept packets only for the destination port, mac and IP14:26
slaweqanyway, I don't have cycles to work on it but for me it looks like maybe it is not really a bug :)14:26
slaweq ralonsoh in each fw driver?14:26
ralonsohat least with native one14:26
haleybi will ask that question in the bug, perhaps this is with iptables hybrid? i don't want lajoskatona wasting cycles with wrong config14:27
mlavalle+114:27
lajoskatonahaleyb: ack, the fw driver can beimportant, I defaulted to ovs fw driver in my mind :-)14:28
haleyblajoskatona: i posted that question, and thanks for taking a look14:28
haleybany other bugs we should discuss?14:28
haleybihrachys is the bug deputy this week14:29
haleybi see lucasgomes as next week, but didn't you mention he's not working on openstack any more ralonsoh ? should i ping him about that so he doesn't do unnecessary work?14:30
ralonsohright14:30
ralonsohI can take his place14:31
haleybralonsoh: ok, thanks. i will ping him later about the next rotation which is way out in March14:32
haleybhopefully things are quiet those weeks anyways14:32
haleybok, moving on14:33
haleyb#topic community goals14:33
haleybi see there was a little movement on neutronclient deprecation14:33
haleyb#link https://review.opendev.org/q/topic:%22bug/1999774%2214:33
lajoskatonayes, some progress, I am lost in some of the tests in horizon for quotas, so in worst case I ask for help from the horizon team14:34
lajoskatona but slow progress anyway14:34
haleyblajoskatona: ack, at least one has a +214:34
haleyband i was surprised py39 is still being run in the other14:35
haleybdoh, although we run it too, my brain has not had coffeee yet14:36
haleybother community goal is eventlet14:37
haleyb#link https://bugs.launchpad.net/neutron/+bugs?field.tag=eventlet-deprecation14:37
ralonsohyes, I'm working in the ovn metadata migration14:37
ralonsohmore in particular in the metadta handler14:37
ralonsohright now we have 2 methods to create the handler: in a thread and spawning processes14:38
ralonsoh(metadata_workers=0 or other number)14:38
ralonsohall these methods use eventlet14:38
ralonsohso I'm going to start migrating only the default one (because metadata_workers=0 by default)14:39
ralonsohand that implies refactoring everything: the http handler, the embeded wsgi server and remove, probably, the haproxy14:39
ralonsoh5 minutes, more or less14:39
ralonsohthe question is: I'll start with this thread model14:39
haleybso no more haproxy?14:40
ralonsohinstead of migrating the other that spawns several process, that would imply creating something like the Neutron API wsgi server (or nova metadata)14:40
ralonsohhaleyb, we don't need it if we have one single embedded server reading from the port14:41
ralonsohso do you agree with this strategy?14:41
ralonsoh(maybe this deserves a mail)14:41
haleybralonsoh: i'm trying to think of other settings we put in the haproxy conf, and if they matter (i truly don't remember at the moment)14:42
haleyblike rate limiting, etc14:42
ralonsohyes, that's something to be considered14:42
ralonsohso maybe haproxy  can stay14:42
slaweqyes, rate limiting is important IMO14:42
slaweqbut with haproxy we have it only for IPv4 or IPv6, not both at the same time14:43
ralonsohand the could be also compatible with a future implementation using multiple processes14:43
ralonsohslaweq, we don't need both14:43
slaweqif we could have something else working for both IP versions would be great14:43
ralonsohat the same time, I mean14:43
ralonsohI would prefer to implement something compatible with the current code14:43
ralonsohmore or less, rather than adding functionalities14:44
slaweqsure14:44
ralonsoh(to be honest, I really don't even know how to start right now)14:44
ralonsohthis requires a level of refactor... huge one14:44
ralonsohso that's all14:44
haleyb"5 minutes, more or less"14:45
haleyb:)14:45
ralonsohI'll send a mail today to provide a feedback channel14:45
ralonsohor 6 mins14:45
haleybralonsoh: so my next question is what do you need help with? i was thinking once there was a patch for one, we could adopt it for the others14:46
haleybor is that a wrong assumption?14:46
ralonsohyes, I'm starting with ovn because I have the env14:47
ralonsohthat should work for both14:47
ralonsohalso ovn metadata only depends on eventlet on this part of the code14:47
ralonsohso maybe we'll be able to create a CI with an eventlet-free ovn metadata14:47
haleybralonsoh: i was even thinking for the other agents too14:48
ralonsohwell, we'll need a new olso.service release14:49
haleybok, and that i'm assuming will happen by E-214:49
haleybi will just wait for now, but would like to help with at least one of the other agents once we have a framework14:51
haleybalright, we can move on14:52
haleyb#topic on-demand14:52
haleybfloor is open for any other topics14:52
fricklerI have another eventlet topic, which is slightly related14:52
haleybfrickler: sure, go ahead14:52
fricklerit is about making neutron work with the latest eventlet release. currently we are still pinned to 0.36.1 in reqs see https://review.opendev.org/c/openstack/requirements/+/93325714:52
fricklerI started checking the neutron failures in the tempest job there and came up with https://review.opendev.org/c/openstack/neutron/+/937353 but this seems to be complicated still14:53
ralonsohthe issue in tempest is not related to this14:53
ralonsohwe are investigating the issues in https://review.opendev.org/c/openstack/neutron/+/93642914:53
ralonsohthere are 5 patches under this one that fixes several issues14:53
ralonsohalthough we are still seeing problems with high API numbers, with 2 Neutron APIs work fine14:54
ralonsohso I would encourage all to review these patches14:54
fricklerralonsoh: did you test any of that with updated eventlet yet?14:55
ralonsohno14:55
ralonsohwe can talk after this meeting14:56
fricklerso I think there are more hidden issues uncovered by newer eventlet, but I'll check your patches and see if they help with that14:56
ralonsoh(also the tempest job is not using wsgi...)14:56
ralonsohhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_3c1/933257/3/check/tempest-full-py3/3c1f7c1/controller/logs/screen-q-svc.txt14:56
haleybRuntimeError: cannot notify on un-acquired lock14:57
frickleryes, I thought that this was a bug in oslo.messaging at first14:58
fricklerbut those errors go away with my above patch14:58
haleybwe can continue discussing, just let me close the meeting14:59
haleybthanks for attending everyone and have a good week15:00
haleyb#endmeeting15:00
opendevmeetMeeting ended Tue Dec 10 15:00:05 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:00
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-12-10-14.00.html15:00
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-12-10-14.00.txt15:00
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-12-10-14.00.log.html15:00
lajoskatonaBye15:00
mlavalle\o15:00
mlavalleHappy Holidays!15:00
ralonsohfrickler, the requirements tempest job is still using the neutron eventlet server15:01
ralonsohI think we can patch the main file that is neutron.cmd.eventlet.server.__init__15:01
haleybmlavalle: enjoy your time off!15:02
fricklerralonsoh: I tried that locally and it still seemed to be too late, which I why I did the patched neutron-server cmd script15:02
ralonsohfrickler, this is created by console_scripts, right?15:03
frickleryes15:04
ralonsohis it not possible to patch this?15:04
ralonsohinstead of re-creating this file in plugin.sh15:04
fricklerit might be, I haven't looked into that yet. I first want to get a working server, which I still havent achieved15:04
ralonsohbut I don't understand15:05
ralonsohwhen we call from neutron.cmd.eventlet.server import main, we are also calling monkey_patch15:05
ralonsohah ah ah, and the lower __init__ imports15:06
ralonsohfrickler, ok, I have an environment and I've reproduced the issue locally15:16
ralonsohwith the eventlet server15:16
ralonsohfrickler, isn't this happening in other CIs?\15:51
ralonsohfor example Nova15:51
fricklerralonsoh: not this particular issue afaict15:53
ralonsohI'm going to try using wsgi15:54
ralonsohif that works, I'll change the tempest definition to use neutron wsgi instead15:54
ralonsohI dont' understand why the enter context of threading.Condition() is not setting the correct counts15:55
slaweqhaleyb I would like to propose new neutron-lib release this week, do You want to wait for https://review.opendev.org/c/openstack/neutron-lib/+/935731 and include it there as well?15:56
slaweqI don't see any other potential candiates to wait for on the list of opened patches15:57
ralonsohfrickler, it seems to work fine so I'll propose the patch right now16:05
opendevreviewSlawek Kaplonski proposed openstack/neutron master: QinQ API extension implementation in the Neutron server  https://review.opendev.org/c/openstack/neutron/+/93737216:06
haleybslaweq: i would, just need to get some reviews on that. It doesn't seem to break anything16:06
slaweqhaleyb I will review it later tonight or tomorrow morning16:06
slaweqand will ping others to review it too :)16:07
opendevreviewMerged openstack/neutron master: Add a detailed debug message in case of segment allocation fail  https://review.opendev.org/c/openstack/neutron/+/93617117:19
slaweqhaleyb I reviewed https://review.opendev.org/c/openstack/neutron-lib/+/935731 and I just left one comment there18:09
haleybslaweq: ack, i can remove that exception class and change the wording in the other18:21
haleybthe quota one too i guess18:22
opendevreviewBrian Haley proposed openstack/neutron-lib master: Support project_id and project_name in ContextBase class  https://review.opendev.org/c/openstack/neutron-lib/+/93573118:27
slaweqhaleyb for the Quota one, it is ok for me as the new name is correct but for the HANetworkConcurrentDeletionProject new name suggest that project was concurently deleted, what is not true18:27
slaweqso I would change only that one18:27
haleybarg, let me add it back18:27
slaweq:)18:28
slaweqthx18:28
slaweqhaleyb I added one more comment there, please check when You will have a minute :)18:50
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Bump hacking  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93685519:01
opendevreviewMerged openstack/neutron-lib master: api-ref: explain nested SNAT behavior is backend specific  https://review.opendev.org/c/openstack/neutron-lib/+/93739219:17
*** atmark_ is now known as atmark19:37
haleybslaweq: i don't exactly understand the question, responded19:51
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent stable/2024.2: Make in_port consistent type  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93748922:52
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent stable/2024.1: Make in_port consistent type  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93749022:52
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent stable/2023.2: Make in_port consistent type  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93749122:52
opendevreviewMerged openstack/neutron-dynamic-routing stable/2024.1: Stable Only: Use branched tempest jobs  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/93673723:46
opendevreviewBrian Haley proposed openstack/neutron master: WIP: Remove false-positive ACLs in OVN DB sync  https://review.opendev.org/c/openstack/neutron/+/93729923:47

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