15:01:21 #startmeeting bgpvpn 15:01:22 Meeting started Tue Jun 2 15:01:21 2015 UTC and is due to finish in 60 minutes. The chair is tmorin. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:01:23 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:01:26 The meeting name has been set to 'bgpvpn' 15:01:33 hi 15:01:38 * pc_m in another meeting...lurking here 15:01:43 hi guys 15:01:45 #link https://wiki.openstack.org/wiki/Meetings/BGPVPN 15:01:49 hi paul, hi petr 15:01:55 hi vikram 15:02:05 hi folks 15:02:10 hi tim 15:02:24 the link is a proposed agenda 15:04:11 hi 15:04:21 sorry fot the delay :) 15:04:37 not sure who else will come, we can wait 1/2 minute and start I guess 15:04:52 many of those present to discuss this in Vancouver are here already 15:05:29 hi mat, hi ildiko 15:06:25 hi 15:07:10 hi paul 15:07:24 hi 15:07:30 hi pc_m 15:07:58 hi folks. I'm in a conf call, so will try to keep one eye on this too :) 15:08:19 ok, I guess we can start 15:08:26 thanks everyone for being present 15:08:36 there is a propsed agenda at https://wiki.openstack.org/wiki/Meetings/BGPVPN 15:08:51 don't hesitate to suggest additional points 15:08:55 first a status 15:09:11 there is an ongoing discussion on the proposed API 15:09:23 this is at https://review.openstack.org/#/c/177740 15:09:38 there are a few pending things to discuss later today 15:09:59 and there is a discussed to progress on how/when to converge with other proposal in VPNaaS 15:10:16 we'll have this discussion later today during the VPNaaS meeting 15:10:28 (16:00 UTC #openstakc-meeting-3) 15:10:42 tmorin: how, when, and if it makes sense 15:10:51 pc_m: yes 15:11:00 the other thing on which we can update the statusis the networking-bgpvpn project 15:11:07 it currently is a stackforge project 15:11:30 and we made, last week, a request for making it an openstakc project under the neutron big tent 15:11:36 https://review.openstack.org/#/c/186041 15:11:54 #link https://review.openstack.org/#/c/186041 15:12:07 this request is being progressed, and as we understand, simply pending the ultimate validation 15:13:06 Great Work:) 15:13:49 if this is validated, as we hope, we will also move the API spec there and abandon the change for Liberty specs discussed in 177740 15:13:50 nice 15:14:13 yes :) 15:14:17 tmorin, any relation and/or comments regarding Mohammad's https://review.openstack.org/#/c/152377/ ? 15:14:53 I feel it's on the agenda :"discussion to follow on possible convergence with other related proposal: during VPNaaS meeting (16:00 UTC #openstack-meeting-3" 15:15:04 yes, this is the idea 15:15:05 vikram : +1 15:15:17 for now we haven't see a clear convergence in the targets 15:15:33 we shoudl try hard ton converge 15:15:36 tmorin: +1 15:15:49 the use cases are very similar 15:16:08 john_a_joyce: well, that does not seem obvious, but as we say --> up for discussion later today 15:16:17 would be good to understand the use cases for each of the two. 15:16:43 tmorin: at a high level I don't see much a difference 15:16:44 pc_me: we hope to have properly documented the usecase in the spec in 177740, but comments are welcome 15:16:57 sure in the details there are differences 15:17:16 tmorin: I'll check it out more 15:17:17 I think the discussion on this should involve Mohamad and others 15:17:27 * pc_m trying to wrap my head around both of these :) 15:17:27 let's discuss this later today 15:17:37 tmorin: makes sense - lets leave it for now 15:17:39 you tell us how we can help 15:17:40 ok 15:17:50 actually, i also want to see both the API together :) 15:18:02 #topic discuss on BGP VPN API, pending items 15:18:31 (apart from question related to difference/convergence with other proposals) 15:18:42 the spec has been updated to address comments made 15:18:50 there aren't much pending points 15:19:06 we steel need to describe in more detail how we want to use the "type" field 15:19:24 how much in detail we want this field to indicate a particular technology 15:19:42 the example being IP VPN with RFC4364, and IP VPN with E-VPN Prefix routes 15:20:24 tmorin, maybe just write in the type field the rfc number? 15:20:41 like type = 'rfc4364' 15:21:00 yes, with some adaptation for specs which aren't rfcs yet 15:21:25 'rfcXXXX+extension' 15:21:40 we had talked about "l2:", "l3:" suffixed by some text indicating the technique 15:21:55 anyhow the API spec will have to list the values and document what they correspond too 15:22:09 the format itself is not that important 15:22:09 yep 15:23:05 then can we just make an initial list of types to submit, and later expand it, if needed? 15:23:21 (leaving current types intact, sure) 15:23:42 i think the inital list could be "l3:rfc4364", "l2:evpn", "l3:evpn-type5" 15:23:57 ack 15:24:32 and we also need to document the API for listing the types, but this one is quite trivial 15:24:53 we'll return a json list of the type strings e.g. [ "l3:rfc4364", "l2:evpn", "l3:evpn-type5" ] 15:25:34 do we really want to name as "l3:rfc4364" 15:25:52 #action tmorin to update spec for type field 15:26:01 maybe not 15:26:17 tmorin, just to be clear: list of types, really supported by the running implementation 15:26:18 ok .. 15:26:33 ipvpn could be good enough, and the API documentation would say 'this means RFC4364' 15:26:42 "ipvpn" is more use friendly 15:27:03 petr: yes, list of supported types 15:27:08 * svinota thinks that an rfc reference is friendly enough 15:27:22 ;) 15:27:30 :) 15:27:39 ok we can keep this open for now 15:28:09 so the API should ask the driver to get supported types 15:28:13 can we have separate section for l2 and l3 and then stating sub-types? 15:28:26 matrohon: yes 15:28:32 what do you mean by section ? 15:28:43 vikram, section == ? 15:28:49 hi carl 15:29:32 i mean one string stating both types and sub-types or two diff arguments for types and su-types separately 15:30:01 am I clear? 15:30:04 yes 15:30:11 this is not a bad idea 15:30:18 there is a reason to state explicitly arguments, agree 15:30:33 not to require any implicit string parsing 15:30:34 one (l2/l3) can be exposed to the tenant, and the other hidden 15:30:36 ok 15:31:03 and we can indicate that the second one can be ommitted, up to the driver to use its default 15:31:35 not sure, if we should hide anything from tenants, but explicit things are better sometimes. 15:31:49 tmorin, just notice, that the list call will take an argument then 15:32:06 yes indeed 15:32:15 tmorin, like list('l2') -> ['evpn'] 15:32:26 yes 15:32:36 svinota: +1 15:32:52 ok lets agree on that and move to the next topic 15:32:58 ack 15:33:09 ok 15:33:25 there is another pending point on whether we should let the API user specify a VXLAN id when VXLAN is used as an encap 15:33:47 diego was the one suggesting that, I suggest we wait for a meeting where he is present to address this 15:34:09 #topic networking-bgpvpn, work items 15:34:33 so we have a few pieces of work on which we'll need to work 15:34:49 moving the API on the new networking-bgpvpn project is one 15:34:52 an easy one 15:35:04 it goes with the creation of the new repo etc 15:35:25 is the neutronclient part done? 15:35:52 vikram : it's included in the bgpvpn upstream code 15:35:57 ok 15:36:12 matrohon, ok for handling the new repo creation etc. ? 15:36:13 #action : matrohon to create the new openstack repo 15:36:15 matrohon: thanks 15:36:35 neutronclient is extendable now :) 15:37:07 :) 15:37:14 yes, installing the networking-bgpvpn package is enough to extend the python-neutronclient so that the new commands can be used 15:37:45 next stuff to do is adapting the code to what we changed in the API since the early proposal 15:37:50 API calls 15:37:53 DB mapping 15:37:56 neutron client 15:38:14 getting supported technique info from the driver 15:38:23 I think these are the four pieces 15:38:40 i can help with few of these 15:39:03 for the API, DB and neutron client, the work specificaly relates to: RDs field, supporting multiple networks 15:39:07 vikram : thanks :) 15:39:14 vikram: kewl :) 15:39:43 I can take up this:) 15:40:01 if unassigned 15:40:08 which piece in partcular ? 15:40:14 anyone else ? 15:40:25 I can help, sure 15:40:32 neutron client to start with 15:40:51 vikram, svinota : I'll create the corresponding blueprint 15:41:02 matrohon, thanks 15:41:12 ok 15:41:37 do we have any deadlines? 15:41:53 tmorin, is it ok if I will take a look onto RD and DB mappings? 15:42:07 svinota : great ! 15:42:11 vikram: "strike while the iron is hot" is our deadline ;) 15:42:23 hehehehe ;) 15:42:23 svinota: +1, excellent 15:42:37 ok 15:42:56 actually, i will on a vacation from 10th - 21st June 15:43:14 so .. just want to confirm 15:43:38 vikram : ok, np 15:43:56 thanks Mathieu 15:44:21 tmorin, matrohon, I'm not familiar with the meeting's black magic, could you write down somewhere on the meeting minutes these assignments? 15:44:26 ok, don't hesitate to ping ourselves on #neutron 15:44:47 yes, the bot will track this based on #action stuff we type 15:45:00 #action matrohon to create blueprint and assign them 15:45:11 #action vikram update python-neutronclient code 15:45:14 #action : svinota to work on DB mappings and RD 15:45:29 perfect :) 15:45:36 huh 15:45:56 (maybe the meetbot likes it better without the colon) 15:46:08 i'm not familiar yet either ;) 15:46:16 next item: unit tests 15:46:52 Question, will there be an issue, if the cli client comes out, and then later a decision is made to blend it into the VPN APIs? 15:46:54 we already have a basic coverage, but I guess it will need to be adapted to the API changes too 15:47:35 pc_m: no magic 15:47:36 pc_m: I feel yes 15:47:58 pc_m : it'll depend on the degree of integration we might have with the VPN project 15:48:11 pc_m: if the API are close enough, we should be able to make that compatible, but we aren't there yet 15:48:15 hard to tell now 15:48:45 concern about an API getting published, and then trying to change it. 15:49:05 pc_m: Make sense. +1 15:49:34 of course, but the starting point is do not delay progress waiting for (a still hypothetical) convergence 15:50:46 it really depends on the VPNaaS V2 API 15:51:00 only 10 mins left, I'd rather use time to cover other items 15:51:15 unit test: API calls are covered but will need an update 15:51:17 pc_m, tmorin , iirc we have API versioning, and I believe that's exactly why it is needed — not to wait for a perfect solution indefinitely 15:51:41 svinota: +1 15:51:56 unit test will also need to cover DB persistency, which they don't do yet 15:52:16 tmorin, unit, not functional? 15:52:20 but versioning is for extending the APIs not to try to converge two into one down the road 15:52:30 makes sense, just worried about introducing more pain down the road. Maybe unneccessarily? 15:52:34 I'm ok to take on this part (persistency unit test for db) 15:52:45 svinota: func tests, next item :) 15:52:53 ok :) 15:53:13 #action tmorin investigate unit test coverage to db persistency 15:53:31 ok, let's talk about func tests now 15:53:50 i know svinota and mat where interested 15:53:51 Neutron has made significant improvment in this area in kilo 15:54:37 the use of nethelpers and the Fixture lib helps on creating what's neede for func test 15:55:14 I start to investigate on how to use the existing work to perform bagpipe func tests 15:55:37 maybe we can plan an IRC discuss later this week on this, with at leat you petr ? 15:55:46 you and matrohon I mean 15:55:52 i make sens for other drivers too 15:56:19 since it helps to create the corresponding neutron conf file etc.... 15:57:12 I agree, lets plan a meeting — this time is almost over 15:57:17 yes 15:57:19 ok 15:57:25 let keep other items for a future meeting 15:57:34 ok 15:57:39 #topic driver status 15:57:45 any update here ? 15:59:00 just we need to work on API convergance fastly to avoid rework 15:59:16 let's discuss dirvers next time 15:59:22 #topic next meetings 15:59:36 vikram: yes - that is probably most important 15:59:41 any opinion on this slot ? 15:59:45 good for everyone ? 15:59:48 everybody's ok with this timeslot? 15:59:52 i am fine 15:59:54 works fine for me 16:00:06 fine for me 16:00:16 ok, then I'll register this slot for bgpvpn weekly meeting 16:00:26 great. 16:00:33 we'll see down the road if we keep the periodicity 16:00:44 thanks for driving us forward tmorin 16:00:58 I think OK for me. 16:01:01 Good job Thomos 16:01:04 not alone though, but thanks 16:01:21 thank you all for your involvment 16:01:33 see you next week 16:01:37 byey 16:01:39 bye 16:01:42 and in a minute on vpnaas :) 16:01:53 thanks and bye 16:02:05 :) 16:02:06 bye 16:02:20 #endmeeting