18:01:07 <SumitNaiksatam> #startmeeting networking_policy 18:01:07 <openstack> Meeting started Thu Sep 8 18:01:07 2016 UTC and is due to finish in 60 minutes. The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot. 18:01:09 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 18:01:11 <openstack> The meeting name has been set to 'networking_policy' 18:01:31 <SumitNaiksatam> #info agenda https://wiki.openstack.org/wiki/Meetings/GroupBasedPolicy#Sept_8th_2016 18:02:05 <rkukura> hi 18:02:13 <SumitNaiksatam> #topic Quality of Service support via NSPs 18:02:19 <SumitNaiksatam> #link https://review.openstack.org/#/c/301701 18:02:39 <SumitNaiksatam> igordcard: thanks for following up on this patch and getting the integration tests to pass 18:03:00 <SumitNaiksatam> i put some comments earlier today 18:03:09 <SumitNaiksatam> but this mostly looks good to me 18:03:23 <SumitNaiksatam> it does what we agreed to in the spec 18:03:24 <rkukura> SumitNaiksatam, igordcard I’m reviewing it, and will have at least one comment that will need to be addressed. 18:03:36 <SumitNaiksatam> rkukura: sure, thanks for the review 18:03:47 <SumitNaiksatam> rkukura: major issue? 18:03:55 <rkukura> just in the devstack config, so far 18:04:09 <SumitNaiksatam> rkukura: ah okay, sure 18:04:20 <SumitNaiksatam> i did go through that 18:05:02 <rkukura> The way its setting ml2 extension_drivers will break apic_aim, and not allow overrding in local.conf. 18:05:15 <SumitNaiksatam> rkukura: ah okay, i did not think of that 18:05:24 <SumitNaiksatam> good point 18:05:53 <rkukura> SumitNaiksatam: This is one example of why we need to get CI running with ENABLE_APIC_AIM, as well as without. 18:05:55 <igordcard> thanks, I'll address that 18:06:10 <SumitNaiksatam> rkukura: yes, i need to add the job 18:06:14 <rkukura> igordcard: I have a recommendation in the comment 18:06:17 <SumitNaiksatam> should have done it before 18:06:56 <SumitNaiksatam> igordcard: not sure you got a chance to see my comments 18:07:31 <SumitNaiksatam> songole: hi, thanks for joining 18:07:39 <igordcard> sorry SumitNaiksatam not yet 18:07:50 <songole> hi 18:08:00 <igordcard> I-m checking now 18:08:10 <SumitNaiksatam> igordcard: okay no problem, i think the only ones which really need fixing are FK name, and the on-delete constraint 18:08:13 <SumitNaiksatam> igordcard: sure 18:08:15 * songole multiplexing. wil be slow to respond.. 18:08:22 <SumitNaiksatam> songole: np 18:08:38 <SumitNaiksatam> songole: we are discussing igordcard’s qos patch 18:08:45 <SumitNaiksatam> songole: which mostly looks good to me 18:09:07 <songole> ok. will review today/tomorrow 18:09:14 <SumitNaiksatam> songole: thanks 18:09:32 <SumitNaiksatam> that patch is actually nicely done and easy to review 18:10:28 <igordcard> oh good catch SumitNaiksatam 18:11:18 <igordcard> yeah I'll address all comments tomorrow 18:11:26 <SumitNaiksatam> igordcard: np 18:11:37 <SumitNaiksatam> anything else on igordcard’s patch? 18:12:40 <SumitNaiksatam> #topic Open Discussion 18:12:57 <SumitNaiksatam> songole: the NFP patches are still pending review 18:13:01 <SumitNaiksatam> songole: any blockers? 18:13:27 <songole> did anyone get a chance to review the patches 18:13:38 <SumitNaiksatam> songole: sorry, i did not 18:13:55 <rkukura> I have not either, sorry 18:13:57 <SumitNaiksatam> songole: but i thought you and hemanthravi were handling these set of patches? 18:14:53 <SumitNaiksatam> songole: let us know if there are any blockers 18:15:07 <SumitNaiksatam> rkukura: anything else you want to discuss today? 18:15:09 <songole> we have tested on neutron. We wanted to test on ACI setup before merging it 18:15:28 <rkukura> SumitNaiksatam: nothing from me today 18:15:30 <SumitNaiksatam> songole: okay, i think that is sane thing to do, so i appreciate you not merging :-) 18:16:06 <SumitNaiksatam> alright, then we will wrap up for today 18:16:13 <SumitNaiksatam> thanks all for joining! 18:16:16 <SumitNaiksatam> bye 18:16:20 <SumitNaiksatam> #endmeeting