15:01:30 <mlavalle> #startmeeting neutron_l3
15:01:31 <openstack> Meeting started Thu Aug  2 15:01:30 2018 UTC and is due to finish in 60 minutes.  The chair is mlavalle. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:33 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:36 <openstack> The meeting name has been set to 'neutron_l3'
15:01:42 <haleyb> hi
15:01:43 <manjeets> hello
15:01:51 <mlavalle> hey, how are you guys?
15:02:55 <haleyb> good, too busy, you?
15:03:08 <mlavalle> about the same ;-)
15:03:30 <manjeets> doing ok
15:03:53 <mlavalle> #topic Announcements
15:04:26 <mlavalle> We are in the middle of wrapping up our RC-1
15:05:10 <mlavalle> I want to thank haleyb, slaweq, manjeets, njohnston for the non stopping reviewing over the past few days
15:05:19 <mlavalle> Thanks to you we are in excellent shape
15:05:35 <mlavalle> we have merged almost everything we wanted to merge in the neutron repo
15:05:38 <manjeets> :)
15:05:57 <mlavalle> The only exception are:
15:06:03 <mlavalle> 1) https://review.openstack.org/#/c/588079
15:06:30 <manjeets> that can go as well, just a minor update needed ?
15:06:33 <mlavalle> This is a release note for port forwardings, that I realized yesterday that we hand't included
15:07:30 <mlavalle> I see manjeets comment and I'll address it when we finish the meeting. I have a question. Is there a specific section for 'config' in the reno format?
15:07:44 <manjeets> mlavalle, other: can be used
15:08:12 <mlavalle> manjeets: ok, I'll use that
15:08:15 <haleyb> and i didn't see any bugs approved for rc1 either, we should probably look through the list and see if anything is important
15:08:17 <slaweq> mlavalle: there is no such section: https://docs.openstack.org/reno/2.1.2/usage.html#editing-a-release-note
15:09:11 <mlavalle> slaweq: thanks. I'll use other as suggested by manjeets
15:09:36 <mlavalle> haleyb: let's do that today and tomorrow
15:09:40 <mlavalle> good idea
15:10:53 <mlavalle> 2) The only other part missing are some patches for neutron-fwaas. I asked them today to attend tomorrow the drivers meeting to check at that point how they are doing with those patches
15:12:11 <mlavalle> I rebased and +2ed yesterday the 2 patches they needed on the Neutron side and slaweq caught and alnded them overnight
15:12:35 <mlavalle> so we have done everything out of their control for them to land the feature in fwaas
15:12:44 <mlavalle> now it's up to them
15:13:19 <mlavalle> it's lookin tight, though, based on what ws mentioned earlier today in the fwaas weekly meeting
15:14:26 <mlavalle> in other news, remember the that the PTG will take place on September 10 - 14 in Denver
15:14:37 <mlavalle> I am staying at the Renaissance hotel
15:14:48 <mlavalle> arriving on Sunday and staying until Saturday
15:15:02 <mlavalle> Looking forward to see you all there
15:15:14 <mlavalle> I will start the etherpad today
15:15:17 <slaweq> I have the same plan like You :)
15:15:32 <mlavalle> so we can brainstorm ideas to be discussed there
15:15:36 <mlavalle> slaweq: Great!
15:15:36 <haleyb> i'm sunday to friday
15:15:51 <manjeets> sunday to friday (Renaissance hotel)
15:16:18 <mlavalle> Now a bit of bad news....
15:16:35 <mlavalle> You guys are stuck with me for another cycle as the Neutron PTL
15:16:40 <davidsha_> :P
15:16:42 <manjeets> lol
15:16:45 <slaweq> LOL
15:17:09 <mlavalle> LOL
15:17:16 <davidsha_> Congratz!
15:17:22 <mlavalle> Thanks!
15:17:27 <slaweq> I was afraid that You will say that You are stepping down :)
15:17:46 <slaweq> Congratz mlavalle!
15:17:58 <mlavalle> I love this team, so it's an honor
15:18:07 <mlavalle> as I said in my nomination
15:18:14 <manjeets> MAGA/MANA ? lol mlavalle for 2018
15:18:24 <mlavalle> the highest in my careet
15:18:27 <mlavalle> career
15:18:39 <mlavalle> ok, let's move on....
15:18:50 <mlavalle> #topic Bugs
15:19:39 <mlavalle> since we have slaweq's attention, I want to start with https://bugs.launchpad.net/neutron/+bug/1766701
15:19:39 <openstack> Launchpad bug 1766701 in neutron "Trunk Tests are failing often in dvr-multinode scenario job" [High,Confirmed] - Assigned to Miguel Lavalle (minsel)
15:20:10 <mlavalle> I've been using the searches that I posted earlier in the bug for kibana
15:20:20 <mlavalle> and I don't get any hist over the past 7 days
15:20:51 <mlavalle> slaweq: have you seen cases of those failures lately in you investigations in gerrit?
15:21:29 <slaweq> no
15:21:42 <slaweq> but it's probably like that because those tests are marked as unstable: https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/scenario/test_trunk.py#L144
15:21:53 <slaweq> so they will be skipped in case of failure
15:23:03 <mlavalle> so I will create a DNM patch removing the the skip on those test and recheck it constantly
15:23:06 <mlavalle> makes sense?
15:23:10 <slaweq> yep
15:23:19 <mlavalle> ok
15:23:22 <mlavalle> thanks
15:23:38 <slaweq> if You will now work on it, we can even merge such patch and remove unstable_tests() in master
15:23:49 <slaweq> will be maybe easier to spot it
15:24:14 <slaweq> and IIRC it was failing in jobs which are non-voting
15:24:21 <mlavalle> right
15:24:30 <mlavalle> ok, let's do that
15:24:38 <mlavalle> I'll propose a patch later today
15:24:45 <slaweq> ok
15:25:05 <mlavalle> moving on to https://bugs.launchpad.net/neutron/+bug/1774459
15:25:05 <openstack> Launchpad bug 1774459 in neutron "Update permanent ARP entries for allowed_address_pair IPs in DVR Routers" [High,Confirmed]
15:25:22 <mlavalle> haleyb: do you think this should be a subject to discuss in the PTG?
15:26:13 <haleyb> mlavalle: yes, based on Swami's last comment it seems it needs some more thought
15:26:32 <mlavalle> ok, I will add it to the etherpad later today, when I start it
15:28:02 <mlavalle> haleyb: do you have handy the DVR bugs that yamamoto pointed out in his bugs report earlier this week?
15:28:12 * mlavalle looking for the message
15:29:00 <yamamoto> https://bugs.launchpad.net/neutron/+bug/1783470  https://bugs.launchpad.net/neutron/+bug/1783654
15:29:00 <openstack> Launchpad bug 1783470 in neutron "get_subnet_for_dvr returns SNAT mac instead of gateway in subnet_info" [Undecided,In progress] - Assigned to Arjun Baindur (abaindur)
15:29:01 <openstack> Launchpad bug 1783654 in neutron "DVR process flow not installed on physical bridge for shared tenant network" [Undecided,New]
15:29:03 <haleyb> think they were the new ones
15:29:28 <mlavalle> yes they are
15:29:44 <mlavalle> thanks yamamoto!
15:29:50 <haleyb> ^^ i'm looking at that now, seems a bug, fix was proposed
15:30:02 <haleyb> https://review.openstack.org/#/c/587234
15:30:45 <haleyb> i can see an optimization there as well that i'll do as a follow-on
15:30:49 <mlavalle> wow, this is a first time contributor!
15:30:58 <mlavalle> time to show how welcoming we are :-)
15:31:20 <haleyb> i think he found a couple of these new bugs
15:31:57 <mlavalle> ok, let's take a look at the bug itself and the proposed patch
15:32:13 <mlavalle> at first glance, it will probably require some tests
15:32:16 <haleyb> patch looks ok, just needs a test
15:32:22 <haleyb> jynx :)
15:32:26 <mlavalle> lol
15:33:01 <mlavalle> the other bug is also from him
15:33:23 <mlavalle> he hasn't proposed a fix yet
15:33:44 <mlavalle> if haleyb doesn't have bandwidth, I can take a look
15:33:56 <haleyb> https://bugs.launchpad.net/neutron/+bug/1783654
15:33:56 <openstack> Launchpad bug 1783654 in neutron "DVR process flow not installed on physical bridge for shared tenant network" [Undecided,New]
15:34:05 <haleyb> doh, you already pasted that
15:34:28 <mlavalle> yeah, I can take a look at it if you don't have bandwidth
15:35:07 <haleyb> mlavalle: if you have bandwidth, else i'll look today/tomorrow
15:35:42 <mlavalle> I'll start with it then and yell for help if I get to deep in c... water
15:36:16 <mlavalle> haleyb: do you have any other bugs?
15:36:38 <haleyb> i think there's 2 more new one
15:36:51 <haleyb> https://bugs.launchpad.net/neutron/+bug/1783728
15:36:51 <openstack> Launchpad bug 1783728 in neutron "DVR: router interface creation failure (load testing)" [High,In progress] - Assigned to Oleg Bondarev (obondarev)
15:37:08 <haleyb> from oleg our old friend
15:37:13 <haleyb> https://review.openstack.org/#/c/586059/
15:37:26 <mlavalle> wow, nice to see him again ;-)
15:37:39 <haleyb> and i already +2'd that, i forgot to add you
15:37:53 <haleyb> i guess if you're ok we have one for rc1
15:37:53 <mlavalle> I just added it to my pile
15:38:19 <mlavalle> I'll take a look. at first glance seems ok for rc1
15:38:48 <haleyb> https://bugs.launchpad.net/neutron/+bug/1784837 was the other
15:38:49 <openstack> Launchpad bug 1784837 in neutron "Test tempest.scenario.test_security_groups_basic_ops.TestSecurityGroupsBasicOps.test_in_tenant_traffic fails in neutron-tempest-dvr-ha-multinode-full job" [Medium,Confirmed]
15:39:04 <haleyb> created by slaweq yesterday
15:40:06 <mlavalle> I think we mentioned in the CI meeting that he was going to look at those failures
15:40:31 <mlavalle> I'll try to take a look this week as part of my bugs deputy duties
15:40:52 <slaweq> yep
15:41:07 <slaweq> I found that this test failed few times recently so I reported it
15:41:17 <slaweq> I don't have any other details about it
15:41:58 <mlavalle> tahnks for the info slaweq
15:42:53 <mlavalle> any other bugs we should discuss today?
15:43:49 <mlavalle> ok, let's move on
15:43:57 <mlavalle> #topic Openflow DVR
15:44:12 <davidsha_> Hey
15:44:27 <mlavalle> we failed to merge this in Rocky
15:44:39 <mlavalle> let's see if we can move it forward in Stein
15:44:56 <davidsha_> kk, We're too late to try for an exception for https://review.openstack.org/#/c/528336/ ?
15:45:09 <mlavalle> I think we are
15:45:18 <davidsha_> kk.
15:45:52 <mlavalle> but let's start looking at merging all this as early as possible during Stein
15:46:38 <davidsha_> ack, code can't be merged until the week after the PTG correct?
15:46:39 <mlavalle> is there any update fon your part davidsha_ ?
15:47:23 <davidsha_> I'm continuing to support Xubo on the work, but it's his feature.
15:47:31 <mlavalle> ok
15:48:06 <davidsha_> Thanks
15:48:35 <mlavalle> maybe you have answered this before, but other than configuring the normal parameters for DVR, is there anything else I need to know to deply this feature in a devstack?
15:48:43 <mlavalle> davidsha_: ^^^^
15:49:15 <davidsha_> Nope, it's just setting the agent_mode=dvr_bridge
15:49:23 <mlavalle> ok, cool
15:49:33 <mlavalle> moving on
15:49:39 <mlavalle> #topic Open agenda
15:50:17 <mlavalle> I will be in China the week of August 13 - 17
15:50:26 <mlavalle> so I might not make it to this meeting
15:51:12 <mlavalle> anything else?
15:51:32 <davidsha_> Nothing from me.
15:51:54 <mlavalle> ok guys, thanks for attending
15:51:58 <davidsha_> Thanks!
15:52:00 <manjeets> I am visiting Uk for 2 weeks, but i'll try to attend meetings
15:52:09 <mlavalle> manjeets: ack
15:52:16 <mlavalle> #endmeeting