Tuesday, 2024-10-08

opendevreviewyatin proposed openstack/neutron master: [CI] Functional: Increase Ulimit to 4096  https://review.opendev.org/c/openstack/neutron/+/92875904:55
opendevreviewyatin proposed openstack/neutron master: [DNM] Check applied limit  https://review.opendev.org/c/openstack/neutron/+/93124704:56
opendevreviewyatin proposed openstack/neutron master: DNM: test functional jobs  https://review.opendev.org/c/openstack/neutron/+/92895305:04
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent master: Add retries to get_device_port_at_ovs  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92882206:50
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent master: Add `-O OpenFlow13` only to ovs-ofctl commands  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93174308:01
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent master: Add `-O OpenFlow13` only to ovs-ofctl commands  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93174308:02
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent master: Add `-O OpenFlow13` only to ovs-ofctl commands  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93174308:09
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent master: Add `-O OpenFlow13` only to ovs-ofctl commands  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93174308:22
opendevreviewMerged openstack/ovn-bgp-agent master: Add retries to get_device_port_at_ovs  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92882209:06
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent master: Add `-O OpenFlow13` only to ovs-ofctl commands  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93174309:45
ravlewHello, I'm encountering an issue with neutron_ovn_metadata_agent, container healthcheck after deployment comes back healthy, however after some time it becomes unhealthy.10:14
ravlewThe healthcheck in question: neutron_ovn_metadata_agent_healthcheck_test: ["CMD-SHELL", "healthcheck_port python {{ ovn_sb_db_port }}"]10:15
ravlewIs this something anyone else has encountered, in latest version?10:16
ravlewit's connected to my work on: https://review.opendev.org/c/openstack/kolla-ansible/+/90495910:17
ralonsohravlew, I don't think this is Neutron related but kolla10:40
ravlewthanks ralonsoh , I was just wondering if the connection is retained by the agent or rather used only when it is needed10:43
ravlewif it's not a constant one I will discuss with kolla guys if healthcheck should be changed10:43
opendevreviewyatin proposed openstack/neutron master: [DNM] Check applied limit  https://review.opendev.org/c/openstack/neutron/+/93124710:45
ralonsohykarel, hello! please check10:55
ralonsoh* https://bugs.launchpad.net/neutron/+bug/2083287/comments/210:55
ralonsoh* https://review.opendev.org/c/openstack/tempest/+/93176510:55
ralonsoh(I'm starting to debug the other bug: https://bugs.launchpad.net/neutron/+bug/2083570)10:56
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [Fullstack] Add active waiting for one HA agent to be active for router  https://review.opendev.org/c/openstack/neutron/+/93176610:58
opendevreviewAleksandr Chudinov proposed openstack/neutron master: [OVN] Update lsp host id when cr port is updated with chassis  https://review.opendev.org/c/openstack/neutron/+/93163211:51
opendevreviewArkady Shtempler proposed openstack/neutron-tempest-plugin master: Neutron&Designate DNS integration - some enhancements  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93018912:20
opendevreviewMerged openstack/neutron master: Ensure the OVSDB and ovn-northd process stop  https://review.opendev.org/c/openstack/neutron/+/93101513:37
ykarelralonsoh, thx13:45
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: Ensure the OVSDB and ovn-northd process stop  https://review.opendev.org/c/openstack/neutron/+/93101413:47
ykarelralonsoh, haleyb can we get https://review.opendev.org/c/openstack/neutron/+/928759 also in13:49
ralonsohsure13:49
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: [OVN] Check metadata HA proxy configuration before restart  https://review.opendev.org/c/openstack/neutron/+/93179013:54
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.1: [OVN] Check metadata HA proxy configuration before restart  https://review.opendev.org/c/openstack/neutron/+/93179113:56
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Oct  8 14:00:23 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
haleybhi everyone14:00
ralonsohhello14:00
cbuggyo/14:00
slaweqo/14:01
lajoskatonao/14:01
haleybalright, will get started14:01
haleyb#topic announcements14:01
ykarelo/14:02
haleybDalmatian has been released, long live Dalmatian!14:02
rubasovo/14:02
ralonsohcongrats everyone14:02
haleybthanks for everyone's work over the past 6 months to get that out14:02
mlavalle+114:02
elvirao/14:02
haleyband no sooner do we get one done than have to plan the next14:03
haleybEpoxy cycle PTG sessions (October 21-25) 14:03
haleyb#link https://etherpad.opendev.org/p/oct2024-ptg-neutron14:03
haleybPlease add topics, i will fill-in ones we've discussed in last few meetings14:04
haleybHave Tue/Wed/Thu/Fri (if needed) timeslots14:04
haleyband Ironic and Nova interlocks14:04
lajoskatonacool, thanks14:04
haleyb#link https://ptg.opendev.org/ptg.html has the calendar with rooms and times14:05
haleyb13:00-17:00 UTC were the times, same as last cycle, if there is an issue with the later ones let me know14:06
haleybwe have not needed all 4 days in the past few cycles14:07
haleybMonday at the PTG there are TC timeslots, assuming some might want to attend them14:08
haleybactually, i now see two timeslots on Wednesday for eventlet-removal, so will try and work around those as well14:09
haleybany other questions about the PTG? this is not anyone's first rodeo :)14:10
haleybok14:10
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:11
haleybwe did not have quorom last week, so i plan on having one this week14:11
mlavalleack14:11
slaweqsure14:11
lajoskatonaack14:12
haleybjust one item at the moment regarding floating IP NAT behavior14:12
haleyb#link https://bugs.launchpad.net/neutron/+bug/208321414:12
haleybLet's continue to use the priorities dashboard for patches in the "ready to merge" state. This could be older changes as well as new ones14:13
haleyb#link https://tinyurl.com/59ubjsv514:14
haleybfull link in the wiki, but too long for pasting14:14
haleybi had no other topics, anything I missed?14:15
haleyb#topic bugs14:16
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/6QCFKHHXGKQZVY3I6YTZR7R7NC3JZ4RV/14:16
haleybralonsoh was the deputy last week14:16
haleybthere were a lot of bugs14:17
ralonsoh(I should be paid more!)14:17
haleybI can pay in beer when we see each other in person :)14:18
ralonsohsorry, should I present the bug list?14:18
haleybi can if you like, up to you, at least the unassigned ones14:18
ralonsohthere some, yes14:19
ralonsohactually some of them are assigned now14:19
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/208360914:19
ralonsohif I'm not wrong, slaweq is working on this one14:19
ralonsohyes, I'll check the assignee 14:19
ralonsohthe next one14:20
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/208368214:20
ralonsohThis is related to a feature already commented14:20
ralonsohI pushed an improvement: https://review.opendev.org/c/openstack/neutron/+/92996714:20
ralonsohbut this seems not to be enough for the Nova API with too many VMs14:20
ralonsohI'll take it because is related to a feature I implemented14:20
haleybright. can we just suggest to use the neutron api to get neutron SGs?14:21
ralonsohI need to reproduce the Nova call14:21
ralonsohjust to know what API call are we issuing14:21
haleybthanks for taking it14:21
ralonsohif the neutron client takes so little time, maybe we can improve that14:21
ralonsohnext one14:22
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/208357014:22
ralonsohah no, I'm investigating this one right now14:22
ralonsohthis is happening in the tempest ipv6 jobs14:22
ralonsohsince the wsgi migration14:22
ralonsohto be honest, I don't know yet what is happening there, but I'll continue working on it14:22
ralonsohthere are 2 bugs to be discussed14:23
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/208352714:23
ralonsoh[OVN] dnat_and_snat rules are setting GATEWAY_PORT when LR pinned to chassis14:23
ralonsohthis is related to the nested routing feature14:23
ralonsohbut in this case there are two routers with GWs, and the inner router has an external network and FIP14:24
ralonsohso the point is if we want to support this architecture14:24
ralonsohI really don't know if we want to have an external network connected to another router as an internal interface, and this router to have another GW network14:25
ralonsohI'll add this to the drivers meeting14:25
ralonsohdo you agree?14:25
lajoskatona+1, perhaps we can invite the the guys owned that feature originally14:25
haleybi haven't gone through all the bug comments yet, but i don't think we want that14:25
ralonsohfor sure, I'll comment that in the bug14:25
ralonsohhaleyb, we'll have time next friday. The point is that with this architecture, we should also snat traffic from this in-the-middle external network14:26
ralonsohand that's not possible for resources outside neutron14:26
ralonsohanyway, this one for the drivers meeting14:26
haleybsure14:27
ralonsohnext one 14:27
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/208376014:27
ralonsohthis is related to a mail chain create by Takashi14:27
ralonsohI replied on monday but I realized that he is expecting more, for example the caller ID14:27
ralonsohnot the caller, the call ID14:27
ralonsohthat actually is NOT printed in the wsgi module14:27
ralonsohso he is proposing to have a common module that prints this information14:28
ralonsohplease check mail "[neutron][nova][oslo][all] Common middleware to log requests"14:28
haleybyes, see ML thread now14:28
ralonsohI think this is interesting and something needed. Maybe we need to wait for this middleware 14:29
ralonsohand that's all14:29
haleybalright, thanks for all the info14:30
haleybany other bugs people want to discuss?14:30
haleybThis week lajoskatona is the bug deputy, next week will be ykarel14:31
haleybdoes that work for both of you?14:31
lajoskatonaI am on it14:31
haleybgreat, thanks14:31
lajoskatonaand I hope ralonsoh emptied the bug cache last week :-)14:31
ykarelif some can switch with me it would be better14:31
ykareli will be out atleast 2 days next week14:31
ykareltue/wed14:31
ralonsohykarel, no problem, I'll do it14:31
ykarelthx ralonsoh , i can do next to next week14:32
slaweqyou were doing it last week so maybe I can switch with ykarel 14:32
elviraI can do it too if you did it this week ralonsoh 14:32
ralonsohwe'll sync offline14:32
ykarel++14:32
haleybok, update the schedule on the wiki if you need to14:33
haleybBug count is currently at 731, down 3 from last week14:34
haleyb#topic community-goals14:34
haleybeventlet deprecation14:34
haleybthere was a ML thread this past week14:35
ralonsohyes, let me finish the document internally and then I'll share it with you14:36
ralonsohwe should have a PTG topic to check the working items14:36
ralonsohsee how to address them14:36
ralonsohand assign the tasks14:36
haleybralonsoh: ok, thanks, i'm trying to find the other ML thread from Steven?14:36
haleybor is it related?14:36
ralonsohI'm talking about Herve's mail14:37
ralonsohin any case, I'll add this info in the PTG etherpad14:37
haleybah yes, that one14:37
haleybi knew it was someone from RH14:37
ralonsohbecause we need to provide feedback to the TC and the community14:37
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/YAX2X7C6FCDDNN4SBFN7BNSUA44Z6VXA/14:38
haleybother community goal was neutronclient deprecation14:39
haleyb#link https://review.opendev.org/q/topic:%22bug/1999774%2214:39
lajoskatonado we have some neutron specific bug or similar to track this progress?14:39
haleyblajoskatona: related to eventlet?14:40
lajoskatonaI mean for eventlet removal?14:40
ralonsohlajoskatona, the eventlet deprecation14:40
ralonsohyes, one sec14:40
ralonsohhttps://bugs.launchpad.net/neutron/+bug/206983114:40
ralonsohthis should be the main one14:40
ralonsohany patch should have a related-bug reference to this LP bug14:40
lajoskatonathanks, I save it14:41
lajoskatonaback to the SDK / remove neutronclient work I still have no time to go back to it14:41
lajoskatonaI even have a Wip one for nova: https://review.opendev.org/c/openstack/nova/+/92802214:42
haleyblajoskatona: ack, i looked at the failures but it wasn't obvious how to fix it :(14:42
haleybin the horizon patch14:42
lajoskatonain its current state I was able to boot a VM, but much more work is necessary on it, yes I suppose a lot of things are broken when you touch the depths of nova, and that's why nobody dares to do it :-)14:43
haleyblajoskatona: i only looked at the horizon patch, will start following the nova one14:44
lajoskatonahaleyb: it is really wip, so I just started to work on it and pushed to gerrit as a PoC that it is possible but just the 1st step of a longer journey :-)14:44
lajoskatonathat's it from me for this topic14:45
haleyblajoskatona: thanks, wish we had more horizon devs14:45
haleyb#topic neutron-lib14:46
haleybnot exactly neutron-lib, more improvements and refactoring, but ihrachys started pushing some mypy patches14:47
haleyb#link https://review.opendev.org/c/openstack/neutron/+/92986414:47
haleyband on that topic we've been having a discussion about abc.ABC classes14:48
haleyb#link https://review.opendev.org/c/openstack/neutron/+/93153314:48
haleybif anyone wants to add to the discussion to help move it forward, i don't want to seem like the ogre14:49
haleyb#topic on-demand14:50
haleybi did not see anything on the wiki, but if you have any topics...14:50
haleybok, thanks for attending, and don't forget to add topics to the PTG etherpad!14:51
haleyb#endmeeting14:51
opendevmeetMeeting ended Tue Oct  8 14:51:33 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:51
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-10-08-14.00.html14:51
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-10-08-14.00.txt14:51
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-10-08-14.00.log.html14:51
ralonsohbye!14:51
lajoskatonaBye14:51
mlavalle\o14:52
slaweqo/14:52
cbuggyo/14:52
haleybykarel: i just wanted to get your opinion on this change, https://review.opendev.org/c/openstack/neutron-lib/+/93168014:52
haleybin addition to moving to py3.12 i removed a non-voting from a job that you had added14:52
haleybfrom what i could tell the job has been stable14:53
ykarelhaleyb, yes should be fine to go voting14:56
ykarelalso i think we no longer need those sqlalchemy master jobs in check queue14:57
haleybykarel: ack, did we move them in neutron? too many patches14:58
ykarelhaleyb, in neutron we moved that to periodic https://github.com/openstack/neutron/commit/793617495f057f2ca63495b21e0070026197136714:58
haleybykarel: right, but i guess we have no periodic in neutron-lib14:59
opendevreviewMerged openstack/neutron master: [OVN] Check metadata HA proxy configuration before restart  https://review.opendev.org/c/openstack/neutron/+/92960415:01
haleybfeel free to comment there, or approve and create a follow-up :)15:01
ykarelack15:03
ihrachyshaleyb: lol. not an ogre. it's a nice argument.15:39
haleybihrachys: i couldn't think of a better word, and shrek jumped into my head15:41
ihrachysI don't even care about consistency on this front. both are fine, and people have preferences. it's like arguing whether a list comprehension is better than a for loop. mildly, sometimes? but it's just style choice in 99% of cases.15:41
haleybihrachys: i tend toward consistency to not have confusion, it's my happy place15:45
ihrachysack; I *will* abandon the switch and revert to metaclass= just so that we can merge these, but I find this a silly reason to -1 a patch.15:47
haleybihrachys: btw, i think you've been interested in updating style things, like getting rid of Class(object), right? I found an old octavia change and was going to play with something similar in neutron15:47
haleyb#link https://review.opendev.org/c/openstack/octavia/+/90559115:47
ihrachyshaleyb: you mean that pyupgrade can be used to automate these conversions?15:47
haleybihrachys: i only put the -1 to have a discussion, sometimes otherwise people assume otherw ise15:48
haleybihrachys: yes, just be careful as it edits files in-place15:48
haleybat least i think that's what it did when i ran it15:48
haleybthen there's autopep8...15:48
ihrachysack; yeah good point, I did this with some weird sed / grep / find ... pipelines and I should use the tool. I even knew about the tool in theory, but forgot it's a thing.15:49
haleybsplitting up into bite-sized chunks is the harder part with the size of our repo15:50
haleyband the rebases15:50
haleybihrachys: anyways, we can add that as a topic for the PTG as well, along with the work you already started15:51
*** sean-k-mooney1 is now known as sean-k-mooney16:27
opendevreviewLajos Katona proposed openstack/neutron-fwaas master: Update jobs based on testing runtime for 2025.1  https://review.opendev.org/c/openstack/neutron-fwaas/+/93150216:50
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: Update jobs based on testing runtime for 2025.1  https://review.opendev.org/c/openstack/neutron-vpnaas/+/93150416:55
gsamfirahi folks! Is this a good place to discuss approaches to fixing a bug in neutron? I've tracked down the root cause and I have a brutal (but working) fix, but I am not faimiliar enough with the codebase (yet) to determine if the approach I took is okay16:56
ralonsohgsamfira, please open a launchpad bug to describe the issue17:29
ralonsohhttps://bugs.launchpad.net/neutron/17:29
ralonsohwith this information we'll be in the same context17:30
ralonsohyou can also push a patch17:30
gsamfiraralonsoh: there's already a bug here: https://bugs.launchpad.net/neutron/+bug/1995078 In my caracal deployment, this fixes it: https://paste.openstack.org/show/bIwytnfsfZbA8FR34Mri/17:31
gsamfiraI can push a PR on gerrit and move the discussion there if you wish17:31
ralonsohgsamfira, please push it to gerrit17:32
ralonsohI would need to understand the rationale of this patch17:32
gsamfirawill do17:32
gsamfirathe router interfaces of networks that get attached to a vrouter are not bound to any chassis. While this is fine for geneve and co, when it comes to VLAN networks in combination with external ports, you want the router interface to be bound to the same ha_chassis_group that the network belongs to 17:34
gsamfirawithout it, external pors have no connectivity17:35
ralonsohthis is why I started https://review.opendev.org/c/openstack/neutron/+/87203317:36
ralonsohto modify the scheduler17:36
ralonsohnot locally the ports17:36
ralonsohanyway, I'll check your patch tomorrow17:36
gsamfirayep. I tried it, but the l3 scheduler seems to only be used for network:router_gateway17:36
gsamfiranot network:router_interface17:36
ralonsoha (internal) router interface is not bound17:37
gsamfirawill probably propose it tomorrow morning GMT+3. Need to get some sleep :D.17:37
gsamfiraI've noticed17:37
gsamfirathat's why I am unsure of this approach17:38
gsamfirait seems to work, not sure of the impact on anything else17:38
opendevreviewMerged openstack/neutron master: [CI] Functional: Increase Ulimit to 4096  https://review.opendev.org/c/openstack/neutron/+/92875919:21
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with eventlet  https://review.opendev.org/c/openstack/neutron/+/93184219:33
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.1: [OVN] Check metadata HA proxy configuration before restart  https://review.opendev.org/c/openstack/neutron/+/93179119:35
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with eventlet  https://review.opendev.org/c/openstack/neutron/+/93184219:37
opendevreviewBrian Haley proposed openstack/neutron master: Add special treatment for 'any' in SG rule API  https://review.opendev.org/c/openstack/neutron/+/92649821:16
cardoehaleyb: Wanted to ping ya about setting up a neutron/ironic PTG time.22:14
opendevreviewMerged openstack/neutron stable/2024.2: [OVN] Check metadata HA proxy configuration before restart  https://review.opendev.org/c/openstack/neutron/+/93179022:47
opendevreviewMerged openstack/neutron master: Use the declarative attribute ``standard_attr_id``  https://review.opendev.org/c/openstack/neutron/+/93040823:51

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