15:07:32 <tmorin> #startmeeting bgpvpn
15:07:32 <openstack> Meeting started Tue Aug 30 15:07:32 2016 UTC and is due to finish in 60 minutes.  The chair is tmorin. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:07:33 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:07:36 <openstack> The meeting name has been set to 'bgpvpn'
15:08:08 <tmorin> hi folks .... rollcall: doude ? eon` ? matrohon ? pcarver ? timirnich ?
15:08:32 <timirnich> hi tmorin
15:09:01 <matrohon> hi
15:09:04 <tmorin> hi, again, tim :)
15:09:05 <pcarver> hi
15:09:45 <tmorin> I guess we can start
15:10:09 <tmorin> #topic announce
15:10:55 <tmorin> the summit agenda is out, and we will have the opportunity to talk about net-bgpvpn in Barcelona...
15:11:02 <tmorin> but you guys already knew :)
15:11:52 <tmorin> we've been discussing with paul and tim on how to articulate the two talks to adapt to the different audiences and cover different aspects in details in the two talks
15:12:37 <tmorin> pcarver, timirnich, feel free to comment
15:12:48 <tmorin> or... next topic...
15:13:21 <pcarver> we can share the Etherpad if anyone wants to have input, but probably don't need to discuss here.
15:13:34 <timirnich> was about to say the same
15:13:38 <matrohon> pcarver, tmorin : please share :)
15:13:46 <pcarver> https://etherpad.openstack.org/p/BGPVPN_talk_summit
15:14:03 <matrohon> pcarver : thanks
15:14:08 <tmorin> matrohon: your comments are much welcome
15:14:51 <tmorin> #topic zuul jobs and gate tests
15:15:13 <tmorin> in the past two weeks some work has been done to improve our jobs
15:15:33 <tmorin> the tempest jobs have been fixed to really run our tempest suite, and only this suite
15:15:47 <tmorin> then some additional tempest jobs have been contributed and merged
15:16:09 <tmorin> the last step is to activate these jobs as gate voting jobs, instead of experimental
15:16:30 <tmorin> this last piece is https://review.openstack.org/352401
15:17:10 <tmorin> #topic other bug fixes, improvements
15:17:18 <matrohon> maybe yolanda can help on that
15:17:40 <tmorin> she's not in this channel apparently
15:18:17 <tmorin> adding her in Cc ?
15:18:33 <matrohon> In just left a msg on #openstack-infra
15:19:03 <tmorin> armax just gave his +1 and andreas his +2, so its just a matter of having a second +2
15:19:18 <tmorin> #undo
15:19:19 <openstack> Removing item from minutes: <ircmeeting.items.Topic object at 0x7f5bbfc7d990>
15:19:25 <tmorin> #topic other bug fixes, improvements
15:19:49 <tmorin> in the past weeks, we also merged an opencontrail driver improvement (uuid check)
15:19:55 <tmorin> and some other minor cleanups
15:20:08 <tmorin> I also unblocked our gate in stable/liberty this morning
15:20:30 <tmorin> well, this last change needs a +2 to proceed
15:21:24 <tmorin> I just added armax as Cc, he or one of the stable maintainers can have a look and hopefully confirm ihrachys +2
15:21:32 <tmorin> this is https://review.openstack.org/#/c/362741/
15:22:54 <tmorin> on the horizon change for tenant workflow: the work is in progress, cedric is going to address our reviews hopefuly next week
15:23:33 <tmorin> matrohon: anything else on bugs/changes that I would have missed ?
15:23:47 <matrohon> tmorin : yep : https://launchpad.net/bgpvpn/+milestone/5.0.0
15:23:50 <matrohon> :)
15:24:14 <tmorin> ok, we can go through the list
15:24:26 <tmorin> before that: https://review.openstack.org/#/c/362808/  -> matrohon, can you have a quick look  ?
15:26:59 <matrohon> tmorin : can you sum up the gasp of it?
15:27:05 <matrohon> not clear to me
15:27:56 <tmorin> it's a tox.ini simplification
15:28:45 <matrohon> well, let's trust our gate an ihrachys on that!
15:28:51 <tmorin> before Aug 11th, the gate was not supporting jobs that would use pip install -c flag, which allow to set upper constraints on versions
15:29:12 <tmorin> so our tox.ini had a tweak to not use -c for some of the jobs
15:29:22 <tmorin> the change just remove this tweak
15:29:34 <matrohon> tmorin, ack
15:29:37 <tmorin> it's a copy-paste from what ihrachys has done in neutron, neutron-lbaas, etc. ...
15:30:57 <tmorin> #topic bug/bp roadmap for next release
15:31:07 <tmorin> so let's go through the list at https://launchpad.net/bgpvpn/+milestone/5.0.0
15:31:20 <tmorin> blueprints first
15:31:38 <tmorin> we need to update the status of the horizon bp: one is done, the other is in progress
15:33:33 <tmorin> on the two other blueprints, they haven't progressed, and it seems unlikely we can do it for 5.0.0
15:33:56 <tmorin> I propose to update them later, once it's sure it can't be done for 5.0.0
15:34:00 <tmorin> matrohon: ?
15:34:18 <matrohon> tmorin : I agree
15:34:27 <tmorin> bugs now...
15:34:53 <tmorin> #1537067 : this one I did look at for some time
15:34:59 <tmorin> bug 1537067
15:34:59 <openstack> bug 1537067 in networking-bgpvpn "enforce constraints on "neutron router-interface-add"" [High,Confirmed] https://launchpad.net/bugs/1537067
15:35:35 <tmorin> I believe we should be able to do a driver-independent check on the service plugin
15:36:29 <matrohon> a call to the neutron API
15:36:31 <matrohon> ?
15:38:05 <matrohon> not sure, we need a hook from the neutron plugin that tell us a subnet is going to be attached to a router
15:38:13 <tmorin> no, to address this limitation, we need to act on an event on the router API
15:38:14 <tmorin> tes
15:38:15 <tmorin> yes
15:38:32 <tmorin> there are events, at least for the reference L3plugin
15:38:45 <tmorin> not sure how it will apply for other drivers
15:38:55 <tmorin> it won't work for contrail, but that is expected
15:39:07 <tmorin> I don't know what the L3plugin options are for ODL
15:39:10 <tmorin> timirnich: ?  ^^
15:39:33 <timirnich> sorry, multitasking
15:39:54 <matrohon> they have their own l3 plugin haven't they?
15:40:13 <timirnich> Yes, ODL has an L3 plugin
15:40:19 <tmorin> actually, I'm asking before having checked: the event we need is possibly fired by ML2
15:40:21 <timirnich> it sits in networking-odl
15:40:50 <tmorin> any L3plugin calling ML2 port update would, I think, work
15:40:54 <matrohon> In the notes, I though about listening to a port-update event
15:40:57 <matrohon> from ML2
15:41:15 <matrohon> I didn't check if a useful event has been added to neutron
15:41:17 <tmorin> it would generate a port BEFORE_UPDATE event that the BGPVPN service plugin can intercept and raise an exception against
15:41:42 <matrohon> tmorin, that was my idea, AFAIR
15:42:04 <tmorin> I think the L3plugin code, which initially was directly updating the L2 db model, now calls ML2, and thus the before_update event is fired
15:42:39 <tmorin> matrohon: yes, very possibly given the bug comments - I had discussions on this with wdeclerq as well
15:45:26 <tmorin> matrohon: do you want to work on it ?
15:46:30 <tmorin> ok... we'll see :)
15:46:31 <matrohon> tmorin : I would love to... not sure to have time, but let me till next week
15:46:36 <tmorin> ok !
15:46:54 <tmorin> bug 1561993
15:46:54 <openstack> bug 1561993 in networking-bgpvpn "support for openstack CLI" [High,Confirmed] https://launchpad.net/bugs/1561993
15:47:22 <tmorin> this one would be important to address for 5.0 - because it's a criteria to stay in the Neutron stadium
15:47:32 <tmorin> (bug 1587511 as well, by the way)
15:47:32 <openstack> bug 1587511 in networking-bgpvpn "demonstrate ability to upgrade" [High,Confirmed] https://launchpad.net/bugs/1587511
15:47:37 <matrohon> I'been through the neutron-mid-cycle report from armax, there were a large topic on it
15:47:56 <tmorin> for bug 1561993, I believe it is a quite easy task
15:47:56 <openstack> bug 1561993 in networking-bgpvpn "support for openstack CLI" [High,Confirmed] https://launchpad.net/bugs/1561993
15:48:30 <tmorin> timirnich, pcarver: would possibly one of you or a colleage be interested in a reasonably easy task to get more familiar with openstack development ?
15:48:51 <matrohon> http://lists.openstack.org/pipermail/openstack-dev/2016-August/102366.html
15:48:55 <tmorin> if so, bug 1561993 would be an good exercise
15:48:58 <matrohon> ^ the report
15:49:11 <timirnich> I can look into this
15:49:29 <timirnich> if I find someone
15:49:33 <timirnich> when do we need this?
15:50:19 <tmorin> i'd say september
15:50:24 <matrohon> rtheis's patches might be a good inspiration for that
15:50:44 <timirnich> Sept is challenging due to ODL Summit
15:50:50 <timirnich> for us
15:50:59 <tmorin> matrohon: wrt to the midcycle comments, I think we are in the easy case: our code is not in python-neutronclient, we just have osc extensions to write
15:51:10 <tmorin> matrohon: can you link the bugs
15:51:20 <tmorin> timirnich: understood
15:51:20 * matrohon searching
15:51:53 <tmorin> timirnich: for stadium criteria the deadline is, I think ocata-1, matrohon do you know when this is ?
15:52:09 <matrohon> I have ocata-1 in mind too
15:52:42 <timirnich> tmorin: can you send me an email reminder that I can use to forward and then I'll see what I can do ok?
15:52:54 <tmorin> ok, timirnich, will do
15:53:28 <tmorin> bug 1587511
15:53:28 <openstack> bug 1587511 in networking-bgpvpn "demonstrate ability to upgrade" [High,Confirmed] https://launchpad.net/bugs/1587511
15:53:34 <tmorin> this one is harder to address
15:53:45 <tmorin> we'll need to mimic what other neutron projects do
15:54:07 <tmorin> some grenade stuff is needed under the devstack/ plugin dir
15:54:13 <matrohon> there is a devref on OSC transition : https://review.openstack.org/#/c/357844/
15:54:28 <tmorin> and I guess some more things are needed for a gate job definition
15:55:42 <tmorin> bug 1501213
15:55:42 <openstack> bug 1501213 in networking-bgpvpn "run tempest tests with bagpipe ovs agent" [High,Triaged] https://launchpad.net/bugs/1501213
15:56:00 <tmorin> this one is also a gate job to create
15:56:17 <tmorin> I'll try to have a look at at least one of these two in early september
15:57:17 <tmorin> bug 1576440 and 1522072 : bagpipe driver small issues, I have to spent some time on these as well
15:57:17 <openstack> bug 1522072 in networking-bgpvpn "bagpipe driver: wrong RTs passed to agent on BGPVPN update or association updates in the case of multiple associations" [Medium,Triaged] https://launchpad.net/bugs/1522072 - Assigned to Thomas Morin (tmmorin-orange)
15:57:19 <openstack> bug 1576440 in networking-bgpvpn "bagpipe: revisit driver now that update notification are fixed" [Medium,Confirmed] https://launchpad.net/bugs/1576440
15:57:25 <tmorin> #topic open discuss
15:57:29 <tmorin> need to wrap up for today
15:57:32 <tmorin> anything else ?
15:57:36 <tmorin> matrohon ? timirnich ?
15:57:46 <timirnich> nope
15:57:50 <matrohon> nein
15:57:58 <timirnich> ;-)
15:58:08 <steve_ruan> hi tmorin
15:58:18 <tmorin> hi steve_ruan !
15:58:24 <steve_ruan> I have spec about add vpn in neutron-dynamic-routing
15:58:25 <steve_ruan> https://review.openstack.org/#/c/360620/
15:58:31 <tmorin> yes
15:58:37 <tmorin> is it still on the agenda ?
15:58:58 <steve_ruan> do you have any comment about define similar api
15:59:05 <steve_ruan> rest API
15:59:30 <tmorin> what needs to be done for this change is updating the API spec taking into account the vnid collision things we have discussed in the comments of this change
16:00:03 <tmorin> we need to wrap up
16:00:14 <tmorin> let's follow up in #openstack-net-bgpvpn
16:00:18 <tmorin> thanks everyone!
16:00:21 <tmorin> #endmeeting