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