| opendevreview | Merged openstack/neutron-lib master: Skip the enforcer undefined rule check https://review.opendev.org/c/openstack/neutron-lib/+/906539 | 01:20 |
|---|---|---|
| opendevreview | Merged openstack/neutron stable/2023.2: Disallow subnet cidr of :: without PD https://review.opendev.org/c/openstack/neutron/+/908973 | 03:00 |
| slaweq | ralonsoh or ykarel or lajoskatona can one of You check https://review.opendev.org/c/openstack/neutron/+/909848 ? thx in advance :) | 07:24 |
| lajoskatona | slaweq: checking | 07:25 |
| ralonsoh | sure, let me check | 07:26 |
| ykarel | ack | 07:27 |
| ralonsoh | slaweq, wouldn't be better to make this change on top of https://review.opendev.org/c/openstack/neutron/+/894399? | 07:27 |
| slaweq | ralonsoh I talked with haleyb yesterday about it and as I want to backport my patch we decided that it will be better to first merge my patch | 07:34 |
| ralonsoh | slaweq, understood | 07:34 |
| slaweq | and I am now rebasing haleyb's patch on top of mine so it should be ready to go also :) | 07:34 |
| ralonsoh | perfect done, approved | 07:34 |
| slaweq | thx you both :) | 07:34 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Make common Metadata Driver classes https://review.opendev.org/c/openstack/neutron/+/894399 | 07:39 |
| slaweq | haleyb ^^ I rebased Your patch on top of Yours :) | 07:39 |
| *** liuxie is now known as liushy | 07:49 | |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Disable exposing remote_ips, when only the lrp prefix is sufficient https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910303 | 07:57 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Trimm interface name consistently https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910304 | 07:58 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Fix event handling for LSP and prefer the options.requested-chassis info https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910305 | 07:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Implement OVN agent metadata extension https://review.opendev.org/c/openstack/neutron/+/898238 | 08:10 |
| lajoskatona | bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki: general question: do you know if you can travel to any of the OpenInfra days (https://openinfra.dev/days )? | 08:58 |
| lajoskatona | if yes we should sync it to meet and discuss (drink a beer :-)) in person and than I can start asking my management for travel budget :-) | 08:59 |
| slaweq | lajoskatona I don't know. I would love to but I doubt I will be able to | 09:00 |
| frickler | lajoskatona: I will not attend any event lacking a proper health policy, even if Berlin didn't even mean travel for me | 09:00 |
| obondarev | lajoskatona: I can't due to personal reasons | 09:00 |
| lajoskatona | slaweq, frickler: thanks | 09:00 |
| lajoskatona | obondarev: thanks | 09:00 |
| opendevreview | liuxie proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 09:01 |
| opendevreview | Liushy proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 09:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Remove OVN_GATEWAY_INVALID_CHASSIS artifact https://review.opendev.org/c/openstack/neutron/+/909305 | 09:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Do not use netaddr.core.ZEROFILL flag with IPv6 addresses https://review.opendev.org/c/openstack/neutron-lib/+/910331 | 09:42 |
| opendevreview | Merged openstack/neutron master: Ensure that haproxy spawned by the metadata agents is active https://review.opendev.org/c/openstack/neutron/+/909848 | 10:03 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Add feature to check if SNAT disabled before exposing tenant networks https://review.opendev.org/c/openstack/ovn-bgp-agent/+/907068 | 10:34 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix address scope test and add address scope unit tests https://review.opendev.org/c/openstack/ovn-bgp-agent/+/907187 | 10:34 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Add support for l3vpn with NB driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906505 | 10:34 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Make scheduling of unhosted gateways aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 10:47 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 10:47 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 10:47 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 10:47 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Ensure OVN DB update on change of number of GW ports https://review.opendev.org/c/openstack/neutron/+/909955 | 10:47 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP: Add documentation for aa-l3-gw-multihoming https://review.opendev.org/c/openstack/neutron/+/899402 | 10:47 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Ensure all routes are deleted when deleting ExtGw https://review.opendev.org/c/openstack/neutron/+/910332 | 10:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Make mandatory the router name in the LRP.external_ids https://review.opendev.org/c/openstack/neutron/+/910222 | 11:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] "Logical_Router" pinned to chassis, OVN L3 scheduler https://review.opendev.org/c/openstack/neutron/+/909194 | 11:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Provide HA functionality to "Logical_Router" chassis pinning https://review.opendev.org/c/openstack/neutron/+/909437 | 11:13 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Trimm interface name consistently https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910304 | 11:48 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Disable exposing remote_ips, when only the lrp prefix is sufficient https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910303 | 11:53 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Fix event handling for LSP and prefer the options.requested-chassis info https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910305 | 11:55 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Fix backward compatibility for setups that export subnets per host https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910306 | 11:55 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Update NB driver to re-use new methods and make code cleaner https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910307 | 11:56 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.2: Ensure that haproxy spawned by the metadata agents is active https://review.opendev.org/c/openstack/neutron/+/910308 | 11:56 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.1: Ensure that haproxy spawned by the metadata agents is active https://review.opendev.org/c/openstack/neutron/+/910309 | 11:57 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/zed: Ensure that haproxy spawned by the metadata agents is active https://review.opendev.org/c/openstack/neutron/+/910335 | 11:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] "Logical_Router" pinned to chassis, OVN L3 scheduler https://review.opendev.org/c/openstack/neutron/+/909194 | 12:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Provide HA functionality to "Logical_Router" chassis pinning https://review.opendev.org/c/openstack/neutron/+/909437 | 12:21 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Fix backward compatibility for setups that export subnets per host https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910306 | 12:26 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Update NB driver to re-use new methods and make code cleaner https://review.opendev.org/c/openstack/ovn-bgp-agent/+/910307 | 12:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Implement OVN agent metadata extension https://review.opendev.org/c/openstack/neutron/+/898238 | 12:56 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 13:00 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Ensure OVN DB update on change of number of GW ports https://review.opendev.org/c/openstack/neutron/+/909955 | 13:00 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Ensure all routes are deleted when deleting ExtGw https://review.opendev.org/c/openstack/neutron/+/910332 | 13:00 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP: Add documentation for aa-l3-gw-multihoming https://review.opendev.org/c/openstack/neutron/+/899402 | 13:00 |
| opendevreview | Bodo Petermann proposed openstack/neutron-lib master: vpnaas: add support for AES CTR https://review.opendev.org/c/openstack/neutron-lib/+/903971 | 13:08 |
| lajoskatona | haleyb, ralonsoh, slaweq: I have a parallel meeting, so can't join today the team meeting | 13:40 |
| haleyb | lajoskatona: ack, np | 13:59 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Feb 27 14:00:17 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 'networking' | 14:00 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 14:00 |
| mlavalle | \o | 14:00 |
| slaweq | o/ | 14:00 |
| frickler | o/ | 14:00 |
| obondarev | hi | 14:00 |
| ykarel | o/ | 14:00 |
| fnordahl | o/ | 14:00 |
| ralonsoh | hello | 14:01 |
| bcafarel | o/ | 14:01 |
| haleyb | ok, lets get started | 14:01 |
| elvira1 | o/ | 14:01 |
| haleyb | #topic announcements | 14:01 |
| haleyb | #link https://releases.openstack.org/caracal/schedule.html | 14:02 |
| ihrachys | o/ | 14:02 |
| haleyb | We are in C-3 milestone week | 14:02 |
| haleyb | that means we shouldn't be landing a feature after this week | 14:03 |
| haleyb | unless we grant an exception | 14:03 |
| haleyb | the week of March 11 is RC1 | 14:04 |
| haleyb | that said, we still have one open feature | 14:04 |
| haleyb | #link https://review.opendev.org/q/topic:%222023-aa-l3-gw-multihoming%22 | 14:04 |
| haleyb | fnordahl has been busy rebasing and addressing comments, would like to merge that since a lot of the changes have landed | 14:06 |
| fnordahl | I for one would be very interested in seeing that land, and happy to answer any questions. 90% of the feature has landed already. The remaining pieces is handling of the BFD records, the anti affinity support for scheduling of LRPs and of course client and documentation. | 14:07 |
| fnordahl | Are there any open issues as you see it preventing the review? | 14:08 |
| haleyb | i cound 7 neutron patches, 2 n-t-p and 2 client | 14:08 |
| haleyb | s/count | 14:08 |
| haleyb | fnordahl: i think you have addressed all my comments, just need to re-review the chain | 14:08 |
| fnordahl | Helpfully it went into merge conflict this morning, which would be the reason for there not being test results on all of them right now, but the results are trickling in | 14:09 |
| mlavalle | I also want to draw attention to https://review.opendev.org/c/openstack/neutron/+/901513. It is the last part of OVN router flavors that we merged earlier during the cycle | 14:10 |
| mlavalle | I'll address lajoskatona comments from yesterday (thanks!) later today | 14:10 |
| haleyb | mlavalle: ack, will take a look | 14:11 |
| mlavalle | haleyb: wait for me to fix what lajoskatona pointed out. I'll ping you | 14:12 |
| mlavalle | thanks! | 14:12 |
| haleyb | sure | 14:12 |
| haleyb | i would encourage people to ping reviewers if it helps to get things in this week | 14:13 |
| haleyb | any other features to talk about? | 14:14 |
| haleyb | one other announcement, i did self-nominate myself for ptl again, so if noone else runs you're stuck with me for 2024.2 :) | 14:15 |
| ralonsoh | +1 | 14:15 |
| obondarev | thanks haleyb | 14:15 |
| mlavalle | Great! | 14:15 |
| mlavalle | thanks! | 14:15 |
| haleyb | also, the TC nomination period is still open through today i think | 14:16 |
| haleyb | please check the schedule to be sure if you are interested | 14:16 |
| haleyb | #topic bugs | 14:17 |
| haleyb | lucasgomes was deputy last week | 14:17 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/PK7JYOJK55WKEC4LGW25V7OWRG6DYEZH/ | 14:17 |
| haleyb | there were only 3 (!) bugs | 14:17 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2054435 | 14:18 |
| haleyb | Testing errors when using new netaddr library | 14:18 |
| ralonsoh | that is related with the topic I would like to comment later | 14:19 |
| ralonsoh | about grouping the netaddr=1.0.0 problems in one LP bug | 14:19 |
| haleyb | ralonsoh: ack, we can discuss in on-demand | 14:20 |
| haleyb | next bug | 14:20 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2054324 | 14:20 |
| haleyb | Iptables rule wrong if created a rule with protocol 4 | 14:20 |
| haleyb | i picked this up and proposed two changes, one in neutron-lib and one in neutron, which is temporary and backportable | 14:21 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/909943 | 14:21 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron-lib/+/909942 | 14:21 |
| haleyb | i wrote up a description in the bug since it's an odd one | 14:22 |
| ralonsoh | qq about the n-lib patch | 14:22 |
| ihrachys | write-up is cool; would be even cooler to have it in the commit message :) | 14:23 |
| ralonsoh | we are adding a new variable to a constants list | 14:23 |
| ralonsoh | that is already used in the API | 14:23 |
| ralonsoh | is that acceptable because this is a bug? | 14:23 |
| ralonsoh | (because that was a missing value in this list) | 14:23 |
| haleyb | ralonsoh: i think it's ok since neutron is the only user of that list afaik, let me check codesearch | 14:24 |
| ralonsoh | but only if Neutron is using it | 14:25 |
| haleyb | yes, it's only used in the iptables firewall driver when determining what to use with iptables-save/restore | 14:25 |
| haleyb | ihrachys: i can put in the commit message, i might be the only one left with that esoteric knowledge of the iptables driver | 14:26 |
| haleyb | the final bug here is an older one | 14:28 |
| haleyb | #link https://bugs.launchpad.net/nova/+bug/2008238 | 14:28 |
| haleyb | SRIOV port binding_profile attributes for OVS hardware offload are stripped on instance deletion or port detachment | 14:28 |
| ralonsoh | we are no longer writing any value in the port binding dict | 14:28 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/898556 | 14:28 |
| ralonsoh | so this is no longer affecting Neutron | 14:29 |
| haleyb | ack, i see you removed neutron from it, so --bug | 14:29 |
| haleyb | Current bug count this week: 739, down 6 from last week | 14:30 |
| haleyb | once we go into freeze i will spend more time pruning, thanks to others that have done the same | 14:30 |
| haleyb | This week bug deputy is jlibosva | 14:31 |
| haleyb | i don't see him here | 14:31 |
| ralonsoh | I'll ping him later | 14:31 |
| haleyb | next week is obondarev - is that good? | 14:31 |
| obondarev | yep | 14:32 |
| haleyb | thanks | 14:32 |
| haleyb | topic #specs | 14:33 |
| haleyb | #link https://review.opendev.org/q/project:openstack%252Fneutron-specs+status:open | 14:33 |
| haleyb | there are still 2 open specs regarding bgp, if we don't merge this cycle will move them to next | 14:33 |
| haleyb | it's really too late to merge any code related to them | 14:34 |
| haleyb | #topic community_goals | 14:35 |
| haleyb | i realize lajoskatona is not able to make it, his horizon patch is waiting for the sdk to be released | 14:35 |
| haleyb | #link https://review.opendev.org/c/openstack/horizon/+/891205 | 14:35 |
| haleyb | so hopefully that happens soon since it's a step to removing rest of the neutronclient code | 14:36 |
| haleyb | moving on | 14:36 |
| haleyb | #topic on_demand | 14:37 |
| haleyb | i did not update wiki, but we have one item from ralonsoh | 14:37 |
| ralonsoh | very quick: we are finding new netaddr>=1.0.0 errors | 14:37 |
| ralonsoh | now we have 2 LP bugs but I found another one this morning | 14:37 |
| ralonsoh | IMO, we should track them all in a single LP bug | 14:37 |
| ralonsoh | or at least have some kind of prefix in the commit message | 14:38 |
| ralonsoh | same as sqlalchemy-20 | 14:38 |
| ykarel | +1 | 14:39 |
| haleyb | ralonsoh: what was the third one? | 14:39 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2054435 | 14:39 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2054203 | 14:40 |
| ralonsoh | no, I didn't open a LP | 14:40 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-lib/+/910331 | 14:40 |
| haleyb | oh, "now we have 2 LP bugs" | 14:40 |
| haleyb | i thought you found a third | 14:40 |
| ralonsoh | yes, this patch is a new bug | 14:40 |
| haleyb | ah, ok | 14:42 |
| haleyb | i did see takashi had proposed a netaddr patch so we could use the old style behavior, which would then require a change in neutron or oslo.utils | 14:43 |
| haleyb | #link https://github.com/netaddr/netaddr/pull/377 | 14:44 |
| haleyb | because we still need to support things like 10/8 because they used to work | 14:44 |
| ralonsoh | this is one of the issues reported in on LP bug | 14:44 |
| ralonsoh | that's the point | 14:44 |
| ralonsoh | nevermind, if we find new bugs, we should open new LP bugs | 14:45 |
| ralonsoh | that's all, thanks | 14:45 |
| haleyb | ralonsoh: ack, thanks | 14:45 |
| haleyb | any other topics? | 14:46 |
| haleyb | neutron-ci meeting is top of hour, irc this time i believe | 14:46 |
| ykarel | yes right | 14:47 |
| haleyb | ok, thanks everyone for attending, reach out on irc for reviews etc as it will be a busy week | 14:48 |
| haleyb | #endmeeting | 14:48 |
| opendevmeet | Meeting ended Tue Feb 27 14:48:30 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:48 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-02-27-14.00.html | 14:48 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-02-27-14.00.txt | 14:48 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-02-27-14.00.log.html | 14:48 |
| mlavalle | \o | 14:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: [netaddrd>=1.0.0] Do not use netaddr.core.ZEROFILL flag with IPv6 https://review.opendev.org/c/openstack/neutron-lib/+/910331 | 14:48 |
| mlavalle | ykarel: video or irc meeting today? | 14:48 |
| ykarel | irc | 14:49 |
| mlavalle | ack | 14:49 |
| fnordahl | \o | 14:49 |
| bcafarel | lajoskatona: and for your earlier question - I do plan to go to OpenInfra Day France (should be manageable budget-wise) | 14:50 |
| ihrachys | ralonsoh wonder if netaddr had a list of incompat changes that projects could go through as a checklist to see if nothing was missed. (unit test failures are fine but if coverage is not enough, ...) | 14:50 |
| ralonsoh | ihrachys, no, that's the point | 14:50 |
| ralonsoh | these changes were made by accident, some of them | 14:50 |
| ralonsoh | my goal was to group all netaddr possible errors somehow, to track them | 14:51 |
| haleyb | i think using the same topic is good for now | 14:52 |
| fnordahl | hrm, aparently the rebase for the merge of https://review.opendev.org/c/openstack/neutron/+/909649 broke https://review.opendev.org/c/openstack/neutron/+/874760/ . We piggy backed on the DelLRouterCommand to ensure no BFD records were left behind. I see there are multiple entry points in ovn_client dealing with removal of router and LRP so I guess we need to propagate it there instead | 14:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] "Logical_Router" pinned to chassis, OVN L3 scheduler https://review.opendev.org/c/openstack/neutron/+/909194 | 14:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Provide HA functionality to "Logical_Router" chassis pinning https://review.opendev.org/c/openstack/neutron/+/909437 | 14:56 |
| ykarel | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Feb 27 15:00:55 2024 UTC and is due to finish in 60 minutes. The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:00 |
| slaweq | o/ | 15:01 |
| ykarel | ping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:01 |
| lajoskatona | o/ | 15:01 |
| lajoskatona | ykarel: is this a video or IRC only meeting today? | 15:01 |
| ykarel | irc | 15:01 |
| lajoskatona | cool , thanks | 15:01 |
| bcafarel | o/ | 15:02 |
| mlavalle | o/ | 15:02 |
| ykarel | k let's start with topics | 15:03 |
| ykarel | #topic Actions from previous meetings | 15:03 |
| ykarel | ykarel to check fullstack failure | 15:04 |
| ykarel | i checked logs but didn't find much there | 15:04 |
| ykarel | will keep an eye on it if it reproduces will open a bug for it | 15:04 |
| ykarel | ykarel to check grenade status | 15:05 |
| ykarel | so it's broken on stable/2023.1 | 15:05 |
| ykarel | https://zuul.openstack.org/builds?job_name=neutron-ovn-grenade-multinode-skip-level&job_name=neutron-ovs-grenade-multinode-skip-level&branch=stable%2F2023.1&skip=0 | 15:05 |
| ykarel | https://review.opendev.org/q/topic:%22bug/2052915%22 | 15:05 |
| ykarel | there are some open patches, and there is discussion whether we should test grenade from unmaintained branch or not | 15:05 |
| lajoskatona | good that you bring it here | 15:06 |
| ykarel | also non skip level job in zed are impacted | 15:06 |
| lajoskatona | to tell the truth I am lost what is the direction and I am not a grenade expert anyway :-( | 15:06 |
| bcafarel | yes skip-level is non-voting for 2023.1, but zed gate is blocked atm | 15:06 |
| ykarel | https://zuul.openstack.org/builds?job_name=neutron-ovs-grenade-multinode&job_name=neutron-ovn-grenade-multinode&branch=stable%2Fzed&skip=0 | 15:07 |
| lajoskatona | I remember somebody said that we should stop pgrade test from unmaintained to anything else | 15:07 |
| ykarel | last run passed, don't know how as not related to patch :) | 15:07 |
| ykarel | may be slaweq did some magic in his patch | 15:08 |
| bcafarel | hmm interesting | 15:08 |
| slaweq | I didn't do any magic for sure :) | 15:09 |
| bcafarel | unconscious magic, even more impressive! :) | 15:09 |
| ykarel | so i think for this we should join the discussion on those patches https://review.opendev.org/q/topic:%22bug/2052915%22 | 15:10 |
| ykarel | and decide next steps, i.e drop or pin jobs to unmaintained/yoga or some commit | 15:10 |
| lajoskatona | both fine for me, if QA or TC has some common thing for all related project that would be the best | 15:11 |
| bcafarel | +1 we can wait a bit and have common fix/way to go from there | 15:12 |
| ykarel | to unblock zed we can make jobs non-voting | 15:12 |
| lajoskatona | +1 | 15:12 |
| lajoskatona | I can pus hthe n-vote patches for zed | 15:13 |
| ykarel | thx lajoskatona | 15:13 |
| ykarel | moving to next | 15:14 |
| ykarel | #topic Stable branches | 15:14 |
| ykarel | bcafarel, anything else apart from zed grenade issue? | 15:14 |
| bcafarel | we already discussed the main topic here :) (grenade in zed/antelope branches) | 15:14 |
| bcafarel | apart from that, all good - it helps with less branches to keep an eye on | 15:14 |
| ykarel | thx for the update | 15:15 |
| ykarel | #topic Stadium projects | 15:15 |
| ykarel | all green in periodic | 15:15 |
| lajoskatona | yes | 15:15 |
| ykarel | lajoskatona, anything else to share for stadiums? | 15:15 |
| lajoskatona | the usual: please keep an eye on the open patches on stadiums :-) thanks for it :-) | 15:15 |
| ykarel | thx lajoskatona | 15:16 |
| ykarel | #topic Rechecks | 15:16 |
| ykarel | overall it's good here as not many rechecks in master | 15:17 |
| ykarel | and just 1 bare rechecks out of 15 | 15:17 |
| ykarel | let's keep doing it i.e no bare rechecks | 15:17 |
| ykarel | #topic fullstack/functional | 15:17 |
| ykarel | we still seeing NetworkInterfaceNotFound in functional jobs | 15:18 |
| ykarel | after ralonsoh fixes, seems it helped as not seeing many such failures as before | 15:18 |
| ralonsoh | do you have links? | 15:18 |
| ykarel | #link https://review.opendev.org/c/openstack/neutron/+/909385 | 15:18 |
| ralonsoh | to check if something else could be done | 15:19 |
| ykarel | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_326/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-uwsgi-fips/32627fb/testr_results.html | 15:19 |
| ykarel | https://f0db0d256d3d4b350b70-3f475577843c6519bda0c8c12c28da61.ssl.cf1.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-sqlalchemy-master/1d7e278/testr_results.html | 15:19 |
| ykarel | https://840d36dd41e668b905ff-5ceaba4a59a2578b108bb2f456b59bef.ssl.cf1.rackcdn.com/910208/2/check/neutron-functional-with-uwsgi/fcf02d9/testr_results.html | 15:19 |
| ralonsoh | these are actually failing on the methods already decorated | 15:20 |
| ykarel | symptom wise in all ^ failed at tenacity retry get/set link_attribute. and from ovs vswitchd logs all had shy port symptom, quickly add/delete/add port | 15:20 |
| ykarel | so we can follow up any further action for this on the same lp bug | 15:23 |
| ykarel | for now may be we can backport your retry patch to stable too? | 15:24 |
| ralonsoh | sure, I'll do it | 15:24 |
| ykarel | thx | 15:24 |
| ykarel | next one | 15:24 |
| ykarel | AttributeError: 'WaitForCreatePortBindingEvent' object has no attribute 'row' | 15:24 |
| ykarel | https://e3258dd049a220460aa0-add3baf7d164319a1fbb45a2e58a56e9.ssl.cf2.rackcdn.com/periodic/opendev.org/openstack/neutron/stable/2023.1/neutron-functional-with-uwsgi-fips/1d1fbd3/testr_results.html | 15:24 |
| ykarel | seen once | 15:24 |
| ralonsoh | very weird, self.row is assigned in the run method, | 15:26 |
| ralonsoh | if that happens, wait will exit | 15:26 |
| ralonsoh | ah ah ah, we don't check that wait result | 15:26 |
| ralonsoh | row_event.wait() | 15:26 |
| ralonsoh | this should be self.assertTrue(row_event.wait()) | 15:27 |
| ralonsoh | at least we check that wait failed or not | 15:27 |
| ralonsoh | I'll push a patch | 15:27 |
| ykarel | thx ralonsoh | 15:27 |
| ykarel | #topic Tempest/Scenario | 15:28 |
| ykarel | https://b2732a041c3f8a8a2227-9262fa59dca60616978c458f4dd8b484.ssl.cf5.rackcdn.com/908973/1/gate/neutron-ovn-tempest-ipv6-only-ovs-release/e79929e/testr_results.html | 15:29 |
| ykarel | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_11c/908973/1/gate/neutron-ovs-tempest-multinode-full/11c9845/testr_results.html | 15:29 |
| ykarel | also similar failure in other grenade job https://22dafa932966946580c1-ab66e702835932941e7e8df246c29428.ssl.cf5.rackcdn.com/909955/4/check/neutron-ovn-grenade-multinode-skip-level/c292b2c/testr_results.html | 15:29 |
| ykarel | any volunteer to check if these some real issues? | 15:30 |
| ralonsoh | I'll check them tomorrow | 15:31 |
| ykarel | thx ralonsoh | 15:31 |
| ykarel | #action ralonsoh to check tempest test_server_actions failures | 15:31 |
| ykarel | #topic Periodic | 15:31 |
| ykarel | all good except some functional test failure that were raised before | 15:31 |
| ykarel | #topic Grafana | 15:32 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:32 |
| ykarel | let's have quick look at it too | 15:32 |
| ykarel | apart from some spikes in check queue rest looks good | 15:33 |
| mlavalle | lgtm as well | 15:33 |
| ykarel | when i was checking failures most of those failures were patch related | 15:33 |
| ykarel | ok moving on | 15:34 |
| ykarel | #topic On Demand | 15:34 |
| ykarel | anything else you would like to raise? | 15:34 |
| mlavalle | not me | 15:35 |
| slaweq | nothing from me | 15:35 |
| lajoskatona | nothing from me | 15:35 |
| ykarel | k in that case let's close early and have everyone 23 minutes back :) | 15:37 |
| ykarel | #endmeeting | 15:37 |
| opendevmeet | Meeting ended Tue Feb 27 15:37:09 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-02-27-15.00.html | 15:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-02-27-15.00.txt | 15:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-02-27-15.00.log.html | 15:37 |
| ralonsoh | bye | 15:37 |
| mlavalle | \o | 15:37 |
| ykarel | thx everyone | 15:37 |
| slaweq | o/ | 15:37 |
| lajoskatona | o/ | 15:37 |
| bcafarel | o/ | 15:39 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Retry ``set|get_link_attribute(s)`` if the interface is not present https://review.opendev.org/c/openstack/neutron/+/910355 | 15:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Retry ``set|get_link_attribute(s)`` if the interface is not present https://review.opendev.org/c/openstack/neutron/+/910356 | 15:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Retry ``set|get_link_attribute(s)`` if the interface is not present https://review.opendev.org/c/openstack/neutron/+/910357 | 15:42 |
| ralonsoh | fnordahl, from https://review.opendev.org/c/openstack/neutron/+/898238 | 16:01 |
| ralonsoh | please explain this extra filtering in https://github.com/openstack/neutron/blob/491fbc890bd6a22e77570982dea9c37786ea412f/doc/source/admin/ovn/l3_scheduler.rst | 16:01 |
| ralonsoh | in a follow up patch | 16:02 |
| ralonsoh | sorry, wrong link | 16:02 |
| ralonsoh | this patch: https://review.opendev.org/c/openstack/neutron/+/873699/ | 16:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][FT] Check ``WaitForCreatePortBindingEvent`` wait result https://review.opendev.org/c/openstack/neutron/+/910363 | 16:13 |
| *** dasm is now known as Guest1195 | 16:51 | |
| *** Guest1195 is now known as dasm | 17:34 | |
| opendevreview | Merged openstack/ovn-bgp-agent master: Add feature to check if SNAT disabled before exposing tenant networks https://review.opendev.org/c/openstack/ovn-bgp-agent/+/907068 | 17:49 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/zed: Stable-Only: make grenade job non-voting on Zed https://review.opendev.org/c/openstack/neutron/+/910379 | 17:56 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Add tap_mirror to extension to OVN supported extensions https://review.opendev.org/c/openstack/neutron/+/905840 | 18:01 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Ensure all routes are deleted when deleting ExtGw https://review.opendev.org/c/openstack/neutron/+/910332 | 18:53 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Ensure OVN DB update on change of number of GW ports https://review.opendev.org/c/openstack/neutron/+/909955 | 18:53 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 18:53 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Document the OVN L3 scheduler: Anti-Affinity https://review.opendev.org/c/openstack/neutron/+/910435 | 19:38 |
| *** elodilles is now known as elodilles_pto | 19:46 | |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix iptables mapping of 'ipip' protocol https://review.opendev.org/c/openstack/neutron/+/909943 | 20:14 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Fix iptables mapping of 'ipip' protocol https://review.opendev.org/c/openstack/neutron-lib/+/909942 | 20:14 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix netaddr library issue in OVS firewall driver https://review.opendev.org/c/openstack/neutron/+/910439 | 20:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!