kyu0[m] | Hello, could you please review my comment? https://review.opendev.org/c/openstack/neutron/+/947324 | 01:25 |
---|---|---|
opendevreview | Kevin Carter proposed openstack/neutron master: feat(OVN): add new mode to only add when missing https://review.opendev.org/c/openstack/neutron/+/941824 | 03:50 |
*** ykarel_ is now known as ykarel | 04:18 | |
ralonsoh | ykarel, I'm checking https://4e87414b0fe3ca1de111-0dd3b493df614cc7e41a46754664ea83.ssl.cf5.rackcdn.com/openstack/a858fb193a21437987d557b6de06dbdc/testr_results.html | 06:01 |
ralonsoh | I'll open a LP bug | 06:01 |
ykarel | ralonsoh, ack this one took ~ 17 seconds to cleanup mac binding, and timeout is 15 sec | 06:02 |
ykarel | not sure why it took that much time | 06:02 |
ykarel | ralonsoh, another failure https://13105f8ef823650ec019-cb65abe58d87a1a010092a9adcbaff91.ssl.cf2.rackcdn.com/openstack/9f625bccccc54f9bb3b76dd44cc78bb9/testr_results.html in this it took ~ 16 seconds | 06:57 |
ralonsoh | I'll add it to the bug (still creating it) | 06:57 |
ykarel | ack i am reporting the unit test issue | 06:58 |
ykarel | reported for unit test https://bugs.launchpad.net/neutron/+bug/2115241 | 07:06 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Wait for the FIP Port_Binding event before checking MAC removal https://review.opendev.org/c/openstack/neutron/+/953195 | 07:46 |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Drop unused method https://review.opendev.org/c/openstack/neutron/+/952977 | 08:34 |
tkajinam | ^^^ seems like we are currently hitting 82% percent boundary and a small change can break it very easily... | 08:35 |
opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove constants for LBaaS https://review.opendev.org/c/openstack/neutron-lib/+/953198 | 09:20 |
opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove unused service type constant for FWaaS https://review.opendev.org/c/openstack/neutron-lib/+/953199 | 09:25 |
opendevreview | Merged openstack/neutron master: Remove compat logic for neutron-lib < 3.9.0 https://review.opendev.org/c/openstack/neutron/+/953111 | 10:21 |
*** LarsErik1 is now known as LarsErikP | 10:30 | |
zigo | haleyb: FYI, I understood what happened. The issue is in both Debian and Ubuntu (I just checked the Ubuntu package): it's not Neutron that is killing the keepalived processes, but systemd ! | 12:14 |
zigo | I will add a killmode=process in the .service to fix it. | 12:15 |
haleyb | zigo: good to hear you figured it out | 12:56 |
haleyb | #startmeeting networking | 13:00 |
opendevmeet | Meeting started Tue Jun 24 13:00:26 2025 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
opendevmeet | The meeting name has been set to 'networking' | 13:00 |
haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, haleyb, ralonsoh | 13:00 |
mlavalle | \o | 13:00 |
obondarev | o/ | 13:00 |
haleyb | hi everyone | 13:00 |
ralonsoh | hello | 13:00 |
slaweq | o/ | 13:00 |
rubasov | o/ | 13:02 |
haleyb | i'll get started | 13:02 |
haleyb | #announcements | 13:02 |
haleyb | We are currently in Week R-3 of Flamingo | 13:02 |
haleyb | Our next milestone in this development cycle will be Flamingo-2, on July 3rd | 13:02 |
haleyb | That is next Thursday 8-o | 13:03 |
haleyb | Final 2025.2 Flamingo release: October 3rd, 2025 | 13:03 |
haleyb | #link https://releases.openstack.org/flamingo/schedule.html | 13:03 |
bcafarel | late o/ | 13:03 |
haleyb | The next OpenInfra PTG[1] will take place October 27-31, 2025 and registration for the event is now open | 13:05 |
haleyb | #link https://ptg.openinfra.org/ | 13:05 |
haleyb | it's virtual and free as it has been for a while | 13: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 | 13:06 |
haleyb | and last announcement | 13:07 |
haleyb | Openstack is changing from CLA to DCO starting July 1st | 13:07 |
lajoskatona | o/ | 13:07 |
haleyb | i can't find the link to the ML thread, but it just means that signed-off-by will be mandatory in reviews | 13:08 |
haleyb | so change your workflow to 'git commit -a -s' or similar | 13:08 |
haleyb | and if anyone knows how to tweak .gitconfig to make that -s always take affect i'd love to know the trick | 13:09 |
haleyb | any other announcements? | 13:09 |
haleyb | #topic bugs | 13:10 |
haleyb | mlavalle was the deputy last week, his report is at | 13:10 |
haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/YNLD3OU44PAA5UOVKXMQDMYVUZHBIGUU/ | 13:10 |
haleyb | first one is | 13:11 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2114732 | 13:11 |
haleyb | grenade-skip-level-always job fails with: Security Group None cannot perform precommit_delete due to Callback | 13:11 |
ralonsoh | I commented there | 13:12 |
ralonsoh | we can use WSGI in 2024.2, but it wasn't stable until we removed eventlet (that happened in 2025.1) | 13:12 |
ralonsoh | this is why we don't move to wsgi until 2025.1 | 13:12 |
haleyb | ralonsoh: yes, just reading your note | 13:12 |
ralonsoh | so with 2024.2 will work, but with some issues | 13:12 |
ralonsoh | in any case, skip jobs are not mandatory in non-slurp releases (as is now) | 13:13 |
ralonsoh | so I would move this job to periodic, as we do in neutron | 13:13 |
haleyb | do we know what job this is failing with? i agree with your assessment | 13:13 |
ralonsoh | we had multiple issues in wsgi and eventlet | 13:13 |
ralonsoh | once we removed everything eventlet related from the API, everything started to work stable | 13:14 |
haleyb | oh grenade-skip-level-always i didn't know that was a job, wonder where it lives | 13:14 |
ralonsoh | in 2024.2+WSGI, the eventlet removal is not finished and we can't guarantee any stability, as we can see in the CI | 13:14 |
ralonsoh | in grenade repo | 13:15 |
haleyb | i see nova, swift and manila run it | 13:15 |
ykarel | ralonsoh, can't we remove "logger" mechanism driver from there to fix that job | 13:16 |
haleyb | manila has it non-voting | 13:16 |
ralonsoh | ykarel, I don't think this is the problem | 13:16 |
ralonsoh | in 2024.2+WSGI, we were still using eventlet | 13:16 |
ykarel | ack when i last checked that looked the issue but can cross check | 13:16 |
ykarel | iirc i mentioned that too | 13:16 |
ykarel | yes in ci meeting | 13:17 |
ykarel | <ykarel> and here i see Loaded mechanism driver names: ['ovn', 'logger'] | 13:17 |
ykarel | <ykarel> so need to remove logger mechanism driver | 13:17 |
ralonsoh | we can try that, for sure | 13:18 |
ykarel | ++ let's do | 13:18 |
ralonsoh | I'll propose a patch | 13:18 |
rubasov | sidenote on git autosignoff: https://paste.opendev.org/show/828233/ | 13:18 |
haleyb | thanks ralonsoh, and if that doesn't work can have them turn-off voting | 13:19 |
lajoskatona | https://opendev.org/openstack/tempest/src/branch/master/zuul.d/integrated-gate.yaml#L463 | 13:19 |
lajoskatona | sorry | 13:19 |
haleyb | rubasov: thanks for that, autoSignOff=true is the trick :) | 13:19 |
ralonsoh | haleyb, rubasov it is better if we integrate it with the openstack hook | 13:20 |
rubasov | plus the git hook | 13:20 |
ralonsoh | for example: https://paste.opendev.org/show/bgqKZ49CDIOpv4ieeIUv/ | 13:20 |
ralonsoh | ^^ that is at the end of the openstack hook | 13:20 |
haleyb | ralonsoh: i'm assuming someone will make a generic change to cover everything? we can worry when it's turned-on next week | 13:22 |
ralonsoh | no idea, this change can be done locally and copy/paste this hook to all repos you have | 13:23 |
ralonsoh | you only need setup your name + mail in git, that is mandatory | 13:23 |
haleyb | ack, i had done that a while back | 13:24 |
haleyb | next bug | 13:24 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2115053 | 13:24 |
haleyb | conntrack flow created for stateless SG rule | 13:24 |
haleyb | slaweq: looks like you picked that up today | 13:24 |
slaweq | yeap | 13:25 |
haleyb | ubuntu noble fails :( | 13:25 |
slaweq | I am triaging it but seems for me like ovn regression | 13:25 |
slaweq | yes, I am now trying to reproduce it on Centos 9 also | 13:26 |
slaweq | but that's still in progress | 13:26 |
haleyb | ack, i'll follow in case there is something that needs a downstream backport | 13:26 |
haleyb | all the other bugs have owners and/or patches | 13:27 |
haleyb | any others to discuss? | 13:27 |
haleyb | lajoskatona: i know we talked last week about you swapping bug deputy weeks with jlibosva, can you still take this week? | 13:28 |
haleyb | i forgot to update the calendar but will right after | 13:28 |
lajoskatona | Yes, I am on it | 13:28 |
haleyb | lajoskatona: ack, thanks! | 13:28 |
haleyb | and next week will be obondarev - is that Ok with you? | 13:29 |
obondarev | sure | 13:29 |
haleyb | great, thanks! | 13:29 |
haleyb | ok, moving on from bugs | 13:30 |
haleyb | #topic community goals | 13:30 |
haleyb | lajoskatona: any update on neutronclient? | 13:30 |
opendevreview | Merged x/whitebox-neutron-tempest-plugin master: [DVR] Add sleep after adding/removing FIPs https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/951111 | 13:31 |
lajoskatona | yes, I pushed one patch for horizon, for rbac API | 13:31 |
lajoskatona | and I checked this morning and it seems only neutron quotas remained to move to SDK (have to double check again) | 13:31 |
lajoskatona | so quite close to finish one project it seems :-) | 13:32 |
lajoskatona | the rbac policies patch (wip): https://review.opendev.org/c/openstack/horizon/+/953116 | 13:32 |
lajoskatona | that's all for SDK vs neutronclient topic | 13:32 |
haleyb | great, thanks for all the patches | 13:33 |
haleyb | ralonsoh: i'll let you talk about eventlet, it's made some progress as well | 13:33 |
ralonsoh | yes but we have a problem | 13:34 |
ralonsoh | fullstack CI | 13:34 |
ralonsoh | 1) Broken with OVS agent eventlet removal: https://review.opendev.org/c/openstack/neutron/+/937765 | 13:34 |
ralonsoh | 2) Broken with the removal of eventlet server code: https://review.opendev.org/c/openstack/neutron/+/952659 | 13:34 |
ralonsoh | I don't know how to replace the API part, to be honest | 13:34 |
ralonsoh | and the OVS agent doesn't work if executed in a eventlet process | 13:34 |
ralonsoh | so, at this point and in order to progress, I would suggest to deactivate the fullstack CI | 13:35 |
ralonsoh | and try to fix API part, if possible | 13:35 |
lajoskatona | yes I think I run into the same for the dhcp agent patch also | 13:35 |
ralonsoh | ^^ right | 13:35 |
ralonsoh | so I would suggest to migrate the jobs, as much as possible, to n-t-p | 13:36 |
ralonsoh | the fact the n-t-p and tempest work with the eventlet-less patches (dhcp, ovs, api) means these patches are working | 13:36 |
ralonsoh | but this fullstack CI is specifically tailor-made for eventlet | 13:37 |
lajoskatona | hard topic..... | 13:37 |
ralonsoh | I know | 13:37 |
ralonsoh | I think many fullstack jobs can be implemented in n-t-p | 13:38 |
haleyb | ralonsoh: ack, i guess we don't have a choice if we want to complete the work, but it will leave a gap | 13:38 |
lajoskatona | few weeks ago I mentioned that migrating to SDK also hard in fullstack as sdk has no no-auth mode..... | 13:38 |
haleyb | what do others think? | 13:38 |
ralonsoh | haleyb, yes but we can close it using neutron-tempest-plugin | 13:38 |
ralonsoh | to be honest, n-t-p is more accurate kind of testing that fullstack | 13:38 |
lajoskatona | let's check if we can migrate those tests to n-t-p and go fo it, we gain simpler gating perhaps | 13:39 |
ralonsoh | I know that for some API tests it was lighter | 13:39 |
lajoskatona | yeah in fullstack we did things like service restart, in pure tempest that is no possible | 13:39 |
haleyb | ralonsoh: understood, it's just it will take time is all | 13:40 |
ralonsoh | haleyb, right | 13:40 |
ralonsoh | lajoskatona, we can have this with tobiko, for example | 13:40 |
ralonsoh | if you don't mind (and if you agree with this, of course) | 13:41 |
ralonsoh | I'll send a mail informing about this | 13:41 |
ralonsoh | are you all OK? | 13:41 |
haleyb | i am Ok with it | 13:42 |
lajoskatona | yes, from side fine, for learning tobiko that's an interesting thing :-) | 13:42 |
ralonsoh | ok, I'll do it and I'll send a patch to remove fullstack | 13:42 |
haleyb | )-; | 13:43 |
ralonsoh | lajoskatona, send me what problems you had with sdk too | 13:43 |
lajoskatona | I can also spend some time on migrating some of the tests | 13:43 |
ralonsoh | just for documentation purposes in the mail | 13:43 |
ralonsoh | I'll create a LP bug related ^^ | 13:43 |
lajoskatona | ralonsoh: ack, I will go back to that problem to collect my fading memories | 13:43 |
ralonsoh | thanks! | 13:44 |
haleyb | alright, moving to last topic | 13:44 |
haleyb | #topic on-demand | 13:44 |
haleyb | ralonsoh: i have a follow-on fix for the metadata-agent LoopingCall code in 2025.1 - https://review.opendev.org/c/openstack/neutron/+/953064 | 13:45 |
haleyb | i broke it with my pep8 change | 13:45 |
ralonsoh | i didn't see it, let me check | 13:45 |
ralonsoh | what was the original change? | 13:46 |
haleyb | https://review.opendev.org/c/openstack/neutron/+/952561 | 13:46 |
haleyb | there was a pep8 error that i just fixed, but should have tested it | 13:47 |
haleyb | pep8 was happy but code silently failed, and it has no tests... | 13:47 |
ralonsoh | ^ but this is my patch | 13:47 |
haleyb | ralonsoh: yes, i shouldn't have fixed the pep8 issue in PS3 | 13:48 |
haleyb | your code was fine, i broke it | 13:48 |
haleyb | well, fine except for failing pep8 job | 13:49 |
ralonsoh | +2, let's merge it asap | 13:50 |
haleyb | either way it needs that change, and i proposed a similar change for all of neutron's calls to that Looping class to make sure they don't get broken | 13:50 |
haleyb | sometimes pylint is not your friend :( | 13:51 |
haleyb | that's all i had, any other topics? | 13:51 |
haleyb | ok, have a good week everyone! | 13:52 |
haleyb | #endmeeting | 13:52 |
opendevmeet | Meeting ended Tue Jun 24 13:52:15 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:52 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2025/networking.2025-06-24-13.00.html | 13:52 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2025/networking.2025-06-24-13.00.txt | 13:52 |
opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2025/networking.2025-06-24-13.00.log.html | 13:52 |
haleyb | o/ | 13:52 |
ralonsoh | bye | 13:52 |
mlavalle | \o | 13:52 |
cbuggy | o/ | 13:52 |
lajoskatona | o/ | 13:52 |
slaweq | o/ | 13:53 |
ykarel | o/ | 13:53 |
obondarev | o/ | 13:54 |
opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Deprecate unused [fwaas] agent_version https://review.opendev.org/c/openstack/neutron-fwaas/+/953212 | 14:41 |
opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Avoid unnecessary iniset_multiline https://review.opendev.org/c/openstack/neutron-fwaas/+/953215 | 14:51 |
danfai | Hi, I got a question regarding host_routes when using segments (e.g. in routed provider networks). It seems we get automatic host_routes for every single subnet. For us this means 100 host_routes, which all point to the same default gateway IP. I was wondering whether this is a bit redundant to have it or what the intended reason was to apply the default gateway only after. | 15:00 |
danfai | As the API already supports using a supernet route, I was adding the default route via gateway IP, which reduces the routes to two identical default gateway routes, which some DHCP clients/init scripts did not like either | 15:02 |
danfai | (relevant method: neutron/services/segments/plugin.py:_calculate_routed_network_host_routes ) | 15:04 |
opendevreview | Lajos Katona proposed openstack/neutron master: Delete tunnel endpoints if endpoint agent is deleted https://review.opendev.org/c/openstack/neutron/+/952988 | 15:09 |
lajoskatona1 | haleyb, ralonsoh (all): I forgot to add this governance discussion to the eventlet topic: https://review.opendev.org/c/openstack/governance/+/952903 | 15:21 |
ildikov | haleyb: mlavalle: Hi, I'm checking in about the OpenStack Ops radio Hour session I mentioned last week. Is either of you available to attend the session and talk about Neutron/networking related topics? | 15:51 |
opendevreview | Dai Dang Van proposed openstack/neutron master: Add dns forwarder l2 extension https://review.opendev.org/c/openstack/neutron/+/951390 | 17:18 |
opendevreview | Dai Dang Van proposed openstack/neutron master: Add dns forwarder l2 extension https://review.opendev.org/c/openstack/neutron/+/951390 | 17:19 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!