18:00:26 <SumitNaiksatam> #startmeeting networking_policy
18:00:27 <openstack> Meeting started Thu Jun 30 18:00:26 2016 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:00:28 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:00:30 <openstack> The meeting name has been set to 'networking_policy'
18:00:47 <SumitNaiksatam> #info agenda  https://wiki.openstack.org/wiki/Meetings/GroupBasedPolicy#June_30th.2C_23rd_2016
18:00:55 <tbachman> SumitNaiksatam: hi!
18:01:06 <igordcard> hi
18:01:20 <SumitNaiksatam> does anyone have anything critical to bring up before we dive into the NFP patches?
18:01:56 <SumitNaiksatam> #topic NFP Implementation patches
18:02:10 <SumitNaiksatam> #link https://review.openstack.org/#/q/topic:bp/gbp-network-services-framework
18:02:17 <rkukura> hi
18:03:07 <SumitNaiksatam> we are currently reviewing patches until: #link https://review.openstack.org/#/c/309145/
18:03:38 <SumitNaiksatam> hemanthravi: thanks to you and the other authors for responding to the review comments
18:04:08 <hemanthravi> most of the review comments have been addressed, currently gate tests are broken and this shoudld be resolved today
18:04:09 <SumitNaiksatam> i think a lot of the pending comments have been addressed
18:04:53 <SumitNaiksatam> hemanthravi: in some cases i noticed that it was mentioned that the comment wil be addressed as a follow bug
18:04:55 <hemanthravi> the dependency chain seems to be broken with one of the recent submit, will resolve this after the irc
18:05:32 <SumitNaiksatam> hemanthravi: the expectation is that the bug be filed, and the bug link be provided in the patch comments
18:05:33 <ivar-laz_> hi
18:05:53 <hemanthravi> ok, will do that
18:05:57 <SumitNaiksatam> hemanthravi: otherwise its difficult to track what was addressed and what needs to be addressed as follow up bugs
18:06:11 <SumitNaiksatam> hemanthravi: i did not see any follow up bugs being filed
18:06:42 <hemanthravi> the team is keeping track of the bugs to be filed, will file these bugs and edit the comment with the bug
18:07:27 <SumitNaiksatam> hemanthravi: great, thanks
18:08:15 <SumitNaiksatam> hemanthravi: my comments in patch set 120 have not been responded to: #link https://review.openstack.org/#/c/309145/120
18:08:31 <SumitNaiksatam> hemanthravi: have these been fixed?
18:10:46 <hemanthravi> these are not fixed yet
18:10:51 <SumitNaiksatam> hemanthravi: okay
18:11:37 <SumitNaiksatam> hemanthravi: until which patch are all the comments completely fixed, or is it that there are few here and there?
18:12:51 <hemanthravi> most of the comments have been addressed, the gate test comments are pending since the failure in the gate test was being resolved
18:12:52 * SumitNaiksatam anticipates this is going to be a slow meeting since we are reviewing patches in real time!
18:13:04 <SumitNaiksatam> hemanthravi: okay
18:13:20 <hemanthravi> all the gate test comments will be addressed
18:14:03 <SumitNaiksatam> hemanthravi: when do you think you will get to that point?
18:14:33 <hemanthravi> should be done fri PST
18:14:40 <hemanthravi> morning
18:14:45 <SumitNaiksatam> hemanthravi: okay great
18:15:35 <hemanthravi> once these are resolved, can we start the merge fri or sat PSt
18:16:04 <SumitNaiksatam> hemanthravi: i am good with that tentative plan, what does the rest of the team think?
18:16:41 <SumitNaiksatam> we might have a good window when the gate is hopefully less loaded over the next few days
18:16:52 <hemanthravi> the main pending ones are to have the gate tests running and the validations added in the gate tests
18:17:23 * tbachman is fine with the plan
18:17:32 <SumitNaiksatam> hemanthravi: yes i am definitely hoping to see the integration test validations
18:17:39 <SumitNaiksatam> tbachman: thanks for the feedback
18:17:44 <tbachman> SumitNaiksatam: sure :)
18:18:07 <hemanthravi> thanks
18:18:30 <SumitNaiksatam> rkukura: ivar-laz_: igordcard: songole: what do you guys think about the plan to start merging tomorrow if the fixes are in place by then?
18:19:00 <songole> SumitNaiksatam: Yes to the merge!!
18:19:12 <SumitNaiksatam> songole: okay
18:19:20 <rkukura> SumitNaiksatam: I’m not opposed, but I have not been doing much reviewing of this code, so I’ll defer to others.
18:19:27 <SumitNaiksatam> rkukura: sure
18:20:02 <igordcard> I'm okay with it
18:20:10 <SumitNaiksatam> igordcard: okay, thanks
18:20:41 <rkukura> I will be pretty much completely offline from tomorrow mid afternoon east coast time until sometime Sunday.
18:20:54 <ivar-laz_> SumitNaiksatam: I'll be on PTO until the 5th, but it sounds good
18:21:06 <SumitNaiksatam> rkukura: thanks for informing us
18:21:10 <SumitNaiksatam> ivar-laz_: thanks for the note
18:21:32 <SumitNaiksatam> so looks like there are no major objections in the team
18:21:38 <tbachman> SumitNaiksatam:  one question
18:21:43 <tbachman> are these only going into master?
18:21:44 <SumitNaiksatam> tbachman: yes sure
18:21:48 <tbachman> or are they going to be backported?
18:21:54 <SumitNaiksatam> tbachman: good question
18:21:56 * tbachman guesses just master
18:22:02 <tbachman> is this considered a new feature?
18:22:16 <SumitNaiksatam> those are tought questions :-P
18:22:19 <tbachman> lol
18:22:19 <rkukura> tbachman: this is GBP!
18:22:20 <SumitNaiksatam> *tough
18:22:20 <songole> they need to be backported as well
18:22:26 <SumitNaiksatam> rkukura: lol!
18:22:35 <tbachman> rkukura: ACK — just wasn’t sure what our policy was :)
18:22:50 <SumitNaiksatam> tbachman: so here is  dimension to this -
18:22:51 * tbachman stirs the pot :P
18:23:18 <SumitNaiksatam> almost all the code that is being introduced by these patches is additive, and is contained new modules
18:23:24 <SumitNaiksatam> *in new
18:24:05 <SumitNaiksatam> hemanthravi: songole: please correct if this is not accurate
18:24:33 <hemanthravi> that's correct, except for one patch in gbp code
18:24:57 <songole> 99% yes. There are a couple of lines of change in 1 file
18:25:46 <hemanthravi> backport is required as the most of the users are moving upto liberty at this stage
18:26:22 <tbachman> so, just to liberty?
18:26:28 * tbachman is fine with that
18:27:37 <songole> tbachman: yes
18:27:40 <rkukura> Is the plan to back-port as we go, or wait til we get to some level of confidence on master before we start back-porting?
18:27:43 <tbachman> songole: thx!
18:27:56 <SumitNaiksatam> rkukura: that is a good point
18:28:13 <SumitNaiksatam> so are we saying that we are definitely not backporting to kilo?
18:28:20 <hemanthravi> currently there is a requirement to backport to liberty for some of the evals
18:28:47 <hemanthravi> not sure on kilo, will check on this
18:29:13 <SumitNaiksatam> hemanthravi: kilo is kind of EoL so its difficult to do this
18:29:48 <hemanthravi> ok
18:30:28 <SumitNaiksatam> so the easier things first, we will try to merge in master, and mitaka backport is automatic since our master is still at mitaka
18:30:37 <SumitNaiksatam> hopefully no issues with that
18:30:53 <SumitNaiksatam> we are saying that we will not do kilo backports
18:31:04 <SumitNaiksatam> that leaves liberty
18:31:16 <hemanthravi> we have also a local backport to liberty that we have been testing
18:31:27 <SumitNaiksatam> and the suggestion over there is that we first test liberty
18:31:31 <hemanthravi> the changes are minor for the liberty backport
18:31:31 <SumitNaiksatam> hemanthravi: ah right on cue
18:32:40 <SumitNaiksatam> so lets first merge master and mitaka backport, and then have the liberty backport patches posted
18:33:07 <SumitNaiksatam> we can then test the stability of the master and mitaka branches before we make a call on merging in liberty
18:33:13 <SumitNaiksatam> does that sound okay to all?
18:33:22 * tbachman nods
18:33:25 <hemanthravi> mostly with i18n being _i18n and some constants for topics in a differnt location
18:33:31 <SumitNaiksatam> tbachman: thanks
18:34:06 <SumitNaiksatam> hemanthravi: yes, that is true but i guess the concern is about the stability of the master branch once this merges
18:34:42 <SumitNaiksatam> we want to be sure that merging does not majorly disrupt the master/mitaka
18:34:46 <SumitNaiksatam> branches
18:34:57 <hemanthravi> ok, as you said we can wait for the stability on master/mitaka
18:35:21 <SumitNaiksatam> hemanthravi: yeah lets get to the point first where have those merged, and then make a call accordingly
18:37:00 <SumitNaiksatam> okay, seems like we are mostly in agreement, and have a reasonable plan moving forward
18:38:09 <SumitNaiksatam> #topic Open Discussion
18:38:25 <SumitNaiksatam> thats pretty much from me for today
18:38:29 <SumitNaiksatam> anyone have anything else?
18:38:38 <igordcard> don't have anything from my side
18:38:47 <SumitNaiksatam> igordcard: QoS next week?
18:39:07 <igordcard> SumitNaiksatam: not sure
18:39:11 <SumitNaiksatam> igordcard: okay
18:39:16 <SumitNaiksatam> igordcard: best effort :-)
18:40:29 <SumitNaiksatam> the deadline for submissions for the Barcelona summit is July 13th
18:40:44 <SumitNaiksatam> fast approaching
18:41:11 <SumitNaiksatam> all right, thanks all for joining, and happy 4th of July to those in the US!
18:41:24 <SumitNaiksatam> bye
18:41:28 <rkukura> SumitNaiksatam: thanks!
18:41:31 <tbachman> SumitNaiksatam: bye!
18:41:33 <rkukura> bye
18:41:34 <igordcard> bye
18:41:34 <hemanthravi> bye
18:41:40 <SumitNaiksatam> #endmeeting