16:02:20 #startmeeting vpnaas 16:02:20 Meeting started Tue Sep 1 16:02:20 2015 UTC and is due to finish in 60 minutes. The chair is pc_m. Information about MeetBot at http://wiki.debian.org/MeetBot. 16:02:21 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 16:02:24 The meeting name has been set to 'vpnaas' 16:02:34 #topic Announcements 16:02:47 o/ 16:02:57 The devstack plugin for VPN has been renamed to neutron-vpnaas (death to q-* :) 16:03:37 The DB models have been separated from DBase logic 16:04:14 Coverage tests have been broken for a while - pushed out commit to fix #link https://review.openstack.org/#/c/217847 16:04:34 Any other announcements, before we hit the agenda items? 16:04:44 Hi 16:04:48 xgerman: ajmiller: hi 16:05:07 #topic Endpoint Groups 16:05:28 So, this is a pre-requisite for the multiple local subnet feature. 16:05:53 Code is out for review, please review it as soon aas possible. Would like to get this in. #link https://review.openstack.org/#/c/212692/ 16:06:08 I'll be working on neutron-client support for this next. 16:06:36 #topic VPN Functional Testing for Neutron Commits 16:06:51 Wish we had this to prevent breakages :( 16:07:13 I talked to Doug, and he suggested I try something, but as of this morning, it failed too. 16:07:30 The tests are not using the neutron patchset that is being tested. 16:07:43 Will circle back with dougwig on this. 16:07:52 just to understand, do we want seperate api job for testing them? 16:08:49 madhu_ak: This is an effort to add a pair of neutron jobs that will run the VPN functional (scenario) tests on any Neutron commit. 16:09:02 Goal is to prevent VPN breakage by Neutron code changes. 16:09:14 gotcha, thanks 16:09:38 The VPN API tests are being moved to neutron-vpnaas as a separate effort (I forget who is on that) 16:09:58 its me ;) 16:10:24 need to circle back with dougwig again 16:10:40 madhu_ak: So this job takes a neutron commit, runs VPN test, stacking and verifies the scenario tests run. Currently, it is doing that, but using the latest from master for neutron change set 16:10:46 madhu_ak: Nice. 16:11:02 #topic Neutron client and VPN devstack plugin 16:11:45 At the Neutron IRC we discussed. Will have two jobs. One that runs the core tests for neutron client, and one that runs tests for advanced services and enables VPN plugin (for now). 16:11:59 I have the project-config and neutronclient code up for review. 16:12:12 Cannot test neutronclient changes, until the job is upstreamed. 16:12:38 #topic Bugs 16:12:47 https://bugs.launchpad.net/neutron/+bugs?field.searchtext=vpnaas&search=Search&field.status%3Alist=NEW&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=FIXCOMMITTED&field.assignee=&field.bug_reporter=&field.omit_dupes=on&field.has_patch=&field.has_no_package=&orderby= 16:12:47 status&start=0 16:12:58 #link https://goo.gl/XNtnLX 16:13:09 nice 16:13:21 These are the bugs we have. All but 2-3 have assignees. 16:13:50 Here are the open reviews #link https://review.openstack.org/#/q/status:open+project:openstack/neutron-vpnaas,n,z 16:14:14 Please help out reviewing. The more +1s we get, the more confidence cores will have to +2/+A these. 16:14:24 We would really like to get reviews on https://review.openstack.org/#/c/216812/. 16:14:46 +1 16:14:53 That fix is destined for stable/kilo, so it needs to get merged into master, then I will be cherry-picking it 16:15:04 ajmiller: Yeah, looks like you mostly need another core. 16:15:44 Plug for my commit... #link https://review.openstack.org/212692 16:16:18 The corresponding dev ref doc review is #link https://review.openstack.org/191944 16:16:44 Anyone have any other bugs they want to bring up to the group? 16:17:54 #topic Open Discussion 16:18:19 Anyone have anything else to discuss? 16:18:33 Just an update.. 16:18:36 * pc_m this will be a short meeting... 16:18:41 Aish: sure 16:19:05 I am about to complete the basic functionalities in teh rally scenario tests.. 16:19:17 Will be addressing your comments in my next patch. 16:19:29 Aish: OK great 16:19:41 pc_m do you think those tests can land in Liberty? 16:20:23 xgerman: good question... I guess they don't have strict deadlines anymore, so maybe it's just a matter of it getting upstreamed in time. 16:20:41 yeah, that’s what I am thinking as well 16:20:46 xgerman: I think the biggest issue is getting core reviewer attention on all things VPN/ 16:21:04 yeah, dougwig... 16:21:48 Which is why I'm pleading with people to review so that cores have a warm fuzzy that the team feels the commits are ready. 16:22:13 Especially as they may not be domain experts. 16:22:31 agreed - on the other had we can ask for more core reviewers ;-) 16:22:47 xgerman: good luck 16:23:23 They are watching the review stats...so if people consistently review VPN code, they could become core reviewers... 16:23:32 exactly!! 16:23:52 Historically, that has been a problem in VPN land. 16:24:13 well, we had an influx of new people so maybe that will get better ;-) 16:24:23 * pc_m fingers crossed 16:24:34 * pc_m and toes 16:25:02 Anyone have any other questions, comments, and/or concerns regarding VPNaaS? 16:25:23 I would love to add cores. :) 16:25:47 dougwig: yeah! Just mentioned we need people to review. 16:25:59 guess we are running through an open door :-) 16:26:44 dougwig: BTW, disabling q-lbaas didn't seem to solve the issue with VPN functional tests running on neutron commits. 16:27:24 #link https://review.openstack.org/#/c/196149/ 16:27:46 dougwig: Let me know when you have some time to discuss. OK? 16:29:09 OK. I guess we can wrap up the meeting. I'll try to work with dougwig on the functional test issue. 16:29:24 Thanks for joining in everyone! 16:29:29 #endmeeting