Friday, 2026-09-18

opendevreviewMerged openstack/neutron-tempest-plugin master: Drop AddressScope.shared from API tests  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/100571500:39
opendevreviewHelen Chen proposed openstack/neutron master: EVPN router always redistributes static routes  https://review.opendev.org/c/openstack/neutron/+/100613300:54
opendevreviewHelen Chen proposed openstack/neutron master: EVPN router always redistributes static routes  https://review.opendev.org/c/openstack/neutron/+/100613300:57
*** ykarel__ is now known as ykarel06:23
opendevreviewKyuyeong Lee proposed openstack/neutron-lib master: port-group: Add API definitions  https://review.opendev.org/c/openstack/neutron-lib/+/100615907:26
zigoQuestion: with the Hibiscus release, is ovn-bgp-agent obsolete?07:33
LarsErikPhaleyb: thanks! 07:52
opendevreviewKyuyeong Lee proposed openstack/neutron master: Open the 2027.1 (Indri) DB branch  https://review.opendev.org/c/openstack/neutron/+/100616108:04
opendevreviewKyuyeong Lee proposed openstack/neutron-lib master: port-group: Add API definitions  https://review.opendev.org/c/openstack/neutron-lib/+/100615908:08
opendevreviewKyuyeong Lee proposed openstack/neutron master: Open the 2027.1 (Indri) DB branch  https://review.opendev.org/c/openstack/neutron/+/100616108:10
opendevreviewRodolfo Alonso proposed openstack/ovsdbapp master: ovn-sb: Set default ``hostname`` in ``chassis_add``  https://review.opendev.org/c/openstack/ovsdbapp/+/100520708:50
opendevreviewRodolfo Alonso proposed openstack/neutron-lib master: api: fix validate_ip_address() to handle None input  https://review.opendev.org/c/openstack/neutron-lib/+/100589308:55
opendevreviewKyuyeong Lee proposed openstack/neutron master: db: Add port group models  https://review.opendev.org/c/openstack/neutron/+/100617108:56
opendevreviewKyuyeong Lee proposed openstack/neutron master: objects: Add port group objects  https://review.opendev.org/c/openstack/neutron/+/100617208:56
opendevreviewKyuyeong Lee proposed openstack/neutron master: api: Add port group CRUD  https://review.opendev.org/c/openstack/neutron/+/100617308:56
opendevreviewRodolfo Alonso proposed openstack/neutron-lib master: api: fix convert_to_boolean_if_not_none() to handle conversion failures  https://review.opendev.org/c/openstack/neutron-lib/+/100590509:11
opendevreviewPiotr Milewski proposed openstack/neutron-vpnaas master: Preserve system timezone in strongSwan namespace  https://review.opendev.org/c/openstack/neutron-vpnaas/+/100618210:03
opendevreviewBharath M V proposed openstack/neutron-tempest-plugin master: [TaaS] Add MTU test to Tap-as-a-Service.  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/100618410:08
opendevreviewMerged openstack/neutron-lib master: db: fix is_retriable() exception handling  https://review.opendev.org/c/openstack/neutron-lib/+/100589110:54
opendevreviewRodolfo Alonso proposed openstack/neutron-lib master: api-ref: fix typos on default SGs listing  https://review.opendev.org/c/openstack/neutron-lib/+/100181611:04
opendevreviewEduardo Olivares proposed openstack/neutron-tempest-plugin master: evpn: Fix disk_over_commit handling in live migration test  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/100600611:15
opendevreviewPiotr Milewski proposed openstack/neutron-vpnaas master: Preserve system timezone in strongSwan namespace  https://review.opendev.org/c/openstack/neutron-vpnaas/+/100618211:40
opendevreviewyatin proposed x/whitebox-neutron-tempest-plugin master: Make db sync work with master podified deployment  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/100619711:48
opendevreviewEduardo Olivares proposed openstack/neutron-tempest-plugin master: evpn: Reuse ip.IPCommand and shell.execute in EVPN provisioner  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/100592212:43
haleyb#startmeeting neutron_drivers13:00
opendevmeetMeeting started Fri Sep 18 13:00:22 2026 UTC and is due to finish in 60 minutes.  The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot.13:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:00
opendevmeetThe meeting name has been set to 'neutron_drivers'13:00
haleybPing list: ykarel, mlavalle, mtomaska, slaweq, tobias-urdin, lajoskatona, haleyb, ralonsoh, cardoe13:00
mlavalle\o13:00
ralonsohhello13:00
lajoskatonao/13:00
haleybi'm not sure slaweq will be here since he added one for next week, but there is one item on the agenda13:01
haleybo/ btw, i'm not all business :)13:01
haleybichen: are you here? you had the one item carried over from last week13:02
mlavalleslaweq is on a business trip, so he might not join13:02
haleyb#link https://bugs.launchpad.net/neutron/+bug/216602713:02
haleyb[RFE] BGP EVPN Type-2 Route support for Neutron ML2/OVN backend13:02
ichenYes, I’m here.13:02
ichenI wrote the EVPN Type-2 spec.  I also submitted the spec.13:03
lajoskatonahttps://review.opendev.org/c/openstack/neutron-specs/+/100490813:04
haleybso since i don't use evpn every day, type-2 is basically per-host? like a /32 or /128 ?13:04
ichenType-2 is always distributed, meaning that only the Mac/ip routes local to a chassis is advertised by the chassis and doesn’t require the chassis routers like for type-5 distributed routing.13:04
haleybor per-IP13:04
ichenType-2 is Mac routes.13:05
ichenIt’s a BGP router saying that to reach MAC1, send to me.13:05
haleybalright, makes sense now13:06
ichenSo, instead of querying for the MAC, the MAC is advertised.13:06
haleybso the downside is a lot more entries?13:06
ichenYes.13:07
haleybi had no more questions, have not read the spec though13:08
lajoskatonabut those are in FRR, or am I wrong?13:08
ichenI don’t know if OVN has the option to do it now or later, but OVN implemented a new option called dynamic-routing-advertise, which can be set to false, if a user needs to cherry-pick to not advertise.  Right now, this option is only available for IP routes, such as static routes and host routes.13:09
ichenlajoskatona: what are in FRR?13:09
haleybdon't know if lajoskatona is still here?13:11
ichenDo you mean that MACs to be advertised?  OVN relays the MAC addresses to FRR/BGP and FRR/BGP advertises them.  The remote site does the same thing and OVN listens for the MAC addresses that FRR/BGP adds into the kernel and creates something (open flow rules or Mac bindings) based on MAC addresses learned from remote site.13:11
lajoskatonasorry, seems like I was dropped, or my network has issues, but back13:12
ichenI also chatted with cardoe separately to understand his request.  I’ll add an option to the spec.13:13
lajoskatona+1 for spec update13:14
ralonsohCan we have both type 2 and type 5 working at the same time? Are these 2 different extensions?13:14
ichenIt is possible.  But the spec is for pure type-2.13:15
ralonsohyes, I know13:16
ichenPure type-2 means one single ethernet segment.13:16
ralonsohbut is possible to have a network configured with evpn type 2 and another network+router with type 5?13:16
ralonsohis that something possible?13:16
ichenA mix of type-2 and type-5 means that the network has multiple ethernet segments and there’s a router to route traffic from one ethernet segment to nother.13:16
ichenralonsoh: yes, that is possible.13:16
ralonsohperfect13:17
mlavallekike a routed network in evpn13:17
ralonsohI have no more questions. I know there are 2 RFEs (distributed and type 2) presented13:19
ralonsohso I don't know what will be the first feature to be implemented13:19
ralonsoh(just to be aware if there is an order)13:19
ichenType-2 is easier to implement now.13:19
ralonsohcool13:19
haleybok, should we vote?13:20
ralonsoh+1 for me (the spec is already there)13:20
lajoskatona+1 from me13:20
haleyb+1 from me, can continue discussion in the spec13:20
mlavalleit seems a reasonable next step to me and besides, we can discuss further in the spec13:20
mlavalleso +113:20
ralonsohthanks ichen!13:21
haleybok, i'll mark it approved13:21
ichenThanks, everyone!13:21
haleybare there any other on-demand topics?13:21
lajoskatonanot from me , thanks13:22
haleybok, then i guess we are done13:22
haleybthanks for attending, have a nice weekend!13:22
haleyb#endmeeting13:22
opendevmeetMeeting ended Fri Sep 18 13:22:58 2026 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:22
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-09-18-13.00.html13:22
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-09-18-13.00.txt13:22
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-09-18-13.00.log.html13:22
lajoskatonao/, have a nice weekend13:23
ralonsohbye!13:23
opendevreviewMohsen Sepandar proposed openstack/neutron-fwaas master: Enable fwaas_v2 driver in DVR mode  https://review.opendev.org/c/openstack/neutron-fwaas/+/98985714:43
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2026.2: dhcp: only use local subnets for dnsmasq --dhcp-range  https://review.opendev.org/c/openstack/neutron/+/100622414:55
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2026.1: dhcp: only use local subnets for dnsmasq --dhcp-range  https://review.opendev.org/c/openstack/neutron/+/100622514:55
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2025.2: dhcp: only use local subnets for dnsmasq --dhcp-range  https://review.opendev.org/c/openstack/neutron/+/100622614:55
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2025.1: dhcp: only use local subnets for dnsmasq --dhcp-range  https://review.opendev.org/c/openstack/neutron/+/100622714:55
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2026.2: ovo: Fix ``qos_network_policy_id`` dirty flag after DB load  https://review.opendev.org/c/openstack/neutron/+/100622814:56
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2026.1: ovo: Fix ``qos_network_policy_id`` dirty flag after DB load  https://review.opendev.org/c/openstack/neutron/+/100622914:57
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2025.2: ovo: Fix ``qos_network_policy_id`` dirty flag after DB load  https://review.opendev.org/c/openstack/neutron/+/100623014:57
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2025.1: ovo: Fix ``qos_network_policy_id`` dirty flag after DB load  https://review.opendev.org/c/openstack/neutron/+/100623114:57
opendevreviewRodolfo Alonso proposed openstack/ovsdbapp master: ovn-sb: Set default ``hostname`` in ``chassis_add``  https://review.opendev.org/c/openstack/ovsdbapp/+/100520715:05
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: vpnaas: Add swanctl (VICI protocol) support for strongSwan  https://review.opendev.org/c/openstack/neutron-vpnaas/+/100380115:07
opendevreviewMerged openstack/neutron master: Open the 2027.1 (Indri) DB branch  https://review.opendev.org/c/openstack/neutron/+/100616115:08
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: conf: Move VPNaaS configuration options to neutron_vpnaas/conf  https://review.opendev.org/c/openstack/neutron-vpnaas/+/100457115:09
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: zuul: add OVN VPNaaS tempest job with swanctl legacy mode  https://review.opendev.org/c/openstack/neutron-vpnaas/+/100382015:09
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: DNM == Test ``neutron-tempest-plugin-vpnaas-ovn-sswan-swanctl`` 20 times  https://review.opendev.org/c/openstack/neutron-vpnaas/+/100443215:10
opendevreviewMerged openstack/neutron-lib master: doc: Document URL path hyphen and underscore equivalence  https://review.opendev.org/c/openstack/neutron-lib/+/100585115:13
stephenfinralonsoh: haleyb: There are a lot more commits on neutron-lib `master` than in `stable/2026.2`. Is there a reason we don't want to backport a lot of the "Add ``is_filter`` to API definition" patches?15:15
stephenfiniiuc, without those you will see HTTP 400 errors if you attempt to use one of the affected params as a filter, which feels like a regression?15:16
stephenfinI ask because I still see failures on https://review.opendev.org/c/openstack/openstacksdk/+/1005450 even after https://review.opendev.org/c/openstack/neutron-lib/+/1005527 merged.15:17
stephenfinI'm putting that down to the library not being installed correctly for neutron (I'm debugging currently) but it's got me wondering about ☝️15:17
ralonsohstephenfin, but all we need is to release a new version of n-lib, right?15:18
ralonsohlet me check if openstacksdk-functional-devstack-networking-ext depends on n-lib15:18
stephenfinn-lib has branched though. To fix this in hibiscus you presumably need to cut a release from stable/2026.2?15:19
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: Rally scenario plugin API changed  https://review.opendev.org/c/openstack/neutron-vpnaas/+/100041715:19
ralonsohyes, I would need to backport and release a new 2026.2 n-lib version15:20
stephenfinOkay. I just wanted to confirm that that was the plan :)15:21
ralonsohstephenfin, the job openstacksdk-functional-devstack-networking-ext is not installing the neutron-lib patch, so we'll need a new version in Indri too15:21
stephenfinyup, I've just seen `Downloading neutron_lib-5.0.1-py3-none-any.whl.metadata (2.4 kB)` in `job-output.txt`15:22
ralonsohlet me check the releases patch that opens the new branches15:22
stephenfinralonsoh: you must not have been reading openstack-discuss recently https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/65GQWCGJRQNCLIA3L3A5AXLVTOPGNZQ7/15:23
stephenfinwe can't release from `master` until requirements branch, which I think doesn't happen until GA in 2 weeks :(15:24
ralonsohstephenfin, so we can still create a new 2026.2 n-lib release?15:24
ralonsohno15:25
stephenfinyes, but you'll need a requirements freeze exception15:25
stephenfingiven this is a regression (right?) that seems an easy one to grant15:25
ralonsohstephenfin, that wasn't a regression, it was a bug since the first implementation15:25
stephenfinoh, isn't the `is_filter` check a new thing in hibiscus? 15:26
stephenfinat least enforcement of it15:26
ralonsohok, yes, in the Neutron API15:27
ralonsohone sec15:27
ralonsohI mean the definition was wrong15:27
ralonsohand we made this mandatory in the API15:27
stephenfinright, that's what I mean15:27
ralonsohso yes, that makes both a regression15:27
haleybwe are well past library freeze (sorry in another meeting)15:27
ralonsohso when can I backport to n-lib and make a new release?15:28
stephenfinhaleyb: Yep, a requirements freeze exception would be needed here15:28
ralonsohand how should I request it? mail?15:28
stephenfinbut as I said above, I believe this is a regression and would warrant one15:28
stephenfinyep. here's an example from last cycle https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/H67NORC72ZELFUPBGFWUXVAKGMCK6TUU/#I2LMBBNY2DOM7YFNNVHS7H36MR75UASC15:29
stephenfinmaybe elodilles hasn't gone home yet and could confirm??15:30
ralonsohok, I think we need to backport a couple of patches (TaaS and VPNaaS)15:30
ralonsohhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_9ab/openstack/9ab3ef4d943b4ea58e43e90221d3cabe/testr_results.html15:31
stephenfinelodilles: if you're still about, we'd like to confirm if a requirememts freeze exception email is required to address a regression in neutron(-lib)15:31
stephenfinerr, s/to address a regression/to release a fix that addresses a regression/15:31
ralonsohstephenfin, we also need the fix in CLI: https://review.opendev.org/c/openstack/python-openstackclient/+/100552515:39
ralonsohso do we need to backport ^ this patch too?15:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: placement: Raise ``PlacementResourceProviderNotFound`` on delete  https://review.opendev.org/c/openstack/neutron-lib/+/100624915:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: taas: Add ``is_filter`` to all TaaS API definitions  https://review.opendev.org/c/openstack/neutron-lib/+/100625015:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: floatingip-pools: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625115:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: network-segment-range: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625215:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: auto-allocated-topology: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625315:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: metering: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625415:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: agent: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625515:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: flavors: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625615:39
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: logging: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625715:40
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: network-ip-availability: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625815:40
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: qos: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625915:40
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: segment: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100626015:40
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: doc: Document URL path hyphen and underscore equivalence  https://review.opendev.org/c/openstack/neutron-lib/+/100626115:40
opendevreviewStephen Finucane proposed openstack/neutron-lib stable/2026.2: db: fix is_retriable() exception handling  https://review.opendev.org/c/openstack/neutron-lib/+/100626215:40
ralonsohstephenfin, I don't think we need all these patches15:40
ralonsohat least for this problem15:40
ralonsohwhat about the openstackclient patch?15:40
stephenfinagreed, but they're all potential regressions, right?15:41
stephenfinwe need the OSC fix also, yes. If you want to propose that I can merge it15:41
ralonsohthese are improvements, nothing was using it15:41
ralonsohI'll propose the backport15:41
ralonsohand I'll send the mail for both projects15:41
ralonsohhaleyb, stephenfin mail sent, waiting for the link15:53
elodilleshi stephenfin ralonsoh . we are quite close to coordinated release (1.5 half weeks from now) so i'd rather not release any library at this point. we are too late for that :/ if the bug is a release critical, then neutron+requirements+release team might need assess the issue. otherwise i'd say if it's "only" regression, then let's wait with the release after the coordinated release and then do 15:53
elodillesa stable release.15:53
elodillesi don't really want any last minute churning because a new lib release causing errors in service deliverables :/15:54
ralonsohelodilles, none of the bugs are critical:15:54
ralonsoh* the neutron-lib one just prevents from using ?name in the API15:54
ralonsoh* the openstack-client can be overriden by using the correct ike policy15:54
ralonsohso no, none of them are critical15:55
ralonsoh--> from using ?name in the API **in TaaS**15:55
elodillesralonsoh: thanks, so i'd really suggest a stable release AFTER the official coordinated 2026.2 Hibiscus release15:55
ralonsohelodilles, can we update the stable/requirements file? just asking15:56
ralonsohafter the release, of course15:56
elodillesyou mean if we release neutron-lib, then the upper-constraints can be bumped?15:56
ralonsohyes, in the stable 2026.2 requirements branches15:56
elodillesyes, that is the usual process15:57
ralonsohperfect, I didn't know about stable branches in requirements, I thought that was static once released15:57
ralonsohthanks for the information! 15:57
elodilleswe do a stable release -> it's upper-constraints is bumped on stable/<series>15:57
ralonsohperfect, we can continue with the backports and release in October15:58
elodillesralonsoh: static for 3rd parties, but the new stable releases have to be consumed somehow ;)15:58
elodilles* the new openstack component's stable releases15:58
*** ralonsoh is now known as ralonsoh_ooo16:02
opendevreviewIvan Anfimov proposed openstack/neutron master: Fix typo  https://review.opendev.org/c/openstack/neutron/+/100629817:27
opendevreviewIvan Anfimov proposed openstack/neutron master: Fix typo  https://review.opendev.org/c/openstack/neutron/+/100629817:29
opendevreviewMerged openstack/neutron master: dhcp: only use local subnets for dnsmasq --dhcp-range  https://review.opendev.org/c/openstack/neutron/+/100586317:44
opendevreviewMerged openstack/neutron-lib stable/2026.2: placement: Raise ``PlacementResourceProviderNotFound`` on delete  https://review.opendev.org/c/openstack/neutron-lib/+/100624918:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: taas: Add ``is_filter`` to all TaaS API definitions  https://review.opendev.org/c/openstack/neutron-lib/+/100625018:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: floatingip-pools: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625118:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: network-segment-range: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625218:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: auto-allocated-topology: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625318:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: metering: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625418:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: agent: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625518:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: flavors: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625618:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: logging: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625718:00
opendevreviewMerged openstack/neutron-lib stable/2026.2: network-ip-availability: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625818:01
opendevreviewMerged openstack/neutron-lib stable/2026.2: qos: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100625918:01
opendevreviewMerged openstack/neutron-lib stable/2026.2: segment: Add ``is_filter`` to API definition  https://review.opendev.org/c/openstack/neutron-lib/+/100626018:06
opendevreviewMerged openstack/neutron-lib stable/2026.2: doc: Document URL path hyphen and underscore equivalence  https://review.opendev.org/c/openstack/neutron-lib/+/100626118:06
opendevreviewMerged openstack/neutron-lib stable/2026.2: db: fix is_retriable() exception handling  https://review.opendev.org/c/openstack/neutron-lib/+/100626218:07
haleybjlibosva: not sure if you're around, was trying to ping Terry about https://review.opendev.org/c/openstack/neutron/+/81885020:12
haleybAlso have to re-work the other issue you commented on20:12
jlibosvahaleyb: I am around :)20:22
jlibosvaI can ping him20:22
haleybjlibosva: great thanks. And regarding the other change, https://review.opendev.org/c/openstack/neutron/+/1005970 - the original bug that Lucas filed mentioned using external_ids to store cidrs. Is that a better option here?20:28
jlibosvahaleyb: that's what I meant with my comment. Except we are not interested in the cidrs for the subnet itself but only the gateway IP. With the traditional Neutron (L2) the gateway IP is outside of OpenStack - it'll be some DC GW. The only place the IP is stored nowadays is the DHCP_Options (which corresponds to the Neutron subnet resource), because the DHCP configures the20:31
jlibosvadefault egress route for the clients.20:31
jlibosvathus the limitation the external network needs to have the DHCP enabled - it doesn't even make sense to not have the DHCP unless the whole spine'n leaf is configured with some DHCP relays, I think20:32
jlibosvabut that said - if we can store the gw IP elsewhere - I think the Logical_Switch external_ids - then we won't need the dependency on the subnet dhcp enabled and we won't need to call the plugin object in the code too20:33
jlibosvabut if there is a better way, we can find it :)20:33
haleybjlibosva: it seems most of the code would live in that file, we'd have to see where it fills DHCP_Options and add something over there. So the BGP code only cares about the gateway_ip it seems?20:42
jlibosvahaleyb: yes, only gw ip21:02
haleybjlibosva: ack, i've sent Claude off with some new instructions, seems to be headed in the correct direction...21:05
jlibosvahaleyb: good :) btw Terry replied on the patch21:10
jlibosvajust fyi :)21:10
opendevreviewBrian Haley proposed openstack/neutron master: Populate external subnet routing info when dhcp disabled  https://review.opendev.org/c/openstack/neutron/+/100597021:11
haleybjlibosva: thanks again, and otherwiseguy too!21:13
opendevreviewTerry Wilson proposed openstack/neutron master: Use Neutron DB for OVN agent health  https://review.opendev.org/c/openstack/neutron/+/81885021:46
opendevreviewTerry Wilson proposed openstack/neutron master: Move OVN agent health monitoring to maintenance worker  https://review.opendev.org/c/openstack/neutron/+/99792421:48
opendevreviewJakub Libosvar proposed openstack/neutron master: evpn: Schedule VRF hosting  https://review.opendev.org/c/openstack/neutron/+/100634823:17
opendevreviewMerged openstack/neutron-lib master: api: fix convert_to_int() to handle None input  https://review.opendev.org/c/openstack/neutron-lib/+/100589423:39

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