15:00:07 #startmeeting neutron_l3 15:00:08 Meeting started Thu Oct 1 15:00:07 2015 UTC and is due to finish in 60 minutes. The chair is mlavalle. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:09 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:13 The meeting name has been set to 'neutron_l3' 15:00:42 Agenda: https://wiki.openstack.org/wiki/Meetings/Neutron-L3-Subteam#Agenda 15:00:53 #chair carl_baldwin 15:00:54 Current chairs: carl_baldwin mlavalle 15:01:18 hi 15:01:22 #topic Announcements 15:01:43 I don't have any annoucements. how about you carl_baldwin ? 15:01:49 Nope 15:02:01 hi 15:02:07 any other annoucements from the team? 15:02:36 hi 15:02:39 ok in that case, let's move on...... 15:02:39 hi 15:02:46 #topic Bugs 15:03:10 so we don't have a lot of change in the list of bugs from last week, pretty much the same ones 15:03:23 there was progress in most of them, though 15:03:46 almost all of them have newer patchsets in their proposed fixes 15:04:14 the one I am curious about is https://bugs.launchpad.net/neutron/+bug/1365473 15:04:14 Launchpad bug 1365473 in neutron "Unable to create a router that's both HA and distributed" [High,In progress] - Assigned to Artur Korzeniewski (artur-korzeniewski) 15:04:51 ownership was trnasferred from amuller to artur-korzeniewski just yesterday 15:05:16 I haven't been able to ping either one of them, so I intend to follow up today 15:05:25 mlavalle: I though amuller was done reviewing this patch. 15:05:44 mlavalle: I reviewed that one yesterday. I can try to contact the new “author" 15:06:00 carl_baldwin: thanks! 15:06:22 https://bugs.launchpad.net/neutron/+bug/1494351 15:06:22 Launchpad bug 1494351 in neutron "Observed StaleDataError in gate-neutron-dsvm-api tests if reference IPAM driver is used" [High,In progress] - Assigned to Pavel Bondar (pasha117) 15:06:31 the review got me thinking of the coupling between the DVR classes in the L3 agent again. Still too much. I hope to have a patch set up soon. 15:06:50 carl_baldwin: ack - that's becoming painful 15:06:54 working on that one (1494351) 15:07:06 pavel_bondar pushed latest PS for this one. Waiting for review, I think 15:07:18 pavel_bondar: I owe you a review on that one. Will bump it up on my list. 15:07:20 pavel_bondar: am i correct? 15:07:33 actually still have some failures 15:07:47 but have an issue with reproducing it locally 15:08:05 anyway reviewving there approach would be helpful 15:08:44 working on this ticket at fullspeed now, so believe will get some results today/tomorrow 15:09:20 pavel_bondar: thank you! 15:09:25 pavel_bondar: ++ 15:09:37 the next up is https://bugs.launchpad.net/neutron/+bug/1486795 15:09:37 Launchpad bug 1486795 in neutron "DVR: create or update port by using notify specific host rather than fanout" [High,In progress] - Assigned to shihanzhang (shihanzhang) 15:10:16 This one got a new patchset, it is waiting for cores reviews.... I believe regXboi gave it a +1 15:10:42 did i? 15:11:00 * regXboi goes and looks 15:11:00 here's the patchset https://review.openstack.org/#/c/221209/ 15:11:11 I did 15:11:38 yeah, this has been through several revisions and I think others should look at it 15:11:38 so anyway, we need core reviewers for this one 15:12:14 Will look today 15:12:34 finally in our list https://bugs.launchpad.net/neutron/+bug/1486828 15:12:34 Launchpad bug 1486828 in neutron "L3: Notify specific agent rather than fanout when associating floatingip" [High,In progress] - Assigned to changzhi (changzhi) 15:12:36 i believe there is a similar change for FIP as well 15:12:43 that one :) 15:13:03 https://review.openstack.org/#/c/215136/ 15:13:23 yeah, he is asking help mocking RPC in unit tests 15:13:38 yes - I pointed amuller at this patch 15:13:54 but the submitter isn't getting the recommendations 15:14:09 this patch still needs test work 15:14:42 so, should we wait for amuller to get back to the submitter with further help? 15:14:56 sorry, what patch? 15:15:16 amuller: https://review.openstack.org/#/c/215136/ 15:15:39 the submitter is asking for help mocking RPC 15:15:41 oh... 15:16:03 I don't think that's the only thing he's asking for =D 15:16:04 * carl_baldwin knows amuller loves helping people mock stuff 15:16:10 heh 15:16:28 I was trying to get him to write proper tests without mocking the entire world 15:17:02 :) 15:17:08 ++ 15:17:17 and I was hoping he could do that without me doing it for him 15:17:21 but it doesn't seem to be the case 15:18:11 I'll try to find a test that's similar enough that he could go on 15:18:31 amuller: thanks! 15:19:17 so that's all the bugs I have today.... regXboi do you have anything? or anyone in the team? 15:20:07 ok, let's move on..... 15:20:17 #topic Routed network segments 15:20:34 Hi 15:20:43 Still collecting reviews. 15:20:49 mlavalle: sorry no - you've hit my list 15:21:27 I would like amuller and kevinbenton to review soon as they both had a lot of feedback before. 15:21:32 #link https://review.openstack.org/#/c/225384/ 15:22:13 I need to start doing the work breakdown for this and get started on the first steps. 15:22:20 That is all from me. 15:22:30 thanks! 15:22:41 #topic DVR 15:22:51 mlavalle:hi 15:22:58 regXboi, haleyb, Swami you are up 15:23:14 I have been working with haleyb on some of the bugs that we have been seeing in upstream 15:23:36 Right now there are two bugs that might be the reason for the inconsistent behavior in the gate. 15:23:45 references? 15:24:06 regXboi: hold on I will paste the patches. 15:24:23 #link https://review.openstack.org/#/c/228582/ 15:24:42 #link https://review.openstack.org/#/c/229561/ 15:25:15 The second patch that I posted, still does not have a bug associated with it. I will file a bug today. Right now I am testing it in the gate to see if I am seeing the floatingip race condition. 15:25:21 I will add them to the wiki so we can track them easilly next week 15:25:32 Also the second patch needs some more work, from the server side to clean up the namespace. 15:25:52 Myself and haleyb were looking at the floatingip race condition yesterday from the log. 15:26:23 Swami: ok, that's the one where the namespace gets deleted from under us with a command in-flight 15:26:36 So in order to avoid this race, I am suggesting that we don't cleanup the fip namespace after every iteration and only clean up when the network is removed. 15:26:39 haleyb: yes 15:27:13 I did a similar change in the server side a while back but now I will use the same context to issue a namespace cleanup in the agent as well. 15:27:36 Once I have the server side up I will open it up for reviews. 15:27:50 from my point I'm working to accumulate some more coding resources for DVR tasks and next week I'll be looking at DVR performance 15:28:06 regXboi: thanks 15:28:26 regXboi: what do you mean by coding resources for DVR, from community or internal. 15:28:35 Swami: I mean internal 15:28:40 regXboi: ok 15:29:04 regXboi: let me know if i can help 15:29:16 haleyb: ack 15:29:17 also there was issue with the snat static routes and there is patch that addresses this issue. 15:29:37 #link https://review.openstack.org/#/c/228026/ 15:30:02 regXboi: please also count me in for any DVR issues that you are targetting upstream. 15:30:26 Swami: ack 15:30:27 i'll track that patchset in the wiki as well 15:30:33 Swami: ack 15:30:41 I’ll have some review time today. 15:30:43 carl_baldwin: regXboi: This patch needs your attention, I think I have addressed all comments. #link https://review.openstack.org/#/c/225319/ 15:31:04 Swami: ack - will look sometime today 15:31:20 I forgot to all one other patch that haleyb is working on 15:31:37 ip_lib patch to check for existence of a device before returns the handle for the device. 15:31:59 s/all/add 15:32:23 #link https://review.openstack.org/#/c/229010/ 15:32:33 that's all I have now for DVR 15:32:52 Swami: thanks! 15:32:56 hopefully by next week we can fix all the instability issues. Keeping my fingers crossed. 15:33:06 haleyb, regXboi: anything else? 15:33:19 mlavalle: nope - I expect to have more next week 15:33:28 nope 15:33:34 once I have the net node baselines in place to compare to 15:34:07 thanks, moving on 15:34:22 #topic BGP dynamic routing 15:34:27 Hi 15:34:33 hi there ! 15:35:41 I'm working on decomposing https://review.openstack.org/#/c/201621/, it's really big 15:36:00 thanks tidwellr :) 15:36:05 Really Big (tm) 15:36:12 carl_baldwin made a brave attempt at a review and ran out of gas 15:36:21 LOL 15:36:23 devvesa too :) 15:36:23 great 15:36:45 tidwellr: Overall, it is looking very god. 15:36:48 good 15:37:05 carl_baldwin: well, you know ;) 15:37:14 yes it does 15:37:27 vikram, how are the agent patches looking? 15:37:31 devvesa: glad to see you ;) 15:37:55 :) 15:37:56 I still need to make route discovery address scope aware 15:38:09 tidwellr: looks good to me ;)... Scheduler needs some attention 15:38:46 tidwellr: I’m going to hit address scopes hard as soon as I wrap up this DVR thing for DVR + HA. 15:39:04 carl_baldwin: it would be helpful if you have a glance over other patches 15:39:21 vikram: ack 15:39:27 yes, the agent patches are in better shape than https://review.openstack.org/#/c/201621/ 15:39:30 carl_baldwin: thanks 15:39:47 you won't run out of gas with those :) 15:39:50 devvesa: also promised to review those ;) 15:40:10 I did. And I will do :) 15:40:41 devvesa, carl_baldwin, tidwellr: thanks 15:40:41 I have been able to take the code we have up for review, deploy it, and actually route to tenant networks with it 15:40:51 ++ 15:40:58 tidwellr: awesome! 15:41:00 cool! 15:41:09 great work! 15:41:18 ++ 15:42:44 unless vikram has anything else, I think that's it on the BGP front 15:43:08 I don't have anything to highlight 15:43:18 tidwellr: We are in sync ;) 15:43:27 tidwellr, vikram thanks for the update 15:43:37 next one.... 15:45:56 #topic DNS 15:45:56 #topic DNS 15:45:56 so, functionality in https://review.openstack.org/#/c/212213/ for floating ips is complete 15:45:57 I have thoroughly tested it in my sandbox and seems to be bug free 15:45:57 I have also incorporated the designate team feedback as far as creating the zones for reverse lookups from and admin user context 15:46:21 mlavalle: Great! 15:46:31 I just realized yesterday that I forgot to make PTR pointers optional based on a config parameter. But that is a minor change that will be taken care of today 15:46:48 that was feedback from the designate team 15:47:18 so I am starting to work on the nova side to create instace ports with dns_name 15:47:32 and that will close the whole circle 15:47:58 mlavalle: Any chance the Jenkins tests can be cleaned up? Those tend to scare reviewers away. 15:48:09 I will also start bugging johnthetubaguy to make sure dns makes it to nova priorities in this cycle 15:48:42 carl_baldwin: I haven't paid attention to that because i think the problem is that we don't have designate in the gate 15:49:21 but over the next few days i plan to run the whoile test suite in my sandbox 15:49:36 so we have a better picture of that 15:49:57 mlavalle: ok 15:49:57 i will also ping anteaya to start working with the infra team to get designate + neutron in the gate 15:50:11 makes sense? 15:50:27 mlavalle: all you need to do is write the tests and jobs 15:50:33 Regarding Nova. I think we should have the code up. Otherwise, I think it will go like it has before. They haven’t come through for us. 15:50:37 mlavalle: I can help you find samples if you need them 15:50:48 mlavalle: find me in infra after the meeting 15:50:51 carl_baldwin: agree 15:51:09 anteaya: i'll ping you today or tomorrow, thanks! 15:51:14 sure 15:51:28 that's all I have today 15:51:56 #topic Address Scopes 15:52:12 carl_baldwin: since you mentioned this above, you want to discuss any further? 15:52:22 I kind of gave an update here. Will start hitting it hard soon. 15:52:31 cool 15:52:36 in that case 15:52:49 #topic Open agenda 15:53:15 I have a personal announcement to share with the team 15:53:24 tomorrow is my last day at Rackspace 15:53:48 this coming Monday I start at IBM as an OpenStack developer focused in Neutron 15:54:07 uopstream, so you guys are not getting rid of me..... :-) 15:54:07 You’ll still be the same mlavalle to us! We’re happy you’ll have time to continue working with us. 15:54:15 mlavalle: Best of luck! 15:54:24 mlavalle: good to see you're still involved in OS. 15:54:27 thanks! 15:54:29 mlavalle: congrats! and best of luck. 15:55:14 any other open topics? 15:56:01 ok, in that case, let's give back 5 minutes back to everybody's agenda 15:56:07 #endmeeting