15:00:37 <ralonsoh> #startmeeting neutron_qos
15:00:38 <openstack> Meeting started Tue Mar 26 15:00:37 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:39 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:42 <openstack> The meeting name has been set to 'neutron_qos'
15:00:45 <ralonsoh> Hi
15:00:48 <davidsha> Hi!
15:00:58 <mlavalle> o/
15:01:27 <rubasov> hi
15:01:53 <ralonsoh> ok, let's start
15:01:57 <ralonsoh> #topic RFEs
15:02:03 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1578989
15:02:05 <openstack> Launchpad bug 1578989 in neutron "[RFE] Strict minimum bandwidth support (egress)" [Wishlist,Fix released] - Assigned to Bence Romsics (bence-romsics)
15:02:19 <lajoskatona> o/
15:02:21 <ralonsoh> except for some doc patches, almost everything is done
15:02:26 <ralonsoh> is that right?
15:02:42 <rubasov> we have some tests remaining
15:03:00 <rubasov> the doc was merged completely
15:03:20 <ralonsoh> that's right, the tempest tests
15:03:25 <rubasov> and a followup rfe to decouple the placement reporter service plugin from ml2
15:03:31 <mlavalle> yeap, the doc is in
15:03:44 <mlavalle> the nova guys were on top of me for that
15:03:45 <ralonsoh> yes, the rfe is in this list
15:04:17 <ralonsoh> ok, so keep reviewing the docs and the tests
15:04:17 <rubasov> lajoskatona: anyhting else I missed?
15:04:20 <ralonsoh> for the records
15:04:21 <lajoskatona> RFE for for decoupling: https://bugs.launchpad.net/neutron/+bug/1818479
15:04:22 <openstack> Launchpad bug 1818479 in neutron "RFE Decouple placement reporting service plugin from ML2" [Wishlist,New] - Assigned to Bence Romsics (bence-romsics)
15:04:22 <ralonsoh> tempest test: #link https://review.openstack.org/629142, #link https://review.openstack.org/629253
15:04:22 <ralonsoh> fullstack: #link https://review.openstack.org/638666, #link https://review.openstack.org/642527
15:04:43 <ralonsoh> yes, we have this RFE in the list
15:04:56 <ralonsoh> just let's finish this one first
15:05:09 <lajoskatona> rubasov: I think that's it
15:05:18 <ralonsoh> Thank you very much
15:05:34 <ralonsoh> next one then
15:05:35 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1818479
15:05:36 <openstack> Launchpad bug 1818479 in neutron "RFE Decouple placement reporting service plugin from ML2" [Wishlist,New] - Assigned to Bence Romsics (bence-romsics)
15:05:59 <ralonsoh> please, lajoskatona
15:06:11 <davidsha> ralonsoh: Is that the right link?
15:06:36 <ralonsoh> for this RFE? yes
15:06:57 <rubasov> slaweq had a code comment on the placement reporter service plugin just before merge we could not fix right there
15:06:58 <davidsha> kk
15:07:14 <rubasov> so I have the task of investigating
15:07:39 <mlavalle> we haven't discussed it in the drivers meeting
15:07:39 <rubasov> summarize the outcome of that in the rfe and then take it to the drivers meeting
15:08:20 <mlavalle> please update the RFE with the results of your research and at that point I will schedule it for discussion
15:08:35 <ralonsoh> rubasov, perfect, do you want me to remove it from the QoS agenda?
15:08:47 <rubasov> mlavalle: I'll do that
15:09:19 <rubasov> ralonsoh: yeah, maybe hide it until the drivers discuss it
15:09:31 <ralonsoh> perfect, thanks!
15:10:00 <ralonsoh> next one
15:10:03 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1777627
15:10:04 <openstack> Launchpad bug 1777627 in neutron "RFE: QoS – rule's management (set,delete, show…) requires <qos-policy> parameter in addition to <rule-id>" [Wishlist,Fix committed] - Assigned to Miguel Lavalle (minsel)
15:10:11 <ralonsoh> Merged! thanks mlavalle
15:10:18 <mlavalle> \o/
15:10:36 <ralonsoh> I'll remove it from the backlog next meeting
15:10:44 <davidsha> This was the one that only the rule id should be needed right?
15:10:44 <mlavalle> I have a question, thoiugh
15:10:48 <ralonsoh> sure
15:10:55 <mlavalle> davidsha: right
15:11:51 <mlavalle> https://docs.openstack.org/neutron/latest/admin/config-qos.html#trusted-projects-policy-json-configuration
15:12:35 <mlavalle> shouldn't we update this section of the docs, now that we have policy in code?
15:13:10 <ralonsoh> I don't understand the question
15:13:46 <mlavalle> I need to add a paragraph indicatin that the alias policies are setup to just follow the other policy
15:13:49 <ralonsoh> ah, you mean the rule permissions will depend on the policy one
15:14:48 <ralonsoh> can you submit it?
15:14:54 <ralonsoh> or do you want me to do it?
15:15:00 <mlavalle> like here: https://review.openstack.org/#/c/613820/11/neutron/conf/policies/qos.py@241
15:15:12 <mlavalle> no, I'll submit it
15:15:26 <ralonsoh> cool, thanks!
15:15:29 <mlavalle> I just wanted to point it out
15:16:39 <ralonsoh> ok, and the last RFE I have is
15:16:41 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1560963
15:16:42 <openstack> Launchpad bug 1560963 in neutron "[RFE] Minimum bandwidth support (egress)" [Wishlist,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:16:56 <ralonsoh> sorry but I was focused on the CI bugs
15:17:11 <ralonsoh> and I didn't start the tunnelled traffic patch
15:17:24 <ralonsoh> I'm planning to start it next week
15:17:27 <mlavalle> thanks for your hard work!
15:17:52 <ralonsoh> (I still have a couple of them tracked and prepared)
15:18:25 <ralonsoh> no more RFEs in the list, next time will be in Train!
15:18:46 <ralonsoh> let's move to bugs
15:18:49 <ralonsoh> #topic Bugs
15:19:04 <ralonsoh> a list of merged and closed ones:
15:19:06 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1815618
15:19:07 <openstack> Launchpad bug 1815618 in neutron "cannot update qos rule" [High,Fix released] - Assigned to Lajos Katona (lajos-katona)
15:19:12 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1818683
15:19:13 <openstack> Launchpad bug 1818683 in neutron "Placement reporter service plugin sometimes creates orphaned resource providers" [Undecided,Fix released] - Assigned to Bence Romsics (bence-romsics)
15:19:20 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1819029
15:19:20 <openstack> Launchpad bug 1819029 in neutron "QoS policies with minimum-bandwidth rule should be rejected on non-physnet ports/networks" [Undecided,Fix released] - Assigned to Slawek Kaplonski (slaweq)
15:19:31 <ralonsoh> thanks lajoskatona rubasov
15:20:21 <ralonsoh> the next one
15:20:22 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1819125
15:20:25 <openstack> Launchpad bug 1819125 in neutron "Fullstack test test_min_bw_qos_policy_rule_lifecycle failing often" [Critical,Fix committed] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)
15:20:31 <ralonsoh> #link https://review.openstack.org/#/c/646082/
15:21:02 <ralonsoh> I have proposed this patch to avoid cleaning the min QoS rules when the agent is started
15:21:17 <ralonsoh> I did it in functional tests but I missed this one for fullstack
15:21:35 <ralonsoh> PS3, as commented, is working
15:21:45 <ralonsoh> PS4 still in the CI
15:22:21 <ralonsoh> and the last one in the etherpad list
15:22:22 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1818696
15:22:23 <openstack> Launchpad bug 1818696 in neutron "frequent ci failures trying to delete qos port" [Undecided,Incomplete]
15:22:32 <ralonsoh> IMO we can close it
15:22:42 <ralonsoh> this problem was related to a specific patch
15:22:57 <ralonsoh> and this problem is not happening again
15:23:27 <mlavalle> cool
15:23:32 <ralonsoh> If we don't see this problem during the next two weeks (next meeting), we'll close it
15:24:14 <ralonsoh> and I don't have more bugs in the list (because QoS doesn't have bugs.....)
15:24:22 <ralonsoh> anyone I'm missing?
15:24:28 <mlavalle> nope
15:24:58 <ralonsoh> cool
15:25:08 <ralonsoh> #topic Open Discussion
15:25:11 <ralonsoh> davidsha, ?
15:25:21 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/1476527
15:25:22 <openstack> Launchpad bug 1476527 in neutron "[RFE] Add common classifier resource" [Wishlist,Triaged] - Assigned to Igor D.C. (igordcard)
15:25:52 <davidsha> Yup, I've fixed that DB issue, going to move onto extending the OvS backend this week
15:26:45 <ralonsoh> for the records: #link https://review.openstack.org/#/c/636333
15:27:17 <ralonsoh> davidsha, do you have any blocker or question?
15:27:21 <davidsha> There is someone working on Neutron Classifier at the moment thats adding extensions to store the classifications in the agent extensions API of the L2 and L3 agents atm.
15:27:43 <davidsha> None from me, but does anyone have questions for me?
15:28:07 <ralonsoh> who's working on this?
15:28:14 <ralonsoh> do you have patches?
15:28:42 <davidsha> Myself on the Neutron side and snierodz is working on the neutron classifier side
15:29:24 <davidsha> As far as patches I have 2 up, 1 for neutron_lib and one for neutron
15:29:41 <ralonsoh> btw, do you have some kind of fullstack or tempest test for this feature?
15:29:48 <ralonsoh> to see how it's working
15:30:19 <davidsha> Not added yet, I've been manually testing it.
15:30:27 <ralonsoh> ok
15:30:47 <davidsha> Neutron: https://review.openstack.org/#/c/636333/ Neutron-lib: https://review.openstack.org/#/c/636330/5
15:31:06 <davidsha> They're all on the same topic
15:31:08 <ralonsoh> yes, I added those two patches to the etherpad
15:31:17 <davidsha> thanks!
15:31:36 <ralonsoh> thank you!
15:31:53 <ralonsoh> any other topic to discuss?
15:32:38 <mlavalle> not from me
15:32:49 <ralonsoh> thank you all and see you in two weeks! And remember to add your topics to the PTG agenda!!
15:33:05 <ralonsoh> #endmeeting