| opendevreview | Youngjun Yoo proposed openstack/python-neutronclient master: refectoring: modifying the value passed as a parameter https://review.opendev.org/c/openstack/python-neutronclient/+/912685 | 01:34 |
|---|---|---|
| opendevreview | Youngjun Yoo proposed openstack/python-neutronclient master: refectoring: simplifying if condition expressions https://review.opendev.org/c/openstack/python-neutronclient/+/912687 | 01:42 |
| opendevreview | Merged openstack/neutron stable/2023.2: Fix TestOVNMechanismDriver ipv6 tests https://review.opendev.org/c/openstack/neutron/+/912072 | 04:19 |
| opendevreview | Merged openstack/neutron stable/2023.1: Fix TestOVNMechanismDriver ipv6 tests https://review.opendev.org/c/openstack/neutron/+/912073 | 04:19 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: Add documentation for aa-l3-gw-multihoming https://review.opendev.org/c/openstack/neutron/+/899402 | 06:47 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/912635 | 07:27 |
| opendevreview | Merged openstack/neutron-vpnaas master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/neutron-vpnaas/+/912641 | 07:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add the network type to the ``Logical_Switch`` register https://review.opendev.org/c/openstack/neutron/+/912347 | 07:38 |
| opendevreview | Merged openstack/neutron-vpnaas stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/neutron-vpnaas/+/912639 | 07:50 |
| opendevreview | Merged openstack/neutron-vpnaas stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/neutron-vpnaas/+/912640 | 07:50 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.1: DNM Just test of the tobiko periodic job https://review.opendev.org/c/openstack/neutron/+/912719 | 08:38 |
| opendevreview | Merged openstack/networking-sfc master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/networking-sfc/+/912644 | 08:43 |
| *** mklejn__ is now known as mklejn | 09:05 | |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: FIX OVN LB Health Monitor checks for IPv6 members https://review.opendev.org/c/openstack/ovn-octavia-provider/+/911413 | 09:37 |
| opendevreview | zhuyawei proposed openstack/neutron master: Try up port, after port binding. https://review.opendev.org/c/openstack/neutron/+/912724 | 10:05 |
| opendevreview | Sebastian Lohff proposed openstack/neutron master: Don't delete already deleted extra router routes https://review.opendev.org/c/openstack/neutron/+/912629 | 10:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Use the LSP update event to update the LRP https://review.opendev.org/c/openstack/neutron/+/912349 | 10:44 |
| ralonsoh | lajoskatona, slaweq hi! Now we have 2024.1 branch in n-lib, I think we can merge this: https://review.opendev.org/c/openstack/neutron-lib/+/912273 | 10:48 |
| zigo | neutron-dynamic-routing is trying to load DEVICE_OWNER_LOADBALANCER from neutorn-lib, but that's gone in Caracal. Where to file a bug? | 10:59 |
| ralonsoh | zigo, where in the code is that happening? | 11:01 |
| ralonsoh | when we merged this patch, we search for these constants in https://codesearch.openstack.org | 11:02 |
| zigo | ralonsoh: https://paste.opendev.org/show/bxUs3Tcp9l4BXzZlSMkQ/ | 11:02 |
| ralonsoh | zigo, this is because n-d-r is using an old version of neutron | 11:04 |
| ralonsoh | I'll update the requirements file | 11:05 |
| zigo | Well, this needs fixing! :) | 11:05 |
| zigo | In packages, it's getting the latest neutron-lib for me. | 11:05 |
| zigo | Thanks! | 11:05 |
| zigo | ralonsoh: With my build log, do you understand where the problem is? Will it be easy to make a patch? | 11:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: Bump Neutron to 24.0.0.0b1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/912737 | 11:13 |
| ralonsoh | zigo, ^ | 11:13 |
| zigo | This will just make it fail in the CI, not repair the issue... | 11:14 |
| ralonsoh | why is so? | 11:14 |
| zigo | Also, the problem is since neutron-lib was released, no? | 11:14 |
| ralonsoh | yes because it removed the lbaas constants | 11:15 |
| zigo | ie: builds with 3.8.0, fails with 3.11.0. | 11:15 |
| ralonsoh | I don't understand what does it mean | 11:15 |
| zigo | Running neutron-dynamic-routing unit tests with neutron-lib 3.8.0 works, but not with 3.11.0. So I fail how changing the version of neutron will do any good. | 11:17 |
| ralonsoh | I think I explained it in the commit message and the LP bug | 11:17 |
| zigo | Oh ok, I understand now, sorry and thanks for explaining ! :) | 11:18 |
| zigo | Will add the constraint in my package, it will pull neutron from experimental and it will fix for me as well. | 11:18 |
| ralonsoh | cool | 11:18 |
| zigo | ralonsoh: Will neutron RC1 be released soon? | 11:18 |
| ralonsoh | let me check the release calendar | 11:20 |
| zigo | This week is RC1 release week ! :) | 11:20 |
| ralonsoh | this week | 11:20 |
| zigo | It's just nicer if it's not released last minute of Friday ... :P | 11:20 |
| ralonsoh | https://releases.openstack.org/caracal/schedule.html | 11:20 |
| zigo | networking-sfc still is using assertDictContainsSubset (gone from Python 3.12). I have a Debian patch for it, but I also don't know where to file a bug for it. | 11:30 |
| ralonsoh | zigo, in Neutron board | 11:31 |
| ralonsoh | https://bugs.launchpad.net/neutron/ | 11:31 |
| zigo | Oh, https://review.opendev.org/c/openstack/networking-sfc/+/906523 ... | 11:31 |
| zigo | IMO, this one needs review + merge for the release. | 11:31 |
| ralonsoh | done | 11:32 |
| zigo | :) | 11:32 |
| zigo | Thanks ! :) | 11:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Group "functional.agent.l3" tests in the same executor https://review.opendev.org/c/openstack/neutron/+/911647 | 11:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Open the 2024.2 (Dalmatian) DB branch https://review.opendev.org/c/openstack/neutron/+/912761 | 11:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "socket" NUMA affinity policy https://review.opendev.org/c/openstack/neutron/+/910594 | 11:45 |
| opendevreview | Merged openstack/networking-bgpvpn master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/networking-bgpvpn/+/912647 | 12:12 |
| frickler | ralonsoh: zigo: fyi, networking-sfc was already branched yesterday, so you'll need to backport | 12:29 |
| ralonsoh | sure | 12:29 |
| opendevreview | Merged openstack/networking-sfc master: python 3.12: assertDictContainsSubset https://review.opendev.org/c/openstack/networking-sfc/+/906523 | 13:02 |
| zigo | frickler: I imported the patch in debian/patches (replacing my patch that was huge...), so that's fine. | 13:46 |
| opendevreview | Merged openstack/networking-bagpipe master: Remove the "new_facade" parameter that is no longer needed https://review.opendev.org/c/openstack/networking-bagpipe/+/905222 | 14:15 |
| fungi | haleyb: (any anyone else interested) just a heads up that i've proposed and backported a bunch of centos-7 removal changes for vpnaas under https://review.opendev.org/q/topic:%22drop-centos-7%22 but for expediency i intend to just go ahead and bypass review/gating to merge them directly in gerrit so we can proceed with identifying further cleanup in advance of friday when that node | 15:34 |
| fungi | label goes away | 15:34 |
| fungi | didn't want it to surprise anyone | 15:34 |
| haleyb | fungi: thanks for the heads up, sgtm | 15:39 |
| fungi | yeah, if we had a longer lead time, and if jobs were actually passing on all those stable and unmaintained branches, we could go through a more typical review process | 15:40 |
| lajoskatona | fungi: thanks, so am I right that no review is necessary? | 15:40 |
| fungi | lajoskatona: a quick double-check would be appreciated just for an extra set of eyeballs | 15:40 |
| fungi | they should really be very trivial deletions from .zuul.yaml and nothing else | 15:41 |
| lajoskatona | fungi: ack | 15:41 |
| otherwiseguy | haleyb: do you know anyone in the oslo-core world I can press for reviews on https://review.opendev.org/c/openstack/oslo.service/+/911627? | 15:42 |
| otherwiseguy | should be pretty safe taking something random that is clearly wrong and adding a sane order to it. :p | 15:43 |
| otherwiseguy | (to fix https://review.opendev.org/c/openstack/neutron/+/911625) essentially, neutron with ml2/ovn doesn exit on sigterm. | 15:44 |
| otherwiseguy | (and stomps on signal handlers that would be set elsewhere in neutron and oslo.service) | 15:45 |
| haleyb | otherwiseguy: i don't know anyone per se, would just look in other changes. is there a bug for that? | 15:45 |
| haleyb | like takashi and stephen | 15:45 |
| otherwiseguy | stephen is on PTO until Mar 19. And I somehow missed that tkajinam was in this channel. :) | 15:46 |
| haleyb | related to https://bugs.launchpad.net/neutron/+bug/2056366 ? | 15:46 |
| otherwiseguy | haleyb: yeah. | 15:47 |
| haleyb | ralonsoh: so are you going to need some more changes in 2024.1? just seeing your comment in one change about getting it merged this week? | 15:50 |
| tkajinam | otherwiseguy, I've been pulled by some release problems, sorry. I'll check it but we already passed requirement freeze deadline (which happend in the week of Feb 26) and I'm not too sure if we can/should have it for 2024.1 at this timing | 15:50 |
| tkajinam | and good to know that Stephen is on PTO now... though I may have to bother him to get his knowledge about sqlalchemy stuffs... | 15:51 |
| tkajinam | otherwiseguy, while I give some thoghts on it, probably we need a bug (probably you add oslo.service to that neutron bug ?) so that we raise exception or get it backported after 2024.1 release | 15:52 |
| otherwiseguy | tkajinam: thanks, understood. Hopefully it is something we can get backported eventually. They only project I saw currently using it was cinder, but neutron *not* using it is a bug in itself. We overwrite oslo's (and our own) signal handlers. | 15:53 |
| otherwiseguy | @tkajinam: I'll create a new oslo bug as well. Thanks. | 15:53 |
| ralonsoh | haleyb, I think you have already discussed LP#2056366 | 16:01 |
| ralonsoh | I hope it could be backportable (the oslo patch) | 16:01 |
| ralonsoh | and about other patches... I don't know really | 16:01 |
| ralonsoh | I was expecting the ovn agent patches to be merged | 16:01 |
| ralonsoh | and https://review.opendev.org/q/topic:%22bug/2052821%22 | 16:02 |
| ralonsoh | haleyb, but I think it is too late for this now | 16:02 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/898238 | 16:03 |
| ralonsoh | 5 months there a very little reviews... what a pity | 16:03 |
| ralonsoh | very few* | 16:03 |
| haleyb | ralonsoh: sorry, i should have looked again last week after you updated, but i didn't see a FFE for it | 16:08 |
| haleyb | and regarding the other bug, it's only marked medium so i didn't know the impact | 16:09 |
| ralonsoh | https://review.opendev.org/q/topic:%22bug/2052821%22 is "kind of" a feature, but identifies as a bug | 16:09 |
| ralonsoh | and the ovn agent metadata extension does not interfere in the current code | 16:10 |
| ralonsoh | but having it in a slurp release, could have been better to start the ovn metadata agent deprecation | 16:10 |
| ralonsoh | I' | 16:10 |
| ralonsoh | sorry | 16:10 |
| ralonsoh | I'll postpone that | 16:10 |
| ralonsoh | actually I need to push a 2024.2 spec for that, to start the ovn metadata agent deprecation | 16:10 |
| fungi | lajoskatona: even if you just wanted to give https://review.opendev.org/912662 a quick once-over that's probably sufficient. the rest are all direct cherry-picks of that. most applied directly, a couple i had to make very minor adjustment to because of nearby churn in the file, but i've already double-checked them too | 16:13 |
| haleyb | ralonsoh: understood. we do have 24 hours if we want to merge the ovn-agent metadata extension | 16:18 |
| ralonsoh | haleyb, that could be awesome | 16:18 |
| ralonsoh | as commented, this is not touching anything that currently runs in any deployment | 16:18 |
| ralonsoh | is just adding the ovn agent extension | 16:18 |
| ralonsoh | that will help us to deprecate the older agent | 16:18 |
| lajoskatona | haleyb, ralonsoh, ykarel: perhaps you can also vote on the patch from fungi to remove centos jobs from vpnaas: https://review.opendev.org/912662, thanks in advance | 16:19 |
| ralonsoh | let me check | 16:19 |
| ralonsoh | +2 | 16:19 |
| haleyb | ralonsoh: will probably need slaweq to look at https://review.opendev.org/c/openstack/neutron/+/898238 as well :) and i will -1 the release patch for now | 16:20 |
| fungi | well, really just giving folks an opportunity to make sure i didn't have a serious typo or something. if we don't merge these, those branches will all have zuul config errors starting day after tomorrow | 16:20 |
| ralonsoh | haleyb, thanks a lot | 16:20 |
| fungi | thanks! | 16:20 |
| opendevreview | Merged openstack/neutron-vpnaas master: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912662 | 16:22 |
| opendevreview | Merged openstack/neutron-vpnaas stable/2024.1: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912663 | 16:22 |
| opendevreview | Merged openstack/neutron-vpnaas stable/2023.2: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912664 | 16:23 |
| opendevreview | Merged openstack/neutron-vpnaas stable/2023.1: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912665 | 16:23 |
| opendevreview | Merged openstack/neutron-vpnaas stable/zed: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912666 | 16:23 |
| opendevreview | Merged openstack/neutron-vpnaas unmaintained/yoga: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912667 | 16:23 |
| opendevreview | Merged openstack/neutron-vpnaas unmaintained/xena: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912668 | 16:23 |
| opendevreview | Merged openstack/neutron-vpnaas unmaintained/wallaby: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912669 | 16:23 |
| opendevreview | Merged openstack/neutron-vpnaas unmaintained/victoria: Zuul: Remove references to centos-7 job nodes https://review.opendev.org/c/openstack/neutron-vpnaas/+/912670 | 16:23 |
| haleyb | slaweq: don't know if you're still around but can you take a look at https://review.opendev.org/c/openstack/neutron/+/898238 ? trying to get that in this cycle | 17:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!