15:00:15 <tidwellr> #startmeeting neutron_l3
15:00:15 <openstack> Meeting started Thu Apr 14 15:00:15 2016 UTC and is due to finish in 60 minutes.  The chair is tidwellr. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:16 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:19 <openstack> The meeting name has been set to 'neutron_l3'
15:01:07 <tidwellr> #link https://wiki.openstack.org/wiki/Meetings/Neutron-L3-Subteam#April_14th.2C_2016
15:01:19 <tidwellr> agenda ^^^
15:01:39 <johnbelamaric> hell
15:01:41 <johnbelamaric> o
15:01:50 <tidwellr> #topic Announcements
15:01:59 <tidwellr> Summit is coming!
15:02:07 <mlavalle> yeah, 2 weeks!
15:02:13 <mlavalle> less actually
15:02:31 <vikram_> hi
15:02:53 <tidwellr> looking forward to hanging out in mlavalle's backyard (semi-sort-of) :)
15:03:05 <vikram_> My best wishes to all the presenters
15:03:14 <mlavalle> looking forward to it!
15:03:21 <carl_baldwin> ++
15:04:09 <tidwellr> any other announcements?
15:05:02 <tidwellr> alright, let's dive in
15:05:08 <tidwellr> #topic Bugs
15:05:42 <tidwellr> https://bugs.launchpad.net/neutron/+bug/1566383
15:05:44 <openstack> Launchpad bug 1566383 in neutron "The creation fip does not endure restarting of l3-agent" [High,In progress] - Assigned to Brian Haley (brian-haley)
15:06:17 <tidwellr> bhaley: anything to share on this?
15:06:40 <mlavalle> haleyb: ^^^^
15:06:47 <mlavalle> lol
15:06:55 <haleyb> i must have udpated the review, let me look
15:07:35 <carl_baldwin> I'll take another look at the review.
15:07:35 <haleyb> tidwellr: review needs an update as it's throwing an exception
15:07:45 <haleyb> i must have missed something
15:08:08 <tidwellr> sounds like it's mostly under control then?
15:08:50 <haleyb> yes, just looks like an EINVAL running /sbin/ip
15:09:18 <tidwellr> cool
15:09:46 <tidwellr> anything else we should know?
15:10:24 <haleyb> not about that one, i'm looking at it now
15:10:37 <carl_baldwin> I think we can just take it to the review and move on.
15:10:43 <tidwellr> sounds good
15:10:46 <tidwellr> https://bugs.launchpad.net/neutron/+bug/1564335
15:10:47 <openstack> Launchpad bug 1564335 in neutron " [Pluggable IPAM] delete subnet in ml2 plugin does not comply with pluggable ipam (deletes ip allocations directly from db)" [High,In progress] - Assigned to Pavel Bondar (pasha117)
15:11:22 <tidwellr> pavel_bondar ^^^
15:11:31 <pavel_bondar> fix is out for review and has one +2
15:11:33 <pavel_bondar> https://review.openstack.org/#/c/300984/
15:11:40 <tidwellr> great!
15:11:57 <carl_baldwin> Good job pavel_bondar
15:12:13 <pavel_bondar> carl_baldwin: Thanks!
15:12:42 <tidwellr> https://bugs.launchpad.net/neutron/+bug/1543094
15:12:44 <openstack> Launchpad bug 1543094 in neutron "[Pluggable IPAM] DB exceeded retry limit (RetryRequest) on create_router call" [High,In progress] - Assigned to Ryan Tidwell (ryan-tidwell)
15:12:50 * tidwellr runs for cover
15:13:27 <carl_baldwin> We'll get this.  I think we're on the right track.
15:13:35 <tidwellr> I think so too, the end is in sight
15:13:47 <tidwellr> I pushed a couple more patch sets last night
15:14:31 <carl_baldwin> tidwellr: I'll have a look.
15:14:46 <tidwellr> what I have up for review will remove all the availability range logic from the code, we'll need a follow-on patch for a migration that removes the schema
15:15:01 <carl_baldwin> tidwellr: cool
15:15:42 <pavel_bondar> tidwellr: sounds good
15:16:25 <tidwellr> alright, on the RFE front
15:16:28 <tidwellr> https://bugs.launchpad.net/neutron/+bug/1566191
15:16:29 <openstack> Launchpad bug 1566191 in neutron "Allow multiple networks with FIP range to be associated with Tenant router" [Medium,Confirmed]
15:16:45 <tidwellr> not sure if anyone has taken a look at this
15:16:59 <carl_baldwin> I need to catch up on this.
15:18:22 <tidwellr> carl_baldwin: sounds good
15:18:40 <tidwellr> #topic Routed Networks
15:19:16 <carl_baldwin> My progress has slowed a little bit while I prepare for summit.  But, it is still moving forward.
15:19:41 <carl_baldwin> I'm going to try to get the CRUD patch in shape before summit and then work a little more on segment / subnet association.
15:20:02 <carl_baldwin> That's really all I have for today.
15:20:18 <tidwellr> sounds good
15:20:27 <tidwellr> moving on
15:20:37 <tidwellr> #topic BGP
15:21:12 <vikram_> hi
15:21:19 <tidwellr> I have nothing to report other than my efforts to catch up with all the work others are doing :)
15:21:40 <vikram_> Need approval for https://review.openstack.org/#/c/268726/
15:21:47 <vikram_> :-)
15:23:17 <tidwellr> In Austin I'd like to catch up with folks about how the split and some of these RFE's can be done during this cycle
15:23:33 <tidwellr> that will be an interesting dance :)
15:24:03 <tidwellr> I'll reach out to people offline
15:24:43 <tidwellr> vikram_, thanks for the persistence with https://bugs.launchpad.net/neutron/+bug/1566191
15:24:44 <openstack> Launchpad bug 1566191 in neutron "Allow multiple networks with FIP range to be associated with Tenant router" [Medium,Confirmed]
15:24:55 <tidwellr> oops, wrong link
15:25:01 <tidwellr> https://review.openstack.org/#/c/268726/
15:25:22 * tidwellr needs to clear his clipboard
15:25:23 <vikram_> np
15:26:00 <tidwellr> alright, moving quickly today
15:26:07 <tidwellr> #topic Open Discussion
15:27:09 <tidwellr> anything we didn't cover that needs some discussion?
15:27:09 <pavel_bondar> One time switch to pluggable ipam via alembic migration is ready for review https://review.openstack.org/#/c/181023/
15:27:21 <carl_baldwin> pavel_bondar: Thanks.
15:27:37 <pavel_bondar> it passes grenade tests, but still don't have clear jenkins pass
15:27:55 <carl_baldwin> pavel_bondar: How are the test failures looking?  Bad?
15:28:29 <pavel_bondar> test failures are different each time, so might be some concurency issue
15:29:59 <pavel_bondar> I put this migration patch on top of the Kevin's review  https://review.openstack.org/#/c/303085/2 that fixes restarting transaction on each concurent access
15:30:29 <carl_baldwin> Okay, I'll have a look at both.
15:30:42 <tidwellr> pavel_bondar: In the commit message I see a comment about IPAvailabilityRange not getting updates, could you elaborate?
15:31:04 <pavel_bondar> tidwellr: ok
15:31:27 <pavel_bondar> this is done to handle cases where users are already using pluggable ipam
15:32:07 <pavel_bondar> so IPAvailabilityRange stops to receive updates, since this tables used only by non-pluggable ipam implementation
15:32:25 <tidwellr> ok, got it
15:32:47 <pavel_bondar> and just doing av range recalc in migration
15:33:09 <carl_baldwin> Makes sense.
15:33:17 <tidwellr> cool, thanks for patiently answering my silly question :)
15:33:40 <pavel_bondar> no prob, case is not trivial anyway)
15:34:38 * tidwellr wonders what would happen if someone was working on removing IP availability ranges...
15:35:19 <tidwellr> pavel_bondar: I don't think this complicates any of the work I'm doing, but I'll be sure to take a look today
15:35:40 <carl_baldwin> I don't think it should complicate it either.
15:36:34 <pavel_bondar> tidwellr: yeah, if remove av_ranges patch is merged before migration I just need to remove ~20 lines of code from migration, so I think we are ok there
15:37:27 <tidwellr> pavel_bondar: let's both keep an eye on each other
15:37:52 <pavel_bondar> sure, taking a look on your patches periodically
15:38:22 <tidwellr> pavel_bondar: timing of which merges first will impact you
15:39:21 <pavel_bondar> tidwellr: right, but it will be easy to fix (as I see)
15:39:36 <tidwellr> pavel_bondar: good to know
15:39:53 <tidwellr> pavel_bondar: anything else on this?
15:40:14 <pavel_bondar> no, that is it
15:40:30 <tidwellr> alright, any other topics for discussion?
15:41:26 <tidwellr> going once
15:41:49 <tidwellr> going twice
15:42:16 <tidwellr> if there's nothing else, we'll give everyone back 15 minutes
15:42:31 <tidwellr> thanks everyone
15:42:33 <tidwellr> #endmeeting