15:00:15 <ralonsoh> #startmeeting neutron_qos
15:00:15 <openstack> Meeting started Tue Apr 11 15:00:15 2017 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:17 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:19 <openstack> The meeting name has been set to 'neutron_qos'
15:00:22 <ralonsoh> hi
15:00:22 <mlavalle> o/
15:00:25 <reedip_> yo
15:01:14 <ralonsoh> slawek is not going to attend
15:01:18 <ralonsoh> but I have his updates
15:01:21 <ralonsoh> let's start
15:01:21 <davidsha> Hi
15:01:27 <ralonsoh> #topic RFEs
15:01:35 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bugs?field.tag=qos+rfe+&field.tags_combinator=ALL RFEs
15:01:42 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1505627
15:01:44 <openstack> Launchpad bug 1505627 in neutron "[RFE] QoS Explicit Congestion Notification (ECN) Support" [Wishlist,New] - Assigned to Reedip (reedip-banerjee)
15:02:05 <ralonsoh> reedip_: are you still working on this spec?
15:02:16 <ralonsoh> there are some comments not addressed
15:02:34 <reedip_> ralonsoh : I was, but couldnt give too much time right now. I will reiterate over it this week.
15:02:49 <ralonsoh> no problem, so it's on hold
15:03:12 <ralonsoh> I know you have other patches ongoing
15:03:17 <ralonsoh> next one
15:03:19 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1596611
15:03:20 <openstack> Launchpad bug 1596611 in neutron "[RFE] Create L3 IPs with qos (rate limit)" [Wishlist,In progress] - Assigned to LIU Yulong (dragon889)
15:03:34 <ralonsoh> This spec needs another iteration
15:03:49 <ralonsoh> IMO, it's almost finished, but we need to review it again
15:03:54 <ralonsoh> Some small nits
15:04:09 <ralonsoh> Please, spend a bit of time taking a look a it
15:04:24 <ralonsoh> (it's a problem we don't have any submitter here)
15:04:39 <ralonsoh> next one
15:04:40 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1639220
15:04:41 <openstack> Launchpad bug 1639220 in neutron "[RFE] Introduce Network QoS policy "is_default" behaviour" [Low,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:05:09 <ralonsoh> I refactored the code, based in some comments from kevinbenton
15:05:27 <ralonsoh> All tests are passing and I made some manual testing. It's working
15:05:47 <ralonsoh> So, the same, please take a bit of time to review it
15:06:11 <ralonsoh> next one
15:06:13 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1560961
15:06:15 <openstack> Launchpad bug 1560961 in neutron "[RFE] Allow instance-ingress bandwidth limiting" [Wishlist,In progress] - Assigned to Slawek Kaplonski (slaweq)
15:06:49 <ralonsoh> I talk to slaweq. The first patch is there, implementing the API
15:07:21 <ralonsoh> There are some problems with tempest tests, but I think it's related with the client version
15:07:56 <ralonsoh> But the code is almost finished. Then we need to start the implementation of any of the backends
15:08:10 <ralonsoh> We will discuss which one should be the first one
15:08:36 <ralonsoh> ok, next one
15:08:38 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1586056
15:08:39 <openstack> Launchpad bug 1586056 in neutron "[RFE] Improved validation mechanism for QoS rules with port types" [Wishlist,In progress] - Assigned to Slawek Kaplonski (slaweq)
15:09:06 <ralonsoh> the same: the code is finished. Reviews please!
15:09:29 <ralonsoh> the last one ( mlavalle )
15:09:30 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1578989
15:09:31 <openstack> Launchpad bug 1578989 in neutron "[RFE] Strict minimum bandwidth support (egress)" [Wishlist,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:09:55 <ralonsoh> mlavalle: how the investigation about the nova API is going?
15:10:02 <davidsha> ralonsoh: is the patch not merged already for validation?
15:10:07 <mlavalle> It's going well
15:10:15 <davidsha> ralonsoh: https://review.openstack.org/#/c/323474/
15:10:26 <mlavalle> you saw the comment I left in the bug
15:10:55 <mlavalle> the only caveat is that the nova team is implementing nested resources in this cycle
15:11:00 <ralonsoh> mlavalle: which one?
15:12:18 <mlavalle> ralonsoh: I think I left it in the spec
15:12:29 <ralonsoh> mlavalle: I'll take a look today
15:12:54 <ralonsoh> davidsha: that was a mistake.  bug 1586056 shouldn't be in this list. Thanks!
15:12:56 <openstack> bug 1586056 in neutron "[RFE] Improved validation mechanism for QoS rules with port types" [Wishlist,In progress] https://launchpad.net/bugs/1586056 - Assigned to Slawek Kaplonski (slaweq)
15:12:59 <mlavalle> ralonsoh: https://review.openstack.org/#/c/396297/
15:13:46 <ralonsoh> mlavalle: perfect, I'll take a look later and we can talk via IRC
15:14:04 <mlavalle> ralonsoh: what I really want the team to look at is http://paste.openstack.org/show/605573/
15:14:37 <mlavalle> ralonsoh: but I am moving ahead with coding
15:14:50 <davidsha> ralonsoh: Actually I was just looking, it has a follow up, but it probably should be another bug: https://review.openstack.org/#/c/426946/
15:15:56 <ralonsoh> davidsha: yes, that was listed before, in the RFEs
15:16:25 <ralonsoh> mlavalle: I know the placement API and os-traits are still in progress
15:16:35 <ralonsoh> mlavalle: this will delay our development
15:16:35 <mlavalle> cool
15:16:56 <ralonsoh> mlavalle: we need to deal with this, no problem
15:17:07 <mlavalle> I faced the same situation with routed networks
15:17:15 <mlavalle> and I developed in parallel with them
15:17:16 <ralonsoh> ok, I think we can move to next topic.
15:17:27 <ralonsoh> mlavalle: yes, for sure
15:17:27 <mlavalle> so by the time they were done, we were also done
15:17:36 <mlavalle> we can do the same here
15:17:47 <mlavalle> as long as I know what the calls looks like
15:17:51 <mlavalle> which is the case
15:18:00 <ralonsoh> mlavalle: and ping me if you need help with some coding or reviews
15:18:08 <mlavalle> yeap, will do
15:18:16 <ralonsoh> Am I missing something in this topic?
15:18:22 <ralonsoh> RFE's
15:18:47 <ralonsoh> ok, next topic
15:18:48 <ralonsoh> #topic Bugs
15:19:10 <ralonsoh> Three easy ones related
15:19:13 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1681728
15:19:14 <openstack> Launchpad bug 1681728 in neutron "QoS rule object should be manage in QosRule metaclass" [Undecided,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:19:19 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1681440
15:19:20 <openstack> Launchpad bug 1681440 in neutron "QoS policy object can't be suitable with 1.2 version of object" [Undecided,In progress] - Assigned to Slawek Kaplonski (slaweq)
15:19:24 <ralonsoh> and
15:19:26 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1677303
15:19:27 <openstack> Launchpad bug 1677303 in neutron "QoS policy object should not raise an exception for version degradation < 1.3" [Undecided,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:20:15 <ralonsoh> davidsha wrote a bug including all of them, but ihrachys suggested to split them in smaller ones
15:20:47 <ralonsoh> All of them are implemented, just waiting for reviews
15:20:59 <ralonsoh> BTW, easy ones!!
15:21:11 <ralonsoh> next one
15:21:14 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1676877
15:21:15 <openstack> Launchpad bug 1676877 in neutron "Increase "TestQosPlugin.test_update_policy_rule" coverage" [Medium,New] - Assigned to Reedip (reedip-banerjee)
15:21:28 <ralonsoh> reedip: how is going?
15:22:31 <ralonsoh> ok, I'll try to ping reedip_ a bit later
15:22:58 <reedip_> ralonsoh : hi , just joined, network issue !
15:23:01 <ralonsoh> but this bug should be easy to implement. There are some cases in QOS testing to be covered
15:23:19 <reedip_> I will pick it up this week,will push a ptch by thursday
15:23:43 <ralonsoh> reedip_: perfect, very fast! thanks
15:24:05 <ralonsoh> and the last one (in my list)
15:24:06 <ralonsoh> https://bugs.launchpad.net/neutron/+bug/1657381
15:24:08 <openstack> Launchpad bug 1657381 in neutron "QoS drivers need to implement a precommit for the actions" [Medium,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:24:52 <ralonsoh> That was ajo's patch, but I continued his progress
15:25:18 <ralonsoh> reedip_ made some comments, but I think the patch is almost ready to be merged
15:25:34 <ralonsoh> but, of course, reviews are welcome
15:25:43 <reedip_> did I ( I forgot :) )
15:25:56 <ralonsoh> https://review.openstack.org/#/c/421818/
15:26:00 <ralonsoh> This one
15:26:13 <mlavalle> ralonsoh: I'll review it today
15:26:18 <ralonsoh> thanks!
15:26:28 <mlavalle> just starred it
15:26:44 <ralonsoh> I don't have any other bug in the list (https://etherpad.openstack.org/p/neutron_qos_meeting_chair)
15:27:00 <ralonsoh> something is missing here?
15:27:06 <reedip_> slaweq, ralonsoh : if there is no issue, can I take this up after my bug submissions?  https://review.openstack.org/#/c/419642/
15:27:20 <ralonsoh> let me check it...
15:27:24 <reedip_> https://launchpad.net/bugs/1649517
15:27:25 <openstack> Launchpad bug 1649517 in neutron "qos policy attached to network, qos_policy_id is reflecting on neutron net-show , but not on the port with neutron port-show" [Wishlist,In progress] - Assigned to Slawek Kaplonski (slaweq)
15:27:48 <ralonsoh> oh, yes
15:28:11 <ralonsoh> this is a low-priority one
15:28:12 <ralonsoh> but
15:28:28 <ralonsoh> it should be nice to have it, of course
15:28:58 <ralonsoh> I'll add it to our list now
15:29:23 <ralonsoh> perfect, more items in this topic?
15:29:28 <reedip_> ralonsoh : ok, I will see if slaweq doesnt push something by the time I finish my patches, I will take it up.
15:29:36 <reedip_> ralonsoh : none from my side
15:30:11 <ralonsoh> reedip_: you can talk together to take this patch, I know slaweq will have no problem
15:30:36 <reedip_> ralonsoh : ok :)
15:30:41 <ralonsoh> I'll skip the topic "other changes"
15:30:45 <ralonsoh> last topic
15:30:46 <ralonsoh> #topic Open Discussion
15:31:09 <mlavalle> I have a question regarding https://review.openstack.org/#/c/426946/
15:31:25 <mlavalle> Is ajo going to review it?
15:31:54 <ralonsoh> ajo is on annual leave this week
15:32:16 <ralonsoh> ping him next one (not on monday, it's bank holiday in Spain)
15:32:37 <mlavalle> ok, my only point is that if he doesn't have the time, I am ready to merge it
15:32:42 <ralonsoh> I know this patch is waiting only for him....
15:33:03 <reedip_> mlavalle : I think that patch needs a reno
15:33:35 <mlavalle> reedip_: that may be
15:33:43 <ralonsoh> hmmm, I don't think so
15:34:01 <reedip_> I mentioned that in the patch but I think it got overlooked :P
15:34:05 <mlavalle> my only point is that during the last meeting I felt the team wanted to move ahead with this pacthset
15:34:08 <ralonsoh> you are not modifying the current behaviour
15:34:44 <ralonsoh> mlavalle: yes, we need this one, it's important to assure the correct behaviour of qos
15:35:00 <mlavalle> so, I'll wait until the end of next week
15:35:05 <mlavalle> I'll ping ajo
15:35:09 <mlavalle> early in the week
15:35:22 <ralonsoh> I'll put this in my calendar
15:35:25 <mlavalle> if by the end, it hasn't been reviewed by him, I'll approve
15:35:35 <ralonsoh> mlavalle: thank you!
15:36:24 <ralonsoh> ok, I think we are done
15:36:31 <ralonsoh> I'll close the meeting
15:36:38 <ralonsoh> #endmeeting