Tuesday, 2024-11-12

opendevreviewIhar Hrachyshka proposed openstack/neutron-lib master: Enable mypy for pep8 tox target  https://review.opendev.org/c/openstack/neutron-lib/+/92148400:32
opendevreviewMerged openstack/neutron master: mypy: enable for all non-test modules  https://review.opendev.org/c/openstack/neutron/+/92986603:01
*** f0o_ is now known as f0o06:21
opendevreviewRodolfo Alonso proposed openstack/networking-sfc master: Fix reference to ``TestModelsMigrations`` class  https://review.opendev.org/c/openstack/networking-sfc/+/93465707:03
opendevreviewRodolfo Alonso proposed openstack/neutron-fwaas master: Use ``alembic.op.alter_column``  https://review.opendev.org/c/openstack/neutron-fwaas/+/93464507:04
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: Use ``alembic.op.alter_column``  https://review.opendev.org/c/openstack/neutron-vpnaas/+/93464407:05
opendevreviewRodolfo Alonso proposed openstack/networking-bgpvpn master: Fix reference to ``TestModelsMigrations`` class  https://review.opendev.org/c/openstack/networking-bgpvpn/+/93465807:05
ralonsohslaweq, hello! please check https://review.opendev.org/c/openstack/neutron/+/934401, in order to close all open (and valid) backports to 2023.1 and propose a releases patch07:39
ralonsohthanks in advance!07:39
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Do not update a deleted Logical_Switch_Port  https://review.opendev.org/c/openstack/neutron/+/93465208:21
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: DNM == [stable-only] Do something with the agent query........  https://review.opendev.org/c/openstack/neutron/+/93473310:01
slaweqralonsoh hi, https://review.opendev.org/c/openstack/neutron/+/934401 approved10:37
ralonsohslaweq, thanks!10:46
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: DNM == [stable-only] Do something with the agent query........  https://review.opendev.org/c/openstack/neutron/+/93473310:50
opendevreviewyatin proposed openstack/neutron-lib stable/2024.2: Skip pylint recommendation "too-many-positional-arguments"  https://review.opendev.org/c/openstack/neutron-lib/+/93473611:10
opendevreviewyatin proposed openstack/neutron-lib stable/2024.2: [2024.2 only] Switch to branched neutron-tempest-plugin jobs  https://review.opendev.org/c/openstack/neutron-lib/+/93466711:11
rpittauhello everyone!when testing the migration to ubuntu noble in ironic we've spotted an  issue compiling the neutron_plugins, anything you're aware about?11:28
rpittaumore info here https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6e5/934150/9/check/ironic-tempest-ovn-uefi-ipmi-pxe/6e507d1/job-output.txt11:28
ralonsohrpittau, let me check what ovs/ovn versions are you using11:34
ralonsohrpittau, better use one of these:11:35
ralonsohOVN_BRANCH: branch-24.03  OVS_BRANCH: branch-3.311:35
ralonsohOVN_BRANCH: "v23.09.0"  OVS_BRANCH: "v3.2.0-20-g1d78a3f31"11:35
ralonsohthe second one is used in our neutron-tempest-plugin jobs11:36
rpittauralonsoh: thanks for checking! I'll try with that11:36
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: DNM == [stable-only] Do something with the agent query........  https://review.opendev.org/c/openstack/neutron/+/93473311:42
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: DNM == [stable-only] Do something with the agent query........  https://review.opendev.org/c/openstack/neutron/+/93473312:04
opendevreviewMerged openstack/neutron stable/2023.1: [OVN] Create the SG rules revision number registers  https://review.opendev.org/c/openstack/neutron/+/93440112:22
ralonsohhi folks, some releases patches to review:12:36
ralonsoh* https://review.opendev.org/c/openstack/releases/+/93466112:36
ralonsoh* https://review.opendev.org/c/openstack/releases/+/93448012:36
*** haleyb|out is now known as haleyb13:53
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Nov 12 14:00:25 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
elvirao/ hi14:00
obondarevo/14:00
ralonsohhello14:00
ykarelo/14:00
lajoskatona1o/14:01
ihrachyso/14:01
s3rj1khi14:01
haleybhi everyone, i'll get started14:01
haleyb#announcements14:01
slaweqo/14:02
haleybi've been out a couple days hopefully it was quiet14:02
rubasovo/14:02
haleybralonsoh: i did notice the release patches, thanks for reminding me14:02
ralonsohyw14:02
haleyb#link https://releases.openstack.org/epoxy/schedule.html14:03
haleybit's week R-20 for Epoxy14:03
haleybEpoxy-1 milestone: November 14th, 202414:03
haleybso in a couple of days14:04
haleyb2025.1 Epoxy final release: April 2nd, 202514:04
haleybso a while to go14:04
haleyb2023.1 is transitioning to unmaintained, one of those release patches is for that, i'll make sure the backlog is clear before approving14:05
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:05
haleybI will send an email by thursday if canceled14:06
haleybLet's continue to use the priorities dashboard for patches in the "ready to merge" state, it has worked well so far - see wiki for link (wiki does not allow url shorteners)14:06
ralonsohthere are many RFEs opened last week14:06
ralonsohI don't know if the authors know about the process14:06
ralonsohcould be better to inform about this in the LP bugs14:07
s3rj1kI've opened 3 RFEs, topic of interest right now is one of them 14:07
haleybralonsoh: ack, i will respond in the bugs, so we will probably have a meeting this friday14:07
haleybi have not triaged any of them, was really off-offline14:08
haleybi did finally send the summary from our epoxy PTG meeting14:09
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/ITHWPHWJHRHRXMNMGDESJL4YAU5Y2JFB/14:09
ralonsohthanks!14:09
s3rj1khaleyb: one of them is on agenda of today's meeting, just in case14:09
haleybplease comment if i missed or mis-stated anything14:09
haleybs3rj1k: ack, will get to it in on-demand14:10
haleybi did have a couple of things to talk about regarding the PTG notes, both regarding large removal items14:11
haleybfirst was regarding IPv6 prefix delegation14:11
haleybihrachys has sent a patch removing it from tree14:11
ralonsohdidn't we have an alternative binary?14:12
haleyb#link https://review.opendev.org/c/openstack/neutron/+/93428314:12
ihrachysralonsoh: elaborate?14:13
haleybralonsoh: i'm not sure, i know there was the option of OVN doing something, but i had a more basic qquestion14:13
lajoskatona1I played with kea14:13
ralonsohright, kea was the proposed alternative14:13
ralonsohanother backend replacing dibbler14:13
lajoskatona1but kea is not for this usecase and misses a lot of things14:13
lajoskatona1but the community is quite responsive and active, so good point for Kea as DHCP backend :-)14:14
ralonsohyes, ovn supports prefix delegation14:14
haleybwhen i was doing my write-up, i didn't see any conclusion in this section regarding removing all the code, and my memory has already faded, so wanted to make sure that was the consensus14:15
ralonsohso is there no interest right now to implement the OVN part?14:15
slaweqI know we had some questions d/s about supporting this feature so do we really want to remove it completely now? 14:15
ihrachyswhat does "completely" mean here? it's still in neutron-lib as an option, a driver can implement it.14:17
haleybihrachys: removing dibbler would be partial, removing IPv6 PD would be completely IMO14:18
lajoskatona1you mean "in neutron-lib: that the api extension?14:18
ralonsohthere is also the code in the Neutron API14:18
ihrachyshaleyb: you refer to ipam parts in that patch?14:19
ralonsohyes, for example14:19
ralonsohand db_base_plugin_v2.py14:19
slaweqby "completely" I meant removal e.g. related bits from the neutron/db/db_base_plugin_v2.py or ipam 14:19
haleybihrachys: yes, and db and dhcp, etc14:19
ihrachysyeah gotcha, I guess I am too trigger happy. I will split this out.14:20
ralonsoh+114:20
haleybright, we shouldn't make it too hard to add a replacement if someone wants to work on it14:21
lajoskatona1+114:21
frickleris "experimental" the same as "deprecated and going to be removed"?14:22
ihrachysI understand the concern. Not that it's hard to revert afterwards if there's interest, but I see that db / ipam layers can be in theory already used elsewhere (though we don't know about it)14:22
ihrachysfrickler: effectively yes, the name is unfortunate14:22
ihrachysit's a cemetery :p14:22
ralonsohthat name was decided because everything under experimental was still tested in the CI but not supported14:23
slaweq"cemetery" is much better name to reflect it's purpose14:23
ralonsohand if the CI started to fail, then the decision was to remove it14:23
rpittauralonsoh: changing OVS and OVN branches worked as a charm, thanks!14:25
ralonsohyw14:25
haleybso to loop back - ihrachys will remove traces of dibbler from the tree14:28
ihrachys+ will update the patch this week to reduce the scope as you suggested14:29
haleyback, ok my second question about the PTG was regarding Linuxbridge, so similar question14:30
haleybi know this has been in the "cemetary" for a while too14:30
ralonsohWe don't have an official migration document, from LB to other backend14:31
ralonsohthat could be a stopper for some users14:31
ralonsohin any case, the first thing should be to communicate this decision in the mailing list and wait for feedback14:31
haleybralonsoh: right, i think that's something maybe we can help with14:31
ralonsoh(not too long)14:31
ralonsohhaleyb, but that is something not easy, to be honest14:32
ralonsohwe can "recommend" this external guide, I don't remember the author14:32
ralonsohthat is moving from LB to OVN, if I recall correctly14:32
haleybif we can communicate waht state the driver is in (untested and unmaintained), and the desire to move to OVN, maybe that will get people using it to look at a transition14:33
haleyband bring up unanswered questions, etc14:33
ralonsohhttps://www.jimmdenton.com/migrating-lxb-to-ovn/14:33
ihrachyswe should probably have a guide / reference even if we don't drop the code14:33
opendevreviewMerged openstack/neutron-lib stable/2024.2: Skip pylint recommendation "too-many-positional-arguments"  https://review.opendev.org/c/openstack/neutron-lib/+/93473614:33
slaweqI'm not sure but maybe it was noonedeadpunk who made this guide, no?14:34
ralonsoh^^ the link is there14:34
ralonsohhttps://www.jimmdenton.com/migrating-lxb-to-ovn/14:34
haleybso i will write something up, with a "strongly recommend migration" due to these reasons14:35
haleyband i do realize there could be unhappy people14:36
haleybso i will work on that, might send it to 1-2 of you for review first14:36
noonedeadpunkit was jamesdenton14:37
slaweqhaleyb there will be unhappy people I guess but from the other hand we were raising this for many years now and nobody really took care of this backend14:37
noonedeadpunkslaweq: nobody will as long as it works14:37
slaweqthx noonedeadpunk, ralonsoh already found it :)14:37
noonedeadpunkand it works "just fine" so far...14:38
haleybslaweq: right, and at least we know CERN is looking at migration14:38
ihrachysthe code can always be spun off into a separate project; it's not like we ban interfaces for linuxbridge.14:38
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: [stable-only] Group by using the same filtered fields  https://review.opendev.org/c/openstack/neutron/+/93473314:38
noonedeadpunkwell, we are looking for migration from OVS and the path seems to be very tricky so far. If possible.14:38
lajoskatona1perhaps this will push NASA also over the treshold to change to something else :üÖ14:39
noonedeadpunkI think same is gonna happen with LXB14:39
noonedeadpunkmainly due to workloads that have disabled port security but running VRRP inside of VMs14:39
slaweqnoonedeadpunk if that would be the case, then let's see who will step in to do https://bugs.launchpad.net/neutron/+bug/2087947 - I believe this may be one of those things which really has to be done if this backend would be still there14:39
haleyblift and shift is sometimes the only answer14:39
haleybslaweq: i will even mention such bugs in my email14:40
noonedeadpunkI was thinking eventlet removal is actually after 2025.1? At least iirc Nova said it was for them14:41
noonedeadpunkto produce another "stable" SLURP14:41
noonedeadpunkas I guess driver removal is not appropriate right now anyway?14:41
ihrachysnoonedeadpunk: the argument is that the deprecated state of linuxbridge was communicated by putting it behind experimental knob, so it's fair game.14:43
noonedeadpunkbut I'd guess this might be also very tricky to replace? https://github.com/openstack/neutron/blob/fa0f8570106257a88275ba59954a509dbf096d0b/neutron/agent/dhcp/agent.py#L13614:43
haleybwe have dug the hole for linuxbridge it's just hard to finsh it off. maybe any email will start a dialogue14:43
ralonsohnoonedeadpunk, not at all, we are planning to migrate the user threads to kernel threads14:44
ralonsohwe'll find issues, mainly in the API part, but we'll solve it14:44
haleybi think we have a plan for these two items, should move onto bugs with only :15 left14:44
slaweq+1 for email - lets send it and see what the reaction will be again14:44
ihrachyslong story short, haleyb will send email and get the ball rolling and we'll see where chips fall14:44
haleybmaybe it's only used with yoga :)14:45
lajoskatona1agree, letás send out the email, and wait ÉüÖ14:45
noonedeadpunklxb case in bug report looks like async thing, but yeah... again - not saying to keep lxb. but potentially easier migration would be to ovs if you're gonna keep/sovle it14:45
haleyb#topic bugs14:45
haleybelvira was the deputy last week, her report is here14:45
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/QIIS7YTCVWYR64BRK7UGASVGU4AGQ3OE/14:45
haleybthere were some unassigned14:46
haleybor at least there were when it was sent14:46
haleyb#link https://bugs.launchpad.net/neutron/+bug/208782414:47
haleybrandom router related test failures since wsgi switch14:47
haleybbut i see there is a patch now14:47
ralonsohI'm on this one, yes14:47
ralonsohand a testing patch on top of it14:47
haleyb#link https://review.opendev.org/c/openstack/neutron/+/93465214:47
haleybok next is14:47
haleyb#link https://bugs.launchpad.net/neutron/+bug/208782214:48
haleybWhen security group rules have the same normalized CIDR, neutron-ovn-db-sync-util has a wrong log message14:48
haleybmentions incorrect message, but think we patched this14:48
haleyb#link https://review.opendev.org/c/openstack/neutron/+/89079914:49
elvirahaleyb: not sure if you mean the patch I linked, but reporter should have that in given their version14:49
slaweqI will try to check it as I was doing this "normalized_cidr" thing in the past14:49
elvirayes, that is the one14:49
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Handle trimming of vlan interface namings  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/91065414:50
haleybslaweq: ack, i can take it if you get too busy, we use that script a lot14:50
slaweqif you have cycles, feel free to do it14:50
slaweqif it will be unassigned next week I can take it then, but if you can earlier, would be great14:51
haleybi should be able to get to it14:51
haleybnext is14:51
haleyb#link https://bugs.launchpad.net/neutron/+bug/208678714:51
haleybneutron fails with postgres on agents14:51
ralonsohI'm on this one14:51
ralonsohbut this is the last postgresql bug I fix14:51
haleyback, thanks, we should have retired it sooner :)14:52
ralonsoh#link https://review.opendev.org/c/openstack/neutron/+/93473314:52
haleyback, thanks14:53
haleybthere were also a few RFEs that i will need to triage, not sure if any saw or commented on them14:54
haleybthat was all i had for bugs14:54
haleybslaweq is the deputy this week, i am next week14:55
noonedeadpunkralonsoh: can you please elaborate a bit on what documentation (and maybe where to place it?) you have in mind for https://review.opendev.org/c/openstack/neutron/+/931495 ?14:55
slaweqack, I'm on it already14:55
haleybthanks14:55
haleybmoving on14:55
haleyb#topic community goals14:55
ralonsohnoonedeadpunk, once the meeting is over14:55
haleybralonsoh: it looks like you started creating bugs for evenlet removal?14:55
ralonsohyes, all the references in 14:56
ralonsoh#link https://etherpad.opendev.org/p/neutron-eventlet-deprecation14:56
ralonsohbut I still need to open the migration to ASGI, that is something that, IMO, should be a consensual decision in the community14:56
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Use neutron_lib constants a device name limiter  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93432714:56
ralonsohI'll attend to the oslo meetings and the eventlet-deprecation ones14:57
ralonsohthat's all14:57
slaweqI marked them all today with "eventlet-deprecation" tag so can be easily found: https://bugs.launchpad.net/neutron/+bugs?field.tag=eventlet-deprecation14:57
haleybralonsoh: ack, thanks14:57
ralonsoh+114:57
lajoskatona1thanks14:57
haleyb#topic on-demand14:58
haleybs3rj1k: i noticed you had added something, although it is an RFE14:58
haleyb#link https://bugs.launchpad.net/neutron/+bug/208677614:59
haleybit might be something more for friday drivers meeting14:59
s3rj1khaleb: yea, I am willing to take this up in case there are no hard objections to proposed RFE14:59
haleybs3rj1k: we would have to discuss and approve in friday's meeting, same time as this one15:00
s3rj1khaleb: got it, thanks15:00
haleybok, will defer that to friday15:01
haleybany other topics, we are over time15:01
haleybok15:02
haleyb#endmeeting15:02
opendevmeetMeeting ended Tue Nov 12 15:02:20 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:02
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-11-12-14.00.html15:02
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-11-12-14.00.txt15:02
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-11-12-14.00.log.html15:02
lajoskatona1o/15:02
ralonsohbye15:02
haleybthanks for attending everyone15:02
elvirao/15:02
mlavalle\o15:02
obondarev o/15:02
slaweqo/15:04
opendevreviewMerged openstack/neutron-lib stable/2024.2: [2024.2 only] Switch to branched neutron-tempest-plugin jobs  https://review.opendev.org/c/openstack/neutron-lib/+/93466715:25
noonedeadpunkralonsoh: so, about https://review.opendev.org/c/openstack/neutron/+/931495  :)16:09
noonedeadpunkyou want to document on how OVN NB DB tables is gonna differ in case of router which is created with external network from geneve?16:10
noonedeadpunkor was that some other aspect?16:11
noonedeadpunkI was actually planning to push some docs, but I also failed to find a good place for that, so I ended up in keeping some ovn-related info in osa repo, ie https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/932616/1/doc/source/app-ovn.rst16:12
opendevreviewMerged openstack/neutron-lib master: Define physical and tunnelled network types  https://review.opendev.org/c/openstack/neutron-lib/+/93414616:17
opendevreviewMerged openstack/neutron master: nit: Add info about ovn version with additional_chassis  https://review.opendev.org/c/openstack/neutron/+/93428616:18
opendevreviewMerged openstack/neutron master: refactor: Move hacking dependency from tox.ini to test-reqs  https://review.opendev.org/c/openstack/neutron/+/93468416:18
opendevreviewMerged openstack/neutron master: [Fullstack] Use only one keepalived-state-change thread in L3 agent  https://review.opendev.org/c/openstack/neutron/+/93424818:13
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Support ovn-bgp agent with DVR disabled  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93393023:26
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Support ovn-bgp agent with DVR disabled  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93393023:26

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