18:01:48 <SumitNaiksatam> #startmeeting networking_policy 18:01:49 <openstack> Meeting started Thu Mar 15 18:01:48 2018 UTC and is due to finish in 60 minutes. The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot. 18:01:50 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 18:01:52 <openstack> The meeting name has been set to 'networking_policy' 18:01:54 <annakk> hi rkukura! 18:02:04 <SumitNaiksatam> (continuing the discussion about summit) 18:02:11 <SumitNaiksatam> annakk: did you have any other talks? 18:02:17 <annakk> no 18:02:48 <SumitNaiksatam> bummer, but thats fine, its still in your backyard so you can attend regardless :-) 18:03:06 <annakk> yes, hope to see you all there as well! 18:03:07 <SumitNaiksatam> #topic Master branch gate breakage 18:03:41 <SumitNaiksatam> so I think this change is something that is causing an issue: 18:03:48 <SumitNaiksatam> #link https://github.com/openstack/neutron/commit/4a0200a2d4e9fdedfc1b74c12330b3b4ee1a70ed 18:04:18 <SumitNaiksatam> i am wondering if we have to do the same thing in our project 18:05:17 <rkukura> SumitNaiksatam: seems likely 18:05:36 <SumitNaiksatam> so i will try it out, but hopefully the stable branches are not blocked (after anna’s patch was merged) 18:05:55 <SumitNaiksatam> *anna’s patches on the stable branches 18:06:25 <SumitNaiksatam> #topic Pending patches 18:06:40 <SumitNaiksatam> happy to see this new driver patch from anna’s team: 18:06:57 <SumitNaiksatam> #link https://review.openstack.org/552902 18:07:29 <SumitNaiksatam> annakk: if you feel comfortable we can quickly discuss this now 18:07:54 <SumitNaiksatam> but no pressure we can do it later as well on gerrit 18:08:13 <annakk> sure, just some context - the nsx policy platform is out with new version 18:08:36 <annakk> old version APIs were marked experimental, so we'll not support them with gbp going forward 18:08:58 <SumitNaiksatam> annakk: thats fine 18:09:17 <annakk> there were some significant changes on platform side, so we'll do some refactoring in gbp driver and in nsxlib which wraps platform apis 18:09:27 <SumitNaiksatam> annakk: okay 18:09:38 <SumitNaiksatam> annakk: you will be enabling the NSX gate job on the master branch? 18:10:25 <annakk> that's challenging, since gates are assuming that all components share same branch 18:10:35 <annakk> but hopefully we'll do that 18:10:37 <SumitNaiksatam> other option is you immediately backport to stable/pike while still in review 18:10:50 <SumitNaiksatam> that way we can see the result on stable/;pike 18:10:57 <annakk> yes, that's the plan for now 18:11:03 <SumitNaiksatam> okay cool 18:11:05 <SumitNaiksatam> that works 18:11:18 <rkukura> annakk: Is your plan to move the stable branches to the new API or leave them on the old API? 18:11:47 <SumitNaiksatam> my understanding is the old driver is being removed from the stable branches as well? 18:11:52 <annakk> pike will go with new one, probably ocata as well 18:12:20 <rkukura> annakk: ok 18:12:41 <SumitNaiksatam> i know rkukura is not going to like it, but perhaps if this is not disruptive to anyone i am fine removing it from stable branches as well 18:13:23 * tbachman wonders if there are are a lot of UTs for the old one 18:13:43 <annakk> few dozen UTs I think 18:13:56 <rkukura> I’m flexible 18:14:02 <tbachman> k — just thinking whether we’ll save much gate time there :) 18:14:04 <annakk> I think it should be easy enough to backport everything, hopefully we'll be only touching code under nsx* 18:14:05 <SumitNaiksatam> (and when i say “rkukura not going to like it” i mean its all for very valid reasons) 18:14:56 <SumitNaiksatam> the other big patch pending review is rkukura’s #link https://review.openstack.org/#/c/529991/ 18:15:05 <SumitNaiksatam> although that one is apic specific 18:15:10 <rkukura> right 18:15:42 * tbachman needs to review that patch, too 18:15:53 <SumitNaiksatam> rkukura: sorry for asking again, but you earlier mentioned that your UTs are triggering this for now 18:16:18 <rkukura> SumitNaiksatam: not sure what you are asking 18:16:18 <SumitNaiksatam> in a real deployment, how will this validation and repair kick in? 18:16:40 <rkukura> initial plan is for a CLI that is run manually, or could be run from cron, etc. 18:17:19 <SumitNaiksatam> so those dont need to be part of this patch and/or posted in parallel, so that say, we can exercise it from the AIM gate job? 18:17:23 <rkukura> I’m trying to get it to the point where validation would pass for a neutron-only (no GBP) deployment 18:17:54 <rkukura> I do think we will want to run it from the AIM job when it is ready to handle everything deployed in that job 18:18:33 <rkukura> I’d prefer to merge the initial version without the CLI, hopefully begining of next week 18:18:59 <rkukura> then add the CLI, and eventually call the CLI from the AIM gate job 18:19:23 <SumitNaiksatam> rkukura: so by that you mean, if GBP is triggering creation of Neutron resources, the current state of the tool will not be able to validate/repair those Neutron resources? 18:20:23 <rkukura> There are cases where GBP PD modifies AIM resources owned by Neutron resources, and those would fail 18:20:44 <SumitNaiksatam> okay 18:21:03 <rkukura> And where GBP creates AIM resources that would be unexpected from Neutron’s point of view 18:21:45 <rkukura> one point might be worth mentioning 18:22:29 <rkukura> there is a test_aim_validation UT class that tests that all the various AIM resource types can be validated and repaired 18:22:45 <SumitNaiksatam> okay 18:23:20 <SumitNaiksatam> rkukura: so the aim gate job runs an exercise script and cleans up the resources after that 18:23:24 <rkukura> But I am also adding _validate() calls to many existing AIM UTs. These don’t attempt to break/repair anything, just make sure everything passes validation 18:23:41 <SumitNaiksatam> so potentially you could your own testing after that 18:23:53 <SumitNaiksatam> *could add 18:24:08 <rkukura> These _validate() calls will add some execution time to those UTs, but I think this is better than duplicating complext scenarios in the test_aim_validation UTs 18:25:13 <rkukura> SumitNaiksatam: In the AIM gate job, I think the main goal would be to make sure things we already do pass validation, not necessarily to test repairs. That might be added later. 18:26:43 <SumitNaiksatam> so keeping in mind the running time of the py27 gate job, if adding this validation tips it over, it might be something you can consider adding to the aim gate job instead 18:27:19 <rkukura> SumitNaiksatam: Agreed, but we need to make sure UTs catch changes that would break validation when possible 18:27:32 <SumitNaiksatam> good point 18:27:52 <SumitNaiksatam> the other patches we have in review are related to the effort around integrating with SFC but still centered around apic 18:27:58 <SumitNaiksatam> these are mostly bug fixes 18:28:09 <SumitNaiksatam> #topic Open Discussion 18:28:23 <SumitNaiksatam> anything else we need to discuss today? 18:28:51 <annakk> not from me 18:28:56 <tbachman> I’m good 18:29:00 <SumitNaiksatam> i was working on optmizing the execution time of at least one of our critical functions 18:29:09 <SumitNaiksatam> had a good chat with rkukura on taht 18:29:11 <rkukura> SumitNaiksatam: Any chance we’ll be able to get either more CPU cores or more time for stable branch gate jobs? 18:29:30 <SumitNaiksatam> was surprised some of the unnecessary things we do 18:29:36 <SumitNaiksatam> will report more details next time around 18:29:57 <SumitNaiksatam> things will can run in milli seconds are running in the order of seconds 18:29:58 <SumitNaiksatam> anyway 18:30:07 <SumitNaiksatam> rkukura: honestly, i have never tried 18:30:32 <annakk> SumitNaiksatam: sounds promising! 18:30:59 <SumitNaiksatam> i have a physcological barrier asking that question :-( 18:31:46 <rkukura> SumitNaiksatam: We definitely treat the stable branches differently than other projects, so I understand 18:32:26 <SumitNaiksatam> rkukura: tbachman lets discuss that when you guys here, next week? 18:32:36 <tbachman> SumitNaiksatam: we’re there the following week 18:32:39 <tbachman> the week of the 25th 18:32:40 <rkukura> right 18:32:44 <SumitNaiksatam> ah okay, cool 18:32:53 <SumitNaiksatam> alrighty, lets wrap up for today then 18:32:56 <rkukura> Monday-Thurday for me 18:32:58 <SumitNaiksatam> thanks for joining 18:33:00 <rkukura> Thursday 18:33:02 <tbachman> rkukura: sam 18:33:04 <tbachman> same 18:33:06 <tbachman> SumitNaiksatam: thx! 18:33:10 <SumitNaiksatam> rkukura: tbachman great 18:33:13 <SumitNaiksatam> bye! 18:33:16 <SumitNaiksatam> #endmeeting