| opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 00:45 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests https://review.opendev.org/c/openstack/neutron/+/896839 | 01:16 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test os-vif revert https://review.opendev.org/c/openstack/neutron/+/896504 | 06:44 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test os-vif revert https://review.opendev.org/c/openstack/neutron/+/896504 | 08:56 |
| opendevreview | Merged openstack/neutron stable/xena: Revert "[OVN][Trunk] Add port binding info on subport when parent is bound" https://review.opendev.org/c/openstack/neutron/+/894794 | 08:57 |
| ralonsoh | bcafarel, slaweq hello! If you have 1 min: https://review.opendev.org/q/Ide3204748274cbab9731310c2e2bd8bd8c9a53ff | 09:01 |
| ralonsoh | thanks in advance | 09:01 |
| ralonsoh | lajoskatona, hi! if you have 10 secs: https://review.opendev.org/c/openstack/neutron-vpnaas/+/896787. pep8 is now passing with this newer version (same a Neutron one) | 09:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: py311: Add py311 job and sqlalchemy-main job to weekly https://review.opendev.org/c/openstack/neutron-vpnaas/+/896744 | 09:04 |
| lajoskatona | ralonsoh: ack | 09:17 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Enhanced external port scheduling https://review.opendev.org/c/openstack/neutron/+/894767 | 09:18 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Enhanced external port scheduling https://review.opendev.org/c/openstack/neutron/+/894767 | 09:22 |
| slaweq | ralonsoh done | 09:53 |
| ralonsoh | thanks# | 09:53 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: py311: Change weekly py310 to py311 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/896740 | 10:19 |
| opendevreview | Merged openstack/neutron-vpnaas master: Bump pylint version to 2.17.4 https://review.opendev.org/c/openstack/neutron-vpnaas/+/896787 | 10:53 |
| opendevreview | Merged openstack/neutron-vpnaas master: py311: Add py311 job and sqlalchemy-main job to weekly https://review.opendev.org/c/openstack/neutron-vpnaas/+/896744 | 10:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/896926 | 11:08 |
| ralonsoh | slaweq, lucasagomes ^^ sorry for that, I found this error today | 11:08 |
| ralonsoh | please check this patch asap, then I need to backport it up to Wallaby | 11:08 |
| ralonsoh | because of https://review.opendev.org/q/I3b7c5d73d2b0d20fb06527ade30af8939b249d75 | 11:09 |
| ralonsoh | (my bad, sorry) | 11:09 |
| * bcafarel notes for upcoming backports | 11:35 | |
| lucasagomes | ralonsoh, checking it now | 12:13 |
| lajoskatona | ralonsoh: Hi, regarding the drivers meeting, I will be late (max 10 minutes) | 12:42 |
| ralonsoh | np | 12:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/896926 | 12:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [OVN] Set the Neutron port status based on "lsp.up" and "lsp.enabled" https://review.opendev.org/c/openstack/neutron/+/896939 | 13:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add a new extension "security-groups-rules-belongs-to-default-sg" https://review.opendev.org/c/openstack/neutron/+/883907 | 13:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove the publish patch in SG rule BEFORE_DELETE and BEFORE_CREATE https://review.opendev.org/c/openstack/neutron/+/896273 | 13:40 |
| ralonsoh | Ping list: ykarel, mlavalle, mtomaska, slawek, obondarev, tobias-urdin, lajoskatona, amotoki | 13:59 |
| mtomaska | O/ | 13:59 |
| mlavalle | o/ | 13:59 |
| ralonsoh | #startmeeting neutron_drivers | 14:00 |
| opendevmeet | Meeting started Fri Sep 29 14:00:01 2023 UTC and is due to finish in 60 minutes. The chair is ralonsoh. 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 |
| ralonsoh | hello | 14:00 |
| ykarel | o/ | 14:00 |
| obondarev | hi, sorry, can't participate today | 14:00 |
| ralonsoh | np | 14:00 |
| slaweq | o/ | 14:01 |
| ralonsoh | ok, we are 4 core drivers here | 14:01 |
| ralonsoh | lajoskatona, will join later | 14:01 |
| ralonsoh | let's start with the agenda | 14:01 |
| ralonsoh | first topic | 14:02 |
| ralonsoh | Liu Xie liushy): [fwaas][rfe] support list type of port_range for firewall rule (I can't find this in the meeting logs) | 14:02 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2028660 | 14:02 |
| ralonsoh | is Liu Xie here? | 14:02 |
| ralonsoh | ok, are you willing to propose this RFE? sorry but I don't have the background for it | 14:03 |
| opendevreview | Merged openstack/neutron stable/2023.1: [OVN] Match LSP_TYPE_VIRTUAL in PortBindingUpdateVirtualPortsEvent https://review.opendev.org/c/openstack/neutron/+/896804 | 14:04 |
| opendevreview | Merged openstack/neutron stable/zed: [OVN] Match LSP_TYPE_VIRTUAL in PortBindingUpdateVirtualPortsEvent https://review.opendev.org/c/openstack/neutron/+/896805 | 14:04 |
| lajoskatona | o/ | 14:04 |
| ralonsoh | ok, I'll update the launchpad bug to request the author of this RFE to propose it during this meeting, next week | 14:04 |
| lajoskatona | I suppose liushy is from pacific, so perhaps we have to warn that there will be a discussion | 14:05 |
| lajoskatona | I should have sone it but this week was a little busy, I can send him/her a mail | 14:05 |
| ralonsoh | lajoskatona, ok, I'll update the LP with the time and dates | 14:05 |
| ralonsoh | and I'll ask him for feedback | 14:05 |
| lajoskatona | cool | 14:05 |
| mlavalle | or we can discuss based on what we know based on the LP and ask questions teher | 14:05 |
| mlavalle | there | 14:06 |
| slaweq | wait a sec with this RFE | 14:06 |
| ralonsoh | mlavalle, ok, but as I said I have no background for this | 14:06 |
| slaweq | I'm checking now our api-ref https://docs.openstack.org/api-ref/network/v2/index.html?expanded=create-firewall-rule-detail#fwaas-v2-0-current-fwaas-firewall-groups-firewall-policies-firewall-rules | 14:06 |
| ralonsoh | so if anyone is willing to do it, perfect | 14:06 |
| slaweq | and it says: "The source port or port range for the firewall rule. A valid value is a port number, as an integer, or a port range, in the format of a : separated range. For a port range, include both ends of the range. For example, 80:90." | 14:06 |
| slaweq | isn't it what this rfe was proposing? | 14:06 |
| mlavalle | yes, it looks like it | 14:06 |
| lajoskatona | even cooler, it is already finished than :-) | 14:07 |
| slaweq | it seems so | 14:07 |
| mlavalle | then maybe is a bug | 14:07 |
| slaweq | but I don't know about implementation | 14:07 |
| ykarel | it's expecting list "source_port": ["22","23","30:80"] | 14:07 |
| slaweq | but instead of list You can create couple of rules | 14:07 |
| slaweq | so I'm not sure if it's good idea to do list there | 14:08 |
| mlavalle | we can add a comment to the LP pointing this out to submitter and ask what is being observed in the implementation | 14:08 |
| slaweq | mlavalle++ | 14:08 |
| lajoskatona | +1 | 14:08 |
| ykarel | +1 | 14:08 |
| ralonsoh | ok, I'll update the LP with the comments made here | 14:09 |
| ralonsoh | so, if I'm not wrong, the suggestion is to create several rules | 14:09 |
| ralonsoh | right? | 14:09 |
| mlavalle | in any case, it doesn't seem to warrant a new API or RFE. It seems more like a bug where the implementation doesn't meet what the API specifies | 14:10 |
| ralonsoh | mlavalle, the API does not accept any list, accepts a single value or a range | 14:10 |
| ralonsoh | so the API now is correct | 14:10 |
| mlavalle | ok | 14:10 |
| ralonsoh | ok, I'll update the LP with your suggestions and, of course, if Liu wants to improve the API, he can propose it | 14:11 |
| mlavalle | then let's explore whether the current API can satisfy the expressed need | 14:11 |
| ralonsoh | but I'm not going to approve yet the LP until we have some feeback from the author | 14:11 |
| mlavalle | yeap | 14:11 |
| mlavalle | that's exactly what we are saying | 14:12 |
| ralonsoh | ok, any other comment? | 14:12 |
| ralonsoh | perfect, thanks! | 14:12 |
| ralonsoh | next one | 14:12 |
| ralonsoh | lucasagomes, [OVN] Allow scheduling external ports on non-gateway nodes | 14:12 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2037294 | 14:13 |
| ralonsoh | lucasagomes, please | 14:13 |
| ralonsoh | ok, let me introduce this RFE | 14:14 |
| ralonsoh | right now the OVN L3 scheduler can schedule the GW ports only on those marked as GW chassis | 14:14 |
| ralonsoh | that is an option in "ovn-cms-options" | 14:14 |
| ralonsoh | there are now no exceptions to this | 14:14 |
| ralonsoh | however, OpenShift would require to schedule these GW ports on non-GW nodes | 14:15 |
| ralonsoh | so a new config option, defining these chassis, is proposed | 14:15 |
| ralonsoh | and of course the OVN L3 scheduler should be aware of this | 14:15 |
| ralonsoh | IMO this is a RFE, because of the impact in the L3 scheduler | 14:16 |
| ralonsoh | what I don't know, and I would like to discuss in advance (maybe in a spec or a patch) is how to implement that in the L3 scheduler | 14:16 |
| ralonsoh | 1) new scheduler or 2) modify the existing ones | 14:16 |
| lajoskatona | Originally I thought a lightweiht RFE, but as you speak about it perhaps a spec would be good for it | 14:17 |
| ralonsoh | I would prefer first to discuss in a small spec what changes are going to be made | 14:17 |
| lajoskatona | +1, thanks for it | 14:18 |
| ralonsoh | there are currently too many ongoing changes in the L3 scheduler | 14:18 |
| mlavalle | and also get a litle more detail on the use case | 14:18 |
| ralonsoh | any other opinion? | 14:19 |
| slaweq | small spec sounds good for me | 14:19 |
| mlavalle | +1 | 14:19 |
| lajoskatona | +1 | 14:20 |
| ykarel | +1 | 14:20 |
| ralonsoh | perfect then, I'll update the LP bug with this info | 14:20 |
| ralonsoh | I have nothing else in the agenda, do you want to add something? | 14:20 |
| mlavalle | just that next Friday I won't attend the meeting, because I'll be off on PTO \o/ | 14:21 |
| ralonsoh | enjoy | 14:21 |
| ralonsoh | thank you very much for attending, this is most probably my last drivers meeting as chair. Have a nice weekend! | 14:21 |
| ykarel | o/ | 14:21 |
| ralonsoh | #endmeeting | 14:21 |
| opendevmeet | Meeting ended Fri Sep 29 14:21:56 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:21 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2023/neutron_drivers.2023-09-29-14.00.html | 14:21 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2023/neutron_drivers.2023-09-29-14.00.txt | 14:21 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2023/neutron_drivers.2023-09-29-14.00.log.html | 14:21 |
| slaweq | thx ralonsoh++ | 14:21 |
| lajoskatona | o/ | 14:21 |
| ralonsoh | see you! | 14:22 |
| slaweq | and see You all | 14:22 |
| mlavalle | thanks for your great leadership ralonsoh! Great job! | 14:22 |
| ralonsoh | thank you | 14:22 |
| mlavalle | cangrats on a job very well done! | 14:22 |
| lajoskatona | +1, thank you | 14:22 |
| lucasagomes | ralonsoh, apologies I didn't see the ping on time | 14:23 |
| ralonsoh | lucasagomes, no problem, I'll update the LP now | 14:23 |
| ralonsoh | a spec will be required, to know how this feature will affect the ovn l3 scheduler | 14:23 |
| ralonsoh | and backwards compatibility | 14:23 |
| lucasagomes | ralonsoh, but you did a good job describing it. There won't be any change on L3 scheduler since this is about external ports btw | 14:24 |
| lucasagomes | ralonsoh, so the gw ports will still be scheduled as always on the nodes marked as enable-chassis-as-gw | 14:24 |
| lucasagomes | what will change however is where the external ports will be scheduled on if the new config is present | 14:24 |
| ralonsoh | ok, that was my mistake then, to think aboyt the l3 scheduler | 14:25 |
| lucasagomes | if the new config is not present it will work just as-is today | 14:25 |
| lucasagomes | so it's 100% backward compatible | 14:25 |
| ralonsoh | but this is only about external ports | 14:25 |
| lucasagomes | yes only external ports | 14:25 |
| ralonsoh | but the external ports are scheduled by OVN | 14:25 |
| ralonsoh | using ha_chassis | 14:25 |
| lucasagomes | ralonsoh, yes | 14:25 |
| ralonsoh | right? | 14:25 |
| ralonsoh | so this must be a core OVN feature | 14:25 |
| lucasagomes | and right now we have a HA chassis group per network | 14:25 |
| lucasagomes | so with my change, we will have a HA Chassis group per external port (limited to 3 members each) | 14:26 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Tap Mirror OVS driver https://review.opendev.org/c/openstack/tap-as-a-service/+/893086 | 14:26 |
| lucasagomes | ralonsoh, not really, we just need to make sure that we create the ha chassis group with the chassis that are eligible to host those external ports | 14:26 |
| lucasagomes | EDPM compute nodes for OpenShift | 14:26 |
| lucasagomes | ralonsoh, then these ports will land on one of the chassis in the group | 14:27 |
| ralonsoh | ok, maybe the scope of this feature is smaller than I though | 14:27 |
| ralonsoh | so the spec won't be needed | 14:27 |
| lucasagomes | ralonsoh, yeah, I don't think it's that big | 14:27 |
| ralonsoh | ok, let me add this topic again in the next meeting | 14:27 |
| ralonsoh | to remove the request of the spec | 14:27 |
| ralonsoh | as I said, I though L3 scheduler was involved | 14:28 |
| lucasagomes | yeah no | 14:28 |
| lucasagomes | I will update the LP to make sure I say that L3 is not involved | 14:28 |
| ralonsoh | perfect | 14:28 |
| lucasagomes | ralonsoh++ thanks much! Sorry for not seeing the ping on time | 14:29 |
| lucasagomes | Updated the LP | 14:30 |
| ralonsoh | haleyb, hey! | 14:53 |
| ralonsoh | the TC has created the caracal etherpad links | 14:53 |
| ralonsoh | and the link is different to the one created | 14:53 |
| ralonsoh | https://etherpad.opendev.org/p/oct2023-ptg-neutron | 14:53 |
| ralonsoh | I've updated it in the Neutron agenda | 14:57 |
| opendevreview | Merged openstack/neutron stable/2023.2: [OVN] Match LSP_TYPE_VIRTUAL in PortBindingUpdateVirtualPortsEvent https://review.opendev.org/c/openstack/neutron/+/896803 | 15:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Forbid the C/D of SG rules in default SG to non-admin users https://review.opendev.org/c/openstack/neutron/+/896222 | 15:47 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: ovn: Spawn metadata resources on additional chassis event https://review.opendev.org/c/openstack/neutron/+/895402 | 17:00 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: docs: Fix releasenote building https://review.opendev.org/c/openstack/tap-as-a-service/+/896956 | 17:19 |
| *** haleyb is now known as haleyb_out | 17:19 | |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: ovn: Spawn metadata resources on additional chassis event https://review.opendev.org/c/openstack/neutron/+/895402 | 17:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!