15:00:57 <ralonsoh> #startmeeting neutron_qos
15:00:59 <openstack> Meeting started Tue Oct 10 15:00:57 2017 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:00 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:03 <openstack> The meeting name has been set to 'neutron_qos'
15:01:11 <ralonsoh> hi there
15:01:17 <davidsha> Hi
15:01:18 <mlavalle> o/
15:01:46 <ralonsoh> slaweq is not going to join us today
15:01:49 <reedip_> o/
15:02:13 <ralonsoh> let's go
15:02:13 <ralonsoh> #topic RFEs
15:02:19 <reedip_> mlavalle : you should just stay on  .. I think you have another meeting or two in this channel today ?? :O
15:02:37 <mlavalle> yeap
15:02:55 <ralonsoh> ok
15:02:56 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1649517
15:02:58 <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 Reedip (reedip-banerjee)
15:03:07 <ralonsoh> reedip_: this code is almost done
15:03:16 <ralonsoh> will you have time to address the comments?
15:03:43 <reedip_> I will ...
15:03:53 <davidsha> There is an issue with pep8 atm though
15:04:05 <ralonsoh> I know
15:04:08 <reedip_> dont know why it is failing
15:04:25 <ralonsoh> I'll take a look at this today
15:04:30 <ralonsoh> #link https://review.openstack.org/#/c/419642/
15:04:47 <ralonsoh> ok, next one
15:04:50 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1596611
15:04:51 <openstack> Launchpad bug 1596611 in neutron "[RFE] Create L3 IPs with qos (rate limit)" [Wishlist,Triaged] - Assigned to LIU Yulong (dragon889)
15:04:55 <ralonsoh> Two patches
15:05:00 <ralonsoh> #link https://review.openstack.org/#/c/453458/
15:05:17 <ralonsoh> The code is done, just to be reviewed by cores
15:05:31 <ralonsoh> this is the TC wrapper
15:05:44 <ralonsoh> and the second one, the qos FIP API
15:05:46 <ralonsoh> #link https://review.openstack.org/#/c/424466/
15:06:03 <ralonsoh> I need to take a second look to this patch (and everyone) this week
15:06:43 <reedip_> ralonsoh : done
15:07:09 <ralonsoh> ok, now DSCP marks and ECN
15:07:16 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1505627
15:07:17 <openstack> Launchpad bug 1505627 in neutron "[RFE] QoS Explicit Congestion Notification (ECN) Support" [Wishlist,Triaged] - Assigned to Reedip (reedip-banerjee)
15:07:17 <ralonsoh> and
15:07:23 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1692951
15:07:25 <openstack> Launchpad bug 1692951 in neutron "[RFE] DSCP mark on the outer header" [Wishlist,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:07:45 <ralonsoh> alisanhaji, reedip_: did you find a solution to join both specs?
15:08:07 <alisanhaji> I think these were not the issue, the DSCP one is separate from ECN
15:08:11 <reedip_> ralonsoh : alisanhaji : you want to say it ?
15:08:25 <ralonsoh> sorry, this one
15:08:26 <alisanhaji> It was rather between two ECN RFEs
15:08:28 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1708460
15:08:29 <openstack> Launchpad bug 1708460 in neutron " [RFE] Reaction to network congestion for qos" [Wishlist,New]
15:08:31 <reedip_> :)
15:08:36 <reedip_> ralonsoh : no issues :)
15:08:39 <alisanhaji> yes :)
15:09:06 <alisanhaji> well, we think that those can still be two implementations in two steps
15:09:24 <ralonsoh> which ones?
15:09:35 <alisanhaji> the first one is reedip's which can react make the VMs activate ECN
15:09:56 <reedip_> as per the discussion which we had, ralonsoh , the ECN one which I proposed with vikram would be the a subset of alisanhaji's proposal
15:10:25 <reedip_> the ECN proposal from my side wont have VM controls on it. alisanhaji's proposal would have it
15:10:40 <alisanhaji> then this one "[RFE] Reaction to network congestion for qos" cqn be a second step where neutron automatically react to congestion when VMs do not do it themselves
15:11:00 <ralonsoh> perfect! so first you need to make VMs and backend aware to ECN
15:11:06 <reedip_> yep
15:11:09 <alisanhaji> yep
15:11:11 <ralonsoh> then implement the ECN logic
15:11:28 <ralonsoh> I'll copy this in the etherpad, thanks!!
15:11:43 <reedip_> yes, so we can have both in parallel and merge them once they are independently functional ... alisanhaji , thats sounds ok ?
15:12:01 <ralonsoh> so let's work on reedip_'s spec: https://review.openstack.org/#/c/445762/
15:12:28 <ralonsoh> reedip_: yes, you can. But you should focus on enabling ECN first
15:12:42 <reedip_> waiting for your comments, guys :)
15:12:57 <ralonsoh> reedip_: ACK
15:13:02 <alisanhaji> yes we can first focus in making the VMs work
15:13:21 <alisanhaji> with ECN then add extra functionalities
15:13:28 <ralonsoh> alisanhaji, reedip_ : thanks guys! Let's push ASAP the ECN spec
15:13:41 <ralonsoh> reedip_: I'll review your spec this week
15:13:59 <ralonsoh> but try to take core reviewers to it
15:14:04 <reedip_> thanks ralonsoh : mlavalle , some inputs from you would also be appreciated :)
15:15:03 <alisanhaji> so for the DSCP RFE https://bugs.launchpad.net/neutron/+bug/1692951
15:15:04 <openstack> Launchpad bug 1692951 in neutron "[RFE] DSCP mark on the outer header" [Wishlist,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:15:04 <ralonsoh> ok, let's move to the last spec
15:15:13 <alisanhaji> ralonsoh: I see that you rebased
15:15:33 <ralonsoh> oh, yes, sorry
15:15:38 <ralonsoh> first this one
15:15:45 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1692951
15:15:50 <alisanhaji> after slaweq_ reviews, I merged the LB with the OVS review
15:15:58 <alisanhaji> I think I can abandon the LB one
15:16:15 <ralonsoh> #link https://review.openstack.org/#/c/501267/
15:16:23 <ralonsoh> why?
15:16:42 <ralonsoh> why are you abandon the LB?
15:17:08 <alisanhaji> Because it's merged with the OVS one
15:17:15 <alisanhaji> this the LB one: https://review.openstack.org/#/c/501271/
15:17:19 <ralonsoh> oh..... sorry
15:17:38 <ralonsoh> yes, you should abandon https://review.openstack.org/#/c/501271/
15:17:42 <alisanhaji> It's not relevant anymore
15:17:52 <ralonsoh> and focus on LB+OVS: https://review.openstack.org/#/c/501267/
15:18:18 <alisanhaji> done
15:18:41 <ralonsoh> you have +1 from slaweq. I'll review this patch today
15:18:44 <alisanhaji> The OVS+LB one needs more reviews
15:18:51 <ralonsoh> btw, I'll add a comment in the LB patch
15:18:53 <alisanhaji> great thanks
15:19:33 <ralonsoh> alisanhaji: thank you!
15:20:03 <ralonsoh> ok, last RFE
15:20:06 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1560963
15:20:07 <openstack> Launchpad bug 1560963 in neutron "[RFE] Minimum bandwidth support (egress)" [Wishlist,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:20:08 <alisanhaji> :)
15:20:26 <ralonsoh> I'm doing the spec in parallel to Nova spec
15:20:43 <ralonsoh> So I'm mainly waiting for Nova spec comments to finish Neutron one
15:20:59 <mlavalle> I reviewed both on Friday I think
15:21:09 <ralonsoh> and that's all... for now. Next 19 is Nova spec freeze so I need to hurry up
15:21:14 <ralonsoh> mlavalle: yes
15:21:20 <mlavalle> did you push a ew revision after that?
15:21:23 <ralonsoh> now we have an "issue"
15:21:27 <mlavalle> new^^^
15:21:44 <ralonsoh> about how to nest resources: HOST->AGENTS->PHYNETS
15:21:46 <ralonsoh> or
15:21:54 <ralonsoh> HOST->PHYSNETS
15:22:12 <ralonsoh> sean-k-mooney is going to add new comments in Nova side
15:22:31 <mlavalle> I think he did
15:22:34 <mlavalle> already
15:22:50 <mlavalle> earlier today
15:23:10 <ralonsoh> mlavalle: now I see them. I'll start addressing the comments at the end of this meeting
15:23:50 <mlavalle> let's keep in mind that we need to get this spec approved by Thursday of nxt week
15:23:50 <ralonsoh> and that's all in the agenda for RFEs
15:23:57 <ralonsoh> I know...
15:24:01 <mlavalle> ok
15:24:09 <ralonsoh> this is my first priority, for sure
15:24:37 <ralonsoh> I'll ping Jay, Sean, Matt and Eric to have this one reviewed
15:25:00 <jaypipes> ralonsoh: in my queue.
15:25:11 <ralonsoh> jaypipes: thanks!!
15:25:49 <ralonsoh> Any other topic in RFE section?
15:26:04 <ralonsoh> let's move to bugs
15:26:14 <ralonsoh> #topic Bugs
15:26:18 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1720077
15:26:19 <openstack> Launchpad bug 1720077 in neutron "Quality of Service (QoS) , document is critical error" [Undecided,Incomplete] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:26:30 <ralonsoh> IMO, this bug is invalid
15:26:47 <reedip_> but it states "critical" :)
15:26:57 <ralonsoh> the configuration options in docs section are correct, as I explain in the bug
15:27:29 <ralonsoh> I think @silently is mixing controller/network/compute node concept
15:27:38 <ralonsoh> I replied in the bug
15:28:09 <ralonsoh> and I tested it (with QoS config options explained in manuals) and worked fine
15:28:25 <ralonsoh> I markef this as incomplete
15:28:41 <ralonsoh> next one
15:28:42 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1702637
15:28:43 <openstack> Launchpad bug 1702637 in neutron "API requests give different results from examples" [Undecided,In progress]
15:29:01 <ralonsoh> I didn't have time to review this one
15:29:09 <ralonsoh> reedip_: do you have time to take a look?
15:29:29 <ralonsoh> reedip_: only if you can
15:30:15 <ralonsoh> I'll try to ping reedip_ a bit later
15:30:24 <ralonsoh> last one
15:30:25 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1628533
15:30:26 <openstack> Launchpad bug 1628533 in neutron "agent file name unclear in config qos doc" [Low,In progress] - Assigned to Boden R (boden)
15:30:52 <ralonsoh> the code is almost done, with several +2
15:30:57 <ralonsoh> so no problem with this one
15:31:03 <ralonsoh> just a doc change
15:31:27 <ralonsoh> and that's all for today!
15:31:40 <ralonsoh> #topic Open Discussion
15:32:05 <ralonsoh> Do you have any other topic?
15:32:37 <reedip_> ralonsoh : will do
15:32:52 <ralonsoh> reedip_: thanks!!
15:33:21 <ralonsoh> ok, let's go back to work hehehe
15:33:27 <davidsha> Thanks!
15:33:29 <ralonsoh> see you in two weeks
15:33:52 <ralonsoh> and thank you for the drivers meeting time change
15:34:02 <ralonsoh> #endmeeting