| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 02:03 |
|---|---|---|
| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 02:10 |
| opendevreview | Jimin Shin proposed openstack/neutron master: Add information to network ip availabilities https://review.opendev.org/c/openstack/neutron/+/965083 | 02:24 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change some unit test code to use project_id https://review.opendev.org/c/openstack/neutron/+/968084 | 04:34 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change some functional test code to use project_id https://review.opendev.org/c/openstack/neutron/+/968085 | 04:35 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change test code to use project_id in dictionaries https://review.opendev.org/c/openstack/neutron/+/968414 | 04:35 |
| *** ykarel_ is now known as ykarel | 07:35 | |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Cleanup stale DHCP ports not bound to host https://review.opendev.org/c/openstack/neutron/+/968770 | 08:02 |
| ralonsoh | hi folks! The 2nd bug fix for the OVS manager FT test: https://review.opendev.org/c/openstack/neutron/+/969107 | 08:45 |
| ykarel | added a comment | 08:52 |
| ralonsoh | thanks! | 08:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Wait for the manager to be created (2) https://review.opendev.org/c/openstack/neutron/+/969107 | 09:11 |
| opendevreview | Merged openstack/neutron stable/2025.2: Don't prevent setting CIDRs as allowed_address_pair for port https://review.opendev.org/c/openstack/neutron/+/969126 | 09:12 |
| opendevreview | Merged openstack/neutron stable/2025.1: Don't prevent setting CIDRs as allowed_address_pair for port https://review.opendev.org/c/openstack/neutron/+/969127 | 09:12 |
| opendevreview | Merged openstack/neutron stable/2024.2: Don't prevent setting CIDRs as allowed_address_pair for port https://review.opendev.org/c/openstack/neutron/+/969128 | 09:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Add extra logs to ``test_create_bridges`` test https://review.opendev.org/c/openstack/neutron/+/969202 | 09:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Docs] Add doc about Virtual IP addresses in Neutron ML2/OVN https://review.opendev.org/c/openstack/neutron/+/968812 | 10:04 |
| zigo | ralonsoh: Hi there! I wonder, what's missing for dynamic-routing eventlet removal to work? | 11:24 |
| zigo | Is it that you don't understand what's going on? | 11:25 |
| ralonsoh | zigo, I don't know, this is why I wrote this mail requesting help on this issue. It seems (maybe) to be a problem in the os-ken library, according to the logs from https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/956747 | 11:26 |
| ralonsoh | but I didn't spend more time on this | 11:27 |
| zigo | ralonsoh: os-ken is still full of eventlet stuff... | 11:27 |
| zigo | Shouldn't we start by it first? | 11:28 |
| ralonsoh | zigo, no, we migrated it to threading. Actually in os-ken 4.0.0, "native" is the default backend | 11:28 |
| zigo | If one sets the env var OSKEN_HUB_TYPE=native, right? | 11:29 |
| ralonsoh | but of course, there could still be some bugs on untested code branchs, like bgp: https://zuul.opendev.org/t/openstack/build/bbfba96502fb40aeb3083a05e2ca08bf/log/controller/logs/screen-q-dr-agent.txt#1236 | 11:29 |
| ralonsoh | branches* | 11:29 |
| zigo | (which seems to be the default) | 11:29 |
| ralonsoh | yes, this is now the default one with 4.0.0 | 11:29 |
| zigo | I'll have to fix my neutron-ipv6-bgp-injector then ! :) | 11:30 |
| zigo | ralonsoh: BTW, I was thinking, but not sure how, to get my neutron-ipv6-bgp-injector integrated with neutron-dynamic-routing. See https://salsa.debian.org/openstack-team/services/neutron-ipv6-bgp-injector/ | 11:31 |
| zigo | Your thoughts?!? | 11:31 |
| zigo | Is it better to keep it a standalone project? | 11:31 |
| zigo | What it does: advertize each /128 over BGP, so that our switch can block traffic for IPs that aren't provisionned on the OpenStack side, eliminating a lot of noise trafic, like scans ... | 11:32 |
| ralonsoh | is that a generic bgp tool? can we create functional test for it? | 11:33 |
| ralonsoh | also is it possible to have it also for ipv4? | 11:33 |
| ralonsoh | that could be added (maybe) as a tool in n-d-r | 11:33 |
| ralonsoh | you can propose a RFE | 11:33 |
| zigo | For IPv4, we use dynamic-routing, but it wouldn't do the work for IPv6. | 11:34 |
| zigo | ie: for IPv6, only /64 are advertized. | 11:34 |
| lajoskatona | zigo, ralonsoh: I plan to go back to ovs-agent and os-ken vs threading this week | 11:35 |
| zigo | For integration in n-d-r, it would need to add a subnet table so one could add ipv6 subnets with the API (instead of the config file). | 11:35 |
| zigo | lajoskatona: great ! :) | 11:35 |
| zigo | And thanks. | 11:35 |
| haleyb | #startmeeting networking | 14:01 |
| opendevmeet | Meeting started Tue Dec 2 14:01:17 2025 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'networking' | 14:01 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, haleyb, ralonsoh | 14:01 |
| elvira | o/ | 14:01 |
| ralonsoh | hedllo | 14:01 |
| ralonsoh | hello* | 14:01 |
| lajoskatona | o/ | 14:01 |
| rubasov | o/ | 14:02 |
| haleyb | #announcements | 14:02 |
| cbuggy | o/ | 14:02 |
| haleyb | We are in week R-17 of Gazpacho | 14:02 |
| haleyb | Our next milestone in this development cycle will be gazpacho-2, on January 8, 2026 | 14:03 |
| haleyb | #link https://releases.openstack.org/gazpacho/schedule.html | 14:03 |
| haleyb | So we should start getting any new features merged | 14:04 |
| haleyb | Continue to use the priorities dashboard for important changes or things ready to merge | 14:04 |
| haleyb | #link https://tinyurl.com/59z278km | 14:04 |
| haleyb | think there was only one patch there yesterday | 14:05 |
| 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:05 |
| haleyb | was there anything important i missed being out most of last week? | 14:06 |
| haleyb | looks like CI has improved :) | 14:06 |
| haleyb | alright can move on | 14:07 |
| haleyb | #topic bugs | 14:07 |
| haleyb | elvira was bug deputy last week, her report is at | 14:07 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/NO5J73HGRWUDT6AOKVTG5QZ4Z7HICJI6/ | 14:07 |
| haleyb | looks like the critical and high ones have owners and fixes proposed/committed | 14:08 |
| haleyb | first medium one | 14:09 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2132342 | 14:09 |
| haleyb | OVN LB / FIP PF does not reply to ARP when VIP is from different subnet than router | 14:09 |
| haleyb | ovn-octavia-provider issue it seems | 14:09 |
| haleyb | looks like OVN behavior changed | 14:10 |
| haleyb | Since i see froyo commented I'll assume he will help address it | 14:11 |
| haleyb | next one is | 14:12 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2133212 | 14:12 |
| haleyb | [S-RBAC] Regular user can't never send project_id in request body, even with custom policy rules which allows that | 14:12 |
| slaweq | I will work on this one | 14:12 |
| haleyb | slaweq: thanks for picking this up, did it need a discussion at all? | 14:12 |
| slaweq | I'm not sure if we need to talk about it really | 14:13 |
| haleyb | ok, will look for the patch :) | 14:13 |
| slaweq | we should probably add new rule in policy to let operators allow to do that for specific roles | 14:13 |
| slaweq | but I think it will be better to discuss in the patch review once it will be pushed to gerrit | 14:14 |
| slaweq | I plan to work on it this week | 14:14 |
| haleyb | ack, thanks | 14:15 |
| haleyb | final unassigned one is | 14:15 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2132382 | 14:15 |
| haleyb | [RFE] Refactor the SR-IOV agent to use the oslo.service launcher | 14:15 |
| ralonsoh | yeah, this is just an improvement | 14:16 |
| ralonsoh | we use oslo.serive launch for all agents except SR-IOV | 14:16 |
| ralonsoh | it could be desirable to have this | 14:16 |
| haleyb | for a second i thought it was eventlet related | 14:16 |
| ralonsoh | nononono | 14:16 |
| haleyb | i'm assuming any change would look like what was done in other agents | 14:16 |
| ralonsoh | yes, that is the point | 14:17 |
| ralonsoh | (and removing the manual signal handling, for example) | 14:17 |
| haleyb | i can take a closer look later | 14:18 |
| ralonsoh | thanks! | 14:18 |
| haleyb | that was all the unassigned/new bugs from last week, any others to discuss? | 14:19 |
| haleyb | this week slaweq is the bug deputy, next week is myself | 14:19 |
| haleyb | slaweq: is that ok for you? | 14:19 |
| slaweq | sure | 14:20 |
| haleyb | ok, moving on | 14:21 |
| haleyb | #topic specs | 14:21 |
| haleyb | #link https://review.opendev.org/q/project:openstack/neutron-specs+status:open | 14:21 |
| haleyb | i really need to review that list | 14:22 |
| haleyb | we can move on | 14:23 |
| haleyb | #topic community goals | 14:23 |
| haleyb | lajoskatona: can you give a neutronclient update? | 14:23 |
| lajoskatona | some, I hope I can push today a nova patch for ports | 14:23 |
| haleyb | #link https://review.opendev.org/c/openstack/nova/+/962604 is probably one of them | 14:24 |
| haleyb | #link https://review.opendev.org/q/topic:%22sdk_for_neutron%22 | 14:24 |
| lajoskatona | and perhaps refresh for some of the older patches as I found some small mistakes during the work on port calls migration to SDK | 14:24 |
| lajoskatona | yes thanks, those are the nova patches | 14:24 |
| lajoskatona | that's it for novaclient | 14:25 |
| lajoskatona | And Miro started to work on moving OSC related code from n-client to OSC, I have to go to test that as well | 14:26 |
| haleyb | ah yes | 14:26 |
| lajoskatona | https://review.opendev.org/c/openstack/python-openstackclient/+/963445 | 14:26 |
| haleyb | lajoskatona: have you had many reviews on the nova patches? just want to make sure they don't get ignored | 14:27 |
| lajoskatona | 0 (zero) | 14:27 |
| lajoskatona | I have t poke them a little | 14:27 |
| ralonsoh | https://www.youtube.com/watch?v=3n6ZY0tYsJg | 14:28 |
| ralonsoh | (sorry for the joke) | 14:28 |
| lajoskatona | :-) | 14:28 |
| haleyb | perfect | 14:29 |
| haleyb | i will put them in my queue | 14:29 |
| haleyb | ralonsoh: are there any more eventlet things you know of? | 14:31 |
| ralonsoh | just the n-d-r issues, related to the eventlet removal | 14:31 |
| ralonsoh | zigo is going to work on this | 14:31 |
| ralonsoh | it could be an error in the os-ken library, a module we don't use in neutron (bgp) | 14:32 |
| haleyb | ah, this was discussion right before meeting, will look through comments | 14:32 |
| ralonsoh | yes | 14:32 |
| ralonsoh | apart from this, I would suggest to remove this topic from the weekly meetings because we can consider that most of the job is done (Neutron and several networking projects) | 14:33 |
| ralonsoh | so we can handle this ad hoc, when needed | 14:33 |
| haleyb | there definitely could be another eventelet bug in os-ken | 14:33 |
| haleyb | ralonsoh: ack, we can just file bugs when we find these issues | 14:33 |
| ralonsoh | so congrats everyone! we had a long journey here but we finished | 14:34 |
| ralonsoh | (almost) | 14:34 |
| lajoskatona | \o/ | 14:35 |
| haleyb | agreed, thanks for all the hard work by a number of people over the past year+ | 14:35 |
| haleyb | #topic on-demand | 14:36 |
| haleyb | ralonsoh: i think you had an item | 14:36 |
| ralonsoh | yes, https://review.opendev.org/c/openstack/neutron/+/968242 | 14:36 |
| ralonsoh | this patch (thanks takashi) is restoring the "multi processing" for the metadata | 14:37 |
| ralonsoh | it is actually using threads, is not exactly the same | 14:37 |
| ralonsoh | but as long as the waiting for the event is not CPU bound, this coudl improve the performance | 14:37 |
| ralonsoh | my only concern is that before this imlementation, we were recommending to use metadata_workers=0 to reduce the SB connections from the compute nodes | 14:38 |
| ralonsoh | this patch is "reverting" that | 14:38 |
| haleyb | right, that was the main thing i noticed | 14:39 |
| ralonsoh | so I would ask to make the assumption that metadata_workers=0 is a valid input and in this implementation metadata_workers=0 == metadata_workers=1 | 14:39 |
| ralonsoh | that needs to be documented and implemented (1 minute coding) | 14:39 |
| ralonsoh | that's all, you can leave a comment in the patch | 14:40 |
| lajoskatona | you mean in the same patch or in a follow-up ? | 14:40 |
| ralonsoh | same patch | 14:40 |
| ralonsoh | because Takashi is multitasking, I'll ask him if I can push a new PS | 14:40 |
| ralonsoh | but after your reviews and comments | 14:41 |
| lajoskatona | +1 | 14:41 |
| ralonsoh | thanks folks! | 14:41 |
| haleyb | and i would agree with the workers==0 comment | 14:42 |
| ralonsoh | cool | 14:42 |
| haleyb | any other on-demand topics? | 14:43 |
| haleyb | ok, thanks for attending and have a great week! | 14:44 |
| haleyb | #endmeeting | 14:44 |
| opendevmeet | Meeting ended Tue Dec 2 14:44:20 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:44 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2025/networking.2025-12-02-14.01.html | 14:44 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2025/networking.2025-12-02-14.01.txt | 14:44 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2025/networking.2025-12-02-14.01.log.html | 14:44 |
| ralonsoh | bye | 14:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: iPXE over IPv6 supported since OVN v23.06.0 https://review.opendev.org/c/openstack/neutron/+/968811 | 14:44 |
| lajoskatona | Bye | 14:44 |
| zigo | ralonsoh: I don't think I have the needed skills to work on eventlet removal for n-d-r. :/ | 14:46 |
| ralonsoh | haleyb, ^ maybe we need a reminder in the mail I sent, just to know if anyone else is able to move this forward | 14:57 |
| haleyb | ralonsoh: sure let me look | 14:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add documentation/reno for the OVN external port N/S routing https://review.opendev.org/c/openstack/neutron/+/968797 | 15:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [OVN] OVN agent configuration options list missing "extensions" https://review.opendev.org/c/openstack/neutron/+/969266 | 15:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove a missing Windows configuration file https://review.opendev.org/c/openstack/neutron/+/969267 | 16:06 |
| opendevreview | Merged openstack/neutron master: [FT] Wait for the manager to be created (2) https://review.opendev.org/c/openstack/neutron/+/969107 | 16:23 |
| opendevreview | Merged openstack/neutron unmaintained/2024.1: Don't prevent setting CIDRs as allowed_address_pair for port https://review.opendev.org/c/openstack/neutron/+/969155 | 18:08 |
| opendevreview | Merged openstack/neutron master: [FT] Add extra logs to ``test_create_bridges`` test https://review.opendev.org/c/openstack/neutron/+/969202 | 20:55 |
| opendevreview | Brian Haley proposed openstack/neutron-dynamic-routing master: [eventlet-removal] Removal of eventlet usage in this repository https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/956747 | 22:45 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!