| *** elodille1 is now known as elodilles_ooo | 06:13 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: vpnaas: Use unique CIDRs per scenario test class https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004437 | 06:32 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: DNM == Test ``neutron-tempest-plugin-vpnaas`` scenario classes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004439 | 06:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: DNM == Test ``neutron-tempest-plugin-vpnaas`` scenario classes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004439 | 06:37 |
| ralonsoh | ykarel, slaweq hello! please check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004437 if you have 1 min | 06:45 |
| ralonsoh | thanks! | 06:45 |
| ralonsoh | the upper testing patch is removing the unstable decorator and testing the same tests many times | 06:45 |
| ralonsoh | https://zuul.opendev.org/t/openstack/status?change=1004437&change=1003820 | 06:46 |
| ykarel | ack | 06:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: vpnaas: Run scenario tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004567 | 07:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: vpnaas: Use unique CIDRs per scenario test class https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004437 | 07:28 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: DNM == Test ``neutron-tempest-plugin-vpnaas`` scenario classes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004439 | 07:28 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: l3: Fix HA state file permissions for keepalived monitor https://review.opendev.org/c/openstack/neutron/+/1004419 | 07:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Remove standalone OVN Metadata agent https://review.opendev.org/c/openstack/neutron/+/998787 | 07:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: conf: Move VPNaaS configuration options to neutron_vpnaas/conf https://review.opendev.org/c/openstack/neutron-vpnaas/+/1004571 | 08:14 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: DNM == Stress the BGP job IPv6 DAD fix https://review.opendev.org/c/openstack/neutron/+/1004573 | 08:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: DNM == Test ``neutron-tempest-plugin-vpnaas`` scenario classes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004439 | 08:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: vpnaas: Add swanctl (VICI protocol) support for strongSwan https://review.opendev.org/c/openstack/neutron-vpnaas/+/1003801 | 08:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: conf: Move VPNaaS configuration options to neutron_vpnaas/conf https://review.opendev.org/c/openstack/neutron-vpnaas/+/1004571 | 08:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: zuul: add OVN VPNaaS tempest job with swanctl https://review.opendev.org/c/openstack/neutron-vpnaas/+/1003820 | 09:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: strongswan: Default ``use_swanctl`` to True https://review.opendev.org/c/openstack/neutron-vpnaas/+/1004575 | 09:17 |
| slaweq | ralonsoh: ykarel lajoskatona hi, can you check couple of my neutron-tempest-plugin patches: https://review.opendev.org/q/project:openstack/neutron-tempest-plugin+owner:skaplons@redhat.com+status:open++label:Verified%252B1 ? Thx in advance | 09:56 |
| ralonsoh | checking now | 09:57 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [bgp] Bump cirros version to 0.6.2 https://review.opendev.org/c/openstack/neutron/+/1002065 | 10:07 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [bgp] Bump cirros version to 0.6.3 https://review.opendev.org/c/openstack/neutron/+/1002065 | 10:12 |
| opendevreview | Merged openstack/neutron master: Implement indirect Floating IPs API https://review.opendev.org/c/openstack/neutron/+/1003262 | 10:19 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [BGP] Remove test from tempest_exclude_regex https://review.opendev.org/c/openstack/neutron/+/1001108 | 10:29 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [BGP] Add new BGP API and scenario tests https://review.opendev.org/c/openstack/neutron/+/1002233 | 10:29 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [bgp] Bump cirros version to 0.6.3 https://review.opendev.org/c/openstack/neutron/+/1002065 | 11:52 |
| *** haleyb|out is now known as haleyb | 12:25 | |
| *** elodilles is now known as elodilles_ooo | 12:47 | |
| haleyb | #startmeeting networking | 13:00 |
| opendevmeet | Meeting started Tue Sep 8 13:00:20 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| opendevmeet | The meeting name has been set to 'networking' | 13:00 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, slaweq, ykarel, lajoskatona, jlibosva, haleyb, ralonsoh, cardoe | 13:00 |
| mtomaska | o/ | 13:00 |
| ralonsoh | hello | 13:00 |
| cardoe | o/ | 13:00 |
| mlavalle | \o | 13:01 |
| cbuggy | o/ | 13:01 |
| haleyb | ok let's get started | 13:01 |
| haleyb | #topic announcements | 13:01 |
| haleyb | We are currently in Week R-3 of Hibiscus | 13:01 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/W6UM44BGT4M45RBZ3ZVEDPJOHZ77C4JB/ | 13:02 |
| haleyb | All deliverables released under a cycle-with-rc model should have a first | 13:02 |
| haleyb | release candidate by the end of the week, from which a stable/2026.2 | 13:02 |
| haleyb | branch will be cut. This branch will track the 2026.2 "Hibiscus" release. | 13:02 |
| haleyb | Once stable/2026.2 has been created, the master branch will be ready to | 13:02 |
| haleyb | switch to 2027.1 "Indri" development. While the master branch will no | 13:02 |
| haleyb | longer be feature-frozen, please prioritize any work necessary for | 13:02 |
| haleyb | completing 2026.2 "Hibiscus" plans. Release-critical bugfixes will need | 13:02 |
| haleyb | to be merged in the master branch first, then backported to the | 13:02 |
| haleyb | stable/2026.2 branch before a new release candidate can be proposed. | 13:02 |
| slaweq | o/ | 13:02 |
| haleyb | Early in the week, the release team will be proposing RC1 patches | 13:03 |
| opendevreview | Helen Chen proposed openstack/neutron-specs master: Propose spec EVPN Type-5 with Distributed Routing https://review.opendev.org/c/openstack/neutron-specs/+/1004590 | 13:03 |
| haleyb | RC1 deadline: 10 September, 2026 (R-3 week) | 13:03 |
| haleyb | Final RC deadline: 24 September, 2026 (R-1 week) | 13:03 |
| opendevreview | Helen Chen proposed openstack/neutron-specs master: Propose spec EVPN Type-5 with Distributed Routing https://review.opendev.org/c/openstack/neutron-specs/+/1004590 | 13:03 |
| haleyb | Final 2026.2 "Hibiscus" release: 30 September, 2026 | 13:03 |
| haleyb | we have quite a few patches on our priority board | 13:04 |
| haleyb | #link https://tinyurl.com/59z278km | 13:04 |
| haleyb | i started going through this morning, will look more later | 13:04 |
| haleyb | there were some FFE's that we approved, please prioritize those reviews | 13:05 |
| opendevreview | Helen Chen proposed openstack/neutron-specs master: Propose spec EVPN Type-5 with Distributed Routing https://review.opendev.org/c/openstack/neutron-specs/+/1004590 | 13:05 |
| haleyb | slaweq: i'll respond to your ML email about yours in a bit, was out friday and yesterday | 13:06 |
| haleyb | mlavalle: since i didn't see any more comments on yours, i'll approve the last two patches later | 13:06 |
| slaweq | haleyb: sure, but actually ralonsoh already approved my patch and it got merged today :) | 13:06 |
| haleyb | slaweq: oh, then it's just n-t-p patches? | 13:07 |
| slaweq | if you will say we shouldn't have it now I can eventually do revert for now | 13:07 |
| slaweq | n-t-p are ready to review | 13:07 |
| slaweq | but not yet merged | 13:07 |
| slaweq | they are green though | 13:07 |
| haleyb | slaweq: no, i think we should have it, i've just been offline and didn't respond | 13:07 |
| slaweq | thank you :) | 13:07 |
| haleyb | and i think all the bgp patches have merged as well | 13:08 |
| haleyb | were there any other FFEs or series that need attention this week? | 13:08 |
| haleyb | if so please set to RP +2 | 13:08 |
| haleyb | Reminder to please use the priority dashboard for anything neutron/networking related | 13:09 |
| haleyb | RP +1 for "ready to merge" changes, RP +2 for gate blockers and similar | 13:09 |
| haleyb | Reminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 13:10 |
| haleyb | there are a few on the list, can meet this friday assuming we have quorum | 13:10 |
| haleyb | and finally, the PTG is coming up in October | 13:10 |
| haleyb | "Indri" PTG October 12-16, 2026 | 13:11 |
| haleyb | PTG Website: https://openinfra.org/ptg/ | 13:11 |
| haleyb | that's all i had for announcements, any others? | 13:11 |
| haleyb | #topic bugs | 13:12 |
| haleyb | slaweq was the deputy last week, his report is at | 13:12 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/46YMU4SJ5H3TAQZHD3PFKDW4E773T27Z/ | 13:12 |
| haleyb | busy week :( | 13:12 |
| slaweq | it seems like it should be your week haleyb :D | 13:13 |
| slaweq | as usually you have so many bugs during the week | 13:13 |
| haleyb | slaweq: hah, i am this week, will hopefully be slow | 13:13 |
| slaweq | haha | 13:13 |
| haleyb | first unassigned is | 13:13 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2166083 | 13:13 |
| haleyb | test_create_event_advances_fsm functional test sporadic failure | 13:14 |
| ralonsoh | I hope mtomaska could have some time to check it ^ | 13:14 |
| mtomaska | ACK | 13:14 |
| slaweq | this one I marked as critical as this is teh gate-failure category | 13:14 |
| ralonsoh | thanks | 13:14 |
| mtomaska | I will check it out | 13:14 |
| haleyb | ack, thanks for looking at it mtomaska | 13:14 |
| haleyb | next unassigned is this | 13:15 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2166018 | 13:15 |
| haleyb | ML2/OVS agent can delete stale per-port flows before reinstalling them, causing intermittent traffic loss on agent re-start | 13:15 |
| haleyb | but i see it has a patch proposed | 13:15 |
| slaweq | ups, I probably missed it | 13:16 |
| haleyb | i will assign to Lajos as he is working on it | 13:16 |
| haleyb | slaweq: i thought the automation would have updated the owner | 13:16 |
| slaweq | yeah, it should | 13:17 |
| * haleyb shrugs | 13:18 | |
| haleyb | the next one is | 13:18 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2166602 | 13:18 |
| haleyb | port list returns 500 (KeyError: binding:vnic_type) for unbound ports with QoS policy | 13:18 |
| ralonsoh | I reviwed it and I have questions | 13:18 |
| ralonsoh | we can: | 13:18 |
| ralonsoh | 1) Leave it as is now. This is happening when we have a failed live migration | 13:19 |
| ralonsoh | the port doesn't have an associated port_binding register. This is an error | 13:19 |
| ralonsoh | if the user see that, he/she will be forced to fix it | 13:19 |
| ralonsoh | 2) Handle it and log a big error | 13:19 |
| ralonsoh | but that will hide the error in the logs | 13:19 |
| ralonsoh | opinions?? | 13:19 |
| slaweq | I set it as High because I don't think we should have any error 500 returned to the user. IMHO we should handle such error and return something "nicer" | 13:20 |
| ralonsoh | we should return the port list, without this info | 13:20 |
| ralonsoh | and log the missing port_binding in the logs | 13:20 |
| ralonsoh | but we can't retunr the list + an error in the CLI | 13:21 |
| ralonsoh | (2) is nicer and doesn't affect other tenants | 13:21 |
| slaweq | we can LOG error or even CRITICAL, no? | 13:21 |
| ralonsoh | yes | 13:21 |
| slaweq | IMO that should be enough | 13:21 |
| ralonsoh | perfect then, I'll go for (2) | 13:21 |
| ralonsoh | that's all | 13:22 |
| haleyb | i would agree, thanks for working on it ralonsoh | 13:22 |
| haleyb | next unassigned one is | 13:22 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2166653 | 13:23 |
| haleyb | IPv4/IPv6 fixed_ips land on different segments for ports on multi-segment-mapped hosts | 13:23 |
| haleyb | ok, so this was created from another one that was split in two | 13:24 |
| ralonsoh | I'll check it tomorrow | 13:24 |
| slaweq | yes, there were two bugs reported as one so I though it will be better to treat them separately | 13:24 |
| slaweq | so I closed original one and reported two others instead | 13:25 |
| opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2026.2: Update .gitreview for stable/2026.2 https://review.opendev.org/c/openstack/os-vif/+/1004592 | 13:26 |
| haleyb | ralonsoh: ack, this looks like something claude could handle pretty well assuming it's all correct | 13:26 |
| opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2026.2: Update TOX_CONSTRAINTS_FILE for stable/2026.2 https://review.opendev.org/c/openstack/os-vif/+/1004593 | 13:26 |
| opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/2026.2 https://review.opendev.org/c/openstack/os-vif/+/1004594 | 13:26 |
| ralonsoh | haleyb, I checked that yesterday with claude | 13:26 |
| ralonsoh | the report is legit and is affecting the dhcp agent | 13:26 |
| haleyb | great | 13:27 |
| haleyb | the sister bug is | 13:27 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2166654 | 13:27 |
| haleyb | DHCP agent (dnsmasq) includes subnets from non-local segments in `--dhcp-range` | 13:27 |
| slaweq | this one seems for me like less impactful then the previous one | 13:27 |
| ralonsoh | agree | 13:28 |
| slaweq | as even if we have all subnets in dnsmasq, we have static assignment for each port so IMHO this is more like minor thing | 13:29 |
| haleyb | i can burn some tokens on it later and see what it comes up with, have to use my quota | 13:29 |
| slaweq | lucky you, we have to safe ours :D | 13:30 |
| slaweq | *save | 13:31 |
| haleyb | i have a new bucket of tokens, just refreshed the other day | 13:31 |
| haleyb | so there was a third bug filed by the same person | 13:31 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2166426 | 13:31 |
| haleyb | neutron-dhcp-agent: dnsmasq processes conflict when two segments of the same network share the same physical_network (routed provider network on a single host) | 13:32 |
| slaweq | yeah, this one I would like someone who is more familiar with routed provider networks to check | 13:32 |
| * slaweq looks at mlavalle :) | 13:32 | |
| haleyb | i was going to do the same | 13:32 |
| mlavalle | I can take a look | 13:33 |
| haleyb | mlavalle: ack, thanks | 13:34 |
| haleyb | that was it for unassigned bugs | 13:34 |
| haleyb | there were also some RFEs, i will look at those and we can discuss in drivers meeting | 13:35 |
| haleyb | this week i am the bug deputy, next week is mlavalle | 13:35 |
| haleyb | is that still good for you mlavalle ? | 13:36 |
| haleyb | Current bug count this week: 608, up 10 from last week | 13:37 |
| haleyb | but it was a busy week | 13:37 |
| haleyb | any other bugs to discuss? | 13:37 |
| haleyb | #topic specs | 13:38 |
| haleyb | #link https://review.opendev.org/q/project:openstack/neutron-specs+status:open | 13:38 |
| haleyb | at this point 2026.2 is past, will need to make a 2027.1 folder and move things there | 13:39 |
| haleyb | i can create the placeholder later, then people can update those | 13:40 |
| ralonsoh | (we need a skill for that, to create these patches...) | 13:40 |
| haleyb | true, the beginning of cycle work can be somewhat automated | 13:41 |
| ichen | I submitted a proposed spec this morning, about EVPN Type-5 route using distributed routing, but we haven’t discussed the RFE yet. | 13:41 |
| haleyb | ichen: yes, saw that, should be able to discuss this friday at meeting if you can attend | 13:42 |
| ichen | Sounds good. | 13:43 |
| haleyb | ok, we can move on | 13:43 |
| haleyb | #topic community goals | 13:43 |
| haleyb | we can go through our active series | 13:43 |
| haleyb | #link https://review.opendev.org/q/topic:%22sdk_for_neutron%22 | 13:43 |
| haleyb | i had reviewed the first few of the series, don't see any other from nova folks, will likely be in next cycle as not critical for them | 13:45 |
| haleyb | this is the set of nova patches to move to the sdk | 13:45 |
| * haleyb realized lajos is not here | 13:46 | |
| haleyb | #link https://review.opendev.org/q/topic:%2522neutron_openstacksdk%2522 | 13:46 |
| haleyb | this is the heat series for neutron sdk | 13:46 |
| haleyb | made progress with reviews | 13:46 |
| haleyb | and the OSC migration series has one OSC patch left, which is now green | 13:47 |
| haleyb | #link https://review.opendev.org/q/topic:%2522migrate_stadium_osc%2522 | 13:47 |
| haleyb | #link https://review.opendev.org/c/openstack/python-openstackclient/+/995991 | 13:47 |
| haleyb | cardoe: oh, i forgot to raise an exception for that last one | 13:48 |
| haleyb | but it's not a big deal, can just finish next cycle | 13:48 |
| haleyb | final community topic is the tenant_id removal work | 13:49 |
| haleyb | #link https://review.opendev.org/q/topic:%22neutron-project-id-migration%22 | 13:49 |
| haleyb | need to wait for nova series ^^ to merge, so will pick this up in 2027.1 | 13:49 |
| haleyb | #topic on-demand | 13:49 |
| haleyb | we already talked about the bug listed there, are there any other topics? | 13:50 |
| mlavalle | I have a topic | 13:50 |
| haleyb | sure | 13:50 |
| mlavalle | I have two patches in the routed networks series both with two +2s. I need someone to merge them: | 13:51 |
| mlavalle | https://review.opendev.org/c/openstack/neutron/+/975111 | 13:51 |
| mlavalle | and: | 13:51 |
| mlavalle | https://review.opendev.org/c/openstack/neutron/+/995612 | 13:51 |
| mlavalle | that's all | 13:51 |
| opendevreview | Merged openstack/os-vif master: Update master for stable/2026.2 https://review.opendev.org/c/openstack/os-vif/+/1004594 | 13:51 |
| haleyb | mlavalle: ack, i will approve after meeting as they have no more comments | 13:52 |
| mlavalle | thanks | 13:52 |
| ralonsoh | hi folks, check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004437 --> I want to remove the unstable decorator for the scenario tests | 13:52 |
| ralonsoh | this decorator: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004439/5/neutron_tempest_plugin/vpnaas/scenario/test_vpnaas.py | 13:52 |
| ralonsoh | that's all | 13:52 |
| haleyb | i had a couple of items i forgot to mention at beginning of meeting | 13:53 |
| haleyb | first, as it's the end of cycle i need to generate the cycle highlights | 13:53 |
| haleyb | if you have something you want in there please let me know, otherwise i will go through things and make a list, then send it for review | 13:54 |
| ralonsoh | send here the patch once in gerrit | 13:54 |
| ralonsoh | and we'll review it too | 13:54 |
| haleyb | sure | 13:54 |
| haleyb | the second was about the contributor/maintainer experience survey | 13:55 |
| haleyb | there was an email to the list last week from Ildiko | 13:55 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/LAQI5DADVUDFSAPQ4YL4Q6L5ZWO34K7K/ | 13:56 |
| opendevreview | Merged openstack/os-vif stable/2026.2: Update .gitreview for stable/2026.2 https://review.opendev.org/c/openstack/os-vif/+/1004592 | 13:56 |
| haleyb | if you haven't already, please take the survey - would be good to have more data points on this | 13:56 |
| haleyb | "During this round the surveys explore what challenges maintainers and contributors face, along with how sustainable they find the time and effort they’ve been investing into the project." | 13:56 |
| haleyb | there are separate ones for maintainer and contributor | 13:57 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [FWaaS] Add basic scenario test for FW attached to L2 ports https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/995695 | 13:57 |
| haleyb | that was all i had, anything else? | 13:57 |
| haleyb | ok, thanks for attending, have a good week everyone! | 13:58 |
| haleyb | #endmeeting | 13:58 |
| opendevmeet | Meeting ended Tue Sep 8 13:58:22 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:58 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2026/networking.2026-09-08-13.00.html | 13:58 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2026/networking.2026-09-08-13.00.txt | 13:58 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2026/networking.2026-09-08-13.00.log.html | 13:58 |
| ralonsoh | bye | 13:58 |
| mlavalle | \o | 13:58 |
| ralonsoh | please | 13:58 |
| ralonsoh | review the doc changes about the core members | 13:58 |
| ralonsoh | https://review.opendev.org/q/topic:%22neutron_core_membership%22 | 13:58 |
| ralonsoh | not a priority, but good to have | 13:58 |
| mlavalle | haleyb, since we are merging the routed networks code, we can include it in the cycle highlights | 13:59 |
| opendevreview | Merged openstack/os-vif stable/2026.2: Update TOX_CONSTRAINTS_FILE for stable/2026.2 https://review.opendev.org/c/openstack/os-vif/+/1004593 | 13:59 |
| haleyb | mlavalle: ack, will do, if you have a specific sentence feel free to send it along | 13:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: qos: Handle missing port binding in resource request https://review.opendev.org/c/openstack/neutron/+/1004651 | 14:33 |
| opendevreview | Merged openstack/neutron stable/2025.1: [CI][FT][2025.1 Only] Drop extra passenv from tox.ini https://review.opendev.org/c/openstack/neutron/+/1004405 | 14:35 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Turn back on `ovn_router_indirect_snat` in the ovn job https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1003633 | 15:03 |
| opendevreview | Merged openstack/neutron master: Fix ungrouped-imports pylint warnings https://review.opendev.org/c/openstack/neutron/+/1000733 | 15:03 |
| opendevreview | Merged openstack/neutron-lib master: Fix arguments-differ pylint warnings https://review.opendev.org/c/openstack/neutron-lib/+/1000907 | 15:05 |
| opendevreview | Merged openstack/neutron-lib master: Fix inconsistent-return-statements pylint warnings https://review.opendev.org/c/openstack/neutron-lib/+/1000912 | 15:05 |
| opendevreview | Merged openstack/neutron-lib master: Remove pylint disable for using-constant-test https://review.opendev.org/c/openstack/neutron-lib/+/1000905 | 15:05 |
| opendevreview | Merged openstack/neutron master: Fix use-dict-literal pylint warnings https://review.opendev.org/c/openstack/neutron/+/1000648 | 17:23 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-tempest-plugin master: Add API tests for subnet leak_routes attribute https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/999016 | 17:24 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Make spine and leaf ansible roles generic https://review.opendev.org/c/openstack/neutron/+/1002215 | 17:30 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-tempest-plugin master: bgp: Add BGP tests with leak_routes attribute https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/999007 | 17:37 |
| haleyb | slaweq: you might be offline already, but https://review.opendev.org/c/openstack/neutron/+/1002499 and the 2025.1 one need a small update. was trying to get new tags out and figure we can include those. tia | 18:30 |
| haleyb | and if you are still there would be good to get this merged as well - https://review.opendev.org/c/openstack/neutron/+/999158 - or i'll ping rodolfo tomorrow | 18:33 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Implement multiple segments per host for routed networks https://review.opendev.org/c/openstack/neutron/+/975111 | 18:33 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Add segment maintenance tasks for routed networks https://review.opendev.org/c/openstack/neutron/+/995612 | 18:51 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Agent extension for segment bridges https://review.opendev.org/c/openstack/neutron/+/982018 | 18:55 |
| mlavalle | haleyb, I had to rebase https://review.opendev.org/c/openstack/neutron/+/975111, https://review.opendev.org/c/openstack/neutron/+/995612 and https://review.opendev.org/c/openstack/neutron/+/982018. I took the opportunity to address a couple of nits you pointed out. Please +2 and approve again | 19:00 |
| haleyb | mlavalle: ack, saw that, will look again soon | 19:01 |
| cardoe | haleyb: https://review.opendev.org/c/openstack/neutron/+/1000485 I'm still hoping that lands before 2026.2 | 19:03 |
| cardoe | That's the basis of making https://docs.openstack.org/neutron/latest/admin/data-center-networks.html#physical-layout work with that allocator | 19:04 |
| cardoe | it also allows me to fix (cannot recall the bug number right now) that Rodolfo filed about the VXLAN allocator being too slow due to populating everything. | 19:05 |
| haleyb | cardoe: i'm trying to remember context, was this the one that we should merge to not have an API change? and it was merged this cycle originally? | 19:05 |
| cardoe | Yeah the API was merged this cycle originally | 19:07 |
| cardoe | So if I can fix the API within cycle it'll let us not have an API break in the future. | 19:07 |
| cardoe | mtomaska__: says he cannot remember the context | 19:08 |
| haleyb | cardoe: right. i'll take a look, we'll need to ping another core as well | 19:08 |
| cardoe | but going through the whole reason Jakub and Helen broke up the API like that was for my asks around the VNI understanding VXLAN underlay. | 19:08 |
| cardoe | It'd allow us to share all the same code. | 19:08 |
| cardoe | But for it to actually work the VXLAN "pool" needs to be different from the VLAN "pool" | 19:09 |
| haleyb | cardoe: maybe we should ping jlibosva then for a review, should be around this week | 19:09 |
| haleyb | and it closes a bug so we shouldn't need an FFE | 19:10 |
| haleyb | and what about https://review.opendev.org/c/openstack/neutron/+/1000482 ? | 19:11 |
| haleyb | they both close the same bug? | 19:12 |
| cardoe | Yes. Two different implementations. | 19:14 |
| cardoe | One added another argument to the function. But I was concerned that the API would be easy to mess up. | 19:15 |
| cardoe | Since it was just func(ctxt, int, int, str, int, int, str) | 19:15 |
| cardoe | So 485 creates some distinct types with dataclasses and it becomes func(ctxt, VniRange, VlanRange) | 19:16 |
| cardoe | I'm happy with either approach. | 19:18 |
| haleyb | cardoe: and were you going to update 485 based on the comments? | 19:29 |
| cardoe | Yeah I can do it now. | 19:31 |
| haleyb | mlavalle: seems you got tripped-up by a recent pylint change - {} is preferred over dict() | 19:44 |
| opendevreview | Doug Goldstein proposed openstack/neutron master: Use separate VNI and VLAN ScopedRanges in VNIVLANAllocator https://review.opendev.org/c/openstack/neutron/+/1000485 | 19:45 |
| haleyb | and remember to RP +1 so they show up on the board again, that isn't sticky | 19:45 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Fixed attribute-defined-outside-init pylint warnings https://review.opendev.org/c/openstack/neutron-lib/+/1000908 | 19:56 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Fix abstract-method pylint warnings https://review.opendev.org/c/openstack/neutron-lib/+/1000906 | 19:57 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Add negative API tests for the indirect Floating IPs API extension https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1003264 | 20:36 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [FWaaS] Clean up firewall group on scenario failure https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004403 | 20:41 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [FWaaS] Use Neutron Tempest plugin test bases https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1004404 | 20:41 |
| opendevreview | Merged openstack/neutron master: doc: expand Neutron core reviewer membership expectations https://review.opendev.org/c/openstack/neutron/+/1003430 | 20:41 |
| opendevreview | Merged openstack/neutron master: doc: rename contributor/contribute.rst to new_extensions.rst https://review.opendev.org/c/openstack/neutron/+/1003913 | 20:41 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: ovn-evpn: Fix race in EVPN FSM functional test https://review.opendev.org/c/openstack/neutron/+/1004703 | 20:47 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Implement multiple segments per host for routed networks https://review.opendev.org/c/openstack/neutron/+/975111 | 20:53 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Add segment maintenance tasks for routed networks https://review.opendev.org/c/openstack/neutron/+/995612 | 20:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: qos: Handle missing port binding in resource request https://review.opendev.org/c/openstack/neutron/+/1004651 | 20:58 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Agent extension for segment bridges https://review.opendev.org/c/openstack/neutron/+/982018 | 21:00 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Ensure rt_tables file exists before reading https://review.opendev.org/c/openstack/ovn-bgp-agent/+/976242 | 21:05 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Wait for link-local before FRR peer ping check https://review.opendev.org/c/openstack/neutron/+/1004718 | 21:28 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!