Thursday, 2014-07-31

*** ivar-lazzaro has joined #openstack-meeting-300:01
*** MarkAtwood has quit IRC00:05
*** ivar-lazzaro has quit IRC00:05
*** mfer has joined #openstack-meeting-300:17
*** eghobo has quit IRC00:20
*** nkubota_ has quit IRC00:25
*** mfer has quit IRC00:26
*** briancurtin has joined #openstack-meeting-300:27
*** rudrarugge has quit IRC00:28
*** xuhanp has joined #openstack-meeting-300:30
*** yamahata has joined #openstack-meeting-300:37
*** hurgleburgler has quit IRC00:38
*** yamamoto has quit IRC00:47
*** banix has joined #openstack-meeting-300:50
*** jamielennox has quit IRC00:54
*** Sukhdev has quit IRC00:55
*** jamielennox_ has joined #openstack-meeting-300:55
*** jamielennox_ is now known as jamielennox00:56
*** xuhanp has quit IRC00:59
*** chuckC has quit IRC01:13
*** nkubota has joined #openstack-meeting-301:17
*** yamahata__ has joined #openstack-meeting-301:34
*** stanzgy has joined #openstack-meeting-301:37
*** baohua has joined #openstack-meeting-301:41
*** carl_baldwin has joined #openstack-meeting-301:54
*** s3wong has quit IRC01:55
*** SumitNaiksatam has quit IRC01:57
*** amotoki has joined #openstack-meeting-301:57
*** sankarshan_away is now known as sankarshan01:58
*** sankarshan is now known as sankarshan_away01:58
*** beagles has quit IRC02:00
*** amotoki has quit IRC02:02
*** beagles has joined #openstack-meeting-302:05
*** SumitNaiksatam has joined #openstack-meeting-302:13
*** carl_baldwin has quit IRC02:13
*** chuckC has joined #openstack-meeting-302:15
*** tomoe_ has joined #openstack-meeting-302:19
*** armax has quit IRC02:22
*** beagles has quit IRC02:25
*** armax has joined #openstack-meeting-302:30
*** beagles has joined #openstack-meeting-302:30
*** mfer has joined #openstack-meeting-302:33
*** armax has quit IRC02:33
*** mfer has quit IRC02:43
*** jamielennox is now known as jamielennox|away02:45
*** jamielennox|away is now known as jamielennox02:48
*** carl_baldwin has joined #openstack-meeting-302:48
*** trinaths has quit IRC03:02
*** trinaths has joined #openstack-meeting-303:03
*** trinaths has quit IRC03:04
*** badveli has quit IRC03:06
*** nelsnelson has quit IRC03:08
*** nkubota has quit IRC03:20
*** nkubota_ has joined #openstack-meeting-303:21
*** yamamoto_ has joined #openstack-meeting-303:28
*** yamamot__ has joined #openstack-meeting-303:29
*** yamamoto_ has quit IRC03:33
*** yamahata has quit IRC03:33
*** yamahata__ has quit IRC03:34
*** carl_baldwin has quit IRC03:34
*** briancurtin has quit IRC03:34
*** yamahata has joined #openstack-meeting-303:35
*** carl_baldwin has joined #openstack-meeting-303:38
*** jamielennox has left #openstack-meeting-303:42
*** carl_baldwin has quit IRC03:44
*** boris-42 has quit IRC03:57
*** amotoki has joined #openstack-meeting-303:58
*** carl_baldwin has joined #openstack-meeting-303:58
*** amotoki has quit IRC04:02
*** HenryG is now known as HenryG_afk04:06
*** boris-42 has joined #openstack-meeting-304:07
*** briancurtin has joined #openstack-meeting-304:23
*** briancurtin has quit IRC04:27
*** banix has quit IRC04:29
*** sankarshan_away is now known as sankarshan04:51
*** sankarshan is now known as sankarshan_away04:51
*** amotoki has joined #openstack-meeting-304:51
*** yamamot__ has quit IRC05:01
*** amotoki has quit IRC05:06
*** MarkAtwood has joined #openstack-meeting-305:08
*** briancurtin has joined #openstack-meeting-305:17
*** yamamoto_ has joined #openstack-meeting-305:18
*** briancurtin has quit IRC05:21
*** carl_baldwin has quit IRC05:22
*** tomoe_ has quit IRC05:47
*** tomoe_ has joined #openstack-meeting-305:49
*** tmazur has joined #openstack-meeting-305:49
*** k4n0 has joined #openstack-meeting-305:51
*** jcoufal has joined #openstack-meeting-306:02
*** briancurtin has joined #openstack-meeting-306:07
*** briancurtin has quit IRC06:11
*** Longgeek has joined #openstack-meeting-306:18
*** Longgeek has quit IRC06:20
*** Longgeek has joined #openstack-meeting-306:22
*** mrunge has joined #openstack-meeting-306:31
*** MaxV has joined #openstack-meeting-306:45
*** iovadia has joined #openstack-meeting-306:46
*** tomoe_ has quit IRC06:47
*** briancurtin has joined #openstack-meeting-307:01
*** tomoe_ has joined #openstack-meeting-307:05
*** briancurtin has quit IRC07:05
*** MaxV has quit IRC07:09
*** MaxV has joined #openstack-meeting-307:09
*** MaxV has quit IRC07:14
*** iovadia1 has joined #openstack-meeting-307:27
*** iovadia2 has joined #openstack-meeting-307:28
*** iovadia has quit IRC07:28
*** iovadia1 has quit IRC07:31
*** pkoniszewski has joined #openstack-meeting-307:32
*** safchain has joined #openstack-meeting-307:34
*** MaxV has joined #openstack-meeting-307:48
*** yamahata has quit IRC07:51
*** MaxV has quit IRC07:53
*** briancurtin has joined #openstack-meeting-307:55
*** briancurtin has quit IRC07:59
*** Longgeek has quit IRC08:04
*** Longgeek has joined #openstack-meeting-308:05
*** tomoe__ has joined #openstack-meeting-308:08
*** MaxV has joined #openstack-meeting-308:10
*** tomoe_ has quit IRC08:12
*** nacim has joined #openstack-meeting-308:13
*** tomoe_ has joined #openstack-meeting-308:16
*** tomoe__ has quit IRC08:17
*** tomoe__ has joined #openstack-meeting-308:17
*** tomoe_ has quit IRC08:20
*** iovadia2 has quit IRC08:22
*** yamamoto_ has quit IRC08:36
*** iovadia has joined #openstack-meeting-308:46
*** MaxV has quit IRC08:48
*** scheuran has joined #openstack-meeting-308:49
*** MaxV has joined #openstack-meeting-308:49
*** briancurtin has joined #openstack-meeting-308:49
*** MaxV has quit IRC08:53
*** yamamoto has joined #openstack-meeting-308:53
*** briancurtin has quit IRC08:54
*** MaxV has joined #openstack-meeting-308:59
*** edleafe has quit IRC09:12
*** edleafe has joined #openstack-meeting-309:12
*** tmazur has quit IRC09:16
*** tomoe__ has quit IRC09:30
*** baohua has quit IRC09:30
*** tomoe_ has joined #openstack-meeting-309:31
*** tmazur has joined #openstack-meeting-309:38
*** briancurtin has joined #openstack-meeting-309:43
*** briancurtin has quit IRC09:48
*** MaxV has quit IRC10:31
*** MaxV has joined #openstack-meeting-310:32
*** MaxV has quit IRC10:37
*** briancurtin has joined #openstack-meeting-310:37
*** jcoufal_ has joined #openstack-meeting-310:37
*** jcoufal has quit IRC10:40
*** briancurtin has quit IRC10:42
*** nkubota_ is now known as zz_nkubota_10:42
*** banix has joined #openstack-meeting-310:44
*** stanzgy has quit IRC10:49
*** beagles has quit IRC10:57
*** beagles has joined #openstack-meeting-311:03
*** beagles is now known as Guest9538111:03
*** MaxV has joined #openstack-meeting-311:05
*** MaxV has quit IRC11:05
*** MaxV has joined #openstack-meeting-311:05
*** Guest95381 has quit IRC11:17
*** beagles_ has joined #openstack-meeting-311:21
*** tmazur has quit IRC11:22
*** beagles_ has quit IRC11:26
*** Longgeek_ has joined #openstack-meeting-311:28
*** beagles_ has joined #openstack-meeting-311:31
*** Longgeek has quit IRC11:31
*** tmazur has joined #openstack-meeting-311:35
*** flaviof_zzz is now known as flaviof11:36
*** banix has quit IRC11:42
*** beagles_ has quit IRC11:57
*** zz_nkubota_ is now known as nkubota11:58
*** Guest28074 has joined #openstack-meeting-312:02
*** alexpilotti has joined #openstack-meeting-312:10
*** baohua has joined #openstack-meeting-312:10
*** k4n0 has quit IRC12:14
*** briancurtin has joined #openstack-meeting-312:26
*** briancurtin has quit IRC12:30
*** Guest28074 is now known as beagles12:38
*** beagles is now known as Guest7845012:38
*** Guest78450 is now known as b3nt_pin12:44
*** b3nt_pin is now known as Guest4791312:44
*** flaviof is now known as flaviof_zzz12:46
*** HenryG_afk is now known as HenryG12:49
*** julim has joined #openstack-meeting-312:59
*** mwagner_lap has joined #openstack-meeting-312:59
*** julim has quit IRC13:00
*** nelsnelson has joined #openstack-meeting-313:00
*** HenryG has quit IRC13:02
*** nkubota is now known as zz_nkubota13:04
*** julim has joined #openstack-meeting-313:04
*** chuckC_ has joined #openstack-meeting-313:12
*** yamahata__ has joined #openstack-meeting-313:14
*** yamamoto_ has joined #openstack-meeting-313:14
*** yamahata_ has quit IRC13:15
*** sdague has quit IRC13:15
*** chuckC has quit IRC13:15
*** chuckC_ is now known as chuckC13:15
*** mwagner_lap has quit IRC13:15
*** yamamoto has quit IRC13:15
*** lsmola has quit IRC13:15
*** anteaya has quit IRC13:15
*** tmazur has quit IRC13:15
*** mwagner_lap has joined #openstack-meeting-313:15
*** tmazur has joined #openstack-meeting-313:15
*** sdague has joined #openstack-meeting-313:16
*** anteaya has joined #openstack-meeting-313:16
*** lsmola has joined #openstack-meeting-313:18
*** peristeri has joined #openstack-meeting-313:18
*** briancurtin has joined #openstack-meeting-313:20
*** briancurtin has quit IRC13:25
*** banix has joined #openstack-meeting-313:30
*** banix has quit IRC13:31
*** banix has joined #openstack-meeting-313:32
*** banix has quit IRC13:34
*** nelsnelson has quit IRC13:36
*** nelsnelson has joined #openstack-meeting-313:37
*** banix has joined #openstack-meeting-313:38
*** mfer has joined #openstack-meeting-313:42
*** nelsnelson has quit IRC13:42
*** Guest47913 has quit IRC13:47
*** briancurtin has joined #openstack-meeting-313:49
*** briancurtin has quit IRC13:49
*** needscoffee has joined #openstack-meeting-313:50
*** needscoffee has joined #openstack-meeting-313:50
*** briancurtin has joined #openstack-meeting-313:50
*** briancurtin has quit IRC13:50
*** briancurtin has joined #openstack-meeting-313:51
*** beagles has joined #openstack-meeting-313:53
*** TravT has joined #openstack-meeting-313:56
*** briancurtin has quit IRC13:57
*** briancurtin has joined #openstack-meeting-313:58
*** mfer has quit IRC13:59
*** briancurtin has quit IRC14:01
*** briancurtin2 has joined #openstack-meeting-314:01
*** thangp has joined #openstack-meeting-314:01
*** needscoffee has quit IRC14:04
*** briancurtin2 has quit IRC14:06
*** briancurtin has joined #openstack-meeting-314:06
*** briancurtin has joined #openstack-meeting-314:06
*** briancurtin has quit IRC14:08
*** otherwiseguy has joined #openstack-meeting-314:10
*** nelsnelson has joined #openstack-meeting-314:16
*** armax has joined #openstack-meeting-314:16
*** briancurtin has joined #openstack-meeting-314:22
*** briancurtin has joined #openstack-meeting-314:22
*** briancurtin has quit IRC14:23
*** yamahata__ has quit IRC14:31
*** mfer has joined #openstack-meeting-314:37
*** Longgeek has joined #openstack-meeting-314:38
*** Longgeek_ has quit IRC14:41
*** jcoufal_ has quit IRC14:41
*** mfer has quit IRC14:41
*** HenryG has joined #openstack-meeting-314:43
*** yyywu has joined #openstack-meeting-314:46
*** briancurtin has joined #openstack-meeting-314:46
*** briancurtin has joined #openstack-meeting-314:46
*** briancurtin has quit IRC14:48
*** mfer has joined #openstack-meeting-314:48
*** briancurtin has joined #openstack-meeting-314:49
*** briancurtin has quit IRC14:49
*** briancurtin has joined #openstack-meeting-314:49
*** Youcef has joined #openstack-meeting-314:50
*** carl_baldwin has joined #openstack-meeting-314:50
*** devvesa_ has joined #openstack-meeting-314:51
*** briancurtin has quit IRC14:54
*** tomoe_ has joined #openstack-meeting-314:55
*** amuller has joined #openstack-meeting-314:55
*** baohua has quit IRC14:55
*** viveknarasimhan has joined #openstack-meeting-314:58
carl_baldwinhi all14:59
amullerHey Carl14:59
armaxcarl_baldwin: hi14:59
*** seizadi has joined #openstack-meeting-314:59
yamamoto_hi14:59
devvesa_hi14:59
carl_baldwinSwami, mrsmith, viveknarasimhan, safchain, pcm, nextone92:  ping15:00
*** xuhanp has joined #openstack-meeting-315:00
*** mrsmith has joined #openstack-meeting-315:00
carl_baldwin#startmeeting neutron_l315:00
openstackMeeting started Thu Jul 31 15:00:23 2014 UTC and is due to finish in 60 minutes.  The chair is carl_baldwin. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: neutron_l3)"15:00
openstackThe meeting name has been set to 'neutron_l3'15:00
carl_baldwin#topic Announcements15:00
*** openstack changes topic to "Announcements (Meeting topic: neutron_l3)"15:00
kevinbentono/15:00
carl_baldwin#link https://wiki.openstack.org/wiki/Meetings/Neutron-L3-Subteam15:00
*** briancurtin has joined #openstack-meeting-315:00
*** briancurtin has joined #openstack-meeting-315:00
mrsmithhowdy carl_baldwin15:00
carl_baldwinAlready a week in to Juno-3.  Things are moving fast.15:01
safchainhi15:01
*** Swami has joined #openstack-meeting-315:01
carl_baldwinjuno-3 is targetted for September 4th.15:01
carl_baldwin#link https://wiki.openstack.org/wiki/Juno_Release_Schedule15:01
*** briancurtin has quit IRC15:01
carl_baldwinAlso, the initial DVR implementation has been merged.  This should enable broader testing.15:02
carl_baldwinThe infra patches to enable experimental job have also merged I think.15:02
armaxcarl_baldwin: the first experimental job is running as we speak15:02
armaxhttp://status.openstack.org/zuul/15:03
Swamigreat.15:03
carl_baldwinarmax: Great.  I was just going to look this morning.15:03
armaxhttps://jenkins06.openstack.org/job/check-tempest-dsvm-neutron-dvr/1/console15:03
*** briancurtin has joined #openstack-meeting-315:03
armaxfor review 108177,1015:03
carl_baldwinDoes this require an explicit “check experimental” to run?15:04
armaxyes15:04
armaxI did post comment ‘check experimental’ explictily15:05
*** TravT has quit IRC15:05
carl_baldwin#topic neutron-ovs-dvr15:05
*** openstack changes topic to "neutron-ovs-dvr (Meeting topic: neutron_l3)"15:05
carl_baldwinall:  Make use of this job on our DVR related patches.15:06
* carl_baldwin goes to run ‘check experimental’ on his reviews15:06
carl_baldwinSwami: Anything to report?15:07
Swamicarl_baldwin: hi15:07
Swamiwe had a couple of issues that we wanted to discuss15:07
SwamiThis is related to migration15:07
*** briancurtin has quit IRC15:08
carl_baldwinYou have the floor.15:08
SwamiThe first question that we have is for a "router-migration", can we make use of "admin_state_up/admin_state_down" first before issuing a router-update15:08
carl_baldwinDo you mean to require that a router be in admin_state_down before migration?15:09
armaxSwami: I think it’s sensible15:09
*** pballand has joined #openstack-meeting-315:09
SwamiThe reason is, when admin issues these commands, the existing state of the routers are cleaned up and then we can move or migrate the routers to the new agent.15:09
Swamicarl_baldwin: Yes15:10
mrsmithit comes down to the admin running 3 commands or one15:10
carl_baldwinSwami: I don’t see a problem with that.  It should be documented.15:10
armaxadmin_state_down/up are there for a reason15:10
*** briancurtin has joined #openstack-meeting-315:10
*** briancurtin has joined #openstack-meeting-315:10
SwamiWe were initially debating that will the admin be ok, in issuing two commands for a migration. First would be to set the 'admin_state" and next would be to do an update.15:10
*** briancurtin has quit IRC15:11
viveknarasimhancould we do this internally15:11
armaxwe could flip the state automatically in the migration process, but I’d vote to be more explicit15:11
viveknarasimhanwithout the admin explicitly running 3 commands.15:11
viveknarasimhandoes he need to know 3 commmands to be executed in a certain order?15:11
yyywui vote for explicit, this is one time thing, right?15:11
armaxthe workflow usually goes like this: you warn your tenant of a maintenance15:11
armaxyou bring the router down15:12
armaxyou migrate15:12
carl_baldwinCan the admin run them one after the other with no delay?15:12
armaxyou bring it up (and hope that everything works)15:12
armaxthen go back to tenant and tell him that everything is okay :)15:12
Swamiviveknarasimhan: I agree with you and that is the reason we wanted concensus from all of us before proceeding.15:12
carl_baldwinOr, does the admin need to wait on something before being allow to run the migration?15:12
*** banix has quit IRC15:12
Swamicarl_baldwin: we need to check it out.15:13
*** tmazur has quit IRC15:13
SwamiSo if we all agree with armax: this is how it has to be done.15:13
carl_baldwinI ask because it could increase router downtime.  But, it is a one time migration and can be planned downtime.15:13
viveknarasimhanif some command in teh 3 fails15:13
viveknarasimhanis there a way to rollback15:13
mrsmithI vote for explicit - it is more straight forward15:14
viveknarasimhanor he need to recreate the centralized router again?15:14
SwamiWe will document that "admin" need to first bring down the router, migrate the router and then tell the tenant to use it.15:14
armaxcarl_baldwin: i imagine that it’s better being explicit15:14
Swamiany questions or concerns there.15:14
armaxwe might want to give us some room before the router going down and the migration15:15
*** mfer has quit IRC15:15
armaxif we do everythin in one shot there’s a risk something gets scheduled on that router in between15:15
yyywuarmax, agreed15:15
Swamiarmax: agreed15:15
armaxscheduled as in something happens  to that router15:16
carl_baldwinI don’t think I’m concerned.15:16
armaxunlikely, but you never know15:16
viveknarasimhanif a problem happens, it could happen in 3 step process as well15:16
yamamoto_explicit sounds less surprising in POV of admins15:16
yyywuand rollback is part of migration failure case, right?15:16
Swamiyyywu: when do you think that rollback should happen15:17
yyywuSawmi: I am thinking if migration failure happened, rollback should kick in.15:17
Swamiright now we are not targeting 'rollback" but we can flag a "migration-error" if something odd happens.15:17
armaxrollback can’t really happen if we don’t implement the distributed->centralized path15:18
yyywuSwami: i think we can live as it.15:18
armaxa recovery procedure would be to destroy and recreate the router (with all the interfaces and gateway associated with it)15:18
Swamiarmax: you are right.15:18
carl_baldwinAnything else on migration / admin_state?15:19
*** briancurtin has joined #openstack-meeting-315:19
Swamicarl_baldwin: admin_state is done.15:19
*** briancurtin has quit IRC15:19
*** briancurtin has joined #openstack-meeting-315:19
SwamiNext question is on the VM migration.15:19
*** briancurtin has quit IRC15:19
*** briancurtin has joined #openstack-meeting-315:20
SwamiHow will vm migration be handled during router conversion.15:20
SwamiThere are two cases.15:20
SwamiOne admin would like to use the same compute node, so they will not disturb the VM, but restart the l3-agent with DVR mode enabled.15:21
SwamiThe other case is where, the admin wants to move all their VM to a greenfiled deployment for DVR enabled Nodes. So they bring up new Compute nodes with DVR enabled L3-agents. In this case the VM migration is out of scope for the dvr team.15:22
carl_baldwinThe first is the only scenario I had in mind.15:22
mrsmithon the first l3-agent would need to be updated as well as ovs15:23
carl_baldwinIs this live migration in the second case?15:23
*** nacim has quit IRC15:23
Swamicarl_baldwin: Ok, if we only target the first scenario, then we will go through the use cases.15:24
yyywuone question, during router conversion, could nova initiate vm migration?15:24
Swamicarl_baldwin: yes it is a kind of live migration.15:24
*** briancurtin has quit IRC15:24
Swamiyyywu: nova has no idea of router conversion, I don't nova will be aware about the router changes.15:25
carl_baldwinI think we should consider the second case out of scope.15:25
armaxSwami: prior to doing the migration every compute host needs to run l2 (with dvr enalbed) and l3 agent15:25
armaxcorrect?15:25
viveknarasimhancorrect armax15:25
Swamiarmax: agreed.15:26
SwamiSo the admin issues a "router_admin_state_down".15:26
armaxdoes it make sense to keep the compute host disabled well during the migrtion?15:26
SwamiThe the admin prepares the compute node for migration.15:26
SwamiAnd the admin updates the router for migration.15:27
armaxmy understanding was that duing a planned upgrade the admin would deploy the right services with the right configs15:27
SwamiThis is for the "case 1" where we use the existing compute nodes.15:27
armaxon the elements of the cloud15:27
*** pballand has quit IRC15:27
armaxbut router migration should probably be a step right after the upgrade is complete15:28
armaxnot 100% true in every case15:28
armaxespecially if the default router type is ‘centralized'15:29
Swamicarl_baldwin: armax: viveknarasimhan: mrsmith: are we all in an agreement15:30
*** scheuran has quit IRC15:30
mrsmithSwami: on VM migration?15:30
mrsmithfocus on case 1 ?15:30
carl_baldwinI think so.  I’m not keen on adding the second case to our scope.15:30
viveknarasimhani agree. we will try to get case 1 fully covered15:31
carl_baldwinMaybe in kilo if there is demand.15:31
viveknarasimhancase 2 looks bit complex15:31
armaxmakes sense15:31
carl_baldwinSwami: anything else?15:31
armaxeven though moving a vm to a new host15:31
Swamimrsmith: carl_baldwin's reply should have answered your question for VM migration. We should reduce our scope to the Case 1: that we discussed.15:31
armaxdoes look like pretty much like a scheduling event15:31
armaxso dvr should handle it just as well15:31
*** briancurtin has joined #openstack-meeting-315:32
Swamicarl_baldwin: that's all from me.15:32
*** briancurtin has quit IRC15:32
carl_baldwinSwami: thanks.  Let’s get pounding more on the DVR code and fixing bugs.  We’ve already got some fixes done and a few more on the way.  Great job!15:32
*** briancurtin has joined #openstack-meeting-315:32
yamamoto_i have a small dvr question15:33
carl_baldwinyamamoto_: go ahead.15:33
Swamicarl_baldwin: np15:33
yamamoto_see https://review.openstack.org/#/c/110188/15:33
*** aleksandr_null has joined #openstack-meeting-315:33
*** pkoniszewski has quit IRC15:33
yamamoto_it's about ofagent but ovs-agent looks same15:33
yamamoto_isn't it a problem for unbind_port_from_dvr?15:33
carl_baldwinyamamoto_: this will take a a bit to look into.  Do you mind if we take the question to the neutron room?15:34
yamamoto_np.  i just wanted dvr folks know.15:35
carl_baldwinNow is a good time to grab them in the neutron room.15:35
carl_baldwin#topic l3-high-availability15:35
*** openstack changes topic to "l3-high-availability (Meeting topic: neutron_l3)"15:35
carl_baldwinsafchain: armax:  Any update here?15:35
safchainhi15:35
armaxcarl_baldwin: going through the review bits15:35
armaxarmax: I need to allocate more time to this though15:36
safchainI addressed comments and reworked base classes15:36
aleksandr_nullHi guys, sorry I'm late, tried to find correct meeting room :)15:36
amullerWorking on l3 agent functional testing: https://review.openstack.org/#/c/109860/15:36
carl_baldwinI said I’d review last week and did not.  But, now with the bulk of DVR merged, I have some review cycles.15:36
amullerSomething basic for starters15:36
safchaincurrently rebasing the scheduler part15:37
amullerThe l3 agent patch itself: https://review.openstack.org/#/c/70700/ - Adds HA routers to the functional tests15:37
*** pballand has joined #openstack-meeting-315:37
amullerOnce that's working I'll be able to respond to reviewer comments and refactor the HA code in the l3 agent so that it isn't as obtrusive15:37
carl_baldwinamuller: do you have a timeline for getting that working?15:38
amullerthe base patch that adds the functional tests is working15:39
amullerthe ha additions in the l3 ha agent patch aren't... I figure I need 2-3 days working on that and I'll start pushing new patchsets that change the code itself and not the functional tests15:39
carl_baldwinamuller: Thanks.  I need to catch up on the progress.  I’ll review today.15:40
carl_baldwinAnything else?15:40
safchainok for me15:40
amullerall good15:40
carl_baldwinThanks15:40
carl_baldwin#topic bgp-dynamic-routing15:40
*** openstack changes topic to "bgp-dynamic-routing (Meeting topic: neutron_l3)"15:40
carl_baldwindevvesa_: hi15:41
devvesa_hi15:41
devvesa_sorry, i was out last week15:41
*** hurgleburgler has joined #openstack-meeting-315:41
carl_baldwinAnything to report?15:41
*** banix has joined #openstack-meeting-315:42
devvesa_keep working on it, i am close to push a WIP patch soon15:42
devvesa_so you can start review it15:42
carl_baldwindevvesa_: That’d be great.15:43
carl_baldwinBe sure to ping me when you post it and I’ll have a look.15:43
devvesa_ok, great15:43
*** tomoe_ has quit IRC15:43
carl_baldwindevvesa_: Anything else?15:44
devvesa_no, anything else for the moment15:44
carl_baldwindevvesa_: thanks15:44
devvesa_thanks carl15:44
*** viveknarasimhan has quit IRC15:44
carl_baldwinAll of the other usual topics are deferred to Kilo.  I’ll defer discussion for now.15:45
carl_baldwin#topic reschedule-routers-on-dead-agents15:45
*** openstack changes topic to "reschedule-routers-on-dead-agents (Meeting topic: neutron_l3)"15:45
*** seizadi has quit IRC15:45
carl_baldwinkevinbenton: hi, this one is yours.15:45
*** Rajeev has joined #openstack-meeting-315:45
kevinbentontopic title is pretty self-explanatory. i would like routers to be taken off of dead agents so they can be automatically rescheduled15:46
kevinbentonhere is one approach https://review.openstack.org/#/c/110893/15:46
amullerSo, the L3 HA blueprint solves the same problem15:46
kevinbentonthis needs to be in icehouse15:47
kevinbentonIMO15:47
kevinbentonso i was hoping for a bugfix15:47
armaxkevinbenton, amuller I think the two overlap15:47
armaxand I see kevinbenton’s approach also a a contingency plan15:47
amullerI was under the impression that people use pacemaker and other solutions currently15:47
carl_baldwinThis might cross line from bug fix to feature.  Might be hard to get in to Icehouse.15:47
armaxthat mitigates the need of relying on external elements to the fail-over process15:48
carl_baldwinamuller: We’ve toyed around with a pacemaker solution.  A colleague gave a talk at the Atl. summit.15:48
*** julim has quit IRC15:48
*** seizadi has joined #openstack-meeting-315:48
aleksandr_nullamuller: I could confirm it from Mirantis Fuel perspective15:49
amullerWe use a Pacemaker based solution in RH OpenStack as well15:49
amullerto solve L3 HA issue15:49
kevinbentonit’s annoying to have to use an external process to do something as simple as rescheduling15:49
*** briancurtin has quit IRC15:49
aleksandr_nullwe using pm/crm to manage l3 agents and forcing rescheduling of routers. Of course with some downtime ;(15:49
carl_baldwinIn our testing, we found it very easy to get in to situations where nodes start shooting themselves.  It turned out to be somewhat difficult to get right.15:49
amullerkevinbenton: I agree, but I'm really conflicted if something like this should be merged... Since L3 HA is the concensus on how to do it, I'd be really careful with making the code any more complicated pre L3 HA15:50
amullerL3 HA == VRRP blueprint15:50
kevinbentoni don’t see how this is the same really15:51
kevinbentonit just does what can be done with the existing API15:51
armaxI think kevinbenton’s proposal targets non HA deployments15:51
armaxgranted we want to minimize potential code conflicts15:51
armaxso let’s see how the two develop and make a call later on when the code is more mature15:51
kevinbentonarmax: right, and i don’t think there would be15:51
aleksandr_nullFrom my point of view VRRP+cn_sync looks easier than rescheduling. In terms of used technologies. But it's not so easy to implement.15:51
kevinbentonaleksandr_null: what?15:52
amullerI know that Rackspace use something similar to your proposal Kevin15:52
kevinbentonaleksandr_null: did you see my patch?15:52
armaxI’d see L3HA the canonical way of doing things15:52
kevinbentonit’s like 10 lines15:52
amullerthey monitor the RPC bus and reschedule routers as needed15:52
aleksandr_nullkevinbenton: Will take a look, of course.15:52
*** jcoufal has joined #openstack-meeting-315:52
aleksandr_nullarmax: +115:52
*** nelsnelson has quit IRC15:52
armaxthat said, there are situations where L3HA as a solution won’t be available15:53
kevinbentonarmax: yes, l3ha is definitely the way to move forward, but I’m trying to address an issue in icehouse15:53
kevinbentonif possible15:53
armaxnow, people might have come up with their own solutions15:53
armaxhomegrown and painful15:53
armaxI think kevinbenton is trying to see whether some of that pain can be taken away :)15:53
kevinbentonit’s embarrassing that a node goes down and we just throw our hands up15:53
aleksandr_nullamuller: But what will happens if something will be wrong with communications inside of the cloud ? MQ fails from time to time, of course its out of scope but VRRP will do that autonomous15:53
carl_baldwinI’m concerned that simply rescheduling will not be enough.  A pacemaker/corosync type solution would shoot the dead node.  This solution would not.  With the agent down, there is no one left around to clean up the old router.15:54
*** briancurtin has joined #openstack-meeting-315:54
*** briancurtin has quit IRC15:54
*** briancurtin has joined #openstack-meeting-315:54
armaxI’d promote that effort, but I’d reserve me the judgment to see whether it’s icehouse/juno material once the code is complete15:54
amullerkevinbenton: If the code doesn't end up being more complicated after it's properly tested, and properly solved the problem, then it's safe enough to merge as it is, but I have a gut feeling that you'll find that it's gonna end up a lot more complicated15:54
armaxkevinbenton: how far off are you?15:54
*** nelsnelson has joined #openstack-meeting-315:55
*** pballand has quit IRC15:55
carl_baldwinIn many situations, the old routers could still be plumbed and moving traffic.15:55
*** nelsnels_ has joined #openstack-meeting-315:55
kevinbentoncarl_baldwin: do your compute nodes frequently lose connectivity to the neutron server?15:55
aleksandr_nullkevinbenton: it depends on architecture of cluster. We had an situation when customer just disabled mgmt/comm network.15:56
carl_baldwinkevinbenton: There are many reasons an agent can be considered dead.15:56
aleksandr_nullFor a while.15:56
*** nelsnels_ has quit IRC15:56
kevinbentonarmax: i have the basic patch there, but it doesn’t address zombie nodes like carl_baldwin mentioned15:56
kevinbentonaleksandr_null, carl_baldwin: how does the openvswitch agent handle a broken management network?15:57
kevinbentonwe have to assume that’s down too then, right?15:57
aleksandr_nullyep.15:57
carl_baldwinkevinbenton: yes, the agent goes inactive.15:58
kevinbentonwell then yes, i wasn’t aware we supported headless operational modes15:58
*** nelsnelson has quit IRC15:59
kevinbentonmy patch is pointless and this isn’t a problem that can be solved from the neutron server15:59
*** nelsnelson has joined #openstack-meeting-315:59
kevinbentonbecause it doesn’t actually know if these routers are online or not16:00
aleksandr_nullIMHO this could be solved only by using autonomous solutions like vrrp, I dont' have any other solutions related to RPC/MQ because they couldn't work autonomous =(16:00
carl_baldwinkevinbenton: It is still something that needs to be addressed.  Our pacemaker / corosync solutions have not been as great as we’d hoped.16:01
armaxkevinbenton’s solution obviously need cooperation between servers an agents16:01
armaxkevinbenton: saying its’ pointless is a bit harsh16:02
armax:)16:02
aleksandr_nullof course mgmt network outage is extraordinary case. Rescheduling that kevinbenton suggests maybe improved by trying to monitor neighborhood nodes and/or mgmt net and if something happens with mgmt net then don't do anything. Just suggestion.16:02
armaxevery solution has tradeoffs16:02
*** mfer has joined #openstack-meeting-316:02
carl_baldwinarmax: +116:02
aleksandr_null+116:02
armaxthe larger question is: do we want to provide some degree of built-in functionality?16:02
armaxwith any of the cons that may have?16:03
armaxexternal pcm/cm also has issues16:03
aleksandr_nullcarl_baldwin: Completely agree. pm/crm doing almost the same the Kevin suggested and also wouldn't work fine if mgmt network is down.16:03
armaxso long as kevin’s proposal is not disruptive to the current effort for L3HA16:03
aleksandr_nullcorosync cluster will just split up.16:03
amullerkevinbenton: We'd appreciate any contributions to the L3HA efforts :)16:03
armaxI’d like to have the option to decide whether to take it or not16:04
aleksandr_nullI could test it in corosync environment and without it16:04
armaxamuller: indeed, would kevin’s time best put to L3HA?16:04
armaxkevinbenton: that’s a question for kevin :)16:04
amullerkevinbenton: testing/reviewing would be awesome, and there's loose ends also16:04
armaxhe might have an hidden customer requiremetn ;)16:04
kevinbentonarmax: that’s not going to happen right now. I need a solution for icehouse16:04
kevinbentonarmax: not hidden :-)16:04
armaxkevinbenton: right, you know what  I mean16:05
armaxkevinbenton: I think it makes sense if you keep on working on this, let’s revise the progress in a week16:05
carl_baldwinI’m glad the discussion is opened.  HA will be a hard nut to crack.  Is this something we want to add to the permanent agenda?16:05
carl_baldwinI just noticed we’re over time.  Anyone else waiting for the room?16:06
armaxand see how far we got16:06
armaxwaaaay over time16:06
* carl_baldwin is really sorry about going over time if someone is waiting for the room.16:06
aleksandr_nulllooks like nobody :)16:06
armaxcarl_baldwin: they would have kicked us out16:06
armax:)16:06
armaxbye everyone16:07
yamamoto_bye16:07
kevinbentonbye16:07
aleksandr_nullbye guys!16:07
carl_baldwinI’ve got to run.  I’d like to discuss rescheduling routers more.  I’ll keep it in the agenda near HA.16:07
carl_baldwinI’ll also get some of our guys with experience with our HA solution on kevinbenton’s review to provide insight.16:07
carl_baldwinThanks all16:07
carl_baldwin#endmeeting16:08
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:08
openstackMeeting ended Thu Jul 31 16:08:06 2014 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:08
openstackMinutes:        http://eavesdrop.openstack.org/meetings/neutron_l3/2014/neutron_l3.2014-07-31-15.00.html16:08
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/neutron_l3/2014/neutron_l3.2014-07-31-15.00.txt16:08
openstackLog:            http://eavesdrop.openstack.org/meetings/neutron_l3/2014/neutron_l3.2014-07-31-15.00.log.html16:08
*** yyywu has quit IRC16:08
*** rudrarugge has joined #openstack-meeting-316:08
*** MaxV has quit IRC16:09
*** MaxV has joined #openstack-meeting-316:09
*** tmazur has joined #openstack-meeting-316:09
*** MaxV has quit IRC16:14
*** Swami has quit IRC16:17
*** xuhanp has quit IRC16:17
*** julim has joined #openstack-meeting-316:18
*** ivar-lazzaro has joined #openstack-meeting-316:20
*** seizadi has left #openstack-meeting-316:20
*** nelsnelson has quit IRC16:23
*** mfer has quit IRC16:29
*** Rajeev has quit IRC16:30
*** mfer has joined #openstack-meeting-316:30
*** flaviof_zzz is now known as flaviof16:36
*** mfer has quit IRC16:37
*** rudrarug_ has joined #openstack-meeting-316:38
*** MaxV has joined #openstack-meeting-316:38
*** rudrarugge has quit IRC16:40
*** tmazur has quit IRC16:42
*** devvesa_ has quit IRC16:43
*** MaxV has quit IRC16:45
*** flaviof is now known as flaviof_zzz16:45
*** MaxV has joined #openstack-meeting-316:46
*** mfer has joined #openstack-meeting-316:48
*** Krishna_ has joined #openstack-meeting-316:49
*** MaxV has quit IRC16:50
*** Krishna_ has quit IRC16:51
*** KrishnaK_ has joined #openstack-meeting-316:51
*** mrsmith has quit IRC16:54
*** banix has quit IRC16:56
*** banix has joined #openstack-meeting-317:00
*** briancurtin has quit IRC17:00
*** amuller has quit IRC17:00
*** briancurtin has joined #openstack-meeting-317:00
*** SumitNaiksatam has quit IRC17:01
*** iovadia has quit IRC17:03
*** yisun has quit IRC17:03
*** otherwiseguy has quit IRC17:03
*** briancurtin has quit IRC17:05
*** otherwiseguy has joined #openstack-meeting-317:08
*** briancurtin has joined #openstack-meeting-317:09
*** briancurtin has quit IRC17:09
*** briancurtin has joined #openstack-meeting-317:09
*** briancurtin has quit IRC17:10
*** briancurtin has joined #openstack-meeting-317:11
*** briancurtin has joined #openstack-meeting-317:11
*** julim has quit IRC17:15
*** SumitNaiksatam has joined #openstack-meeting-317:15
*** julim has joined #openstack-meeting-317:16
*** otherwiseguy has quit IRC17:16
*** julim has quit IRC17:17
*** otherwiseguy has joined #openstack-meeting-317:25
*** mestery has quit IRC17:31
*** mestery has joined #openstack-meeting-317:31
*** HenryG_ has joined #openstack-meeting-317:31
*** HenryG has quit IRC17:34
*** banix has quit IRC17:37
*** HenryG_ is now known as HenryG17:39
*** banix has joined #openstack-meeting-317:40
*** LouisF has joined #openstack-meeting-317:42
*** banix has quit IRC17:43
*** banix has joined #openstack-meeting-317:43
*** Sukhdev has joined #openstack-meeting-317:46
*** MaxV has joined #openstack-meeting-317:48
*** MaxV has quit IRC17:48
*** MaxV has joined #openstack-meeting-317:49
*** MaxV has quit IRC17:53
*** briancurtin has quit IRC17:53
*** rkukura has joined #openstack-meeting-317:55
*** briancurtin has joined #openstack-meeting-317:56
*** briancurtin has joined #openstack-meeting-317:56
*** Qijing has joined #openstack-meeting-317:58
SumitNaiksatamhi17:58
rkukurahi SumitNaiksatam17:58
banixhallo17:58
*** OSM has joined #openstack-meeting-317:58
*** OSM is now known as songole17:58
*** s3wong has joined #openstack-meeting-317:59
SumitNaiksatamrkukura: banix: hi18:00
SumitNaiksatamlets get started18:00
*** yyywu has joined #openstack-meeting-318:00
SumitNaiksatam#startmeeting networking_policy18:00
openstackMeeting started Thu Jul 31 18:00:45 2014 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.18:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.18:00
*** openstack changes topic to " (Meeting topic: networking_policy)"18:00
openstackThe meeting name has been set to 'networking_policy'18:00
SumitNaiksatamhttps://wiki.openstack.org/wiki/Meetings/Neutron_Group_Policy#July_31st.2C_201418:01
*** mandeep has joined #openstack-meeting-318:01
SumitNaiksatam#info agenda https://wiki.openstack.org/wiki/Meetings/Neutron_Group_Policy#July_31st.2C_201418:01
SumitNaiksatam#topic Patches in review18:01
*** openstack changes topic to "Patches in review (Meeting topic: networking_policy)"18:01
SumitNaiksatam#link https://wiki.openstack.org/wiki/Meetings/Neutron_Group_Policy/Patches18:01
SumitNaiksatamthere is a long standing -2 on the first patch of the series from markmcclain18:02
SumitNaiksatami have pointed this out in the -dev mailing list18:02
SumitNaiksatamwe also requested markmcclain to attend this meeting18:03
SumitNaiksatamhowever, he does not seem to be online18:03
mandeepmestery: How should we proceed on this?18:03
SumitNaiksatamwe have one +2 from rkukura on the patch18:03
mesterymandeep: It's hard to proceed unless markmcclain is here, I believe he's on a plane returning from the nova mid-cycle at the moment, but I'm not 100% sure.18:04
SumitNaiksatammestery: okay18:04
mandeepmestery: Quoting salvatore from the mailing list:18:04
mandeepI try to avoid -2s as much as possible. I put a -2 only when I reckon your18:04
mandeeppatch should never be merged because it'll make the software unstable or18:04
mandeeptries to solve a problem that does not exist. -2s stick across patches and18:04
SumitNaiksatammestery: from a project process/policy perspective, is there any remediation to this?18:04
mandeeptend to put off other reviewers.18:04
*** Youcef has quit IRC18:05
SumitNaiksatammestery: the issue has been raised in the mailing list as well18:05
mandeepmestery: Would you consider this appropriate use of -2 (assuming that definition)?18:05
rkukuraI don’t think it would hurt to encourage other core reviewers to review the initial patches and place their votes.18:05
mesterySumitNaiksatam: As I've indicated before, I can't make people remove -2s. We need Mark to respond to this and remove the -2 or justify it at this point, and he hasn't done either.18:05
mandeepmestery: And also raised on 1-1 email and IRC chats18:05
SumitNaiksatammestery: my suggestion, and as stated before as well, was never to ask you to ask markmcclain to remove his -218:06
mandeepmestery: Can you, as the PTL, take the action item to follow up on this?18:06
SumitNaiksatammestery: the suggestion was to find out why the -2 still persists18:06
mesterymandeep: I have already spoken to Mark many times about this to no avail.18:06
mesterySumitNaiksatam: ++18:06
*** briancurtin2 has joined #openstack-meeting-318:06
mesterySumitNaiksatam: That's really the crux, and I haven't had luck in getting that information out.18:06
SumitNaiksatammestery: okay, we know you have tried18:07
SumitNaiksatammestery: at this point, and in this forum, we are just trying to find out, from a process perspective, what needs to be done18:07
*** regXboi has joined #openstack-meeting-318:07
regXboisorry all - ODL TSC call running late18:07
regXboiso I'm here - but not here18:08
regXboiso please move my agenda item to near the end18:08
ivar-lazzaromestery: should there be a "no answer" time limit before the community takes action in these situations? (e.g. voting in the ML so that the PTL removes the -2?)18:08
SumitNaiksatammestery: alternatively, if there is no process/policy in place for this, we probably need to formulate a reasonable one18:08
SumitNaiksatamivar-lazzaro: +118:08
SumitNaiksatami believe, as a core, i am accountable to the -2 i put18:08
regXboiok... here now18:09
SumitNaiksatamso, if i put a -2, and a new patch set is posted, i review the -2 again18:09
SumitNaiksatami believe that should be an explicit process that everyone should follow18:09
regXboiSumitNaiksatam: you can say that here18:09
ivar-lazzaroSumitNaiksatam: +118:10
SumitNaiksatamif a new patchset is posted, i think the onus is on the core who put the -2, to justify why it should persist18:10
regXboibut since there hasn't been any email on the ML archives, I can't point to any public record of asking for the -2 to be reviewed18:10
regXboithere is finally something in the patch set archives, but that's semi-ephermeal18:10
*** briancurtin has quit IRC18:10
SumitNaiksatamregXboi: there are explicit guidelines to not hound reviewers on the -dev mailing list for reviews18:10
mandeepmestery: Do you agree that the process is broken and needs to be addressed? It was also being discussed on the mailing list todayt morning18:10
rkukuraregXboi: I explicitly asked Mark to either remove or justifiy his -2 in my review where I voted +2.18:11
mandeepmestery: http://lists.openstack.org/pipermail/openstack-dev/2014-July/041651.html18:11
*** briancurtin2 has quit IRC18:11
regXboirkukura: yes, that's the comment I was referring to18:11
regXboiand mandeep: that's a reasonable email18:11
mandeepregXboi: We are forbidden to ask for reviews on the mailing list.18:11
*** briancurtin has joined #openstack-meeting-318:11
mandeepregXboi: And that is why we have been sending 1-1 emails to ask for re-review18:12
SumitNaiksatamregXboi: regardless, you cant reason that the -2 persists because there wasnt an email in the mailing list pointing to it, its the duty to of the core to remove the -2 if the circumstances under which he put the -2 have changed18:12
* regXboi notes that rule is not in the interests of transparency18:12
mandeepregXboi: I am asking mestery if the process should be updated if multiple teams are hitting similar issues?18:13
regXboiSumitNaiksatam: that is not my point at all18:13
regXboimy point is that a rule not asking for a re-review on the ML is not in the interests of transparency18:13
SumitNaiksatamokay, we dont intend this to be a meeting where we fix neutorn process/policy issues, i think this team deserves to know why the patch is not allowed to make progress18:14
regXboibecause my first reaction would be (at this point) to say "d*mn the rules" :-)18:14
regXboibut that's me18:14
SumitNaiksatamhence this is being brought up in the agenda here18:14
SumitNaiksatamthe hope was that markmcclain would have been able to attend so that he could explain to the team18:14
mesteryregXboi: ++, there isn't any concern with sending these types of requests to the mailing list, in fact, in this case, that's where it should be discussed18:15
SumitNaiksatamunfortunately it seems that he is travel18:15
mandeepregXboi: And we are asking mestery (PTL) for direction18:15
SumitNaiksatammestery: so how is asking for a request to review a -2 different from asking for a request to review a patch (the latter is explicitly prohibited)18:16
mesteryMy point here is that in this case, with a lingering -2, I fail to see how requesting on the list is out of bounds, in fact, it's like a last resort IMHO.18:16
*** dkehnx has joined #openstack-meeting-318:16
SumitNaiksatammestery: okay, thanks for that guidance18:17
SumitNaiksatammestery: that said, this has already been raised in the email that i sent out yesterday18:17
ivar-lazzaromestery: well, technically they core could ignore the ML thread as well :D18:17
*** julim has joined #openstack-meeting-318:17
ivar-lazzarothey -> the18:18
regXboiivar-lazzaro: true, but then the weight of public opinion shifts18:18
ivar-lazzaroI had the impression that -2s were a technical issue, not a political one18:18
SumitNaiksatamregXboi: why should we have to waste time in the first place having to build the weight of the public opinion in one way or the other//18:18
mandeepregXboi: I hope we are not becoming politicians and responding to public opinion as opposed to technical concerns18:19
SumitNaiksatammandeep: exactly my point18:19
regXboiok folks, let's be real here18:19
regXboiany time you have more than 2 people and money on the table, you have policics18:19
regXboier politics18:19
rkukurawe need some checks and balances, it seems18:19
SumitNaiksatamok i will have to reign it here!18:19
regXboieven in the perceived optics of the situation there are politics18:19
s3wongIn terms of policy change - one way to do so may be how contributors can vote for PTL, contributors can also vote for or against cores18:19
*** jerryz has joined #openstack-meeting-318:20
regXboiSumitNaiksatam: ++ :)18:20
SumitNaiksatamok so moving on for the time being18:20
SumitNaiksatamall good suggestions18:20
SumitNaiksatamhad to allow everyone (including myself) to vent a little! ;-P18:20
mandeeps3wong: That is a good idea18:20
ivar-lazzaros3wong: +118:20
mandeeps3wong: If the public opinion is going to decide it, let the ATC decide it18:20
SumitNaiksatamthe other update on the patch series is that all the three series are linear now18:21
SumitNaiksatamthat means you can go to GP-PLG-3 and be able to pull everything before it18:21
mandeepSumitNaiksatam: Thanks. That helps a lot!18:21
SumitNaiksatamGP-PLG-3 did not pass jenkins because a db migration is dated18:21
SumitNaiksatambut it passes UTs, and should be functional18:22
SumitNaiksatammestery: before i forget, thanks for stopping by and providing guidance18:22
SumitNaiksatamany questions on series 1, 2, and 3?18:23
regXboiother than I have to check that I've got +1s on all of them - no :)18:23
SumitNaiksatamregXboi: thanks ;-)18:23
SumitNaiksatamthanks for reviewing that is18:23
*** yamamoto_ has quit IRC18:24
*** nelsnelson has joined #openstack-meeting-318:24
SumitNaiksatamGP-*-1, and GPM-*-1 were rebased until yesterday18:24
*** nelsnelson has quit IRC18:24
SumitNaiksatamGP-*-2/3 need to rebased18:24
SumitNaiksatam#topic Mapping model/driver update18:24
*** openstack changes topic to "Mapping model/driver update (Meeting topic: networking_policy)"18:25
SumitNaiksatamrkukura: over to you, anything new to report here?18:25
mandeepwith DB migrations in place we will have to rebase often, and that will keep wiping off all the +1s - perhaps gerrit should not wipe out +1s for dbmigration updates18:25
regXboimandeep: that's actually a good point for reviewers - be prepared to do a lot of repeat +1s :/18:25
SumitNaiksatammandeep: ah, did not think of that, not sure how easy/difficult it is to achive that18:25
rkukurathe DB migration for gpm-db-1 was a bit of a challenge, but thanks to HenryG and akamyshnikova, its all worked out18:25
SumitNaiksatamregXboi: thanks for accepting that responsibility! :-)18:26
*** thangp has quit IRC18:26
SumitNaiksatamrkukura: that was great that you were able to fix those issues18:26
regXboiSumitNaiksatam: I have been - I've got filters on all the re-review mails so I can hit them first thing in the morning :)18:26
SumitNaiksatamrkukura: any short pointers for the rest of the team here in terms what you did to get around the issue?18:26
mandeepregXboi: I agree :-(18:26
rkukuraI’ll probably include some more validation checks in a gpm-rmd-1 update at some point18:26
*** nelsnelson has joined #openstack-meeting-318:26
SumitNaiksatamregXboi: sounds very organized, great!18:27
rkukuraOne issue was that alembic sometimes does not generate the foriegn key constraints that are in the model18:27
rkukuraAlso, when removing columns in a downgrade, some constraint types need to be removed first, and others don’t18:28
SumitNaiksatamrkukura: ah okay, so you had to manually add those?18:28
rkukuraIts very easy to test the migrations if you’ve got a devstack environment18:28
SumitNaiksatamrkukura: i mean the foreign key18:28
regXboirkukura: is *that* what that was all about18:28
* regXboi mutters ugly things about code that sometimes does things and sometimes doesn't18:29
SumitNaiksatamfor reference, its the DB migration in this patch #link https://review.openstack.org/#/c/10179518:29
rkukuraI had to manually add the code add and remove foreign keys, and remove the generated code for removing unique constraints18:29
SumitNaiksatamrkukura: ok cool, i guess we can follow that example in the future18:30
s3wongrkukura: wouldn't the code be regenerated to remove your manual changes?18:30
rkukuraakamyshnikova is very helpful on reviewing these, and tested the latest version with postgresql as well18:30
SumitNaiksatamrkukura: sweet18:30
rkukurawe don’t usually regenerate the migration code18:30
rkukuraits a generate once, then edit, process18:30
SumitNaiksatamakamyshnikova: much appreciated!18:30
rkukurathe generated migrations always need some cleanup18:30
*** mfer has quit IRC18:31
SumitNaiksatamrkukura: also the issues that you ran into for generating the migration18:31
SumitNaiksatamrkukura: is the env module change merged upstream?18:31
rkukuraSumitNaiksatam: Right, not sure if my workaround would still be needed. Hopefully its fixed.18:31
SumitNaiksatamrkukura: ah ok18:32
SumitNaiksatamany questions for rkukura on the mapping part? (since hopefully you are reviewing that right now ;-P )18:32
rkukuraSo s3wong is making good progress on  the next step of the mapping - creating SGs for enforcement of the policy rules.18:33
SumitNaiksatamrkukura: yeah thats next on the agenda item18:33
SumitNaiksatam#topic Security Groups mapping update18:33
*** openstack changes topic to "Security Groups mapping update (Meeting topic: networking_policy)"18:33
SumitNaiksatams3wong: your turn :-)18:33
s3wongSo I have been coding up the contract=>SG mapping18:34
s3wongso far I coded up the create/update/delete_contract part, and the create/update_EPG part (the latter almost done)18:35
s3wongstill need to test, get them ready for gerrit18:35
SumitNaiksatams3wong: okay nice18:35
regXbois3wong: I see you want to defer the question of "which SG is matched" to the drivers, which is fair18:35
s3wongalso adding unit test as I go along18:35
SumitNaiksatamregXboi: good point to bring up18:35
regXboibut that begs the question of how the driver in the patch set will handle that18:35
SumitNaiksatamso at this point, we think that the SG mapping is neutron resource mapping driver specific18:36
regXboiSumitNaiksatam: yes I think defering to the drivers is the right answer18:36
regXboibut if this includes a sample driver, that has to answer it :/18:36
s3wongregXboi: yes, now it is rendered directly by the mapping driver18:36
*** flaviof_zzz is now known as flaviof18:36
SumitNaiksatamregXboi: in terms of patch logistics, this will be a new GPM-RMD-3 patch, right s3wong?18:37
s3wongSumitNaiksatam: that's correct18:37
regXboiworks for me18:37
regXboiand I'll be reading when it arrives :)18:37
SumitNaiksatamokay any other questions for s3wong?18:38
s3wongso yeah, guys - stay tuned, patches will come soon18:38
SumitNaiksatams3wong: any blockers for you at this point?18:38
s3wongSumitNaiksatam: no18:38
s3wongSumitNaiksatam: able to proceed well since you rebased18:39
SumitNaiksatams3wong: ok good to hear18:39
SumitNaiksatammoving on18:39
SumitNaiksatam#topic CLI/Client18:39
*** openstack changes topic to "CLI/Client (Meeting topic: networking_policy)"18:39
*** rudrarugge has joined #openstack-meeting-318:39
SumitNaiksatamregXboi: we will get to your part in a bit18:39
SumitNaiksatamlets first get the update18:39
SumitNaiksatamsongole: there?18:40
songoleyes SumitNaiksatam18:40
songoleUpdate: posted patches for all GBP resources18:40
songoletests are pending18:40
songoleNeed to handle mapping extension18:41
LouisFsongole: what are patch numbers?18:41
mandeepsongole: I have created a couple of public devstack VMs to do integration of the mapping driver, let me know if you need access to that.18:41
songolehttps://review.openstack.org/#/c/104013/18:42
songoleit has dependency links to others18:42
regXboium18:42
regXboithat patch doesn't cover anything past -118:42
regXboiwhen are -2 and -3 going to be added?18:42
songoleregXboi: they are all added18:43
mandeepregXboi: https://review.openstack.org/#/c/110798/18:43
*** rudrarug_ has quit IRC18:43
mandeepregXboi: https://review.openstack.org/#/c/110806/18:43
SumitNaiksatamsongole: i am not sure i understood “Need to handle mapping extension”18:43
songolemandeep: thanks18:43
mandeepregXboi: (added links for the record)18:43
regXboium... thos patches aren't listed in the wiki?!?!18:44
regXboi*those18:44
songoleSumitNaiksatam: end point can take a port, l2 policy could take a network18:44
songolethose options need to be added to CLI-118:44
SumitNaiksatamsongole: oh okay, got it18:44
SumitNaiksatamregXboi: point noted18:45
*** briancurtin has left #openstack-meeting-318:45
SumitNaiksatamsongole: can you please update the wiki page: https://wiki.openstack.org/wiki/Meetings/Neutron_Group_Policy/Patches18:45
songolemandeep: I will ping on reg those public devstack setups18:45
songoleSumitNaiksatam: will do18:45
regXboiSumitNaiksatam, songole: thanks - I've been driving reviews based on that webpage18:45
mandeepsongole: OK18:45
SumitNaiksatambtw, comment not directed just to songole, anyone if you see the wiki is outdated, please update with your patch link18:46
SumitNaiksatam*links18:46
regXboi++18:46
SumitNaiksatamregXboi: that was the reason we put the wiki, so you are right in going by that18:46
mandeepregXboi: agreed18:46
SumitNaiksatamsongole: thats excellent progress, thanks much!18:46
SumitNaiksatamok now to regXboi’s suggestion18:47
SumitNaiksatam#topic “Profiled” API18:47
*** openstack changes topic to "“Profiled” API (Meeting topic: networking_policy)"18:47
SumitNaiksatamregXboi: the floor is yours :-)18:47
regXboifolks have probably all seen the email, but let's link it in for the record18:47
regXboithe idea is to bring back support for the 2-group approach with minimal intrusiveness18:48
regXboi#link http://lists.openstack.org/pipermail/openstack-dev/2014-July/041467.html18:48
SumitNaiksatamregXboi: thanks was just about to post the link18:48
regXboithe suggestion is to do this as a patch on CLI-3 (call it CLI-4) because it is a departure from normal openstack convention18:49
regXboiand we can go that direction18:49
SumitNaiksatamregXboi: ah cool18:49
regXboiwith the caveat that we may have to pry open the API to allow the CLI patch to make a single call to neutron18:49
mandeepregXboi: That also answer a few questions on what a "profiled API" is,18:50
regXboiIIRC the 2-group approach didn't discuss about reuse of policy information, so I have some freedom in trying to make that work18:50
regXboimandeep: the idea of a profiled API come from IETF and other places18:50
* regXboi goes and gets links18:50
mandeepregXboi: The patch can also include diffs to other resources (as required), and we can review them as a single change18:51
SumitNaiksatamregXboi mandeep: i am still mystified by the term “profiled”, but i am just ignorant!18:51
regXboiexamples of what I mean:18:51
regXboiOASIS (for SAMLv2): http://docs.oasis-open.org/security/saml/v2.0/saml-profiles-2.0-os.pdf18:51
regXboiEncryption and Checksum Specifications for Kerberos 5: http://tools.ietf.org/html/rfc396118:51
regXboiInternet X.509 Public Key Infrastructure Certificate and Certificate Revocation List (CRL) Profile: http://www.ietf.org/rfc/rfc3280.txt18:51
SumitNaiksatamregXboi: thanks18:51
mandeepregXboi: I am not familiar wit that, but it sounds like an interesting idea.18:51
regXboiInternet X.509 Public Key Infrastructure Qualified Certificates Profile: http://ftp.isi.edu/in-notes/rfc3039.txt18:51
mandeepregXboi: I will check the link. Thanks.18:52
regXboithe point being to define common sets of values for otherwise complex things18:52
SumitNaiksatamregXboi: can you explain what you mean by “we may have to pry open the API”18:52
regXboiSumiNaiksatam: sure18:52
*** thomasem has joined #openstack-meeting-318:52
banixSumitNaiksatam: looking at neutron client i do not see any command that leads to anything but a single crud call to neutron. don’t you think departure from that will be difficult?18:52
regXboithe issue that may arise is that the CLI is going to make multiple calls to neutron backend18:52
regXboiso if folks complain that is uncoventional, we may have to allow the CLI to make a single call and move the profile in to the API side of the patch18:53
SumitNaiksatambanix: yeah, this amounts to some orchestration on the client side18:53
songoleregXboi: CLI is not an orchestrator today. Do you think heat might be a better place to put it?18:53
regXboipoint folks: HEAT is already doing some orchestration on the client side18:53
regXboisongole: no... because that bypasses horizon18:54
SumitNaiksatamregXboi: i think we have some freedom to do the same orchestration in Horizon18:54
regXboithat's doing double the necessary work18:54
mandeepregXboi: If new orchestration is involved, do you think that we will need do a BP as well?18:54
*** mfer has joined #openstack-meeting-318:54
*** thomasem has quit IRC18:54
regXboimandeep: I don't believe so but I don't *know*18:55
SumitNaiksatami dont think we need a new BP if only client side changes are involved18:55
regXboimestery might have some input here....18:55
mandeepregXboi: Horizon equivalent of orchestration can be a wizard, but if needed it should be able to do the orchestration18:55
SumitNaiksatammandeep: yeah, thats what i was thinking as well18:55
mesteryregXboi: Lets discuss this and see, I don't know off the top of my head18:55
mandeepregXboi: But I will need to check if that is being done for now18:55
regXboimandeep: that's not my point - somebody sharp might go "why are you doing this twice?"18:56
SumitNaiksatammestery: any restrictions on doing some orchestration on the client side?18:56
regXboiand we won't have a good answer at that point18:56
SumitNaiksatammestery: i am not aware that there are, but i dont think that there is a precedent either18:56
mesterySumitNaiksatam: ++, I'm not sure either18:56
* regXboi is known for setting precedents :)18:56
SumitNaiksatamregXboi: i think if its reasonable, there is always a first!18:57
regXboianyway... that's the idea18:57
regXboiI was going to say I needed CLI-3 as a starting point18:57
SumitNaiksatamregXboi: okay thanks18:57
regXboibut now that is there :)18:57
SumitNaiksatamregXboi: yes we have to thank songole for getting it in expeditiously18:57
mandeepregXboi: A quick look at the example profile API links, I would have expected a BP for such a chnage.18:57
LouisFregXboi: should CLI be: neutron policy-apply [contract] [src group] [destination group]18:58
regXboiLouisF: I think policy-apply will need to change to be consistent with CLI-318:58
rkukuraOne thing to keep in mind is that the GP API in Juno will be considered experimental, so tweeks to make it more usable in Kilo are certainly possible, even breaking backward compatibility. This CLI approach could be thought of as a temporary workaround.18:58
regXboirkukura: ++18:58
LouisFrkukura: ++18:59
regXboimandeep: I'm not sure I agree with you, but that's just my opinion18:59
*** mfer has quit IRC18:59
banixmandeep: regXboi is not suggesting a generic profiled cli; just use of the concept; even that in my opinion it wont get through but i am a bit too cynic at this point18:59
regXboibanix: thanks for pointing that out18:59
SumitNaiksatamrkukura: agree18:59
regXboiI am *NOT* suggesting a general framework - that's a whole different kettle of fish18:59
regXboiI'm talking about a one off19:00
mandeepbanix: Thanks for the clarification19:00
mandeepregXboi: But hopefully a one-off that sets a precedence for other similar updates ... ;-)19:00
SumitNaiksatamregXboi: it sounds perfectly reasonable to me suggest this approach in a patch19:00
regXboianyway - we're at the top of the hour - any more questions?19:00
mandeepSumitNaiksatam: +119:00
SumitNaiksatamregXboi: thanks for that update and clarification19:00
banixI left a comment at #link https://review.openstack.org/#/c/103755/ a few days ago on Patch Set 3; i think regardless of what Ryan is suggesting this should be trivial to do; do you guys see any reason why it cannot be done?19:00
SumitNaiksatambanix: sorry i missed that19:01
regXboiand if I'm not here - reply to the thread on the ML19:01
SumitNaiksatambanix: until you pointed out in the email thread19:01
SumitNaiksatambanix: i will respond19:01
SumitNaiksatam#topic API Intercept19:01
*** openstack changes topic to "API Intercept (Meeting topic: networking_policy)"19:01
* regXboi pads away19:01
SumitNaiksatamjust a quick plug for the work kevinbenton has been doing #link https://review.openstack.org/#/c/10990119:01
SumitNaiksatam#topic Open Discussion19:02
*** openstack changes topic to "Open Discussion (Meeting topic: networking_policy)"19:02
SumitNaiksatamwe will need to wrap up quickly since we are over19:02
SumitNaiksatamwe did not touch on the vendor drivers today19:02
rkukuraI’ve started looking at the intercept patch, but need to spend more time to understand it.19:02
SumitNaiksatambut hopefully people are thinking about it and are not overly blocked19:02
mandeepSame here19:02
SumitNaiksatamrkukura mandeep: thanks19:03
rms_13+119:03
SumitNaiksatamrms_13: any quick update on Horizon?19:03
SumitNaiksatamhorizon integration that is19:03
rms_13Not today. For sure next thursday as I am going to work full time on it tomorrow19:03
SumitNaiksatamrms_13: nice, looking forward to it19:03
rms_13Abishek is going to do initial glance19:03
SumitNaiksatamrms_13: okay19:03
SumitNaiksatamrms_13: you might want to look at songole’s new patches19:04
rms_13Yes. That is going to change my current patch19:04
SumitNaiksatamalright anything else, anyone?19:04
SumitNaiksatamalright, thanks all for joining19:05
banixbye19:05
rms_13by19:05
songolebye19:05
rkukurabye19:05
SumitNaiksatamuntil next week, but please keep reviewing19:05
LouisFrms_13: has horizon ptach been updated?19:05
SumitNaiksatambye19:05
*** regXboi has left #openstack-meeting-319:05
*** banix has quit IRC19:05
mandeepbye19:05
SumitNaiksatam#endmeeting19:05
*** rkukura has left #openstack-meeting-319:05
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"19:05
openstackMeeting ended Thu Jul 31 19:05:45 2014 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)19:05
openstackMinutes:        http://eavesdrop.openstack.org/meetings/networking_policy/2014/networking_policy.2014-07-31-18.00.html19:05
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/networking_policy/2014/networking_policy.2014-07-31-18.00.txt19:05
openstackLog:            http://eavesdrop.openstack.org/meetings/networking_policy/2014/networking_policy.2014-07-31-18.00.log.html19:05
rms_13LouisF: no19:05
SumitNaiksatamLouisF: i think rms_13 plans to do that tomorrow19:05
*** songole has left #openstack-meeting-319:05
SumitNaiksatamor starting tomorrow19:06
LouisFrms_13: thx19:06
rms_13But its always good to take a look.19:06
rms_13Whatever is there is going to stay with API changes only19:06
rms_13So review comments will be appreciated19:06
*** Qijing has quit IRC19:07
*** KrishnaK_ has quit IRC19:07
*** ivar-lazzaro has quit IRC19:12
*** mandeep has quit IRC19:13
*** pballand has joined #openstack-meeting-319:19
*** LouisF has quit IRC19:23
*** coolsvap has quit IRC19:24
*** banix has joined #openstack-meeting-319:27
*** yamamoto has joined #openstack-meeting-319:29
*** coolsvap has joined #openstack-meeting-319:29
*** yamamoto has quit IRC19:34
*** jcoufal has quit IRC19:39
*** jcoufal has joined #openstack-meeting-319:40
*** Sukhdev has quit IRC19:46
*** chuckC has quit IRC19:47
*** jaypipes has joined #openstack-meeting-319:55
*** thomasem has joined #openstack-meeting-320:05
*** peristeri has quit IRC20:05
*** chuckC has joined #openstack-meeting-320:08
*** thomasem has quit IRC20:09
*** peristeri has joined #openstack-meeting-320:10
*** eghobo has joined #openstack-meeting-320:17
*** peristeri has quit IRC20:23
*** thomasem has joined #openstack-meeting-320:26
*** SumitNaiksatam has quit IRC20:30
*** thomasem has quit IRC20:44
*** nelsnelson has quit IRC20:46
*** yamamoto has joined #openstack-meeting-320:48
*** thomasem has joined #openstack-meeting-320:51
*** yamamoto has quit IRC20:53
*** SumitNaiksatam has joined #openstack-meeting-320:55
*** nelsnelson has joined #openstack-meeting-320:57
*** mfer has joined #openstack-meeting-320:57
*** thomasem_ has joined #openstack-meeting-321:02
*** sgordon has joined #openstack-meeting-321:02
*** thomasem has quit IRC21:05
*** thomasem has joined #openstack-meeting-321:07
*** openstack has joined #openstack-meeting-321:08
*** ChanServ sets mode: +o openstack21:08
*** flaviof is now known as flaviof_zzz21:08
*** armax has quit IRC21:08
*** yamamoto has joined #openstack-meeting-321:09
*** sgordon has left #openstack-meeting-321:10
*** thomasem_ has quit IRC21:10
*** thomasem has quit IRC21:11
*** mrunge has quit IRC21:11
*** yamamoto has quit IRC21:22
*** banix has quit IRC21:32
*** ivar-lazzaro has joined #openstack-meeting-321:40
*** mestery has quit IRC21:53
*** mestery has joined #openstack-meeting-321:54
*** nelsnelson has quit IRC21:57
*** yamamoto has joined #openstack-meeting-321:59
*** rfolco has quit IRC22:00
*** yamamoto has quit IRC22:03
*** otherwiseguy has quit IRC22:04
*** otherwiseguy has joined #openstack-meeting-322:09
*** mfer has quit IRC22:15
*** armax has joined #openstack-meeting-322:22
*** mestery has quit IRC22:22
*** mestery has joined #openstack-meeting-322:23
*** rms_13 has quit IRC22:27
*** igordcard has joined #openstack-meeting-322:27
*** mfer has joined #openstack-meeting-322:38
*** jcoufal has quit IRC22:39
*** jcoufal has joined #openstack-meeting-322:40
*** mikal_ is now known as mikal22:40
*** stanzgy has joined #openstack-meeting-322:42
*** HenryG has quit IRC22:45
*** safchain has quit IRC22:46
*** igordcard has quit IRC22:55
*** igordcard has joined #openstack-meeting-322:56
*** yamamoto has joined #openstack-meeting-322:59
*** carl_baldwin has quit IRC23:01
*** yamamoto has quit IRC23:03
*** tsufiev has quit IRC23:04
*** jcoufal has quit IRC23:05
*** tsufiev has joined #openstack-meeting-323:09
*** pballand has quit IRC23:18
*** jaypipes has quit IRC23:27
*** pballand has joined #openstack-meeting-323:40
*** s3wong has quit IRC23:43
*** pballand has quit IRC23:44
*** yamamoto has joined #openstack-meeting-323:48
*** yamamoto has quit IRC23:53
*** alexpilotti has quit IRC23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!