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