18:31:43 <SumitNaiksatam> #startmeeting Networking FWaaS
18:31:44 <openstack> Meeting started Wed Jan 21 18:31:43 2015 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:31:45 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:31:47 <pc_m> hi
18:31:48 <openstack> The meeting name has been set to 'networking_fwaas'
18:31:53 <SumitNaiksatam> #info metting agenda https://wiki.openstack.org/wiki/Meetings/FWaaS#Agenda_for_Next_Meeting
18:32:04 <SumitNaiksatam> #info Kilo-2 is Feb 5th
18:32:34 <SumitNaiksatam> anything else anyone wants to share/announce upfront?
18:33:00 <SumitNaiksatam> #topic Bugs
18:33:23 <SumitNaiksatam> there was a critical bug (UTs breaking) but it was fixed and merged
18:34:02 <SumitNaiksatam> other than that i dont see any new critical/major bugs
18:34:16 <SridarK> SumitNaiksatam: yes nothing critical present
18:34:18 <SumitNaiksatam> SridarK: vishwanathj: anything show up on your radar?
18:34:23 <SumitNaiksatam> SridarK: okay
18:34:53 <SumitNaiksatam> badveli_: hi
18:34:59 <badveli_> hello sumit and all
18:35:04 <SridarK> badveli_: hi
18:35:08 <SumitNaiksatam> badveli_: anything show up on your bug radar?
18:35:11 <badveli_> hello sridark
18:35:17 <vishwanathj> jenkins test fails for our code because we have not uploaded the L3 router plugin code
18:36:17 <SridarK> vishwanathj: on ur review patch ? That is fine now
18:36:24 <vishwanathj> Ok
18:36:32 <SumitNaiksatam> vishwanathj: yeah, will come to the vendor patches in a min
18:37:42 <SumitNaiksatam> badveli_: any progress on #link https://bugs.launchpad.net/neutron/+bug/1386543 after I last posted the message?
18:38:15 <SumitNaiksatam> i dont see any update on #link https://bugs.launchpad.net/neutron/+bug/1335375
18:38:30 <SumitNaiksatam> badveli_: just checking in case you happened to spend time on this
18:39:42 <SumitNaiksatam> ok moving on from bugs
18:39:59 <SumitNaiksatam> #topic Docs
18:40:02 <badveli_> i did not see any update
18:40:24 <SumitNaiksatam> SridarK: anything on your radar on this front?
18:40:37 <SridarK> SumitNaiksatam: no updates on this
18:40:43 <SumitNaiksatam> SridarK: okay
18:40:52 <SumitNaiksatam> #topic Firewall Insertion
18:41:10 <SumitNaiksatam> SridarK: good discussion last week
18:41:16 <SumitNaiksatam> SridarK: any progress?
18:41:27 <SridarK> SumitNaiksatam: have done a first pass on the subtasks
18:41:50 <SridarK> SumitNaiksatam: will need to look at foreign keys from neutron db
18:41:56 <SridarK> (routers)
18:42:27 <SridarK> SumitNaiksatam: i think this should not be a problem - u have probab done this sort of thing for GBP
18:42:54 <SumitNaiksatam> SridarK: yeah, i can help you with the FK, etc
18:42:55 <SridarK> SumitNaiksatam: most of the work is on the Plugin side, from what i see - the agent should not be too bad
18:43:04 <SridarK> SumitNaiksatam: thanks for that
18:43:13 <pc_m> SridarK: Let me know when you have a handle on how long it'll take to get this in, so I can coordinate refactoring for L3.
18:43:15 <SumitNaiksatam> SridarK: feel free to post the WIP patch, we can take it from there
18:43:22 <SridarK> SumitNaiksatam: i am trying to see if i can get a WIP patch out next week
18:43:40 <SumitNaiksatam> SridarK: sooner might be better :-)
18:43:42 <SridarK> SumitNaiksatam: u took the keys out of my keyboard :-)
18:43:55 <SridarK> SumitNaiksatam: will definitely try for that
18:44:01 <SumitNaiksatam> SridarK: since kilo-2 is feb 5th
18:44:10 <SridarK> pc_m: yes will do so on L3 agent refactor
18:44:19 <SridarK> SumitNaiksatam: yes i agree
18:44:34 <SumitNaiksatam> pc_m: thanks for bringing that up, yes, we owe you an estimate on that
18:45:00 <SridarK> SumitNaiksatam: there are some details on how we handle state with multiple routers now that we will track the routers
18:45:01 <SumitNaiksatam> SridarK: so far any blocking issues?
18:45:12 <pc_m> With the WIP, we can probably start to look at what is needed for refactoring.
18:45:13 <SumitNaiksatam> SridarK: okay
18:45:19 <SumitNaiksatam> pc_m: agree
18:45:25 <SridarK> SumitNaiksatam: we can see if we need to do this now or it can be outside the scope of this work
18:45:42 <SumitNaiksatam> SridarK: lets set some time aside tomorrow and white board this
18:45:53 <SumitNaiksatam> SridarK: does that work for you?
18:45:54 <SridarK> pc_m: yes agreed - on the agent side it seems mostly it is removing code out
18:46:06 <SridarK> SumitNaiksatam: most definitely thanks for the time
18:46:18 <SridarK> SumitNaiksatam: will coordinate with u offline
18:46:29 <SumitNaiksatam> SridarK: sure, others feel free to join as well
18:46:56 <SridarK> SumitNaiksatam: ok that were the basic things i wanted to bring up
18:47:03 <SridarK> *those
18:47:05 <SumitNaiksatam> SridarK: so you havent hit any major blockers yet, right?
18:47:11 <SridarK> SumitNaiksatam: not yet
18:47:13 <vishwanathj> SumitNaiksatam, Please send the webex meeting if you plan to have one
18:47:20 <SumitNaiksatam> vishwanathj: sure
18:47:37 <SumitNaiksatam> SridarK: we will need some changes to the CLI for this, right?
18:47:45 <SumitNaiksatam> SridarK: if so i can add those
18:47:52 <SridarK> SumitNaiksatam: yes will need that ok greate
18:48:04 <SridarK> *great
18:48:37 <SumitNaiksatam> any other questions for SridarK on this?
18:48:48 <SridarK> SumitNaiksatam: ur help as always is most appreciated
18:49:11 <SumitNaiksatam> SridarK: thanks for working on this
18:49:23 <SridarK> SumitNaiksatam: no worries at all
18:49:36 <SumitNaiksatam> #topic FWaaS: L3 Agent restructure
18:49:39 <SumitNaiksatam> pc_m: hi
18:49:47 <SumitNaiksatam> we already touched on this a little bit today
18:49:47 <pc_m> howdy
18:50:01 <SumitNaiksatam> anything you want to add/bring up besides that?
18:50:09 <pc_m> Yeah, once we have a feel for when insertion will be done, we can look at the refactoring.
18:50:13 <SumitNaiksatam> any developments since we disucssed last?
18:50:46 <pc_m> no. just a holding pattern right now, which is OK, as the router object work is still ongoing.
18:51:33 <pc_m> For VPN, once the router object is ready, we'll do more refactoring. For FW, I don't know, may or may not have to do anything.
18:52:04 <SumitNaiksatam> pc_m: yes, hopefully we are able to determing that soon
18:52:57 <SumitNaiksatam> pc_m: thanks, keep us in the loop on anything else that we need to know on the l3-agent refactoring side (and that might potentially affect fwaas)
18:53:04 <SumitNaiksatam> #topic Service Groups
18:53:10 <pc_m> ok
18:53:25 <SumitNaiksatam> badveli_: any update?
18:54:44 <SumitNaiksatam> perhaps not
18:54:53 <badveli_> sumit i am thinking of two patches
18:55:01 <SumitNaiksatam> badveli_: okay
18:55:26 <badveli_> one as the service objects and the other one as the reference implementation
18:55:46 <badveli_> reference implementation as the firewall
18:55:55 <SumitNaiksatam> badveli_: okay, sounds reasonable to me
18:56:21 <SumitNaiksatam> badveli_: just keep in mind that people will probably start reviewing in earnest only after they see both patches
18:56:37 <SridarK> badveli_: for the first one will there be a backend ?
18:56:46 <badveli_> thanks sumit, keeping in mind about the reviews, i will go head and do this kind of split
18:56:59 <SumitNaiksatam> SridarK: i believe the second patch is the backend for the first patch
18:57:07 <badveli_> no there will be no backend kind of implementation
18:57:23 <SridarK> badveli_:, SumitNaiksatam: ok so one review split as 2 patches
18:57:27 <SridarK> ok got it
18:57:28 <badveli_> in the first patch, the second one is a reference implementation
18:58:37 <SumitNaiksatam> any questions for badveli_?
18:58:56 <badveli_> thanks for your help, i will go in this direction
18:59:04 <SumitNaiksatam> badveli_: thanks for the update
18:59:25 <SumitNaiksatam> oh one thing, it seems like these patches will get split across repos
18:59:32 <badveli_> thanks sumit, all . Let me try
18:59:57 <badveli_> yes one will be in the neutron
19:00:07 <SumitNaiksatam> so thats a bit of an unchartered territory for us
19:00:53 <SumitNaiksatam> pc_m: have you attempted any simultaneuos changes to the neutron repo and one of the services’ repos?
19:01:08 <SumitNaiksatam> pc_m: say for example when you were doing the VPN agent refactor?
19:01:13 <pc_m> no. Not sure if we can.
19:01:17 <SumitNaiksatam> pc_m: hmmm
19:01:41 <SumitNaiksatam> pc_m: i was going to ask if one can make a patch dependent across the repos
19:01:45 <pc_m> I created the new stuff in Neutron, committed, and then did change in VPN to use.
19:02:01 <pc_m> SumitNaiksatam: Not sure how to do that.
19:02:06 <SumitNaiksatam> because i think in badveli_’s case his second patch in the fwaas repo should be dependent on the first patch in the neutron repo
19:02:46 <pc_m> sounds like it will be pioneering in that area :)
19:02:48 <SumitNaiksatam> pc_m: in this case the first patch will not merge without people seeing the second patch, kind of a chicken and egg situation! :-)
19:03:00 <SumitNaiksatam> SridharRamaswamy: hi
19:03:14 <badveli_> Sumit we will have the issue if it is one patch or two patches
19:03:19 <pc_m> SumitNaiksatam: No way to decouple?
19:03:27 <SumitNaiksatam> badveli_: yes thats true
19:03:38 <SridarK> badveli_: i think u have to have 2 patches
19:03:59 <SumitNaiksatam> the bigger concern is that the second patch is probably not going to pass UTs until the first patch merges
19:04:11 <SumitNaiksatam> that is if we are not able to put the dependency
19:04:51 <SumitNaiksatam> badveli_: you are going to be blazing a trail here I guess! ;-)
19:05:22 <SumitNaiksatam> pc_m: i am not sure how we could completely decouple the second patch
19:05:33 <badveli_> but there could be a chance of atleast getting one patch i guess?
19:05:45 <SumitNaiksatam> pc_m: since the second patch implements the extensions
19:06:24 <pc_m> Does first patch define the extensions?
19:06:35 <SumitNaiksatam> pc_m: yes
19:06:57 <pc_m> Is there some way for a temporary stub implementation in Neutron?
19:06:57 <SumitNaiksatam> in fact even the DB is on the fwaas repo side
19:07:07 * pc_m just thinking out loud
19:07:09 <SridarK> So the extensions will need to go into neutron
19:07:21 <SumitNaiksatam> so only the extension definition will go into neutron
19:07:38 <SumitNaiksatam> SridarK: yeah
19:07:45 <SumitNaiksatam> pc_m: that is a possibility
19:08:04 <SridarK> SumitNaiksatam: so that will need to be a separate patch anyway
19:08:25 <SumitNaiksatam> pc_m: actually, for the first patch to work, no stub is required
19:08:43 <SridarK> SumitNaiksatam: we will land up with extension and api and no backend or a stub in neutron
19:09:04 <SumitNaiksatam> if we can figure out how the second patch can make use of the extension definition while the first patch is still being reviewed, we are good
19:09:14 <SumitNaiksatam> SridarK: correct
19:09:47 <SridarK> SumitNaiksatam: ok this is definitely trailblazing - hope we don't get burned
19:10:03 <SumitNaiksatam> SridarK: :-)
19:10:16 <SumitNaiksatam> SridarK: what about your patch, do you anticipate a similar issue?
19:10:37 <SumitNaiksatam> SridarK: since you need to update the fwaas extension definition as well
19:10:45 <badveli_> sumit, sridar, please let me know if we need a discussion, we can take it offline also
19:10:56 <SridarK> SumitNaiksatam: yes exactly -
19:11:21 <SumitNaiksatam> :-)
19:11:44 <SridarK> SumitNaiksatam: extensions will need to go into neutron but nothing much more - will need to use routers resource from neutron (but that is not an issue)
19:12:00 <SridarK> SumitNaiksatam: not as bad as service objects case
19:12:27 <SumitNaiksatam> SridarK: perhaps
19:12:30 <SridarK> SumitNaiksatam: also on the topic of extensions - will have some impact from the pecan framework changes
19:12:45 <SumitNaiksatam> SridarK: yeah whenever it lands
19:12:58 <SridarK> SumitNaiksatam: ok
19:14:02 <SumitNaiksatam> SridarK: it will be good if you can identify all the changes you need to make in the extension, DB, and plugin, that way we can work through this surgically
19:14:09 <SumitNaiksatam> SridarK: we need to do this at the earliest
19:14:14 <SridarK> SumitNaiksatam: ok will do so
19:14:19 <badveli_> Sumit, Sridar thanks for your updates. I will sync up offline and think about this
19:14:37 <SumitNaiksatam> SridarK: the agent is on the fwaas repo side, so dont foresee as many issues there
19:14:42 <SridarK> SumitNaiksatam: yes this needs more thought for sure
19:14:58 <SridarK> SumitNaiksatam: yes the agent is quite straightforward not much in terms of changes
19:15:26 <SumitNaiksatam> this is the kind of blocker i was asking about, and was hoping that we had not run into! :-(
19:15:52 <SumitNaiksatam> anyway, perhaps also a good idea to send to the -dev mailer and check if someone else has thought about this already
19:16:09 <SridarK> SumitNaiksatam: yes need to get used to thinking in terms of multiple repos - glad we had this conversation today - had not really thought about that
19:16:27 <pc_m> +1
19:16:37 <SumitNaiksatam> #topic Vendor Plugins/Drivers
19:16:45 <SridarK> SumitNaiksatam: so although the extensions are defined in neutron - hopefully the migration will stay in fwaas repo
19:16:46 <SumitNaiksatam> vishwanathj: your turn, now
19:17:01 <SumitNaiksatam> SridarK: the DB is on the neutron-fwaas side
19:17:15 <SumitNaiksatam> SridarK: so yes, migration is not in neutron
19:17:21 <vishwanathj> I have uploaded the code and can be viewed at https://review.openstack.org/#/c/147600/
19:17:28 <SridarK> SumitNaiksatam: yes
19:17:39 <SumitNaiksatam> vishwanathj: thanks, sorry havent had a chance to look
19:17:55 <SridarK> vishwanathj: same here - need some time to take a look
19:18:02 <vishwanathj> ok
19:18:16 <SumitNaiksatam> badveli_: perhaps you can also help with reviewing vishwanathj’s patch
19:19:11 <SridarK> SumitNaiksatam: we will also upstream a vendor patch once our vendor repo is set up for the dependency (L3)
19:19:16 <badveli_> fine sumit
19:19:28 <SumitNaiksatam> vishwanathj: do you have any blockers at this point (apart from getting reviewer attention)?
19:19:38 <SumitNaiksatam> SridarK: yes, i was coming to your patch
19:19:47 <SridarK> oops sorry :-(
19:19:55 <SumitNaiksatam> SridarK: np, :-)
19:20:21 <vishwanathj> SumitNaiksatam, nothing that i can think of, the meeting with you, SridarK, pc_m last week was beneficial, a big thanks to you folks
19:20:29 <SumitNaiksatam> vishwanathj: the way i understand your patch is based on the current framework and implementation, right?
19:20:37 <vishwanathj> yes
19:20:56 <SumitNaiksatam> vishwanathj: no worries, we appreciate your sincere effort, and for getting conversation started
19:21:25 <SumitNaiksatam> vishwanathj: as i mentioned we need at least one more core reviewer to help review
19:21:41 <vishwanathj> ok
19:21:53 <SumitNaiksatam> so lets try to identify who can help with that
19:22:20 <SumitNaiksatam> SridarK: sorry, go ahead
19:22:36 <vishwanathj> SumitNaiksatam, any suggestions on who I can approach
19:22:53 <SumitNaiksatam> SridarK: so you wont be submitting the patch to neutron-fwaas?
19:23:11 <SridarK> SumitNaiksatam: no worries - just wanted to add that we will have 2 patches(plugin, agent) pushed up once we have our vendor repo setup for L3
19:23:23 <SridarK> SumitNaiksatam: it will be to neutron-fwaas
19:23:40 <SumitNaiksatam> vishwanathj: i believe all the neutron-drivers are cores on fwaas, and then there are a couple of more
19:23:42 <SridarK> SumitNaiksatam: we will need to pull out our Router implementation from neutron
19:23:54 <SumitNaiksatam> vishwanathj: i will add them to your review, you can proceed from there, sound okay?
19:23:59 <vishwanathj> ok
19:24:17 <SumitNaiksatam> SridarK: ah, so the external repo dependency for the l3 plugin being pulled out?
19:24:23 <SridarK> SumitNaiksatam: exactly
19:24:30 <SumitNaiksatam> SridarK: got it
19:24:43 <SridarK> SumitNaiksatam: i think we have figured out the mechanics for this and should happen this week
19:24:46 <SumitNaiksatam> SridarK: but you can still post the patch in neutron-fwaas, right?
19:24:55 <SumitNaiksatam> WIP, i mean
19:25:02 <SridarK> SumitNaiksatam: yes we can - did not want to risk a -2
19:25:14 <SumitNaiksatam> okay
19:26:08 <SridarK> SumitNaiksatam: ok - can do that - our code is done - wanted to put a note on the vendor repo dependency (point to a wiki or something) and get our stuff out so folks realize that there is a plan in place for vendor repo
19:26:36 <SridarK> SumitNaiksatam: we may be good by this week or worst case will get it out by early next week with a WIP
19:26:49 <SumitNaiksatam> SridarK: yes sure, i think WIP should insulate against the possibility of an overzealous -2 ;-)
19:26:58 <SridarK> SumitNaiksatam: ok :-)
19:27:06 <SumitNaiksatam> ok we are hitting the hour mark
19:27:10 <SumitNaiksatam> #topic Open Discussion
19:27:22 <SumitNaiksatam> and i was thinking we would have had a shorter meeting today!
19:27:30 <SumitNaiksatam> anything else folks?
19:27:33 <SridarK> SumitNaiksatam: :-)
19:27:49 <SumitNaiksatam> vishwanathj: i have added the reviewers
19:27:56 <vishwanathj> SumitNaiksatam, Thanks
19:28:16 <SumitNaiksatam> alrighty, lets call it a wrap for this meeting
19:28:21 <SridarK> Bye all
19:28:22 <SumitNaiksatam> thanks all for joining!
19:28:24 <SumitNaiksatam> bye
19:28:27 <pc_m> bye
19:28:35 <SumitNaiksatam> #endmeeting