Tuesday, 2014-03-18

openstackgerritCarl Baldwin proposed a change to openstack/neutron: L3 agent gives preference to processing RPC messages over full sync (WIP)  https://review.openstack.org/7881900:01
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS support for VPN service admin state change and reporting  https://review.openstack.org/8112400:02
*** carl_baldwin has quit IRC00:03
*** sfox has quit IRC00:04
*** pcm_ has quit IRC00:05
_cjones_ Anyone here to answer a hopefully simple question?00:09
_cjones_If I have Neutron enabled to do my networking, when Nova makes a call to network_api "allocate_for_instance()" where does this call end up in Neutron?00:10
*** dosaboy_ has quit IRC00:14
*** dosaboy has joined #openstack-neutron00:15
*** sweston has quit IRC00:15
*** mlavalle has quit IRC00:16
*** sweston has joined #openstack-neutron00:16
*** julim has joined #openstack-neutron00:17
*** baoli has joined #openstack-neutron00:20
*** thuc_ has quit IRC00:20
*** thuc has joined #openstack-neutron00:20
*** sweston has quit IRC00:21
*** julim has quit IRC00:24
lifeless_cjones_: that RPC is within the nova network RPC abstraction, so you should read nova/network/neutronv2/api.py stuff00:24
sc68cal^ +100:24
*** thuc has quit IRC00:25
*** emagana has quit IRC00:27
*** matsuhashi has joined #openstack-neutron00:27
*** Sukhdev has joined #openstack-neutron00:27
_cjones_lifeless, sc68cal. Thanks.00:28
sc68calI *think* it basically does a port_create call via the REST API00:29
sc68calthen eventually it percolates down to the virt driver for plugging00:29
*** manishg has quit IRC00:29
sc68caland libvirt for example, creates a bridge interface and wires up the tap device to it00:30
sc68cal_cjones_: http://docs.openstack.org/developer/neutron/devref/openvswitch_agent.html00:30
*** pbeskow has quit IRC00:35
openstackgerritSean M. Collins proposed a change to openstack/neutron: Ensure entries in dnsmasq belong to a subnet using DHCP  https://review.openstack.org/6457800:35
*** pbeskow has joined #openstack-neutron00:36
*** inara` has quit IRC00:36
_cjones_sc68cal. Yeah, I was just curious as to the entry point in Neutron. I'm getting closer.00:40
*** yamahata has joined #openstack-neutron00:40
_cjones_sc68cal. I've already created the bulk of my ML2 driver, but I was curious about other bits like where the IP address gets picked in "allocate_for_instance()". I'm still swimming through the code.00:41
*** banix has joined #openstack-neutron00:41
sc68cal_cjones_: base plugin v200:41
sc68calfetching link00:41
*** leseb has joined #openstack-neutron00:42
sc68cal_cjones_: https://github.com/openstack/neutron/blob/master/neutron/db/db_base_plugin_v2.py#L34100:42
_cjones_Yup. Got it. Thanks. I've nearly pieced it all together. Thanks.00:42
sc68calfeel free to write some doc up and contribute it00:42
sc68calif you're keeping notes00:43
sc68calwe could use some additional doc in the devref00:43
_cjones_Heh. Okay will do.00:43
sc68calhttps://github.com/openstack/neutron/tree/master/doc/source/devref00:43
_cjones_This would all come in through q-svc, no?00:43
sc68calyeah00:43
sc68calI have some doc pending about how q-svc starts up and fires up the REST API00:44
sc68calhttps://review.openstack.org/#/c/79675/00:44
*** inara has joined #openstack-neutron00:44
*** BuSerD has quit IRC00:46
*** leseb has quit IRC00:46
_cjones_Do you have any locations of diagrams describing the flow of messaging just within Neutron.00:47
_cjones_?00:47
*** markmcclain has joined #openstack-neutron00:49
_cjones_I was hoping to just have a 10,000 foot picture of how q-svc, q-agt, q-dhcp q-l3 q-meta are all related (or not for that matter)00:49
*** dkehn-pad has joined #openstack-neutron00:50
dkehn-paddkeh_: ping00:51
*** thuc has joined #openstack-neutron00:51
dkehn-paddkehn_ ping00:51
*** dkehn-pad has quit IRC00:51
*** tomoe_ has joined #openstack-neutron00:53
*** alexpilotti has quit IRC00:55
*** yamahata has quit IRC01:03
*** yamahata has joined #openstack-neutron01:03
*** _cjones_ has quit IRC01:04
oda-gHi core reviewes, would you pick up https://review.openstack.org/#/c/79858/ which needs one more +2 ?01:07
openstackgerritKevin Benton proposed a change to openstack/neutron: Allow CIDRs with non-zero masked portions  https://review.openstack.org/8113701:09
*** tomoe_ has quit IRC01:10
openstackgerritberlin proposed a change to openstack/neutron: Add session persistence support for NVP advanced LBaaS  https://review.openstack.org/5914601:11
*** sweston has joined #openstack-neutron01:11
*** markmcclain has quit IRC01:16
*** banix has quit IRC01:18
*** networkstatic has quit IRC01:18
*** tomoe_ has joined #openstack-neutron01:27
*** morganfainberg is now known as morganfainberg_Z01:30
*** ijw has quit IRC01:41
*** banix has joined #openstack-neutron01:41
*** dave_tucker_zzz is now known as dave_tucker01:42
*** ijw has joined #openstack-neutron01:42
*** HenryG has joined #openstack-neutron01:43
*** alagalah has quit IRC01:46
*** ijw has quit IRC01:47
openstackgerritKevin Benton proposed a change to openstack/neutron: Show neutron API request body with debug enabled  https://review.openstack.org/8110901:47
*** ljjjustin has joined #openstack-neutron01:48
*** morganfainberg_Z is now known as morganfainberg01:49
*** BuSerD has joined #openstack-neutron01:54
*** ram5391 has joined #openstack-neutron02:01
*** ljjjustin has quit IRC02:03
*** xianghui has joined #openstack-neutron02:04
ram5391hi, after deploying quantum-gateway with juju I try to login to horizon, I recieve a 'Connection to neutron failed: Maximum attempts reached' error02:04
openstackgerritshihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API  https://review.openstack.org/7380202:04
ram5391I am on the quantum-gateway machine and the neutron service is running, does rabbitmq and neutron need to be on the same macine?02:05
swestonram5391: no, they are two independent services.02:06
*** terrylhowe has left #openstack-neutron02:07
ram5391sweston: right, could this be a username/password mismatch issue between neutron and rabbimq?02:08
*** morganfainberg is now known as morganfainberg_Z02:08
swestonram5391: I don't see how they could have gotten out of sync ...02:10
swestonram5391: possibly a connection limit in your database?  check your open connections in your database server02:11
*** manishg has joined #openstack-neutron02:12
ram5391sweston: currently mysql is set at -1 for connection limit through juju. No idea if that is what it should be02:13
swestonram5391: can you make a connection to your mysql server?  if you can, then I'd say it would be safe to rule that out as an issue ;-)02:15
*** banix has quit IRC02:15
swestonram5391: what conditions lead you up to this error?02:16
*** manishg_ has joined #openstack-neutron02:16
ram5391sweston: that would be a sound assumption, I can connect fine. All I am doing is deploying a new installation of openstack via juju.02:16
*** manishg has quit IRC02:17
ram5391the service are spread out on vms running on the rc/cc, when I login to horizon, I get the error of not being able to contact neutron02:17
ram5391literal string is 'Connection to neutron failed: Maximum attempts reached'02:18
*** RajeshMohan has quit IRC02:18
swestonram5391: i just looked at the code.  that error actually means your web server cannot contact the neutron server.02:20
*** harlowja is now known as harlowja_away02:21
ram5391sweston: interesting.. stupid question, does the quantum gateway machine need to be exposed?02:22
swestonram5391: I would check your neutron server process ... it is complaining about the api service, so unless juju changed the port, it should be on 969602:24
*** manishg_ has quit IRC02:24
swestonram5391: that's a possibility that you had a route in your neutron installation that borked things up for you02:25
openstackgerritA change was merged to openstack/neutron: Create new IPv6 attributes for Subnets  https://review.openstack.org/5298302:26
ram5391sweston: okay, I'm not super familiar (if you coulden't tell) with neutron, so what exactly would be a "gotcha" when looking at the api service?02:26
*** spandhe has quit IRC02:27
*** Sukhdev has quit IRC02:27
ram5391If I navigate to the url :9696 I recieve a valid xml document02:27
ram5391however, telnet from the horizon machine to the quantum machine does not show an xml document02:28
ram5391sweston: nevermind, upon curling I get the xml02:29
swestonram5391: that's alright, we are all learning :-).  try this command wget -O - http://<your-neutron-ip>:969602:30
*** thuc has quit IRC02:32
openstackgerritA change was merged to openstack/neutron: Calculate stateless IPv6 address  https://review.openstack.org/5618402:32
ram5391sweston: request came back 200 okay02:32
openstackgerritKevin Benton proposed a change to openstack/neutron: Ignore PortNotFound exceptions on lockless delete  https://review.openstack.org/8102502:32
*** thuc has joined #openstack-neutron02:32
swestonram5391: are you making the connection from your horizon server to the neutron ip?02:33
ram5391sweston: yes indeed02:33
ram5391sweston: i'm even using the dns instead of IP02:34
*** thuc has quit IRC02:37
marunmestery: ping02:37
marunrkukura: ping02:38
swestonram5391: the neutron client is the only place where that error is thrown from.  So you have reached the end of my thought process, unfortunately.  If I were you, next I would try getting the stack trace out of horizon ... you will have to know where it's making the connection from/to in order to proceed.02:38
*** arosen-home has joined #openstack-neutron02:39
ram5391sweston: ah okay, thanks for the help, perhaps I should re-install horizon just for good measure02:39
swestonram5391: that's one way to approach it ;-).  Good luck.02:40
ram5391sweston: can't hurt to try, thanks!02:41
swestonram5391: sure thing :-)02:42
*** tomoe_ has quit IRC02:43
*** gongysh has joined #openstack-neutron02:44
ram5391also, for anyone else who might be there, running any command on the quantum-gateway machine (such as neutron agent-list) also returns 'Connection to neutron failed: Maximum attempts reached'02:48
*** edhall_ is now known as edhall02:49
*** gongysh has quit IRC02:49
*** suresh12 has quit IRC02:51
*** crc32 has quit IRC02:55
*** alagalah has joined #openstack-neutron02:57
*** dave_tucker is now known as dave_tucker_zzz03:00
*** tomoe_ has joined #openstack-neutron03:02
*** alagalah has quit IRC03:02
*** gongysh has joined #openstack-neutron03:02
openstackgerritXu Han Peng proposed a change to openstack/neutron: Permit ICMPv6 RAs only from known routers  https://review.openstack.org/7225203:03
*** dvorkinista has joined #openstack-neutron03:03
swestonyamata: ping03:05
*** harlowja_away is now known as harlowja03:06
*** BuSerD has quit IRC03:07
*** gongysh has quit IRC03:08
*** BuSerD has joined #openstack-neutron03:10
*** matsuhashi has quit IRC03:14
*** tomoe_ has quit IRC03:20
*** matsuhashi has joined #openstack-neutron03:20
*** gongysh has joined #openstack-neutron03:21
*** dvorkinista has quit IRC03:24
*** gongysh has quit IRC03:26
*** baoli has quit IRC03:29
*** baoli has joined #openstack-neutron03:30
*** matsuhashi has quit IRC03:30
*** matsuhashi has joined #openstack-neutron03:33
*** BuSerD has quit IRC03:34
*** baoli has quit IRC03:34
*** BuSerD has joined #openstack-neutron03:37
*** gongysh has joined #openstack-neutron03:39
*** matsuhashi has quit IRC03:42
*** gongysh has quit IRC03:44
*** morganfainberg_Z is now known as morganfainberg03:52
*** BuSerD has quit IRC03:52
*** gongysh has joined #openstack-neutron03:57
*** BuSerD has joined #openstack-neutron03:57
*** BuSerD has quit IRC03:58
*** sfox has joined #openstack-neutron04:01
*** suresh12 has joined #openstack-neutron04:01
*** kashyap has joined #openstack-neutron04:02
openstackgerritshihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API  https://review.openstack.org/7380204:02
*** suresh12 has quit IRC04:06
*** vkozhukalov_ has joined #openstack-neutron04:07
*** ramishra has joined #openstack-neutron04:07
*** ramishra_ has joined #openstack-neutron04:08
*** WackoRobie has quit IRC04:09
*** WackoRobie has joined #openstack-neutron04:10
*** ramishra has quit IRC04:12
*** WackoRobie has quit IRC04:14
*** gongysh has quit IRC04:22
*** mwagner_lap is now known as mwagner_zzz04:25
*** dvorkinista has joined #openstack-neutron04:26
*** _cjones_ has joined #openstack-neutron04:26
*** tomoe_ has joined #openstack-neutron04:29
*** chandan_kumar has joined #openstack-neutron04:31
*** _cjones_ has quit IRC04:31
*** linuxaddicts has quit IRC04:32
*** dvorkinista has quit IRC04:32
*** gongysh has joined #openstack-neutron04:34
openstackgerritA change was merged to openstack/neutron: Added missing l3_update call in update_network  https://review.openstack.org/8088704:37
*** WackoRobie has joined #openstack-neutron04:37
*** gongysh has quit IRC04:40
*** WackoRob_ has joined #openstack-neutron04:40
*** WackoRobie has quit IRC04:40
*** WackoRob_ has quit IRC04:42
*** WackoRobie has joined #openstack-neutron04:42
*** vkozhukalov_ has quit IRC04:43
*** WackoRobie has quit IRC04:43
*** WackoRobie has joined #openstack-neutron04:44
*** linuxaddicts has joined #openstack-neutron04:48
*** WackoRobie has quit IRC04:49
*** sweston has quit IRC04:52
*** gongysh has joined #openstack-neutron04:53
*** pradipta_away is now known as pradipta04:53
*** matsuhashi has joined #openstack-neutron04:55
*** gongysh has quit IRC04:59
*** tomoe_ has quit IRC05:01
*** linuxaddicts has quit IRC05:01
*** tomoe_ has joined #openstack-neutron05:04
*** singhs has joined #openstack-neutron05:10
openstackgerritKevin Benton proposed a change to openstack/neutron: Ignore PortNotFound exceptions on lockless delete  https://review.openstack.org/8102505:11
*** gongysh has joined #openstack-neutron05:12
*** singhs_ has joined #openstack-neutron05:12
*** singhs has quit IRC05:14
*** singhs_ is now known as singhs05:14
*** gongysh has quit IRC05:17
openstackgerritIan Wienand proposed a change to openstack/neutron: Record and log reason for dhcp agent resync  https://review.openstack.org/8117305:20
*** irenab has joined #openstack-neutron05:21
*** linuxaddicts has joined #openstack-neutron05:23
*** linuxaddicts has quit IRC05:26
*** yfried has quit IRC05:27
*** tomoe_ has quit IRC05:28
*** gongysh has joined #openstack-neutron05:30
*** gongysh has quit IRC05:35
*** networkstatic has joined #openstack-neutron05:35
*** dvorkinista has joined #openstack-neutron05:39
*** harlowja is now known as harlowja_away05:40
*** WackoRobie has joined #openstack-neutron05:43
*** linuxaddicts has joined #openstack-neutron05:43
*** ramishra_ has quit IRC05:46
*** WackoRobie has quit IRC05:48
*** gongysh has joined #openstack-neutron05:48
*** gongysh has quit IRC05:53
*** ramishra has joined #openstack-neutron05:53
*** sweston has joined #openstack-neutron06:01
*** Jabadia has joined #openstack-neutron06:04
*** gdubreui has quit IRC06:06
*** dvorkinista has quit IRC06:08
*** dvorkinista has joined #openstack-neutron06:08
*** dvorkinista has quit IRC06:12
*** rotbeard has joined #openstack-neutron06:14
*** gdubreui has joined #openstack-neutron06:18
marunlifeless: If you have a moment, I have a question regarding foreign keys and locking.06:18
lifelessshoot06:19
lifelessC is in bed now06:19
lifelessso I can talk06:19
*** vkozhukalov_ has joined #openstack-neutron06:22
marunlifeless: will locking a parent record have any impact on other transactions being able to insert child records?06:23
marunlifeless: i.e. if I 'select for update' the parent row as part of deletion, can I avoid having to worry about record insertion by other transactions in related tables?06:25
maruner, e.g.06:25
lifelessone of the transactions will definitely abort to the best of my knowledge06:26
lifelessI know more in the postgresql world about the specific details here06:26
lifelessSpamapS: / mordred: ^ mysql behaviour ?06:26
lifelessmarun: presuming you have on delete cascade, of course06:27
marunlifeless: no.  there lies the rub06:27
lifelessmarun: so codewise, I don't understand the problem you were describing before06:27
lifelessmarun: oh, no cascade delete? whyever not ?06:27
marunlifeless: well, there are a couple of reasons06:28
marunlifeless: but mainly part of the referential integrity is managed in code06:28
marunlifeless: or maybe I'm abusing the term06:28
lifelessmarun: thats usually a bad idea, but go on06:28
marunlifeless: erm, I'm struggling to pick a good example06:30
lifelessmarun: pick *an* example then :)06:30
*** matsuhashi has quit IRC06:30
marunlifeless: there isn't a delete-cascade on network <-> ports06:30
lifelesscan a port be owned by multiple networks ?06:31
marunlifeless: no06:31
marunlifeless: but there are different classes of ports06:31
lifelessso 1 network : M ports06:31
marunyes06:31
lifelessmarun: I don't understand why classes of ports affects the schema06:31
marunin the case of user-defined ports, they can be deleted no problem06:32
marunin the case of admin-defined ports, they can't be06:32
lifelessmarun: I mean, I understand why behavioural code will want different things stored, and possibly other related objects, but your serialisation technique shouldn't be tied in06:32
mordredlifeless: waht?06:32
lifelessmarun: can you expand on the admin port issue ?06:32
marunthis is why I'm struggling, I have the feeling I'm missing something.06:32
openstackgerritshihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API  https://review.openstack.org/7380206:32
*** thuc has joined #openstack-neutron06:33
lifelessmordred: mysql - table A and B, B has a FK pointing to A's PK. select for update a row in A.06:33
marunlifeless: bad example.  my brain is fried from looking at this for too long.06:33
mordredtoo early in morning.06:33
lifelessmordred: can other transactions add rows in B with the FK set to that row in A ? concurrently - and commit, without causing a rollback of the other transaction06:33
lifelessmordred: morning ?06:33
mordredlifeless: I'm in a weird timezone06:33
*** SridharG has joined #openstack-neutron06:33
lifelessmordred: orly, i'm SO surprised.06:34
marunlifeless: a better reason for not using delete cascade is that the ml2 plugin now has events surrounding create/update/delete of individual resources06:34
marunlifeless: maybe a better reason?06:34
marunlifeless: (if we can lock the parent and prevent addition during deletion, maybe not a good reason)06:35
marunlifeless: does any of that make sense?06:35
lifelessso, needing to raise events makes sense sure.06:36
lifelessI think the simplest thing, as I said before, is to make sure you've locked the range (FK=$id) on the table, which will prevent inserts06:36
*** thuc has quit IRC06:37
marunand will locking that range affect related table insertion?06:37
openstackgerritJenkins proposed a change to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/8118506:37
lifelessmarun: you lock that on the related tables06:38
marunso we have to hold locks on every related table?06:38
lifelessselect for update foo where fk_column=pk_of_parent_table06:38
lifelessmarun: yes, of course.06:38
marun:(06:38
maruntoo many f'ing tables06:38
mordredlifeless, marun: I've asked stewart to join us - he's smart06:39
openstackgerritA change was merged to openstack/neutron: Validate multicast ip range in Cisco N1kv Plugin  https://review.openstack.org/7529106:40
*** networkstatic has quit IRC06:41
*** networkstatic has joined #openstack-neutron06:41
marunlifeless: most of the relationships in the neutron db are actually managed by fk's with cascade deletion06:42
*** gongysh has joined #openstack-neutron06:42
SpamapSlifeless: o/06:43
*** WackoRobie has joined #openstack-neutron06:43
*** tomoe_ has joined #openstack-neutron06:43
marunSpamapS: so, is this a stupid line of inquiry?06:43
SpamapSno, let me see if I understand.06:43
*** saju_m has joined #openstack-neutron06:44
SpamapSselect for update foo where fk_column=pk_of_parent_table; ... the only lock here is on foo.06:44
SpamapSwhat's the next question?06:44
*** WackoRobie has quit IRC06:44
*** WackoRobie has joined #openstack-neutron06:45
marunso if I have table A with 10 tables with fk's for A06:45
*** saju_m has quit IRC06:45
marunand I want to delete row x in table A06:45
SpamapSwait, you have table A, with 10 tables?06:45
maruntable A + 10 tables that relate to A06:46
SpamapSAH ok06:46
*** saju_m has joined #openstack-neutron06:46
SpamapSRight you can't delete things in A unless all 10 FK's are deleted first.06:46
marunRight06:46
*** yfried has joined #openstack-neutron06:46
marunAnd I need to generate an event for many of the rows in related tables when they are removed06:47
marunevent -> python call06:47
SpamapSSo it sounds like you want to do a transaction, emit events, then commit it?06:48
marunThat's part of it.06:48
marunBut in the case of a deletion, what's the best strategy to ensure I emit events for all rows in related tables?06:48
*** gongysh has quit IRC06:49
SpamapSthe way I said it would work..06:49
*** matsuhashi has joined #openstack-neutron06:49
marunIf cascade deletion is used (and it is), rows added by other transactions would potentially be deleted without having events generated for them.06:49
*** WackoRobie has quit IRC06:49
SpamapSyou could also go a step further and use XA.. but that limits databases you can use, and also has _HORRIBLE_ performance... http://dev.mysql.com/doc/refman/5.6/en/xa.html06:49
SpamapSmarun: wait ok ignore the XA comment for a second.06:50
SpamapSmarun: if cascading deletion is used, you won't know what is deleted.06:50
marunexactly06:50
SpamapSmarun: so IMO, it isn't really an option if you must send events for all deletes.06:50
SpamapSmarun: you do, in fact, need to use select for update.. and then you'll have those rows locked (and the gaps between any ranges of rows in any single table locked as well)06:51
*** skraynev_afk is now known as skraynev06:51
SpamapSmarun: or, as lifeless was suggesting.. on pgsql, you won't lock the gaps, but some other transactions may fail if they would fall within the gaps06:52
marunyee06:52
SpamapSthey're retryable06:52
SpamapSbut you have to retry the whole thing, all the way back to the select for update06:52
marunso if events are necessary, either use cascade deletion and each related table needs to be locked via 'select for update', or cascade deletion should not be used and detection of insertion can be indicated by an integrity error?06:52
SpamapSyou lost me06:53
SpamapSIf you want to send events on deletion of rows, you have to use 'select for update' to get those ids, and in the same transaction, use delete to get rid of them.06:53
*** gdubreui has quit IRC06:53
openstackgerritberlin proposed a change to openstack/neutron: Fix duplicate name of NVP LBaaS objs not allowed on vShield Edge  https://review.openstack.org/7390506:53
marunso if events (e.g. resource deletion) are required06:54
maruncascade deletion doesn't indicate when records have been added, so locking via 'select for update' is required.06:54
SpamapSI don't understand what you mean.06:54
SpamapSWhat records will you be adding?06:54
marunfirst transaction - deleting parent resource and related resources06:54
marunsecond transaction - adding  related resources06:55
marunresources -> rows06:55
SpamapSalso if you do 'select for update * from table_Z where FK_on_A=999', then FD_on_A=999 is locked in table_Z .. nobody can add that record until you commit.06:55
marunright.06:55
SpamapSeven if it is not there06:55
SpamapSthe index slot is locked06:56
lifelessSpamapS: on postgresql stuff in the gaps won't be affected at all :)06:56
maruni'm worried at the number of tables that will need to be locked in some cases06:56
lifelessSpamapS: its better than that :>06:56
marunlifeless: gaps?06:56
SpamapSlifeless: right, in the scenario above, with pgsql, a second transaction will move forward, but if the first transaction hasn't committed yet, won't the second rollback?06:57
SpamapShm.. I bet only if there was a deadlock actually06:58
SpamapSlifeless: I know you're going to troll me at every turn with that.. but .. I'm going to troll you with "Facebook and Google disagree with you" back... so let's just put down our sabers on this one. :)06:58
lifelessSpamapS: why would it? unless the second transaction is for the locked range06:59
openstackgerritXu Han Peng proposed a change to openstack/neutron: Permit ICMPv6 RAs only from known routers  https://review.openstack.org/7225206:59
SpamapSright "liveness, blah blah, better optimizer... foo foo.."06:59
SpamapSlet's not get into that06:59
lifelessSpamapS: and since the isolation fixes, the locked range includes everything queries by the transaction, so all the rows with fk=xxx, but nothing adjacent to them06:59
SpamapSmarun: why are you worried about the number of tables that need to be locked? It is _rows_ you are locking, not tables.07:00
lifelessSpamapS: seriously not trying to troll07:00
marunSpamapS: actually, I don't think it's such an issue.  not all related tables require events.07:00
SpamapSlifeless: I'm sure you aren't, but I'm unusually grumpy today. :-P07:00
lifelessSpamapS: orly :P07:00
*** tomoe_ has quit IRC07:00
lifelessmarun: how many hundreds of tables ?07:00
marunIf we're using facebook usage as an indicator of quality, shouldn't we switch to php?07:00
lifelessmarun: (teasing slightly)07:00
marunlifeless: I'm kind of afraid of the neutron codebase.  it doesn't take 100s of tables to cause problems.07:01
lifelessmarun: more seriously, its a db round trip per lock select, which is sad, but if the schema has that many tables, that many locks are required no matter the strategy (unless you choose an eventual consistency strategy)07:01
*** gongysh has joined #openstack-neutron07:01
marunI think the current strategy is 'fake it till you make it'07:02
marunhttps://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L43207:02
lifelessmarun: if liveness is a particular concern, perhaps an eventual consistency strategy is what you should be refactoring too07:02
SpamapSAlso the gap locking thing is really not that big of a deal.07:02
marunso in this method, the strategy is to loop over the network and keep trying until there are no more related resources so the network can be deleted.07:03
lifelessif you've got 10's of thousands of ports, the gaps that are locked are small07:03
SpamapSIt matters _a lot_ in some cases, like mass deletes/updates.. but usually there are very few gaps locked for long at all.07:03
lifelessmarun: this method == eventual consistency ?07:03
oda-gHi core reviewers, would you pick up to review https://review.openstack.org/#/c/79858/ which needs one more +2 ?07:04
marunlifeless: not really, it just retries07:04
lifelessmarun: I was asking what you meant07:04
lifelessmarun: when yousaid 'in this method'07:04
marunlifeless: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L43207:04
lifelessoh the code07:04
SpamapSIt seems weird07:05
marunit is :)07:05
lifelessmarun: so thats basically what you'll need no matter what07:05
lifelessmarun: because concurrency in sql is expressed as integrityerror exceptions07:05
SpamapSok so yeah, select for update on the key to prevent another transaction interfering with this one is probably the right answer.07:05
marunlifeless: 'select for update' wouldn't work?07:06
SpamapSselect for update is going to prevent the case of a concurrent insert pwning the FK's..07:06
lifelessmarun: and either the end user or someone else has to deal with that07:06
SpamapSbut you may still have transactions roll back.07:06
lifelessmarun: select for update will fail if there is another transaction open that would prevent the update07:06
*** gongysh has quit IRC07:07
lifelessmarun: thats the whole point of it; it doesn't make the concurrency go away, it means you find out earlier07:07
marunso is there any point to using 'select for update'?07:07
lifelessyes, you find out earlier ;)07:08
lifelesswhich means you don't find out at the end, when you think you're all done, that really all that work was wasted because some numpty added a row07:08
lifelessbut let me describe an eventual consistency approach07:08
lifelesswe're talking network right, so I'll describe a hypothetical case there07:09
lifelessadd a new table network_ops_journal07:09
*** emagana has joined #openstack-neutron07:09
lifelessall *reads* that explicitly or implicitly depend on network should join through that table (left outer)07:09
lifelessif there's a row in network_ops_journal with the network PK + op of 'start_delete', they should treat the network as deleted regardless of the state of the network in the network table07:10
lifelessdelete_network becomes insert into network_ops_journal (network_pk, 'start_delete', timestamp)07:11
lifelessadd a background looping call to the server which looks for start_deletes without finish_deletes, and a timestamp more than <maximum transaction length> in the past07:11
maruni've worked on systems that had the idea of an in-table flag to indicate deletion07:12
lifeless[also, have the db set to kill transactions longer than that length...]07:12
SpamapSlifeless: this is starting to sound like an MVCC implementation.. :)07:12
lifelessthen, you *know* noone will be updating or adding to the network or any child tables subsequent to that point07:12
marunWhat's the advantage of having it in a separate table, other than separating out the deletion fields from the model?07:12
lifelessand your deleting become a lot simpler07:12
lifelessmarun: if you need to journal creates, its hard to do in the same table :)07:13
lifelessmarun: really depends on your liveness issues07:13
lifelessSpamapS: not really, journals in postgresql are useful too for liveness07:13
*** tomoe_ has joined #openstack-neutron07:13
*** D7 has joined #openstack-neutron07:13
marunHmmm07:13
lifelessmarun: but sure, you could do a soft_delete field and look for that07:13
*** Dseven has quit IRC07:13
lifelesspoint is that you can commit the delete immediately without doing schema contents cleanup07:14
marunright.  +1 for that for sure.07:14
lifelessand then in the background raise events and cleanup at your own pace07:14
SpamapSwhy +1 for that?07:14
SpamapSif DELETE of a resource is expensive..07:14
SpamapSthe API should simply respond with "OK I will delete that!"07:14
SpamapSand just do it in the DB07:14
maruner07:15
marunthat's not really the problem07:15
marunwe have issues with eventlet + mysql + async events causing deadlock because transactions hold locks for too long07:15
lifelessliveness ;)07:15
lifelesswhich generally means you're doing too much work in transactions07:15
SpamapSWhy are transactions being held open for a long time?07:15
marunyes07:16
lifelessor you are suffering lock contention07:16
marunthat too07:16
lifelessor both07:16
lifelesstoo much work can drive lock contention07:16
lifelessits a vicious circle07:16
SpamapSone thing about gap locking.. it affords MySQL instant deadlock detection.07:16
maruntoo much confusing and complex abstraction -> too much work in transactions07:16
maruncan you please explain gap locking?07:16
marunor point me at a resource that can?07:17
lifelessmarun: you're familiar with btrees07:17
marunyes07:17
marunwell, i should say i know what they are07:17
lifelessmarun: mysql locks the adjacency in the tree, not just the row selected07:17
marunof an index you mean?07:17
SpamapSmarun: gap locking is when MySQL locks the empty gaps between two locked rows. It only occurrs when you select/delete/update a range of rows, and it helps to make sure the range query result does not change during the transaction.07:17
lifelessmarun: which prevents inserts on either side with keys that are < or > than the index key in question, bounded by the next gkey up or down07:18
lifelessSpamapS: is it PK order only, or done on the indices?07:18
SpamapSto be clear, it does not happen on single key reads. There's no point there.07:18
lifelessI haven't actually read the implementation...07:18
SpamapSlifeless: it is always in index order.07:18
lifelessgotta run, crying baby07:18
SpamapSbe well07:19
marunlifeless: thanks for you help!07:19
*** gongysh has joined #openstack-neutron07:20
marunSpamapS: hmm, I wonder if gap locks have been causing us grief in the gate.07:20
SpamapSmarun: so one thing to do is to avoid doing large ranged index reads in transactions if you can.07:20
marunSpamapS: so narrow or single index reads are fine?07:21
SpamapSmarun: right, and there is an isolation level you can use which doesn't do it..07:21
marunSpamapS: READ COMMITTED?07:22
SpamapSmarun: http://www.mysqlperformanceblog.com/2012/08/28/differences-between-read-committed-and-repeatable-read-transaction-isolation-levels/07:22
SpamapSmarun: yeah I think thats the one07:22
*** networkstatic has quit IRC07:23
*** networkstatic has joined #openstack-neutron07:23
SpamapSmarun: "If you run an UPDATE that is not well indexed you will lock many rows"07:23
marunSpamapS: but is there any real advantage if indexes are used for most things?07:23
SpamapSlikely cause of problems :)07:23
SpamapSmarun: be 100% sure indexes are being used as you think they are :)07:24
*** tomoe_ has quit IRC07:24
marunfair enough07:24
SpamapSmarun: for the "delete a network" code, I think read committed would actually be fine.07:25
*** gongysh has quit IRC07:25
marunis that cross-db? i.e. does pg support the same thing?07:26
*** enikanorov__ has joined #openstack-neutron07:28
SpamapSmarun: http://www.postgresql.org/docs/9.1/static/transaction-iso.html07:30
*** enikanorov_ has quit IRC07:31
marunSpamapS: ok.07:33
*** ajo has joined #openstack-neutron07:33
SpamapSmarun: some things would fail horribly with read committed though, so YMMV :)07:33
marunSpamapS: and what would the advantage of moving to read committed for network deletion?07:33
marunSpamapS: deletion is handled by primary key for network, subnet and ports, so I'm not sure I see how it would help.07:34
marunSpamapS: :(07:35
marunhttps://review.openstack.org/#/c/78880/6/neutron/db/db_base_plugin_v2.py07:35
marunSpamapS: this patch just merged, and it removes any guarantee that port insertion can't happen during delete.07:36
*** tomoe_ has joined #openstack-neutron07:37
SpamapSmarun: ah. Wel then.. looping will just happen more often right? :-/07:37
marunSpamapS: except that this method doesn't have a wrapper that does the looping.07:38
*** gongysh has joined #openstack-neutron07:38
SpamapSmarun: OH!07:38
SpamapSmarun: I see now07:38
SpamapSmarun: no transaction..07:38
marunSpamapS: right07:38
*** ajo has quit IRC07:38
SpamapSso.. "this consistency thing is causing locking, remove it"07:39
marunSpamapS: We don't have enough people clueful of db-related stuff :/07:39
SpamapSbut..07:39
marunSpamapS: I'm a bit more clueful than most, but as you can see, not much more.07:39
SpamapSthis is like.. transactions 10107:39
*** ajo has joined #openstack-neutron07:39
marunYeah07:39
marunSpamapS: The troubling part is less that the patch is proposed - and by a core member no less - but that it gets approved without anyone pointing out the obvious.07:40
*** tomoiaga has joined #openstack-neutron07:40
SpamapSmarun: yeah.. major red flag07:40
marunSpamapS: I think we really need a db session at summit.07:40
SpamapSmarun: unless there is good cause to believe that it is o-k for ports to be added after the initial select.07:40
*** dkehn_ has quit IRC07:41
SpamapSmarun: but really.. this is a single index delete (network_id) ... so why would you _WANT_ to unblock other things trying to add ports to the network you're trying to delete.07:41
*** dkehn_ has joined #openstack-neutron07:41
SpamapSmarun: have to go to sleep now. But anyway, agreed that more awareness of how databases work is crucial for writing queries and updates. :)07:42
* SpamapS sleeps07:42
marunSpamapS: thanks for your help!07:42
*** WackoRobie has joined #openstack-neutron07:43
*** gongysh has quit IRC07:43
lifelessbtw07:46
lifelessone easy way to make adding ports cause locks / block on the network row would be to update the network row (e.g. tweak a timestamp()07:47
lifelessterrible for liveness, great if you want these broad locks07:47
lifeless but honestly, deletes are rare, optimise the other cases07:47
*** WackoRobie has quit IRC07:48
*** tomoe_ has quit IRC07:50
openstackgerritMaru Newby proposed a change to openstack/neutron: Stop removing ip allocations on port delete  https://review.openstack.org/8119607:52
*** amuller has joined #openstack-neutron07:56
*** gongysh has joined #openstack-neutron07:56
*** emagana has quit IRC07:58
*** ramishra has quit IRC08:00
*** gongysh has quit IRC08:01
*** luqas has joined #openstack-neutron08:08
*** dvorkinista has joined #openstack-neutron08:09
*** nati_ueno has joined #openstack-neutron08:11
*** ramishra has joined #openstack-neutron08:13
*** dvorkinista has quit IRC08:13
*** gongysh has joined #openstack-neutron08:14
*** gongysh has quit IRC08:19
*** dave_tucker_zzz is now known as dave_tucker08:20
marunlifeless: deletes are rare, except in the gate :p08:21
*** ramishra has quit IRC08:23
openstackgerritAssaf Muller proposed a change to openstack/neutron: Change report_interval from 4 to 30, agent_down_time from 9 to 75  https://review.openstack.org/8082908:25
*** jistr has joined #openstack-neutron08:26
*** arosen-home has quit IRC08:26
*** jistr is now known as jistr|mtg08:27
*** carl_baldwin has joined #openstack-neutron08:28
lifelessmarun: even so :)08:30
*** ramishra has joined #openstack-neutron08:31
*** _cjones_ has joined #openstack-neutron08:32
*** gongysh has joined #openstack-neutron08:32
*** morganfainberg is now known as morganfainberg_Z08:34
openstackgerritshihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API  https://review.openstack.org/7380208:35
*** gongysh has quit IRC08:37
*** _cjones_ has quit IRC08:37
*** sfox has quit IRC08:39
*** WackoRobie has joined #openstack-neutron08:43
*** jgallard has joined #openstack-neutron08:44
*** ramishra has quit IRC08:47
*** jp_at_hp has joined #openstack-neutron08:47
*** WackoRobie has quit IRC08:47
*** jlibosva has joined #openstack-neutron08:48
*** djoreilly has joined #openstack-neutron08:49
*** saju_m has quit IRC08:50
*** rossella_s has joined #openstack-neutron08:50
*** gongysh has joined #openstack-neutron08:50
*** ygbo has joined #openstack-neutron08:54
*** saju_m has joined #openstack-neutron08:56
*** gongysh has quit IRC08:56
openstackgerritberlin proposed a change to openstack/neutron: Add Error Handling to NVP advanced LBaaS/FWaaS  https://review.openstack.org/5962508:57
*** leseb has joined #openstack-neutron09:02
*** ramishra has joined #openstack-neutron09:03
*** gongysh has joined #openstack-neutron09:09
*** dvorkinista has joined #openstack-neutron09:10
openstackgerritRoman Podoliaka proposed a change to openstack/neutron: WIP: sync db code  https://review.openstack.org/7720509:12
*** tomoiaga has left #openstack-neutron09:12
*** nati_ueno has quit IRC09:13
*** dvorkinista has quit IRC09:14
*** gongysh has quit IRC09:14
*** pradipta is now known as pradipta_away09:15
*** singhs has quit IRC09:16
*** rotbeard has quit IRC09:16
*** leseb has quit IRC09:16
*** leseb has joined #openstack-neutron09:17
*** pradipta_away is now known as pradipta09:18
*** gdubreui has joined #openstack-neutron09:19
*** leseb has quit IRC09:21
*** leseb has joined #openstack-neutron09:24
openstackgerritPhil Day proposed a change to openstack/python-neutronclient: Allow user ID for authentication  https://review.openstack.org/7859609:25
*** gongysh has joined #openstack-neutron09:27
*** tomoe_ has joined #openstack-neutron09:30
*** tomoe_ has quit IRC09:31
*** tomoe_ has joined #openstack-neutron09:31
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator  https://review.openstack.org/8121109:32
*** afazekas_ has joined #openstack-neutron09:34
*** rotbeard has joined #openstack-neutron09:35
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator  https://review.openstack.org/8121109:37
*** WackoRobie has joined #openstack-neutron09:43
*** bada has joined #openstack-neutron09:44
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: NSX plugin: return 400 for invalid gw certificate  https://review.openstack.org/8094809:47
*** WackoRobie has quit IRC09:48
*** sphoorti has joined #openstack-neutron09:56
*** amuller has quit IRC09:56
openstackgerritshihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API  https://review.openstack.org/7380209:57
openstackgerritSphoorti proposed a change to openstack/neutron: Add unit test for add_vxlan in test_linux_ip_lib  https://review.openstack.org/8055410:04
*** dvorkinista has joined #openstack-neutron10:10
*** dvorkinista has quit IRC10:16
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator  https://review.openstack.org/8121110:16
*** sweston has quit IRC10:16
*** nati_ueno has joined #openstack-neutron10:20
*** pradipta is now known as pradipta_away10:25
*** matsuhashi has quit IRC10:25
*** pcm_ has joined #openstack-neutron10:27
*** networkstatic is now known as networkstatic_zZ10:29
*** bvandenh has joined #openstack-neutron10:31
*** pcm_ has quit IRC10:31
*** pcm_ has joined #openstack-neutron10:32
*** nati_ueno has quit IRC10:33
openstackgerritZang MingJie proposed a change to openstack/neutron: db migration failed when plugin name alias used in config file  https://review.openstack.org/8122110:43
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: Fix enums usage for postgres in migrations  https://review.openstack.org/5311210:43
*** WackoRobie has joined #openstack-neutron10:43
openstackgerritRobert Kukura proposed a change to openstack/neutron: ML2: Remove validate_port_binding() and unbind_port()  https://review.openstack.org/7951110:44
*** Jabadia_ has joined #openstack-neutron10:47
*** Jabadia has quit IRC10:47
*** WackoRobie has quit IRC10:48
*** baoli has joined #openstack-neutron10:48
*** baoli has quit IRC10:48
*** baoli has joined #openstack-neutron10:49
*** yamahata has quit IRC10:49
*** jistr|mtg has quit IRC11:05
*** dvorkinista has joined #openstack-neutron11:12
*** tomoe_ has quit IRC11:13
*** rwsu has quit IRC11:13
*** jgallard has quit IRC11:13
*** amuller has joined #openstack-neutron11:14
*** rwsu has joined #openstack-neutron11:14
*** tomoe_ has joined #openstack-neutron11:14
*** enikanorov has quit IRC11:14
*** enikanorov has joined #openstack-neutron11:16
*** dvorkinista has quit IRC11:16
*** sphoorti_ has joined #openstack-neutron11:26
*** gdubreui has quit IRC11:31
*** mwagner_zzz has quit IRC11:35
*** xianghui has quit IRC11:39
*** leseb has quit IRC11:39
*** leseb has joined #openstack-neutron11:40
*** leseb_ has joined #openstack-neutron11:41
*** leseb has quit IRC11:45
*** sphoorti_ has quit IRC11:46
*** yamahata has joined #openstack-neutron11:56
*** sphoorti has quit IRC11:58
*** tomoe_ has quit IRC12:00
*** ramishra has quit IRC12:01
openstackgerritOleg Bondarev proposed a change to openstack/neutron: LBaaS: remove orphan haproxy instances on agent start  https://review.openstack.org/8124212:03
*** ramishra has joined #openstack-neutron12:03
*** obondarev1 has joined #openstack-neutron12:06
*** baoli has quit IRC12:07
*** jp_at_hp has quit IRC12:07
*** baoli has joined #openstack-neutron12:08
*** yamahata_ has quit IRC12:08
*** jp_at_hp has joined #openstack-neutron12:08
*** yamahata_ has joined #openstack-neutron12:08
*** obondarev has quit IRC12:09
*** eezhova19 has quit IRC12:09
*** nplanel has quit IRC12:09
*** eezhova19 has joined #openstack-neutron12:09
*** ajo has quit IRC12:09
*** jgrimm has quit IRC12:09
*** bada has quit IRC12:10
*** bada has joined #openstack-neutron12:11
*** dvorkinista has joined #openstack-neutron12:12
*** dvorkinista has quit IRC12:17
*** afazekas_ has quit IRC12:18
*** nplanel has joined #openstack-neutron12:20
*** jgrimm has joined #openstack-neutron12:20
*** ajo has joined #openstack-neutron12:21
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup.  https://review.openstack.org/8026112:25
*** luqas has quit IRC12:26
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup.  https://review.openstack.org/8026112:30
*** vkozhukalov_ has quit IRC12:30
*** afazekas has joined #openstack-neutron12:31
*** dims has quit IRC12:31
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS support for VPN service admin state change and reporting  https://review.openstack.org/8112412:31
*** afazekas has quit IRC12:32
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup.  https://review.openstack.org/8026112:33
*** dims has joined #openstack-neutron12:33
openstackgerritOleg Bondarev proposed a change to openstack/neutron: LBaaS: small cleanup in agent device driver interface  https://review.openstack.org/8124812:34
*** gongysh has quit IRC12:35
*** bada_ has joined #openstack-neutron12:35
ajoobondarev1,  ^12:36
*** gongysh has joined #openstack-neutron12:36
ajodoesn't it require the same change in the load balancer implementations?12:36
obondarev1ajo: which implementations?12:37
ajoobondarev1, I don't actually know that part :)12:37
obondarev1ajo: do you mean drivers?12:37
ajobut, as they were abstract methods, I suppose there must be drivers/actual implementations12:37
obondarev1ajo: ok, currently this change affects only haproxy namespace driver as it is the only implemented agent device driver12:38
obondarev1ajo: in haproxy driver these methods are overriden without 'context' param12:38
ajoaha, and the change doesn't need to remove the 'context' also form the haproxy implementation?12:38
ajoahhh12:38
ajook12:38
ajounderstood :)12:38
ajoform->from12:39
*** bada has quit IRC12:39
obondarev1ajo: so it was kind od typo in abstract driver12:39
ajoit's always good to cleanup :)12:39
obondarev1yeah)12:40
*** xuhanp has joined #openstack-neutron12:42
*** salv-orlando has quit IRC12:43
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: WIP Implement test  https://review.openstack.org/7652012:43
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: WIP Add testing for database from oslo  https://review.openstack.org/7651912:43
*** carl_baldwin_ has joined #openstack-neutron12:43
*** carl_baldwin has quit IRC12:45
*** carl_baldwin_ is now known as carl_baldwin12:45
*** amotoki has joined #openstack-neutron12:47
*** vkozhukalov_ has joined #openstack-neutron12:50
*** markmcclain has joined #openstack-neutron12:55
*** jecarey has joined #openstack-neutron12:57
*** yfried1 has joined #openstack-neutron12:58
*** yfried has quit IRC12:59
*** heyongli has joined #openstack-neutron12:59
*** thuc has joined #openstack-neutron12:59
*** roeyc has joined #openstack-neutron12:59
*** thuc_ has joined #openstack-neutron13:00
*** jgallard has joined #openstack-neutron13:02
*** thuc has quit IRC13:04
*** yfried has joined #openstack-neutron13:11
*** yfried1 has quit IRC13:13
*** dvorkinista has joined #openstack-neutron13:13
*** changbl has quit IRC13:14
*** julim has joined #openstack-neutron13:14
*** ramishra has quit IRC13:14
*** salv-orlando has joined #openstack-neutron13:15
obondarev1markmcclain: ping13:16
*** yfried1 has joined #openstack-neutron13:16
*** obondarev1 is now known as obondarev13:18
*** yfried has quit IRC13:19
*** dvorkinista has quit IRC13:20
markmcclainobondarev: pong13:20
obondarevmarkmcclain: a question regarding https://review.openstack.org/#/c/66899/13:20
obondarevmarkmcclain: is it deferred for Juno?13:21
markmcclainyes… I'll update the comment on that review13:21
obondarevmarkmcclain: gotcha13:22
*** peristeri has joined #openstack-neutron13:29
amotokimarkmcclain: ping13:35
markmcclainamotoki: pong13:36
amotokisorry for my absense in the weekly meeting. I am trying to sync oslo code now. It is good to sync it before icehouse release.13:36
amotokimarkmcclain: what do you think about oslo sync.13:36
amotokimarkmcclain: there is a change in oslo db.api and we need some changes in neutron.db code.13:37
markmcclainamotoki: we'll have to target only limited portions to sync13:37
markmcclainwe had change sneak into the neutron.openstack.common rpc that prevents us from sync'ing the rpc tree13:38
amotokimarkmcclain: I got it. I will investigate each of the changes in oslo and will propose sync if needed.13:39
markmcclainsounds good… I think db is the only real portion of the tree that has not been sync'ed recently13:40
*** jecarey has quit IRC13:41
amotokimarkmcclain: I think so too. db code in neutron openstack.common is out-of-date for a long time and i think it is better to sync it for better maintainability for stable release,13:41
amotoki but we need to investigate it carefully.13:42
*** sbadia has quit IRC13:42
markmcclainyeah.. the late merge does concern me, so let me know if anything stands out as risy13:42
markmcclain*risky13:42
openstackgerritOleg Bondarev proposed a change to openstack/neutron: LBaaS: remove orphan haproxy instances on agent start  https://review.openstack.org/8124213:43
*** mwagner_lap has joined #openstack-neutron13:43
amotokimarkmcclain: sure.13:43
*** aveiga has joined #openstack-neutron13:44
*** yfried1 has quit IRC13:45
*** yfried has joined #openstack-neutron13:50
*** dvorkinista has joined #openstack-neutron13:50
*** sbadia has joined #openstack-neutron13:51
*** leseb_ has quit IRC13:51
*** leseb has joined #openstack-neutron13:51
*** Jabadia_ has quit IRC13:53
*** Jabadia has joined #openstack-neutron13:53
*** jp_at_hp has quit IRC13:53
*** afazekas has joined #openstack-neutron13:54
*** tomoe_ has joined #openstack-neutron13:56
*** sbadia has quit IRC13:56
*** thuc_ has quit IRC13:57
*** carl_baldwin has quit IRC13:57
*** thuc has joined #openstack-neutron13:57
*** jp_at_hp has joined #openstack-neutron13:59
*** banix has joined #openstack-neutron13:59
*** sbadia has joined #openstack-neutron13:59
*** otherwiseguy has joined #openstack-neutron14:00
*** heyongli has quit IRC14:00
*** thuc has quit IRC14:02
*** shshang has joined #openstack-neutron14:02
*** luqas has joined #openstack-neutron14:03
*** leseb has quit IRC14:04
*** leseb has joined #openstack-neutron14:04
*** salv-orlando has quit IRC14:08
openstackgerritDane LeBlanc proposed a change to openstack/neutron: Cisco Nexus: maximum recursion error in ConnectionContext.__del__  https://review.openstack.org/8073714:10
*** jobewan has joined #openstack-neutron14:10
*** vkozhukalov_ has quit IRC14:12
*** amuller has quit IRC14:12
*** dvorkinista has quit IRC14:14
openstackgerritZang MingJie proposed a change to openstack/neutron: Ensure host_routes order in subnet  https://review.openstack.org/7894614:16
*** amuller has joined #openstack-neutron14:19
*** alexpilotti has joined #openstack-neutron14:21
*** thuc has joined #openstack-neutron14:22
*** overlayer has joined #openstack-neutron14:23
*** thuc has quit IRC14:23
*** thuc has joined #openstack-neutron14:24
*** vkozhukalov_ has joined #openstack-neutron14:25
*** dguitarbite has quit IRC14:31
*** thuc has quit IRC14:34
*** thuc has joined #openstack-neutron14:34
*** dguitarbite has joined #openstack-neutron14:36
*** jecarey has joined #openstack-neutron14:38
*** thuc has quit IRC14:39
*** thedodd has joined #openstack-neutron14:41
*** gongysh has quit IRC14:42
*** otherwiseguy has quit IRC14:44
*** carl_baldwin has joined #openstack-neutron14:49
openstackgerritBerezovsky Irena proposed a change to openstack/neutron: Add update binding:profile with physicla _network  https://review.openstack.org/8128114:51
*** coolsvap has joined #openstack-neutron14:51
*** carl_baldwin has quit IRC14:52
*** irenab has quit IRC14:53
*** otherwiseguy has joined #openstack-neutron14:56
*** alagalah has joined #openstack-neutron14:59
*** thuc has joined #openstack-neutron14:59
*** afazekas has quit IRC15:00
*** shshang has quit IRC15:00
sc68calxuhanp: I'll e-mail you a draft15:00
sc68calbefore I post it to the ML, since I'm sort of talking about your patch15:01
sc68calbut I didn't write it :)15:01
*** skraynev is now known as skraynev_afk15:01
*** alagalah has quit IRC15:01
*** thuc has quit IRC15:01
*** alagalah has joined #openstack-neutron15:01
*** thuc has joined #openstack-neutron15:02
*** chandan_kumar has quit IRC15:02
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup  https://review.openstack.org/8026115:03
*** chandan_kumar has joined #openstack-neutron15:04
*** cgoncalves has quit IRC15:07
*** cgoncalves has joined #openstack-neutron15:07
*** networkstatic_zZ is now known as networkstatic15:09
JoeJulianI changed a peer_gateway address through ipsec-site-connection-update but that change is not reflected in ipsec status. Is that expected behavior?15:09
*** changbl has joined #openstack-neutron15:12
*** orion195 has joined #openstack-neutron15:13
orion195Hi guys, on a newly installed openstack, I get this: ERROR 1146 (42S02) at line 1: Table 'neutron.migrate_version' doesn't exist15:14
orion195while running openstack-db --init --service neutron15:14
orion195installation done using the openstack manua, no packstack15:15
*** thuc has quit IRC15:15
sc68calbaoli: When you did that neutron subnet and router create15:19
sc68calwas that before the force_gateway_on_subnet flag was introduced?15:19
*** Jabadia has quit IRC15:20
*** Jabadia has joined #openstack-neutron15:20
*** vkozhukalov_ has quit IRC15:22
*** networkstatic has quit IRC15:22
*** saju_m has quit IRC15:23
xuhanpsc68cal, thanks for letting me know about the email. I will have a look15:23
*** Jabadia has quit IRC15:25
*** xuhanp has quit IRC15:25
ram5391Hi, where does the "keystone uri" field get configured for neutronclient.client? The value is wrong, and I can't find it in api-paste.ini or neutron.conf15:26
*** networkstatic has joined #openstack-neutron15:26
markmcclainorion195: openstack-db is not used by Neutron15:30
markmcclainram5391: depends where is the neutronclient being invoked?15:31
*** vkozhukalov_ has joined #openstack-neutron15:33
ram5391markmcclain: while on the quantum machine, if I issue a "neutron -v agent-list" I get all valid responses until the last one, which looks for the keystone uri at https://127.0.0.1. Which keystone is on a different machine, and not operating via https15:33
*** markwash has joined #openstack-neutron15:36
*** devlaps has joined #openstack-neutron15:37
*** yamahata has quit IRC15:39
*** yamahata has joined #openstack-neutron15:39
markmcclainram5391 not sure I'm understanding15:39
markmcclainlast response is what?15:40
*** saju_m has joined #openstack-neutron15:40
*** yfried has quit IRC15:43
openstackgerritSean M. Collins proposed a change to openstack/neutron: Advanced Services documentation  https://review.openstack.org/8082515:45
ram5391markmcclain: sorry, I'm in the process of rebuilding all my relationships. That may fix the problem15:47
*** overlayer has quit IRC15:47
*** overlayer has joined #openstack-neutron15:47
*** markwash has quit IRC15:51
*** thuc has joined #openstack-neutron15:56
*** rupsky has joined #openstack-neutron15:56
*** saju_m has quit IRC15:57
*** thuc_ has joined #openstack-neutron15:57
*** rupsky has quit IRC15:58
*** sfox has joined #openstack-neutron15:59
*** jorgem has joined #openstack-neutron16:00
*** cgoncalves has quit IRC16:00
*** thuc has quit IRC16:01
*** cgoncalves has joined #openstack-neutron16:02
*** cgoncalves has quit IRC16:03
*** cgoncalves has joined #openstack-neutron16:04
*** krtaylor has quit IRC16:04
*** sbadia has quit IRC16:05
*** dvorkinista has joined #openstack-neutron16:05
*** blogan has joined #openstack-neutron16:07
*** sbalukoff has quit IRC16:08
*** SumitNaiksatam has quit IRC16:11
*** _cjones_ has joined #openstack-neutron16:13
*** sfox has quit IRC16:16
*** arosen1 has joined #openstack-neutron16:17
*** spandhe has joined #openstack-neutron16:20
*** sweston has joined #openstack-neutron16:22
*** sfox has joined #openstack-neutron16:23
*** vkozhukalov_ has quit IRC16:25
sdagueso what's up with this in every single neutron log fie?16:29
sdague2014-03-18 13:59:27.701 3242 ERROR neutron.common.legacy [-] Skipping unknown group key: firewall_driver16:29
sdagueis that something that devstack is setting incorrectly?16:29
*** dvorkinista has quit IRC16:31
*** chandankumar_ has joined #openstack-neutron16:34
*** chandan_kumar has quit IRC16:37
*** pete5 has quit IRC16:38
anteayahttps://review.openstack.org/#/c/80989/ is failing in the gate16:42
*** dguitarbite has quit IRC16:42
openstackgerritBerezovsky Irena proposed a change to openstack/neutron: Add update binding:profile with physical_network  https://review.openstack.org/8128116:42
*** ijw has joined #openstack-neutron16:43
*** networkstatic has quit IRC16:44
*** networkstatic has joined #openstack-neutron16:44
*** ijw has quit IRC16:45
*** bada__ has joined #openstack-neutron16:45
*** ijw has joined #openstack-neutron16:46
*** bada_ has quit IRC16:48
*** overlayer has quit IRC16:49
*** overlayer has joined #openstack-neutron16:51
openstackgerritDane LeBlanc proposed a change to openstack/neutron: When the Cisco nexus plugin is configured on DevStack, the tempest Neutron API test test_port_list_filter_by_router_id fails with the following error:      ParseError: no element found: line 1, column 0  https://review.openstack.org/8130416:51
*** spandhe has quit IRC16:52
*** harlowja_away is now known as harlowja16:53
openstackgerritÉdouard Thuleau proposed a change to openstack/neutron: OVS lib defer apply doesn't handle concurrency  https://review.openstack.org/6391716:54
openstackgerritYves-Gwenael Bourhis proposed a change to openstack/neutron: Make dnsmasq aware of all names  https://review.openstack.org/5293016:54
*** arosen1 has quit IRC16:55
*** spandhe has joined #openstack-neutron16:56
*** arosen1 has joined #openstack-neutron16:56
*** markwash has joined #openstack-neutron16:57
*** overlayer has quit IRC16:58
*** overlayer has joined #openstack-neutron16:59
*** markwash has quit IRC17:01
*** morganfainberg_Z is now known as morganfainberg17:01
openstackgerritDane LeBlanc proposed a change to openstack/neutron: Cisco plugin fails test_port_list_filter_by_router_id with ParseError no element found  https://review.openstack.org/8130417:02
*** SumitNaiksatam has joined #openstack-neutron17:03
*** markwash has joined #openstack-neutron17:07
openstackgerritDarragh O'Reilly proposed a change to openstack/neutron: lb-agent: reduce delete_vlan_bridge() logging severity  https://review.openstack.org/7599317:12
pba1Hello, I'm trying to figure out how plugins/extensions work in neutron, so I read NeutronDevelopment/API docs and started poking code. I've made some small changes in one of the extensions (neutron/extensions/firwall.py (it works (unchanged version))) to see if I'm poking in the right place but it seams this file wasn't read after my change (.pyc file is older) so i guess I need to17:13
pba1"reinstall/reload" extensions somehow but I don't know how to do it. I'm using devstack and ./unstack && ./stack insn't working. Sorry for asking about such simple thing but i can't find any information about it on wiki.17:13
*** tomoe_ has quit IRC17:16
*** amotoki has quit IRC17:17
*** otherwiseguy has quit IRC17:17
*** networkstatic has quit IRC17:17
_cjones_pba1: You shouldn't need to restack. Just restart the Neutron service.17:18
pba1but even restacking isn't working17:20
*** roeyc has quit IRC17:21
*** alagalah_ has joined #openstack-neutron17:22
*** alagalah has quit IRC17:22
*** rotbeard has quit IRC17:22
*** devlaps1 has joined #openstack-neutron17:23
_cjones_Hey guys, I'm occasionally observing this error while developing in Neutron: 2014-03-18 10:16:55.390 TRACE neutron.openstack.common.rpc.amqp TimeoutError: QueuePool limit of size 10 overflow 20 reached, connection timed out, timeout 1017:23
*** devlaps has quit IRC17:24
_cjones_pba1: It should. Perhaps that code isn't being executed.17:24
_cjones_pba1: Bust out the debugger.17:24
pba1I don't know if I understand corectly if I change get_resources method (RESOURCE_ATTRIBUTE_MAP to be exact) in extension it should infuance neutron API right ?17:24
*** arosen1 has quit IRC17:24
_cjones_pba1: I'm not that intimately familiar. I'd have to look.17:25
pba1or should I change also sowhere else17:25
_cjones_pba1: You could also delete the pyc file. If that code isn't run, no new pyc will be genenrated. I suspect you're in this case if you dont' notice any changes.17:26
pba1i did ;p17:26
pba1all of them ^^17:26
pba1and new are not beeing generated17:26
pba1and everytinhg works17:27
_cjones_pba1: If they're not being generated, they're not being executed.17:27
pba1yes, so when exactly extensions are executed17:27
pba1should I load them manualy after changing them ?17:28
pba1somehow17:28
*** sbalukoff has joined #openstack-neutron17:28
*** leseb has quit IRC17:30
*** leseb has joined #openstack-neutron17:30
*** thedodd has quit IRC17:31
*** kashyap has left #openstack-neutron17:32
cgoncalvesSumitNaiksatam: hi! could you clarify me on what the "consumed_contracts" attribute on the EPG model (group policy) is used for please? why do we need such attribute?17:33
*** bada has joined #openstack-neutron17:33
cgoncalvesSumitNaiksatam: I can comment on the presentation doc if you prefer17:33
*** leseb has quit IRC17:34
SumitNaiksatamcgoncalves: hi17:34
SumitNaiksatamcgoncalves: am in a meeting17:34
*** bada__ has quit IRC17:35
*** aveiga has quit IRC17:35
SumitNaiksatamcgoncalves: good question17:36
*** ram5391 has quit IRC17:37
cgoncalvesSumitNaiksatam: oh, sorry. we can chat a bit later or bring this question to the presentation doc17:37
*** arosen1 has joined #openstack-neutron17:37
*** arosen1 is now known as arosen-home17:37
SumitNaiksatamcgoncalves: yeah sure17:37
*** morganfainberg is now known as morganfainberg_Z17:38
SumitNaiksatamcgoncalves: give me about 90 mins17:38
cgoncalvesSumitNaiksatam: ok, thanks17:38
*** geekinutah has quit IRC17:38
*** aveiga has joined #openstack-neutron17:39
*** geekinutah has joined #openstack-neutron17:40
*** otherwiseguy has joined #openstack-neutron17:43
jlibosvamarkmcclain: hi, do you have a minute to discuss https://review.openstack.org/#/c/75924/ ?17:44
*** ygbo has quit IRC17:44
*** crc32 has joined #openstack-neutron17:50
*** sbadia has joined #openstack-neutron17:55
*** jp_at_hp has quit IRC17:56
*** morganfainberg_Z is now known as morganfainberg17:58
*** aveiga has quit IRC18:00
*** harlowja is now known as harlowja_away18:01
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: Cancelling thread start while unit tests running.  https://review.openstack.org/8132318:01
*** harlowja_away is now known as harlowja18:02
*** devvesa has joined #openstack-neutron18:03
sballeI am looking for information on the Provider Network. I did a search and found bits and pieces. Can somebody point me at a good place to look for informaiton on this feature?18:04
*** shakamunyi has joined #openstack-neutron18:04
geekinutahsballe: join #openstack and I'll help you in there18:06
sballeok thx18:06
openstackgerritA change was merged to openstack/neutron: NSX: Ensure gateway devices are usable after upgrade  https://review.openstack.org/8098918:07
*** ijw has quit IRC18:07
*** aveiga has joined #openstack-neutron18:10
*** aveiga has quit IRC18:10
markmcclainjlibosva: yes18:15
*** suresh12 has joined #openstack-neutron18:15
jlibosvamarkmcclain: oh, I'm on a meeting right now, can I ping you later? (50 mins)18:16
markmcclainsure18:17
jlibosvathx18:17
*** shakamunyi has quit IRC18:18
*** devvesa has quit IRC18:23
*** D7 has quit IRC18:24
*** shakamunyi has joined #openstack-neutron18:27
*** amuller has quit IRC18:28
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup  https://review.openstack.org/8026118:28
*** ijw has joined #openstack-neutron18:30
*** jlibosva has quit IRC18:31
*** yfried1 has joined #openstack-neutron18:31
*** _sweston_ has joined #openstack-neutron18:31
*** jgallard has quit IRC18:33
*** sweston has quit IRC18:33
*** alagalah_ has quit IRC18:34
*** ijw has quit IRC18:34
*** ijw has joined #openstack-neutron18:36
*** _sweston_ has quit IRC18:36
*** luqas has quit IRC18:37
openstackgerritJohn Eckersberg proposed a change to openstack/neutron: Use InnoDB storage engine when creating tables in migrations  https://review.openstack.org/8133418:42
*** thuc_ has quit IRC18:43
*** thuc has joined #openstack-neutron18:44
*** BuSerD has joined #openstack-neutron18:44
*** thuc_ has joined #openstack-neutron18:45
*** networkstatic has joined #openstack-neutron18:45
*** overlayer has quit IRC18:47
*** vkozhukalov_ has joined #openstack-neutron18:48
*** thuc has quit IRC18:48
*** shakamunyi has quit IRC18:49
*** spandhe has quit IRC18:52
*** spandhe has joined #openstack-neutron18:52
*** alagalah has joined #openstack-neutron18:52
*** jecarey has quit IRC18:55
*** jecarey has joined #openstack-neutron18:55
*** jlibosva has joined #openstack-neutron18:59
sc68calpba1: what plugin are you running19:00
sc68calthere should be a log line for each extension that the plugin loads19:00
sc68calwhen neutron server starts19:00
sc68calso look for a line that says the plugin loaded the extension you are modifying19:00
*** gdubreui has joined #openstack-neutron19:01
*** krtaylor has joined #openstack-neutron19:03
*** otherwiseguy has quit IRC19:11
*** carl_baldwin has joined #openstack-neutron19:14
*** arosen-home has quit IRC19:18
*** saju_m has joined #openstack-neutron19:18
*** arosen1 has joined #openstack-neutron19:18
*** djoreilly has quit IRC19:21
*** overlayer has joined #openstack-neutron19:24
*** zzelle has joined #openstack-neutron19:25
*** _sweston_ has joined #openstack-neutron19:30
*** spandhe has quit IRC19:34
*** samuelbercovici has joined #openstack-neutron19:34
*** spandhe has joined #openstack-neutron19:35
*** alagalah has quit IRC19:37
bloganmarkmcclain: when you have time can you look at https://review.openstack.org/#/c/80816/ its a really simple one19:39
*** salv-orlando has joined #openstack-neutron19:42
*** nati_ueno has joined #openstack-neutron19:43
*** dguitarbite has joined #openstack-neutron19:52
*** julim has quit IRC20:02
*** salv-orlando has quit IRC20:03
*** sfox has quit IRC20:06
*** ianw has joined #openstack-neutron20:11
ianwobondarev: re https://review.openstack.org/#/c/81173/ do you mean file a bug describing the debug output?20:12
openstackgerritDane LeBlanc proposed a change to openstack/neutron: Cisco plugin fails with ParseError no elem found  https://review.openstack.org/8130420:13
openstackgerritKevin Benton proposed a change to openstack/neutron: ML2 BigSwitch: Don't modify parent context  https://review.openstack.org/8136020:14
*** dvorkinista has joined #openstack-neutron20:16
*** salv-orlando has joined #openstack-neutron20:16
*** leseb has joined #openstack-neutron20:16
*** peristeri has quit IRC20:16
*** jlibosva has quit IRC20:20
*** singhs has joined #openstack-neutron20:31
marunrkukura: ping20:34
rkukuramarun: hi20:34
marunrkukura: hi Bob20:34
marunrkukura: I wanted to make sure I understood some of your comments re: migration script20:35
rkukuramarun: OK20:35
marunrkukura: so re: port bindings, migrate port id and host, and set vif type to UNBOUND?20:36
marunrkukura: I think that's my only question actually. Good catch on not allowing for ovs configured for vxlan tunneling, I was totally blind to that.20:37
rkukuramarun:  That may be the way to go, provided that the patch I'm working on gets merged.20:37
marunrkukura: what to do if it doesn't?20:37
marunrkukura: or should the migration patch also not merge?20:37
rkukuraIf we don't get my patch in, the migration tool would need to figure out the "right" values for vif_type (easy), for driver (also easy), for vif_details (easy now but changing with Nachi's VIF security patch), and segment (maybe harder)20:39
marunrkukura: erg20:39
marunmarkmcclain: ^^20:39
*** alagalah has joined #openstack-neutron20:40
rkukuramarun: Its just plain cleaner to let ML2 and its drivers do the binding than to try to get the migration tool to do it.20:41
*** alagalah has quit IRC20:41
marunrkukura: fair enough, I'll trust you on that.20:41
marunrkukura: is there a reason that your patch won't land in icehouse?20:42
*** marun has quit IRC20:42
rkukuraI don't see any reason, other than running out of time.20:42
*** marun has joined #openstack-neutron20:45
*** dave_tucker is now known as dave_tucker_zzz20:46
marunrkukura: sorry, lost my connection20:46
rkukuramarun: Just replied that I don't see any reason my fix won't make it, unless we run out of time.20:46
marunrkukura: ok.20:47
rkukuraTrying to complete it today.20:47
marunrkukura: so I should rebase on your patch to ensure an explicit dependency20:47
*** tvardeman has joined #openstack-neutron20:48
*** tvardeman_ has joined #openstack-neutron20:48
*** ijw has quit IRC20:48
*** dave_tucker_zzz is now known as dave_tucker20:48
*** BuSerD has quit IRC20:49
markmcclainmarun: -1 on that20:50
rkukuramarun: Probably. I'd suggest making the other updates and letting the review continue on yours in the mean time.20:50
marunmarkmcclain: ok20:50
markmcclainwe should keep them independent as one is a bug and the other is a blueprint20:50
markmcclainI'd rather merge an imperfect migration that we can clean up with bob's patch20:51
marunmarkmcclain: so is there another way to ensure correct migration of port bindings?20:51
markmcclainrkukura knows that stuff better than me20:51
*** jlibosva has joined #openstack-neutron20:51
marunmarkmcclain: ok20:53
marunrkukura: so, given not adding a dependency, do I leave the binding migration as is or take it in a different direction?20:53
*** manishg has joined #openstack-neutron20:53
*** BuSerD has joined #openstack-neutron20:54
rkukuramarun, markmcclain: The piece that's I'm pretty sure is critical is setting binding.segment to the UUID of the (only) network segment of the port's migrated network.20:54
rkukuraWithout that, the get_device_details RPC from the L2 agent won't return the info needed.20:55
jlibosvamarkmcclain: hi, I accidentally wrote you on openstack-meeting20:56
marunrkukura: what do you mean by '(only)'?20:56
*** salv-orlando has quit IRC20:56
rkukuraSo things would continue to work, but if an L2 agent was restarted, they would break.20:56
* jlibosva is not used on this irc client20:56
rkukuramarun: By "only", I mean that since the monolithic plugins don't support multi-segment networks, the migrated networks will have only one segment.20:57
*** ajo has quit IRC20:57
*** ijw has joined #openstack-neutron21:00
*** ijw_ has joined #openstack-neutron21:01
markmcclainjlibosva: I'll be multitasking in the project meeting.. what's up?21:01
jlibosvamarkmcclain: I just wanted to ask about https://review.openstack.org/#/c/75924/ but we can discuss this tomorrow (again :)) ) if you want21:03
openstackgerritBerezovsky Irena proposed a change to openstack/neutron: Add update binding:profile with physical_network  https://review.openstack.org/8128121:03
*** spandhe has quit IRC21:03
marunrkukura: ah, so it should be easy enough to ensure that the bindings have the required segment id21:04
markmcclainjlibosva: ok so what did you want to discuss21:04
*** ijw has quit IRC21:04
*** manishg_ has joined #openstack-neutron21:05
jlibosvamarkmcclain: oleg told me you were against modifying scripts that are already part of previous releases. In this case migration chain from folsom to icehouse contains two identical scripts for lbaas21:05
jlibosvamarkmcclain: the only difference is that the one between folsom and grizzly is never used because is missing plugin dependency21:05
*** spandhe has joined #openstack-neutron21:05
*** spandhe has quit IRC21:05
markmcclainyeah we can't modify the timeline because those tracking trunk21:05
markmcclainor on a release version do not get the modifications21:06
jlibosvamarkmcclain: this is what I don't understand, I don't see a reason not to do it because it won't cause any harm or maybe I just don't see it21:06
rkukuramarun: right, so maybe go with that for now, and then we can revisit later21:06
*** zhipeng has joined #openstack-neutron21:07
markmcclainbut it also doesn't cause any benefit for those stuck on other points of the timeline21:07
*** spandhe has joined #openstack-neutron21:07
*** singhs has quit IRC21:07
markmcclainI was considering spending sometime discussing in Atlanta on how we can clarify our migration editing policy21:08
*** manishg has quit IRC21:08
*** manishg_ is now known as manishg21:08
marunrkukura: ok21:08
jlibosvamarkmcclain: it does - icehouse will benefit from the change when upgrading from havana. Currently if lbaas is used in havana, migration chain breaks because it attempts to re-create the tables again21:08
jlibosvamarkmcclain: +1 on discussing it21:09
rkukuramarun: thanks21:09
*** banix_ has joined #openstack-neutron21:11
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch ML2: Include bound_segment in port  https://review.openstack.org/8136721:12
*** dims has quit IRC21:12
markmcclainjlibosva: right.. need to revisit the implications of offline migrating21:13
*** ajo has joined #openstack-neutron21:16
*** banix has quit IRC21:16
*** banix_ is now known as banix21:16
*** banix has quit IRC21:17
jlibosvaI was wrong, db migration fails between grizzly and havana, but the point is the same :)21:17
*** dave_tucker is now known as dave_tucker_zzz21:21
*** zhipeng has quit IRC21:23
*** dims has joined #openstack-neutron21:26
*** blogan has quit IRC21:30
*** krtaylor has quit IRC21:31
*** jlibosva has quit IRC21:35
*** leseb has quit IRC21:38
*** zhipeng has joined #openstack-neutron21:41
*** saju_m has quit IRC21:42
*** _cjones_ has quit IRC21:45
*** _cjones_ has joined #openstack-neutron21:46
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch ML2: Include bound_segment in port  https://review.openstack.org/8136721:47
*** thuc_ has quit IRC21:48
*** ijw_ has quit IRC21:50
*** _cjones_ has quit IRC21:50
*** spandhe has quit IRC21:51
*** alagalah has joined #openstack-neutron21:51
*** dave_tucker_zzz is now known as dave_tucker21:51
*** markwash_ has joined #openstack-neutron21:52
openstackgerritNachi Ueno proposed a change to openstack/neutron: Add enable_security_group option  https://review.openstack.org/6728121:54
*** markwash has quit IRC21:54
*** markwash_ is now known as markwash21:54
*** salv-orlando has joined #openstack-neutron21:55
*** alagalah has quit IRC21:57
*** jgrimm has quit IRC21:57
openstackgerritClaudiu Belu proposed a change to openstack/neutron: Fixes Hyper-V agent security groups enable issue  https://review.openstack.org/8138221:58
*** spandhe has joined #openstack-neutron22:00
*** networkstatic has quit IRC22:04
*** _sweston_ is now known as sweston22:06
oda-gHi core reviewers, would you pick up to review https://review.openstack.org/#/c/79858/ which needs one more +2 ?22:07
*** jecarey has quit IRC22:08
*** claudiub has joined #openstack-neutron22:09
openstackgerritKevin Benton proposed a change to openstack/neutron: Allow CIDRs with non-zero masked portions  https://review.openstack.org/8113722:09
openstackgerritAkihiro Motoki proposed a change to openstack/neutron: NEC plugin: Rename quantum_id column to neutron_id  https://review.openstack.org/7885422:12
*** ajo has quit IRC22:12
*** pcm_ has quit IRC22:14
*** markmcclain1 has joined #openstack-neutron22:17
*** markmcclain has quit IRC22:17
*** leseb has joined #openstack-neutron22:17
*** leseb has quit IRC22:19
*** leseb has joined #openstack-neutron22:19
*** mwagner_lap has quit IRC22:21
*** markmcclain1 has quit IRC22:24
*** jobewan has quit IRC22:28
*** tvardeman_ has quit IRC22:30
*** tvardeman has joined #openstack-neutron22:30
*** banix has joined #openstack-neutron22:31
*** tvardeman has quit IRC22:32
*** BuSerD has quit IRC22:35
*** _cjones_ has joined #openstack-neutron22:39
*** thuc has joined #openstack-neutron22:40
*** banix has quit IRC22:40
*** BuSerD has joined #openstack-neutron22:52
*** alagalah has joined #openstack-neutron22:53
*** vkozhukalov_ has quit IRC22:56
*** zhipeng has quit IRC22:58
oda-gHi, is there somebody core reviewer ?22:59
*** dvorkini_ has joined #openstack-neutron22:59
*** devlaps has joined #openstack-neutron23:00
*** dvorkinista has quit IRC23:02
*** devlaps1 has quit IRC23:02
*** nati_ueno has quit IRC23:07
oda-gnati_ueno: ping23:07
*** claudiub has quit IRC23:08
*** leseb has quit IRC23:11
openstackgerritClaudiu Belu proposed a change to openstack/neutron: Fixes the Hyper-V agent individual ports metrics  https://review.openstack.org/7879523:11
*** leseb has joined #openstack-neutron23:12
*** crc32 has quit IRC23:13
*** leseb has quit IRC23:16
*** alagalah has quit IRC23:19
*** zhipeng has joined #openstack-neutron23:22
*** alagalah has joined #openstack-neutron23:22
*** changbl has quit IRC23:24
openstackgerritCarl Baldwin proposed a change to openstack/neutron: Don't use root to list namespaces  https://review.openstack.org/8109823:26
*** blogan has joined #openstack-neutron23:27
*** dvorkinista has joined #openstack-neutron23:28
*** BuSerD has quit IRC23:30
*** dvorkini_ has quit IRC23:31
*** spandhe has quit IRC23:31
*** krtaylor has joined #openstack-neutron23:31
*** spandhe has joined #openstack-neutron23:33
*** blogan has quit IRC23:37
*** dvorkini_ has joined #openstack-neutron23:47
*** carl_baldwin has quit IRC23:48
*** dvorkinista has quit IRC23:48
*** networkstatic has joined #openstack-neutron23:51
*** alexpilotti has quit IRC23:52
*** zzelle has quit IRC23:53
openstackgerritBrandon Logan proposed a change to openstack/neutron: Added config value help text in ns metadata proxy  https://review.openstack.org/8081623:54
*** blogan has joined #openstack-neutron23:55

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