15:30:29 #startmeeting neutron-drivers 15:30:29 Meeting started Wed Jun 17 15:30:29 2015 UTC and is due to finish in 60 minutes. The chair is mestery. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:30:31 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:30:33 The meeting name has been set to 'neutron_drivers' 15:30:41 #link https://wiki.openstack.org/wiki/Meetings/NeutronDrivers Agenda (same as every week) 15:30:53 carl_baldwin: ping if you're here 15:31:13 pong cuz I’m here 15:31:25 awesome! 15:31:28 OK lets get rolling 15:31:31 #topic Spec Discussion 15:31:40 #link https://review.openstack.org/#/c/184857/ 15:31:41 "Get Me a Network" 15:31:56 The plan with this one is to work the design out next week in Fort Collins, especially the concurrency issues 15:32:03 And once we do that, find someone to implement it :) 15:32:10 Any comments? 15:32:11 Concerns? 15:32:31 no, that jives with what i thought would work best. 15:33:14 cool 15:33:20 Next up 15:33:24 #link https://review.openstack.org/#/c/169612 Availability zones 15:33:49 amotoki had a lot of comments on revision 23 of htis one 15:33:55 which need addressing, thanks for hte review amotoki 15:34:03 AZ one generally looks good. i posted some clarifications. 15:34:23 amotoki: Great! 15:34:33 I think this is a nice one to get in for Liberty. carl_baldwin dougwig, have you had a chance to review? 15:34:46 i have not 15:34:54 OK 15:35:04 mestery: Not much review there. I can soon if needed. 15:35:23 amotoki: Thank you for your review! I will update tomorrow. 15:35:23 carl_baldwin: No worries, I know russellb amotoki and I have reviewed, among others 15:35:24 hmm, i see one issue, though it's small. ignores services entirely. 15:35:32 dougwig: Good catch! 15:35:40 And worth a re-spin to accomodate that 15:35:47 i'll comment today 15:35:55 dougwig: i noticed it too, but it can be deferred I think. 15:36:41 but it is worth mentioned at least. 15:37:16 amotoki_: yes, it could, though i find it frustrating when features are supported piecemeal. 15:37:24 Agreed 15:37:28 agree. 15:37:49 Next up 15:37:53 #link https://review.openstack.org/#/c/94612/ VLAN Aware VMs 15:38:04 I know armax has been dilligently following up on this, and there was a meeitng yesterday on it 15:38:10 armax: Can you summarize if you're around? :) 15:38:53 And if armax is busy, the tl;dr is that some progress was made on this one 15:38:54 mestery: I think we agreed to choose first class citizen for trunk port resource 15:38:59 armax: Thanks :) 15:39:07 they’d need to extend the port resource to track subports 15:39:20 they’ll respin the patch and hopefully we’ll start seeing some code 15:39:23 #info trunk ports will be a first class citizen but subports will be an extension of ports 15:39:26 awesome 15:39:28 thanks armax! 15:39:30 Questions? 15:39:32 and we’ll take it from there 15:40:15 there is another spin of the spec, and we can check it. 15:40:23 amotoki_: Yes 15:40:35 i think kevinbenton had some issues with subports. was he at the meeting? else just flag him to review. 15:40:44 He was not there sadly 15:41:01 dougwig: no he wasn't 15:41:12 amotoki was there though 15:41:29 dougwig: we’ll ping him 15:41:39 armax: ah.... you're right. 15:41:50 cool. i do want to see trunk ports happen. 15:42:26 we all do 15:42:40 OK, last one 15:42:46 #link https://review.openstack.org/#/c/172244/ Routed Networks 15:42:48 carl_baldwin: This is your baby :) 15:43:03 The good news is Kris from godaddy is hopefully going to make the mid-cyclen ext week to discuss this with us 15:43:16 And how it can (hopefully) be evolved to address the segment support those folks want 15:43:21 mestery: That is good news. I look forward to the discussion. 15:43:54 should we put this spec review on hold until the mid-cycle, then? 15:44:19 dougwig: +1 15:44:21 dougwig: +1 15:44:27 It could use just a bit more definition. 15:44:40 +1 15:44:56 … and some validation from others that my ideas are sane would be good. 15:45:02 carl_baldwin: ha, your specs are usually among the most detailed already. :) 15:45:14 * carl_baldwin blushes 15:45:26 carl_baldwin: I know regXoi wants to discuss this in your L3 meeting tomorrow, look for him there 15:45:32 He reached out to me this morning on this one 15:45:45 mestery: Thanks, I will look for him. 15:46:19 OK, does anyone want to bringup any other particular spec? 15:46:37 question: are there any important specs in *aaS areas? 15:47:06 Bug #1464465 15:47:06 bug 1464465 in neutron "binding_type improvements to simplify Nova interactions" [Undecided,Confirmed] https://launchpad.net/bugs/1464465 15:47:34 amotoki_: for lbaas, the big items are l7 redirection, horizon, and octavia. l7 was a backlog from kilo, octavia is happening in a different repo, but we do need to talk at some point about how to get horizon support in for lbaas v2. 15:47:52 for fwaas, i think xgerman is putting together use cases in prep for taking a look at maybe a v2 api 15:47:53 dougwig: So, lbaas is covered 15:48:01 For fwaas, we're in the process of rebooting it a bit, so nothing at this time. 15:48:03 i'm not sure about vpn. 15:48:10 dougwig: ++ 15:48:18 I need to talk to pc_m about vpn today 15:48:27 dougwig: for horizon support, I can help all efforst and I can discuss offline 15:48:27 Sorry are we not at RFEs yet? 15:48:29 It's on my list, it's just not burning me enough to get my attention yet :) 15:48:31 HenryG: Not yet 15:48:32 :) 15:48:40 amotoki_: Awesome, thank you! 15:48:52 amotoki_: ok, let's do that. i'm also interested in if we can do that panel in-repo, instead of having it all in horizon (like the devstack model) 15:48:54 for vpnaas, I need to clarify what are prioritized. 15:49:30 dougwig: horizon team are exploring how to scale dahsborad efforts. 15:49:44 amotoki_: ok, we'll take our cue on how to proceed from you, then. 15:49:56 Great! 15:50:04 Any other specs before we move to RFEs? 15:50:09 dougwig: I will share efforts in horizon later. 15:50:33 for vpnaas, I will talk paul what should be prioritized. 15:50:45 ah... i wrote already.. 15:50:58 :) 15:51:02 OK, lets move to RFEs then 15:51:04 #topic RFEs 15:51:11 #link https://bugs.launchpad.net/neutron/+bugs?field.tag=rfe RFE bug link 15:51:48 HenryG: You had one for us? 15:51:54 We can start with yours :) 15:51:59 Bug #1464465 15:52:00 bug 1464465 in neutron "binding_type improvements to simplify Nova interactions" [Undecided,Confirmed] https://launchpad.net/bugs/1464465 15:52:11 It's from ijw 15:52:16 better query: 15:52:17 https://bugs.launchpad.net/neutron/+bugs?field.searchtext=&orderby=-importance&field.status%3Alist=NEW&field.status%3Alist=TRIAGED&assignee_option=any&field.assignee=&field.bug_reporter=&field.bug_commenter=&field.subscriber=&field.structural_subscriber=&field.tag=rfe&field.tags_combinator=ANY&field.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&fi 15:52:17 eld.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field.has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on&search=Search 15:52:23 man, let me shorten that 15:52:32 dougwig: rookie mistake! :) 15:52:44 https://goo.gl/dfUdUf 15:52:52 only new and triaged RFE's. ^^ 15:52:59 cool 15:53:37 Why not "confirmed"? 15:54:07 my understanding is that 'confirmed' is 'approved by the drivers/LT'. do i need to go read that process doc again? sec. 15:54:22 * HenryG may have wongly set the RFE to confirmed 15:54:29 :) 15:54:50 ahh, triaged means approved. 15:54:52 one sec. 15:55:33 https://goo.gl/xtBJkU 15:55:49 HenryG: Seems as if the binding RFE proposed by ijw is dependent on a nova spec, true? 15:56:02 mestery: yes 15:56:32 I think he is asking for approval to work on the Neutron side if that goes ahead 15:56:56 confirmed means they have been acked as RFE bugs 15:56:57 it seems we need to consider it along with the nova vif script spec. 15:57:19 triaged means someone from the drivers team has seen them and started working on them 15:57:38 as in talking and chatting and fighting 15:57:44 insulting…you name it 15:57:54 in progress 15:58:26 is when someone hopelessly start coding believing that the rfe will be made available only to realize that a final -2 will put his/her hopes in tatters 15:58:26 rofl 15:58:30 https://goo.gl/xtBJkU <-- new + confirmed RFE's 15:58:39 I think binding improvements to match Nova is a reasonable request, hence I marked it as confirmed 15:58:44 dougwig: How many more URLs will you generate for us? 15:59:01 HenryG: cool 15:59:07 mestery: gonna start being some ascii goats soon. 15:59:10 I have no idea if I have the authority to mark as confirmed ;) 15:59:23 HenryG: We'll slap you on the wrist if you do something wrong, don't worry 15:59:57 I removed the RFE tag from this one: https://bugs.launchpad.net/neutron/+bug/1464190 15:59:58 Launchpad bug 1464190 in neutron "Fujitsu Neutron ML2 Mechanism Driver for ISM" [Wishlist,Confirmed] - Assigned to Yushiro FURUKAWA (y-furukawa-2) 16:00:27 This one needs an update from the proposer: https://bugs.launchpad.net/neutron/+bug/1457034 16:00:27 Launchpad bug 1457034 in neutron "BGPVPN extension" [Undecided,New] 16:00:50 We may as well just mark this as triaged: https://bugs.launchpad.net/neutron/+bug/1458890 16:00:51 Launchpad bug 1458890 in neutron "Add segment support to Neutron" [Undecided,Confirmed] 16:00:57 Because it's gonna happen, so many operators need this one 16:01:14 For such cases, do we set the status to Imcomplete? or keep it New? 16:01:38 i am talking about BGPVPN one. 16:01:40 amotoki_: We set it to "Triaged" 16:01:52 amotoki_: Lets leave it as "Confirmed" since we've alreayd looked at it 16:02:10 Thoughts on https://bugs.launchpad.net/neutron/+bug/1460720 ??? 16:02:11 Launchpad bug 1460720 in neutron "Add API to set ipv6 gateway" [Undecided,New] - Assigned to Abishek Subramanian (absubram) 16:02:18 we need a wiki page or doc cheat sheet. "rfe bug states for dummies" 16:02:21 <-- the dummy 16:02:35 #action dougwig to write "RFE for dummies" page 16:02:40 dougwig: Sound good? ;) 16:02:42 You see what I did there? 16:02:45 lol 16:02:47 lol, fine. :) 16:02:51 :-) 16:03:18 mestery: that RFE can at least be marked confirmed 16:03:29 HenryG: done 16:03:52 HenryG: It's an API change, but it seems sensible to me. 16:04:03 It was discussed but I have not seen any work started yet 16:04:15 (in the L3 meeting) 16:04:15 HenryG: I am fine moving this to triaged and allowing work to proceed 16:04:19 anyone have any issues there? 16:04:22 going once 16:04:24 going twice 16:04:27 * mestery waits for 30 seconds 16:04:32 +1 16:04:40 Thanks carl_baldwin 16:04:42 sold to carl_baldwin ! :) 16:04:42 We came up with a good plan, I think. 16:04:45 triaged it is! 16:04:52 yay 16:05:04 Now on to armax's favorite RFE: https://bugs.launchpad.net/neutron/+bug/1461133 16:05:05 Launchpad bug 1461133 in neutron "Supporting multiple L3 backends" [Undecided,Confirmed] 16:05:12 Just as long as sold to doesn’t mean assigned to 16:05:19 carl_baldwin: lol 16:05:22 heh 16:06:10 I think everyone thinks we need ML3 16:06:16 It is actually ML3 work. it sounds reasonable 16:06:18 And I think the way forward is to use flavors (where's dougwig and my flavors update) 16:06:27 amotoki_: Lets move this one to Triaged then and be done with it 16:06:32 We can iterate more on the code and devref 16:06:43 mestery: agree. 16:06:57 cool, done! 16:06:58 next up 16:07:04 https://bugs.launchpad.net/neutron/+bug/1463594 16:07:05 Launchpad bug 1463594 in neutron "LBaaS drivers can be queried to determine whether they support a feature the API exposes" [Undecided,Confirmed] 16:07:08 #link https://bugs.launchpad.net/neutron/+bug/1463594 16:07:16 I discussed this with blogan and dougwig yesterday I believe 16:07:28 The result is mentioned in comment #4 16:07:34 blogan will write a spec with specific use cases 16:07:40 Because an RFE for this one wasn't enough 16:07:43 dougwig: Did I capture that accurately>? 16:07:49 yes, we can punt this until we see a spec. 16:08:06 cool 16:08:19 OK, next up: 16:08:21 #link https://bugs.launchpad.net/neutron/+bug/1464361 16:08:22 Launchpad bug 1464361 in neutron "Support for multiple gateways in Neutron subnets" [Undecided,New] - Assigned to Shraddha Pandhe (shraddha-pandhe) 16:08:33 carl_baldwin: I think you'd want to review this one 16:08:50 i'm leaning towards -1 16:08:54 requires data model changes 16:08:58 and dhcp agent changes 16:09:00 kevinbenton: Was thinking the same 16:09:10 kevinbenton: Ignore the implmentation details for a minute 16:09:15 THe use case seems valid to me 16:09:18 Agree? 16:09:49 maybe 16:09:56 carl_baldwin: your thoughts on the use case? 16:09:56 use cases looks valid 16:09:58 maybe? 16:10:02 is it common to give servers different gateways in the same subnets? 16:10:09 but i am not sure the approach looks good or not. 16:10:40 kevinbenton: Lets comment on the bug, if the use case is valid, we can then work a solution 16:10:43 I’m not sure how common this is. I’ve never seen it but that does mean much. 16:10:45 Thus, the RFE process in action! 16:12:00 I've commented and marked it confirmed 16:12:12 Last one 16:12:13 #link https://bugs.launchpad.net/neutron/+bug/1464793 16:12:14 Launchpad bug 1464793 in neutron "Add a driver for isc-dhcpd" [Undecided,New] - Assigned to Shraddha Pandhe (shraddha-pandhe) 16:12:34 +1 16:12:39 only question is where it should live 16:12:55 right 16:12:59 you mean, in it's own repo? 16:13:15 yeah, or do we just pull it into neutron? 16:14:04 I would not like to pull this in, if we do the reference split (still on the table), it could go in that repo 16:14:25 I am not sure the current neutron driver impl can support proper driver mechanism 16:14:34 amotoki_: There's also that :) 16:14:35 after ref split, we can do it in that repo. 16:15:01 i think it's a no brainer to confirm this one, though. 16:15:03 yeah, we have, but we only support dnsmasq at now. 16:15:24 i guess it can just go in wherever the reference lives for now 16:15:25 dougwig: Already confirmed 16:15:31 and commented based on this discussion 16:15:57 That was the last RFE 16:16:00 #topic Open Discussion 16:16:08 Anything else or shall we close this meeting? 16:16:37 shall we discuss the deadlock issue really quick? 16:16:45 kevinbenton: Please, go ahead! 16:16:56 I know carl_baldwin is here and you have been discussing that with him on the ML, so yes, makes sense 16:17:12 it has come to my attention that with galera multi-master, any random thing can deadlock to test a programmers patience 16:17:40 should we just jam the retry mechanism high up in the API layer for every operation and call it a day? 16:18:11 I'd be curious what jaypipes has to say about that given his DB experience kevinbenton. 16:18:21 Have you run that by him by chance? 16:18:26 what's the root of the issue? crazy txn nesting? not locking tables in a defined order? 16:18:28 mestery: +1 16:18:30 i will in my secret meeting with him in 10 minutes 16:18:34 rofl 16:18:40 and have him reply to the ML 16:18:44 OK, let us know how it goes, his guidance is apprecaited 16:18:46 cool 16:18:50 We should also make sure keep zzzeek in the loop 16:18:54 +1 16:19:02 dougwig: transactions are verified after they are committed 16:19:03 dougwig: There is a thread you should catch up on. 16:19:22 how did i miss this thread? ok, i'll go look. 16:19:29 dougwig: so once galera realizes it regrets it commitment, it throws a deadlock 16:19:58 * mestery points dougwig at this thread: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067059.html 16:19:58 dougwig: the thread is titled "quota enforcement" to throw you off the trail 16:20:00 #link http://lists.openstack.org/pipermail/openstack-dev/2015-June/067059.html 16:20:04 kevinbenton carl_baldwin: That's the one, right? 16:20:16 mestery: yeah 16:20:20 cool 16:20:26 HenryG: Thanks for beating me to getting the link 16:20:28 it led to that topic because the discussion of locks came up 16:20:33 er, mestery 16:20:43 ??? 16:20:50 ty 16:20:54 mestery: stop beating carl_baldwin 16:20:55 Thanks mestery for getting the link. 16:21:10 rofl 16:21:13 OK, lets close this thing down. 16:21:26 Thanks for all your hard work and guidance folks! 16:21:28 #endmeeting