15:01:33 <tmorin> #startmeeting bgpvpn
15:01:34 <openstack> Meeting started Tue Mar 21 15:01:33 2017 UTC and is due to finish in 60 minutes.  The chair is tmorin. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:35 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:37 <openstack> The meeting name has been set to 'bgpvpn'
15:01:46 <pcarver> hi
15:01:55 <tmorin> hi pcarver !
15:02:00 <tmorin> let's see who else is around...
15:02:35 <tmorin> matrohon ? doude ? timirnich ? lhinds ?
15:03:50 <doude> hi
15:04:23 <tmorin> ok, let's start
15:04:35 <tmorin> here is what I have as an agenda:
15:04:51 <tmorin> a) neutron-lib related work
15:04:59 <tmorin> b) API evolution discussion
15:05:09 <tmorin> c) bagpipe driver status update
15:05:22 <tmorin> d) contrail driver status update (? if you want doude)
15:05:43 <tmorin> #topic neutron-lib related work
15:05:51 <tmorin> pcarver: do you want to update us ?
15:06:11 <tmorin> I've just seen a refresh of https://review.openstack.org/#/c/395585/
15:06:52 <pcarver> tmorin: I've been spending more time troubleshooting tox failures in multiple environments trying to find one that works reliably than I have actually making code changes. But I just pushed a patch set that has a portion of the changes and passes tox locally.
15:07:31 <pcarver> I left out the attribute map parts because those were causing tox to fail. I'm going to bring those in next.
15:07:41 <tmorin> ok, looks good
15:07:52 <tmorin> seems going in the right direction
15:07:53 <pcarver> But I wanted to push a partial PS and see if it passes CI
15:08:01 <tmorin> yes
15:08:40 <tmorin> we haven't discussed https://review.openstack.org/#/c/439193/ recently
15:08:47 <tmorin> ("Point API entry to neutron-lib API Reference")
15:08:57 <tmorin> this one does not seem really ready
15:09:07 <tmorin> I failed to comment on it though
15:09:19 <pcarver> tmorin: yeah, it didn't render correctly. I need to figure out the right syntax
15:09:55 <pcarver> assuming there is a correct syntax to embed a remote link in the middle of a generated TOC
15:09:58 <tmorin> I think that keeping the toc as is would be better, and that a link to the API could be done *in* the existing api.rst
15:10:24 <pcarver> oh, so keep the file api.rst and just remove all the content and replace it with a hyperlink?
15:10:31 <tmorin> by replacing its content by a simple sentence: This API is documented `here <http://....>`.
15:10:54 <pcarver> that's possible and would work. I think it's a little ugly to make the reader click an extra link
15:11:26 <tmorin> Or better: This API is documented in the `Neutron API reference <http://....>`.
15:12:10 <tmorin> yeah, I see what you mean, but I think it's even more ugly to have an item in the TOC that links outside the document
15:12:20 <pcarver> OK. I think it would be nicer to jump directly to it without having to click through an interstitial page, but at least it will get it done
15:13:21 <pcarver> I'll do it that way
15:13:42 <tmorin> pcarver: ok !
15:14:03 <tmorin> thanks parver for working on these items !
15:14:06 <tmorin> next topic ?
15:14:14 <tmorin> #topic APi evolution discussion
15:14:43 <tmorin> we have discussed API evolution in the past weeks
15:14:50 <tmorin> but not very intensively
15:15:04 <tmorin> there is one blueprint on which having reviews would help
15:15:14 <tmorin> #link https://blueprints.launchpad.net/bgpvpn/+spec/port-association
15:15:27 <tmorin> details are in the corresponding etherpad: https://blueprints.launchpad.net/bgpvpn/+spec/port-association/+edit
15:15:48 <tmorin> sorry wrong link
15:15:51 <tmorin> #link https://etherpad.openstack.org/p/bgpvpn_advanced_features
15:16:07 <tmorin> wrong link again ...:-/
15:16:41 <tmorin> #link https://etherpad.openstack.org/p/bgpvpn_port_association
15:16:48 <tmorin> this is the right one
15:17:35 <tmorin> doude: do you think you or someone else working on OpenContrail could have a look and give feedback ?
15:18:40 <tmorin> timirnich: (hi!) I see you're logged in, perhaps you would know if someone working on ODL could have a look at https://etherpad.openstack.org/p/bgpvpn_port_routes
15:18:47 <tmorin> #undo
15:18:48 <openstack> Removing item from minutes: #link https://etherpad.openstack.org/p/bgpvpn_port_association
15:18:50 <tmorin> #undo
15:18:51 <openstack> Removing item from minutes: #link https://etherpad.openstack.org/p/bgpvpn_advanced_features
15:18:54 <tmorin> #undo
15:18:54 <openstack> Removing item from minutes: #link https://blueprints.launchpad.net/bgpvpn/+spec/port-association
15:19:07 <tmorin> #link https://blueprints.launchpad.net/bgpvpn/+spec/port-routes
15:19:17 <tmorin> #link https://etherpad.openstack.org/p/bgpvpn_port_routes
15:19:30 <doude> tmorin: i can have a look and give you my feedback
15:19:35 <tmorin> doude: thanks!
15:20:07 <timirnich> tmorin: I'll have a look and pass on if needed
15:20:33 <timirnich> sorry for not really joining the call - triple booked
15:20:39 <tmorin> timirnich: thanks!  jan had been contributing at some point in the past
15:20:49 <tmorin> timirnich: np, we understand :)
15:22:12 <tmorin> tmorin: I'll try to reach out with Nuage folks as well
15:23:07 <tmorin> tmorin: I'll try to also push a proposal for an APi extension allowing the control of local_pref and communities/extended-communities
15:23:29 <tmorin> there is also another API evolution item in the pipe
15:24:02 <tmorin> ODL folks have revived the work around adding the "vni" attribute (allowing to force the VNI for a BGPVPN of type L2)
15:24:22 <tmorin> #link https://review.openstack.org/#/c/332711/
15:24:47 <tmorin> this will have to be adapted to be pushed/discussed in neutron-lib first
15:25:54 <tmorin> next topic ?
15:27:45 <tmorin> #topic c) bagpipe driver status update
15:31:06 <tmorin> the work on merging bagpipe-bgp into networking-bagpipe is now quite complete
15:31:13 <tmorin> its and improvements here and there
15:31:50 <tmorin> some unrelated cleaning patches are pending, waiting for neutron changes to merge
15:32:01 <tmorin> nothing else much to add...
15:32:11 <tmorin> #topic contrail driver status
15:32:19 <tmorin> doude ? if you want to ...
15:47:02 <doude> tmorin: oops I missed your ping
15:47:06 <doude> few minutes left
15:47:16 <tmorin> np, you still have some time :)
15:48:16 <doude> actually the contrail modifications were merged and will be integrated in the release 4.0 (scheduled middle of May)
15:48:48 <doude> and the BGPVPN driver also merge in the Contrail plugin/drivers repository
15:49:06 <doude> that support l2 and l3 VPN and network and router assocations
15:49:15 <doude> it does not support RD
15:49:23 <tmorin> ok!
15:49:31 <tmorin> excellent news !
15:49:45 <tmorin> this is a pretty extensive API coverage
15:49:49 <doude> I'll push a review on bgpvpn to depreciate beta driver with warning message
15:49:56 <tmorin> ok!
15:51:03 <tmorin> can you also update the doc under networking-bgpvpn/doc/source/contrail ... to specify the new package.path.to.contriail_v2:Driver
15:51:06 <tmorin> ?
16:01:15 <tmorin> ok
16:01:20 <tmorin> forgot to close
16:01:22 <tmorin> thanks everyone
16:01:27 <tmorin> #endmeeting