| ykarel | ralonsoh, ack | 04:48 |
|---|---|---|
| ykarel | hmm so it fails even with 1 concurrency | 04:54 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job on top of patch 967844 https://review.opendev.org/c/openstack/ovsdbapp/+/967843 | 05:30 |
| opendevreview | yatin proposed openstack/ovsdbapp master: Revert "Rework venv to support arbitrary schemas" https://review.opendev.org/c/openstack/ovsdbapp/+/967844 | 05:53 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job on top of patch 967844 https://review.opendev.org/c/openstack/ovsdbapp/+/967843 | 05:54 |
| ykarel | ralonsoh, ^ works with full revert | 06:30 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 06:54 |
| ralonsoh | ykarel, so the problem is this patch | 06:54 |
| ykarel | yeap | 06:55 |
| ralonsoh | ok, good to know. For now, we need to revert it and release again ovsdbapp | 06:55 |
| ralonsoh | once Terry is back, I'll ping him | 06:55 |
| ykarel | yeap agree | 06:55 |
| ralonsoh | ykarel, what patch are you reverting? | 06:55 |
| ralonsoh | because https://review.opendev.org/c/openstack/ovsdbapp/+/967844 has two | 06:56 |
| ykarel | ralonsoh, both needed | 06:56 |
| ralonsoh | ahhh ok | 06:56 |
| ykarel | as second one was dependent on the first | 06:56 |
| ykarel | that's why your original attempt to test the revert failed | 06:57 |
| ralonsoh | yeah, I see it now. Thanks! | 06:57 |
| opendevreview | Sergey Kraynev proposed openstack/neutron master: Replace response body in after hook for 404 error https://review.opendev.org/c/openstack/neutron/+/966498 | 06:58 |
| ralonsoh | ok, I'm fast approving this and creating a release patch | 06:59 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job on top of patch 967844 https://review.opendev.org/c/openstack/ovsdbapp/+/967843 | 07:03 |
| opendevreview | yatin proposed openstack/neutron master: Revert "[eventlet-removal] Handle stop DHCP agent" https://review.opendev.org/c/openstack/neutron/+/967808 | 08:01 |
| ykarel | ralonsoh, rebased and updated commit message with relevant bugs^ | 08:02 |
| ralonsoh | thanks | 08:02 |
| opendevreview | Merged openstack/ovsdbapp master: Revert "Rework venv to support arbitrary schemas" https://review.opendev.org/c/openstack/ovsdbapp/+/967844 | 08:59 |
| ralonsoh | haleyb, please check https://review.opendev.org/c/openstack/releases/+/967967 | 09:05 |
| opendevreview | Merged openstack/neutron stable/2024.2: [FT] Wait for the manager to be created https://review.opendev.org/c/openstack/neutron/+/967339 | 10:19 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2024.2: Ensure that bridge exist and UP before adding tables https://review.opendev.org/c/openstack/ovn-bgp-agent/+/947152 | 10:37 |
| opendevreview | Lajos Katona proposed openstack/neutron master: [eventlet-removal] Handle systemctl stop for DHCP agent https://review.opendev.org/c/openstack/neutron/+/967902 | 10:43 |
| *** tkajinam is now known as Guest31836 | 11:05 | |
| ykarel | slaweq, lajoskatona please do check the revert https://review.opendev.org/c/openstack/neutron/+/967808 | 11:23 |
| opendevreview | Merged openstack/neutron master: [OVN] Initialize OVN agent in ``start`` method https://review.opendev.org/c/openstack/neutron/+/967219 | 11:24 |
| lajoskatona | ykarel: sure, I updated my patch on top of it | 11:58 |
| opendevreview | aarefiev proposed openstack/os-ken master: BGP: allow create multiple speakers https://review.opendev.org/c/openstack/os-ken/+/967688 | 13:17 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2025.1: Ensure that bridge exist and UP before adding tables https://review.opendev.org/c/openstack/ovn-bgp-agent/+/947151 | 13:31 |
| cardoe | Am I in the wrong channel for the drivers meeting? | 14:03 |
| mlavalle | haleyb, are we meeting today? | 14:03 |
| haleyb | mlavalle: yes, i think i got the time wrong again | 14:04 |
| cardoe | mlavalle: I wanted to talk to you about segment binding... specifically your spec and https://review.opendev.org/c/openstack/neutron/+/840418 wrt to https://bugs.launchpad.net/networking-generic-switch/+bug/2114451 | 14:04 |
| haleyb | let me start the meeting | 14:05 |
| haleyb | #startmeeting neutron_drivers | 14:05 |
| opendevmeet | Meeting started Fri Nov 21 14:05:55 2025 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:05 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:05 |
| opendevmeet | The meeting name has been set to 'neutron_drivers' | 14:05 |
| haleyb | Ping list: ykarel, mlavalle, mtomaska, slaweq, tobias-urdin, lajoskatona, haleyb, ralonsoh | 14:05 |
| mlavalle | \o | 14:06 |
| cardoe | o/ | 14:06 |
| haleyb | i know rodolfo was online earlier but don't see him now | 14:06 |
| mtomaska | o/ | 14:06 |
| slaweq | o/ | 14:06 |
| ralonsoh | hi (late) | 14:06 |
| TheJulia | o/ | 14:07 |
| ralonsoh | sorry, is the drivers meeting ongoing now? | 14:07 |
| haleyb | ralonsoh: yes, i was late as well, just started | 14:07 |
| ralonsoh | cool | 14:07 |
| haleyb | we had a few things on the list | 14:08 |
| haleyb | first one was from dsan | 14:08 |
| haleyb | #link bugs.launchpad.net/neutron/+bug/2123836 | 14:08 |
| haleyb | there is also a patch | 14:08 |
| haleyb | i'm not sure they are here, can move to second item | 14:09 |
| dsan | o/ | 14:09 |
| cardoe | lol ran in as the doors closed | 14:10 |
| haleyb | dsan: hi, i had commented in the bug and think others had looked at the change | 14:10 |
| slaweq | I just looked at it now | 14:10 |
| slaweq | and it looks ok for me | 14:10 |
| ralonsoh | same for me, being this option configurable and disabled by default | 14:11 |
| ralonsoh | just a question: is it possible to have something else apart from unit tests? | 14:11 |
| ralonsoh | something that spawns dnsmasq with this option and other agent making this query? | 14:12 |
| haleyb | my only comment (in the bug) is should the dhcp-agent be using this as well as liveness check, and restart any dnsmasq that doesn't respond? | 14:12 |
| haleyb | but if others are fine with it... | 14:12 |
| ralonsoh | haleyb, we never consider that | 14:13 |
| ralonsoh | but I think that could be a follow up RFE | 14:13 |
| slaweq | haleyb: IMO this could be potential follow up improvement | 14:13 |
| haleyb | sure, that would work. it's not something you can put on a dashboard but could help | 14:14 |
| haleyb | ralonsoh: regarding your testing ask, that would be good - dsan is that doable? | 14:15 |
| slaweq | I'm not sure what you want to test there? Wouldn't it be testing of dnsmasq itself actually? | 14:16 |
| slaweq | do we need that really? | 14:16 |
| ralonsoh | yes, that will test dnsmasq | 14:16 |
| dsan | meaning like adding a dns check ? | 14:17 |
| ralonsoh | yes | 14:17 |
| dsan | guess it's doable | 14:17 |
| dsan | there's not much input validation as is | 14:18 |
| haleyb | so more a functional test | 14:18 |
| dsan | i was wondering if and how to do that | 14:18 |
| dsan | inside dnsmasq it's only string manipulation | 14:19 |
| dsan | not really if it's a valid txt record | 14:19 |
| dsan | right now if we feed a wrong config value | 14:20 |
| dsan | dnsmasq won't start and output an error | 14:20 |
| dsan | not sure if it's acceptable | 14:20 |
| dsan | or if a preflick check is needed | 14:20 |
| slaweq | IMO it is acceptable | 14:21 |
| dsan | ok | 14:21 |
| haleyb | i'm thinking just the positive check would be necessary, i.e. start dnsmasq and use dig to get the txt message and verify it's what it should be? | 14:22 |
| dsan | then I look into adding a little dns test | 14:22 |
| haleyb | ralonsoh: is that what you were thinking? | 14:22 |
| ralonsoh | yes | 14:22 |
| haleyb | great, so we should vote, +1 from me | 14:23 |
| mlavalle | +1 | 14:23 |
| slaweq | +1 | 14:23 |
| ralonsoh | +1 | 14:23 |
| haleyb | ok, i'll mark it approved, and thanks for working on it dsan | 14:23 |
| haleyb | ralonsoh: the next topic was yours | 14:24 |
| ralonsoh | thanks | 14:24 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/1970944 | 14:24 |
| ralonsoh | So the goal is to replace the keepalived-state-change script for something not so heavy (in RAM terms) | 14:25 |
| ralonsoh | we already discussed that in the PTG | 14:25 |
| ralonsoh | initially we were using keepalived notify_scripts | 14:25 |
| ralonsoh | but we move to the python script in https://review.opendev.org/c/openstack/neutron/+/125384 because the errors present when multiple quick transitions happened | 14:26 |
| ralonsoh | however this error was still present in next released | 14:26 |
| ralonsoh | this is why I proposed https://review.opendev.org/q/I70037da9cdd0f8448e0af8dd96b4e3f5de5728ad (6 years ago) | 14:26 |
| opendevreview | Merged openstack/neutron master: Revert "[eventlet-removal] Handle stop DHCP agent" https://review.opendev.org/c/openstack/neutron/+/967808 | 14:27 |
| ralonsoh | so the point is going back to the notify_scripts, but this time making it configurable (for a period of time) | 14:27 |
| ralonsoh | so a user will be able to choose between python script or notify_scripts | 14:27 |
| ralonsoh | if we switch to notfy_scripts by default and that works, we can remove the python script implementation | 14:28 |
| ralonsoh | POC: https://review.opendev.org/c/openstack/neutron/+/965909 | 14:28 |
| ralonsoh | 2 questions: | 14:28 |
| ralonsoh | 1) Do you agree with this strategy | 14:28 |
| ralonsoh | 2) I'm not going to implement it, someone have time for this? Someone interested in the ML2/OVS backend | 14:28 |
| haleyb | I agree we should fix the memory issue, but also cannot implement it | 14:30 |
| slaweq | I like this new approach, I think it should be better | 14:31 |
| ralonsoh | so if we vote for it, haleyb will you send a mail to ask for volunteers? | 14:32 |
| haleyb | so you remember who noticed the memory issue? not to task them but just so they can be involved | 14:32 |
| ralonsoh | zigo ^ | 14:32 |
| haleyb | ralonsoh: yes, i could send something to the ML | 14:32 |
| haleyb | he might be offline | 14:34 |
| * zigo is looking this up | 14:34 | |
| ralonsoh | from PTG etherpad: (zigo) Some numbers from our public cloud related to that: in a random network node in production, we have 673 instances of this process. That's 65 GB of RAM | 14:35 |
| zigo | ralonsoh: It's be nicer if the neutron/agent/l3/keepalived-state-change.sh file was located in /usr/bin and had "neutron-" at the begining of its name. | 14:35 |
| zigo | Also, you're not removing the older script, are you? | 14:35 |
| ralonsoh | this is another alternative | 14:35 |
| ralonsoh | we are discussing using notify_scripts (form keepalived) again | 14:36 |
| ralonsoh | that will be much easier | 14:36 |
| ralonsoh | (ok, you are already talking about https://review.opendev.org/c/openstack/neutron/+/965909/1/neutron/agent/l3/keepalived-state-change.sh) | 14:37 |
| lajoskatona | o/ | 14:37 |
| zigo | Yeah, I was. | 14:37 |
| ralonsoh | so yes, that could be changed. The final location of the file depends on the toml file, that moves this file where needed | 14:37 |
| zigo | I'm just not a fan of having executed files in private directories. Also, the distro policies say you shouldn't use ".sh" as extension: from a user standpoint, the implementation language has no value, and therefore the extension shouldn't be used. | 14:38 |
| zigo | If you don't do all of this, I'll have to cary a Debian specific patch, which is very annoying. | 14:39 |
| zigo | (these are just pieces of advices on the implementation... nothing blocking though) | 14:39 |
| haleyb | zigo: well, we are also looking for someone to do the work | 14:40 |
| zigo | About using keepalived own implementation: I have no idea if it made progress over the last 10 years or not. | 14:40 |
| zigo | Though it worries me that its call was removed in 2014, and we don't know why. | 14:40 |
| lajoskatona | zigo: but the basic idea to replace python with shell script is acceptable from distro maintaner perspective and can be accepted? | 14:40 |
| zigo | Yeah, I like the idea. | 14:40 |
| lajoskatona | ak | 14:41 |
| lajoskatona | ack | 14:41 |
| haleyb | or have the choice? i think that's what ralonsoh said above? i.e. a config option | 14:41 |
| zigo | An implementation in a better language could be cool too (C / Rust anymore ? :P ) | 14:41 |
| ralonsoh | initially we can implement both and make it configurable | 14:42 |
| zigo | Maybe on a side proect to depend on? | 14:42 |
| ralonsoh | but there is not need for this | 14:42 |
| ralonsoh | keepalived notify_script is expecting a simple script, for example a bash script | 14:42 |
| ralonsoh | nothing complex (actually we need to write a file and send a socket message) | 14:43 |
| zigo | Well, I still don't like the fact that it doesn't scale. Reducing the memory footprint is cool, but what if I get 6000 instances instead of 600 ? | 14:43 |
| zigo | I'd say it's a good and fast approach to fix things quick. | 14:43 |
| ralonsoh | sorry, I dont' understand | 14:44 |
| zigo | (well, to tell the truth: if I had 6000 HA router on a network node, I'd have other issues too ... :P) | 14:44 |
| ralonsoh | if you have 6000 instances you will still need 6K instances of keepalived, not any python script to monitor the state change | 14:44 |
| lajoskatona | what if we say that here's a reference implementation in shel, this is the interface in the documentation how it should behave do it yourself if you need something more | 14:44 |
| slaweq | lajoskatona: you mean to make that notify script configurable? | 14:45 |
| slaweq | and give operators way to bring their own one | 14:46 |
| zigo | ralonsoh: The point I'm making is that the current situation where we spawn a long-lived daemon forever isn't ideal, because (not depending on how much memory it takes) it's going to always grow... | 14:46 |
| slaweq | if yes, I like that idea | 14:46 |
| lajoskatona | slaweq: yes,something like that | 14:46 |
| ralonsoh | zigo, what daemon? | 14:46 |
| zigo | The current script no? | 14:46 |
| ralonsoh | no | 14:46 |
| ralonsoh | notify_scripts doens't work like this | 14:46 |
| zigo | Is it spawned, then dies? | 14:46 |
| ralonsoh | yes | 14:47 |
| zigo | Ok, then I shut up. :P | 14:47 |
| ralonsoh | keepalived will call it when active/backuop event happens | 14:47 |
| ralonsoh | and the execution should take milliseconds | 14:47 |
| zigo | Then maybe, my idea of writing it in something less heavy than bash isn't that stupid. | 14:48 |
| zigo | ralonsoh: You'll be surprised how heavy bash is. | 14:48 |
| zigo | If you do something in Bash, I'll write an MR to do it at least in dash... :P | 14:48 |
| zigo | It will spawn at least 3 times faster. | 14:48 |
| zigo | Anyways, yeah, all my support for this ! | 14:49 |
| ralonsoh | as commented before, we need someone to take the lead on this implementation | 14:49 |
| ralonsoh | I won't be able to continue | 14:49 |
| slaweq | I think that we can use "sh" maybe instead of bash but I wouldn't go with something like dash there | 14:49 |
| ralonsoh | so if you are able to do this or someone else you know, that wil be perfect | 14:49 |
| zigo | slaweq: /bin/sh in Debian is dash, which is why I'm writing this. | 14:50 |
| lajoskatona | yes we have to select something that first works in the CI :-) | 14:50 |
| zigo | dash == iso shell, no bashism involved. | 14:50 |
| slaweq | ok, I didn't know that | 14:50 |
| ralonsoh | ok, let's keep the implementatikon for the patch | 14:50 |
| haleyb | ok, we still have one more item on agenda so should finish up | 14:50 |
| ralonsoh | we should vote for this idea | 14:50 |
| haleyb | still need someone to own it | 14:50 |
| zigo | ralonsoh: Your previous bash implementation was really fine, IMO. | 14:50 |
| zigo | At least as a first approach. | 14:50 |
| ralonsoh | that is discarded now | 14:51 |
| zigo | :/ | 14:51 |
| haleyb | i vote to go forward with this, pending an owner | 14:51 |
| slaweq | +1 | 14:51 |
| ralonsoh | +1 | 14:51 |
| lajoskatona | +1, I check with my management (I have anyway long discussions with them in the recent weeks....) | 14:52 |
| zigo | I'm +1 on anything that fixes the current situation, whatever implementation that is. | 14:52 |
| mlavalle | +1 | 14:52 |
| opendevreview | Bodo Petermann proposed openstack/neutron master: Allow plugins to add periodics to maintenance worker https://review.opendev.org/c/openstack/neutron/+/939817 | 14:52 |
| haleyb | ok, so i guess we can change the existing bug to have rfe tags, and i can send an email to list asking for an owner | 14:52 |
| lajoskatona | +1 | 14:53 |
| haleyb | ok, last topic was from cardoe | 14:53 |
| haleyb | and sorry for not having a lot of time | 14:53 |
| cardoe | Well happy to do what I can. | 14:53 |
| cardoe | I can throw a meetpad link but Ironic boxes will likely take longer than 5 minutes to boot all the way up | 14:54 |
| ralonsoh | it's ok, no problem | 14:55 |
| TheJulia | The ask was for some sort of demo to advance the basic understanding... so maybe why not and just keep going in a separate forum ? | 14:55 |
| ralonsoh | we can close the drivers meeting and join the meetpad | 14:55 |
| cardoe | https://meetpad.opendev.org/neutron-vxlan-demo | 14:55 |
| lajoskatona | I am on mobile-net in a car currently, but I try.... | 14:55 |
| haleyb | alright i will end this meeting then, i don't have a conflict for 30 minutes | 14:56 |
| haleyb | #endmeeting | 14:56 |
| opendevmeet | Meeting ended Fri Nov 21 14:56:58 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:56 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2025/neutron_drivers.2025-11-21-14.05.html | 14:56 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2025/neutron_drivers.2025-11-21-14.05.txt | 14:56 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2025/neutron_drivers.2025-11-21-14.05.log.html | 14:56 |
| opendevreview | Merged openstack/neutron master: [OVN] The external networks GW chassis must the same as the GW LRP https://review.opendev.org/c/openstack/neutron/+/962155 | 15:11 |
| opendevreview | Jakub Libosvar proposed openstack/ovsdbapp master: Add Neutron functional test to the gate queue https://review.opendev.org/c/openstack/ovsdbapp/+/968031 | 15:23 |
| haleyb | and i think i killed that meeting | 16:01 |
| cardoe | Thanks everyone for taking the time with us today. | 16:26 |
| cardoe | TheJulia: I didn't kill the call did I? | 16:33 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Add event to create BGP chassis resources https://review.opendev.org/c/openstack/neutron/+/961854 | 16:57 |
| TheJulia | cardoe: no, you didn't. it was all good | 17:15 |
| noonedeadpunk | hey folks - any chance to get this reviewed? https://review.opendev.org/c/openstack/neutron/+/962985 | 17:35 |
| noonedeadpunk | this one actually fixes the issue which ralonsoh attempted to fix as well | 17:36 |
| noonedeadpunk | but also would be finally to fix auto-allocations docs: https://review.opendev.org/c/openstack/neutron/+/936643 | 17:37 |
| noonedeadpunk | *would be nice | 17:38 |
| opendevreview | Merged openstack/neutron master: [SGL] Ignore port groups that don't come from SGs https://review.opendev.org/c/openstack/neutron/+/967583 | 18:07 |
| opendevreview | Merged openstack/neutron master: [doc] Update plugins needed for auto-allocation https://review.opendev.org/c/openstack/neutron/+/936643 | 19:00 |
| opendevreview | Merged openstack/neutron master: [OVN] Sync the LRP Gateway_Chassis with the network HCG https://review.opendev.org/c/openstack/neutron/+/964381 | 20:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Maintenance method to update the LRP from GC to HCG https://review.opendev.org/c/openstack/neutron/+/947906 | 21:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use HA_Chassis_Group in the OVN L3 scheduler https://review.opendev.org/c/openstack/neutron/+/947317 | 21:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove check support for NB ``Gateway_Chassis`` table https://review.opendev.org/c/openstack/neutron/+/965351 | 21:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Remove ``_add_gateway_chassis`` auxiliary method https://review.opendev.org/c/openstack/neutron/+/965354 | 21:08 |
| opendevreview | Brian Haley proposed openstack/neutron master: Add ovn-agent to metadata caching doc https://review.opendev.org/c/openstack/neutron/+/968080 | 21:18 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change some common test code to use project_id https://review.opendev.org/c/openstack/neutron/+/968083 | 21:31 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change some unit test code to use project_id https://review.opendev.org/c/openstack/neutron/+/968084 | 21:31 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change some functional test code to use project_id https://review.opendev.org/c/openstack/neutron/+/968085 | 21:31 |
| opendevreview | Merged openstack/neutron-vpnaas master: Fix report_status function in ovn setups https://review.opendev.org/c/openstack/neutron-vpnaas/+/963493 | 21:52 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!