| opendevreview | Brian Haley proposed openstack/neutron master: Update .pylintrc https://review.opendev.org/c/openstack/neutron/+/906772 | 01:43 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 01:55 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP: Add documentation for aa-l3-gw-multihoming https://review.opendev.org/c/openstack/neutron/+/899402 | 01:56 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute https://review.opendev.org/c/openstack/neutron/+/878531 | 01:56 |
| opendevreview | Takashi Kajinami proposed openstack/ovn-bgp-agent master: Use .coveragerc to omit directories https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906666 | 07:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: [sqlalchemy-20] Use sqlalchemy.orm.DeclarativeBase https://review.opendev.org/c/openstack/neutron-lib/+/906627 | 07:30 |
| ralonsoh | hi folks, if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/905836 | 08:04 |
| ralonsoh | hi lajoskatona! if you have less than 1 min | 08:05 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-lib/+/903841 | 08:05 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/903842 | 08:05 |
| ralonsoh | thanks! | 08:06 |
| lajoskatona | ralonsoh: checking | 08:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add ``subnetpool-prefix-ops`` extension to ML2/OVN mech driver https://review.opendev.org/c/openstack/neutron/+/906816 | 08:28 |
| maximkorezkij[m] | frickler ralonsoh yes, i was talking about the subnetpool-prefix-ops. Did you guys already created a bug or should i do it ? | 08:31 |
| ralonsoh | maximkorezkij[m], ^^ no, I just added the extension to the OVN driver with a reference to the bug that implemented that feature | 08:32 |
| ralonsoh | in the upper patch | 08:32 |
| maximkorezkij[m] | perfect, sorry i didnt see it | 08:33 |
| maximkorezkij[m] | then i will have an eye open for that patch, if i can help with something to speed this up, let me know | 08:34 |
| opendevreview | Merged openstack/neutron-fwaas master: Update python classifier with py3.10 & py3.11 in setup.cfg https://review.opendev.org/c/openstack/neutron-fwaas/+/905301 | 08:53 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/networking-bgpvpn/+/900287 | 09:00 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Use .coveragerc to omit directories https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906666 | 09:23 |
| *** ravlew is now known as Guest444 | 10:09 | |
| opendevreview | Merged openstack/neutron master: python-3.12: do not use datetime.datetime.utcnow() https://review.opendev.org/c/openstack/neutron/+/903842 | 10:33 |
| opendevreview | Merged openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 10:33 |
| opendevreview | Merged openstack/neutron-lib master: Python-3.12: do not use datetime.datetime.utcnow() https://review.opendev.org/c/openstack/neutron-lib/+/903841 | 10:44 |
| opendevreview | Merged openstack/neutron-lib master: Tap mirror API def and API ref https://review.opendev.org/c/openstack/neutron-lib/+/885354 | 10:44 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [ovn] Enable IPv6 metadata tests for OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/894027 | 10:44 |
| opendevreview | Merged openstack/neutron-lib master: Add Tap Mirror exception https://review.opendev.org/c/openstack/neutron-lib/+/895603 | 10:45 |
| opendevreview | Merged openstack/neutron-lib master: Add some neutron constants https://review.opendev.org/c/openstack/neutron-lib/+/905360 | 10:45 |
| opendevreview | Merged openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute https://review.opendev.org/c/openstack/neutron/+/878531 | 10:45 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Use .coveragerc to omit directories https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906497 | 10:50 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: ensure that cleaning DHCP process with one segment happens first https://review.opendev.org/c/openstack/neutron/+/905617 | 11:05 |
| opendevreview | Fernando Royo proposed openstack/ovn-bgp-agent master: Add support to PF OVN LBs for NB Driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905504 | 11:28 |
| opendevreview | Merged openstack/neutron stable/yoga: Revert "Add sleep before checking if ovs port is in the namespace" https://review.opendev.org/c/openstack/neutron/+/906295 | 11:44 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Use .coveragerc to omit directories https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906497 | 12:13 |
| opendevreview | James Page proposed openstack/networking-sfc master: python 3.12: assertDictContainsSubset https://review.opendev.org/c/openstack/networking-sfc/+/906523 | 13:02 |
| haleyb | #startmeeting neutron_drivers | 14:00 |
| opendevmeet | Meeting started Fri Jan 26 14:00:30 2024 UTC and is due to finish in 60 minutes. The chair is haleyb. 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 'neutron_drivers' | 14:00 |
| slaweq | o/ | 14:00 |
| mlavalle | \o | 14:00 |
| haleyb | Ping list: ykarel, mlavalle, mtomaska, slawek, obondarev, tobias-urdin, lajoskatona, amotoki | 14:00 |
| obondarev | o/ | 14:00 |
| lajoskatona | o/ | 14:00 |
| ralonsoh | hi | 14:01 |
| haleyb | alright, we have quorum | 14:02 |
| mlavalle | \o/ | 14:02 |
| mlavalle | first time this year | 14:02 |
| haleyb | i forgot to update the wiki, but seems we have 3 things to review | 14:03 |
| haleyb | first rfe | 14:03 |
| haleyb | https://bugs.launchpad.net/neutron/+bug/2047849 | 14:03 |
| haleyb | [RFE] Start using oslo messaging namespaces | 14:03 |
| haleyb | mtomaska__: are you here? | 14:04 |
| haleyb | although i see ralonsoh sent out the patch | 14:04 |
| ralonsoh | I've started with a POC: https://review.opendev.org/c/openstack/neutron/+/905309 | 14:04 |
| ralonsoh | this is duplicating the server RPCs: one with namespace, the other one without it | 14:04 |
| ralonsoh | to allow the upgrades (newer servers, older agents) | 14:05 |
| ralonsoh | of course, I found some issues... | 14:05 |
| ralonsoh | in any case, the question here is if that should be approved or not | 14:05 |
| ralonsoh | I'll start: yes, the RPC segregation should have been done since the beginning | 14:06 |
| ralonsoh | RPC channels between different agents should have different namespaces and different APIs | 14:06 |
| lajoskatona | +1 | 14:06 |
| ralonsoh | (now we are sharing some of them...) | 14:06 |
| mlavalle | it's really technical debt | 14:07 |
| mlavalle | so yes | 14:07 |
| ralonsoh | exactly | 14:07 |
| lajoskatona | but as you mentioned it isi tricky for the upgrade, C is slurp, am I right? | 14:07 |
| haleyb | ralonsoh: so what happens during an upgrade? | 14:07 |
| slaweq | for me there is no question if we should or not do that - we should BUT we need to have good plan and test well upgrades | 14:08 |
| ralonsoh | during the upgrade the server will be listening to two namespaces | 14:08 |
| slaweq | with keeping in mind SLURP releases | 14:08 |
| ralonsoh | slaweq, right, this is mandatory and probably we'll need something more that grenade jobs | 14:08 |
| slaweq | lajoskatona yes, C will be SLURP | 14:08 |
| mlavalle | I don't think there is any doubt we should do this. The question is how | 14:09 |
| obondarev | so what's the main issue that namespaces are solving? Is it just method names collisions? | 14:09 |
| ralonsoh | yes, we had that in the DHCP agent recently | 14:09 |
| ralonsoh | so each API should be independent | 14:10 |
| obondarev | I see, thanks | 14:10 |
| ralonsoh | and each RPC server should be listening only to th specific agents | 14:10 |
| haleyb | and to the SLURP question, it seems we should get this done in C to be useful for next SLURP | 14:11 |
| ralonsoh | hmmm to be honest, i don't know if I'll be able to finish that in C | 14:11 |
| ralonsoh | I know our current release cadence | 14:12 |
| lajoskatona | that is my feeling also we are quite in the middle of C for this | 14:13 |
| haleyb | ack | 14:13 |
| ralonsoh | in any case, this is not a high priority, is not affecting any deployment and should be considered as a tech debt | 14:13 |
| haleyb | right, thanks | 14:14 |
| lajoskatona | as I see we can prepare it (perhaps discuss during the ptg in detail) and activate it in E | 14:14 |
| mlavalle | yes, we don't have a deluge of cases affected by this. so we have time | 14:16 |
| slaweq | ++ | 14:18 |
| haleyb | ok. and i guess that once it's complete we can decide if we activate in D or E | 14:18 |
| haleyb | lets vote | 14:18 |
| mlavalle | +1 | 14:18 |
| haleyb | +1 | 14:18 |
| obondarev | +1 | 14:18 |
| ralonsoh | +1 | 14:19 |
| slaweq | +1 | 14:19 |
| haleyb | great, and should i assign to you ralonsoh ? | 14:19 |
| ralonsoh | yeah | 14:19 |
| lajoskatona | +! | 14:19 |
| haleyb | ok, there was another RFE filed by ralonsoh this week | 14:22 |
| haleyb | https://bugs.launchpad.net/neutron/+bug/2049623 | 14:22 |
| haleyb | [RFE] Refactor OVS Trunk plugin to have one single port | 14:22 |
| ralonsoh | If you don't mind, I'll first try to create a POC | 14:22 |
| ralonsoh | if that works, then I'll present it here | 14:22 |
| haleyb | ralonsoh: that was going to be my first question | 14:22 |
| ralonsoh | if not, I'll close the bug | 14:22 |
| opendevreview | morice proposed openstack/neutron master: [ovn] AZs distribution in L3 port scheduler https://review.opendev.org/c/openstack/neutron/+/892604 | 14:23 |
| ralonsoh | so please, discard this one for now | 14:23 |
| haleyb | ok, and the other one we were also going to wait for a POC | 14:24 |
| haleyb | https://bugs.launchpad.net/neutron/+bug/2045058 | 14:24 |
| haleyb | so i guess there is nothing else on the agenda | 14:25 |
| haleyb | any other things anyone wants to discuss? | 14:25 |
| mlavalle | not from me | 14:25 |
| ralonsoh | no thanks | 14:26 |
| lajoskatona | nothing from me | 14:26 |
| slaweq | nothing from me | 14:26 |
| haleyb | ok, thanks for attending and have a nice weekend! | 14:26 |
| haleyb | #endmeeting | 14:26 |
| opendevmeet | Meeting ended Fri Jan 26 14:26:36 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:26 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2024/neutron_drivers.2024-01-26-14.00.html | 14:26 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2024/neutron_drivers.2024-01-26-14.00.txt | 14:26 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2024/neutron_drivers.2024-01-26-14.00.log.html | 14:26 |
| mlavalle | \o | 14:26 |
| slaweq | o/ | 14:26 |
| ralonsoh | bye | 14:26 |
| mlavalle | have a nice weekend y'all | 14:26 |
| lajoskatona | o/ | 14:27 |
| obondarev | o/ | 14:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: If method ``set_netns`` fails, restore previous device namespace https://review.opendev.org/c/openstack/neutron/+/906855 | 15:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: If method ``set_netns`` fails, restore previous device namespace https://review.opendev.org/c/openstack/neutron/+/906856 | 15:45 |
| opendevreview | Merged openstack/tap-as-a-service master: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/tap-as-a-service/+/900639 | 15:47 |
| opendevreview | Brian Haley proposed openstack/neutron master: [ovn] Make scheduling of unhosted gateways aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 15:49 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 15:50 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Exclude tests directory from coverage calculation https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/906663 | 16:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Document the OVN L3 scheduler: AZs distribution https://review.opendev.org/c/openstack/neutron/+/906868 | 16:13 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add documentation about NB DB driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903407 | 16:18 |
| opendevreview | Merged openstack/tap-as-a-service master: Enables external network mirroring https://review.opendev.org/c/openstack/tap-as-a-service/+/896515 | 16:44 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add documentation about NB DB driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903407 | 16:47 |
| haleyb | starting to see a openstack-tox-py311-with-sqlalchemy-master failure in all jobs :( | 16:57 |
| ralonsoh | log? | 16:58 |
| haleyb | ralonsoh: https://zuul.opendev.org/t/openstack/build/c48468391e6140858f35519a44ddffed is one example | 16:59 |
| haleyb | i quickly looked an noticed three different changes same failure | 16:59 |
| haleyb | i won't have cycles for a couple of hours | 17:00 |
| ralonsoh | maybe is related to https://review.opendev.org/c/openstack/neutron/+/903842 | 17:02 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add documentation about NB DB driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903407 | 17:03 |
| haleyb | ralonsoh: could be, even though the test passed there | 17:14 |
| haleyb | ralonsoh: timeutils.utcnow() does have more elements than that test mocks | 17:22 |
| haleyb | datetime.datetime(2024, 1, 26, 17, 21, 39, 560872) | 17:23 |
| opendevreview | Merged openstack/neutron master: If method ``set_netns`` fails, restore previous device namespace https://review.opendev.org/c/openstack/neutron/+/905836 | 17:36 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix test_update_timestamp unit test failure https://review.opendev.org/c/openstack/neutron/+/906918 | 20:26 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix test_update_timestamp unit test failure https://review.opendev.org/c/openstack/neutron/+/906918 | 20:57 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix test_update_timestamp unit test failure https://review.opendev.org/c/openstack/neutron/+/906918 | 21:25 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix test_update_timestamp unit test failure https://review.opendev.org/c/openstack/neutron/+/906918 | 22:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!