21:02:16 <markmcclain> #startmeeting Networking
21:02:17 <openstack> Meeting started Mon Jul  8 21:02:16 2013 UTC.  The chair is markmcclain. Information about MeetBot at http://wiki.debian.org/MeetBot.
21:02:18 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
21:02:20 <openstack> The meeting name has been set to 'networking'
21:02:33 <markmcclain> #link https://wiki.openstack.org/wiki/Network/Meetings
21:02:47 <markmcclain> #topic Announcements
21:03:32 <markmcclain> I think we mostly survived the rename in one piece.  Thanks to the infra team transitioning the repositories and CI infrastructure on Saturday.
21:03:43 <garyk> hi
21:03:47 <gongysh> hi
21:03:48 <nati_ueno> hi!
21:04:01 <nati_ueno> markmcclain: sorry late
21:04:05 <enikanorov> hi
21:04:16 <emagana> hello all
21:04:47 <markmcclain> If you haven't already make sure to update your git configs to point at the new repo locations
21:05:22 <arosen1> hi
21:05:25 <markmcclain> #link https://launchpad.net/neutron/+milestone/havana-2
21:06:21 <markmcclain> H2 is right around the corner.  All blueprints for H2 need to be proposed by Thursday, so that we we have time to review them before the branch is cut.
21:07:05 <markmcclain> We've only merged 25% of the targeted blueprints, so I expect the next week to busy
21:07:54 <markmcclain> If you have code out for review, you will need to rebase it to pick up the name changes otherwise it will fail on merging
21:08:11 <markmcclain> #topic renaming
21:08:40 <markmcclain> We removed most references to quantum with the code itself over the weekend.
21:09:23 <emagana> markmcclain: terrific job!!!
21:09:25 <markmcclain> The code passes unit tests, but we are unable to validate the vendor plugins.  I've reached out to the plugin contacts to ask they test their plugins and report back.
21:09:32 <Swami> hi all
21:10:28 <markmcclain> emagana: thanks
21:10:51 <emagana> I want to ask if someone is experiencing this devstack problem after renaming: https://bugs.launchpad.net/devstack/+bug/1199126
21:10:52 <uvirtbot> Launchpad bug 1199126 in devstack "starting neutron agent fails" [Undecided,New]
21:11:15 <HenryG> I ran a quick check on the Cisco plugin on devstack and didn't notice any problems.
21:11:31 <markmcclain> HenryG: good to know
21:12:01 <markmcclain> emagana: looks like a devstack bug
21:12:04 <gongysh> when we publish a new neutron client
21:12:07 <gongysh> ?
21:12:48 <markmcclain> gongysh: the NeutronClient (2.2.4) that was published last week was equivalent to quantumclient 2.2.3
21:13:05 <gongysh> ok
21:13:14 <markmcclain> I'd like to get the rest of the integrated projects switched
21:13:18 <emagana> script is not able to find this function: neutron_plugin_configure_plugin_agent
21:13:29 <markmcclain> and then we can push 2.2.5 with newer features
21:14:32 <markmcclain> Any questions about renaming?
21:14:48 <gongysh> when we update doc?
21:15:05 <gongysh> hope they were done too.
21:15:08 <markmcclain> gongysh: emagana started a bit of that last week
21:15:15 <emagana> gongysh: I am working on them!
21:15:22 <gongysh> ok
21:15:34 <nati_ueno> submitter should do (1) Rebase the patch to the neutron (2) replace s/Quantum/Neutron/ s/quantum/neutron/ in the newer added part ?
21:15:57 <nati_ueno> emagana: oh the rumor get truth :)
21:16:12 <salv-orlando> nati_ueno: that's one part of it. In general look for conflict and solve them just like with any other patch
21:16:22 <emagana> nati_ueno: ;-)
21:16:35 <nati_ueno> salv-orlando: I got it. IMO, we should have wiki for this instructions
21:16:39 <markmcclain> nati_ueno: text substitution works for most cases, but there a few plugins were quantum_* is actually exchanged with the backend
21:16:51 <mestery> nati_ueno: It's really quite easy, any errors will be caught in flake8 or UT.
21:17:05 <SumitNaiksatam> markmcclain: thanks for fixing in the plugins as well
21:17:08 <nati_ueno> mestery: I see! good point!
21:17:28 <salv-orlando> yep, the only issues I had so far are with imports. But it's easy to pick out which ones you added/changed/removed with git diff
21:17:28 <SumitNaiksatam> make sure you use "git mv" when you rename files/directories
21:17:42 <SumitNaiksatam> else it will show up as new files in existing patches
21:18:00 <markmcclain> SumitNaiksatam: most of that should have been taken care of by the big rename this weekend
21:18:05 <nati_ueno> ok I would like to write wiki page for this. Or Should I add this information for existing page?
21:18:19 <markmcclain> nati_ueno: you can add instructions on the existing
21:18:23 <mestery> I vote for adding to the existing page.
21:18:33 <nati_ueno> markmcclain: I got it
21:18:47 <SumitNaiksatam> markmcclain: i was referring to patches in review which introduce new files
21:19:51 <markmcclain> right.. I've found that rebasing generally does the right thing
21:20:11 <amotoki> SumitNaiksatam: reagrding patches, rebase will address renaming issues.
21:20:39 <SumitNaiksatam> at least, it did not happen in my case
21:20:56 <amotoki> when you add new files, "git mv" will help you.
21:21:01 <mestery> SumitNaiksatam: I had to manually use "git mv" to move files in one of my patchsets which was new in my patchset.
21:21:52 <markmcclain> Last thing on renaming.. if you find a bug make sure to file it in launchpad so that we can track them and prioritize them
21:21:56 <SumitNaiksatam> amotoki, mestery: yeah, you need to do a "git mv" for new files (i noticed lots of new files in patches that were in review, probably not doing a git mv)
21:21:57 <markmcclain> #topic bugs
21:22:25 <markmcclain> We've this bug still to close: https://bugs.launchpad.net/neutron/+bug/1184484
21:22:28 <uvirtbot> Launchpad bug 1184484 in tripleo "Quantum default settings will cause deadlocks due to overflow of sqlalchemy_pool" [Critical,Triaged]
21:22:43 <markmcclain> lifeless: is the bug you wanted to discuss?
21:22:45 <garyk> markmcclain - does the oslo db merge take care of that?
21:22:46 <salv-orlando> I am working on it. Expecting to have a fix or an headache tomorrow
21:22:55 <nati_ueno> updated https://wiki.openstack.org/wiki/Network/neutron-renaming#How_to_rename_existing_patches
21:22:59 <lifeless> markmcclain: it is one of two.
21:23:04 <markmcclain> salv-orlando: thanks for looking into it
21:23:11 <markmcclain> lifeless: what is the other one?
21:23:12 <salv-orlando> garyk: I am trying with the new modelbase
21:23:15 <lifeless> markmcclain: bug 1189385
21:23:16 <uvirtbot> Launchpad bug 1189385 in tripleo "quantum-server hung up it's listening port" [Critical,Triaged] https://launchpad.net/bugs/1189385
21:23:24 <lifeless> markmcclain: we've supplied all the info asked for
21:23:30 <lifeless> markmcclain: (AFAICT)
21:23:31 <garyk> salv-orlando ok thanks
21:23:31 <salv-orlando> hopefully that will be solution otherwise, we might have a bad problem with sqlalchemy leaking sessions
21:24:02 <salv-orlando> for bug 1189385, all that I can see is that is seems unrelated to but 1184484
21:24:40 <markmcclain> lifeless: thanks for adding the tracebacks… I'll dig into 1189385
21:24:46 <garyk> markmcclain we also should discuss the sporadic failures of tempest
21:24:54 <garyk> https://bugs.launchpad.net/nova/+bug/1186161
21:24:55 <uvirtbot> Launchpad bug 1186161 in nova "Tempest tests sometimes get: [Fail] Couldn't ping server" [Undecided,New]
21:25:00 <markmcclain> garyk:  yes can do that here
21:25:12 <markmcclain> this check fails in the gate randomly
21:25:22 <markmcclain> and running the gate again it is likely to pass
21:25:32 <lifeless> markmcclain: thank you!
21:25:57 <markmcclain> garyk: I know you were looking into 1186161 have found anything of interest?
21:27:16 <garyk> markmcclain - i ahve yet been able to reproduce this when i run tempest
21:27:42 <markmcclain> ok
21:27:51 <garyk> markmcclain - i will continue to look into it
21:28:02 <markmcclain> thanks
21:28:15 <markmcclain> Any other high priority bugs to discuss?
21:28:46 <garyk> markmcclain - https://bugs.launchpad.net/nova/+bug/1192287
21:28:47 <uvirtbot> Launchpad bug 1192287 in nova "Creating server did not fail when exceeded Quantum quota limit" [Undecided,New]
21:29:12 <garyk> i think that this is more nova related than quantum. the quotas check is done on the compute node instead of at the api level
21:29:28 <markmcclain> right
21:29:35 <markmcclain> arosen1: we're you working on this too?
21:30:06 <garyk> markmcclain, arosen1: i can look into this if you guys want
21:30:18 <armax> I think this is similar bug 1172808
21:30:19 <uvirtbot> Launchpad bug 1172808 in nova "Nova fails on Quantum port quota too late" [Wishlist,Triaged] https://launchpad.net/bugs/1172808
21:30:25 <arosen1> yea ^
21:30:33 <salv-orlando> I think this goes back to the 'fail fast' discussions we had early in the release cycle
21:30:35 <arosen1> we need to move the port creation to nova-api to fix that.
21:30:39 <salv-orlando> year that thing
21:31:19 <garyk> any objections if i take it?
21:31:22 <salv-orlando> arosen1 has a blueprint for it; but community had mixed feelings about it.
21:31:51 <arosen1> https://blueprints.launchpad.net/nova/+spec/nova-api-quantum-create-port
21:33:15 <markmcclain> cool.. we probably link the bugs and blueprint to prevent more duplicate entires
21:33:16 <arosen1> garyk: it's a big change. I was planing on bitting this off next but if you want it, it's all yours :)
21:33:52 <garyk> arosen1: i'll take a look at the bp and let you know
21:34:00 <arosen1> garyk:  k
21:34:20 <markmcclain> Ok let's run through the sub teams real quick
21:34:23 <markmcclain> #topic API
21:34:27 <salv-orlando> hi
21:34:34 <markmcclain> salv-orlando: look like we're in good shape for H2
21:34:54 <salv-orlando> very quickly: we are risking to miss the H-2 deadline for service type refactoring
21:35:04 <salv-orlando> because of new concerns around design
21:35:17 <salv-orlando> enikanorov is going to push this to the mailing list soon.
21:35:26 <salv-orlando> please note that we already have patches dependent on it
21:36:00 <salv-orlando> and that with 'service types' providers or whatever we want to call them there's a very thin line between a productive discussion and speculation
21:36:24 <salv-orlando> so let's get that to what matters for our goals, and ensure its merged by H-2 deadline!
21:36:27 <salv-orlando> that's all
21:37:17 <nati_ueno> enikanorov said he will support full db based implementation in hanava, so I'm going to +2 for the patch after checking it again
21:37:47 <markmcclain> ok.. looks like we have enough cores on it
21:38:01 <markmcclain> Anything else besides this blocker?
21:38:02 <salv-orlando> nati_ueno I am not sure about you mean with 'full db based', but I think we need to give room to next topic
21:38:07 <salv-orlando> nope, that is all
21:38:08 <nati_ueno> so salv-orlando and akihiro looks already +1 or +2 for that patch, I'm expecting that patch will land soon
21:38:18 <markmcclain> #topic VPN
21:38:27 <markmcclain> nati_ueno: hi
21:38:48 <nati_ueno> Regarding to service type discussion, we decided to remove vpn_type
21:39:02 <nati_ueno> We will add support for service type framework in H3
21:39:16 <enikanorov> ok
21:39:26 <nati_ueno> One news is how to test page have horizon ui instruction also
21:39:45 <nati_ueno> https://wiki.openstack.org/wiki/Quantum/VPNaaS/HowToInstall
21:40:03 <markmcclain> great.. minus pep8 errors.. like folks should review VPN
21:40:17 <nati_ueno> Yes since h2 end is close,
21:40:41 <nati_ueno> so it is great if we could have cores on REST API and Driver patch
21:41:04 <nati_ueno> I'm reviewing REST API so one additional for REST API and 2 core for driver patch
21:41:22 <markmcclain> I'll be one of the cores on this
21:41:33 <nati_ueno> markmcclain: Thanks!
21:41:43 <salv-orlando> markmcclain: if you cannot take both patches, I can do the api one
21:42:09 <markmcclain> salv-orlando: cool.. I'll let you have one and I'll review the driver
21:42:10 <nati_ueno> salv-orlando: Thanks!
21:42:27 <nati_ueno> I need one more for drivers
21:42:44 <amotoki> i will look the driver one.
21:42:49 <nati_ueno> amotoki: Thanks!
21:42:51 <markmcclain> amotoki: thanks
21:43:05 <markmcclain> Anything else on VPN?
21:43:19 <nati_ueno> markmcclain: salv-orlando: amotoki: OK We are renaming code. After we check it working with new name, we will ping you guys
21:43:29 <nati_ueno> markmcclain: That's all from us
21:43:29 <salv-orlando> k
21:43:39 <markmcclain> nati_ueno: sounds good
21:43:40 <nati_ueno> Thank you for your help
21:43:42 <markmcclain> #topic firewalling
21:44:09 <markmcclain> SumitNaiksatam: hi
21:44:12 <SumitNaiksatam> hi
21:44:17 <SumitNaiksatam> API, Agent and Driver patches have been rebased with the name change. CLI patch still needs to be rebased.
21:44:29 <SumitNaiksatam> API and Driver patches are out of WIP. Unit tests are in the works on the agent patch, hence it's still WIP.
21:44:39 <markmcclain> thanks for getting them rebased so quickly
21:44:40 <SumitNaiksatam> RajeshMohan, who is the author of the driver patch, is traveling to India today, so expect some delays on review responses at his end.
21:44:53 <markmcclain> ok
21:44:55 <SumitNaiksatam> I need to add the db migration script to the API patch.
21:45:07 <SumitNaiksatam> SridarK do you have anything more to add on the agent work?
21:45:28 <SridarK> Nothing more - will get UT done by Wed
21:45:46 <SumitNaiksatam> in general, there have been cores on the API patch (thanks Salvatore, amotoki, arisen, others)
21:45:59 <SumitNaiksatam> we need more on the driver and agent patch (when its ready)
21:46:10 <SumitNaiksatam> arisen -> arosen
21:46:11 <nati_ueno> I'll review them :)
21:46:21 <SumitNaiksatam> thanks nati_ueno !! :-)
21:46:38 <markmcclain> great
21:46:41 <SumitNaiksatam> thats our update
21:46:47 <nati_ueno> SumitNaiksatam: Thank you for your sharing test instruction. It is great to if we could have wiki page for that
21:46:53 <markmcclain> thanks for updating
21:46:58 <markmcclain> #topic ml2
21:47:02 <SumitNaiksatam> nati_ueno: its on the wiki page
21:47:07 <markmcclain> mestery: hi
21:47:11 <nati_ueno> SumitNaiksatam: Ah sorry. Thanks
21:47:12 <mestery> markmcclain: hi
21:47:28 <mestery> ML2 changes are progressing nicely, though it's unclear if we'll get much more merged before H2 ends.
21:47:39 <mestery> I think the GRE and VXLAN type drivers have a good chance if we can get some core reviewers.
21:47:46 <mestery> And the MechanismDriver change as well.
21:48:04 <markmcclain> I took a look at the mech driver this afternoon
21:48:05 <mestery> Also, I'm hoping to get gongysh to review this one again https://review.openstack.org/#/c/33107/ (I added a link to a design document there).
21:48:23 <gongysh> sorry for the delay to look at it.
21:48:27 <gongysh> I will do it today
21:48:32 <mestery> Other than that, we have a sub-team meeting Wednesday to discuss in detail ML2.
21:48:38 <markmcclain> gongysh: will you be able to look an unlblock?
21:48:39 <mestery> gongysh: Thanks! I plan to work on that tomorrow.
21:48:47 <markmcclain> gongysh: thanks
21:48:56 <mestery> That's it for ML2.
21:49:16 <markmcclain> so this https://blueprints.launchpad.net/neutron/+spec/ml2-portbinding
21:49:19 <markmcclain> will miss H2?
21:49:37 <mestery> I think so, I haven't seen any review for it yet.
21:49:45 <mestery> rkukura was going to try to get a review out early this week though.
21:49:54 <markmcclain> ok.. I'll hold it in for now
21:50:01 <mestery> Thanks!
21:50:05 <markmcclain> mestery: thanks for updating
21:50:12 <markmcclain> #topic lbaas
21:50:24 <enikanorov> hi
21:50:40 <enikanorov> the agent scheduling patch isstill waiting for cores
21:51:04 <enikanorov> nothing has changed from past week except now the patch depends on another one
21:51:14 <enikanorov> which divides agent scheduling to dhcp & l3
21:51:25 <enikanorov> (and lbaas, in lbaas-related patch)
21:51:47 <markmcclain> gongysh: can you take a look at this one: https://review.openstack.org/#/c/35266/
21:51:50 <enikanorov> so those two reviews require core's attention
21:52:16 <nati_ueno> I'll review it also
21:52:26 <gongysh> sure
21:52:32 <markmcclain> nati_ueno, gongysh: thanks
21:52:39 <markmcclain> enikanorov: anything else?
21:52:42 <enikanorov> that's all from lbaas for h2
21:52:57 <markmcclain> enikanorov: thanks for updating
21:53:08 <markmcclain> #topic nova
21:53:19 <garyk> we have spoken about bugs above
21:53:20 <markmcclain> garyk: I think we covered most of the pressing items earlier
21:53:44 <markmcclain> we had same thought :)
21:53:47 <markmcclain> #topic stable
21:53:50 <garyk> markmcclain agreed. there is one issue though that it would be nice if quantum guys could look at (1 sec)
21:53:56 <gongysh> https://review.openstack.org/#/c/34968/
21:53:59 <gongysh> nova side
21:54:11 <garyk> https://review.openstack.org/#/c/33054/
21:54:57 <garyk> gongysh - i think that this was reverte in nova as they felt it was the cause f the tempest problems
21:55:23 <gongysh> no, not this one.
21:55:23 <markmcclain> monkeypatching eventlet causes some problems deep down with httplib2
21:55:42 <gongysh> it is the one with shared quantum client.
21:55:56 <garyk> ok
21:56:25 <markmcclain> I'll look at the token issue because we've similar stuff internally
21:56:40 <markmcclain> #topic stable
21:56:49 <garyk> no updates here
21:57:01 <markmcclain> Now that we've renamed the code the backports are going to get much harder
21:57:12 <garyk> a few people have been dong backports (sorry i do not recall the names - but thanks!)
21:57:24 <markmcclain> we're also farther into the dev cycle, so might need to raise the bar for what we allow as a backport
21:58:01 <garyk> hopefully neutron will have fewer problems than quantum
21:58:10 <markmcclain> :)
21:58:18 <mestery> garyk: Heh :)
21:58:50 <markmcclain> #topic Horizon
21:58:57 <amotoki> hi
21:58:58 <markmcclain> amotoki: anything to share?
21:59:13 <amotoki> markmcclain: is renaming patch from you ready for reivew?
21:59:18 <markmcclain> yes
21:59:32 <amotoki> thank. will check it.
21:59:40 <markmcclain> great!
21:59:55 <amotoki> for H2, security group support is the only one related to neutron.
22:00:25 <markmcclain> cool.. so most of the services will land in H3?
22:00:35 <amotoki> i think so.
22:00:42 <amotoki> vpnass has a good progress.
22:01:13 <amotoki> i have no concerns for fwaas too.
22:01:17 <markmcclain> good to hear
22:01:30 <SumitNaiksatam> KC is working on the CLI, but should be on FWaaS shortly
22:01:50 <amotoki> SumitNaiksatam: good to hear
22:01:59 <amotoki> all from me.
22:02:08 <markmcclain> Any questions on Horizon<>Neutron integration?
22:02:29 <markmcclain> #topic Open Discussion
22:02:54 <gongysh> I will begin the old work on multihost feature this week.
22:03:03 <garyk> i am going to crash. good night. have a good one
22:03:09 <markmcclain> garyk: night
22:03:11 <amotoki> garyk: night!
22:03:13 <nati_ueno> garyk: night!
22:03:13 <gongysh> since the host-id is in.
22:03:17 <markmcclain> gongysh: this will be h3?
22:03:22 <gongysh> garky: dream.
22:03:26 <gongysh> yes
22:04:31 <markmcclain> Anything other items to cover?
22:04:36 <gongysh> and about concurrent lock
22:04:42 <gongysh> this is a big problem
22:05:53 <gongysh> overlapping cidr is just a case of them.
22:05:58 <marun> gongysh: that's a db issue
22:06:08 <marun> gongysh: a concurrent lock is not required
22:06:14 <gongysh> no, that is a business logic problem.
22:06:29 <marun> gongysh: yes, one that needs to be enforced at the db level
22:06:31 <gongysh> not all problem can be fixed by a constraint.
22:06:32 <marun> gongysh: not in code
22:07:07 <marun> gongysh: this isn't a hard problem.  it's been done by db's since the 1970s
22:07:35 <gongysh> haha,
22:08:07 <marun> gongysh: if you are not aware of this, I respectfully suggest that you are not the best person to address the problem.  There is a reason I assigned the bugs to markmcclain.
22:08:12 <gongysh> ok, what is your opinion of the overlapping cidr checking.
22:09:30 <nati_ueno> Which bug are you discussing?
22:09:46 <markmcclain> We're out of time.. if gongysh, marun want to hop into #openstack-neutron we can talk about this more
22:10:01 <gongysh> ok
22:10:19 <markmcclain> and those that are interested can join us there too
22:10:59 <gongysh> nati_ueno: Bug #1194206
22:11:02 <uvirtbot> Launchpad bug 1194206 in neutron "allow_overlapping_ips=False fails for Concurrent requests" [High,In progress] https://launchpad.net/bugs/1194206
22:11:04 <nati_ueno> gongysh: thanks
22:11:04 <markmcclain> Thanks for everyone who stepped up to review important patches for H2
22:11:09 <markmcclain> #endmeeting