14:00:05 <mlavalle> #startmeeting neutron_drivers
14:00:06 <openstack> Meeting started Fri Dec  7 14:00:05 2018 UTC and is due to finish in 60 minutes.  The chair is mlavalle. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:07 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:09 <openstack> The meeting name has been set to 'neutron_drivers'
14:00:18 <njohnston> o/
14:00:48 <mlavalle> hi njohnston
14:01:09 <njohnston> How are you doing mlavalle?  Happy Friday!
14:01:23 <mlavalle> same to you, njohnston
14:01:37 <Chenjie> mlavalle: hi, I have another 2 RFE to discuss in future driver meetings
14:01:46 <Chenjie> https://bugs.launchpad.net/neutron/+bug/1803494
14:01:47 <openstack> Launchpad bug 1803494 in neutron "[RFE] Add l2pop support for floating IP resources" [Undecided,New] - Assigned to ChenjieXu (midone)
14:01:53 <Chenjie> https://bugs.launchpad.net/neutron/+bug/1806316
14:01:54 <openstack> Launchpad bug 1806316 in neutron "[RFE] Add RPC query API to l2pop for FDB resync" [Wishlist,New] - Assigned to ChenjieXu (midone)
14:02:20 <mlavalle> Chenjie: ack
14:02:31 <Chenjie> mlavalle: Thank you so much!
14:02:43 <mlavalle> haleyb indicated that he might be a little late
14:02:55 <mlavalle> slaweq: you around? feeling better?
14:03:12 <mlavalle> amotoki: ping
14:03:59 <rubasov> o/
14:04:07 <mlavalle> hey rubasov
14:04:14 <rubasov> hello
14:04:48 <rubasov> I'm just here to listen to learn how this goes
14:05:06 <amotoki> hi
14:05:14 <mlavalle> so slaweq sent an email last night indicating he was sick
14:05:26 <mlavalle> so he is very unlikely to attend today
14:05:39 <mlavalle> amotoki is here :-)
14:05:55 <mlavalle> and I know haleyb will be here, although a little late
14:06:17 <mlavalle> let's give him a few minutes to achieve quorum
14:07:52 <mlavalle> yamamoto is also sick today, so he won't attend
14:08:19 <mlavalle> it must be the cold weather in the northern hemisphere
14:08:41 <njohnston> yes, so many are sick
14:09:27 <amotoki> I am staying a southern island in Japan this week for an event and it is much warmer than Tokyo by 10~15 degree :)
14:10:08 <amotoki> I need to care myself when I return to Tokyo tomorrow
14:10:14 <mlavalle> although, at least today, it is not that cold in Wroclaw (where slaweq lives). it is about 10C (51F)
14:11:06 <mlavalle> well, today Tokyo is ~9C (50F)
14:11:10 <mlavalle> not that cold
14:11:11 <amotoki> It still sounds cold to me :p
14:11:19 <mlavalle> LOL
14:11:21 <mlavalle> yeap
14:11:30 <amotoki> it is 25~28C here
14:11:56 <mlavalle> by all means, amotoki, please take care of yourself when you go back to Tokyo
14:13:05 <amotoki> thanks
14:13:20 <mlavalle> Let's give haleyb 3 more minutes and then we start discussing Chenjie's RFE, which is where we left off last week
14:13:34 <njohnston> I just sent haleyb a message in a side channel
14:14:01 <mlavalle> njohnston: he sent me an email earlier today indicating he had to run some errands
14:14:07 <njohnston> ah ok
14:14:35 <slaweq> hi, sorry for being late
14:14:49 <slaweq> I'm feeling bit better so I can attend our meeting
14:15:16 <mlavalle> slaweq: are you sure? we understand if you need to rest
14:15:36 <mlavalle> #topic RFEs
14:15:42 <slaweq> no, it's fine. I rest all day :)
14:15:51 <mlavalle> Great!
14:15:57 <amotoki> slaweq: good to hear that you are getting better :)
14:16:07 <mlavalle> so let's start here: https://bugs.launchpad.net/neutron/+bug/1793653
14:16:08 <openstack> Launchpad bug 1793653 in neutron "[RFE] Enable other subprojects to extend l2pop fdb information" [Wishlist,In progress] - Assigned to ChenjieXu (midone)
14:16:11 <njohnston> slaweq: Glad you'redoing better!
14:16:20 <slaweq> thx guys
14:17:25 <mlavalle> Chenjie: I read it carefully yesterday and overall it makes sense to me
14:17:46 <mlavalle> I have a a couple of questions, though:
14:18:03 <Chenjie> mlavalle: thanks!
14:19:05 <mlavalle> 1) For this to work, there must a corresponding change in a subproject, I suspect in this case networking-bgpvpn, right?
14:19:36 <mlavalle> Chenjie: ^^^
14:19:51 <haleyb> sorry i'm late, darn laptop
14:20:13 <mlavalle> haleyb: no problem. thanks for showing up
14:23:01 <Chenjie> mlavalle: yes,it should be networking-bgpvpn in this case.
14:23:24 <Chenjie> mlavalee: sorry for late reply. I didn't see your question.
14:23:57 <mlavalle> Chenjie: has that changed been approved in that subproject or is it under consideration? I looked in Gerrit and couldn't find anything: https://review.openstack.org/#/q/project:openstack/networking-bgpvpn+owner:%22ChenjieXu+%253Cchenjie.xu%2540intel.com%253E%22
14:24:23 <mlavalle> Chenjie: of course I'm assuming you are the person implementing that change
14:25:10 <mlavalle> Chenjie: so what is the status on the bgpvpn side?
14:25:31 <Chenjie> mlavalle: It has not been proposed to make the change in bgpvpn.
14:25:54 <Chenjie> mlavalle: I think the change will be proposed after this RFE merged.
14:26:16 <mlavalle> Chenjie: has there at least been any discussion with them?
14:27:02 <Chenjie> mlavalle: sorry, I don't know. I can ask.
14:30:17 <mlavalle> Chenjie: second question 2) Have you deployed the proposed Neutron patch (https://review.openstack.org/#/c/599319) with bgpvpn (with a corresponding change to register a fdb extending function) and shown it works?
14:32:22 <Chenjie> mlavalle: Sorry, I didn't test this with BGPVPN. I just test the RFE with a simple case written by myself.
14:33:37 <Chenjie> mlavalle: I just sent an email to ask whether the bgpvpn side will be upstreamed.
14:35:26 <mlavalle> Chenjie: mhhhh. can you clarify that. so when we are talking about bgpvpn, aren't we talking about https://github.com/openstack/networking-bgpvpn?
14:37:52 <Chenjie> mlavalle: yes, I can. We are talking about this project.
14:38:49 <mlavalle> Chenjie: so there is nothing to be upstreamed. This project is already "upstreamed" in the the open
14:39:41 <mlavalle> Chenjie: and that means a corresponding change has to be proposed to https://github.com/openstack/networking-bgpvpn, correct?
14:40:28 <Chenjie> https://github.com/starlingx-staging/stx-networking-bgpvpn
14:40:51 <amotoki> what is the relationship between networking-bgpvpn and stx-networking-bgpvpn?
14:40:59 <Chenjie> mlavalle: the change in this project should be upstreamed to project networking-bgpvpn
14:41:25 <Chenjie> mlavalle: stx-networking-bgpvpn is based on networking-bgpvpn
14:41:35 <mlavalle> Chenjie: is it a fork?
14:41:47 <Chenjie> mlavalle: stx-networking-bgpvpn is downstream.
14:42:01 <Chenjie> mlavalle: yes.
14:43:28 <amotoki> looking at the repo, stx-netwokring-bgpvpn is a fork, not  a downstream.
14:43:47 <mlavalle> Chenjie: so the corresponding change has to be upstreamed to https://github.com/openstack/networking-bgpvpn, correct?
14:44:44 <Chenjie> mlavalle:yes, but I don't know will they propose to upstream those changes. I just sent an email to ask about that.
14:45:08 <mlavalle> amotoki: I think that this type of repo will disappear in the near future. The idea is that StarlingX converges with OpenStack master in gneral
14:45:34 <Chenjie> mlavalle: yes!
14:45:35 <amotoki> mlavalle: yeah, my undestanding is same
14:45:41 <mlavalle> the plan is for this convergence to take place sonn after te end of Stein
14:47:10 <Chenjie> mlavalle: thank you for your information!
14:48:01 <amotoki> it seems we first need to discuss a RFE for a specific user-visible feature like l2pop support in bgpvpn. the current RFE we are discussing is about how to achieve this.
14:48:09 <mlavalle> Chenjie: I am hesitant to make this change in the Neutron side if we don't have the buy in from the the upstream networking-bgpvpn team
14:48:57 <mlavalle> why make this change in Neutron when the the bgpvpn upstream might say it won't work for some reason
14:49:00 <mlavalle> ?
14:49:40 <Chenjie> mlavalle: https://bugs.launchpad.net/neutron/+bug/1803494
14:49:41 <openstack> Launchpad bug 1803494 in neutron "[RFE] Add l2pop support for floating IP resources" [Undecided,New] - Assigned to ChenjieXu (midone)
14:50:06 <slaweq> but IMO even if bgpvpn will not use it, can't it be used by other projects as well? Maybe some 3rd party projects?
14:50:24 <Chenjie> mlavalle: this RFE can alse be a use case for RFE "Enable other projects to extend l2pop fdbs"
14:50:54 <Chenjie> mlavalle: this RFE just propose to make changes in neutron.
14:52:09 <mlavalle> slaweq: yes.... but in principle why add a feature that doesn't have an approved use case yet.....
14:52:56 <mlavalle> having said that, I buy the posibility that this latest RFE might be such use case: https://bugs.launchpad.net/neutron/+bug/1803494
14:52:58 <openstack> Launchpad bug 1803494 in neutron "[RFE] Add l2pop support for floating IP resources" [Undecided,New] - Assigned to ChenjieXu (midone)
14:53:22 <mlavalle> but now we need to take a look at this one
14:53:44 <Chenjie> mlavalle: that's ok with me.
14:54:29 <mlavalle> Chenjie: let me be clear, I am not trying to shut dowan your RFE. I also wear a StarlingX hat, as you know
14:54:46 <mlavalle> I want StarlingX to succeed
14:54:54 <Chenjie> mlavalle: during this period, I think I can ask about the opinions of BGPVPN team.
14:55:09 <Chenjie> mlavalle: yes, I understand.
14:55:19 <mlavalle> here's what I propose to go forward
14:55:32 <mlavalle> 1) Let's review this latest RFE
14:56:27 <mlavalle> 2) I volunteer to facilitate a conversation with the networking-bgpvpn team. I want to hear what they have to say
14:57:28 <mlavalle> Chenjie: does it make sense^^^^?
14:57:43 <Chenjie> mlavalle:it makes sense to me.
14:58:00 <Chenjie> mlavalle: thank you for your proposal!
14:58:10 <mlavalle> any other thoughts, amotoki, haleyb, slaweq, njohnston, rubasov?
14:58:29 <haleyb> none from me
14:58:30 <slaweq> sounds good to me
14:58:44 <njohnston> sounds good to me
14:58:48 <amotoki> sounds good
14:59:15 <mlavalle> ok, thanks for attending. Have a great weekend!
14:59:19 <amotoki> mlavalle: which stadium projects do we cover by neutron RFEs?
14:59:39 <amotoki> we sometimes cover fwaas RFEs. how aobut networking-bgpvpn?
15:00:02 <mlavalle> amotoki: I think is a project by project. I'll ask tmorin
15:00:11 <mlavalle> #endmeeting