15:00:14 <ralonsoh> #startmeeting neutron_qos
15:00:14 <openstack> Meeting started Tue Feb 26 15:00:14 2019 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:15 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:17 <openstack> The meeting name has been set to 'neutron_qos'
15:00:21 <ralonsoh> hello everyone!
15:00:25 <davidsha> Hi
15:01:17 <mlavalle> ralonsoh: I won't be able to attend today
15:01:24 <ralonsoh> mlavalle, np
15:02:30 <ralonsoh> ok, the quorum is quite low today
15:02:30 <slaweq> hi
15:02:39 <ralonsoh> perfect, one more folk
15:02:44 <ralonsoh> ok, let's start
15:02:44 <slaweq> :)
15:02:44 <liuyulong> hi
15:02:47 <liuyulong> I'm here
15:02:51 <ralonsoh> hi!
15:02:59 <ralonsoh> #topic RFEs
15:03:15 <ralonsoh> I would prefer to wait for rubasov or lajoskatona
15:03:21 <ralonsoh> so I'll start with this one
15:03:23 <lajoskatona> o/
15:03:28 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1777627
15:03:29 <openstack> Launchpad bug 1777627 in neutron "RFE: QoS – rule's management (set,delete, show…) requires <qos-policy> parameter in addition to <rule-id>" [Wishlist,In progress] - Assigned to Miguel Lavalle (minsel)
15:03:40 <ralonsoh> #link https://review.openstack.org/#/c/613820/
15:03:53 <ralonsoh> some comments still pending
15:04:19 <ralonsoh> because Miguel is not attending, I'll call him later
15:04:28 <ralonsoh> next one
15:04:29 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1578989
15:04:29 <openstack> Launchpad bug 1578989 in neutron "[RFE] Strict minimum bandwidth support (egress)" [Wishlist,In progress] - Assigned to Lajos Katona (lajos-katona)
15:04:35 <ralonsoh> #link https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:minimum-bandwidth-allocation-placement-api
15:04:35 <ralonsoh> #link https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/bandwidth-resource-provider
15:04:35 <ralonsoh> #link https://review.openstack.org/#/q/topic:minimum-bandwidth-allocation-placement-api+(status:open+OR+status:merged)+project:openstack/tempest
15:04:45 <ralonsoh> let me focus on the Neutron patches
15:05:06 <ralonsoh> most of them with +W
15:05:06 <lajoskatona> ralonsoh: yeah as previously I said the most critical ones are from rubasov
15:05:27 <ralonsoh> lajoskatona, in Nova
15:05:33 <ralonsoh> right?
15:06:17 <lajoskatona> ralonsoh: as I know in nova the patches are progressing, perhaps 5 is still under review to have the basic functionality ready in stein
15:06:51 <lajoskatona> ralonsoh: that means to boot/delete  VMs and detach interfaces with ports with min bw
15:07:08 <ralonsoh> lajoskatona, I've follow those ones, yes
15:07:29 <ralonsoh> ralonsoh, I'm not a Nova reviewer but I'll give a try to those ones
15:07:35 <ralonsoh> to speedup the reviews
15:07:43 <lajoskatona> ralonsoh: thanks
15:07:48 <ralonsoh> For the record: those 2 Neutron patches need reviews
15:07:51 <ralonsoh> #link https://review.openstack.org/#/c/580672/
15:08:00 <ralonsoh> #link https://review.openstack.org/#/c/574783/
15:08:21 <lajoskatona> ralonsoh: exactly, thanks for linking them.
15:08:41 <lajoskatona> ralonsoh: rubasov started to work on documentation, and I am working on tests
15:09:07 <ralonsoh> lajoskatona, tempest (for the records) https://review.openstack.org/#/q/topic:minimum-bandwidth-allocation-placement-api+(status:open+OR+status:merged)+project:openstack/tempest
15:09:21 <lajoskatona> so I suppose this week there will be patch(es) for documenting the feature
15:09:44 <ralonsoh> lajoskatona, cool, mostly on Neutron, is that right?
15:09:55 <lajoskatona> ralonsoh: yeah, I think I will ping QA guys for review.
15:10:00 <ralonsoh> sure
15:10:02 <lajoskatona> ralonsoh: yes
15:10:10 <slaweq> lajoskatona: I will review it too
15:10:19 <lajoskatona> slaweq: thanks
15:10:44 <ralonsoh> lajoskatona, any blocker/stopper/something else?
15:11:22 <lajoskatona> ralonsoh: I think nothing. We hoped to be readye much before the stein freeze but hope that it will go through
15:11:43 <davidsha> I think tomorrow is the freeze
15:12:07 <davidsha> it was mentioned in the last meeting I think
15:12:29 <ralonsoh> davidsha, yes, it's tomorrow, but we can always ask for an extension
15:12:42 <davidsha> True
15:12:42 <ralonsoh> for specific features
15:13:09 <ralonsoh> lajoskatona, thank you very much (and ruabsov)
15:13:34 <ralonsoh> Last one
15:13:35 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1560963
15:13:36 <openstack> Launchpad bug 1560963 in neutron "[RFE] Minimum bandwidth support (egress)" [Wishlist,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:13:42 <ralonsoh> #link https://review.openstack.org/#/q/topic:bug/1560963+(status:open)
15:13:49 <ralonsoh> most of the patches are merged
15:14:07 <ralonsoh> just one with +2 for the pyroute changes
15:14:22 <ralonsoh> and, of course, the bug one is still there (without reviews)
15:14:50 <ralonsoh> I'll start, next week, coding the tunnelled interfaces patch
15:15:04 <ralonsoh> on top of the big one (for physical bridges)
15:15:39 <ralonsoh> and that's all for me
15:15:54 <ralonsoh> any other comment in this topic? RFEs
15:16:28 <ralonsoh> ok, let's move
15:16:29 <ralonsoh> #topic Bugs
15:16:37 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1785189
15:16:38 <openstack> Launchpad bug 1785189 in neutron "Floatingip and router bandwidth speed limit failure" [Medium,In progress]
15:16:41 <ralonsoh> #link: https://review.openstack.org/#/c/596637/
15:16:59 <ralonsoh> As commented last week, I can't reproduce the problem spotted out in the bug
15:17:22 <ralonsoh> So I recommended to abandon this patch, but still waiting
15:17:32 <ralonsoh> I'll remove it from the list of bugs next week
15:17:54 <ralonsoh> next one
15:17:55 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1784006
15:17:56 <openstack> Launchpad bug 1784006 in neutron "Instances miss neutron QoS on their ports after unrescue and soft reboot" [Medium,Confirmed] - Assigned to Miguel Lavalle (minsel)
15:18:35 <ralonsoh> Because Miguel is not here, we'll wait until he start the development (or maybe I'll ask him to take this one).
15:18:39 <ralonsoh> I'll ping him later
15:19:03 <ralonsoh> (or anyone else, if you have time, for sure)
15:19:20 <ralonsoh> last one
15:19:21 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1815618
15:19:22 <openstack> Launchpad bug 1815618 in neutron "cannot update qos rule" [High,In progress] - Assigned to Lajos Katona (lajos-katona)
15:19:25 <ralonsoh> https://review.openstack.org/#/c/638714/
15:19:28 <rubasov> late o/
15:19:34 <ralonsoh> rubasov, no problem
15:19:55 <ralonsoh> rubasov, if you have comments on the RFE, you can talk in the open discussion
15:20:03 <ralonsoh> (in 5 mins)
15:20:22 <ralonsoh> lajoskatona, I wrote a couple of comments on the patch
15:20:39 <lajoskatona> ralonsoh: for the bug: I will check your comments,
15:20:50 <ralonsoh> IMO (just my opinion), it's cleaner if you just update the check_rules_conflict method
15:21:26 <ralonsoh> but anyway, the bug is addressed and you found the problem, what is great
15:21:34 <lajoskatona> ralonsoh: I check it, thanks for reviewing
15:21:38 <ralonsoh> np
15:21:55 <ralonsoh> I don't have more bugs in the list
15:22:01 <ralonsoh> Am I missing something?
15:22:43 <ralonsoh> cool
15:22:51 <ralonsoh> #topic Open Discussion
15:23:05 <ralonsoh> rubasov, do you have any topic?
15:23:18 <rubasov> ralonsoh: no I don't
15:24:14 <ralonsoh> perfect
15:24:17 <davidsha> ralonsoh: ok if I give a quick update on the neutron-classifier qos POC?
15:24:27 <ralonsoh> davidsha, yes, I was going to ping you
15:24:30 <ralonsoh> sure!
15:25:26 <davidsha> I got the database base migration script to use the classification-id and the policy-id as a unique key tuple, thanks for linking me to the directional example for bw-limit!
15:25:42 <ralonsoh> hehehe you are welcome
15:25:55 <davidsha> It works for classification-id = None as well so it's backwards compatible as well.
15:26:29 <davidsha> I've also started into propagating the classification to the L2-agent but thats a little less refined at the moment
15:26:36 <ralonsoh> davidsha, perfect, that was my concern last day
15:27:39 <ralonsoh> davidsha, one question, about the RPC modifications
15:27:49 <ralonsoh> davidsha, how are you going to handle it?
15:28:01 <davidsha> I should have the next PS up soon, just running pep8 atm, I have the QoS driver listening for the topics at the moment, but was thinking of using the new classifier l2 extension to add the classification to a dict in the agent extensions API
15:28:02 <ralonsoh> davidsha, you are going to add new RPC callbacks to the mech drivers
15:28:19 <ralonsoh> davidsha, " I have the QoS driver listening" hahahaha
15:28:28 <ralonsoh> ok
15:28:51 <ralonsoh> the point is this should be an extension, only loaded if enabled
15:29:12 <davidsha> True
15:29:27 <ralonsoh> perfect, just to be aligned
15:29:47 <ralonsoh> btw, only for L2? or L3 too is needed
15:29:53 <davidsha> I added it just to make sure it was broadcasting, was already looking into adding it to the extensions API ;P
15:30:39 <davidsha> atm only L2, but I can duplicate the work from L2 to L3, they are both made symmetrical in their extension APIs
15:30:56 <ralonsoh> cool
15:31:17 <ralonsoh> because so far there are no L2/L3 shared extensions
15:31:19 <ralonsoh> qos?
15:31:28 <davidsha> qos is, isn't it?
15:31:42 <davidsha> it has FIP bw limit
15:31:47 <ralonsoh> yes, but with different interfaces
15:32:01 <davidsha> kk, I think FWaas might have both
15:32:05 <ralonsoh> it's not possible to have a single interface for both
15:32:32 <davidsha> kk
15:33:02 <ralonsoh> do you have any patch upstream?
15:33:29 <davidsha> I have patches, just waiting on pep8 before upstreaming them
15:33:34 <ralonsoh> ok
15:34:03 <ralonsoh> any other update?
15:34:06 <davidsha> It's these: https://review.openstack.org/#/c/636330/ https://review.openstack.org/#/c/636333/
15:34:19 <davidsha> Nope, thats me, thanks!
15:34:29 <ralonsoh> I'll add both to the etherpad
15:34:55 <davidsha> thanks!
15:35:15 <ralonsoh> thank you
15:35:32 <ralonsoh> any other update from you folks?
15:36:12 <ralonsoh> Thank you very much for attending
15:36:16 <ralonsoh> see you in two weeks
15:36:19 <ralonsoh> #endmeeting