| opendevreview | Miguel Lavalle proposed openstack/neutron master: Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 00:43 |
|---|---|---|
| opendevreview | Merged openstack/neutron master: Replace network type names by constants https://review.opendev.org/c/openstack/neutron/+/902046 | 01:19 |
| opendevreview | Merged openstack/neutron master: Remove ovn_l3_mode configuration option https://review.opendev.org/c/openstack/neutron/+/901505 | 01:48 |
| opendevreview | Merged openstack/neutron master: Fix releasenote location https://review.opendev.org/c/openstack/neutron/+/901631 | 01:48 |
| opendevreview | yatin proposed openstack/neutron stable/xena: [DNM] Check xena nftables job issue https://review.opendev.org/c/openstack/neutron/+/902296 | 06:48 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [OVN] Fail to start Neutron server when OVN and FIP PF config is invalid https://review.opendev.org/c/openstack/neutron/+/892542 | 06:53 |
| opendevreview | yatin proposed openstack/neutron stable/xena: [DNM] Check xena nftables job issue https://review.opendev.org/c/openstack/neutron/+/902296 | 07:05 |
| opendevreview | yatin proposed openstack/neutron stable/xena: [DNM] Check xena nftables job issue https://review.opendev.org/c/openstack/neutron/+/902296 | 07:23 |
| sahid | o/ | 08:55 |
| sahid | an other question guys regarding ovs, on a host without any VMs running we can see it receiving lot of ARPs packets | 08:56 |
| sahid | and it looks like this creates high cpu consumtion from ovs-vswitchd | 08:56 |
| sahid | when i shou stat of the fdb entries I can see more than 5000 rows | 08:57 |
| sahid | looks like related to https://bugs.launchpad.net/neutron/+bug/1732067 | 09:16 |
| opendevreview | Merged openstack/neutron stable/yoga: [DHCP agent] Fetch OVN Metadata port from plugin https://review.opendev.org/c/openstack/neutron/+/901850 | 09:33 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Ensure proxy arp and ndp is configure for flat provider networks https://review.opendev.org/c/openstack/ovn-bgp-agent/+/902263 | 09:43 |
| mgoddard | lucasagomes: Hi! Do you know if ML2/OVN works with (bluefield v3) smart NICs on (ironic) bare metal? | 10:16 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Ensure proxy arp and ndp is configure for flat provider networks https://review.opendev.org/c/openstack/ovn-bgp-agent/+/902263 | 10:55 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Ensure proxy arp and ndp is configure for flat provider networks https://review.opendev.org/c/openstack/ovn-bgp-agent/+/902384 | 11:10 |
| mnasiadka | Hello - is there an option to force sync of ovn-octavia-provider LBs to OVN NB? like Neutron has a sync script? | 11:12 |
| opendevreview | Merged openstack/neutron master: Remove agent veth_mtu configuration option https://review.opendev.org/c/openstack/neutron/+/901476 | 11:27 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Ensure proxy arp and ndp is configure for flat provider networks https://review.opendev.org/c/openstack/ovn-bgp-agent/+/902384 | 11:31 |
| opendevreview | Merged openstack/neutron stable/wallaby: [DHCP agent] Fetch OVN Metadata port from plugin https://review.opendev.org/c/openstack/neutron/+/901852 | 12:27 |
| opendevreview | Merged openstack/neutron stable/wallaby: [DHCP agent] Fix route to OVN metadata port for non-isolated networks https://review.opendev.org/c/openstack/neutron/+/901781 | 12:39 |
| opendevreview | Merged openstack/neutron stable/2023.2: Fix the common/ovn functional tests https://review.opendev.org/c/openstack/neutron/+/902154 | 12:39 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Update the contributor information https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/901043 | 12:39 |
| opendevreview | Merged openstack/neutron stable/2023.2: [fullstack] Unify ``TestQoSPolicyIsDefault`` tests https://review.opendev.org/c/openstack/neutron/+/898956 | 13:11 |
| opendevreview | Merged openstack/neutron stable/2023.2: [fullstack] Unify ``TestMTUScenarios`` tests https://review.opendev.org/c/openstack/neutron/+/898955 | 13:11 |
| opendevreview | Merged openstack/neutron master: Remove a print statement from the iptables unit test https://review.opendev.org/c/openstack/neutron/+/897251 | 13:11 |
| opendevreview | Merged openstack/neutron master: [ovn]Only synchronize non dynamic segments https://review.opendev.org/c/openstack/neutron/+/891930 | 13:11 |
| opendevreview | Merged openstack/neutron master: TestSegmentHostMappingNoStore class is missing config https://review.opendev.org/c/openstack/neutron/+/897252 | 13:11 |
| opendevreview | Merged openstack/neutron master: Add python3.10 support in testing runtime https://review.opendev.org/c/openstack/neutron/+/890275 | 13:11 |
| opendevreview | Merged openstack/neutron stable/xena: docs: update default value of metadata workers for ml2/ovn https://review.opendev.org/c/openstack/neutron/+/901499 | 13:11 |
| sahid | quick auestion, why don't we have this option configured to true by default for OVS? explicitly_egress_direct | 13:21 |
| sahid | my understanding is that without this option networking equipments do not handle the packet as a normal packet | 13:26 |
| sahid | and for example it seems that ovs does not learn mac addresses | 13:27 |
| sahid | but I would be interested to understand the reason of having it defaulted to False | 13:27 |
| sahid | I guess there are side effect as-well | 13:27 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove the SubresourceTest class https://review.opendev.org/c/openstack/neutron/+/902335 | 13:38 |
| *** ykarel is now known as ykarel|away | 13:51 | |
| haleyb | #startmeeting neutron_drivers | 14:00 |
| opendevmeet | Meeting started Fri Dec 1 14:00:40 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 'neutron_drivers' | 14:00 |
| mlavalle | \o | 14:00 |
| lajoskatona | o/ | 14:00 |
| haleyb | i'm hoping ihrachys_ saw my email about his RFE | 14:01 |
| haleyb | but we don't have quorom yet, lets wait | 14:02 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2045058 | 14:02 |
| haleyb | in case anyone wants to read | 14:02 |
| obondarev | o/ | 14:02 |
| ihrachys_ | I did not see it yet but I will check | 14:04 |
| * mlavalle reading | 14:05 | |
| haleyb | ihrachys_: since it was tagged as an rfe i added it to the agenda | 14:05 |
| ihrachys_ | so long story short, neutron manually manages long RPC calls; and oslo.messaging already supports something similar (or better) so we could adopt it | 14:05 |
| ihrachys_ | (better because clients heartbeat and not just timeout on long calls) | 14:06 |
| haleyb | ihrachys_: yes, it seemed to me like a good change, especially since another project has a blueprint to follow | 14:07 |
| obondarev | can we adopt it without disabling neutron mechanism? Thus we'll have some backup mechanism and potentially we can see which is working better in our case | 14:07 |
| obondarev | and eventually disable another | 14:08 |
| lajoskatona | This is the oslo patch which introduces the option (I can't find the doc); https://review.opendev.org/c/openstack/oslo.messaging/+/546763 | 14:08 |
| haleyb | i forgot the ping... | 14:08 |
| haleyb | ping ykarel, mlavalle, mtomaska, slawek, obondarev, tobias-urdin, lajoskatona, amotoki | 14:08 |
| mlavalle | LOL | 14:08 |
| ihrachys_ | I think we can adopt step by step and it seems like a good approach | 14:08 |
| mlavalle | I'm here | 14:08 |
| haleyb | it was just a blind copy/paste :) | 14:08 |
| lajoskatona | The nova patch you referenced in RFE do this only for longer than 60sec calls, or am I misunderstood? | 14:09 |
| ihrachys_ | backoff client activates its logic on MessagingTimeout so if oslo.msg doesn't return it, it should be NOOP | 14:09 |
| ihrachys_ | lajoskatona I think so and I believe it's because they don't expect timeouts from calls that are shorter than 60s? | 14:09 |
| lajoskatona | ihrachys_: good explanation, so possible | 14:10 |
| obondarev | ihrachys_: perfect then, so no need to immediately disable neutron lib backoff | 14:10 |
| ihrachys_ | probably some optimization not to send too many heartbeats; good point and we should check with nova why they do it | 14:10 |
| ihrachys_ | I can check with dan who posted the nova patch | 14:11 |
| ihrachys_ | actually he's here dansmith | 14:11 |
| mlavalle | wow, what an honor... dansmith is here! | 14:12 |
| ihrachys_ | "keep the failure timing characteristics that our code likely expects (from history)" in code, not sure what it means :) | 14:12 |
| haleyb | was that really from 5 years ago in nova? wow | 14:13 |
| mlavalle | that;s what the commit says | 14:14 |
| ihrachys_ | yes. Dan told me about it half a year ago and I just got to post it in launchpad so... | 14:14 |
| haleyb | i would be curious if it worked as they intended, and also if it code has changed since then | 14:15 |
| lajoskatona | +1 | 14:16 |
| ihrachys_ | Dan mentioned it to me when he heard that we still manually bump timeouts in high scale envs; and said this approach works for them fine. | 14:16 |
| ihrachys_ | (and so we should adopt it) | 14:17 |
| haleyb | ihrachys_: ack | 14:17 |
| haleyb | so i think the only problem is we don't have quorom to vote, i only counted 4 of us | 14:17 |
| mlavalle | it looks very reasonable to me | 14:18 |
| mlavalle | ihrachys_: will you implement it? | 14:18 |
| haleyb | but i think it's a good idea, on the fence if it even needs a spec | 14:18 |
| ihrachys_ | mlavalle: I will | 14:18 |
| mlavalle | I don't feel we need a spec. Maybe use the first few revisions to the patch as a PoC and see how it goes | 14:19 |
| lajoskatona | agree | 14:19 |
| mlavalle | count my vote as +1, either today or next meeting | 14:22 |
| lajoskatona | I would say, let's add some more details to the RFE like how it works for nova, how to handle shorter that 60sec calls etc, and push PoCn ad if we need formal vote lets do it in a following drivers meting or even team meeting | 14:22 |
| ihrachys_ | ack | 14:22 |
| lajoskatona | +1 from me also | 14:22 |
| ihrachys_ | I will probably post patches in the next year though :) | 14:22 |
| haleyb | all sounds good, +1 from me as well, if you want to start work we can just table vote until next meeting | 14:23 |
| mlavalle | lajoskatona: good approach. this way we don't waste ihrachys_ time and he can move forward, whenever the upcoming holidays season allow him | 14:23 |
| obondarev | +1 for the RFE | 14:23 |
| lajoskatona | +1for holiday season :P | 14:24 |
| haleyb | ihrachys_: i will keep on agenda for next week if you like and will be around | 14:24 |
| mlavalle | ihrachys_: I don't think we are in a rush for this. We've waited 5 years, I think we will survive another couple of months :-) | 14:24 |
| mlavalle | and thanks for bringing this up! | 14:25 |
| mlavalle | I think it is a worthwhile optimization | 14:25 |
| haleyb | are there any other rfe's to discuss? otherwise i can let everyone go back to Review Days! :) | 14:25 |
| haleyb | i will take that as a No | 14:26 |
| ihrachys_ | o/ | 14:27 |
| haleyb | have a good weekend everyone, and thanks ihrachys_ | 14:27 |
| haleyb | #endmeeting | 14:27 |
| opendevmeet | Meeting ended Fri Dec 1 14:27:19 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2023/neutron_drivers.2023-12-01-14.00.html | 14:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2023/neutron_drivers.2023-12-01-14.00.txt | 14:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2023/neutron_drivers.2023-12-01-14.00.log.html | 14:27 |
| lajoskatona | Bye, Have a nice weekend | 14:27 |
| mlavalle | haleyb, lajoskatona, obondarev and others participating in the reviews day: https://review.opendev.org/c/openstack/neutron/+/883988 is good to review. The coverage job is punishing me for modules I didn't even touch, so I think I deserve a review | 14:29 |
| haleyb | mlavalle: ack, i'll need more coffee for that :) | 14:46 |
| haleyb | maybe i should have created an etherpad for everyone to put a list, instead of going to the large list | 14:46 |
| haleyb | https://review.opendev.org/q/project:openstack/neutron+status:open | 14:46 |
| haleyb | although that only shows around 200 open reviews in neutron | 14:47 |
| mlavalle | haleyb: so I have two questions: 1) Just review away what draws our attention for as long as possible today? | 14:55 |
| mlavalle | would it be helpful if I focus on https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353. Or do you want me to look at other things? | 14:56 |
| haleyb | mlavalle: that was my plan, looking back should have created a board for things | 14:56 |
| mlavalle | haleyb: ok, I'll look at https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353 and then review other things as time allows | 14:59 |
| haleyb | regarding the vpnaas change, and this is just my opinion, i'd rather not spend half the day looking at one change if possible, but you might need the dedicated time to get through it. i'm just herding the cats | 15:00 |
| mlavalle | haleyb: ok, then I'll just review from the Neutron queue | 15:00 |
| mlavalle | and will look at the vpnaas change some other day | 15:01 |
| haleyb | i didn't mean to pop that balloon :( | 15:01 |
| mlavalle | haleyb: I'll return to vpnaas other day soon | 15:02 |
| mlavalle | so you didn't really pop anything | 15:02 |
| lajoskatona | haleyb, mlavalle: ok, so no priority list or similar now for the review day? | 15:19 |
| haleyb | lajoskatona: that's my fault, for next week maybe i create an etherpad | 15:20 |
| haleyb | because there are changes that block other changes, like the IPv6 metadata one | 15:21 |
| haleyb | https://review.opendev.org/c/openstack/neutron/+/894026 | 15:22 |
| lajoskatona | if you (or others who need quick review) set the review priority works also for me | 15:26 |
| haleyb | lajoskatona: oh yeah we do have that field | 15:33 |
| haleyb | mlavalle: just finished quick review of OVN router flavors, coffee has kicked-in | 15:34 |
| mlavalle | haleyb: Thanks! How do you want me to deal with the coverage issue? I'm thinking create another patch adding some unit tests (since the guilty modules were not touched by me) and rebase my patch on top of it. Makes sense? | 15:37 |
| haleyb | mlavalle: i suppose. looking at the coverage report it does list neutron/services/ovn_l3/service_providers/user_defined.py at 0%, but it looked like you just moved the code? | 15:41 |
| haleyb | i almost don't want to look closely at that report as it will distract me | 15:44 |
| mlavalle | haleyb: good point. In that case I'll add unit tests for neutron/services/ovn_l3/service_providers/user_defined.py in the current patch and see if that fixes the issue. And of course I'll address your comments. Thanks | 15:46 |
| haleyb | mlavalle: thanks, and not sure all my comments are valid, thanks for working on the patch! | 15:47 |
| opendevreview | Lajos Katona proposed openstack/neutron master: FIP QoS: check policy id before blindly updating FIP https://review.opendev.org/c/openstack/neutron/+/899469 | 15:48 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: Remove usage of LBaaS constants https://review.opendev.org/c/openstack/networking-sfc/+/902427 | 15:52 |
| opendevreview | Merged openstack/networking-bgpvpn master: Add python3.10 support in testing runtime https://review.opendev.org/c/openstack/networking-bgpvpn/+/894865 | 16:10 |
| opendevreview | Eduardo Olivares proposed openstack/neutron stable/zed: [DNM] Skip StatelessSecurityGroup Tobiko tests https://review.opendev.org/c/openstack/neutron/+/902430 | 16:34 |
| opendevreview | Merged openstack/neutron-lib master: Update default for BFD/ECMP router extra attributes https://review.opendev.org/c/openstack/neutron-lib/+/893026 | 17:22 |
| opendevreview | Eduardo Olivares proposed openstack/neutron stable/zed: [DNM] Skip StatelessSecurityGroup Tobiko tests https://review.opendev.org/c/openstack/neutron/+/902430 | 17:55 |
| opendevreview | Merged openstack/neutron master: Remove ovs_integration_bridge configuration option https://review.opendev.org/c/openstack/neutron/+/901474 | 18:20 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM - testing functional test failures https://review.opendev.org/c/openstack/neutron/+/902447 | 21:30 |
| haleyb | mlavalle: thanks for all the reviews, i'm done for the day. if you're looking for an old one i just noticed https://review.opendev.org/c/openstack/neutron/+/618208 8-o | 23:07 |
| opendevreview | Merged openstack/neutron master: l3_extra_gws: Listing GW ports requires admin context https://review.opendev.org/c/openstack/neutron/+/897798 | 23:58 |
| opendevreview | Merged openstack/neutron master: l3_extra_gws: Fix KeyError when removing extra GW port https://review.opendev.org/c/openstack/neutron/+/897799 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!