opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Enhanced external port scheduling https://review.opendev.org/c/openstack/neutron/+/894767 | 08:55 |
---|---|---|
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Update the External Ports documentation https://review.opendev.org/c/openstack/neutron/+/900030 | 08:55 |
opendevreview | Merged openstack/neutron stable/yoga: [DHCP agent] Add route to OVN metadata port if exists https://review.opendev.org/c/openstack/neutron/+/900198 | 08:56 |
mnederlof | Hi, would someone be able to review this change? https://review.opendev.org/c/openstack/neutron/+/896883 | 09:00 |
opendevreview | Merged openstack/neutron master: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900647 | 10:58 |
opendevreview | Bartosz Bezak proposed openstack/neutron stable/2023.2: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900881 | 11:02 |
opendevreview | Bartosz Bezak proposed openstack/neutron stable/2023.1: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900882 | 11:03 |
opendevreview | Bartosz Bezak proposed openstack/neutron stable/zed: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900883 | 11:13 |
opendevreview | Bartosz Bezak proposed openstack/neutron stable/zed: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900883 | 11:25 |
opendevreview | Bartosz Bezak proposed openstack/neutron stable/yoga: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900885 | 11:27 |
opendevreview | Bartosz Bezak proposed openstack/neutron stable/wallaby: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900886 | 11:29 |
opendevreview | Roberto Acosta proposed openstack/neutron-dynamic-routing master: WIP: Allow to configure the BGP peer connect mode https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/899505 | 11:46 |
opendevreview | Jayce Houtman proposed openstack/neutron master: Change exception messages to error log messages for DNS integration. https://review.opendev.org/c/openstack/neutron/+/900212 | 11:52 |
lajoskatona | haleyb: (anybody): Hi, shall I ask 2 quick reviews for taas: https://review.opendev.org/c/openstack/tap-as-a-service/+/884239 & https://review.opendev.org/c/openstack/tap-as-a-service/+/896515 | 12:28 |
lajoskatona | haleyb: I tested both, and it looks really useful | 12:29 |
haleyb | lajoskatona: i'll try and take a look | 13:13 |
haleyb | zigo: so the amd64 regression is gone, i see the arm64 one fails, but it's not using the new code, should i worry? | 13:14 |
zigo | haleyb: No, you should just be patient ! :) | 13:15 |
zigo | The arm64 hosts are usually a lot slower to process new packages. | 13:15 |
haleyb | my slide rule is faster :) | 13:15 |
* haleyb just wants to merge that and release a new neutron-lib | 13:17 | |
zigo | haleyb: It's been 20 days I'm waiting for Ceph 18.2.0 to build in mips64el: https://buildd.debian.org/status/package.php?p=ceph&suite=experimental :/ | 13:18 |
zigo | Hopefully the new buildd machines will speed this up. | 13:18 |
haleyb | 8-o | 13:19 |
lajoskatona | haleyb: thanks | 13:44 |
opendevreview | Jakub Libosvar proposed openstack/neutron-lib master: api-ref: Document only unbound ports vnic_type updates https://review.opendev.org/c/openstack/neutron-lib/+/900826 | 13:55 |
haleyb | ping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 14:00 |
jlibosva | o/ | 14:00 |
haleyb | #startmeeting networking | 14:00 |
opendevmeet | Meeting started Tue Nov 14 14:00:47 2023 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 'networking' | 14:00 |
mlavalle | o/ | 14:00 |
rubasov | o/ | 14:01 |
obondarev | o/ | 14:01 |
bcafarel | o/ | 14:01 |
haleyb | hi everybody | 14:01 |
lajoskatona | o/ | 14:01 |
frickler | \o | 14:01 |
haleyb | #topic announcements | 14:01 |
haleyb | This week is the C-1 milestone, and library releases have started to happen | 14:02 |
haleyb | os-ken was released and python-neutronclient is ready to do | 14:02 |
haleyb | ovsdbapp had a few things ready to merge and neutron-lib has some more | 14:03 |
haleyb | #link https://review.opendev.org/q/project:openstack/neutron-lib | 14:03 |
haleyb | if there is anything you want merged for C-1 there please update it and ping for reviews, i'll be updating the release patch either tonight or tomorrow to move it along | 14:04 |
haleyb | i think the only thing i had was fixing some unit tests to support sqlalchemy 2 - https://review.opendev.org/c/openstack/neutron-lib/+/900557 | 14:05 |
haleyb | it's at least passed the debian amd64 testing | 14:05 |
haleyb | next announcement | 14:06 |
slaweq | o/ | 14:06 |
slaweq | sorry for being late | 14:06 |
haleyb | i was going to cancel the drivers meeting this friday as i'm on pto, and next friday since it's a US holiday, unless someone wants to run them | 14:06 |
haleyb | slaweq: o/ | 14:07 |
mlavalle | do we have any proposals that my warrant a meeting? | 14:08 |
mlavalle | or can we wait until december? | 14:08 |
haleyb | there is nothing at https://wiki.openstack.org/wiki/Meetings/NeutronCI at the moment | 14:08 |
mlavalle | If needed I can run the meeting this coming Friday | 14:08 |
haleyb | and no new bugs up to today | 14:08 |
mlavalle | next week's I have the same holiday as you \o/ | 14:09 |
haleyb | mlavalle: sure, we can decide thursday based on need, thanks! | 14:09 |
mlavalle | yeap | 14:09 |
lajoskatona | +1 for deciding on thursday | 14:09 |
haleyb | the other announcement is just with this being C-1 release week, I'm guessing there will be a neutron release proposed as well, so if there is anything you need there work on getting reviews | 14:12 |
slaweq | also, regarding releases - yoga is going to be unmaintained soon | 14:12 |
slaweq | we have many not released patches there - maybe it's worth to make one more release before this "final" step | 14:12 |
lajoskatona | elodilles sent out a mail with useful links: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/SYPTRTPUNWZFJNSL2PEYK4NX67RQ5LLW/ | 14:13 |
haleyb | slaweq: ack, thanks | 14:13 |
lajoskatona | the neutron etherpad from the mail: https://etherpad.opendev.org/p/neutron-stable-yoga-eom | 14:14 |
haleyb | i know we had gone through the stable branches last week trying to merge things, lets take a look again | 14:14 |
haleyb | fwiw i know someone here at canonical was going to propose releases for all the neutron stable branches, i will ping him to get that started | 14:15 |
haleyb | so since i'm dense, yoga will be deleted but u/v/w/x will still be around? | 14:18 |
haleyb | or maybe the *deleted* doesn't mean what i think | 14:19 |
lajoskatona | Perhaps ask on the release channel as I know things are fluid | 14:20 |
lajoskatona | but I just hve rumors from elodilles :-) | 14:20 |
haleyb | just yoga-em ? | 14:20 |
haleyb | sure i'll ask | 14:21 |
haleyb | any other announcements? | 14:21 |
haleyb | #action (haleyb) ask what *deleted* means in yoga release email | 14:21 |
haleyb | #topic bugs | 14:22 |
haleyb | lucasagomes was the deputy last week, his report is at | 14:22 |
haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/PXVY4XBS32G5RDDU5MZJTQ5PL5LIID7H/ | 14:22 |
haleyb | not too busy, and as of this morning only one was unassigned | 14:23 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2042941 | 14:23 |
haleyb | i think yatin is off today, but that might be related to the oslo.utils issue we saw last week | 14:24 |
haleyb | oh, i misread the bug | 14:24 |
haleyb | they are not installing later versions? | 14:25 |
haleyb | is anyone more familiar with the job syntax on that to confirm? | 14:27 |
lajoskatona | I can check it | 14:28 |
haleyb | lajoskatona: thanks | 14:28 |
haleyb | my weekly bug query shows we're down 2 bugs to 765 so moving in the right direction | 14:29 |
haleyb | hopefully more will get auto-closed as patches merge | 14:29 |
haleyb | this week's deputy is jlibosva - are you ok doing it? | 14:30 |
jlibosva | yes :) | 14:30 |
haleyb | and next week is obondarev | 14:31 |
obondarev | o/ | 14:31 |
haleyb | jlibosva: ack, and i see zero bugs so far :) | 14:31 |
haleyb | any other bugs anyone wants to discuss? | 14:32 |
haleyb | #topic specs | 14:33 |
haleyb | #link https://review.opendev.org/q/project:openstack%252Fneutron-specs+status:open | 14:33 |
haleyb | currently 3 open, the two BGP ones are close to merging, i just had some nits :-o | 14:34 |
haleyb | otherwiseguy had raised a question on the OVN interconnect one but hasn't had any follow-up yet | 14:36 |
haleyb | #link https://review.opendev.org/c/openstack/neutron-specs/+/891204 | 14:36 |
haleyb | so i'll just ask if you have an opinion to comment there | 14:37 |
haleyb | #topic community_goals | 14:39 |
haleyb | lajoskatona: any updates? | 14:40 |
lajoskatona | nothing, sorry | 14:41 |
haleyb | ack | 14:41 |
haleyb | moving on | 14:41 |
haleyb | #topic on_demand | 14:41 |
haleyb | i had some items, but will let others talk first if they have something | 14:42 |
haleyb | ok, in no particular order | 14:43 |
haleyb | OVN support for VPNaas. Does anyone have cycles to take a look? | 14:43 |
haleyb | #link https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353 | 14:43 |
haleyb | it has one +2 from slaweq and a +1 from lajoskatona, it's just an XXL patch | 14:44 |
haleyb | someone on the ML asked about this and it came up at our PTG | 14:44 |
mlavalle | wow 4k+ lines | 14:44 |
lajoskatona | I checked and stated it in devstack, so I am fine with it basically, but not deep testing as I am not vpn master | 14:45 |
opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add initial support for local OVN cluster instead of kernel-networking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/881779 | 14:46 |
haleyb | right, who are the contributors working on vpnaas? besides bodo and dmitriy i guess | 14:46 |
haleyb | would be good to have another +1 from someone familiar with vpn | 14:47 |
mlavalle | yeap | 14:47 |
lajoskatona | mnaser perhaps came earlier with questions or bugs | 14:47 |
haleyb | "This looks quite fair to me and it's confirmed to work by multiple operators. This also passes tempest test, so might worth having a shot and do a follow-up work whenever needed." - last comment from dmitriy | 14:48 |
mlavalle | ok, I'll take a stab at it over the next 2 weeks. I'll go slowly and deploy in devstack | 14:49 |
haleyb | if i have time i'll look through it, i guess as long as the db migration is sane, etc we can always fix the bugs later | 14:49 |
mlavalle | I might ping lajoskatona for some guidance | 14:49 |
haleyb | mlavalle: ack, thanks | 14:49 |
lajoskatona | +1 | 14:50 |
haleyb | ok, my second topic is... | 14:50 |
mlavalle | if by next Tuesday I feel I'm not making progress, I'll let you know during this meeting | 14:50 |
haleyb | Is anyone familiar with the neutron-lib API reporting tool? It has not been maintained and does not currently run | 14:50 |
haleyb | #link https://review.opendev.org/c/openstack/neutron-lib/+/899009 | 14:51 |
haleyb | has anyone run 'tox -e api-report' in neutron-lib in the past 7 years? | 14:51 |
lajoskatona | I checked first when saw a patch around it, and now it is not working | 14:52 |
slaweq | haleyb what is this "api-report"? | 14:52 |
slaweq | it's first time I see that :) | 14:52 |
haleyb | slaweq: you answered my question, i had no idea it was there either | 14:52 |
haleyb | and it doesn't run | 14:53 |
haleyb | #link https://review.opendev.org/c/openstack/neutron-lib/+/338571 | 14:54 |
haleyb | introduced there ^^ | 14:54 |
haleyb | we can either remove it, or fix the issue with the shebang and let someone try and get it working | 14:55 |
haleyb | but it seems we have enough on our plates right now | 14:55 |
slaweq | just a note: it was removed from pep8 job with https://review.opendev.org/c/openstack/neutron-lib/+/355563 | 14:56 |
mlavalle | I say remove it | 14:58 |
haleyb | mlavalle: ack, just wanted a second opinion, someone can always revert and fix it | 14:58 |
lajoskatona | + | 14:58 |
mlavalle | yeap | 14:58 |
lajoskatona | +1 | 14:58 |
haleyb | ok, that patch just needs an update to remove the rest of it, will ping submittor | 14:58 |
slaweq | ++ | 14:58 |
haleyb | last topic with one minute left | 14:59 |
haleyb | i had mentioned review days at the PTG, is everyone good at trying it for a day or two? | 14:59 |
mlavalle | yes | 15:00 |
lajoskatona | yes | 15:00 |
mlavalle | well, I am | 15:00 |
mlavalle | I'll let others speak for themselves | 15:00 |
slaweq | ++ | 15:00 |
haleyb | ok, i guess that makes four of us at least :) | 15:01 |
slaweq | just please let me know when it will be so I will organize other stuff in the way to prioritize this :) | 15:01 |
haleyb | i was going to propose next week, but begin a short week maybe the week after? does november 30th/dec 1st work? or can do later | 15:02 |
mlavalle | how about sending a doodle. Let's target second week of december | 15:02 |
mlavalle | the week after Thanksgiving is also good | 15:03 |
haleyb | mlavalle: that's a good idea, will send one | 15:03 |
haleyb | #action (haleyb) send a doodle about review days | 15:03 |
mlavalle | +1 | 15:03 |
haleyb | ok, we are over time | 15:03 |
haleyb | there is no CI meeting today, will be next week i believe | 15:04 |
slaweq | ok, thx for the info :) | 15:04 |
mlavalle | nope, Yatin if off today | 15:04 |
lajoskatona | thanks, just wanted to ask | 15:04 |
haleyb | lajoskatona: sorry, did you have a question? | 15:05 |
mlavalle | no, he was asking about the CI meeting | 15:05 |
lajoskatona | no, just thanks for the info that no CI meeting today | 15:05 |
mlavalle | answered | 15:05 |
haleyb | ah, ok | 15:05 |
lajoskatona | exactly | 15:05 |
haleyb | #endmeeting | 15:06 |
opendevmeet | Meeting ended Tue Nov 14 15:06:04 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:06 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2023/networking.2023-11-14-14.00.html | 15:06 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-11-14-14.00.txt | 15:06 |
opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2023/networking.2023-11-14-14.00.log.html | 15:06 |
mlavalle | o/ | 15:06 |
haleyb | have a good week o/ | 15:06 |
lajoskatona | o/ | 15:06 |
opendevreview | Merged openstack/ovn-bgp-agent master: Add initial support for local OVN cluster instead of kernel-networking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/881779 | 15:06 |
opendevreview | Brian Haley proposed openstack/neutron master: Enable liberal TCP connection tracking for SNAT namespaces https://review.opendev.org/c/openstack/neutron/+/618208 | 15:10 |
opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove unmaintained API report tooling https://review.opendev.org/c/openstack/neutron-lib/+/899009 | 15:10 |
opendevreview | Brian Haley proposed openstack/neutron-lib master: api-ref: Document only unbound ports vnic_type updates https://review.opendev.org/c/openstack/neutron-lib/+/900826 | 15:14 |
haleyb | slaweq: can you take a look at https://review.opendev.org/c/openstack/neutron-lib/+/900557 ? it seems to fix the issue based on the debian build logs | 15:22 |
opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove unmaintained API report tooling https://review.opendev.org/c/openstack/neutron-lib/+/899009 | 15:26 |
slaweq | haleyb done | 15:26 |
otherwiseguy | @haleyb: on the ovn interconnect db-sync issue, I just can't think of any reason for the sync code to assume, unless otherwise instructed with a special flag on each potential object type, that it is free to delete it. Seems better to use the flags we already set and only touch neutron-owned stuff. | 15:28 |
otherwiseguy | historically we've treated the ovn db like it is just ours, but back in the day there was no reason not to. | 15:29 |
otherwiseguy | Now that there is the possibility of sharing the db with others, it just seems better to only touch our stuff instead of enumerating all of the possible things we might share with and looking for their signatures and creating an "ignore-this!" flag for things that don't have obvious signatures. | 15:30 |
opendevreview | Jakub Libosvar proposed openstack/neutron-lib master: api-ref: Document only unbound ports vnic_type updates https://review.opendev.org/c/openstack/neutron-lib/+/900826 | 15:39 |
haleyb | otherwiseguy: right, it somehow seems more correct to look for things with our fingerprints whenever possible | 15:41 |
elodilles | haleyb: the unmaintained process will be something like this: 1) tag the HEAD of stable/$series ($series-eom == end-of-maintenance); 2) delete stable/$series; 3) cut unstable/$series from $series-eom tag | 15:42 |
elodilles | (and tag $series-eol at some point at the HEAD of unstable/$series branch) | 15:46 |
haleyb | elodilles: ok, so yoga-em is created and stable/yoga goes away. i don't understand the unstable part there | 15:48 |
haleyb | oh, and i guess this means we should tag a 20.5.0 or such before this | 15:48 |
elodilles | haleyb: when we had EM then the last release was tagged with $series-em, now we tag the HEAD of stable/$series instead. so the release is necessary if you decide it to be important | 15:50 |
haleyb | elodilles: i think it's important for us downstream, let me ask our build team, just makes for easier work i think | 15:51 |
frickler | elodilles: s/unstable/unmaintained/ | 15:51 |
frickler | (for the branch names) | 15:52 |
elodilles | frickler: /o\ thanks | 15:59 |
haleyb | frickler: i don't see any neutron branches with 'unmaintained' in their names, just tags with -em and -eol, so i'm still confused on that part | 16:03 |
opendevreview | Jakub Libosvar proposed openstack/neutron master: Forbid updating vnic type on a bound port https://review.opendev.org/c/openstack/neutron/+/892815 | 16:04 |
opendevreview | Jakub Libosvar proposed openstack/neutron master: Forbid updating vnic type on a bound port https://review.opendev.org/c/openstack/neutron/+/892815 | 16:05 |
haleyb | jlibosva: hey, can you look at this one too? https://review.opendev.org/c/openstack/neutron-lib/+/895940 - thanks in advance | 16:50 |
haleyb | bcafarel: can you look at https://review.opendev.org/c/openstack/neutron/+/900196 and related changes? i see the yoga one got merged but not some of the others | 17:19 |
bcafarel | haleyb: ah yes I remember looking at that chain, some backports had failing CI and missing conflicts - now looks good, let me review/approve them | 17:20 |
haleyb | recheck; recheck; recheck worked :) | 17:20 |
opendevreview | Merged openstack/neutron-lib master: Change test_sqlalchemytypes.py to support sqlalchemy 2.0 https://review.opendev.org/c/openstack/neutron-lib/+/900557 | 17:24 |
haleyb | zigo: your build looks clean and patch has merged, does that need to be backported? otherwise i'll just update the currently planned release to include that | 17:37 |
opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove unmaintained API report tooling https://review.opendev.org/c/openstack/neutron-lib/+/899009 | 17:38 |
haleyb | otherwiseguy: when you get a chance can you look at my comment in https://review.opendev.org/c/openstack/ovsdbapp/+/896686 ? trying to tag the ovsdbapp release this week, thanks! | 17:44 |
frickler | haleyb: the switch hasn't happened yet, and yoga will be the first series which is transitioned. how to handle older branches in EM status is still to be discussed | 17:48 |
opendevreview | Merged openstack/neutron master: Remove obsolete PID files before start https://review.opendev.org/c/openstack/neutron/+/895832 | 18:02 |
opendevreview | Merged openstack/tap-as-a-service master: Support for ovs-dpdk https://review.opendev.org/c/openstack/tap-as-a-service/+/884239 | 19:07 |
opendevreview | Merged openstack/neutron stable/2023.1: [DHCP agent] Add route to OVN metadata port if exists https://review.opendev.org/c/openstack/neutron/+/900196 | 19:22 |
opendevreview | Merged openstack/neutron stable/zed: [DHCP agent] Add route to OVN metadata port if exists https://review.opendev.org/c/openstack/neutron/+/900197 | 19:22 |
opendevreview | Merged openstack/neutron stable/xena: [DHCP agent] Add route to OVN metadata port if exists https://review.opendev.org/c/openstack/neutron/+/900199 | 19:22 |
opendevreview | Merged openstack/neutron stable/wallaby: [DHCP agent] Add route to OVN metadata port if exists https://review.opendev.org/c/openstack/neutron/+/900200 | 19:22 |
opendevreview | Miro Tomaska proposed openstack/neutron master: Metadata: handle process exceptions https://review.opendev.org/c/openstack/neutron/+/890986 | 19:40 |
opendevreview | Jakub Libosvar proposed openstack/neutron stable/wallaby: Revert "[OVN][Trunk] Add port binding info on subport when parent is bound" https://review.opendev.org/c/openstack/neutron/+/900896 | 21:05 |
opendevreview | Jakub Libosvar proposed openstack/neutron stable/wallaby: Revert "[OVN][Trunk] Add port binding info on subport when parent is bound" https://review.opendev.org/c/openstack/neutron/+/900896 | 21:23 |
opendevreview | Merged openstack/neutron stable/wallaby: Partially revert "Do not link up HA router gateway in backup node" https://review.opendev.org/c/openstack/neutron/+/845406 | 23:03 |
opendevreview | Merged openstack/neutron stable/victoria: Partially revert "Do not link up HA router gateway in backup node" https://review.opendev.org/c/openstack/neutron/+/847633 | 23:03 |
opendevreview | Merged openstack/neutron stable/2023.1: Ensure ovn loadbalancer FIPs are centralized upon neutron restarts https://review.opendev.org/c/openstack/neutron/+/900882 | 23:48 |
haleyb | otherwiseguy: you still around? didn't know if you saw my comment on https://review.opendev.org/c/openstack/ovsdbapp/+/896686 | 23:54 |
otherwiseguy | I hadn't seen it. I'll investigate. | 23:55 |
haleyb | just a question on the func test, i hate adding code without tests :) | 23:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!