15:01:46 <tmorin> #startmeeting bgpvpn
15:01:47 <openstack> Meeting started Tue Sep 27 15:01:46 2016 UTC and is due to finish in 60 minutes.  The chair is tmorin. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:48 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:50 <openstack> The meeting name has been set to 'bgpvpn'
15:02:01 <pcarver> Hi
15:02:14 <tmorin> hi bfernando, doude, eon`, lewo, matrohon, pcarver, timirnich, wdeclercq
15:02:26 <matrohon> hi
15:02:33 <wdeclercq> hey
15:03:48 <tmorin> let's start ?
15:04:16 <matrohon> yep
15:04:17 <tmorin> #topic last week wrapup
15:04:31 <tmorin> last week we merged quite a lof of changes
15:04:54 <tmorin> among them changes needed to unbreak the gate (thx matrohon)
15:05:17 <tmorin> we merged a simple change to allow the ODL plugin to do BGPVPNs of type l2
15:05:28 <tmorin> (looking forward to see this work!)
15:06:00 <tmorin> and another significant change merged was the last portion of the Horizon work
15:06:16 <tmorin> #info we now have a full Horizon UI !  yay :)
15:06:39 <tmorin> #topic related neutron fixes
15:07:10 <tmorin> last week me and matrohon worked on neutron's fixes
15:07:26 <tmorin> one for https://launchpad.net/bugs/1537067
15:07:27 <openstack> Launchpad bug 1537067 in networking-bgpvpn "enforce constraints on "neutron router-interface-add"" [High,In progress] - Assigned to Mathieu Rohon (mathieu-rohon)
15:07:39 <tmorin> and two for the bagpipe driver (issues in newton's ovs agent)
15:07:51 <tmorin> all this has now merged into neutron's stable/newton
15:07:58 <tmorin> so we should be good for the release
15:08:29 <tmorin> #topic work in progress
15:08:39 <tmorin> some things are in the pipe...
15:08:57 <tmorin> a fix to bug 1537067, as mentioned above
15:08:58 <openstack> bug 1537067 in networking-bgpvpn "enforce constraints on "neutron router-interface-add"" [High,In progress] https://launchpad.net/bugs/1537067 - Assigned to Mathieu Rohon (mathieu-rohon)
15:09:23 <tmorin> and a fix to bug 1627645
15:09:24 <openstack> bug 1627645 in networking-bgpvpn "bagpipe driver: need to co-exist with Neutron routers" [Medium,In progress] https://launchpad.net/bugs/1627645 - Assigned to Thomas Morin (tmmorin-orange)
15:09:34 <tmorin> this one I think will be solved soonish
15:10:11 <tmorin> it addresses a longstanding limitation of bagpipe driver
15:10:37 <tmorin> additionally we also have work in progress by pcarver on the API documentation side
15:10:45 <tmorin> https://review.openstack.org/#/c/373395/
15:11:04 <NikoHermannsEric> hey
15:11:09 <tmorin> pcarver: do you want to comment ?
15:11:32 <pcarver> tmorin: It's coming along. I've made a bunch of changes that I haven't pushed in the latest patch set yet
15:11:45 <tmorin> hi NikoHermannsEric
15:11:50 <pcarver> Also, I still need to disambiguate the parameter names
15:11:56 <tmorin> ok
15:12:13 <tmorin> would you like to keep this in net-bgpvpn for now, or submit soonish in neutron-lib ?
15:12:24 <pcarver> It looks like there's a lot of cleanup going on in neutron-lib as well for ambiguous parameter names
15:12:46 <pcarver> so the examples I was looking at contained information that hadn't been fixed yet
15:15:12 <tmorin> pcarver: np, understood, sounds like "work in progress"  / very reasonable :)
15:15:29 <wdeclercq> Sorry im a bit late to review https://review.openstack.org/#/c/369413/4/networking_bgpvpn/neutron/db/bgpvpn_db.py  ... does the query return NoResultFound .. ever without doing .all() on it?
15:15:46 <tmorin> pcarver: do you plan to look at how to integrate that into the doc generated by doc/source, or... ?
15:16:36 <pcarver> tmorin: I think once I get it all cleaned up I can then move it into neutron-lib pretty easily
15:16:53 <pcarver> but I'm not sure about getting it in before the end of the current cycle
15:17:05 <tmorin> wdeclercq: that's ok, you're not so late: I haven't review it much myself :)
15:17:17 <pcarver> might make sense to get it into networking-bgpvpn first and then migrate it to neutron-lib right after the summit
15:17:35 <tmorin> pcarver: publishing to the doc site happens independently of releases we make, so that's not a pb
15:18:15 <tmorin> pcarver: ok, so the plan would be: clean it up here (manually checking the result with tox -eapi-ref) and then submit in neutron-lib ?
15:18:22 <tmorin> pcarver: ok
15:18:46 <pcarver> tmorin: yes. It belongs in neutron-lib eventually as long as networking-bgpvpn is a stadium sub-project
15:18:48 <tmorin> wdeclercq: maybe matrohon can reply ?
15:18:53 <tmorin> pcarver: yes
15:19:06 <pcarver> but there was some issue with getting the networking-sfc docs into neutron-lib
15:19:46 <pcarver> I decided to follow the networking-sfc model and get the docs created first, then tackle the issue of getting them into neutron-lib
15:19:58 <matrohon> wdeclercq, tmorin : I'm revisiting thsi patch now that the counter part as merged in neutron
15:20:12 <matrohon> any review is welcomed
15:20:13 <pcarver> I'm still not sure I 100% understand Armando's objection when I tried to move the networking-sfc API docs into neutron-lib
15:21:04 <pcarver> I'm pretty sure it wasn't an objection to them being there, just an objection with the mechanics of the change
15:21:35 <tmorin> pcarver: for our information, can you point at the corresponding change ?
15:23:41 <pcarver> tmorin: Oh, I see I was misremembering the sequence of events. I'm thinking of https://review.openstack.org/#/c/351407/
15:24:12 <pcarver> I wasn't moving them into neutron-lib, I was trying to enable publishing of them where Louis had created them
15:24:48 <pcarver> But in the comments I asked about how to move them and Armando's reply was "Louis and I talked about this extensively. Moving now is not the right thing to do, there's a checklist of things to go through before we can move the API reference, which is really the last item to worry about."
15:25:08 <pcarver> So I dropped it for the time being.
15:25:25 <pcarver> I don't know what the checklist is
15:25:49 <tmorin> pcarver: ok, understood
15:25:53 <pcarver> But by working on it in the networking-bgpvpn repo I can at least get the content of the API docs complete and correct
15:26:20 <pcarver> I'll make a point of asking Armando about moving it after I've got it complete and correct
15:28:11 <tmorin> ok !
15:28:14 <tmorin> next topic ?
15:29:34 <tmorin> #topic  https://review.openstack.org/#/c/369413
15:29:41 <tmorin> back to this change
15:30:05 <tmorin> wdeclercq: just to keep track of that, can you comment in the change and then matrohon can follow up ?
15:30:11 <matrohon> tmorin, I started taking your comments into account
15:30:29 <tmorin> tmorin: wdeclercq's comment I guess
15:30:30 <matrohon> tmorin, we just discussed offline
15:30:33 <tmorin> ok
15:30:36 <wdeclercq> tmorin: I'll submit a change on the current patch set. matrohon knows
15:30:38 <tmorin> good then
15:30:42 <matrohon> wdeclercq, will submit a cleanup patch
15:30:51 <tmorin> what else should we discuss today ?
15:30:57 <matrohon> wdeclercq, you can submit a change to master
15:31:28 <tmorin> about https://bugs.launchpad.net/bgpvpn/+bug/1576440 ...
15:31:29 <openstack> Launchpad bug 1576440 in networking-bgpvpn "bagpipe: revisit driver now that update notification are fixed" [Medium,Confirmed]
15:31:38 <tmorin> this bug is really a code cleanup improvement
15:31:48 <tmorin> it does not have to be "Medium" importance
15:32:02 <tmorin> we can downgrade it to clean up our list for release 5.0.0
15:32:19 <matrohon> we should move it to ocata?
15:32:42 <tmorin> yes, indeed
15:33:00 <tmorin> #topic work for stadium inclusion / ocata-1
15:33:24 <tmorin> we have a few things that we need to work on to stay in the stadium
15:33:34 <tmorin> the deadline seems to be confirmed at ocata-1
15:33:38 <tmorin> which is November 17th
15:33:50 <tmorin> not very close, but not far either
15:34:02 <tmorin> #help we need people to volunteer
15:34:34 <pcarver> tmorin: Do we have a list?
15:35:06 <matrohon> pcarver, the two remaining bugs for newton :)
15:35:10 <tmorin> some of the bugs we marked for 5.0.0 relate to that
15:35:14 <tmorin> yes
15:35:17 <tmorin> bug 1561993
15:35:18 <openstack> bug 1561993 in networking-bgpvpn "support for openstack CLI" [High,Confirmed] https://launchpad.net/bugs/1561993
15:35:19 <matrohon> https://launchpad.net/bgpvpn/+milestone/5.0.0
15:35:20 <tmorin> bug 1587511
15:35:21 <openstack> bug 1587511 in networking-bgpvpn "demonstrate ability to upgrade" [High,Confirmed] https://launchpad.net/bugs/1587511
15:35:53 <tmorin> and there is a checklist here as well: http://docs.openstack.org/developer/neutron/stadium/governance.html#checklist  (thanks matrohon)
15:36:37 <tmorin> pcarver: neutron-lib api-ref is actually in this checklist
15:37:08 <matrohon> tmorin, migrating to openstack client too
15:37:10 <tmorin> there is grafana as well -- not sure how important this is, nor how much work it will require
15:37:27 <tmorin> matrohon: yes, but we already track this one
15:38:06 <matrohon> tmorin, yep but examples are provided in the checklist, it's helpful
15:39:12 <tmorin> yes
15:40:03 <tmorin> wdeclercq, NikoHermannsEric, pcarver, bfernando... anyone...: if you can think about helping on one of these, that could be nice
15:40:27 <pcarver> tmorin: I'll take a look
15:40:33 <tmorin> tmorin: it would be nice to kickstart these before the summit,since Nov-17 will be very close afterwards
15:40:41 <tmorin> great pcarver !:)
15:42:54 <tmorin> do we have another topic to discuss ?
15:43:00 <tmorin> #topic open discussion
15:43:11 <NikoHermannsEric> tmorin: I will look at the list
15:43:21 <tmorin> thanks NikoHermannsEric
15:45:31 <tmorin> ok, seems like we are done for today...!
15:45:36 <tmorin> thanks everyone!
15:45:40 <tmorin> #endmeeting