| opendevreview | Merged openstack/neutron-lib master: Remove fip64 API definition and documentation https://review.opendev.org/c/openstack/neutron-lib/+/986742 | 01:03 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: Change ovn_router_indirect_snat config option to True https://review.opendev.org/c/openstack/neutron/+/987308 | 04:00 |
| *** ralonsoh is now known as ralonsoh_ooo | 06:24 | |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [OVN] Only set NAT gateway_port when distributed FIP is enabled https://review.opendev.org/c/openstack/neutron/+/987187 | 06:30 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add compute1 node to BGP multinode tempest job https://review.opendev.org/c/openstack/neutron/+/986075 | 07:23 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [OVN] Only set NAT gateway_port when distributed FIP is enabled https://review.opendev.org/c/openstack/neutron/+/987187 | 07:45 |
| opendevreview | Merged openstack/neutron stable/2026.1: Populate OVN AgentCache in the db-sync tool https://review.opendev.org/c/openstack/neutron/+/987122 | 09:40 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Use SDK for Nova notifications https://review.opendev.org/c/openstack/neutron/+/983905 | 10:20 |
| opendevreview | Harald Jensås proposed openstack/neutron master: ML2: tri-state checks for vlan_transparent/qinq https://review.opendev.org/c/openstack/neutron/+/987334 | 11:03 |
| tafkamax | Hi! | 11:25 |
| tafkamax | What are the thoughts on this patch? https://review.opendev.org/c/openstack/neutron/+/986349 | 11:26 |
| opendevreview | Harald Jensås proposed openstack/neutron-lib master: Document ML2 VLAN tri-state capability behavior https://review.opendev.org/c/openstack/neutron-lib/+/987337 | 11:26 |
| tafkamax | nudging it along for my own benefit aswell :) | 11:27 |
| tafkamax | i know everybody has work to do... | 11:27 |
| lajoskatona | tafkamax: Hi, you can try to ping ralonsoh here (perhaps He is on PTO) or try to bring this topic to the team meetings On-Demand agenda this afternoon (the meeting is from 1300UTC and the on-demand part is the last one, so around end-of-the-hour) | 11:40 |
| tafkamax | ok, will try to be online then, thx | 11:41 |
| opendevreview | yatin proposed openstack/neutron master: [service-type] Clear warnings for using ServicePluginBase https://review.opendev.org/c/openstack/neutron/+/987349 | 12:36 |
| haleyb | #startmeeting networking | 13:00 |
| opendevmeet | Meeting started Tue May 5 13:00:34 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 | 13:00 |
| mlavalle | \o | 13:00 |
| ykarel | o/ | 13:01 |
| slaweq | o/ | 13:01 |
| lajoskatona | o/ | 13:02 |
| haleyb | alright, we have quorum | 13:02 |
| haleyb | #topic announcements | 13:02 |
| haleyb | We are currently in Week R-21 of Hibiscus | 13:02 |
| haleyb | #link https://releases.openstack.org/hibiscus/schedule.html | 13:02 |
| bcafarel | late o/ | 13:02 |
| haleyb | Hibiscus-1 milestone is next week | 13:03 |
| haleyb | so hopefully we can get some good early changes merged that need some soaking | 13:04 |
| haleyb | 2024.2/Dalmatian transitioned to EOL, still trying to get our n-t-p jobs cleaned-up and remove traces of them | 13:05 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-lib master: evpn: Do not set evpn_vni to None by default https://review.opendev.org/c/openstack/neutron-lib/+/987354 | 13:06 |
| haleyb | i actually split into two pieces, was just looking for first part | 13:06 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/986964 | 13:06 |
| haleyb | the n-d-r jobs are randomly failing still | 13:06 |
| haleyb | Just a reminder to use the priority dashboard for anything neutron/networking related | 13:08 |
| haleyb | #link https://tinyurl.com/59z278km | 13:08 |
| haleyb | RP +1 for "ready to merge" changes, RP +2 for gate blockers and similar | 13:08 |
| lajoskatona | the n-d-r failures are on master? | 13:08 |
| haleyb | lajoskatona: randomly on master, 2025.2 and 2025.1 | 13:08 |
| lajoskatona | ohhh, nice | 13:08 |
| haleyb | resource_setup_container ? i think that's CI ? | 13:09 |
| lajoskatona | https://codesearch.openstack.org/?q=resource_setup_container&i=nope&literal=nope&files=&excludeFiles=&repos= | 13:10 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-tempest-plugin master: evpn: Add API router tests for EVPN extension https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/987355 | 13:10 |
| haleyb | i mean, it could be something else, os-ken is in there as well, but just a single test failure | 13:10 |
| lajoskatona | seems like smething for the tempest tests for dynamic-routing | 13:10 |
| lajoskatona | yeah, os-ken is everywhere, and can be a good suspect | 13:11 |
| ykarel | in that test some docker setup was done, so coule be related to that | 13:12 |
| haleyb | os_ken.tests.integrated.common.docker_base.CommandError | 13:12 |
| ykarel | ok confirmed in logs, it's related to that | 13:13 |
| ykarel | ERROR: failed to build: failed to solve: process "/bin/sh -c apt-get install -qy --no-install-recommends telnet tcpdump quagga-bgpd" did not complete successfully: exit code: 100 | 13:13 |
| ykarel | : os_ken.tests.integrated.common.docker_base.CommandError | 13:13 |
| ykarel | from https://564427108cabbad8b493-532d928235cdca0f1f7d2e9417dacd2f.ssl.cf1.rackcdn.com/openstack/8007a1732cc04596ae093d964dac1538/controller/logs/tempest_log.txt | 13:13 |
| haleyb | so trying to build the quagga con tainer fails | 13:13 |
| haleyb | oh, could be related to the ubuntu mirrors perhaps? | 13:14 |
| ykarel | it's not using CI mirrors, which should have avoided/limited such issues related to external acceess | 13:14 |
| ykarel | so let's have a bug and fix it like ^ ? | 13:15 |
| haleyb | ykarel: yes, we should open a bug then, error 100 is the generic error from apt but maybe we can narrow it down | 13:16 |
| ykarel | +1 , will you open the bug or want me to do that? | 13:17 |
| haleyb | AI says to change to apt-get update && apt-get install | 13:17 |
| ykarel | that's not true here :) | 13:17 |
| haleyb | ykarel: can you do that while i type away | 13:17 |
| ykarel | ok will do | 13:18 |
| haleyb | ykarel: and i don't know if that's the issue, i'd assume an apt-get update was run at some point | 13:18 |
| ykarel | #5 308.2 W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/focal/InRelease Connection failed [IP: 91.189.92.23 80] | 13:18 |
| ykarel | #5 308.2 W: Some index files failed to download. They have been ignored, or old ones used instead. | 13:18 |
| ykarel | so we should avoid these external mirrors and instead use ci mirrors and that should help here | 13:19 |
| haleyb | i think we are still having issues after a ddos attack last week | 13:19 |
| haleyb | but yes, maybe there is some change we can make | 13:20 |
| haleyb | alright will continue | 13:21 |
| haleyb | i will be traveling next week, so will have limited time for neutron (and irc) | 13:22 |
| haleyb | can someone lead this meeting next week? and possibly drivers if there is an agenda? | 13:22 |
| ykarel | reported https://bugs.launchpad.net/neutron/+bug/2151194 | 13:23 |
| haleyb | i didn't plan for it to be H-1, so slaweq you will need to do the release approval when those patches drop if you don't see me around | 13:24 |
| ykarel | also i see it's using focal, so may be can also consider moving to newer versions | 13:24 |
| haleyb | ykarel: ack, thanks! | 13:24 |
| slaweq | sure | 13:24 |
| haleyb | ykarel: oh, everything should be jammy or noble, maybe theres a definition issue, don't know if it will help | 13:25 |
| ykarel | haleyb, its about that container image which being built in the jobs, we should have all our jobs in jammy/noble already | 13:25 |
| haleyb | right | 13:26 |
| haleyb | ok, so any taker for running this meeting next week? or everyone can draw straws? :) | 13:27 |
| haleyb | alright, i'll assume as a collective someone will take over | 13:28 |
| slaweq | I can run it next week if there is no other volunteers | 13:28 |
| lajoskatona | I can :-) | 13:28 |
| lajoskatona | Ohh, slaweq was quicker :-) | 13:29 |
| haleyb | slaweq, lajoskatona: thanks, was going to do eenie minee moe | 13:29 |
| slaweq | LOL | 13:29 |
| haleyb | alright, i think we can move onto bugs | 13:30 |
| haleyb | #topic bugs | 13:30 |
| haleyb | ykarel was the deputy, his email is at | 13:30 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/X4BCDF75GJBZBDU4ZQZ6YWULKPFIUR7E/ | 13:30 |
| haleyb | all the high/med/low have owners, don't know about patches | 13:31 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2150682 | 13:32 |
| haleyb | [ovn-octavia-provider] VIP ports are unprotected: empty device_id allows them to be attached to other workloads, breaking the LB | 13:32 |
| haleyb | #link https://review.opendev.org/c/openstack/ovn-octavia-provider/+/986730 | 13:32 |
| haleyb | that just needs a review | 13:33 |
| haleyb | is froyo around? | 13:33 |
| haleyb | oh, i see he approved something yesterday so that's the answer | 13:35 |
| froyo | yeah! | 13:35 |
| froyo | haleyb, I will take a look there | 13:35 |
| haleyb | froyo: hi! can you take a look at that one? i'm sure a lot of AI generation but looks ok | 13:35 |
| haleyb | thanks | 13:35 |
| froyo | yw | 13:36 |
| haleyb | next one | 13:36 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2150864 | 13:36 |
| haleyb | networking-sfc unit tests are failing with filtervalidation extension enabled | 13:36 |
| haleyb | lajoskatona: you asked me about this yesterday | 13:36 |
| lajoskatona | yes | 13:37 |
| lajoskatona | I will push the sfc patch to make the gate happy | 13:37 |
| haleyb | great, thanks | 13:37 |
| lajoskatona | and after that I will reopen the patch to move sfc api definitions to n-lib | 13:37 |
| lajoskatona | actually to use the definitions from n-lib | 13:38 |
| haleyb | lajoskatona: right, you mentioned the abandoned changes | 13:38 |
| lajoskatona | this one: https://review.opendev.org/c/openstack/networking-sfc/+/553819 | 13:38 |
| haleyb | last high priority bug is | 13:39 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2150866 | 13:39 |
| haleyb | connectivity lost with FIP after ovn-controller restarted | 13:39 |
| haleyb | lajoskatona: yes, right | 13:39 |
| haleyb | oh, that's really old | 13:39 |
| haleyb | regarding the ovn-controller one, there is a patch up | 13:40 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/987187 | 13:40 |
| haleyb | that was from Eduardo | 13:40 |
| haleyb | and i'll skip to the unassigned list of bugs | 13:42 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2150879 | 13:42 |
| haleyb | OVN BGP Agent missing egress from ip rules and default gateway route in br-ex table | 13:42 |
| haleyb | kuba confirmed the bug looks legit, don't know if submittor will work on it | 13:43 |
| haleyb | so if someone wants an ovn-bgp-agent bug it's available :) | 13:43 |
| haleyb | and the last unassigned one | 13:44 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2150754 | 13:44 |
| haleyb | resource_request used only for QoS, cannot be used for physnet scheduling | 13:44 |
| haleyb | ralonsoh had asked some questions, awaiting answers | 13:44 |
| lajoskatona | baremetal is for ironinc, am I wrong? | 13:45 |
| haleyb | ok, any other bugs to discuss? | 13:45 |
| haleyb | lajoskatona: right, that would explain why i think cardoe responded to a ping from nate in channel? or i at least thought he did? | 13:47 |
| haleyb | i lost some of my scrollback | 13:47 |
| haleyb | or maybe it was on the ML | 13:48 |
| lajoskatona | ack, so perhaps this is again some Ironic-Neutron x-project (+Nova/placement) thing :-) | 13:48 |
| haleyb | maybe, yes | 13:49 |
| cardoe | oh dang I'm here now. | 13:49 |
| haleyb | ah yes, it was in a response to the drivers meeting last week | 13:50 |
| cardoe | I'm happy to discuss it at a drivers meeting. | 13:50 |
| haleyb | cardoe: yeah, we were just discussing a bug that i think you responded to on the ML from Nate Harper? | 13:50 |
| cardoe | Yes. So it actually relates to a question I think I asked at the PTG. | 13:51 |
| haleyb | cardoe: yes, we can talk about it then (friday) | 13:51 |
| cardoe | It's about use case and visibility of resources. | 13:51 |
| cardoe | The model with VMs treats everything as uniform and the only difference is bandwidth maybe which the QoS support for resource_request does. | 13:52 |
| cardoe | There's a common thread through all of this. | 13:52 |
| cardoe | And that's having some understanding of what you're built on to make the right connection choices. | 13:53 |
| cardoe | Like Helen's type-5 spec. Say you want to build a box in one room vs another room in the future. | 13:53 |
| cardoe | Since that spec is letting you bridge rooms together with OVN. | 13:53 |
| cardoe | Yes bandwidth is something you can quantify those two rooms... but it's not necessarily correct. | 13:54 |
| haleyb | cardoe: ack, we can loop back friday, or in a few minutes, sorry i have to wrap-up the meeting | 13:55 |
| haleyb | but thanks for responding and the ML response, you know more about this space than all of us :) | 13:55 |
| haleyb | ok, 5 minutes left, can go through the rest quickly | 13:56 |
| haleyb | #topic specs | 13:56 |
| haleyb | #link https://review.opendev.org/q/project:openstack/neutron-specs+status:open | 13:56 |
| haleyb | the type-5 spec was merged :) | 13:56 |
| haleyb | there is at least one in the list ready for review (security default statefulness) | 13:57 |
| haleyb | cardoe: did you want to go forward with https://review.opendev.org/c/openstack/neutron-specs/+/952166 ? | 13:57 |
| haleyb | hard to tell based on last comment | 13:57 |
| haleyb | feel free to respond in the spec review as well | 13:58 |
| haleyb | and on the topic of specs | 13:58 |
| 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:58 |
| haleyb | #link https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 13:58 |
| haleyb | ok, 2 minutes | 13:59 |
| cardoe | Honestly I'd love to just remove the restriction on vxlan to allow a physical_network | 13:59 |
| haleyb | cardoe: maybe something else to discuss friday | 13:59 |
| haleyb | #topic community goals | 13:59 |
| haleyb | #link https://review.opendev.org/q/topic:%22sdk_for_neutron%22 | 13:59 |
| haleyb | #link https://review.opendev.org/q/topic:%2522migrate_stadium_osc%2522 | 14:00 |
| lajoskatona | slow progress with both, for nova patches I wait for review | 14:00 |
| haleyb | right, i think some of the OSC patches have merged though | 14:01 |
| lajoskatona | yes, and for bgpvpn there's also a +2 | 14:01 |
| haleyb | maybe only cleanup is https://review.opendev.org/c/openstack/python-neutronclient/+/977975 ? | 14:01 |
| haleyb | ok, need to wrap up | 14:02 |
| haleyb | #topic on-demand | 14:02 |
| haleyb | anything else to discuss? | 14:02 |
| lajoskatona | nothing from me | 14:03 |
| haleyb | okay, as we're over i'll let everyone go | 14:03 |
| haleyb | thanks for attending and have a good week! | 14:03 |
| haleyb | #endmeeting | 14:03 |
| opendevmeet | Meeting ended Tue May 5 14:03:24 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:03 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2026/networking.2026-05-05-13.00.html | 14:03 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2026/networking.2026-05-05-13.00.txt | 14:03 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2026/networking.2026-05-05-13.00.log.html | 14:03 |
| mlavalle | \o | 14:03 |
| lajoskatona | o/ | 14:03 |
| ykarel | o/ | 14:03 |
| tafkamax | I was late | 14:06 |
| tafkamax | But nudging along this | 14:06 |
| tafkamax | https://review.opendev.org/c/openstack/neutron/+/986349 | 14:06 |
| cardoe | haleyb: I'm happy to help in any way I can. I'll admit my $TIME has been lower than I wanted but the network tooling is important to me. | 14:09 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: Enable name as filter for SFC API https://review.opendev.org/c/openstack/networking-sfc/+/987371 | 14:40 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Add support for Python 3.13 https://review.opendev.org/c/openstack/ovn-octavia-provider/+/982908 | 14:40 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change ovn_router_indirect_snat config option to True https://review.opendev.org/c/openstack/neutron/+/987308 | 14:50 |
| haleyb | cardoe: ack, sorry was in downstream meeting | 14:51 |
| cardoe | no worries | 14:51 |
| cardoe | I was thinking about making an extension that added a "fabric" field to network. Which could then be used for scheduling. | 14:53 |
| cardoe | Like during one of the calls Rodolfo mentioned that VXLAN is an overlay network and it's assumed that everything is connected together in one VRF. | 14:54 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: sfc: make name fields filter in API https://review.opendev.org/c/openstack/neutron-lib/+/987372 | 14:54 |
| cardoe | But we're seeing asks around multiple VRFs (the type-5 spec for example is adding extra VRFs via config file and then creating pools of VNIs separately) | 14:54 |
| stephenfin | haleyb: RE: your type hint questions, it's all been hand-written [*] so far 😅 I've repeatedly experimented with claude and different models, but found it kept trying to "cheat" with `type: ignore` statements or excessive use of `typing.Any` | 14:55 |
| cardoe | We can continue to treat VXLAN and OVN as an overlay when physical_network = None | 14:55 |
| cardoe | Otherwise the fabric can field can interact with physical_network. | 14:55 |
| haleyb | stephenfin: impressive, so i guess there is no easy way out for neutron* repos, maybe someone will have the time eventually | 14:57 |
| * haleyb is not asking you to do anything btw, but maybe i'll spend some tokens to see how claude does | 14:58 | |
| stephenfin | I mean, perhaps someone can drive an agent better that I have, and the fact that a lot of the depdency libraries are now typed should help (since you can verify its output) | 14:58 |
| stephenfin | ...but the "don't cheat" bit was the hardest one to enforce. It just kept ignoring me 😅 | 14:59 |
| stephenfin | Wholly unrelated: the networking-extra job in SDK appears to be permafailing since this time last week https://zuul.opendev.org/t/openstack/builds?job_name=openstacksdk-functional-devstack-networking-ext&project=openstack/openstacksdk | 15:00 |
| stephenfin | Looking at the jobs, it seems to be a bug in fwaas https://zuul.opendev.org/t/openstack/build/89bace86508a43b7833ba9f0b9588484 Did the API suddenly get stricter there? | 15:01 |
| stephenfin | Ah, looks like this was ralonsoh https://review.opendev.org/c/openstack/neutron/+/978463 | 15:07 |
| haleyb | stephenfin: yes, was just looking for that one, it broke a couple of repos | 15:08 |
| haleyb | and i'm not good at driving AI either, i can say it doesn't listen but it would say i didn't explain well enough | 15:08 |
| stephenfin | haleyb: the resource currently says `name` is a valid filter but the error says that's not true https://github.com/openstack/openstacksdk/blob/master/openstack/network/v2/firewall_policy.py#L38 | 15:11 |
| haleyb | lajoskatona: seems fwaas is broken same way networking-sfc is ^^^ ? | 15:11 |
| haleyb | stephenfin: we'll have to take a look | 15:11 |
| stephenfin | iiuc, the API "spec" lives here https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/firewall_v2.py I'm not sure how to read that to figure out what the expected filters are though | 15:12 |
| stephenfin | understood. Does it make sense to (a) drop the filters or (b) make the job non-voting for now? | 15:12 |
| stephenfin | ...given there's a few things backed up behind this now | 15:12 |
| haleyb | stephenfin: https://review.opendev.org/c/openstack/neutron-lib/+/987372 is fixing it for sfc, probably need similar for firewall_v2 | 15:13 |
| haleyb | stephenfin: is that a job in openstacksdk? | 15:13 |
| stephenfin | yes https://zuul.opendev.org/t/openstack/builds?job_name=openstacksdk-functional-devstack-networking-ext&project=openstack/openstacksdk | 15:13 |
| stephenfin | master is currently rather unhappy as a result https://review.opendev.org/q/project:openstack/openstacksdk+branch:master+is:open | 15:14 |
| haleyb | guess we could make non-voting until we release a new neutron-lib with fixes, vpnaas probably broken as well | 15:15 |
| stephenfin | ack https://review.opendev.org/c/openstack/openstacksdk/+/987374 | 15:24 |
| lajoskatona | haleyb, stephenfin: anything can happen, strange that our weekly periodic was green, and only sfc failed | 15:39 |
| lajoskatona | It was strange for me, but liked the idea that we have such robust software :-) | 15:40 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Update definitions to support name field filter https://review.opendev.org/c/openstack/neutron-lib/+/987378 | 15:51 |
| haleyb | lajoskatona: think that will help, untested | 15:51 |
| haleyb | we do have an experimental job in neutron for openstacksdk-functional-devstack-networking that could maybe catch it? | 15:53 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test name field filtering changes with sdk https://review.opendev.org/c/openstack/neutron/+/987381 | 15:56 |
| stephenfin | haleyb: lajoskatona: I'm just cross-referencing the SDK definitions with what's in neutron-lib and I see a few other differences. Who would be the best person to say whether it's SDK or neutron-lib that is wrong for each? | 16:31 |
| stephenfin | For example, firewall groups. SDK: https://github.com/openstack/openstacksdk/blob/master/openstack/network/v2/firewall_group.py#L33-L42; neutron-lib: https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/firewall_v2.py#L107-L150 | 16:32 |
| stephenfin | We're fixing name but apparently description, egress_firewall_policy_id, ingress_firewall_policy_id, shared, status and ports are all missing `is_filter`, while project_id is missing entirely | 16:35 |
| haleyb | stephenfin: i think that will be addressed with https://review.opendev.org/c/openstack/neutron-lib/+/987378 - i'm testing in https://review.opendev.org/c/openstack/neutron/+/987381 | 16:35 |
| haleyb | if i'm understanding correctly | 16:35 |
| stephenfin | That's only adding name though? Do we need to add `is_filter` to e.g. `ingress_firewall_policy_id` also? | 16:36 |
| stephenfin | Basically are there any other valid filters _except_ for name? | 16:36 |
| haleyb | stephenfin: i'd have to look | 16:37 |
| stephenfin | Ack. If not, we need to drop everything else from the `_query_mapping` fields in the various fwaas resources | 16:37 |
| haleyb | i mean, for things like subnets you can filter on anything pretty much | 16:41 |
| haleyb | i was shooting for the quick hit to make the gates green | 16:41 |
| stephenfin | fari | 16:46 |
| stephenfin | *fair | 16:46 |
| stephenfin | I left a comment on the change but, as noted there, didn't -1 since this can all be done separately. Hopefully the links are useful at least. | 16:47 |
| haleyb | the api-ref does say "The Networking API v2.0 supports filtering based on all top level attributes of a resource. Filters are applicable to all list requests." | 16:54 |
| haleyb | i do see a lot of these recently migrated to OSC aren't very feature-rich in that filter department | 16:54 |
| haleyb | i actually need to add the -ext job to my test patch | 17:02 |
| cardoe | haleyb: https://review.opendev.org/c/openstack/neutron/+/985837 is the change for allowing vxlan to take physical_network for example. | 17:32 |
| opendevreview | Merged openstack/neutron master: bgp: Update main router policies when chassis is added https://review.opendev.org/c/openstack/neutron/+/985895 | 17:43 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Update definitions to support name field filter https://review.opendev.org/c/openstack/neutron-lib/+/987378 | 18:15 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test name field filtering changes with sdk https://review.opendev.org/c/openstack/neutron/+/987381 | 18:16 |
| haleyb | cardoe: i just posted some more comments there | 18:30 |
| cardoe | haleyb: thanks! I've also added a topic to the drivers meeting. Hopefully correctly this time. | 18:32 |
| haleyb | cardoe: ack, it's in the section where i'll notice it now I have selective eyesight :) | 18:34 |
| cardoe | haleyb: so I've got a crazy vision for where neutron controls the underlay and all the BGP bits as well as the overlay that the VMs connect to. | 18:56 |
| cardoe | I think I've got most of the pieces together. | 18:56 |
| cardoe | I just want to make sure I present things properly. | 18:56 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-tempest-plugin master: evpn: Add API router tests for EVPN extension https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/987355 | 19:58 |
| *** kleini_ is now known as kleini | 20:33 | |
| opendevreview | Brian Haley proposed openstack/neutron master: WIP: Stop using neutron-lib project info code https://review.opendev.org/c/openstack/neutron/+/986961 | 20:37 |
| cardoe | haleyb: ugh you made me panic that somehow neutron wasn't doing project_id enforcement | 21:40 |
| cardoe | on the VXLAN type | 21:40 |
| haleyb | no, just wondering about the filters there based on existing code | 21:42 |
| cardoe | yeah that's what I meant | 21:42 |
| cardoe | I do need to tweak something. But there's duplication. | 21:43 |
| cardoe | https://opendev.org/openstack/neutron/src/commit/f95a2c61393b59a6bc2fb67323c31d7e7ca8af13/neutron/plugins/ml2/drivers/type_vlan.py#L242 | 21:46 |
| cardoe | because of that line we need to build up the dict again while the other paths can use what's supplied. | 21:46 |
| opendevreview | Doug Goldstein proposed openstack/neutron-lib master: remove reference to non-existent method https://review.opendev.org/c/openstack/neutron-lib/+/987437 | 22:12 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Remove target_tenant from RBAC calls https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/987438 | 22:24 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!