17:01:21 <Sukhdev> #startmeeting networking_l2gw 17:01:22 <openstack> Meeting started Mon Jul 18 17:01:21 2016 UTC and is due to finish in 60 minutes. The chair is Sukhdev. Information about MeetBot at http://wiki.debian.org/MeetBot. 17:01:24 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:01:26 <openstack> The meeting name has been set to 'networking_l2gw' 17:01:45 <Sukhdev> can I see a show of hands who is here to attend L2GW meeting? 17:02:07 <oferby> I'm here 17:02:35 <Sukhdev> I see bunch of guys on the channel - wonder who else is here 17:02:50 <Sukhdev> #topic: Agenda 17:02:54 <Sukhdev> #link: https://wiki.openstack.org/wiki/Meetings/L2Gateway#Meeting_July_18.2C_2016 17:03:12 <Sukhdev> #topic: Announcements 17:03:29 <Sukhdev> oferby is a new core member of this team 17:03:36 <Sukhdev> congratulations oferby 17:03:43 <oferby> :-) thanks 17:04:08 <Sukhdev> #topic: Patches under review 17:04:21 <Sukhdev> #link: https://review.openstack.org/#/q/status:open+project:openstack/networking-l2gw,n,z 17:04:36 <Sukhdev> I approved some patches over the weekend 17:05:09 <Sukhdev> I noticed new ones have popped up this morning 17:05:24 <Sukhdev> I am glad to see oferby is here to help out with these 17:05:46 <oferby> I add small change just for small cleanup 17:06:05 <Sukhdev> cleanup is always good :-) 17:06:19 <oferby> if you are ok with this I will continue ... 17:07:16 <Sukhdev> oferby : I will review it later this evening and post comments 17:07:27 <oferby> ok 17:07:39 <Sukhdev> also, I will be available on-line this evening, should you need to reach me 17:07:55 <oferby> ok 17:08:14 <Sukhdev> #topic: OVSLIB 17:08:28 <Sukhdev> oferby : thanks for taking on this 17:08:53 <Sukhdev> one of my concerns is keeping this updated with the on-going changes 17:09:19 <oferby> I totally understand and this was my concern too - 17:09:43 <oferby> but could not convince OVS people ... 17:10:10 <oferby> the issue is not that big - 17:10:29 <oferby> the schema is used by IDL only to ask for tables and columns 17:10:32 <Sukhdev> I have just added russell as a reviewer to this patch - to seek his feedback 17:10:56 <oferby> he was involved before Ben .... 17:11:11 <Sukhdev> is he not still involved? 17:11:34 <oferby> stopped after Ben made his view on this 17:12:08 <Sukhdev> hmm.... I do not see him on IRC - 17:12:29 <Sukhdev> will try to ping him later during the day and ask him to provide feedback - 17:13:27 <Sukhdev> armax: are you here? 17:15:29 <Sukhdev> oferby : I will try to reach armax as well russell later during the day to see if we can get him to review the patch as well 17:15:44 <oferby> ok. 17:15:54 <Sukhdev> I like what I saw - I think this is a good staring point to get things moving forward 17:16:20 <oferby> I have few thinks that I would like to do in neutron-lib to help us - 17:16:37 <oferby> it would be nice if you could ask armax to join. 17:16:56 <Sukhdev> I will 17:17:21 <Sukhdev> feel free to shoot him an email - if your timing on the IRC does not work out - 17:17:29 <oferby> just for you to know what it is about - 17:17:49 <oferby> there are some classes in neutron that we need - 17:17:58 <oferby> Connection, Transaction, etc - 17:18:12 <Sukhdev> right 17:18:23 <oferby> I don't want to copy them, I rather use small package like neutron-lib - 17:18:38 <oferby> so I would like to move them from neutron to lib 17:19:00 <oferby> I would like to have armax ok before doing the work ... 17:19:20 <oferby> you view on it?! 17:19:21 <Sukhdev> why can't we simply import it? 17:19:44 <oferby> because neutron is too big of a code and we need only fraction. 17:20:04 <oferby> if you have a node with only the l2gw agent - 17:20:20 <Sukhdev> can't we just import the classes that we are interested in? 17:20:21 <oferby> you will need to install neutron-server code. 17:20:32 <oferby> no. dependencies ..... 17:20:41 <Sukhdev> ah I see 17:20:54 <oferby> also installation script will not allow this. 17:21:14 <Sukhdev> I do not like the idea of running neutron server on the hypervisors 17:21:31 <oferby> you do not run it 17:21:47 <oferby> just install all the code to use small part of it. 17:21:55 <oferby> silly .... 17:22:10 <Sukhdev> understood 17:22:49 <Sukhdev> but, I have a feeling, if you import neutron-lib anywhere, you are getting the code anyways 17:23:31 <oferby> you don't get all neutron code which is big. only small part. 17:24:14 <Sukhdev> I mean the lib - not all of neutron server 17:26:53 <Sukhdev> let me follow up with armax and russell on this 17:27:01 <Sukhdev> will get back to you 17:28:08 <Sukhdev> #topic: L2GW as cloud inter-connect 17:29:12 <Sukhdev> oferby: looks like you were disconnected 17:29:20 <oferby> yes I was... 17:29:40 <Sukhdev> so, on the cloud inter-connect 17:30:22 <Sukhdev> The spec looks good to me 17:30:36 <Sukhdev> I noticed few more reviews - which is always good 17:30:48 <Sukhdev> We should wrap this up - 17:31:08 <oferby> right 17:32:53 <oferby> did you get disconnected this time?! 17:33:08 <Sukhdev> I will give it a final read tonight and get this done 17:33:19 <oferby> great. 17:33:20 <Sukhdev> probably 17:34:02 <Sukhdev> #topic: Open Discussion 17:34:35 <Sukhdev> oferby : anything else you would like to discuss? 17:34:48 <oferby> I spoke with Swami earlier - 17:34:53 <Sukhdev> oh good 17:34:59 <oferby> asked him to add comments on the DVR 17:35:02 <Sukhdev> Swami : are you here? 17:35:09 <oferby> he promised he will today. 17:35:19 <Swami> Sukhdev: yes here 17:35:32 <Sukhdev> Swami : we need to close off on that patch - 17:35:46 <Swami> I just posted a couple of comments in there, once I get the answers I will be comfortable giving a +1 17:36:09 <Sukhdev> oferby : have you read the comments? 17:36:24 <Sukhdev> or lets quickly go over those comments here 17:36:35 <oferby> ok. looking. 17:36:51 <Sukhdev> Swami : do you mind articulating your concern here - so that we can try to get a closure here 17:37:00 <Swami> Sukhdev: sure 17:37:33 <Swami> The question that I had was what does the endpoints contain? Does it have all the vxlan tunnel endpoints configured. 17:38:09 <Swami> The next question is, if you are matching with the endpoint[host] to the agent['host'] how do you determine if this is the active host, if HA is enabled. 17:38:49 <oferby> the endpoint contain host information. I look for the host name and resolve its IP address 17:39:19 <Swami> oferby: ok, 17:40:15 <Swami> oferby: so how do you determine, if this is the valid agent if HA is enabled for the network node, is there anyplace that you check for those conditions. 17:40:53 <oferby> in HA case there will be multiple hosts and I'm selecting the first active one. 17:40:57 <Swami> oferby: Or is that taken care somewhere else that is not in this patch. 17:41:20 <Swami> oferby: so do you check for the active state of the agent in the network node. 17:41:39 <oferby> yes. the db query return only active nodes. 17:42:28 <Swami> oferby: you mean the filter with 'alive['True']' will take care of it. 17:42:36 <oferby> right 17:42:43 <Swami> oferby: ok, no problem. 17:42:56 <Swami> I think I got what I wanted, I will +1 it right now. 17:43:02 <Swami> oferby: Sukhdev: thanks 17:43:08 <oferby> great. thanks. 17:43:24 <Swami> oferby: jenkins is still failing. 17:43:30 <Sukhdev> Swami : you can post your comments based upon this discussion - 17:43:31 <oferby> looking. 17:43:55 <Sukhdev> oferby : can you please take care of the py27 errors? 17:44:14 <Sukhdev> Once Swami gives a +1, and jenkins is happy, I will approve this for merge 17:44:20 <oferby> I sure will. my tomorrow .... 17:44:29 <Sukhdev> sounds good 17:44:45 <Sukhdev> Swami : thanks for your time - much appriciated 17:44:54 <Swami> Sukhdev: no problem 17:45:23 <oferby> I see the problem. it is a fix for package that was missing in unit tests. 17:45:34 <oferby> the fix was merged. 17:45:37 * Sukhdev owes Swami a beer in Barcelona 17:45:53 <Swami> Sukhdev: sure, lets sync up 17:46:27 <Sukhdev> oferby : so, rebasing may take care of it then 17:46:39 <oferby> it will. I will do that tomorrow. 17:46:53 <oferby> your tonight... 17:47:01 <Sukhdev> sounds good - thanks oferby 17:47:18 <Sukhdev> anything else that we need to discuss? 17:47:22 <oferby> nop. 17:47:29 <Sukhdev> good - so, we are done then 17:47:37 <Sukhdev> Thanks for attending 17:47:43 <Sukhdev> bye 17:47:50 <oferby> ok. have a great day. bye. 17:48:01 <Sukhdev> #endmeeting