opendevreview | Merged openstack/neutron stable/2024.1: Correct logic error when associating FIP with OVN LB https://review.opendev.org/c/openstack/neutron/+/929998 | 00:53 |
---|---|---|
opendevreview | Ihar Hrachyshka proposed openstack/tap-as-a-service master: Remove unused sections from setup.cfg https://review.opendev.org/c/openstack/tap-as-a-service/+/930160 | 01:15 |
opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove upgrade check for old removed options https://review.opendev.org/c/openstack/neutron/+/930161 | 01:16 |
opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Drop ovn migration for TripleO deployments https://review.opendev.org/c/openstack/neutron/+/930163 | 01:19 |
opendevreview | Merged openstack/neutron stable/2023.2: Correct logic error when associating FIP with OVN LB https://review.opendev.org/c/openstack/neutron/+/929999 | 01:28 |
opendevreview | Merged openstack/neutron master: refactor: remove some unused variables https://review.opendev.org/c/openstack/neutron/+/929810 | 01:29 |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Convert [ml2] physical_network_mtus to DictOpt https://review.opendev.org/c/openstack/neutron/+/930245 | 01:49 |
lajoskatona | ralonsoh: 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 |
lajoskatona | https://review.opendev.org/c/openstack/devstack/+/930237 | 07:59 |
lajoskatona | In 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 paths | 08:05 |
lajoskatona | To 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 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider master: Add OVN DB sync in OVN driver https://review.opendev.org/c/openstack/ovn-octavia-provider/+/925324 | 08:19 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider master: Add sync floating IP support https://review.opendev.org/c/openstack/ovn-octavia-provider/+/929039 | 08:19 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client and sync cmd https://review.opendev.org/c/openstack/ovn-octavia-provider/+/925747 | 08:19 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider stable/2023.1: Replace python-neutronclient with openstacksdk https://review.opendev.org/c/openstack/ovn-octavia-provider/+/930273 | 08:23 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider stable/2023.1: Fix: Avoid neutron ops registration conflict https://review.opendev.org/c/openstack/ovn-octavia-provider/+/930274 | 08:32 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider unmaintained/zed: Fix: Avoid neutron ops registration conflict https://review.opendev.org/c/openstack/ovn-octavia-provider/+/930275 | 08:34 |
sylvr | Good 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 |
sylvr | 2024-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 |
sylvr | I'm thinking I may have misconfigured something, could you help me troubleshoot this ? thanks :) | 08:46 |
opendevreview | Merged openstack/neutron master: Remove install document for TripleO deployments https://review.opendev.org/c/openstack/neutron/+/930164 | 08:53 |
opendevreview | Merged openstack/neutron master: Drop ovn migration for TripleO deployments https://review.opendev.org/c/openstack/neutron/+/930163 | 08:53 |
zigo | Heya ! 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 |
ihrachys | zigo: I think it is spawn by neutron main process but ralonsoh may confirm | 13:03 |
zigo | Yeah thhanks. But "main process" means what? Will that work under neutron-api over uwsgi + neutron-rpc-server ? | 13:04 |
ihrachys | ok maybe I'm wrong, see https://review.opendev.org/c/openstack/devstack/+/922125 | 13:04 |
ihrachys | need ralonsoh to chime in :) | 13:05 |
ralonsoh | ihrachys, sorry, I'm on PTO today, I'll check that issue tomorrow | 13:05 |
ihrachys | see https://github.com/rdo-packages/neutron-distgit/blob/rpm-master/neutron-periodic-workers.service | 13:05 |
opendevreview | Lajos Katona proposed openstack/neutron master: Functional: Increase Ulimit to 6144 https://review.opendev.org/c/openstack/neutron/+/928759 | 13:06 |
opendevreview | Lajos Katona proposed openstack/neutron master: DNM: test functional jobs https://review.opendev.org/c/openstack/neutron/+/928953 | 13:06 |
ralonsoh | but in a nutshell: with wsgi we only provide API workers; the others (periodic, maintenance, rpc) must be spawned in different ervices | 13:06 |
opendevreview | Merged openstack/neutron unmaintained/wallaby: [unmaintained-only] Periodic jobs will be weekly only https://review.opendev.org/c/openstack/neutron/+/930210 | 13:11 |
zigo | ralonsoh: Oh ok, thanks. I'll write the init scripts then. | 13:12 |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Drop workaround for eventlet < 0.22.0 https://review.opendev.org/c/openstack/neutron/+/930327 | 13:36 |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Remove logic for Windows operating systems https://review.opendev.org/c/openstack/neutron/+/930328 | 13:36 |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Remove logic for Windows operating systems https://review.opendev.org/c/openstack/neutron/+/930328 | 13:40 |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Remove remaining figure for TripleO installation document https://review.opendev.org/c/openstack/neutron/+/930333 | 13:54 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Functional tests] Add logging router interfaces in metadata IPv6 tests https://review.opendev.org/c/openstack/neutron/+/930336 | 13:59 |
haleyb | #startmeeting networking | 14:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'networking' | 14:00 |
haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh | 14:00 |
mlavalle | \o | 14:00 |
lajoskatona | o/ | 14:00 |
obondarev | o/ | 14:00 |
frickler | \o | 14:00 |
bcafarel | o/ | 14:00 |
elvira | o/ | 14:00 |
haleyb | #topic announcements | 14:01 |
ihrachys | o/ | 14:01 |
haleyb | lets get started | 14:01 |
haleyb | RC1 has been released, we have identified a few fixes and I will propose an RC2 right after this meeting | 14:02 |
haleyb | September 26th, 2024 is the deadline for final 2024.2 Dalmatian release candidates | 14:02 |
haleyb | so I'm hoping RC2 is the last | 14:02 |
haleyb | We will then enter a quiet period until we tag the final release on October 2nd, 2024 | 14:02 |
haleyb | Watch for any translation patches coming through on the stable/2024.2 branch and merge them quickly | 14:02 |
rubasov | o/ | 14:02 |
slaweq | o/ | 14:02 |
haleyb | Release-critical bugfixes will need to be merged in the master branch first, then backported to the stable/2024.2 | 14:02 |
haleyb | Master branch has switched to 2025.1 Epoxy development, but please prioritize any work necessary for completing 2024.2 Dalmatian plans | 14:03 |
haleyb | It's time to start planning the 2025.1 Epoxy development cycle, including discussing PTG sessions content, in preparation of Project Teams Gathering | 14:03 |
mlavalle | are we going to use the priority list to mark what needs to be reviewed? | 14:03 |
haleyb | I will start on that next week | 14:03 |
haleyb | mlavalle: yes, we can continue to use that, but for RC* ping in the channel might be quicker with the timeline | 14:04 |
mlavalle | ack, ping me if necessary | 14:05 |
haleyb | at this point I think RC2 will be it, and I believe it's all been merged to stable/2024.2 | 14:05 |
haleyb | to 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 me | 14:06 |
haleyb | ok... any question on that? | 14:08 |
* mlavalle is not sure email is older than irc | 14:08 | |
haleyb | well i won't be available over nntp either :-p | 14:09 |
haleyb | my daughter was so happy to tell us she'd been sending a lot of emails lately, kids today use snapchat for everything | 14:10 |
haleyb | so i'm old | 14:10 |
mlavalle | LOL, yeap | 14:10 |
slaweq | :) | 14:12 |
haleyb | my only other announcement was I started doing the normal beginning of cycle patches for updating the gate jobs | 14:13 |
haleyb | https://review.opendev.org/q/topic:%222025.1-neutron-jobs%22 | 14:13 |
lajoskatona | thanks | 14:13 |
haleyb | one of those is waiting on a zuul-jobs patch still i think | 14:13 |
haleyb | actually, one is an openstack-zuul-jobs patch, but there is another dependency | 14:14 |
frickler | also note that devsstack hasn't branched yet, so jobs may still be missing | 14:14 |
frickler | (for stable/2024.2) | 14:14 |
haleyb | either way, it's good to get those in early so any reviews are great | 14:15 |
haleyb | frickler: oh, like grenade probably | 14:15 |
frickler | yes, same thing for grenade | 14:15 |
haleyb | most of the patches I did were just to move python versions forward and enable our skip-level job | 14:16 |
ihrachys | haleyb: what's the minimal py version now? | 14:16 |
haleyb | py3.9 is minimal, py3.12 is maximal (?) the ones in-between are now peridoc | 14:16 |
ihrachys | was it py38 before? | 14:17 |
mlavalle | yes, we had py38 | 14:17 |
haleyb | https://review.opendev.org/c/openstack/neutron/+/929788 has a link to the governance doc, but was only 3.9 before | 14:17 |
ihrachys | ack; of new syntax, probably just dict1 | dict2 instead of dict1.copy(); dict.update() is now useful to us | 14:18 |
haleyb | https://review.opendev.org/c/openstack/governance/+/926150/3/reference/runtimes/2025.1.rst | 14:18 |
haleyb | anyways, that's all the announcements I had | 14:20 |
haleyb | #topic bugs | 14:21 |
haleyb | jlibosva was the bug deputy last week, his report is at | 14:21 |
haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/K64H67NE3OJGL73AL5KL33AFXWDKUWIY/ | 14:21 |
haleyb | the first we knew about since the other day | 14:22 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2081173 | 14:22 |
haleyb | security-groups-rules-belongs-to-default-sg slows down SG GET operations | 14:22 |
haleyb | it was related to another bug filed last week, which i don't see in this list | 14:23 |
ihrachys | maybe because the other was already fixed/ | 14:24 |
haleyb | right, but should still be in the list | 14:24 |
haleyb | just had to find it | 14:24 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2081087 | 14:24 |
haleyb | Performance regression in neutron-server from 2023.1 to 2024.1 when fetching a Security Group | 14:24 |
haleyb | ralonsoh fixed that and we backported, but more work is required on other places that could benefit from using 'selectin' | 14:25 |
ihrachys | where's the "more work" tracked? | 14:26 |
haleyb | ihrachys: i need to open another bug for investitagion, will take some benchmarking of each case to figure out | 14:28 |
ihrachys | yeah at the very least some investigation tracker. we have one for rally scenarios check-up (also in the list) but not for selectin | 14:29 |
haleyb | right. the patch I created to change them all is at least a roadmap | 14:30 |
haleyb | #link https://review.opendev.org/c/openstack/neutron/+/929851 | 14:30 |
haleyb | but it shouldn't merge as there are cases where it makes things slower | 14:30 |
haleyb | So i will create one, and i was going to add it to the PTG list | 14:31 |
haleyb | the next one is related | 14:32 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2081108 | 14:32 |
opendevreview | Merged openstack/ovn-bgp-agent master: Drop Python 3.6/7 support https://review.opendev.org/c/openstack/ovn-bgp-agent/+/930110 | 14:32 |
opendevreview | Merged openstack/ovn-bgp-agent master: Bump hacking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/930111 | 14:32 |
haleyb | Tighten / improve rally SG tests to catch performance regressions | 14:32 |
haleyb | when we saw the regression, it seems like rally should have triggered something | 14:32 |
haleyb | i'm just not sure it's setup to find these things, for example, i think it only adds 20 SGs | 14:33 |
haleyb | so if we can automate something better it would be good | 14:34 |
ihrachys | maybe it should add 1000 and have more tight thresholds | 14:34 |
ihrachys | the bug is open ended, yes | 14:34 |
lajoskatona | +1 to have jobs for perf tests perhaps as periodic or similar | 14:35 |
haleyb | of 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 them | 14:35 |
slaweq | I don't think rally can compare somehow results with older runs | 14:35 |
ihrachys | slaweq: it compares with a hardcoded threshold I think | 14:35 |
slaweq | That would be imho good to check and raise failures if there is regresion | 14:36 |
ihrachys | so put threshold at x2 what you observe; then you catch x20 regressions | 14:36 |
ihrachys | I suspect one could with enough motivation build some chart from collected rally results that would show trends. | 14:36 |
slaweq | Worth to try for sure | 14:37 |
ihrachys | haleyb: ++ on being more anal / conservative about touching db layer and other foundational parts | 14:37 |
slaweq | I 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 sure | 14:38 |
haleyb | ihrachys: i know on kernel netdev there is usually a lot of data in the commit message when a "this is faster/better" change is proposed | 14:39 |
haleyb | maybe a good conversation for the ML so we don't reinvent the wheel | 14:39 |
ihrachys | seems like common sense to add relevant data / explanations to commit messages... not sure what to "reinvent" here except culture? | 14:40 |
ihrachys | in 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 |
haleyb | ihrachys: yes, but there was still a new lazy='joined' if i'm remembering correctly | 14:42 |
haleyb | typically it's a distro user that finds these regressions, at least it seems that way to me, and that means a years-old change, etc | 14:43 |
ihrachys | reviewers 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 |
haleyb | ihrachys: and yes, part is culture, my reinvent comment was more aimed at if other services are somehow doing this already | 14:45 |
ihrachys | it never hurts to ask around :) | 14:45 |
haleyb | i mean, can there be a job that does a before/after with a single patch? that's one way | 14:46 |
haleyb | anyways, this is perfect fodder for the PTG :) | 14:46 |
lajoskatona | +1 for PTG discussion | 14:46 |
haleyb | i will get an etherpad created, etc so i don't forget all these things | 14:47 |
haleyb | that was it for high bugs, any others to discuss? | 14:49 |
haleyb | #topic community-goals | 14:50 |
haleyb | i think there is only one of the initial eventlet changes left | 14:51 |
haleyb | #link https://review.opendev.org/q/topic:%22bug/2069581%22 | 14:51 |
haleyb | the tempest patch blocking it merged, but need to wait for rodolfo to go through the latest run on that one | 14:51 |
haleyb | #topic on-demand | 14:52 |
haleyb | i did forget to mention, obondarev is the bug deputy this week, ralonsoh is next week | 14:53 |
haleyb | hopefully not many bugz | 14:53 |
obondarev | on it, not many so far :) | 14:53 |
haleyb | nice | 14:53 |
haleyb | any other topics? | 14:53 |
haleyb | ok, have a nice week, i'll get RC2 proposed | 14:54 |
haleyb | #endmeeting | 14:54 |
opendevmeet | Meeting ended Tue Sep 24 14:54:38 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:54 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-24-14.00.html | 14:54 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-24-14.00.txt | 14:54 |
opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-24-14.00.log.html | 14:54 |
lajoskatona | o/ | 14:54 |
obondarev | o/ | 14:54 |
ihrachys | o/ | 14:54 |
slaweq | o/ | 14:56 |
haleyb | RC2 release patch: https://review.opendev.org/c/openstack/releases/+/930345 | 15:02 |
lajoskatona | \o/ | 15:13 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix pep8 with pylint 3.3.0 https://review.opendev.org/c/openstack/ovn-octavia-provider/+/930347 | 15:13 |
opendevreview | Merged openstack/neutron unmaintained/wallaby: [Wallaby Only][CI] Change sdk job to lioadm helper https://review.opendev.org/c/openstack/neutron/+/929944 | 15:15 |
haleyb | bcafarel: 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 downstream | 15:16 |
bcafarel | haleyb: sure thing, will look later today | 15:16 |
haleyb | thanks! | 15:17 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider master: Add OVN DB sync in OVN driver https://review.opendev.org/c/openstack/ovn-octavia-provider/+/925324 | 15:28 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider master: Add sync floating IP support https://review.opendev.org/c/openstack/ovn-octavia-provider/+/929039 | 15:28 |
opendevreview | Rico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client and sync cmd https://review.opendev.org/c/openstack/ovn-octavia-provider/+/925747 | 15:28 |
opendevreview | Merged openstack/tap-as-a-service master: Remove unused sections from setup.cfg https://review.opendev.org/c/openstack/tap-as-a-service/+/930160 | 15:49 |
opendevreview | Merged openstack/tap-as-a-service master: Remove unused openstack-common.conf https://review.opendev.org/c/openstack/tap-as-a-service/+/930159 | 15:49 |
opendevreview | Arkady Shtempler proposed openstack/neutron-tempest-plugin master: Neutron&Designate DNS integration - some enhancements https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/930189 | 15:50 |
opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: DNM: test master of vpnaas https://review.opendev.org/c/openstack/neutron-vpnaas/+/930208 | 16:59 |
opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Introduce LSP address column parsing functions https://review.opendev.org/c/openstack/ovn-bgp-agent/+/925801 | 17:57 |
opendevreview | Jakub 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/+/924531 | 17:58 |
opendevreview | Merged openstack/neutron master: Remove remaining figure for TripleO installation document https://review.opendev.org/c/openstack/neutron/+/930333 | 18:03 |
opendevreview | Merged openstack/neutron master: Fix usage of removed external_network_bridge option https://review.opendev.org/c/openstack/neutron/+/930162 | 18:03 |
opendevreview | Merged openstack/neutron master: refactor: unindent some indents in _handle_lb_fip_cmds https://review.opendev.org/c/openstack/neutron/+/929793 | 21:14 |
opendevreview | Merged openstack/neutron master: refactor: don't calculate list of attached lbs for every lb https://review.opendev.org/c/openstack/neutron/+/929794 | 21:17 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!