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