14:00:01 <ralonsoh> #startmeeting neutron_drivers
14:00:01 <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:01 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:01 <opendevmeet> The meeting name has been set to 'neutron_drivers'
14:00:05 <ralonsoh> hello
14:00:08 <ykarel> o/
14:00:18 <obondarev> hi, sorry, can't participate today
14:00:25 <ralonsoh> np
14:01:18 <slaweq> o/
14:01:41 <ralonsoh> ok, we are 4 core drivers here
14:01:45 <ralonsoh> lajoskatona, will join later
14:01:54 <ralonsoh> let's start with the agenda
14:02:06 <ralonsoh> first topic
14:02:11 <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:16 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/2028660
14:02:56 <ralonsoh> is Liu Xie here?
14:03:49 <ralonsoh> ok, are you willing to propose this RFE? sorry but I don't have the background for it
14:04: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:07 <opendevreview> Merged openstack/neutron stable/zed: [OVN] Match LSP_TYPE_VIRTUAL in PortBindingUpdateVirtualPortsEvent  https://review.opendev.org/c/openstack/neutron/+/896805
14:04:12 <lajoskatona> o/
14:04:54 <ralonsoh> ok, I'll update the launchpad bug to request the author of this RFE to propose it during this meeting, next week
14:05:08 <lajoskatona> I suppose liushy is from pacific, so perhaps we have to warn that there will be a discussion
14:05:25 <lajoskatona> I should have sone it but this week was a little busy, I can send him/her a mail
14:05:33 <ralonsoh> lajoskatona, ok, I'll update the LP with the time and dates
14:05:40 <ralonsoh> and I'll ask him for feedback
14:05:50 <lajoskatona> cool
14:05:51 <mlavalle> or we can discuss based on what we know based on the LP and ask questions teher
14:06:05 <mlavalle> there
14:06:08 <slaweq> wait a sec with this RFE
14:06:15 <ralonsoh> mlavalle, ok, but as I said I have no background for this
14:06:17 <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:23 <ralonsoh> so if anyone is willing to do it, perfect
14:06:29 <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:38 <slaweq> isn't it what this rfe was proposing?
14:06:49 <mlavalle> yes, it looks like it
14:07:10 <lajoskatona> even cooler, it is already finished than :-)
14:07:21 <slaweq> it seems so
14:07:23 <mlavalle> then maybe is a bug
14:07:28 <slaweq> but I don't know about implementation
14:07:37 <ykarel> it's expecting list "source_port": ["22","23","30:80"]
14:07:59 <slaweq> but instead of list You can create couple of rules
14:08:12 <slaweq> so I'm not sure if it's good idea to do list there
14:08:17 <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:26 <slaweq> mlavalle++
14:08:42 <lajoskatona> +1
14:08:53 <ykarel> +1
14:09:32 <ralonsoh> ok, I'll update the LP with the comments made here
14:09:48 <ralonsoh> so, if I'm not wrong, the suggestion is to create several rules
14:09:51 <ralonsoh> right?
14:10:02 <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:22 <ralonsoh> mlavalle, the API does not accept any list, accepts a single value or a range
14:10:29 <ralonsoh> so the API now is correct
14:10:39 <mlavalle> ok
14:11:14 <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:14 <mlavalle> then let's explore whether the current API can satisfy the expressed need
14:11:48 <ralonsoh> but I'm not going to approve yet the LP until we have some feeback from the author
14:11:55 <mlavalle> yeap
14:12:03 <mlavalle> that's exactly what we are saying
14:12:23 <ralonsoh> ok, any other comment?
14:12:47 <ralonsoh> perfect, thanks!
14:12:49 <ralonsoh> next one
14:12:58 <ralonsoh> lucasagomes, [OVN] Allow scheduling external ports on non-gateway nodes
14:13:03 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/2037294
14:13:06 <ralonsoh> lucasagomes, please
14:14:00 <ralonsoh> ok, let me introduce this RFE
14:14:22 <ralonsoh> right now the OVN L3 scheduler can schedule the GW ports only on those marked as GW chassis
14:14:29 <ralonsoh> that is an option in "ovn-cms-options"
14:14:42 <ralonsoh> there are now no exceptions to this
14:15:06 <ralonsoh> however, OpenShift would require to schedule these GW ports on non-GW nodes
14:15:24 <ralonsoh> so a new config option, defining these chassis, is proposed
14:15:35 <ralonsoh> and of course the OVN L3 scheduler should be aware of this
14:16:12 <ralonsoh> IMO this is a RFE, because of the impact in the L3 scheduler
14:16:41 <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:51 <ralonsoh> 1) new scheduler or 2) modify the existing ones
14:17:27 <lajoskatona> Originally I thought a lightweiht RFE, but as you speak about it perhaps a spec would be good for it
14:17:59 <ralonsoh> I would prefer first to discuss in a small spec what changes are going to be made
14:18:12 <lajoskatona> +1, thanks for it
14:18:17 <ralonsoh> there are currently too many ongoing changes in the L3 scheduler
14:18:19 <mlavalle> and also get a litle more detail on the use case
14:19:21 <ralonsoh> any other opinion?
14:19:40 <slaweq> small spec sounds good for me
14:19:46 <mlavalle> +1
14:20:08 <lajoskatona> +1
14:20:11 <ykarel> +1
14:20:23 <ralonsoh> perfect then, I'll update the LP bug with this info
14:20:43 <ralonsoh> I have nothing else in the agenda, do you want to add something?
14:21:09 <mlavalle> just that next Friday I won't attend the meeting, because I'll be off on PTO \o/
14:21:15 <ralonsoh> enjoy
14:21:40 <ralonsoh> thank you very much for attending, this is most probably my last drivers meeting as chair. Have a nice weekend!
14:21:52 <ykarel> o/
14:21:56 <ralonsoh> #endmeeting