| zhouhenglc | slaweq: a small patch https://review.opendev.org/c/openstack/neutron-fwaas/+/831478, when you have time, please check it :) | 02:20 |
|---|---|---|
| opendevreview | liuyulong proposed openstack/neutron master: Config option for link down/up ext gw port HA router in backup node https://review.opendev.org/c/openstack/neutron/+/834260 | 05:54 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron-specs master: OVS: multiple routed provider segments per host https://review.opendev.org/c/openstack/neutron-specs/+/823823 | 06:21 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron-specs master: OVS: multiple routed provider segments per host https://review.opendev.org/c/openstack/neutron-specs/+/823823 | 06:35 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron-specs master: OVS: multiple routed provider segments per host https://review.opendev.org/c/openstack/neutron-specs/+/823823 | 06:36 |
| opendevreview | yangjianfeng proposed openstack/neutron master: [docs] L3 router support ndp proxy https://review.opendev.org/c/openstack/neutron/+/822253 | 06:54 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/victoria: Retry logical switch associations to load balancers https://review.opendev.org/c/openstack/ovn-octavia-provider/+/833884 | 08:00 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Add FIPS enabled scenario jobs https://review.opendev.org/c/openstack/neutron/+/828143 | 08:45 |
| opendevreview | Liangyi Ma proposed openstack/neutron-fwaas master: replace tearDown with addCleanup https://review.opendev.org/c/openstack/neutron-fwaas/+/834264 | 08:49 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider master: Fix zuul templates for functional and tempest tests https://review.opendev.org/c/openstack/ovn-octavia-provider/+/833798 | 08:53 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: DNM Just testing functional and fullstack fips jobs https://review.opendev.org/c/openstack/neutron/+/834265 | 08:57 |
| opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: replace tearDown with addCleanup https://review.opendev.org/c/openstack/neutron-fwaas/+/834264 | 09:57 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider master: Fix zuul templates for functional and tempest tests https://review.opendev.org/c/openstack/ovn-octavia-provider/+/833798 | 10:21 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Decrease OS_TEST_TIMEOUT to 240 for dsvm-fullstack https://review.opendev.org/c/openstack/neutron/+/827488 | 10:25 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Retry logical switch associations to load balancers https://review.opendev.org/c/openstack/ovn-octavia-provider/+/833885 | 10:35 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Retry logical switch associations to load balancers https://review.opendev.org/c/openstack/ovn-octavia-provider/+/833882 | 10:36 |
| opendevreview | yangjianfeng proposed openstack/neutron-tempest-plugin master: [WIP] Add ndp proxy API tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/827791 | 11:08 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/yoga: [Stable only] Fix CI jobs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/834282 | 12:41 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/yoga: [Stable only] Fix CI jobs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/834282 | 12:44 |
| *** TheJulia is now known as needsleep | 12:57 | |
| *** needsleep is now known as TheJulia | 12:57 | |
| *** obondarev_ is now known as obondarev | 13:18 | |
| *** dasm|off is now known as dasm | 13:22 | |
| lajoskatona | damiandabrowski[m]: Hi, is today's drivers meeting ok for you for the GARP vs HA routers issue? | 13:25 |
| lajoskatona | trident: ---^ | 13:31 |
| damiandabrowski[m] | hey! I'm available if You need me but I didn't have enough time to prepare due to the internal incident in my company :/ | 13:31 |
| damiandabrowski[m] | so we can either discuss it today or postpone to next Friday | 13:32 |
| trident | Hey, yeah, I think it would probably make sense to actually postpone to next week. Same company and same incident has taken most of my time this week as well. Next week looks much more promising timewise for me. | 13:34 |
| *** arne_wiebalck is now known as a2ew6k | 13:36 | |
| lajoskatona | trident: ok, thanks, I keep it in the agenda then (https://wiki.openstack.org/wiki/Meetings/NeutronDrivers#Agenda ) | 13:36 |
| damiandabrowski[m] | thank You! | 13:37 |
| *** TheJulia is now known as needssleep | 13:38 | |
| slaweq | lajoskatona: I have urgent issue at work today and will probably not be able to participate actively in drivers meeting | 13:47 |
| slaweq | but I will be lurking here :) | 13:47 |
| slaweq | sorry | 13:47 |
| trident | Thanks very much lajoskatona! | 13:47 |
| lajoskatona | slaweq: thanks | 13:50 |
| lajoskatona | #startmeeting neutron_drivers | 14:00 |
| opendevmeet | Meeting started Fri Mar 18 14:00:25 2022 UTC and is due to finish in 60 minutes. The chair is lajoskatona. 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 |
| lajoskatona | o/ | 14:00 |
| mlavalle | o/ | 14:00 |
| obondarev | hi | 14:00 |
| yamamoto | hi | 14:01 |
| noonedeadpunk | o/ | 14:01 |
| amotoki | hi | 14:01 |
| haleyb | hi | 14:02 |
| lajoskatona | I think we can start | 14:03 |
| lajoskatona | The first topic from the list regarding GARP are not sent during ha router master state transition we can postpone | 14:03 |
| lajoskatona | So the first one fortoday: | 14:04 |
| lajoskatona | [QoS][OvS] implement QoS bandwidth limitation by leveraging ovs meter (#link https://bugs.launchpad.net/neutron/+bug/1964342 ) | 14:04 |
| noonedeadpunk | Are we? | 14:04 |
| mlavalle | yeah, I think that's trident's topic | 14:04 |
| noonedeadpunk | Because that's quite critical | 14:04 |
| slaweq | o/ | 14:04 |
| noonedeadpunk | Basic functionality is quite broken and this topic has not been taken for meeting for 3 weeks now | 14:04 |
| lajoskatona | noonedeadpunk: I jsut asked trident if we can discuss it today, but they had no time for testing the possible solutions this week | 14:04 |
| mlavalle | yeap | 14:05 |
| slaweq | is it about ingress or egress traffic (from vm point of view)? | 14:05 |
| slaweq | or both? | 14:05 |
| trident | noonedeadpunk: Yeah, me and damiandabrowski[m] discussed a bit and there is more testing on our side that needs to be done before we really have anything new to report on it. | 14:06 |
| noonedeadpunk | I'm not sure about egress | 14:06 |
| noonedeadpunk | Ingress for sure is broken | 14:06 |
| slaweq | https://review.opendev.org/c/openstack/neutron/+/832662 that should fix current implementation of the ingress bw limit in ml2/ovs | 14:07 |
| noonedeadpunk | I'm not neutron code expert, but can try to help out. But eventuaalyy any solutions I saw has own flaws... | 14:07 |
| lajoskatona | noonedeadpunk: could you sync with trident and than next week we can check the topic | 14:08 |
| noonedeadpunk | I don't think it will as VIP is not functional and mentioned patch not touching that part? | 14:08 |
| lajoskatona | noonedeadpunk: is that work for you? | 14:08 |
| noonedeadpunk | lajoskatona: yes, thanks! | 14:08 |
| lajoskatona | noonedeadpunk: cool | 14:08 |
| mlavalle | so talking now about QoS? | 14:09 |
| noonedeadpunk | sorry for interrupting:) I just joined to check out on the bug that was skipped without mentioning reason again :) | 14:09 |
| lajoskatona | yes I think we can jump on it | 14:09 |
| lajoskatona | noonedeadpunk: true, I forgot to mention that I asked about it trident | 14:09 |
| lajoskatona | So the coming rfe: [QoS][OvS] implement QoS bandwidth limitation by leveraging ovs meter (#link https://bugs.launchpad.net/neutron/+bug/1964342 ) | 14:10 |
| lajoskatona | this is from liuyulong | 14:10 |
| lajoskatona | If I understand correctly it is to have another QoS Bandwidth limit driver with os-ken and and ovs meter rules | 14:11 |
| slaweq | sorry, I though we are talking about this one all the time :) | 14:12 |
| mlavalle | LOL | 14:12 |
| mlavalle | yeah I figured | 14:12 |
| * slaweq is focused mostly on other meeting now | 14:12 | |
| slaweq | regarding this qos bw limit rfe - what's the point of having 2 different ways to configure it in ovs agent? I would just go with one which works better | 14:13 |
| mlavalle | agree | 14:13 |
| slaweq | so if using meters will be more accurate/efficient/better in other way - lets go with it and replace current implementation | 14:13 |
| slaweq | if it's not better, what's the point of adding it to the code? | 14:14 |
| lajoskatona | slaweq: valid point | 14:14 |
| slaweq | that's my opinion at least | 14:14 |
| mlavalle | I agree | 14:14 |
| slaweq | but tbh I would like to see ralonsoh's thoughts on that one before approving/rejecting it | 14:14 |
| mlavalle | it seems he is trying to justify this from the off loading perspective | 14:14 |
| mlavalle | to smart-nic | 14:15 |
| mlavalle | but then he states that tc can also be off-loaded | 14:15 |
| mlavalle | I bit confusing | 14:15 |
| lajoskatona | the current driver uses ovsdb | 14:15 |
| lajoskatona | sorry half finished sentence.... | 14:15 |
| lajoskatona | so that one was confusing for me also, as I don't see why offloading is better with the proposed solution | 14:16 |
| obondarev | agree, need to clarify what are the benefits | 14:17 |
| lajoskatona | ok, I add the questions to the bug, and ask liuyulong to present the asked details, is that ok? | 14:18 |
| obondarev | +§ | 14:19 |
| obondarev | +1 | 14:19 |
| mlavalle | perfect | 14:20 |
| mlavalle | +1 | 14:20 |
| amotoki | +1 | 14:20 |
| lajoskatona | ok, thanks | 14:20 |
| lajoskatona | Next topic is: | 14:20 |
| lajoskatona | DHCP agent scheduler filtering ignored when agent service restarted (#link https://bugs.launchpad.net/neutron/+bug/1964765 ) | 14:21 |
| lajoskatona | it's from BBC as I see | 14:21 |
| amotoki | honestly I am not sure this is a bug or not.... | 14:22 |
| lajoskatona | summary is to have possiblity to schedule network on a group of dhcp agents specified by some regex filtering or similar | 14:23 |
| obondarev | I think it’s a fair expectation to have consistent filtering of dhcp agents when network (auto)scheduling | 14:23 |
| lajoskatona | amotoki: your decision to mark it a srfe was really good, thanks | 14:23 |
| lajoskatona | obondarev: true, I can imagine usecases to be useful like the one in the bug report | 14:24 |
| obondarev | from the other hand changing this behaviour (skip filtering on net auto schedule) might break things for someone | 14:26 |
| obondarev | for this I think a new parameter in the Filter itself might be added - whether filter should be applied on net auto-schedule (agent restart) | 14:27 |
| amotoki | is it the right thing to skip filtering? I think it is better to apply consistent filtering if there is some inconsistency. | 14:27 |
| obondarev | this will let us avoid new config option | 14:28 |
| lajoskatona | and what about a new scheduler and if somebody needs this with possible traps, they select it in config? | 14:29 |
| obondarev | new scheduler or new filter? | 14:29 |
| lajoskatona | I thought new scheduler, as that is configurable | 14:30 |
| amotoki | I think two topics are mixed. the one is about a new filtering/scheduler and the other point is that filtering is skipped when agent restart. | 14:31 |
| amotoki | i am not sure the second point only affects the new filtering mentioned in the bug. | 14:31 |
| obondarev | afaiu only segment related filtering is currently done on agent restart | 14:34 |
| obondarev | all others are skipped, both old and new ones | 14:34 |
| lajoskatona | ok so the LP can be split to an RFE and an actual bug? | 14:35 |
| obondarev | I think the bug is just to decide if we should enable filters on agent restart | 14:37 |
| amotoki | obondarev: agree | 14:38 |
| obondarev | and my point is - let's enable filtering on agent restart for those filters that explicitly say "apply me on restart" - then we can decide if existing filters should add this flag or not | 14:38 |
| lajoskatona | +1 | 14:39 |
| amotoki | +1 | 14:40 |
| amotoki | it looks like that the bug author does not request to add a new filter in neutron. the author just uses their new filter as an example to explain what happens. | 14:40 |
| amotoki | so I totally agree with obondarev's point | 14:40 |
| lajoskatona | amotoki: possible, I concerned more on the filter they created | 14:41 |
| obondarev | not sure this new filter is even going to be upstreamed | 14:42 |
| *** a2ew6k is now known as arne_wiebalck | 14:43 | |
| obondarev | I mean if author has such plans | 14:43 |
| lajoskatona | ok, than the summary of this LP is: allow new flag for filters to execute them on agent restart, and set this to False for already existing filters | 14:43 |
| lajoskatona | obondarev: possible | 14:44 |
| obondarev | or to accept it as False if filter doesn't have this parameter | 14:44 |
| fnordahl | lajoskatona: (re question about smartnic_dpu docs, screnshot and possible marketing page reference) apologies for the tardy response, that is an excellent idea. Are you asking for expansion of the existing doc page or some oob screenshot of stuff in action? | 14:44 |
| lajoskatona | obondarev: +1 | 14:45 |
| lajoskatona | fnordahl: just a sec we are at the end of the meeting | 14:46 |
| lajoskatona | ok regarding DHCP scheduling/filtering: is it ok this way for everybody? | 14:46 |
| obondarev | +1 | 14:46 |
| amotoki | I am still unclear a bit. I think AZ schduler does the similar thing so it might be affected, while network AZ and network segments are used together. | 14:47 |
| amotoki | s/are not used together/ | 14:48 |
| obondarev | so do you suggest to enable it for all filters on agent restart? | 14:51 |
| amotoki | what is "this way"? is it to accept filtering in dhcp RPC as False? | 14:51 |
| amotoki | if so, I am okay with it. | 14:52 |
| amotoki | If the current behavior affects the existing scheduler, we can fix it separately. | 14:52 |
| lajoskatona | amotoki: for this way I meant this : "allow new flag for filters to execute them on agent restart, and set this to False for already existing filters" | 14:53 |
| obondarev | IOW nothing changes for current filters, new filters can specify if they should filter on agent restart | 14:53 |
| lajoskatona | yes that's my understanding also | 14:53 |
| amotoki | lajoskatona: obondarev: thanks for the clarification. I got it. | 14:53 |
| lajoskatona | amotoki: :-) this is a tricky LP bug, but really good discussion | 14:54 |
| lajoskatona | ok, could you vote on in please if it is ok for you? | 14:55 |
| mlavalle | +1 | 14:55 |
| amotoki | +1 | 14:55 |
| lajoskatona | +1 | 14:55 |
| obondarev | +1 | 14:55 |
| yamamoto | +1 | 14:55 |
| lajoskatona | ok, thanks | 14:56 |
| lajoskatona | if there is nothing more to discuss, we can close the meeting | 14:56 |
| *** arne_wiebalck is now known as a2ew6k | 14:57 | |
| lajoskatona | #endmeeting | 14:57 |
| opendevmeet | Meeting ended Fri Mar 18 14:57:29 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:57 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-03-18-14.00.html | 14:57 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-03-18-14.00.txt | 14:57 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-03-18-14.00.log.html | 14:57 |
| lajoskatona | o/ | 14:57 |
| mlavalle | o/ | 14:57 |
| obondarev | o/ | 14:57 |
| amotoki | have a good weekend o/ | 14:57 |
| yamamoto | good night | 14:57 |
| lajoskatona | thanks, have a good weekend :-) | 14:58 |
| fnordahl | lajoskatona: sorry about that, I just jumped on the highlight not realizing a meeting was in session, my bad :) | 14:58 |
| lajoskatona | fnordahl: no problem | 14:58 |
| *** a2ew6k is now known as arne_wiebalck | 14:59 | |
| lajoskatona | fnordahl: the question was coming from diabl_rojo for Yoga marketing page, and they need more screenshots and such things | 14:59 |
| lajoskatona | fnordahl: I think the doc pages are ok for them, at least yesterday when I asked there was no complaints :-) | 15:00 |
| fnordahl | lajoskatona: we'd be happy to provide them, we have a talk on the schedule for the Berlin summit which will include a demo and we can take screenshots from that preparation. What are the timelines and where should we send them? | 15:01 |
| lajoskatona | fnordahl: cool, that sounds really good, and something that is more digestable for marketing people :-) | 15:02 |
| lajoskatona | I think today | 15:02 |
| lajoskatona | let me check the mail | 15:02 |
| *** arne_wiebalck is now known as a2ew6k | 15:02 | |
| lajoskatona | fnordahl: it's today, but no info which timezone today is closing | 15:03 |
| *** a2ew6k is now known as arne_wiebalck | 15:05 | |
| fnordahl | lajoskatona: right, I'll see what I can do, is this e-mail on the list i.e. do you have a link to it, or would it be possible for you to forward it to me? | 15:06 |
| *** arne_wiebalck is now known as a2ew6k | 15:07 | |
| *** a2ew6k is now known as arne_wiebalck | 15:08 | |
| lajoskatona | fnordahl: I forwarded it to frode.nordahl@canonical.com | 15:29 |
| fnordahl | lajoskatona: excellent, thanks! | 15:29 |
| opendevreview | Balazs Gibizer proposed openstack/neutron-lib master: Add port-mac-address-override shim extension https://review.opendev.org/c/openstack/neutron-lib/+/831935 | 15:43 |
| opendevreview | Balazs Gibizer proposed openstack/neutron master: Update port MAC from binding profile for PFs https://review.opendev.org/c/openstack/neutron/+/829247 | 15:45 |
| gibi_pto | lajoskatona, rubasov, ralonsoh: ^^ update the bugfix to overwrite the data in the DB and proposed a shim API extension for it | 15:46 |
| lajoskatona | gibi_pto: thanks | 15:55 |
| opendevreview | Balazs Gibizer proposed openstack/neutron-lib master: Add port-mac-address-override shim extension https://review.opendev.org/c/openstack/neutron-lib/+/831935 | 16:05 |
| opendevreview | Balazs Gibizer proposed openstack/neutron master: Update port MAC from binding profile for PFs https://review.opendev.org/c/openstack/neutron/+/829247 | 16:06 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Code cleaning: make RPC method signatures more meaningful https://review.opendev.org/c/openstack/tap-as-a-service/+/833432 | 16:15 |
| opendevreview | Balazs Gibizer proposed openstack/neutron-lib master: Add port-mac-address-override shim extension https://review.opendev.org/c/openstack/neutron-lib/+/831935 | 16:24 |
| opendevreview | Balazs Gibizer proposed openstack/neutron master: Update port MAC from binding profile for PFs https://review.opendev.org/c/openstack/neutron/+/829247 | 16:26 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider master: Avoid loadbalancer stuck in PENDING_X if delete_vip_port fails https://review.opendev.org/c/openstack/ovn-octavia-provider/+/834335 | 17:05 |
| opendevreview | Miguel Lavalle proposed openstack/ovn-octavia-provider master: Remove incorrect character in f-string https://review.opendev.org/c/openstack/ovn-octavia-provider/+/834345 | 18:09 |
| *** jlibosva is now known as Guest2502 | 18:23 | |
| hjensas | I have a question regarding black magic retrying in neutron ML2 drivers. If the driver knows it failed in a non-recoverable way, how can the driver raise an exception that won't be retried? | 20:40 |
| hjensas | p | 20:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!