17:01:56 #startmeeting Networking Advanced Services 17:01:56 yo 17:01:56 Meeting started Tue Dec 2 17:01:56 2014 UTC and is due to finish in 60 minutes. The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot. 17:01:57 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:02:00 The meeting name has been set to 'networking_advanced_services' 17:02:04 pc_m: SridarK glebo: hi 17:02:11 o/ 17:02:11 #info https://wiki.openstack.org/wiki/Meetings/AdvancedServices#Agenda 17:02:46 copy pasting from the neutron IRC for the benefit who missed it - 17:02:47 #info SPD is 12-8-2014 and SAD is 12-15-2014 17:02:53 #info Kilo-1 is 12-18-2014 17:03:24 Could you expand those acronyms please? 17:03:29 any other info/announcements people would like to share? (apart from the services’ split discussion, of course) 17:03:40 Spec Proposal Deadline 17:03:43 Spec approval deadline, and the first is the submission deadline (not sure on the P) 17:03:46 Spec Approval Deadline 17:03:46 Thanks! 17:04:17 it'll be a SAD day 17:04:28 because you spec doesn't make it? 17:04:33 badum-tish! 17:04:40 not knowing what that is, probably no 17:04:48 blogan: exactly - seemed like SAD had become common neutron lingo 17:04:58 sorry; wrong window 17:05:53 hello everyone! 17:05:58 okay next topic - our raison d'être 17:06:22 #topic Advanced services’ spin out 17:06:26 SumitNaiksatam: seems Paris has left its impression on u :-) 17:06:43 SridarK: oh good catch! 17:06:47 Services' split: #link https://review.openstack.org/#/c/136835 17:07:21 and there is a somewhat related spec which factors into the extensions’ discussions: #link https://review.openstack.org/#/c/136760 17:07:35 to summarize for those who missed the meeting in the morning 17:07:40 please note the imminent changes that are coming, based on today's neutron meeting: 17:07:42 https://www.irccloud.com/pastebin/5qRvFm1X 17:08:12 * glebo yeah, thinks 06:00 is PAINFUL, but was commuting through inches of rain at the time. 17:08:14 dougwig: thanks, i was just about to summarize that, seems like you had it prepared ;-) 17:08:27 glebo: yeah, commute was pretty painful 17:08:35 that was my gerrit comment when I did -A just a few ago. :) 17:09:11 dougwig: you mean workflow-a 17:09:12 neat 17:09:17 *workflow-1 17:09:43 Hi 17:10:01 dougwig: those read like executive decisions ;-P 17:10:23 i realized after hitting submit that i should've put, "from the neutron meeting..." :) 17:10:33 SumitNaiksatam: That's because they were discussed and agreed to in the neutron meeting today 17:10:38 dougwig: ++ 17:10:52 mestery: yes sure, just pulling dougwig’s leg here 17:10:58 lol 17:11:14 with both the split and flavors, it's a good thing that i'm secretly a spider. 17:11:31 Wait, are we now getting to accuse dougwig of being a merciless tyrant? 17:11:38 mestery: glad you could make it to this meeting as well 17:11:50 * mestery is omniscient 17:12:16 sbalukoff: that we thought was already established, no? ;-) 17:12:40 mestery: lol 17:12:43 As long as I get to carry around a pitchfork and torch, I'm happy. 17:13:02 okay, so any thoughts feedback on those points, or anything else in the latest spec that we need discuss today/here? 17:13:03 Aaanyway. 17:13:05 sbalukoff likes to be the only merciless tyrant in town 17:13:08 sbalukoff: wait, we already passed halloween, no? 17:13:20 xgerman: +1 17:13:48 I feel like the current spec is pretty close to what we need to get progress to happen. 17:14:00 +1 17:14:01 sbalukoff: okay 17:14:20 As such, if you've got objections, please make specific comments to that effect soon! 17:14:37 and if not +1 17:14:41 the work needs to start next week at the latest, so we need to get things solid. 17:14:43 xgerman: +1 17:14:59 yep, originally split was aimed for k1 17:15:07 and we can still make it 17:15:10 i don't mind some rework later, but if you've got something major, please don't wait. 17:15:28 I'm new here then, How to propose a spec here? 17:15:34 And make time *now* to review if you haven't done so and feel like you have a horse in this race. 17:15:46 trinaths1: hi, same as neutron spec 17:15:50 trinaths1: our specs are neutron specs 17:16:12 there is a wiki somewhere explaining the process 17:16:41 is pc_m here? 17:16:46 yes sir 17:16:47 okay. How will they be pointed to Advanced services? 17:17:14 trinaths1: thats not needed, its all one neutron track 17:17:21 okay. 17:17:37 SumitNaiksatam: understood. 17:17:39 #topic Services’ impact by L3 agent refactoring 17:17:58 this would impact fwaas and vpnaas 17:18:10 pc_m: you want to summarize where we are at with this? 17:18:15 sure 17:18:35 Carl has a BP out for the effort (I think 131535) 17:19:08 I did a commit for review to start the observer pattern to allow agents to register with L3 agent to get notifications for events. 17:19:24 136549 17:19:57 Assaf has refined it some, and will do a updated patch that merges his with mine. Will have it tomorrow or Thur I suspect. 17:20:13 Currently, we're trying to work out the notifications. 17:20:14 #link https://review.openstack.org/#/c/131535/ (for the spec) 17:20:43 Seems like there may be some variation as to where notifications will occur. 17:21:02 Will need help from FW folks to see about where hooks are needed for that agent. 17:21:06 #link https://review.openstack.org/#/c/136549/ (for pc_m’s patch) 17:21:31 pc_m: i volunteer 17:21:43 I think Assaf's is #link https://review.openstack.org/#/c/137668 17:21:51 SridarK: thanks! 17:22:22 We can hook up offline. Looking at router add/delete/update and where agents need to be notified, for agent specific actions. 17:22:27 pc_m: i can also have Gary Duan from our team take a look 17:22:29 pc_m: there are few places that notifications are needed esp on router stuff 17:22:37 pc_m: yes exactly 17:22:51 glebo: super! The more the merrier. 17:23:21 I think we want to get Assaf's and my commits merged, and then it is a matter of working out the notification points and refine the notification method. 17:23:24 pc_m: so this does not propose to move away from the l3 agent inheriting from the fwaas agent? 17:23:37 pc_m: Is is the plan to still go for multiple agents (per service) or service specfic classes somehow loaded by a single agent? 17:24:00 SumitNaiksatam: I think it will split them up. We're trying to tackle it piecemeal. 17:24:08 pc_m: ah ok 17:24:18 hareeshp: yeah, my earlier question 17:24:26 SumitNaiksatam: yes ;) 17:24:46 I posted on ML (a few weeks back) some steps Carl and I discussed, and I think Carl has it in his BP. 17:24:54 SumitNaiksatam: It is proposed to move away from the inheritence for FW. Very soon. 17:25:02 carl_baldwin: okay 17:25:17 pc_m: so your patch is the first one to be reviewed in this chain, right? 17:25:57 SumitNaiksatam: Sort of... Assaf's is dependent on mine. There are a few others as well. 17:26:14 Carl has one to split up the L3 agent into modules. 17:26:24 ah there is a topic for this: 17:26:26 #link https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:bp/restructure-l3-agent,n,z 17:26:48 As soon as we identify how FW needs to be notified and what callbacks are needed on the l3_agent and router then we should be able to break the inheritence relationship. 17:26:52 Take a look at the BP first of all, so that we can get closure on carl_baldwin's BP. 17:27:07 at the end of this discussion, i just want to make sure that my naive assumption that this work exists in parallel to the split is actually true, and we don't have to block on each other. 17:27:14 carl_baldwin: pc_m: sounds good 17:27:38 carl_baldwin: currently it is mostly on the router_interface_add 17:27:48 dougwig: good question, and concern 17:28:05 so we should be able to do this with some minimal restructure of the code 17:28:16 (FW agent code) 17:28:21 dougwig: i would tend to think that this refactoring of the agent has to be accomplished prior to the fwaas and vpnaas split 17:28:54 so that the respective agents can move with the services when the split happens 17:28:57 what is the timeline for that? alternately, i'm ok with a temporary circular dependency with the l3 agent 17:29:16 temporary == never shipped. 17:29:31 dougwig: yeah i agree, if there is a workaround, we should consider that 17:29:33 also since we are now splitting into 3 repos do we eed to split all at once? 17:29:43 dougwig: We're hot on it now... not sure how long to tease it all apart. 17:29:47 dougwig: This is currently our highest priority. The bp just merged yesterday and so we should working on it in earnest. 17:30:31 we have this, split, rest refactor, and vendor split, all aimed straight at each other. anything we can do to sever dependencies will make the impending management nightmare easier. 17:30:44 +1 17:30:57 dougwig: +1 17:31:00 +1 17:31:19 +1 17:31:24 +1 17:31:28 agreed, so it might help to get some of these things in first, so that we dont have too many loose ends after the split 17:31:28 +1 17:31:38 +1 17:31:49 SumitNaiksatam: that is the opposite of severing dependencies. :) 17:31:59 (confused) what does a split mean? 17:32:06 * pc_m +1 to dougwig's comment 17:32:08 dougwig: not all dependencies are equal 17:32:17 trinaths1: https://review.openstack.org/#/c/136835/ 17:32:43 So REST refactoring starts next week at the mid-cycle? 17:32:56 anytime you say "before" or "after", it is a dependency, and adds risk, IMO. 17:33:04 mestery: if you are still around, is there a timeline we are targeting for the services’ repo split? 17:33:34 dougwig: thanks 17:34:02 it's next week, unless the rest refactor makes it impossible. 17:34:04 SumitNaiksatam at the summit we said K1 17:34:09 dougwig: Did you have a plan for moving FW out of tree without breaking the inheritence hierarchy? 17:34:13 I would still aim for that 17:34:16 * carl_baldwin has not completely read the bp. Sorry. 17:34:23 that would help to evaluate if its feasible at all to address any percieved dependenices for the split 17:34:25 carl_baldwin: yes, but you'll need a bucket handy. 17:34:43 :) 17:35:30 carl_baldwin: the short-form would be treating the classes/repos as if they're not separate for the very short-term. 17:35:56 dougwig: okay. thanks for the clarity. 17:36:02 then the fw team can refactor to independence on their own timeline. 17:36:05 * carl_baldwin wonders where he left that bucket. ;) 17:36:13 i did warn you. 17:36:49 SumitNaiksatam: The work will be done next week in SLC 17:37:03 So, initially, the two repos are co-dependent? 17:37:08 so the idea would be that neutron would depend on adv services’ repo, and adv services’ repo would depend on neutron? 17:37:21 carl_baldwin: yeah, you summarized it better :-) 17:37:23 carl_baldwin: until the l3 refactor is done, yes. that's the only circular link that i know of. 17:37:44 dougwig: I think that is fine for the (very) short term but I’ll be looking again for the bucket if Kilo ships like that. 17:37:55 i don't think we can allow kilo to ship like that. 17:38:06 i guess we would need the co-dependency would be needed regardless of the L3 agent dependency issue 17:38:33 * i guess the co-dependency would be needed regardless of the L3 agent dependency issue 17:38:48 only for the upgrade scenario. the plan is to have a hacking rule to prevent any other circular usages. 17:38:53 with an exception for l3 for now 17:39:22 dougwig: okay 17:40:02 so if the spin out is planned for the next week, i am guessing that is probably too short a time frame to achieve the l3 agent refactoring 17:40:16 dougwig: okay. Does it mean more complexity in the build / test environment to support this co-dependent thing? 17:40:42 SumitNaiksatam: yup 17:40:43 no. the circular link is already there for the upgrade scenarios. it just means abusing it slightly for fun and profit. 17:40:51 the only extra work is the exception to the new hacking rule 17:42:20 dougwig: okay. I’m okay going forward with that. We can rebase our work to react to it. 17:42:21 perhaps we can set the services’ agents (fwaas/vpnaas) as a target for K2, that way we are more or less guaranteed to get it done in Kilo 17:43:12 if it lands earlier, great, but we can proceed with the split regardless 17:43:13 i'll include this refactor and breaking this link in the split timeline. for K2 as SumitNaiksatam suggests? 17:43:39 dougwig: i think that will be good to call out 17:43:45 +1 17:43:50 will do. 17:44:03 dougwig: at least it identifies what we need to follow up on after the split as an immediate requirement 17:44:04 look for a spec with a detailed roadmap by lunchtime. 17:44:09 SumitNaiksatam: I’d like to target an initial break out of the inheritence hierarchy for a couple weeks from now. I’m not sure how much work will be needed to get to where you would like the service agents to be. 17:44:10 dougwig: thanks 17:44:42 carl_baldwin: agreed, i think, like SridarK mentioned, on the fwaas side we dont anticipate it to be as much 17:44:44 I’m focused on detangling the L3 agent first. Service agents second. 17:44:55 perhaps pc_m can talk to for the vpn side of things 17:44:56 SumitNaiksatam: dougwig: +1 17:45:11 SumitNaiksatam: carl_baldwin: yes should not be too bad on FW 17:45:34 I have probab jinxed it by saying that. :-) 17:45:37 okay we have 15 mins left, and we haven’t ye reached “flavors” :-) 17:45:43 SumitNaiksatam: pc_m: i can help out if any vpn work spills over 17:45:43 I don't think there is too much on VPN. only a handful of notifications. 17:45:46 Heh! 17:45:58 SridharRamaswamy: pc_m: okay thanks 17:46:04 #topic Flavors 17:46:11 #link https://review.openstack.org/#/c/102723 17:46:23 the spec has been repurposed from Juno ^^^ 17:46:29 It's back from the grave! 17:46:38 and it wants your brains 17:46:39 * sbalukoff prepares his pitchfork and torch again 17:46:46 err, eyes. 17:46:48 i meant eyes. 17:46:53 Yes, eyes. 17:47:00 dougwig: thanks for resurrecting it in time for the SPD 17:47:07 Mmmm, the flavor... 17:47:32 it's roughly similar to mark's spec from juno. did anyone have major issues with that one? 17:47:43 and by roughly, i mean almost entirely identical. 17:47:45 I loved it@@ 17:47:50 not to rat hole into names, but i believe there were some suggestions to drop the use of “flavor” terminology in general in OpenStack 17:48:24 do we have a good alternative? 17:48:27 if those still persist, perhpas they will show up early in the review 17:48:47 dougwig: not sure, i was just trying to look up the threads that i recall reading a week or so back 17:48:56 If we want this approved before the SAD date, those reviews / objections better come quick! 17:49:04 sbalukoff: agree 17:49:17 sbalukoff: +100 17:49:20 dougwig: will look up and post if i can dig it out 17:49:26 SumitNaiksatam: thanks 17:49:54 anything more of “flavors”? 17:50:05 not sure if people had a chance to read it since it was resurrected 17:50:22 Well, they should take the opportunity now. :) 17:50:24 * pc_m unfortunately not yet 17:50:42 ok moving on 17:51:07 #topic Open Discussion (and any FWaaS/LBaaS/VPNaaS updates) 17:51:24 for lbaas updates, scroll up one hour to the meeting minutes. :) 17:51:30 dougwig: :-) 17:51:31 Haha 17:51:35 :) 17:51:42 quick admin question: do we have a separate mailing list for advanced services? 17:51:55 sunil_: not really 17:52:03 SumitNaiksatam: When can I propose the FW Spec for K? 17:52:04 sunil_: in fact there isnt one for neutron either 17:52:16 SumitNaiksatam: FWaaS putting together spec for FW insertion on router - should have it out by tomorrow 17:52:19 trinaths1: you can and should before SPD (dec 8th) 17:52:20 my understanding is the BPs on the flight need to be reproposed against advanced services spec repo ? Is this correct ? 17:52:38 SridharRamaswamy: there isnt a separate repo 17:52:40 there is only one spec repo 17:52:55 okay, just to clarify for everyone’s benefit - 17:53:00 SridharRamaswamy: no, only neutron-specs, and only re-propose if it was from juno and needs to be for kilo 17:53:11 if you need propose a spec, please propose a spec in Neutron Specs, nothing has changed 17:53:12 SumitNaiksatam: Okay, Here is my spec https://review.openstack.org/#/c/126187/ 17:53:13 re: FW insertion… 17:53:21 how are we doing on the use cases? 17:53:30 I've lost track since I went on TG break 17:53:40 SumitNaiksatam: please consider it for K. 17:53:42 if you need to send an email regarding advanced services, you send the email to the -dev ML, and prefix with: 17:54:15 SumitNaiksatam: okay, I've a vpn spec already in neutron-spec/kilo #link https://review.openstack.org/#/c/136693/ 17:54:16 [Neutron][advanced-services] or [Neutron][LBaaS] or [Neutron][FWaaS] or [Neutron][VPNaaS] 17:54:26 trinaths1: SridharRamaswamy: great 17:54:45 SridarK: I'll keep my eyes open for the insertion on router spec 17:54:47 SumitNaiksatam: :) reviews bless the specs.. 17:54:48 on the fwaas side, SridarK will be posting a new spec for the insertion 17:54:54 glebo: ^^^ 17:54:57 * reviewers 17:54:57 dougwig: this is a new spec for Kilo 17:55:01 glebo: we sill need the use cases :-) 17:55:30 glebo: yes have sent a reminder to FW folks on that 17:55:31 pc_m: SridharRamaswamy: anything you would like to add for VPNaaS that is relevant to the rest of the adv services’ team here? 17:55:44 nothing from me. 17:56:05 nothing from me, waiting for all the refactors to fly in :) 17:56:10 * pc_m still trying to get unburied from vacation and 4 days of no internet 17:56:18 altighty, if nothing else, we can call it a wrap and take back 4 mins! 17:56:26 who hoo 17:56:29 thanks all for joining! 17:56:30 bye 17:56:33 bye all 17:56:35 bye 17:56:37 bye 17:56:38 bye all 17:56:41 bye 17:56:41 #endmeeting