Tuesday, 2024-09-24

opendevreviewMerged openstack/neutron stable/2024.1: Correct logic error when associating FIP with OVN LB  https://review.opendev.org/c/openstack/neutron/+/92999800:53
opendevreviewIhar Hrachyshka proposed openstack/tap-as-a-service master: Remove unused sections from setup.cfg  https://review.opendev.org/c/openstack/tap-as-a-service/+/93016001:15
opendevreviewIhar Hrachyshka proposed openstack/neutron master: Remove upgrade check for old removed options  https://review.opendev.org/c/openstack/neutron/+/93016101:16
opendevreviewIhar Hrachyshka proposed openstack/neutron master: Drop ovn migration for TripleO deployments  https://review.opendev.org/c/openstack/neutron/+/93016301:19
opendevreviewMerged openstack/neutron stable/2023.2: Correct logic error when associating FIP with OVN LB  https://review.opendev.org/c/openstack/neutron/+/92999901:28
opendevreviewMerged openstack/neutron master: refactor: remove some unused variables  https://review.opendev.org/c/openstack/neutron/+/92981001:29
opendevreviewTakashi Kajinami proposed openstack/neutron master: Convert [ml2] physical_network_mtus to DictOpt  https://review.opendev.org/c/openstack/neutron/+/93024501:49
lajoskatonaralonsoh: Good morning, yesterday I had a chat with some infra guys about our issue with "too many files open", and they suggested to understand better what files  are open, and the result is this patch:07:59
lajoskatonahttps://review.opendev.org/c/openstack/devstack/+/93023707:59
lajoskatonaIn the file tracker logs (i.e.: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_894/928953/13/check/neutron-functional-with-uwsgi-7/894de3f/controller/logs/screen-file_tracker.txt ) if more 15000 files are open dumps the file paths08:05
lajoskatonaTo tell the truth I don't see big difference from the working one (i.e.: https://c5f1c0adf580e9de5276-bc554c95a41b41301246f2ff3ad0cf6a.ssl.cf5.rackcdn.com/928953/13/check/neutron-functional-with-uwsgi-6/85b1f0b/controller/logs/screen-file_tracker.txt )08:08
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add OVN DB sync in OVN driver  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92532408:19
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add sync floating IP support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92903908:19
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client and sync cmd  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92574708:19
opendevreviewRico Lin proposed openstack/ovn-octavia-provider stable/2023.1: Replace python-neutronclient with openstacksdk  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93027308:23
opendevreviewRico Lin proposed openstack/ovn-octavia-provider stable/2023.1: Fix: Avoid neutron ops registration conflict  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93027408:32
opendevreviewRico Lin proposed openstack/ovn-octavia-provider unmaintained/zed: Fix: Avoid neutron ops registration conflict  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93027508:34
sylvrGood morning ! I'm studying Cloud Operation and I'm trying to deploy an OpenStack with kayobe. However I'm encountering issue when I try to create an instance, it seems like neutron can't bind port to my instances causing them to timeout during build this is the error I get in /var/log/kolla/neutron/neutron-server.log :08:46
sylvr2024-09-24 08:37:40.295 22 ERROR neutron.plugins.ml2.managers [req-c21a9ea8-1ecf-4a10-a3c0-c7b266fa8021 req-80f12b11-1123-4b19-8b49-5dcc389ce8ad 607a43b606a54e11bdf89c46577db5f2 53a3ffb6389641e1934e0ac97a9be36e - - default default] Failed to bind port 0d12f559-2fa2-485b-ae31-3b75ad9a2a72 on host compute2 for vnic_type normal using segments [{'id': '97ee4fe9-cf1f-47d4-9736-128f4982abc9', 'network_type': 'flat', 'physical_network':08:46
sylvr'network_id': 'd933b37d-9e74-44ef-aa82-cbc3ba67a7b1'}]08:46
sylvrI'm thinking I may have misconfigured something, could you help me troubleshoot this ? thanks :)08:46
opendevreviewMerged openstack/neutron master: Remove install document for TripleO deployments  https://review.opendev.org/c/openstack/neutron/+/93016408:53
opendevreviewMerged openstack/neutron master: Drop ovn migration for TripleO deployments  https://review.opendev.org/c/openstack/neutron/+/93016308:53
zigoHeya ! I was wondering how to handle neutron-periodic-workers from the packaging point of view. Is this a new daemon so I need to wirte a systemd .service ? Or will it be just spawn by neutron-api (ie: forked form a uwsgi process)?11:59
ihrachyszigo: I think it is spawn by neutron main process but ralonsoh may confirm13:03
zigoYeah thhanks. But "main process" means what? Will that work under neutron-api over uwsgi + neutron-rpc-server ?13:04
ihrachysok maybe I'm wrong, see https://review.opendev.org/c/openstack/devstack/+/92212513:04
ihrachysneed ralonsoh to chime in :)13:05
ralonsohihrachys, sorry, I'm on PTO today, I'll check that issue tomorrow13:05
ihrachyssee https://github.com/rdo-packages/neutron-distgit/blob/rpm-master/neutron-periodic-workers.service13:05
opendevreviewLajos Katona proposed openstack/neutron master: Functional: Increase Ulimit to 6144  https://review.opendev.org/c/openstack/neutron/+/92875913:06
opendevreviewLajos Katona proposed openstack/neutron master: DNM: test functional jobs  https://review.opendev.org/c/openstack/neutron/+/92895313:06
ralonsohbut in a nutshell: with wsgi we only provide API workers; the others (periodic, maintenance, rpc) must be spawned in different ervices13:06
opendevreviewMerged openstack/neutron unmaintained/wallaby: [unmaintained-only] Periodic jobs will be weekly only  https://review.opendev.org/c/openstack/neutron/+/93021013:11
zigoralonsoh: Oh ok, thanks. I'll write the init scripts then.13:12
opendevreviewTakashi Kajinami proposed openstack/neutron master: Drop workaround for eventlet < 0.22.0  https://review.opendev.org/c/openstack/neutron/+/93032713:36
opendevreviewTakashi Kajinami proposed openstack/neutron master: Remove logic for Windows operating systems  https://review.opendev.org/c/openstack/neutron/+/93032813:36
opendevreviewTakashi Kajinami proposed openstack/neutron master: Remove logic for Windows operating systems  https://review.opendev.org/c/openstack/neutron/+/93032813:40
opendevreviewTakashi Kajinami proposed openstack/neutron master: Remove remaining figure for TripleO installation document  https://review.opendev.org/c/openstack/neutron/+/93033313:54
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [Functional tests] Add logging router interfaces in metadata IPv6 tests  https://review.opendev.org/c/openstack/neutron/+/93033613:59
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Sep 24 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
lajoskatonao/14:00
obondarevo/14:00
frickler\o14:00
bcafarelo/14:00
elvirao/14:00
haleyb#topic announcements14:01
ihrachyso/14:01
haleyblets get started14:01
haleybRC1 has been released, we have identified a few fixes and I will propose an RC2 right after this meeting14:02
haleybSeptember 26th, 2024 is the deadline for final 2024.2 Dalmatian release candidates14:02
haleybso I'm hoping RC2 is the last14:02
haleybWe will then enter a quiet period until we tag the final release on October 2nd, 202414:02
haleybWatch for any translation patches coming through on the stable/2024.2 branch and merge them quickly14:02
rubasovo/14:02
slaweqo/14:02
haleybRelease-critical bugfixes will need to be merged in the master branch first, then backported to the stable/2024.214:02
haleybMaster branch has switched to 2025.1 Epoxy development, but please prioritize any work necessary for completing 2024.2 Dalmatian plans14:03
haleybIt's time to start planning the 2025.1 Epoxy development cycle, including discussing PTG sessions content, in preparation of Project Teams Gathering14:03
mlavalleare we going to use the priority list to mark what needs to be reviewed?14:03
haleybI will start on that next week14:03
haleybmlavalle: yes, we can continue to use that, but for RC* ping in the channel might be quicker with the timeline14:04
mlavalleack, ping me if necessary14:05
haleybat this point I think RC2 will be it, and I believe it's all been merged to stable/2024.214:05
haleybto throw a wrench in it I'm mostly out the rest of the week, will only be checking email sporadically, so please use that (old) communication style if you need me14:06
haleybok... any question on that?14:08
* mlavalle is not sure email is older than irc14:08
haleybwell i won't be available over nntp either :-p14:09
haleybmy daughter was so happy to tell us she'd been sending a lot of emails lately, kids today use snapchat for everything14:10
haleybso i'm old14:10
mlavalleLOL, yeap14:10
slaweq:)14:12
haleybmy only other announcement was I started doing the normal beginning of cycle patches for updating the gate jobs14:13
haleybhttps://review.opendev.org/q/topic:%222025.1-neutron-jobs%2214:13
lajoskatonathanks14:13
haleybone of those is waiting on a zuul-jobs patch still i think14:13
haleybactually, one is an openstack-zuul-jobs patch, but there is another dependency14:14
frickleralso note that devsstack hasn't branched yet, so jobs may still be missing14:14
frickler(for stable/2024.2)14:14
haleybeither way, it's good to get those in early so any reviews are great14:15
haleybfrickler: oh, like grenade probably14:15
frickleryes, same thing for grenade14:15
haleybmost of the patches I did were just to move python versions forward and enable our skip-level job14:16
ihrachyshaleyb: what's the minimal py version now?14:16
haleybpy3.9 is minimal, py3.12 is maximal (?) the ones in-between are now peridoc14:16
ihrachyswas it py38 before?14:17
mlavalleyes, we had py3814:17
haleybhttps://review.opendev.org/c/openstack/neutron/+/929788 has a link to the governance doc, but was only 3.9 before14:17
ihrachysack; of new syntax, probably just dict1 | dict2 instead of dict1.copy(); dict.update() is now useful to us14:18
haleybhttps://review.opendev.org/c/openstack/governance/+/926150/3/reference/runtimes/2025.1.rst14:18
haleybanyways, that's all the announcements I had14:20
haleyb#topic bugs14:21
haleybjlibosva was the bug deputy last week, his report is at14:21
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/K64H67NE3OJGL73AL5KL33AFXWDKUWIY/14:21
haleybthe first we knew about since the other day14:22
haleyb#link https://bugs.launchpad.net/neutron/+bug/208117314:22
haleybsecurity-groups-rules-belongs-to-default-sg slows down SG GET operations14:22
haleybit was related to another bug filed last week, which i don't see in this list14:23
ihrachysmaybe because the other was already fixed/14:24
haleybright, but should still be in the list14:24
haleybjust had to find it14:24
haleyb#link https://bugs.launchpad.net/neutron/+bug/208108714:24
haleybPerformance regression in neutron-server from 2023.1 to 2024.1 when fetching a Security Group14:24
haleybralonsoh fixed that and we backported, but more work is required on other places that could benefit from using 'selectin'14:25
ihrachyswhere's the "more work" tracked?14:26
haleybihrachys: i need to open another bug for investitagion, will take some benchmarking of each case to figure out14:28
ihrachysyeah at the very least some investigation tracker. we have one for rally scenarios check-up (also in the list) but not for selectin14:29
haleybright. the patch I created to change them all is at least a roadmap14:30
haleyb#link https://review.opendev.org/c/openstack/neutron/+/92985114:30
haleybbut it shouldn't merge as there are cases where it makes things slower14:30
haleybSo i will create one, and i was going to add it to the PTG list14:31
haleybthe next one is related14:32
haleyb#link https://bugs.launchpad.net/neutron/+bug/208110814:32
opendevreviewMerged openstack/ovn-bgp-agent master: Drop Python 3.6/7 support  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93011014:32
opendevreviewMerged openstack/ovn-bgp-agent master: Bump hacking  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93011114:32
haleybTighten / improve rally SG tests to catch performance regressions14:32
haleybwhen we saw the regression, it seems like rally should have triggered something14:32
haleybi'm just not sure it's setup to find these things, for example, i think it only adds 20 SGs14:33
haleybso if we can automate something better it would be good14:34
ihrachysmaybe it should add 1000 and have more tight thresholds14:34
ihrachysthe bug is open ended, yes14:34
lajoskatona+1 to have jobs for perf tests perhaps as periodic or similar14:35
haleybof course now i can get on my soap box and say we should get better at asking for perf numbers on DB patches, since the person making the change is probably best suited to providing them14:35
slaweqI don't think rally can compare somehow results with older runs14:35
ihrachysslaweq: it compares with a hardcoded threshold I think14:35
slaweqThat would be imho good to check and raise failures if there is regresion14:36
ihrachysso put threshold at x2 what you observe; then you catch x20 regressions14:36
ihrachysI suspect one could with enough motivation build some chart from collected rally results that would show trends.14:36
slaweqWorth to try for sure14:37
ihrachyshaleyb: ++ on being more anal / conservative about touching db layer and other foundational parts14:37
slaweqI think Ironic has some job where they have simple script to create many resources with API and measure time of responses but I would need to check that to be sure14:38
haleybihrachys: i know on kernel netdev there is usually a lot of data in the commit message when a "this is faster/better" change is proposed14:39
haleybmaybe a good conversation for the ML so we don't reinvent the wheel14:39
ihrachysseems like common sense to add relevant data / explanations to commit messages... not sure what to "reinvent" here except culture?14:40
ihrachysin this scenario though, I think the regression was introduced by a new api, no? no one claimed it's "faster". it was a functional change that regressed something else.14:41
haleybihrachys: yes, but there was still a new lazy='joined' if i'm remembering correctly14:42
haleybtypically it's a distro user that finds these regressions, at least it seems that way to me, and that means a years-old change, etc14:43
ihrachysreviewers should definitely pay attention to these things (among lots of other things); yes. but I feel most active reviewers on the project are quite stretched so quality necessarily suffers. it's also a strain on contributors to collect all the requested data, run all the tests etc. etc.14:44
haleybihrachys: and yes, part is culture, my reinvent comment was more aimed at if other services are somehow doing this already14:45
ihrachysit never hurts to ask around :)14:45
haleybi mean, can there be a job that does a before/after with a single patch? that's one way14:46
haleybanyways, this is perfect fodder for the PTG :)14:46
lajoskatona+1 for PTG discussion14:46
haleybi will get an etherpad created, etc so i don't forget all these things14:47
haleybthat was it for high bugs, any others to discuss?14:49
haleyb#topic community-goals14:50
haleybi think there is only one of the initial eventlet changes left14:51
haleyb#link https://review.opendev.org/q/topic:%22bug/2069581%2214:51
haleybthe tempest patch blocking it merged, but need to wait for rodolfo to go through the latest run on that one14:51
haleyb#topic on-demand14:52
haleybi did forget to mention, obondarev is the bug deputy this week, ralonsoh is next week14:53
haleybhopefully not many bugz14:53
obondarevon it, not many so far :)14:53
haleybnice14:53
haleybany other topics?14:53
haleybok, have a nice week, i'll get RC2 proposed14:54
haleyb#endmeeting14:54
opendevmeetMeeting ended Tue Sep 24 14:54:38 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:54
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-24-14.00.html14:54
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-24-14.00.txt14:54
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-24-14.00.log.html14:54
lajoskatonao/14:54
obondarevo/14:54
ihrachyso/14:54
slaweqo/14:56
haleybRC2 release patch: https://review.opendev.org/c/openstack/releases/+/93034515:02
lajoskatona\o/15:13
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Fix pep8 with pylint 3.3.0  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93034715:13
opendevreviewMerged openstack/neutron unmaintained/wallaby: [Wallaby Only][CI] Change sdk job to lioadm helper  https://review.opendev.org/c/openstack/neutron/+/92994415:15
haleybbcafarel: hey bernard, can you take a look at https://review.opendev.org/c/openstack/neutron/+/926666 and yoga friend? these were based on a change RH did downstream15:16
bcafarelhaleyb: sure thing, will look later today15:16
haleybthanks!15:17
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add OVN DB sync in OVN driver  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92532415:28
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add sync floating IP support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92903915:28
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client and sync cmd  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92574715:28
opendevreviewMerged openstack/tap-as-a-service master: Remove unused sections from setup.cfg  https://review.opendev.org/c/openstack/tap-as-a-service/+/93016015:49
opendevreviewMerged openstack/tap-as-a-service master: Remove unused openstack-common.conf  https://review.opendev.org/c/openstack/tap-as-a-service/+/93015915:49
opendevreviewArkady Shtempler proposed openstack/neutron-tempest-plugin master: Neutron&Designate DNS integration - some enhancements  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93018915:50
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: DNM: test master of vpnaas  https://review.opendev.org/c/openstack/neutron-vpnaas/+/93020816:59
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Introduce LSP address column parsing functions  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92580117:57
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: nb driver: Don't expose FIP if the external_mac is not set  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92453117:58
opendevreviewMerged openstack/neutron master: Remove remaining figure for TripleO installation document  https://review.opendev.org/c/openstack/neutron/+/93033318:03
opendevreviewMerged openstack/neutron master: Fix usage of removed external_network_bridge option  https://review.opendev.org/c/openstack/neutron/+/93016218:03
opendevreviewMerged openstack/neutron master: refactor: unindent some indents in _handle_lb_fip_cmds  https://review.opendev.org/c/openstack/neutron/+/92979321:14
opendevreviewMerged openstack/neutron master: refactor: don't calculate list of attached lbs for every lb  https://review.opendev.org/c/openstack/neutron/+/92979421:17

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