opendevreview | liuyulong proposed openstack/neutron-tempest-plugin master: Re-enable the router test_migration cases https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/936318 | 01:46 |
---|---|---|
opendevreview | liuyulong proposed openstack/neutron master: Add basical functionalities for metadata path extension https://review.opendev.org/c/openstack/neutron/+/881535 | 01:57 |
opendevreview | liuyulong proposed openstack/neutron master: Add metadata path extension openflows https://review.opendev.org/c/openstack/neutron/+/888097 | 01:57 |
opendevreview | liuyulong proposed openstack/neutron master: Fullstack case for metadata path https://review.opendev.org/c/openstack/neutron/+/888098 | 01:57 |
opendevreview | liuyulong proposed openstack/neutron master: Add devstack plugin to enable ovs metadata_path https://review.opendev.org/c/openstack/neutron/+/928586 | 01:57 |
opendevreview | yatin proposed openstack/neutron master: [DNM] wsgi switch higher workers https://review.opendev.org/c/openstack/neutron/+/936272 | 08:15 |
opendevreview | Merged openstack/neutron-fwaas master: Drop extra space from nflog-prefix https://review.opendev.org/c/openstack/neutron-fwaas/+/937361 | 08:37 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Set always the GW LRP "gateway_mtu" option https://review.opendev.org/c/openstack/neutron/+/937026 | 08:37 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Optimization in the router port options generation https://review.opendev.org/c/openstack/neutron/+/937025 | 08:38 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Optimization in the router port options generation https://review.opendev.org/c/openstack/neutron/+/937025 | 08:39 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Set always the GW LRP "gateway_mtu" option https://review.opendev.org/c/openstack/neutron/+/937026 | 08:39 |
opendevreview | liuyulong proposed openstack/neutron-tempest-plugin master: Re-enable the router test_migration cases https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/936318 | 09:12 |
*** ralonsoh_ is now known as ralonsoh | 09:21 | |
opendevreview | liuyulong proposed openstack/neutron-tempest-plugin master: Skip nested snat validation for DVR https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/937443 | 09:29 |
opendevreview | Fernando Royo proposed openstack/ovsdbapp master: [DNM] Adding some logs to debug txn commit issue https://review.opendev.org/c/openstack/ovsdbapp/+/937445 | 09:42 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936952 | 09:46 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936952 | 09:48 |
opendevreview | Kien Nguyen proposed openstack/neutron master: Use the correct input for OVN agent deletion https://review.opendev.org/c/openstack/neutron/+/936988 | 11:33 |
opendevreview | Fernando Royo proposed openstack/ovsdbapp master: [DNM] Adding some logs to debug txn commit issue https://review.opendev.org/c/openstack/ovsdbapp/+/937445 | 11:39 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936952 | 11:58 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936952 | 11:58 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936952 | 12:04 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with 4 workers https://review.opendev.org/c/openstack/neutron/+/936429 | 12:20 |
opendevreview | Merged 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/+/937237 | 13:24 |
haleyb | #startmeeting networking | 14:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'networking' | 14:00 |
haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh | 14:00 |
mlavalle | \o | 14:00 |
lajoskatona | o/ | 14:00 |
obondarev | o/ | 14:00 |
ralonsoh | hello | 14:00 |
ykarel | o/ | 14:00 |
s3rj1k | hi all | 14:00 |
frickler | \o | 14:00 |
jlibosva | o/ | 14:00 |
haleyb | ok, let's get started | 14:01 |
haleyb | #announcements | 14:01 |
slaweq | o/ | 14:01 |
haleyb | #link https://releases.openstack.org/epoxy/schedule.html | 14:01 |
haleyb | We are currently in week R-16 | 14:01 |
haleyb | Epoxy-2 milestone R-12 (week of Jan 6th) | 14:01 |
haleyb | Reminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:02 |
haleyb | i see one item on the wiki for this week, unfortunately i might not be able to attend | 14:03 |
haleyb | it actually doesn't look like an RFE anyways | 14:03 |
haleyb | so maybe no meeting is required | 14:04 |
lajoskatona | +1, no meeting is good meeting :-) | 14:04 |
opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 14:04 |
rubasov | late o/ | 14:04 |
haleyb | i will ping him after meeting for status | 14:05 |
lajoskatona | especially on Friday afternoon :-) | 14:05 |
haleyb | the 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 weeks | 14:06 |
* mlavalle will be off starting tomorrow | 14:06 | |
haleyb | i was going to cancel the meeting for the 24th | 14:07 |
haleyb | but with E-2 coming up shortly after, i'm not sure if we will have things to discuss on 12/31 | 14:07 |
slaweq | I won't be there 24th nor 31st :) | 14:08 |
haleyb | like mlavalle, i will also be taking a break, but will be working a bit to make sure the wheels don't fall off | 14:08 |
lajoskatona | I also will be on PTO | 14:09 |
haleyb | i 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 January | 14:09 |
mlavalle | first time in 40 years that I take such a long break | 14:09 |
haleyb | mlavalle: you deserve it, and enjoy you're time off | 14:10 |
opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 14:10 |
lajoskatona | mlavalle: enjoy it :-) | 14:10 |
haleyb | so should i also cancel the meeting on the 31st? | 14:11 |
slaweq | ++ | 14:11 |
ykarel | ++ | 14:11 |
mlavalle | ++ | 14:11 |
ihrachys | yes! some locations / culture consider NY eve as much a holiday as Christmas | 14:11 |
lajoskatona | +1 | 14:11 |
haleyb | ok, done. Meeting next week on the 17th, then next on Jan 7th | 14:12 |
haleyb | I will still be on the beach on the 7th, can someone volunteer to lead that one? I'm back on the 8th | 14:12 |
mlavalle | yes I can | 14:13 |
haleyb | mlavalle: thanks. and that might be a busy week being E-2 with releases being proposed | 14:13 |
haleyb | i will send an email to the list regarding the meetings in case someone is not here | 14:14 |
haleyb | last announcement i have is my weekly reminder | 14:15 |
haleyb | Let's continue to use the priorities dashboard for patches in the "ready to merge" state (weekly reminder) | 14:15 |
haleyb | any other announcements? | 14:16 |
haleyb | #topic bugs | 14:16 |
haleyb | mlavalle was the deputy this week, his report is at | 14:17 |
haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/3HIN2NODJM6S3BF3LKXQW6BMHSNQPKWP/ | 14:17 |
haleyb | i see most have owners and/or patches | 14:17 |
haleyb | this one does not | 14:18 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2091211 | 14:18 |
mlavalle | yeah that one I marked as incomplete, but submitter responded, so it needs follow up | 14:19 |
haleyb | it's on yoga/zed | 14:19 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: DNM: debug o-api <--> ovsdbapp timeout https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936952 | 14:19 |
haleyb | first step would be to reproduce on master branch, not sure if anyone has cycles or a setup running ml2/ovs | 14:21 |
lajoskatona | haleyb: I can check, I try to spare some time for it | 14:21 |
haleyb | lajoskatona: ack thanks. this also could be a deployment issue, that was using ansible | 14:22 |
slaweq | isn't it normal what is in this bug / | 14:22 |
slaweq | when you run tcpdump your nic is in promiscous mode, right? | 14:23 |
slaweq | or am I wrong? | 14:23 |
slaweq | it would be bad if there would be packets from some other network | 14:23 |
slaweq | but from the same one I think it is kind of ok, no? | 14:23 |
lajoskatona | possible, I have to check | 14:24 |
ralonsoh | in OVS? TAP ports should receive VM traffic only | 14:24 |
haleyb | slaweq: it could be normal, all those things you describe seem valid, i guess with port security i would not have expected | 14:24 |
haleyb | it is a flat provider network | 14:25 |
slaweq | but port security is not filtering incoming packets based on the dst mac | 14:25 |
slaweq | IIRC | 14:25 |
ralonsoh | we do, we accept packets only for the destination port, mac and IP | 14:26 |
slaweq | anyway, 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 |
ralonsoh | at least with native one | 14:26 |
haleyb | i will ask that question in the bug, perhaps this is with iptables hybrid? i don't want lajoskatona wasting cycles with wrong config | 14:27 |
mlavalle | +1 | 14:27 |
lajoskatona | haleyb: ack, the fw driver can beimportant, I defaulted to ovs fw driver in my mind :-) | 14:28 |
haleyb | lajoskatona: i posted that question, and thanks for taking a look | 14:28 |
haleyb | any other bugs we should discuss? | 14:28 |
haleyb | ihrachys is the bug deputy this week | 14:29 |
haleyb | i 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 |
ralonsoh | right | 14:30 |
ralonsoh | I can take his place | 14:31 |
haleyb | ralonsoh: ok, thanks. i will ping him later about the next rotation which is way out in March | 14:32 |
haleyb | hopefully things are quiet those weeks anyways | 14:32 |
haleyb | ok, moving on | 14:33 |
haleyb | #topic community goals | 14:33 |
haleyb | i see there was a little movement on neutronclient deprecation | 14:33 |
haleyb | #link https://review.opendev.org/q/topic:%22bug/1999774%22 | 14:33 |
lajoskatona | yes, 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 team | 14:34 |
lajoskatona | but slow progress anyway | 14:34 |
haleyb | lajoskatona: ack, at least one has a +2 | 14:34 |
haleyb | and i was surprised py39 is still being run in the other | 14:35 |
haleyb | doh, although we run it too, my brain has not had coffeee yet | 14:36 |
haleyb | other community goal is eventlet | 14:37 |
haleyb | #link https://bugs.launchpad.net/neutron/+bugs?field.tag=eventlet-deprecation | 14:37 |
ralonsoh | yes, I'm working in the ovn metadata migration | 14:37 |
ralonsoh | more in particular in the metadta handler | 14:37 |
ralonsoh | right now we have 2 methods to create the handler: in a thread and spawning processes | 14:38 |
ralonsoh | (metadata_workers=0 or other number) | 14:38 |
ralonsoh | all these methods use eventlet | 14:38 |
ralonsoh | so I'm going to start migrating only the default one (because metadata_workers=0 by default) | 14:39 |
ralonsoh | and that implies refactoring everything: the http handler, the embeded wsgi server and remove, probably, the haproxy | 14:39 |
ralonsoh | 5 minutes, more or less | 14:39 |
ralonsoh | the question is: I'll start with this thread model | 14:39 |
haleyb | so no more haproxy? | 14:40 |
ralonsoh | instead of migrating the other that spawns several process, that would imply creating something like the Neutron API wsgi server (or nova metadata) | 14:40 |
ralonsoh | haleyb, we don't need it if we have one single embedded server reading from the port | 14:41 |
ralonsoh | so do you agree with this strategy? | 14:41 |
ralonsoh | (maybe this deserves a mail) | 14:41 |
haleyb | ralonsoh: 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 |
haleyb | like rate limiting, etc | 14:42 |
ralonsoh | yes, that's something to be considered | 14:42 |
ralonsoh | so maybe haproxy can stay | 14:42 |
slaweq | yes, rate limiting is important IMO | 14:42 |
slaweq | but with haproxy we have it only for IPv4 or IPv6, not both at the same time | 14:43 |
ralonsoh | and the could be also compatible with a future implementation using multiple processes | 14:43 |
ralonsoh | slaweq, we don't need both | 14:43 |
slaweq | if we could have something else working for both IP versions would be great | 14:43 |
ralonsoh | at the same time, I mean | 14:43 |
ralonsoh | I would prefer to implement something compatible with the current code | 14:43 |
ralonsoh | more or less, rather than adding functionalities | 14:44 |
slaweq | sure | 14:44 |
ralonsoh | (to be honest, I really don't even know how to start right now) | 14:44 |
ralonsoh | this requires a level of refactor... huge one | 14:44 |
ralonsoh | so that's all | 14:44 |
haleyb | "5 minutes, more or less" | 14:45 |
haleyb | :) | 14:45 |
ralonsoh | I'll send a mail today to provide a feedback channel | 14:45 |
ralonsoh | or 6 mins | 14:45 |
haleyb | ralonsoh: 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 others | 14:46 |
haleyb | or is that a wrong assumption? | 14:46 |
ralonsoh | yes, I'm starting with ovn because I have the env | 14:47 |
ralonsoh | that should work for both | 14:47 |
ralonsoh | also ovn metadata only depends on eventlet on this part of the code | 14:47 |
ralonsoh | so maybe we'll be able to create a CI with an eventlet-free ovn metadata | 14:47 |
haleyb | ralonsoh: i was even thinking for the other agents too | 14:48 |
ralonsoh | well, we'll need a new olso.service release | 14:49 |
haleyb | ok, and that i'm assuming will happen by E-2 | 14:49 |
haleyb | i will just wait for now, but would like to help with at least one of the other agents once we have a framework | 14:51 |
haleyb | alright, we can move on | 14:52 |
haleyb | #topic on-demand | 14:52 |
haleyb | floor is open for any other topics | 14:52 |
frickler | I have another eventlet topic, which is slightly related | 14:52 |
haleyb | frickler: sure, go ahead | 14:52 |
frickler | it 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/+/933257 | 14:52 |
frickler | I 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 still | 14:53 |
ralonsoh | the issue in tempest is not related to this | 14:53 |
ralonsoh | we are investigating the issues in https://review.opendev.org/c/openstack/neutron/+/936429 | 14:53 |
ralonsoh | there are 5 patches under this one that fixes several issues | 14:53 |
ralonsoh | although we are still seeing problems with high API numbers, with 2 Neutron APIs work fine | 14:54 |
ralonsoh | so I would encourage all to review these patches | 14:54 |
frickler | ralonsoh: did you test any of that with updated eventlet yet? | 14:55 |
ralonsoh | no | 14:55 |
ralonsoh | we can talk after this meeting | 14:56 |
frickler | so I think there are more hidden issues uncovered by newer eventlet, but I'll check your patches and see if they help with that | 14:56 |
ralonsoh | (also the tempest job is not using wsgi...) | 14:56 |
ralonsoh | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_3c1/933257/3/check/tempest-full-py3/3c1f7c1/controller/logs/screen-q-svc.txt | 14:56 |
haleyb | RuntimeError: cannot notify on un-acquired lock | 14:57 |
frickler | yes, I thought that this was a bug in oslo.messaging at first | 14:58 |
frickler | but those errors go away with my above patch | 14:58 |
haleyb | we can continue discussing, just let me close the meeting | 14:59 |
haleyb | thanks for attending everyone and have a good week | 15:00 |
haleyb | #endmeeting | 15:00 |
opendevmeet | Meeting ended Tue Dec 10 15:00:05 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-12-10-14.00.html | 15:00 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-12-10-14.00.txt | 15:00 |
opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-12-10-14.00.log.html | 15:00 |
lajoskatona | Bye | 15:00 |
mlavalle | \o | 15:00 |
mlavalle | Happy Holidays! | 15:00 |
ralonsoh | frickler, the requirements tempest job is still using the neutron eventlet server | 15:01 |
ralonsoh | I think we can patch the main file that is neutron.cmd.eventlet.server.__init__ | 15:01 |
haleyb | mlavalle: enjoy your time off! | 15:02 |
frickler | ralonsoh: I tried that locally and it still seemed to be too late, which I why I did the patched neutron-server cmd script | 15:02 |
ralonsoh | frickler, this is created by console_scripts, right? | 15:03 |
frickler | yes | 15:04 |
ralonsoh | is it not possible to patch this? | 15:04 |
ralonsoh | instead of re-creating this file in plugin.sh | 15:04 |
frickler | it might be, I haven't looked into that yet. I first want to get a working server, which I still havent achieved | 15:04 |
ralonsoh | but I don't understand | 15:05 |
ralonsoh | when we call from neutron.cmd.eventlet.server import main, we are also calling monkey_patch | 15:05 |
ralonsoh | ah ah ah, and the lower __init__ imports | 15:06 |
ralonsoh | frickler, ok, I have an environment and I've reproduced the issue locally | 15:16 |
ralonsoh | with the eventlet server | 15:16 |
ralonsoh | frickler, isn't this happening in other CIs?\ | 15:51 |
ralonsoh | for example Nova | 15:51 |
frickler | ralonsoh: not this particular issue afaict | 15:53 |
ralonsoh | I'm going to try using wsgi | 15:54 |
ralonsoh | if that works, I'll change the tempest definition to use neutron wsgi instead | 15:54 |
ralonsoh | I dont' understand why the enter context of threading.Condition() is not setting the correct counts | 15:55 |
slaweq | haleyb 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 |
slaweq | I don't see any other potential candiates to wait for on the list of opened patches | 15:57 |
ralonsoh | frickler, it seems to work fine so I'll propose the patch right now | 16:05 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: QinQ API extension implementation in the Neutron server https://review.opendev.org/c/openstack/neutron/+/937372 | 16:06 |
haleyb | slaweq: i would, just need to get some reviews on that. It doesn't seem to break anything | 16:06 |
slaweq | haleyb I will review it later tonight or tomorrow morning | 16:06 |
slaweq | and will ping others to review it too :) | 16:07 |
opendevreview | Merged openstack/neutron master: Add a detailed debug message in case of segment allocation fail https://review.opendev.org/c/openstack/neutron/+/936171 | 17:19 |
slaweq | haleyb I reviewed https://review.opendev.org/c/openstack/neutron-lib/+/935731 and I just left one comment there | 18:09 |
haleyb | slaweq: ack, i can remove that exception class and change the wording in the other | 18:21 |
haleyb | the quota one too i guess | 18:22 |
opendevreview | Brian Haley proposed openstack/neutron-lib master: Support project_id and project_name in ContextBase class https://review.opendev.org/c/openstack/neutron-lib/+/935731 | 18:27 |
slaweq | haleyb 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 true | 18:27 |
slaweq | so I would change only that one | 18:27 |
haleyb | arg, let me add it back | 18:27 |
slaweq | :) | 18:28 |
slaweq | thx | 18:28 |
slaweq | haleyb I added one more comment there, please check when You will have a minute :) | 18:50 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Bump hacking https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936855 | 19:01 |
opendevreview | Merged openstack/neutron-lib master: api-ref: explain nested SNAT behavior is backend specific https://review.opendev.org/c/openstack/neutron-lib/+/937392 | 19:17 |
*** atmark_ is now known as atmark | 19:37 | |
haleyb | slaweq: i don't exactly understand the question, responded | 19:51 |
opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent stable/2024.2: Make in_port consistent type https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937489 | 22:52 |
opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent stable/2024.1: Make in_port consistent type https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937490 | 22:52 |
opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent stable/2023.2: Make in_port consistent type https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937491 | 22:52 |
opendevreview | Merged openstack/neutron-dynamic-routing stable/2024.1: Stable Only: Use branched tempest jobs https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/936737 | 23:46 |
opendevreview | Brian Haley proposed openstack/neutron master: WIP: Remove false-positive ACLs in OVN DB sync https://review.opendev.org/c/openstack/neutron/+/937299 | 23:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!