| opendevreview | Miguel Lavalle proposed openstack/neutron-specs master: Multiple backend support for L3 routers in ML2/OVN https://review.opendev.org/c/openstack/neutron-specs/+/885324 | 00:11 |
|---|---|---|
| opendevreview | Miguel Lavalle proposed openstack/neutron-specs master: Multiple backend support for L3 routers in ML2/OVN https://review.opendev.org/c/openstack/neutron-specs/+/885324 | 00:13 |
| opendevreview | huanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations https://review.opendev.org/c/openstack/neutron/+/880922 | 03:04 |
| opendevreview | Merged openstack/neutron-fwaas-dashboard master: remove unicode prefix from code https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/854476 | 06:11 |
| opendevreview | Maxim Korezkij proposed openstack/neutron master: [neutron-api] remove leader_only for sb connection https://review.opendev.org/c/openstack/neutron/+/885240 | 07:07 |
| opendevreview | huanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations https://review.opendev.org/c/openstack/neutron/+/880922 | 07:13 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp/agent: fix 'get_metadata_bind_interface' driver call https://review.opendev.org/c/openstack/neutron/+/884674 | 08:07 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rpc/dhcp: avoid get_network_info to return segments if not needed https://review.opendev.org/c/openstack/neutron/+/880131 | 08:07 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp/agent: add more detail to a todo note regarding call_driver https://review.opendev.org/c/openstack/neutron/+/885341 | 08:07 |
| opendevreview | Merged openstack/neutron stable/zed: Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/885173 | 09:39 |
| opendevreview | Merged openstack/neutron stable/xena: Start metadata proxy even if IPv6 DAD fails https://review.opendev.org/c/openstack/neutron/+/885271 | 09:39 |
| opendevreview | Merged openstack/neutron stable/wallaby: Start metadata proxy even if IPv6 DAD fails https://review.opendev.org/c/openstack/neutron/+/885273 | 09:39 |
| opendevreview | Merged openstack/neutron stable/victoria: Start metadata proxy even if IPv6 DAD fails https://review.opendev.org/c/openstack/neutron/+/885274 | 09:39 |
| ralonsoh | lajoskatona, hi! Please approve https://review.opendev.org/c/openstack/neutron/+/885174 if you have time. Once this patch is merged, I'll push new releases patches | 09:54 |
| ralonsoh | thanks in advance!! | 09:54 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: WIP: Tap mirror API def and API ref https://review.opendev.org/c/openstack/neutron-lib/+/885354 | 10:09 |
| lajoskatona | ralonsoh: done :-) | 10:10 |
| ralonsoh | thanks! | 10:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/885154 | 10:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: Replace "tenant_id" with "project_id" https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/882940 | 11:09 |
| *** EugenMayer44 is now known as EugenMayer4 | 11:21 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add port hardware offload extension https://review.opendev.org/c/openstack/neutron-lib/+/882726 | 11:34 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: WIP: GRE/ERSPAN mirroring for taas https://review.opendev.org/c/openstack/tap-as-a-service/+/885357 | 11:38 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Fix issues due to rcent RBAC changes https://review.opendev.org/c/openstack/networking-bgpvpn/+/883057 | 12:10 |
| opendevreview | Merged openstack/neutron stable/xena: Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/885175 | 12:15 |
| opendevreview | Merged openstack/neutron stable/wallaby: Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/885176 | 12:23 |
| lajoskatona | slaweq, ralonsoh: Hi, could you check these 2 sfc patches when you have some free time: https://review.opendev.org/c/openstack/networking-sfc/+/883078 & https://review.opendev.org/c/openstack/networking-sfc/+/881240 | 12:59 |
| ralonsoh | sure | 12:59 |
| lajoskatona | slaweq, ralonsoh: today I again have to skip the meetings :-( | 12:59 |
| ralonsoh | no problem | 12:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Stop the RPC connections when the agent exits https://review.opendev.org/c/openstack/neutron/+/883685 | 13:22 |
| *** obondarev_ is now known as obondarev | 13:55 | |
| ralonsoh | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 13:59 |
| mlavalle | o/ | 13:59 |
| mtomaska1 | o/ | 13:59 |
| ykarel | o/ | 14:00 |
| bcafarel | o/ | 14:00 |
| ralonsoh | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Jun 6 14:00:08 2023 UTC and is due to finish in 60 minutes. The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'networking' | 14:00 |
| ralonsoh | hello all | 14:00 |
| obondarev | hi | 14:00 |
| haleyb | o/ | 14:00 |
| isabek | Hi | 14:00 |
| frickler | \o | 14:00 |
| ralonsoh | slaweq and lajoskatona won't attend today | 14:00 |
| ralonsoh | ok, let's start, we have quorum | 14:00 |
| ralonsoh | #topic announcements | 14:01 |
| ralonsoh | #link https://releases.openstack.org/bobcat/schedule.html | 14:01 |
| ralonsoh | next week is the Vancouver PTG | 14:01 |
| ralonsoh | this is the schedule | 14:01 |
| ralonsoh | #link Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 14:01 |
| ralonsoh | sorry | 14:01 |
| mlavalle | o/ | 14:01 |
| mtomaska1 | o/ :) | 14:01 |
| ralonsoh | #link https://vancouver2023.openinfra.dev/a/schedule | 14:01 |
| bcafarel | this is to see if we react automatically :) | 14:01 |
| ykarel | :) | 14:02 |
| ralonsoh | yeah... sorry, bad copy/paste | 14:02 |
| ralonsoh | in this agenda you can filter by day and topic | 14:02 |
| ralonsoh | btw, I don't think we'll be able to have video meetings | 14:02 |
| ralonsoh | but we'll try using out own laptops | 14:02 |
| ralonsoh | ah, because of the PTG, there will be no meetings (team, CI and drivers) | 14:03 |
| ralonsoh | and that's all I have in this topic | 14:03 |
| ralonsoh | something I'm missing? | 14:03 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Do not dispose local_vlan_hints https://review.opendev.org/c/openstack/neutron/+/880334 | 14:03 |
| mlavalle | no CI meeting today either | 14:04 |
| ralonsoh | yes, I had this for the last minute topic | 14:04 |
| ralonsoh | ok, let's move on | 14:04 |
| ralonsoh | #topic bugs | 14:04 |
| ralonsoh | last week report is from mtomaska1 | 14:04 |
| ralonsoh | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-June/033962.html | 14:05 |
| ralonsoh | all bugs are assigned (that's good) | 14:05 |
| ralonsoh | there are two pending ones | 14:05 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2022360 | 14:05 |
| ralonsoh | this one was already assigned | 14:05 |
| ralonsoh | (but not in the LP bug) | 14:05 |
| ralonsoh | and there is a patch: https://review.opendev.org/c/openstack/neutron/+/883235 (reviews are welcome) | 14:05 |
| ralonsoh | the last one is | 14:06 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2022058 | 14:06 |
| ralonsoh | if I'm not wrong, what is expected from this bug is to put the general ML2/OVN config option for FIP distributed in the router dict | 14:06 |
| ralonsoh | so n-d-r can read it | 14:06 |
| ralonsoh | in this case the router resource will be different to the ML2/OVS one, because this will be a read-only parameter | 14:07 |
| dmitriis | o/ | 14:07 |
| dmitriis | I guess I can comment | 14:07 |
| ralonsoh | dmitriis, but ^^ is this what you expected? | 14:07 |
| ralonsoh | at least from Neutron | 14:08 |
| dmitriis | That's one way of doing it, I thought of an alternative as well | 14:08 |
| ralonsoh | please, go on | 14:08 |
| dmitriis | where we'd allow the distributed attribute to turn the distributed fips for OVN routers on a per-router basis | 14:08 |
| dmitriis | i.e. the addition of `external_mac` to OVN would depend on whether the global config option is enabled and whether `distributed == True` | 14:09 |
| ralonsoh | yeah, this is actually the "proper" way to implement this | 14:09 |
| ralonsoh | but of course this is much more complex | 14:09 |
| ralonsoh | and, IMO, this second alternative could be considered as a RFE | 14:10 |
| dmitriis | the other part of the bug is the NDR part | 14:10 |
| dmitriis | because currently /32 routes for `distributed == True` routers are created on the basis of floatingip_agent_gateway ports | 14:11 |
| dmitriis | but those aren't created with ML2/OVN | 14:11 |
| dmitriis | I'm not sure if this was a design decision to conserve IP space or not (compared to ML2/ovs there is an IP address per node per external network consumed for those) | 14:11 |
| dmitriis | but they also allow for southbound routing to be done directly to a compute node with /32 routes | 14:12 |
| dmitriis | so there are 2 things to consider: | 14:12 |
| dmitriis | 1. if there are no floatingip_agent_gateway ports, we should probably have a fallback to /32 routes with the centralized gateway port IP as a next-hop | 14:13 |
| dmitriis | that way we fix the issue with no routes being advertised for OVN routers that have `distributed == True` | 14:13 |
| dmitriis | 2. consider if we need to optionally enable the creation of floatingip_agent_gateway ports with ML2/OVN akin to ML2/OVS for routing purposes | 14:14 |
| dmitriis | (1) is probably a bug-fix type of a change. (2) is more of an RFE | 14:14 |
| dmitriis | I hope it makes sense. There are some links to the code in the bug which show what I mean (just in case) | 14:15 |
| ralonsoh | ok, I'll take this conversation out of this meeting but I would like to understand why these "floatingip_agent_gateway" ports are needed | 14:15 |
| ralonsoh | I'll ping you after this meeting just to retrieve more info | 14:15 |
| dmitriis | ok, sounds good | 14:15 |
| ralonsoh | and it could be useful to have these thoughs in the LP | 14:15 |
| ralonsoh | ok, any other bug you want to discuss? | 14:16 |
| dmitriis | no bugs, just a review at the end of the agenda | 14:16 |
| ralonsoh | sure, please add it in the on demand section | 14:16 |
| ralonsoh | the week bcafarel is the bug deputy, next week is lajoslatona | 14:17 |
| ralonsoh | because there is nothing related to os-ken, let's jump to the next section | 14:17 |
| dmitriis | ralonsoh: ack, it's the last one here (https://wiki.openstack.org/wiki/Network/Meetings#On_Demand_Agenda) | 14:17 |
| ralonsoh | #topic specs | 14:17 |
| ralonsoh | dmitriis, don't worry, I'll ask for this at the end | 14:18 |
| dmitriis | ack | 14:18 |
| ralonsoh | we have one active spec: https://review.opendev.org/q/project:openstack%252Fneutron-specs+status:open | 14:18 |
| ralonsoh | #link https://review.opendev.org/c/openstack/neutron-specs/+/885324 | 14:18 |
| ralonsoh | Multiple backend support for L3 routers in ML2/OVN | 14:18 |
| ralonsoh | this RFE was discussed (and approved) last Friday | 14:18 |
| ralonsoh | so, as usual, reviews are more than welcome | 14:18 |
| ralonsoh | (when there is something in the spec) | 14:19 |
| mlavalle | I'm writing the spec, that is WIP in gerrit for now | 14:19 |
| ralonsoh | sure | 14:19 |
| ralonsoh | something else in this topic? | 14:19 |
| ralonsoh | ok, I'll skip next one | 14:20 |
| ralonsoh | because | 14:20 |
| ralonsoh | 1) slaweq is not here and there are no bugs related to sRBAC | 14:20 |
| ralonsoh | 2) lajos is not here and there are no new patches related to the client migration | 14:21 |
| ralonsoh | so this is going to be a very very fast meeting | 14:21 |
| ralonsoh | #topic on_demand | 14:21 |
| ralonsoh | I have one | 14:21 |
| ralonsoh | as commented, we don't have the CI meeting during the next 2 weeks | 14:21 |
| ralonsoh | but, of course, if there is something urgent/blocker (tox5, sqlalchemy3.0 or similar) | 14:22 |
| ralonsoh | please ping any core in this channel and open a critical LP bug | 14:22 |
| ralonsoh | dmitriis, please | 14:22 |
| frickler | don't make jokes like that, we don't even have sqla2.0 running | 14:22 |
| ralonsoh | the CI is now passing (for Neutron) | 14:22 |
| ralonsoh | so at least we are good (I think so) | 14:23 |
| frickler | neutron yes, openstack ... meh | 14:23 |
| ralonsoh | yeah... | 14:23 |
| dmitriis | ralonsoh: are you asking about my item? | 14:23 |
| ralonsoh | sure | 14:23 |
| dmitriis | ah, ok | 14:23 |
| dmitriis | so I have a review for the multiple gateway support spec implementation | 14:24 |
| dmitriis | https://review.opendev.org/c/openstack/neutron/+/873593 | 14:24 |
| dmitriis | just looking for someone to look at it to have 1 more core review on it | 14:24 |
| dmitriis | I understand it's quite a large one to review | 14:25 |
| dmitriis | but it's quite self-contained as far as the functionality goes | 14:25 |
| ralonsoh | for sure, I'll check it tomorrow morning, you have my word | 14:26 |
| fnordahl | \o/ | 14:26 |
| fnordahl | there is of course a series of reviews stacked on top of it, and they are smaller | 14:26 |
| dmitriis | ralonsoh: tyvm, just trying to get it in early to avoid rushing close to the change freeze | 14:26 |
| dmitriis | that's all I had for this | 14:26 |
| ralonsoh | yes and you have worked hard in this one, for sure | 14:26 |
| ralonsoh | dmitriis, thanks | 14:27 |
| frickler | are there some docs for it somewhere? the reno is pretty terse | 14:27 |
| dmitriis | We'll add some as a follow-up, just want to make sure we have the functionality set in place | 14:28 |
| dmitriis | Some of the functionality is documented in the API ref change | 14:28 |
| dmitriis | but definitely deserves a separate doc entry, I agree | 14:28 |
| ralonsoh | thank you | 14:29 |
| frickler | might help reviews to see some example scenario to compare against | 14:29 |
| frickler | but I'll check the API ref change | 14:29 |
| dmitriis | frickler: ack | 14:30 |
| ralonsoh | any other topic? | 14:31 |
| dmitriis | not from me | 14:31 |
| ralonsoh | thank you all for attending | 14:32 |
| ralonsoh | #endmeeting | 14:32 |
| opendevmeet | Meeting ended Tue Jun 6 14:32:04 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:32 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2023/networking.2023-06-06-14.00.html | 14:32 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-06-06-14.00.txt | 14:32 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2023/networking.2023-06-06-14.00.log.html | 14:32 |
| mlavalle | o/ | 14:32 |
| ykarel | o/ | 14:32 |
| obondarev | o/ | 14:32 |
| mtomaska1 | o/ | 14:32 |
| fnordahl | \o thanks all | 14:32 |
| dmitriis | o/ | 14:32 |
| opendevreview | Merged openstack/networking-sfc master: Remove the old CLI code from sfc and fix issuse due to RBAC changes https://review.opendev.org/c/openstack/networking-sfc/+/883078 | 15:05 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Set DVR qr-xyz interfaces DOWN on backup node https://review.opendev.org/c/openstack/neutron/+/869741 | 15:14 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Add a new option to enable signals https://review.opendev.org/c/openstack/neutron/+/884711 | 15:27 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Register GMR with config https://review.opendev.org/c/openstack/neutron/+/884842 | 15:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Add a "port" child table "porthardwareoffloadtype" https://review.opendev.org/c/openstack/neutron/+/882832 | 16:36 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/wallaby: Handle no more IP addresses available during a network sync https://review.opendev.org/c/openstack/neutron/+/885390 | 17:00 |
| opendevreview | Merged openstack/neutron stable/yoga: Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/885174 | 17:23 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/2023.1: Handle no more IP addresses available during a network sync https://review.opendev.org/c/openstack/neutron/+/885391 | 17:52 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/zed: Handle no more IP addresses available during a network sync https://review.opendev.org/c/openstack/neutron/+/885392 | 17:52 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/yoga: Handle no more IP addresses available during a network sync https://review.opendev.org/c/openstack/neutron/+/885393 | 17:53 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/xena: Handle no more IP addresses available during a network sync https://review.opendev.org/c/openstack/neutron/+/885394 | 17:53 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/2023.1: [OVN][Migration] Enable settings backup subnet for NFS clients https://review.opendev.org/c/openstack/neutron/+/885395 | 17:54 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/zed: [OVN][Migration] Enable settings backup subnet for NFS clients https://review.opendev.org/c/openstack/neutron/+/885396 | 17:54 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/xena: [OVN][Migration] Enable settings backup subnet for NFS clients https://review.opendev.org/c/openstack/neutron/+/885397 | 17:54 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/yoga: [OVN][Migration] Enable settings backup subnet for NFS clients https://review.opendev.org/c/openstack/neutron/+/885398 | 17:55 |
| opendevreview | Merged openstack/neutron master: Fix some new pylint "C" warnings https://review.opendev.org/c/openstack/neutron/+/883608 | 18:34 |
| opendevreview | Merged openstack/neutron master: Improve the ``PortBindingUpdateVirtualPortsEvent`` match filter https://review.opendev.org/c/openstack/neutron/+/883681 | 18:34 |
| opendevreview | Merged openstack/neutron master: gate: bump ovn to the latest LTS release (22.03) https://review.opendev.org/c/openstack/neutron/+/880890 | 18:34 |
| frickler | slaweq: lajos: please have a look at https://review.opendev.org/c/openstack/tap-as-a-service/+/837059 , I want to finally get rid of all zuul config errors | 18:42 |
| opendevreview | Merged openstack/neutron-lib master: Add FirewallGroupPortNotSupported exception https://review.opendev.org/c/openstack/neutron-lib/+/884599 | 23:31 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Don't check exit status when nc_client is spawned https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/885108 | 23:34 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [fwaas]Disassociate all ports before deleting firewall group https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883826 | 23:34 |
| opendevreview | Miguel Lavalle proposed openstack/neutron-specs master: Multiple backend support for L3 routers in ML2/OVN https://review.opendev.org/c/openstack/neutron-specs/+/885324 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!