18:02:41 <SumitNaiksatam> #startmeeting networking_policy
18:02:42 <openstack> Meeting started Thu Feb  9 18:02:41 2017 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:02:43 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:02:46 <openstack> The meeting name has been set to 'networking_policy'
18:02:56 <tbachman> SumitNaiksatam: hi!
18:03:11 <SumitNaiksatam> #info agenda #link https://wiki.openstack.org/wiki/Meetings/GroupBasedPolicy#Feb_9th_2017
18:03:37 <igordcard> hi SumitNaiksatam
18:03:43 <SumitNaiksatam> igordcard: ah right on cue
18:03:55 <SumitNaiksatam> #topic QoS patch in master
18:04:13 <SumitNaiksatam> #link https://review.openstack.org/#/c/426436
18:04:38 <SumitNaiksatam> igordcard: so have the issues been resolved regarding the NFP gate job?
18:04:52 <rkukura> hi
18:05:01 <songole> hello
18:05:01 <SumitNaiksatam> igordcard: we know that the NFP job has issues beyond the changes in your patch, since it times out
18:05:04 <SumitNaiksatam> songole: hi
18:05:30 <igordcard> SumitNaiksatam: ashutosh was working on the nfp enablement
18:05:41 <SumitNaiksatam> igordcard: ah okay, in the other patch
18:05:51 <igordcard> #link https://review.openstack.org/#/c/429523/
18:05:57 <SumitNaiksatam> igordcard: so my understanding is that the other patch is for experimentation?
18:06:00 <songole> SumitNaiksatam: ashutosh has found a few issues
18:06:06 <SumitNaiksatam> songole: okay
18:06:34 <igordcard> SumitNaiksatam: yeah I think so..
18:08:01 <igordcard> qos needs to be enabled in local-nfp.conf too though, it seems
18:08:11 <SumitNaiksatam> songole: igordcard: it would have been better if the ashutosh’s patch was made dependent on igordcard’s patch
18:08:16 <igordcard> so that is at least one more change to make on my patch
18:08:53 <SumitNaiksatam> as it stands now, its a cherry-pick and its kind of difficult to tell what are the changes made in addition to igordcard’s changes
18:09:03 <SumitNaiksatam> igordcard: unless you are keeping a track :-)
18:09:13 <songole> SumitNaiksatam: okay
18:09:37 <SumitNaiksatam> songole: i am not suggesting that we redo, but a note for the future
18:09:48 <igordcard> SumitNaiksatam: I haven't been following closely, but my impression is that ashutosh's patch can completely replace mine as soon as it passses dsvm-nfp
18:10:13 <SumitNaiksatam> songole: perhaps ashutosh is keeping track of the changes he is making and is going to relay that to igordcard via comments on igordcard’s patch (which i believe is what is currently happening)
18:10:16 <igordcard> the equivalent of my changes were done on the cherry-pick too
18:10:50 <SumitNaiksatam> igordcard: hmmm
18:11:28 <SumitNaiksatam> igordcard: i will leave it to you and ashutosh to decide, but let the team know which patch they should review
18:11:31 <igordcard> SumitNaiksatam: as soon as I see dsvm-nfp green I will discuss with ashutosh the next steps
18:11:44 <igordcard> SumitNaiksatam: will do
18:12:04 <SumitNaiksatam> igordcard: sounds good, so right now we are waiting for ashutosh to identify the changes
18:12:21 <igordcard> SumitNaiksatam: I'll leave a comment on his patch too now
18:12:26 <SumitNaiksatam> igordcard: thanks
18:13:15 <SumitNaiksatam> i had reviewed this patch earlier and had made changes on top of it, so its little painful to review the cherry-pick now because we lose context of the original patch
18:13:33 <SumitNaiksatam> perhaps doing a diff between the patchsets will help (assuming no rebase has been done :-))
18:13:59 <SumitNaiksatam> anwyay, igordcard anything else we need to discuss on this?
18:14:14 <igordcard> SumitNaiksatam: I don't mind pulling the cherry-pick and applying the changes to my latest patchset, and submitting as a new patchset so we can view the diff
18:14:24 <SumitNaiksatam> igordcard: that works
18:14:47 <SumitNaiksatam> igordcard: i right away saw one issue with how the UT setup was being changed in ashutosh’s patch
18:14:53 <SumitNaiksatam> i will add a note
18:14:53 <igordcard> SumitNaiksatam: but only when I see dsvm-nfp green or ashutosh patch is otherwise ready
18:15:17 <SumitNaiksatam> igordcard: the problem is that the nfp job is failing on other patches as well on account of time out
18:15:39 <SumitNaiksatam> songole: so on that, did you check with the team on removing the test we discussed?
18:15:45 <SumitNaiksatam> test -> scenario
18:16:30 <songole> SumitNaiksatam: yes. But, ash says having both would catch more issues
18:16:48 <SumitNaiksatam> songole: sure but right now its not catching any because it does not complete :-)
18:16:49 <songole> such as clean up issues with the previous firewall..
18:17:12 <songole> he thinks he knows the issues..
18:17:15 <SumitNaiksatam> songole: i would also prefer to not remove any scenarios
18:17:26 <songole> or understands what is happening
18:17:28 <SumitNaiksatam> songole: okay, any chance he has a ETA?
18:17:44 <SumitNaiksatam> songole: because as we discussed, currently it makes the process very slow of all other patches
18:18:13 <SumitNaiksatam> songole: we need to have an interim solution if its going to take longer to get to a solution
18:18:30 <SumitNaiksatam> solution -> final solution
18:18:56 <songole> SumitNaiksatam: I will get back on ETA
18:19:04 <SumitNaiksatam> songole: thanks much
18:19:19 <SumitNaiksatam> igordcard: songole: thanks for the update here
18:19:21 <SumitNaiksatam> moving on
18:19:30 <SumitNaiksatam> #topic Application Policy Group
18:19:51 <SumitNaiksatam> we had discussed this spec before #link https://review.openstack.org/339272
18:20:11 <SumitNaiksatam> but its still pending votes/approval
18:20:36 <SumitNaiksatam> i have also posted a WIP implementation patch: #link https://review.openstack.org/430616
18:20:52 <songole> SumitNaiksatam: will review the spec
18:20:55 <SumitNaiksatam> there are some changes in the impl which i have not pushed to gerrit
18:20:57 <SumitNaiksatam> songole: thanks
18:21:12 <SumitNaiksatam> the spec is pretty short and the idea is pretty basic
18:21:59 <SumitNaiksatam> we mostly expect the clients to use this, however the apic driver will also have a specific back-end mapping for this construct
18:22:31 <SumitNaiksatam> rkukura: tbachman igordcard: in case you get a chance please take a look again
18:22:37 <tbachman> SumitNaiksatam: ack
18:22:38 <SumitNaiksatam> on the spec and impl patch
18:23:16 <SumitNaiksatam> to be clear, this is a new resource being added to the GBP model, its not a driver-specific extension
18:23:26 <rkukura> SumitNaiksatam: I will look again, but did we just have a discussion about this?
18:23:29 <SumitNaiksatam> hence everyone should be aware
18:24:01 <SumitNaiksatam> rkukura: we had discussions in the past, but the spec is still un-approved
18:24:18 <igordcard> [previous topic] I've reverted my own qos patch to the same state as before ash's fork, to prevent confusion
18:24:22 <SumitNaiksatam> the implemention patch will be out of WIP soon
18:24:29 <SumitNaiksatam> igordcard: nice, thanks
18:25:34 <SumitNaiksatam> #topic Open Discussion
18:25:44 <SumitNaiksatam> anyone have anything else to bring up today?
18:26:03 <SumitNaiksatam> songole: this following patch was posted: #link https://review.openstack.org/#/c/431350/
18:26:11 <SumitNaiksatam> regarding adding port-security
18:26:26 <SumitNaiksatam> but it fails the nfp gate for the time out reason
18:26:39 <SumitNaiksatam> should we wait for the gate job to be fixed to merge this?
18:26:57 <SumitNaiksatam> (merge after review, that is)
18:27:22 <songole> SumitNaiksatam: i think this is the fix ash was proposing. looks like there is still some work that needs to be done
18:27:51 <songole> this is regarding nfp gate job failure
18:27:53 * tbachman saw a patch fly by the other day on GBP from someone named Ann?
18:28:24 <SumitNaiksatam> tbachman: yes we need to review that
18:28:40 <SumitNaiksatam> tbachman: she is a new member of the GBP community, from VMW
18:29:08 <SumitNaiksatam> songole: okay, but its completely removing the fw_lb.sh exercise script
18:29:33 <SumitNaiksatam> tbachman was referring to this patch #link https://review.openstack.org/#/c/431244/
18:30:01 <SumitNaiksatam> tbachman: so to give you context, Anna was integration the resource mapping driver on top of the VMW monolithic neutron plugin
18:30:09 <SumitNaiksatam> tbachman: and she ran into this issue
18:30:32 <tbachman> SumitNaiksatam: cool - thx (and great to hear that we have a new member to the community :-) )
18:30:39 * tbachman wonders if Ann knows about our IRC meetings
18:30:55 <rkukura> SumitNaiksatam: how is the newton work going?
18:31:01 <SumitNaiksatam> tbachman: good point, i think she does
18:31:18 <SumitNaiksatam> rkukura: i am trying to wrap up the mitaka stuff before resuming the newton sync
18:31:27 <rkukura> ok
18:31:31 <SumitNaiksatam> rkukura: i am stuck in the newton sync with one wierd issue
18:31:42 <SumitNaiksatam> rkukura: the aim tables are not getting created in the UTs
18:32:23 <SumitNaiksatam> rkukura: i spent quite a bit of time on that until i had to context switch sometime in the middle of last week, i havent been able to get back to that after it
18:32:42 <rkukura> SumitNaiksatam: let me know if you’d like me to take a look when you get back to this
18:32:43 <SumitNaiksatam> but yeah, the newton sync is the big priority right now
18:33:17 <SumitNaiksatam> rkukura: yes, once i hit a complete wall i will ping you (with a rundown of the symptoms)
18:33:30 <rkukura> ok
18:34:11 <SumitNaiksatam> rkukura: you  are attending the PTG in atlanta, right?
18:34:21 <rkukura> SumitNaiksatam: right
18:34:26 <SumitNaiksatam> is anyone else going?
18:34:42 <songole> SumitNaiksatam: what is ptg?
18:35:50 <SumitNaiksatam> #link https://www.openstack.org/ptg/
18:35:59 <SumitNaiksatam> Project Teams Gathering (PTG)
18:36:03 <SumitNaiksatam> i had to look it up :-)
18:36:33 <songole> SumitNaiksatam: thanks :)
18:36:36 <SumitNaiksatam> this is the new design summit (separate from the conference, which will be in Boston later)
18:36:49 <SumitNaiksatam> totally stole our acronym there!
18:37:14 <SumitNaiksatam> tbachman: igordcard: are you going?
18:37:22 <tbachman> SumitNaiksatam: unfortunately not
18:37:47 <SumitNaiksatam> tbachman: okay, would have been a short flight
18:38:16 <SumitNaiksatam> alrighty, if nothing else lets wrap up for today
18:38:23 <SumitNaiksatam> thanks all for joining
18:38:28 <SumitNaiksatam> #endmeeting