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