15:00:37 <mlavalle> #startmeeting neutron_l3
15:00:38 <openstack> Meeting started Thu Jun  7 15:00:37 2018 UTC and is due to finish in 60 minutes.  The chair is mlavalle. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:39 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:41 <openstack> The meeting name has been set to 'neutron_l3'
15:00:46 <manjeets_> hello
15:00:46 <mlavalle> Hi there!
15:00:50 <haleyb> hi
15:01:22 <mlavalle> #topic Announcements
15:02:07 <mlavalle> This week we are cutting Rocky-2:
15:02:14 <mlavalle> #link https://review.openstack.org/#/c/572666/
15:03:38 <mlavalle> CFP for the OpenStack summit in Berlin is now open:
15:03:45 <mlavalle> #link https://www.openstack.org/summit-login/login?BackURL=%2Fsummit%2Fberlin-2018%2Fcall-for-presentations%2F
15:04:13 <mlavalle> it closes at the end of this month
15:04:25 <mlavalle> don't let the deadline catch you off-guard ;-)
15:04:47 <manjeets_> thats very quick and short this time
15:05:14 <mlavalle> The final announcement I have this week (and the most important) is that manjeets_ has indicated that he will attend this meeting regularly
15:05:25 <mlavalle> so welcome to the team!
15:05:33 <manjeets_> thanks !
15:06:06 <mlavalle> Any other annoucements I might be missing?
15:06:16 <haleyb> yes, welcome manjeets !
15:07:33 <mlavalle> #topic Bugs
15:08:01 <mlavalle> The first one I have today is https://bugs.launchpad.net/neutron/+bug/1757482
15:08:03 <openstack> Launchpad bug 1757482 in neutron "IP address for a router interface allowed outside the allocation range of subnet" [High,Confirmed] - Assigned to Miguel Lavalle (minsel)
15:08:53 <mlavalle> This bug is the consolidation of the other bigs that refer to the same problem
15:09:03 <mlavalle> it came back last week as a CVE
15:09:26 <mlavalle> and then we realized that there was no need to embargo it since the issue was already public
15:09:37 <mlavalle> I am working on a fix
15:10:13 <haleyb> mlavalle: ack, if you need me to take it let me know
15:10:56 <mlavalle> the fix should be around here, I think: https://github.com/openstack/neutron/blob/master/neutron/db/l3_db.py#L786
15:12:14 <mlavalle> My first inclination was to forbid assigning an interface to the port if the subnet belongs to the external network
15:12:22 <manjeets_> so adding adding a validation that ip belongs to subnet before adding an interface ?
15:13:38 <mlavalle> but then in the bug I saw a comment that was passed along from kevinbenton suggesting to prevent taking the subnet's gateway ip if you are not the owner of the subnet
15:13:53 <mlavalle> any preferences in the team as to what is better?
15:15:16 <manjeets_> 2nd solution looks better though no preference
15:16:04 <haleyb> mlavalle: i haven't gone through those scenarios, but something that checks subnet owner i would guess, since the external is owned by admin right?
15:16:33 <mlavalle> right, you need to be admin to create external network
15:16:50 <mlavalle> so, I will follow second approach
15:17:02 <mlavalle> expect a patch over the next few days along these lines
15:18:10 <mlavalle> Next one is https://bugs.launchpad.net/neutron/+bug/1766701
15:18:12 <openstack> Launchpad bug 1766701 in neutron "Trunk Tests are failing often in dvr-multinode scenario job" [High,Confirmed] - Assigned to Miguel Lavalle (minsel)
15:18:52 <mlavalle> I created a patch for this one: https://review.openstack.org/#/c/571043/
15:19:08 <mlavalle> it seems I made some progress but still need to debug further
15:20:35 <mlavalle> If you have any quick suggestions, it would be appreciated: http://logs.openstack.org/43/571043/1/check/neutron-tempest-plugin-scenario-linuxbridge/2ff702a/job-output.txt.gz#_2018-05-30_00_34_25_498852
15:22:07 <haleyb> mlavalle: that was the same line you fixed right?
15:22:13 <mlavalle> yeah
15:22:21 <mlavalle> but I shifted the problem
15:22:32 <mlavalle> it accepts the command now
15:22:48 <mlavalle> but now it is lokking for '.' user
15:23:22 <mlavalle> I wonder if it is the period in "dhclient .*$IFACE.%(tag)d" || '
15:23:35 <mlavalle> I'll give it a try today
15:24:31 <mlavalle> Finally I have https://bugs.launchpad.net/neutron/+bug/1711463
15:24:32 <openstack> Launchpad bug 1711463 in neutron "TestNetworkBasicOps.test_network_basic_ops failed with "Timed out waiting for to become reachable"" [High,Confirmed] - Assigned to Miguel Lavalle (minsel)
15:25:39 <mlavalle> I will create Kibana query for this one
15:25:45 <mlavalle> to keep track of it
15:26:41 <mlavalle> In regards to this revert: https://review.openstack.org/#/c/570941/
15:27:18 <mlavalle> it seems the submitter got stuck
15:27:34 <mlavalle> If we don't hear from him
15:27:39 <mlavalle> by tomorrow
15:27:45 <haleyb> swami also started https://review.openstack.org/#/c/572168/
15:27:51 <mlavalle> I'm going to +2 Swami's https://review.openstack.org/#/c/572168/
15:28:08 <mlavalle> LOL, we were looking at the same patch
15:28:17 <mlavalle> so let's merge it by tomorrow
15:28:18 <mlavalle> ok?
15:28:47 <haleyb> it also has a shortened commit message, and it probably needs the bug it fixes in there
15:29:27 <mlavalle> ack
15:29:41 <mlavalle> any other bugs we should discuss today?
15:29:43 <haleyb> i'm going to click on the original review revert button and see what happens
15:29:57 <mlavalle> that's a good idea, generally
15:29:59 <mlavalle> LOL
15:30:11 <mlavalle> that's actually what we said last week
15:30:13 <haleyb> oh, it makes it short too, so we just need to update with the new bug being fixed, minor update
15:30:13 <mlavalle> thanks
15:30:24 <mlavalle> ok
15:30:29 <haleyb> i'll update the commit message
15:30:30 <mlavalle> thanks
15:30:43 <mlavalle> any other bugs to discuss today
15:30:45 <mlavalle> ?
15:32:01 <haleyb> not from me
15:32:08 <manjeets_> none from me
15:32:31 <mlavalle> ok
15:32:45 <mlavalle> #topic Port forwarding
15:34:12 <mlavalle> We have two patchsets up for review:
15:34:16 <mlavalle> https://review.openstack.org/#/c/535647
15:34:30 <mlavalle> https://review.openstack.org/#/c/533850
15:34:38 <mlavalle> They are making good progress
15:35:04 <mlavalle> I suggest to wait for the next revisions and review, if you have a chance
15:35:15 <mlavalle> #Topic Open Agenda
15:35:26 <mlavalle> Any other topics we should disucss today?
15:36:19 <haleyb> i saw there was an update to the dvr openflow change, have not had time to look yet
15:36:51 <mlavalle> ahh, great
15:37:00 <mlavalle> thanks for bringing it up
15:37:12 <mlavalle> yeah I thoght he was going to attend today's meeting
15:37:22 <manjeets_> haleyb, https://review.openstack.org/#/c/472289/ you mean this ?
15:37:41 <haleyb> yes, that one
15:38:08 <manjeets_> cool ! I'll review it
15:38:21 <mlavalle> manjeets_: he is your co-worker
15:38:37 <mlavalle> I think he is in California, though
15:38:41 <manjeets_> mlavalle, yes he's based at santa clara i guess
15:38:42 <manjeets_> yes
15:38:55 <mlavalle> ok, cool
15:39:01 <mlavalle> have a nice day
15:39:09 <mlavalle> #endmeeting