15:00:20 <ralonsoh> #startmeeting neutron_qos
15:00:21 <openstack> Meeting started Tue Jan 28 15:00:20 2020 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:22 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:25 <openstack> The meeting name has been set to 'neutron_qos'
15:00:42 <slaweq> hi ralonsoh :)
15:00:47 <ralonsoh> hello!
15:00:58 <ralonsoh> don't worry, the agenda is almost empty today
15:01:10 <ralonsoh> David is not going to attend, I'll update the RFE status
15:01:35 <ralonsoh> let's start then
15:01:41 <ralonsoh> #topic RFEs
15:01:46 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1476527
15:01:46 <openstack> Launchpad bug 1476527 in neutron "[RFE] Add common classifier resource" [Wishlist,Triaged] - Assigned to Igor D.C. (igordcard)
15:01:58 <ralonsoh> the spec is merged
15:02:14 <slaweq> and there is also one small follow up for this spec, right?
15:02:14 <ralonsoh> but there is a patch to amend it
15:02:15 <ralonsoh> https://review.opendev.org/#/c/703431/
15:02:15 <patchbot> patch 703431 - neutron-specs - Missing caveat in NC DSCP spec - 2 patch sets
15:02:20 <slaweq> right :)
15:02:23 <ralonsoh> exactly
15:02:32 <ralonsoh> IMO, it's ok now
15:02:37 <ralonsoh> I'll +2 it
15:02:48 <slaweq> thx
15:02:50 <ralonsoh> David is going to push the last patch for the n-lib patch
15:03:08 <lajoskatona> o/
15:03:09 <ralonsoh> #link https://review.opendev.org/#/c/670049/
15:03:09 <patchbot> patch 670049 - neutron-lib - Neutron-Classifier service plugin - 9 patch sets
15:03:13 <ralonsoh> lajoskatona, hi!
15:03:27 <ralonsoh> there are some errors in the CI but nothing serious
15:03:31 <lajoskatona> Just for info: I started to write some tempest API tests for it
15:03:37 <ralonsoh> IMO, the n-lib is almost ready
15:03:47 <ralonsoh> lajoskatona, the classifier?
15:03:54 <lajoskatona> ralonsoh: yes
15:04:10 <ralonsoh> lajoskatona, that's fantastic! please, add this info to the LP
15:04:18 <ralonsoh> or use the bug number
15:04:20 <lajoskatona> as I see there's no tempest for it in the original repo, and I thought that can be useful
15:04:30 <lajoskatona> ralonsoh: will do
15:04:39 <ralonsoh> for sure that will be very helpful
15:04:40 <slaweq> lajoskatona++ thx a lot :)
15:05:18 <ralonsoh> and I have no more updates for this RFE
15:05:29 <lajoskatona> your welcome
15:05:58 <ralonsoh> let's see if, in 2 weeks, we have the neutron patch ready
15:06:14 <slaweq> ralonsoh: I hope so :)
15:06:35 <ralonsoh> ok, next one
15:06:41 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1851362
15:06:41 <openstack> Launchpad bug 1851362 in neutron "[RFE] ports do not inherit their associated network's policy" [Low,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:06:47 <ralonsoh> last patch is in the gate
15:06:55 <ralonsoh> #link https://review.opendev.org/#/c/693244/
15:06:56 <patchbot> patch 693244 - neutron - Add "qos_network_policy_id" to port definition - 4 patch sets
15:06:58 <ralonsoh> 7 rechecks
15:07:03 <ralonsoh> but this is the last one
15:07:11 <ralonsoh> I'll remove this RFE from the list next meeting
15:07:15 <ralonsoh> thanks for the reviews!
15:07:34 <ralonsoh> btw, I have some requests to backport this feature
15:07:41 <ralonsoh> this is not possible: there is an API change
15:07:58 <ralonsoh> and because this is not a bug, only a RFE, this is not going to be possible
15:08:09 <slaweq> yes
15:08:42 <ralonsoh> ok, next one
15:08:46 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1843165
15:08:46 <openstack> Launchpad bug 1843165 in neutron "RFE: Adding support for direct ports with qos in ovs" [Wishlist,Fix released] - Assigned to waleed mousa (waleedm)
15:09:03 <ralonsoh> #link https://review.opendev.org/#/c/611605/
15:09:04 <patchbot> patch 611605 - neutron - Add support for direct ports with QoS in OVS (MERGED) - 22 patch sets
15:09:07 <ralonsoh> merged last week
15:09:19 <ralonsoh> thanks to the smart nics QoS support
15:09:37 <ralonsoh> the patch "only" enables to use those ports with the QoS extension
15:09:44 <ralonsoh> of course, we have sanity checks for this
15:09:57 <ralonsoh> I'll remove this RFE from the list next week
15:10:09 <ralonsoh> and that's all I have in the RFE section
15:10:14 <ralonsoh> something else?
15:10:27 <slaweq> nope
15:10:42 <ralonsoh> let's move on
15:10:43 <ralonsoh> #topic Bugs
15:10:57 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1845176
15:10:57 <openstack> Launchpad bug 1845176 in neutron "Removing of QoS queue in neutron-ovs-agent fails due to existing references" [Medium,Fix released] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:11:08 <ralonsoh> #link https://review.opendev.org/#/c/687922/
15:11:09 <patchbot> patch 687922 - neutron - Add OVS QoS driver cache for minimum bandwidth rules (MERGED) - 15 patch sets
15:11:14 <ralonsoh> the patch was merged last week
15:11:32 <ralonsoh> this patch, I think so, will reduce the number of errors in the CI
15:11:40 <ralonsoh> related to fullstack QoS tests
15:11:46 <slaweq> fingers crossed :)
15:11:53 <ralonsoh> (I don't remember now the LP bug for this)
15:12:00 <ralonsoh> let's find it during the next meeting
15:12:14 <ralonsoh> I'll remove this bug from the list for the next meeting
15:12:34 <ralonsoh> next one
15:12:38 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1853171
15:12:38 <openstack> Launchpad bug 1853171 in neutron "Deprecate and remove any "ofctl" code in Neutron and related projects " [Medium,In progress] - Assigned to David Shaughnessy (david-shaughnessy)
15:13:04 <ralonsoh> as commented, this is going to be a long life bug
15:13:21 <ralonsoh> and maybe we won't be able to remove ofctl from Neutron
15:13:38 <ralonsoh> there are some parts, like the firewall, using it
15:13:45 <lajoskatona> ralonsoh: I started to work on bagpipe, but I have slow progress regarding this ofctl one
15:14:06 <ralonsoh> lajoskatona, yes, and bagpipe
15:14:06 <slaweq> so it's "cross project" effort, right?
15:14:11 <ralonsoh> it is
15:14:39 <lajoskatona> yeah, as I remember 3-4 project use neutron ofctl methods
15:14:39 <ralonsoh> but also there are some sections in the firewall I don't know how to implement using native implementation
15:14:57 <ralonsoh> so it's a bit more complicated as I expected
15:15:14 <ralonsoh> the main think is: no new development should use it
15:15:24 <ralonsoh> thing*
15:15:34 <slaweq> maybe we can ask otherwiseguy for help with those hard parts?
15:15:46 <ralonsoh> yes, he can help us a lot
15:16:03 <ralonsoh> but when I talked to David, he told me that this is a low priority bug
15:16:22 <ralonsoh> and he will start implementing the migration once the classifier is merged
15:16:27 <ralonsoh> or the main part
15:16:31 <ralonsoh> makes sense
15:17:03 <ralonsoh> I would reduce the "importace" to low
15:17:07 <slaweq> yes, it's definitely not urgent thing
15:18:05 <ralonsoh> and last bug is
15:18:09 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1824095
15:18:09 <openstack> Launchpad bug 1824095 in neutron "Fullstack job broken in stable/stein branch" [High,Fix committed] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:18:16 <ralonsoh> The patch was merged
15:18:20 <ralonsoh> one sec...
15:18:35 <ralonsoh> #link https://review.opendev.org/#/c/651458/
15:18:35 <patchbot> patch 651458 - neutron (stable/stein) - Specify physical network in QoS fullstack tests (MERGED) - 2 patch sets
15:18:57 <ralonsoh> a simple cherry-pick from a previous one
15:19:16 <ralonsoh> I'll remove it from the list
15:19:20 <ralonsoh> and that's all I have
15:19:43 <ralonsoh> #topic Open Discussion
15:19:56 <ralonsoh> do you have something to discuss?
15:21:00 <slaweq> I'm good
15:21:44 <ralonsoh> perfect then, thank you very much for attending today. Next meeting in 40 mins in this channel (Neutron CI)
15:21:51 <ralonsoh> #endmeeting