15:07:13 <tmorin> #startmeeting bgpvpn
15:07:14 <openstack> Meeting started Tue Feb  7 15:07:13 2017 UTC and is due to finish in 60 minutes.  The chair is tmorin. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:07:15 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:07:18 <openstack> The meeting name has been set to 'bgpvpn'
15:07:21 <tmorin> hi everyone
15:07:24 <pcarver> hi
15:07:31 <tmorin> hi paul
15:07:37 <tmorin> hi matrohon
15:07:39 <tmorin> hi doude
15:10:42 <bfernando> o/
15:11:35 <matrohon> hi
15:11:40 <tmorin> hi bfernando, matrohon
15:11:45 <tmorin> let's start
15:11:49 <tmorin> on the agenda:
15:12:05 <tmorin> a) OSC client plugin (doude, if you're around)
15:12:21 <tmorin> b) BGPVPN API definition and doc in neutron-lib (pcarver)
15:12:29 <doude> hi tmorin
15:12:36 <tmorin> c) filtering on GET API calls
15:12:38 <tmorin> hi doude!
15:12:48 <tmorin> c) is by doude as well
15:12:58 <tmorin> d) bagpipe driver status
15:13:04 <tmorin> e) open discuss
15:14:53 <tmorin> #topic OSC client plugin
15:14:55 <tmorin> doude ?
15:15:04 <tmorin> the status is short I guess :)
15:15:28 <doude> I get one review during last night I should take in account
15:15:43 <doude> I'll try to do that before the end of the day
15:16:02 <doude> nothing since last meeting
15:16:07 <tmorin> doude: I tried to provide code snippets to help address this one quickly
15:16:07 <doude> *more
15:16:12 <tmorin> yep
15:16:19 <doude> yes I saw, thanks tmorin
15:16:34 <tmorin> you're welcome
15:16:42 <tmorin> #topic BGPVPN API definition in neutron-lib
15:16:47 <tmorin> pcarver ?
15:16:58 <tmorin> this one is a marathon :)
15:17:03 <pcarver> tmorin: getting pretty close. Need to resolve a merge conflict
15:17:07 <tmorin> but the finish line seems close now :)
15:17:14 <pcarver> when I tried to git review yesterday it looked weird
15:17:32 <tmorin> yes, and adapting the validator to address boden's comments as well
15:17:39 <pcarver> it looks like my commit is introducing a field changed_at but I'm really not
15:17:48 <tmorin> #link https://review.openstack.org/#/c/413182/
15:18:11 <pcarver> it looks like maybe something that someone else removed that I'm inadvertently putting back in.
15:18:23 <pcarver> just need to verify before I delete it
15:18:51 <pcarver> oh, sorry I'm talking about API ref, not def
15:18:58 <tmorin> ah
15:19:15 <pcarver> I'll get to the def right after I resolve the merge conflict on the ref
15:19:27 <tmorin> ok, this one indeed has only a merge issue to resolve
15:19:42 <tmorin> ping me if you need help
15:19:59 <tmorin> what about the API code definition ?
15:20:14 <tmorin> do you want help to adapt the validator code as suggested by boden ?
15:20:50 <pcarver> I think I understand about adapting the validator but haven't started it yet. I'll give it a try and let you know if I run into problems.
15:22:19 <tmorin> pcarver: ok
15:22:28 <tmorin> next topic ?
15:22:37 <tmorin> #topic filtering on GET API calls
15:23:01 <tmorin> we've just merged a change by doude to allow filtering by network or router on GET API calls
15:23:06 <tmorin> this is very useful !
15:23:20 <tmorin> #link https://review.openstack.org/426309
15:23:25 <tmorin> thanks doude!
15:23:54 <tmorin> the driver API is changed a bit, but we've checked and drivers are not impacted
15:24:14 <doude> you're welcome tmorin
15:24:22 <tmorin> the one out-of-tree driver that was using it (odlv2 driver, work in progress) has been updated later today
15:24:46 <pcarver> Is that the standard fields behavior?
15:25:02 <tmorin> yes
15:25:30 <tmorin> GET .../bgpvpn/bgpvpns?networks=...
15:26:10 <tmorin> its standard except that we weren't supporting it for attributes that were lists
15:27:54 <pcarver> does it also apply to network associations and router associations?
15:30:09 <tmorin> these API resources don't have attributes that would be lists
15:30:21 <tmorin> so they were already supporting being listed with a filter
15:30:56 <pcarver> ok
15:31:09 <tmorin> next topic ?
15:31:15 <tmorin> #topic bagpipe driver status
15:31:19 <tmorin> not much to say
15:31:30 <tmorin> some cleanup changes are in the pipe
15:31:38 <tmorin> they depend on cleanups in neutron repo
15:31:47 <tmorin> hopefully all this will merge for ocata
15:32:06 <tmorin> #topic open discussion
15:32:18 <tmorin> any question/comment anyone ?
15:35:23 <tmorin> ok !
15:35:28 <tmorin> we're done then!
15:35:45 <tmorin> i'm not sure there'll be a meeting next week
15:35:49 <tmorin> I'll be off
15:36:06 <tmorin> matrohon: would you chair next week ?
15:36:36 <tmorin> and the following week will be the PTG so no meeting either
15:36:40 <tmorin> bye everyone
15:37:20 <pcarver> bye
15:39:30 <matrohon> sorry
15:39:33 <matrohon> I'll do
15:39:45 <matrohon> tmorin, ^
15:39:53 <tmorin> thanks matrohon!
15:40:05 <tmorin> closing the meeting...
15:40:10 <tmorin> bye everyone
15:40:16 <tmorin> #endmeeting