openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: L3 agent gives preference to processing RPC messages over full sync (WIP) https://review.openstack.org/78819 | 00:01 |
---|---|---|
openstackgerrit | Paul Michali proposed a change to openstack/neutron: VPNaaS support for VPN service admin state change and reporting https://review.openstack.org/81124 | 00:02 |
*** carl_baldwin has quit IRC | 00:03 | |
*** sfox has quit IRC | 00:04 | |
*** pcm_ has quit IRC | 00: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 IRC | 00:14 | |
*** dosaboy has joined #openstack-neutron | 00:15 | |
*** sweston has quit IRC | 00:15 | |
*** mlavalle has quit IRC | 00:16 | |
*** sweston has joined #openstack-neutron | 00:16 | |
*** julim has joined #openstack-neutron | 00:17 | |
*** baoli has joined #openstack-neutron | 00:20 | |
*** thuc_ has quit IRC | 00:20 | |
*** thuc has joined #openstack-neutron | 00:20 | |
*** sweston has quit IRC | 00:21 | |
*** julim has quit IRC | 00:24 | |
lifeless | _cjones_: that RPC is within the nova network RPC abstraction, so you should read nova/network/neutronv2/api.py stuff | 00:24 |
sc68cal | ^ +1 | 00:24 |
*** thuc has quit IRC | 00:25 | |
*** emagana has quit IRC | 00:27 | |
*** matsuhashi has joined #openstack-neutron | 00:27 | |
*** Sukhdev has joined #openstack-neutron | 00:27 | |
_cjones_ | lifeless, sc68cal. Thanks. | 00:28 |
sc68cal | I *think* it basically does a port_create call via the REST API | 00:29 |
sc68cal | then eventually it percolates down to the virt driver for plugging | 00:29 |
*** manishg has quit IRC | 00:29 | |
sc68cal | and libvirt for example, creates a bridge interface and wires up the tap device to it | 00:30 |
sc68cal | _cjones_: http://docs.openstack.org/developer/neutron/devref/openvswitch_agent.html | 00:30 |
*** pbeskow has quit IRC | 00:35 | |
openstackgerrit | Sean M. Collins proposed a change to openstack/neutron: Ensure entries in dnsmasq belong to a subnet using DHCP https://review.openstack.org/64578 | 00:35 |
*** pbeskow has joined #openstack-neutron | 00:36 | |
*** inara` has quit IRC | 00: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-neutron | 00: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-neutron | 00:41 | |
sc68cal | _cjones_: base plugin v2 | 00:41 |
sc68cal | fetching link | 00:41 |
*** leseb has joined #openstack-neutron | 00:42 | |
sc68cal | _cjones_: https://github.com/openstack/neutron/blob/master/neutron/db/db_base_plugin_v2.py#L341 | 00:42 |
_cjones_ | Yup. Got it. Thanks. I've nearly pieced it all together. Thanks. | 00:42 |
sc68cal | feel free to write some doc up and contribute it | 00:42 |
sc68cal | if you're keeping notes | 00:43 |
sc68cal | we could use some additional doc in the devref | 00:43 |
_cjones_ | Heh. Okay will do. | 00:43 |
sc68cal | https://github.com/openstack/neutron/tree/master/doc/source/devref | 00:43 |
_cjones_ | This would all come in through q-svc, no? | 00:43 |
sc68cal | yeah | 00:43 |
sc68cal | I have some doc pending about how q-svc starts up and fires up the REST API | 00:44 |
sc68cal | https://review.openstack.org/#/c/79675/ | 00:44 |
*** inara has joined #openstack-neutron | 00:44 | |
*** BuSerD has quit IRC | 00:46 | |
*** leseb has quit IRC | 00: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-neutron | 00: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-neutron | 00:50 | |
dkehn-pad | dkeh_: ping | 00:51 |
*** thuc has joined #openstack-neutron | 00:51 | |
dkehn-pad | dkehn_ ping | 00:51 |
*** dkehn-pad has quit IRC | 00:51 | |
*** tomoe_ has joined #openstack-neutron | 00:53 | |
*** alexpilotti has quit IRC | 00:55 | |
*** yamahata has quit IRC | 01:03 | |
*** yamahata has joined #openstack-neutron | 01:03 | |
*** _cjones_ has quit IRC | 01:04 | |
oda-g | Hi core reviewes, would you pick up https://review.openstack.org/#/c/79858/ which needs one more +2 ? | 01:07 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow CIDRs with non-zero masked portions https://review.openstack.org/81137 | 01:09 |
*** tomoe_ has quit IRC | 01:10 | |
openstackgerrit | berlin proposed a change to openstack/neutron: Add session persistence support for NVP advanced LBaaS https://review.openstack.org/59146 | 01:11 |
*** sweston has joined #openstack-neutron | 01:11 | |
*** markmcclain has quit IRC | 01:16 | |
*** banix has quit IRC | 01:18 | |
*** networkstatic has quit IRC | 01:18 | |
*** tomoe_ has joined #openstack-neutron | 01:27 | |
*** morganfainberg is now known as morganfainberg_Z | 01:30 | |
*** ijw has quit IRC | 01:41 | |
*** banix has joined #openstack-neutron | 01:41 | |
*** dave_tucker_zzz is now known as dave_tucker | 01:42 | |
*** ijw has joined #openstack-neutron | 01:42 | |
*** HenryG has joined #openstack-neutron | 01:43 | |
*** alagalah has quit IRC | 01:46 | |
*** ijw has quit IRC | 01:47 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Show neutron API request body with debug enabled https://review.openstack.org/81109 | 01:47 |
*** ljjjustin has joined #openstack-neutron | 01:48 | |
*** morganfainberg_Z is now known as morganfainberg | 01:49 | |
*** BuSerD has joined #openstack-neutron | 01:54 | |
*** ram5391 has joined #openstack-neutron | 02:01 | |
*** ljjjustin has quit IRC | 02:03 | |
*** xianghui has joined #openstack-neutron | 02:04 | |
ram5391 | hi, after deploying quantum-gateway with juju I try to login to horizon, I recieve a 'Connection to neutron failed: Maximum attempts reached' error | 02:04 |
openstackgerrit | shihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API https://review.openstack.org/73802 | 02:04 |
ram5391 | I 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 |
sweston | ram5391: no, they are two independent services. | 02:06 |
*** terrylhowe has left #openstack-neutron | 02:07 | |
ram5391 | sweston: right, could this be a username/password mismatch issue between neutron and rabbimq? | 02:08 |
*** morganfainberg is now known as morganfainberg_Z | 02:08 | |
sweston | ram5391: I don't see how they could have gotten out of sync ... | 02:10 |
sweston | ram5391: possibly a connection limit in your database? check your open connections in your database server | 02:11 |
*** manishg has joined #openstack-neutron | 02:12 | |
ram5391 | sweston: currently mysql is set at -1 for connection limit through juju. No idea if that is what it should be | 02:13 |
sweston | ram5391: 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 IRC | 02:15 | |
sweston | ram5391: what conditions lead you up to this error? | 02:16 |
*** manishg_ has joined #openstack-neutron | 02:16 | |
ram5391 | sweston: 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 IRC | 02:17 | |
ram5391 | the 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 neutron | 02:17 |
ram5391 | literal string is 'Connection to neutron failed: Maximum attempts reached' | 02:18 |
*** RajeshMohan has quit IRC | 02:18 | |
sweston | ram5391: 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_away | 02:21 | |
ram5391 | sweston: interesting.. stupid question, does the quantum gateway machine need to be exposed? | 02:22 |
sweston | ram5391: I would check your neutron server process ... it is complaining about the api service, so unless juju changed the port, it should be on 9696 | 02:24 |
*** manishg_ has quit IRC | 02:24 | |
sweston | ram5391: that's a possibility that you had a route in your neutron installation that borked things up for you | 02:25 |
openstackgerrit | A change was merged to openstack/neutron: Create new IPv6 attributes for Subnets https://review.openstack.org/52983 | 02:26 |
ram5391 | sweston: 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 IRC | 02:27 | |
*** Sukhdev has quit IRC | 02:27 | |
ram5391 | If I navigate to the url :9696 I recieve a valid xml document | 02:27 |
ram5391 | however, telnet from the horizon machine to the quantum machine does not show an xml document | 02:28 |
ram5391 | sweston: nevermind, upon curling I get the xml | 02:29 |
sweston | ram5391: that's alright, we are all learning :-). try this command wget -O - http://<your-neutron-ip>:9696 | 02:30 |
*** thuc has quit IRC | 02:32 | |
openstackgerrit | A change was merged to openstack/neutron: Calculate stateless IPv6 address https://review.openstack.org/56184 | 02:32 |
ram5391 | sweston: request came back 200 okay | 02:32 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Ignore PortNotFound exceptions on lockless delete https://review.openstack.org/81025 | 02:32 |
*** thuc has joined #openstack-neutron | 02:32 | |
sweston | ram5391: are you making the connection from your horizon server to the neutron ip? | 02:33 |
ram5391 | sweston: yes indeed | 02:33 |
ram5391 | sweston: i'm even using the dns instead of IP | 02:34 |
*** thuc has quit IRC | 02:37 | |
marun | mestery: ping | 02:37 |
marun | rkukura: ping | 02:38 |
sweston | ram5391: 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-neutron | 02:39 | |
ram5391 | sweston: ah okay, thanks for the help, perhaps I should re-install horizon just for good measure | 02:39 |
sweston | ram5391: that's one way to approach it ;-). Good luck. | 02:40 |
ram5391 | sweston: can't hurt to try, thanks! | 02:41 |
sweston | ram5391: sure thing :-) | 02:42 |
*** tomoe_ has quit IRC | 02:43 | |
*** gongysh has joined #openstack-neutron | 02:44 | |
ram5391 | also, 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 edhall | 02:49 | |
*** gongysh has quit IRC | 02:49 | |
*** suresh12 has quit IRC | 02:51 | |
*** crc32 has quit IRC | 02:55 | |
*** alagalah has joined #openstack-neutron | 02:57 | |
*** dave_tucker is now known as dave_tucker_zzz | 03:00 | |
*** tomoe_ has joined #openstack-neutron | 03:02 | |
*** alagalah has quit IRC | 03:02 | |
*** gongysh has joined #openstack-neutron | 03:02 | |
openstackgerrit | Xu Han Peng proposed a change to openstack/neutron: Permit ICMPv6 RAs only from known routers https://review.openstack.org/72252 | 03:03 |
*** dvorkinista has joined #openstack-neutron | 03:03 | |
sweston | yamata: ping | 03:05 |
*** harlowja_away is now known as harlowja | 03:06 | |
*** BuSerD has quit IRC | 03:07 | |
*** gongysh has quit IRC | 03:08 | |
*** BuSerD has joined #openstack-neutron | 03:10 | |
*** matsuhashi has quit IRC | 03:14 | |
*** tomoe_ has quit IRC | 03:20 | |
*** matsuhashi has joined #openstack-neutron | 03:20 | |
*** gongysh has joined #openstack-neutron | 03:21 | |
*** dvorkinista has quit IRC | 03:24 | |
*** gongysh has quit IRC | 03:26 | |
*** baoli has quit IRC | 03:29 | |
*** baoli has joined #openstack-neutron | 03:30 | |
*** matsuhashi has quit IRC | 03:30 | |
*** matsuhashi has joined #openstack-neutron | 03:33 | |
*** BuSerD has quit IRC | 03:34 | |
*** baoli has quit IRC | 03:34 | |
*** BuSerD has joined #openstack-neutron | 03:37 | |
*** gongysh has joined #openstack-neutron | 03:39 | |
*** matsuhashi has quit IRC | 03:42 | |
*** gongysh has quit IRC | 03:44 | |
*** morganfainberg_Z is now known as morganfainberg | 03:52 | |
*** BuSerD has quit IRC | 03:52 | |
*** gongysh has joined #openstack-neutron | 03:57 | |
*** BuSerD has joined #openstack-neutron | 03:57 | |
*** BuSerD has quit IRC | 03:58 | |
*** sfox has joined #openstack-neutron | 04:01 | |
*** suresh12 has joined #openstack-neutron | 04:01 | |
*** kashyap has joined #openstack-neutron | 04:02 | |
openstackgerrit | shihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API https://review.openstack.org/73802 | 04:02 |
*** suresh12 has quit IRC | 04:06 | |
*** vkozhukalov_ has joined #openstack-neutron | 04:07 | |
*** ramishra has joined #openstack-neutron | 04:07 | |
*** ramishra_ has joined #openstack-neutron | 04:08 | |
*** WackoRobie has quit IRC | 04:09 | |
*** WackoRobie has joined #openstack-neutron | 04:10 | |
*** ramishra has quit IRC | 04:12 | |
*** WackoRobie has quit IRC | 04:14 | |
*** gongysh has quit IRC | 04:22 | |
*** mwagner_lap is now known as mwagner_zzz | 04:25 | |
*** dvorkinista has joined #openstack-neutron | 04:26 | |
*** _cjones_ has joined #openstack-neutron | 04:26 | |
*** tomoe_ has joined #openstack-neutron | 04:29 | |
*** chandan_kumar has joined #openstack-neutron | 04:31 | |
*** _cjones_ has quit IRC | 04:31 | |
*** linuxaddicts has quit IRC | 04:32 | |
*** dvorkinista has quit IRC | 04:32 | |
*** gongysh has joined #openstack-neutron | 04:34 | |
openstackgerrit | A change was merged to openstack/neutron: Added missing l3_update call in update_network https://review.openstack.org/80887 | 04:37 |
*** WackoRobie has joined #openstack-neutron | 04:37 | |
*** gongysh has quit IRC | 04:40 | |
*** WackoRob_ has joined #openstack-neutron | 04:40 | |
*** WackoRobie has quit IRC | 04:40 | |
*** WackoRob_ has quit IRC | 04:42 | |
*** WackoRobie has joined #openstack-neutron | 04:42 | |
*** vkozhukalov_ has quit IRC | 04:43 | |
*** WackoRobie has quit IRC | 04:43 | |
*** WackoRobie has joined #openstack-neutron | 04:44 | |
*** linuxaddicts has joined #openstack-neutron | 04:48 | |
*** WackoRobie has quit IRC | 04:49 | |
*** sweston has quit IRC | 04:52 | |
*** gongysh has joined #openstack-neutron | 04:53 | |
*** pradipta_away is now known as pradipta | 04:53 | |
*** matsuhashi has joined #openstack-neutron | 04:55 | |
*** gongysh has quit IRC | 04:59 | |
*** tomoe_ has quit IRC | 05:01 | |
*** linuxaddicts has quit IRC | 05:01 | |
*** tomoe_ has joined #openstack-neutron | 05:04 | |
*** singhs has joined #openstack-neutron | 05:10 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Ignore PortNotFound exceptions on lockless delete https://review.openstack.org/81025 | 05:11 |
*** gongysh has joined #openstack-neutron | 05:12 | |
*** singhs_ has joined #openstack-neutron | 05:12 | |
*** singhs has quit IRC | 05:14 | |
*** singhs_ is now known as singhs | 05:14 | |
*** gongysh has quit IRC | 05:17 | |
openstackgerrit | Ian Wienand proposed a change to openstack/neutron: Record and log reason for dhcp agent resync https://review.openstack.org/81173 | 05:20 |
*** irenab has joined #openstack-neutron | 05:21 | |
*** linuxaddicts has joined #openstack-neutron | 05:23 | |
*** linuxaddicts has quit IRC | 05:26 | |
*** yfried has quit IRC | 05:27 | |
*** tomoe_ has quit IRC | 05:28 | |
*** gongysh has joined #openstack-neutron | 05:30 | |
*** gongysh has quit IRC | 05:35 | |
*** networkstatic has joined #openstack-neutron | 05:35 | |
*** dvorkinista has joined #openstack-neutron | 05:39 | |
*** harlowja is now known as harlowja_away | 05:40 | |
*** WackoRobie has joined #openstack-neutron | 05:43 | |
*** linuxaddicts has joined #openstack-neutron | 05:43 | |
*** ramishra_ has quit IRC | 05:46 | |
*** WackoRobie has quit IRC | 05:48 | |
*** gongysh has joined #openstack-neutron | 05:48 | |
*** gongysh has quit IRC | 05:53 | |
*** ramishra has joined #openstack-neutron | 05:53 | |
*** sweston has joined #openstack-neutron | 06:01 | |
*** Jabadia has joined #openstack-neutron | 06:04 | |
*** gdubreui has quit IRC | 06:06 | |
*** dvorkinista has quit IRC | 06:08 | |
*** dvorkinista has joined #openstack-neutron | 06:08 | |
*** dvorkinista has quit IRC | 06:12 | |
*** rotbeard has joined #openstack-neutron | 06:14 | |
*** gdubreui has joined #openstack-neutron | 06:18 | |
marun | lifeless: If you have a moment, I have a question regarding foreign keys and locking. | 06:18 |
lifeless | shoot | 06:19 |
lifeless | C is in bed now | 06:19 |
lifeless | so I can talk | 06:19 |
*** vkozhukalov_ has joined #openstack-neutron | 06:22 | |
marun | lifeless: will locking a parent record have any impact on other transactions being able to insert child records? | 06:23 |
marun | lifeless: 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 |
marun | er, e.g. | 06:25 |
lifeless | one of the transactions will definitely abort to the best of my knowledge | 06:26 |
lifeless | I know more in the postgresql world about the specific details here | 06:26 |
lifeless | SpamapS: / mordred: ^ mysql behaviour ? | 06:26 |
lifeless | marun: presuming you have on delete cascade, of course | 06:27 |
marun | lifeless: no. there lies the rub | 06:27 |
lifeless | marun: so codewise, I don't understand the problem you were describing before | 06:27 |
lifeless | marun: oh, no cascade delete? whyever not ? | 06:27 |
marun | lifeless: well, there are a couple of reasons | 06:28 |
marun | lifeless: but mainly part of the referential integrity is managed in code | 06:28 |
marun | lifeless: or maybe I'm abusing the term | 06:28 |
lifeless | marun: thats usually a bad idea, but go on | 06:28 |
marun | lifeless: erm, I'm struggling to pick a good example | 06:30 |
lifeless | marun: pick *an* example then :) | 06:30 |
*** matsuhashi has quit IRC | 06:30 | |
marun | lifeless: there isn't a delete-cascade on network <-> ports | 06:30 |
lifeless | can a port be owned by multiple networks ? | 06:31 |
marun | lifeless: no | 06:31 |
marun | lifeless: but there are different classes of ports | 06:31 |
lifeless | so 1 network : M ports | 06:31 |
marun | yes | 06:31 |
lifeless | marun: I don't understand why classes of ports affects the schema | 06:31 |
marun | in the case of user-defined ports, they can be deleted no problem | 06:32 |
marun | in the case of admin-defined ports, they can't be | 06:32 |
lifeless | marun: 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 in | 06:32 |
mordred | lifeless: waht? | 06:32 |
lifeless | marun: can you expand on the admin port issue ? | 06:32 |
marun | this is why I'm struggling, I have the feeling I'm missing something. | 06:32 |
openstackgerrit | shihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API https://review.openstack.org/73802 | 06:32 |
*** thuc has joined #openstack-neutron | 06:33 | |
lifeless | mordred: mysql - table A and B, B has a FK pointing to A's PK. select for update a row in A. | 06:33 |
marun | lifeless: bad example. my brain is fried from looking at this for too long. | 06:33 |
mordred | too early in morning. | 06:33 |
lifeless | mordred: 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 transaction | 06:33 |
lifeless | mordred: morning ? | 06:33 |
mordred | lifeless: I'm in a weird timezone | 06:33 |
*** SridharG has joined #openstack-neutron | 06:33 | |
lifeless | mordred: orly, i'm SO surprised. | 06:34 |
marun | lifeless: a better reason for not using delete cascade is that the ml2 plugin now has events surrounding create/update/delete of individual resources | 06:34 |
marun | lifeless: maybe a better reason? | 06:34 |
marun | lifeless: (if we can lock the parent and prevent addition during deletion, maybe not a good reason) | 06:35 |
marun | lifeless: does any of that make sense? | 06:35 |
lifeless | so, needing to raise events makes sense sure. | 06:36 |
lifeless | I 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 inserts | 06:36 |
*** thuc has quit IRC | 06:37 | |
marun | and will locking that range affect related table insertion? | 06:37 |
openstackgerrit | Jenkins proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/81185 | 06:37 |
lifeless | marun: you lock that on the related tables | 06:38 |
marun | so we have to hold locks on every related table? | 06:38 |
lifeless | select for update foo where fk_column=pk_of_parent_table | 06:38 |
lifeless | marun: yes, of course. | 06:38 |
marun | :( | 06:38 |
marun | too many f'ing tables | 06:38 |
mordred | lifeless, marun: I've asked stewart to join us - he's smart | 06:39 |
openstackgerrit | A change was merged to openstack/neutron: Validate multicast ip range in Cisco N1kv Plugin https://review.openstack.org/75291 | 06:40 |
*** networkstatic has quit IRC | 06:41 | |
*** networkstatic has joined #openstack-neutron | 06:41 | |
marun | lifeless: most of the relationships in the neutron db are actually managed by fk's with cascade deletion | 06:42 |
*** gongysh has joined #openstack-neutron | 06:42 | |
SpamapS | lifeless: o/ | 06:43 |
*** WackoRobie has joined #openstack-neutron | 06:43 | |
*** tomoe_ has joined #openstack-neutron | 06:43 | |
marun | SpamapS: so, is this a stupid line of inquiry? | 06:43 |
SpamapS | no, let me see if I understand. | 06:43 |
*** saju_m has joined #openstack-neutron | 06:44 | |
SpamapS | select for update foo where fk_column=pk_of_parent_table; ... the only lock here is on foo. | 06:44 |
SpamapS | what's the next question? | 06:44 |
*** WackoRobie has quit IRC | 06:44 | |
*** WackoRobie has joined #openstack-neutron | 06:45 | |
marun | so if I have table A with 10 tables with fk's for A | 06:45 |
*** saju_m has quit IRC | 06:45 | |
marun | and I want to delete row x in table A | 06:45 |
SpamapS | wait, you have table A, with 10 tables? | 06:45 |
marun | table A + 10 tables that relate to A | 06:46 |
SpamapS | AH ok | 06:46 |
*** saju_m has joined #openstack-neutron | 06:46 | |
SpamapS | Right you can't delete things in A unless all 10 FK's are deleted first. | 06:46 |
marun | Right | 06:46 |
*** yfried has joined #openstack-neutron | 06:46 | |
marun | And I need to generate an event for many of the rows in related tables when they are removed | 06:47 |
marun | event -> python call | 06:47 |
SpamapS | So it sounds like you want to do a transaction, emit events, then commit it? | 06:48 |
marun | That's part of it. | 06:48 |
marun | But 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 IRC | 06:49 | |
SpamapS | the way I said it would work.. | 06:49 |
*** matsuhashi has joined #openstack-neutron | 06:49 | |
marun | If 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 IRC | 06:49 | |
SpamapS | you 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.html | 06:49 |
SpamapS | marun: wait ok ignore the XA comment for a second. | 06:50 |
SpamapS | marun: if cascading deletion is used, you won't know what is deleted. | 06:50 |
marun | exactly | 06:50 |
SpamapS | marun: so IMO, it isn't really an option if you must send events for all deletes. | 06:50 |
SpamapS | marun: 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 skraynev | 06:51 | |
SpamapS | marun: 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 gaps | 06:52 |
marun | yee | 06:52 |
SpamapS | they're retryable | 06:52 |
SpamapS | but you have to retry the whole thing, all the way back to the select for update | 06:52 |
marun | so 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 |
SpamapS | you lost me | 06:53 |
SpamapS | If 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 IRC | 06:53 | |
openstackgerrit | berlin proposed a change to openstack/neutron: Fix duplicate name of NVP LBaaS objs not allowed on vShield Edge https://review.openstack.org/73905 | 06:53 |
marun | so if events (e.g. resource deletion) are required | 06:54 |
marun | cascade deletion doesn't indicate when records have been added, so locking via 'select for update' is required. | 06:54 |
SpamapS | I don't understand what you mean. | 06:54 |
SpamapS | What records will you be adding? | 06:54 |
marun | first transaction - deleting parent resource and related resources | 06:54 |
marun | second transaction - adding related resources | 06:55 |
marun | resources -> rows | 06:55 |
SpamapS | also 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 |
marun | right. | 06:55 |
SpamapS | even if it is not there | 06:55 |
SpamapS | the index slot is locked | 06:56 |
lifeless | SpamapS: on postgresql stuff in the gaps won't be affected at all :) | 06:56 |
marun | i'm worried at the number of tables that will need to be locked in some cases | 06:56 |
lifeless | SpamapS: its better than that :> | 06:56 |
marun | lifeless: gaps? | 06:56 |
SpamapS | lifeless: 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 |
SpamapS | hm.. I bet only if there was a deadlock actually | 06:58 |
SpamapS | lifeless: 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 |
lifeless | SpamapS: why would it? unless the second transaction is for the locked range | 06:59 |
openstackgerrit | Xu Han Peng proposed a change to openstack/neutron: Permit ICMPv6 RAs only from known routers https://review.openstack.org/72252 | 06:59 |
SpamapS | right "liveness, blah blah, better optimizer... foo foo.." | 06:59 |
SpamapS | let's not get into that | 06:59 |
lifeless | SpamapS: 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 them | 06:59 |
SpamapS | marun: why are you worried about the number of tables that need to be locked? It is _rows_ you are locking, not tables. | 07:00 |
lifeless | SpamapS: seriously not trying to troll | 07:00 |
marun | SpamapS: actually, I don't think it's such an issue. not all related tables require events. | 07:00 |
SpamapS | lifeless: I'm sure you aren't, but I'm unusually grumpy today. :-P | 07:00 |
lifeless | SpamapS: orly :P | 07:00 |
*** tomoe_ has quit IRC | 07:00 | |
lifeless | marun: how many hundreds of tables ? | 07:00 |
marun | If we're using facebook usage as an indicator of quality, shouldn't we switch to php? | 07:00 |
lifeless | marun: (teasing slightly) | 07:00 |
marun | lifeless: I'm kind of afraid of the neutron codebase. it doesn't take 100s of tables to cause problems. | 07:01 |
lifeless | marun: 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-neutron | 07:01 | |
marun | I think the current strategy is 'fake it till you make it' | 07:02 |
marun | https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L432 | 07:02 |
lifeless | marun: if liveness is a particular concern, perhaps an eventual consistency strategy is what you should be refactoring too | 07:02 |
SpamapS | Also the gap locking thing is really not that big of a deal. | 07:02 |
marun | so 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 |
lifeless | if you've got 10's of thousands of ports, the gaps that are locked are small | 07:03 |
SpamapS | It matters _a lot_ in some cases, like mass deletes/updates.. but usually there are very few gaps locked for long at all. | 07:03 |
lifeless | marun: this method == eventual consistency ? | 07:03 |
oda-g | Hi core reviewers, would you pick up to review https://review.openstack.org/#/c/79858/ which needs one more +2 ? | 07:04 |
marun | lifeless: not really, it just retries | 07:04 |
lifeless | marun: I was asking what you meant | 07:04 |
lifeless | marun: when yousaid 'in this method' | 07:04 |
marun | lifeless: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L432 | 07:04 |
lifeless | oh the code | 07:04 |
SpamapS | It seems weird | 07:05 |
marun | it is :) | 07:05 |
lifeless | marun: so thats basically what you'll need no matter what | 07:05 |
lifeless | marun: because concurrency in sql is expressed as integrityerror exceptions | 07:05 |
SpamapS | ok so yeah, select for update on the key to prevent another transaction interfering with this one is probably the right answer. | 07:05 |
marun | lifeless: 'select for update' wouldn't work? | 07:06 |
SpamapS | select for update is going to prevent the case of a concurrent insert pwning the FK's.. | 07:06 |
lifeless | marun: and either the end user or someone else has to deal with that | 07:06 |
SpamapS | but you may still have transactions roll back. | 07:06 |
lifeless | marun: select for update will fail if there is another transaction open that would prevent the update | 07:06 |
*** gongysh has quit IRC | 07:07 | |
lifeless | marun: thats the whole point of it; it doesn't make the concurrency go away, it means you find out earlier | 07:07 |
marun | so is there any point to using 'select for update'? | 07:07 |
lifeless | yes, you find out earlier ;) | 07:08 |
lifeless | which 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 row | 07:08 |
lifeless | but let me describe an eventual consistency approach | 07:08 |
lifeless | we're talking network right, so I'll describe a hypothetical case there | 07:09 |
lifeless | add a new table network_ops_journal | 07:09 |
*** emagana has joined #openstack-neutron | 07:09 | |
lifeless | all *reads* that explicitly or implicitly depend on network should join through that table (left outer) | 07:09 |
lifeless | if 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 table | 07:10 |
lifeless | delete_network becomes insert into network_ops_journal (network_pk, 'start_delete', timestamp) | 07:11 |
lifeless | add 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 past | 07:11 |
marun | i've worked on systems that had the idea of an in-table flag to indicate deletion | 07:12 |
lifeless | [also, have the db set to kill transactions longer than that length...] | 07:12 |
SpamapS | lifeless: this is starting to sound like an MVCC implementation.. :) | 07:12 |
lifeless | then, you *know* noone will be updating or adding to the network or any child tables subsequent to that point | 07:12 |
marun | What's the advantage of having it in a separate table, other than separating out the deletion fields from the model? | 07:12 |
lifeless | and your deleting become a lot simpler | 07:12 |
lifeless | marun: if you need to journal creates, its hard to do in the same table :) | 07:13 |
lifeless | marun: really depends on your liveness issues | 07:13 |
lifeless | SpamapS: not really, journals in postgresql are useful too for liveness | 07:13 |
*** tomoe_ has joined #openstack-neutron | 07:13 | |
*** D7 has joined #openstack-neutron | 07:13 | |
marun | Hmmm | 07:13 |
lifeless | marun: but sure, you could do a soft_delete field and look for that | 07:13 |
*** Dseven has quit IRC | 07:13 | |
lifeless | point is that you can commit the delete immediately without doing schema contents cleanup | 07:14 |
marun | right. +1 for that for sure. | 07:14 |
lifeless | and then in the background raise events and cleanup at your own pace | 07:14 |
SpamapS | why +1 for that? | 07:14 |
SpamapS | if DELETE of a resource is expensive.. | 07:14 |
SpamapS | the API should simply respond with "OK I will delete that!" | 07:14 |
SpamapS | and just do it in the DB | 07:14 |
marun | er | 07:15 |
marun | that's not really the problem | 07:15 |
marun | we have issues with eventlet + mysql + async events causing deadlock because transactions hold locks for too long | 07:15 |
lifeless | liveness ;) | 07:15 |
lifeless | which generally means you're doing too much work in transactions | 07:15 |
SpamapS | Why are transactions being held open for a long time? | 07:15 |
marun | yes | 07:16 |
lifeless | or you are suffering lock contention | 07:16 |
marun | that too | 07:16 |
lifeless | or both | 07:16 |
lifeless | too much work can drive lock contention | 07:16 |
lifeless | its a vicious circle | 07:16 |
SpamapS | one thing about gap locking.. it affords MySQL instant deadlock detection. | 07:16 |
marun | too much confusing and complex abstraction -> too much work in transactions | 07:16 |
marun | can you please explain gap locking? | 07:16 |
marun | or point me at a resource that can? | 07:17 |
lifeless | marun: you're familiar with btrees | 07:17 |
marun | yes | 07:17 |
marun | well, i should say i know what they are | 07:17 |
lifeless | marun: mysql locks the adjacency in the tree, not just the row selected | 07:17 |
marun | of an index you mean? | 07:17 |
SpamapS | marun: 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 |
lifeless | marun: which prevents inserts on either side with keys that are < or > than the index key in question, bounded by the next gkey up or down | 07:18 |
lifeless | SpamapS: is it PK order only, or done on the indices? | 07:18 |
SpamapS | to be clear, it does not happen on single key reads. There's no point there. | 07:18 |
lifeless | I haven't actually read the implementation... | 07:18 |
SpamapS | lifeless: it is always in index order. | 07:18 |
lifeless | gotta run, crying baby | 07:18 |
SpamapS | be well | 07:19 |
marun | lifeless: thanks for you help! | 07:19 |
*** gongysh has joined #openstack-neutron | 07:20 | |
marun | SpamapS: hmm, I wonder if gap locks have been causing us grief in the gate. | 07:20 |
SpamapS | marun: so one thing to do is to avoid doing large ranged index reads in transactions if you can. | 07:20 |
marun | SpamapS: so narrow or single index reads are fine? | 07:21 |
SpamapS | marun: right, and there is an isolation level you can use which doesn't do it.. | 07:21 |
marun | SpamapS: READ COMMITTED? | 07:22 |
SpamapS | marun: http://www.mysqlperformanceblog.com/2012/08/28/differences-between-read-committed-and-repeatable-read-transaction-isolation-levels/ | 07:22 |
SpamapS | marun: yeah I think thats the one | 07:22 |
*** networkstatic has quit IRC | 07:23 | |
*** networkstatic has joined #openstack-neutron | 07:23 | |
SpamapS | marun: "If you run an UPDATE that is not well indexed you will lock many rows" | 07:23 |
marun | SpamapS: but is there any real advantage if indexes are used for most things? | 07:23 |
SpamapS | likely cause of problems :) | 07:23 |
SpamapS | marun: be 100% sure indexes are being used as you think they are :) | 07:24 |
*** tomoe_ has quit IRC | 07:24 | |
marun | fair enough | 07:24 |
SpamapS | marun: for the "delete a network" code, I think read committed would actually be fine. | 07:25 |
*** gongysh has quit IRC | 07:25 | |
marun | is that cross-db? i.e. does pg support the same thing? | 07:26 |
*** enikanorov__ has joined #openstack-neutron | 07:28 | |
SpamapS | marun: http://www.postgresql.org/docs/9.1/static/transaction-iso.html | 07:30 |
*** enikanorov_ has quit IRC | 07:31 | |
marun | SpamapS: ok. | 07:33 |
*** ajo has joined #openstack-neutron | 07:33 | |
SpamapS | marun: some things would fail horribly with read committed though, so YMMV :) | 07:33 |
marun | SpamapS: and what would the advantage of moving to read committed for network deletion? | 07:33 |
marun | SpamapS: deletion is handled by primary key for network, subnet and ports, so I'm not sure I see how it would help. | 07:34 |
marun | SpamapS: :( | 07:35 |
marun | https://review.openstack.org/#/c/78880/6/neutron/db/db_base_plugin_v2.py | 07:35 |
marun | SpamapS: this patch just merged, and it removes any guarantee that port insertion can't happen during delete. | 07:36 |
*** tomoe_ has joined #openstack-neutron | 07:37 | |
SpamapS | marun: ah. Wel then.. looping will just happen more often right? :-/ | 07:37 |
marun | SpamapS: except that this method doesn't have a wrapper that does the looping. | 07:38 |
*** gongysh has joined #openstack-neutron | 07:38 | |
SpamapS | marun: OH! | 07:38 |
SpamapS | marun: I see now | 07:38 |
SpamapS | marun: no transaction.. | 07:38 |
marun | SpamapS: right | 07:38 |
*** ajo has quit IRC | 07:38 | |
SpamapS | so.. "this consistency thing is causing locking, remove it" | 07:39 |
marun | SpamapS: We don't have enough people clueful of db-related stuff :/ | 07:39 |
SpamapS | but.. | 07:39 |
marun | SpamapS: I'm a bit more clueful than most, but as you can see, not much more. | 07:39 |
SpamapS | this is like.. transactions 101 | 07:39 |
*** ajo has joined #openstack-neutron | 07:39 | |
marun | Yeah | 07:39 |
marun | SpamapS: 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-neutron | 07:40 | |
SpamapS | marun: yeah.. major red flag | 07:40 |
marun | SpamapS: I think we really need a db session at summit. | 07:40 |
SpamapS | marun: 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 IRC | 07:41 | |
SpamapS | marun: 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-neutron | 07:41 | |
SpamapS | marun: 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 sleeps | 07:42 | |
marun | SpamapS: thanks for your help! | 07:42 |
*** WackoRobie has joined #openstack-neutron | 07:43 | |
*** gongysh has quit IRC | 07:43 | |
lifeless | btw | 07:46 |
lifeless | one 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 |
lifeless | terrible for liveness, great if you want these broad locks | 07:47 |
lifeless | but honestly, deletes are rare, optimise the other cases | 07:47 |
*** WackoRobie has quit IRC | 07:48 | |
*** tomoe_ has quit IRC | 07:50 | |
openstackgerrit | Maru Newby proposed a change to openstack/neutron: Stop removing ip allocations on port delete https://review.openstack.org/81196 | 07:52 |
*** amuller has joined #openstack-neutron | 07:56 | |
*** gongysh has joined #openstack-neutron | 07:56 | |
*** emagana has quit IRC | 07:58 | |
*** ramishra has quit IRC | 08:00 | |
*** gongysh has quit IRC | 08:01 | |
*** luqas has joined #openstack-neutron | 08:08 | |
*** dvorkinista has joined #openstack-neutron | 08:09 | |
*** nati_ueno has joined #openstack-neutron | 08:11 | |
*** ramishra has joined #openstack-neutron | 08:13 | |
*** dvorkinista has quit IRC | 08:13 | |
*** gongysh has joined #openstack-neutron | 08:14 | |
*** gongysh has quit IRC | 08:19 | |
*** dave_tucker_zzz is now known as dave_tucker | 08:20 | |
marun | lifeless: deletes are rare, except in the gate :p | 08:21 |
*** ramishra has quit IRC | 08:23 | |
openstackgerrit | Assaf 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/80829 | 08:25 |
*** jistr has joined #openstack-neutron | 08:26 | |
*** arosen-home has quit IRC | 08:26 | |
*** jistr is now known as jistr|mtg | 08:27 | |
*** carl_baldwin has joined #openstack-neutron | 08:28 | |
lifeless | marun: even so :) | 08:30 |
*** ramishra has joined #openstack-neutron | 08:31 | |
*** _cjones_ has joined #openstack-neutron | 08:32 | |
*** gongysh has joined #openstack-neutron | 08:32 | |
*** morganfainberg is now known as morganfainberg_Z | 08:34 | |
openstackgerrit | shihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API https://review.openstack.org/73802 | 08:35 |
*** gongysh has quit IRC | 08:37 | |
*** _cjones_ has quit IRC | 08:37 | |
*** sfox has quit IRC | 08:39 | |
*** WackoRobie has joined #openstack-neutron | 08:43 | |
*** jgallard has joined #openstack-neutron | 08:44 | |
*** ramishra has quit IRC | 08:47 | |
*** jp_at_hp has joined #openstack-neutron | 08:47 | |
*** WackoRobie has quit IRC | 08:47 | |
*** jlibosva has joined #openstack-neutron | 08:48 | |
*** djoreilly has joined #openstack-neutron | 08:49 | |
*** saju_m has quit IRC | 08:50 | |
*** rossella_s has joined #openstack-neutron | 08:50 | |
*** gongysh has joined #openstack-neutron | 08:50 | |
*** ygbo has joined #openstack-neutron | 08:54 | |
*** saju_m has joined #openstack-neutron | 08:56 | |
*** gongysh has quit IRC | 08:56 | |
openstackgerrit | berlin proposed a change to openstack/neutron: Add Error Handling to NVP advanced LBaaS/FWaaS https://review.openstack.org/59625 | 08:57 |
*** leseb has joined #openstack-neutron | 09:02 | |
*** ramishra has joined #openstack-neutron | 09:03 | |
*** gongysh has joined #openstack-neutron | 09:09 | |
*** dvorkinista has joined #openstack-neutron | 09:10 | |
openstackgerrit | Roman Podoliaka proposed a change to openstack/neutron: WIP: sync db code https://review.openstack.org/77205 | 09:12 |
*** tomoiaga has left #openstack-neutron | 09:12 | |
*** nati_ueno has quit IRC | 09:13 | |
*** dvorkinista has quit IRC | 09:14 | |
*** gongysh has quit IRC | 09:14 | |
*** pradipta is now known as pradipta_away | 09:15 | |
*** singhs has quit IRC | 09:16 | |
*** rotbeard has quit IRC | 09:16 | |
*** leseb has quit IRC | 09:16 | |
*** leseb has joined #openstack-neutron | 09:17 | |
*** pradipta_away is now known as pradipta | 09:18 | |
*** gdubreui has joined #openstack-neutron | 09:19 | |
*** leseb has quit IRC | 09:21 | |
*** leseb has joined #openstack-neutron | 09:24 | |
openstackgerrit | Phil Day proposed a change to openstack/python-neutronclient: Allow user ID for authentication https://review.openstack.org/78596 | 09:25 |
*** gongysh has joined #openstack-neutron | 09:27 | |
*** tomoe_ has joined #openstack-neutron | 09:30 | |
*** tomoe_ has quit IRC | 09:31 | |
*** tomoe_ has joined #openstack-neutron | 09:31 | |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator https://review.openstack.org/81211 | 09:32 |
*** afazekas_ has joined #openstack-neutron | 09:34 | |
*** rotbeard has joined #openstack-neutron | 09:35 | |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator https://review.openstack.org/81211 | 09:37 |
*** WackoRobie has joined #openstack-neutron | 09:43 | |
*** bada has joined #openstack-neutron | 09:44 | |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NSX plugin: return 400 for invalid gw certificate https://review.openstack.org/80948 | 09:47 |
*** WackoRobie has quit IRC | 09:48 | |
*** sphoorti has joined #openstack-neutron | 09:56 | |
*** amuller has quit IRC | 09:56 | |
openstackgerrit | shihanzhang proposed a change to openstack/neutron: Prevent dhcp port deletion from the API https://review.openstack.org/73802 | 09:57 |
openstackgerrit | Sphoorti proposed a change to openstack/neutron: Add unit test for add_vxlan in test_linux_ip_lib https://review.openstack.org/80554 | 10:04 |
*** dvorkinista has joined #openstack-neutron | 10:10 | |
*** dvorkinista has quit IRC | 10:16 | |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator https://review.openstack.org/81211 | 10:16 |
*** sweston has quit IRC | 10:16 | |
*** nati_ueno has joined #openstack-neutron | 10:20 | |
*** pradipta is now known as pradipta_away | 10:25 | |
*** matsuhashi has quit IRC | 10:25 | |
*** pcm_ has joined #openstack-neutron | 10:27 | |
*** networkstatic is now known as networkstatic_zZ | 10:29 | |
*** bvandenh has joined #openstack-neutron | 10:31 | |
*** pcm_ has quit IRC | 10:31 | |
*** pcm_ has joined #openstack-neutron | 10:32 | |
*** nati_ueno has quit IRC | 10:33 | |
openstackgerrit | Zang MingJie proposed a change to openstack/neutron: db migration failed when plugin name alias used in config file https://review.openstack.org/81221 | 10:43 |
openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Fix enums usage for postgres in migrations https://review.openstack.org/53112 | 10:43 |
*** WackoRobie has joined #openstack-neutron | 10:43 | |
openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: Remove validate_port_binding() and unbind_port() https://review.openstack.org/79511 | 10:44 |
*** Jabadia_ has joined #openstack-neutron | 10:47 | |
*** Jabadia has quit IRC | 10:47 | |
*** WackoRobie has quit IRC | 10:48 | |
*** baoli has joined #openstack-neutron | 10:48 | |
*** baoli has quit IRC | 10:48 | |
*** baoli has joined #openstack-neutron | 10:49 | |
*** yamahata has quit IRC | 10:49 | |
*** jistr|mtg has quit IRC | 11:05 | |
*** dvorkinista has joined #openstack-neutron | 11:12 | |
*** tomoe_ has quit IRC | 11:13 | |
*** rwsu has quit IRC | 11:13 | |
*** jgallard has quit IRC | 11:13 | |
*** amuller has joined #openstack-neutron | 11:14 | |
*** rwsu has joined #openstack-neutron | 11:14 | |
*** tomoe_ has joined #openstack-neutron | 11:14 | |
*** enikanorov has quit IRC | 11:14 | |
*** enikanorov has joined #openstack-neutron | 11:16 | |
*** dvorkinista has quit IRC | 11:16 | |
*** sphoorti_ has joined #openstack-neutron | 11:26 | |
*** gdubreui has quit IRC | 11:31 | |
*** mwagner_zzz has quit IRC | 11:35 | |
*** xianghui has quit IRC | 11:39 | |
*** leseb has quit IRC | 11:39 | |
*** leseb has joined #openstack-neutron | 11:40 | |
*** leseb_ has joined #openstack-neutron | 11:41 | |
*** leseb has quit IRC | 11:45 | |
*** sphoorti_ has quit IRC | 11:46 | |
*** yamahata has joined #openstack-neutron | 11:56 | |
*** sphoorti has quit IRC | 11:58 | |
*** tomoe_ has quit IRC | 12:00 | |
*** ramishra has quit IRC | 12:01 | |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: remove orphan haproxy instances on agent start https://review.openstack.org/81242 | 12:03 |
*** ramishra has joined #openstack-neutron | 12:03 | |
*** obondarev1 has joined #openstack-neutron | 12:06 | |
*** baoli has quit IRC | 12:07 | |
*** jp_at_hp has quit IRC | 12:07 | |
*** baoli has joined #openstack-neutron | 12:08 | |
*** yamahata_ has quit IRC | 12:08 | |
*** jp_at_hp has joined #openstack-neutron | 12:08 | |
*** yamahata_ has joined #openstack-neutron | 12:08 | |
*** obondarev has quit IRC | 12:09 | |
*** eezhova19 has quit IRC | 12:09 | |
*** nplanel has quit IRC | 12:09 | |
*** eezhova19 has joined #openstack-neutron | 12:09 | |
*** ajo has quit IRC | 12:09 | |
*** jgrimm has quit IRC | 12:09 | |
*** bada has quit IRC | 12:10 | |
*** bada has joined #openstack-neutron | 12:11 | |
*** dvorkinista has joined #openstack-neutron | 12:12 | |
*** dvorkinista has quit IRC | 12:17 | |
*** afazekas_ has quit IRC | 12:18 | |
*** nplanel has joined #openstack-neutron | 12:20 | |
*** jgrimm has joined #openstack-neutron | 12:20 | |
*** ajo has joined #openstack-neutron | 12:21 | |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup. https://review.openstack.org/80261 | 12:25 |
*** luqas has quit IRC | 12:26 | |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup. https://review.openstack.org/80261 | 12:30 |
*** vkozhukalov_ has quit IRC | 12:30 | |
*** afazekas has joined #openstack-neutron | 12:31 | |
*** dims has quit IRC | 12:31 | |
openstackgerrit | Paul Michali proposed a change to openstack/neutron: VPNaaS support for VPN service admin state change and reporting https://review.openstack.org/81124 | 12:31 |
*** afazekas has quit IRC | 12:32 | |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup. https://review.openstack.org/80261 | 12:33 |
*** dims has joined #openstack-neutron | 12:33 | |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: small cleanup in agent device driver interface https://review.openstack.org/81248 | 12:34 |
*** gongysh has quit IRC | 12:35 | |
*** bada_ has joined #openstack-neutron | 12:35 | |
ajo | obondarev1, ^ | 12:36 |
*** gongysh has joined #openstack-neutron | 12:36 | |
ajo | doesn't it require the same change in the load balancer implementations? | 12:36 |
obondarev1 | ajo: which implementations? | 12:37 |
ajo | obondarev1, I don't actually know that part :) | 12:37 |
obondarev1 | ajo: do you mean drivers? | 12:37 |
ajo | but, as they were abstract methods, I suppose there must be drivers/actual implementations | 12:37 |
obondarev1 | ajo: ok, currently this change affects only haproxy namespace driver as it is the only implemented agent device driver | 12:38 |
obondarev1 | ajo: in haproxy driver these methods are overriden without 'context' param | 12:38 |
ajo | aha, and the change doesn't need to remove the 'context' also form the haproxy implementation? | 12:38 |
ajo | ahhh | 12:38 |
ajo | ok | 12:38 |
ajo | understood :) | 12:38 |
ajo | form->from | 12:39 |
*** bada has quit IRC | 12:39 | |
obondarev1 | ajo: so it was kind od typo in abstract driver | 12:39 |
ajo | it's always good to cleanup :) | 12:39 |
obondarev1 | yeah) | 12:40 |
*** xuhanp has joined #openstack-neutron | 12:42 | |
*** salv-orlando has quit IRC | 12:43 | |
openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: WIP Implement test https://review.openstack.org/76520 | 12:43 |
openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: WIP Add testing for database from oslo https://review.openstack.org/76519 | 12:43 |
*** carl_baldwin_ has joined #openstack-neutron | 12:43 | |
*** carl_baldwin has quit IRC | 12:45 | |
*** carl_baldwin_ is now known as carl_baldwin | 12:45 | |
*** amotoki has joined #openstack-neutron | 12:47 | |
*** vkozhukalov_ has joined #openstack-neutron | 12:50 | |
*** markmcclain has joined #openstack-neutron | 12:55 | |
*** jecarey has joined #openstack-neutron | 12:57 | |
*** yfried1 has joined #openstack-neutron | 12:58 | |
*** yfried has quit IRC | 12:59 | |
*** heyongli has joined #openstack-neutron | 12:59 | |
*** thuc has joined #openstack-neutron | 12:59 | |
*** roeyc has joined #openstack-neutron | 12:59 | |
*** thuc_ has joined #openstack-neutron | 13:00 | |
*** jgallard has joined #openstack-neutron | 13:02 | |
*** thuc has quit IRC | 13:04 | |
*** yfried has joined #openstack-neutron | 13:11 | |
*** yfried1 has quit IRC | 13:13 | |
*** dvorkinista has joined #openstack-neutron | 13:13 | |
*** changbl has quit IRC | 13:14 | |
*** julim has joined #openstack-neutron | 13:14 | |
*** ramishra has quit IRC | 13:14 | |
*** salv-orlando has joined #openstack-neutron | 13:15 | |
obondarev1 | markmcclain: ping | 13:16 |
*** yfried1 has joined #openstack-neutron | 13:16 | |
*** obondarev1 is now known as obondarev | 13:18 | |
*** yfried has quit IRC | 13:19 | |
*** dvorkinista has quit IRC | 13:20 | |
markmcclain | obondarev: pong | 13:20 |
obondarev | markmcclain: a question regarding https://review.openstack.org/#/c/66899/ | 13:20 |
obondarev | markmcclain: is it deferred for Juno? | 13:21 |
markmcclain | yes… I'll update the comment on that review | 13:21 |
obondarev | markmcclain: gotcha | 13:22 |
*** peristeri has joined #openstack-neutron | 13:29 | |
amotoki | markmcclain: ping | 13:35 |
markmcclain | amotoki: pong | 13:36 |
amotoki | sorry 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 |
amotoki | markmcclain: what do you think about oslo sync. | 13:36 |
amotoki | markmcclain: there is a change in oslo db.api and we need some changes in neutron.db code. | 13:37 |
markmcclain | amotoki: we'll have to target only limited portions to sync | 13:37 |
markmcclain | we had change sneak into the neutron.openstack.common rpc that prevents us from sync'ing the rpc tree | 13:38 |
amotoki | markmcclain: I got it. I will investigate each of the changes in oslo and will propose sync if needed. | 13:39 |
markmcclain | sounds good… I think db is the only real portion of the tree that has not been sync'ed recently | 13:40 |
*** jecarey has quit IRC | 13:41 | |
amotoki | markmcclain: 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 IRC | 13:42 | |
markmcclain | yeah.. the late merge does concern me, so let me know if anything stands out as risy | 13:42 |
markmcclain | *risky | 13:42 |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: remove orphan haproxy instances on agent start https://review.openstack.org/81242 | 13:43 |
*** mwagner_lap has joined #openstack-neutron | 13:43 | |
amotoki | markmcclain: sure. | 13:43 |
*** aveiga has joined #openstack-neutron | 13:44 | |
*** yfried1 has quit IRC | 13:45 | |
*** yfried has joined #openstack-neutron | 13:50 | |
*** dvorkinista has joined #openstack-neutron | 13:50 | |
*** sbadia has joined #openstack-neutron | 13:51 | |
*** leseb_ has quit IRC | 13:51 | |
*** leseb has joined #openstack-neutron | 13:51 | |
*** Jabadia_ has quit IRC | 13:53 | |
*** Jabadia has joined #openstack-neutron | 13:53 | |
*** jp_at_hp has quit IRC | 13:53 | |
*** afazekas has joined #openstack-neutron | 13:54 | |
*** tomoe_ has joined #openstack-neutron | 13:56 | |
*** sbadia has quit IRC | 13:56 | |
*** thuc_ has quit IRC | 13:57 | |
*** carl_baldwin has quit IRC | 13:57 | |
*** thuc has joined #openstack-neutron | 13:57 | |
*** jp_at_hp has joined #openstack-neutron | 13:59 | |
*** banix has joined #openstack-neutron | 13:59 | |
*** sbadia has joined #openstack-neutron | 13:59 | |
*** otherwiseguy has joined #openstack-neutron | 14:00 | |
*** heyongli has quit IRC | 14:00 | |
*** thuc has quit IRC | 14:02 | |
*** shshang has joined #openstack-neutron | 14:02 | |
*** luqas has joined #openstack-neutron | 14:03 | |
*** leseb has quit IRC | 14:04 | |
*** leseb has joined #openstack-neutron | 14:04 | |
*** salv-orlando has quit IRC | 14:08 | |
openstackgerrit | Dane LeBlanc proposed a change to openstack/neutron: Cisco Nexus: maximum recursion error in ConnectionContext.__del__ https://review.openstack.org/80737 | 14:10 |
*** jobewan has joined #openstack-neutron | 14:10 | |
*** vkozhukalov_ has quit IRC | 14:12 | |
*** amuller has quit IRC | 14:12 | |
*** dvorkinista has quit IRC | 14:14 | |
openstackgerrit | Zang MingJie proposed a change to openstack/neutron: Ensure host_routes order in subnet https://review.openstack.org/78946 | 14:16 |
*** amuller has joined #openstack-neutron | 14:19 | |
*** alexpilotti has joined #openstack-neutron | 14:21 | |
*** thuc has joined #openstack-neutron | 14:22 | |
*** overlayer has joined #openstack-neutron | 14:23 | |
*** thuc has quit IRC | 14:23 | |
*** thuc has joined #openstack-neutron | 14:24 | |
*** vkozhukalov_ has joined #openstack-neutron | 14:25 | |
*** dguitarbite has quit IRC | 14:31 | |
*** thuc has quit IRC | 14:34 | |
*** thuc has joined #openstack-neutron | 14:34 | |
*** dguitarbite has joined #openstack-neutron | 14:36 | |
*** jecarey has joined #openstack-neutron | 14:38 | |
*** thuc has quit IRC | 14:39 | |
*** thedodd has joined #openstack-neutron | 14:41 | |
*** gongysh has quit IRC | 14:42 | |
*** otherwiseguy has quit IRC | 14:44 | |
*** carl_baldwin has joined #openstack-neutron | 14:49 | |
openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Add update binding:profile with physicla _network https://review.openstack.org/81281 | 14:51 |
*** coolsvap has joined #openstack-neutron | 14:51 | |
*** carl_baldwin has quit IRC | 14:52 | |
*** irenab has quit IRC | 14:53 | |
*** otherwiseguy has joined #openstack-neutron | 14:56 | |
*** alagalah has joined #openstack-neutron | 14:59 | |
*** thuc has joined #openstack-neutron | 14:59 | |
*** afazekas has quit IRC | 15:00 | |
*** shshang has quit IRC | 15:00 | |
sc68cal | xuhanp: I'll e-mail you a draft | 15:00 |
sc68cal | before I post it to the ML, since I'm sort of talking about your patch | 15:01 |
sc68cal | but I didn't write it :) | 15:01 |
*** skraynev is now known as skraynev_afk | 15:01 | |
*** alagalah has quit IRC | 15:01 | |
*** thuc has quit IRC | 15:01 | |
*** alagalah has joined #openstack-neutron | 15:01 | |
*** thuc has joined #openstack-neutron | 15:02 | |
*** chandan_kumar has quit IRC | 15:02 | |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup https://review.openstack.org/80261 | 15:03 |
*** chandan_kumar has joined #openstack-neutron | 15:04 | |
*** cgoncalves has quit IRC | 15:07 | |
*** cgoncalves has joined #openstack-neutron | 15:07 | |
*** networkstatic_zZ is now known as networkstatic | 15:09 | |
JoeJulian | I 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-neutron | 15:12 | |
*** orion195 has joined #openstack-neutron | 15:13 | |
orion195 | Hi guys, on a newly installed openstack, I get this: ERROR 1146 (42S02) at line 1: Table 'neutron.migrate_version' doesn't exist | 15:14 |
orion195 | while running openstack-db --init --service neutron | 15:14 |
orion195 | installation done using the openstack manua, no packstack | 15:15 |
*** thuc has quit IRC | 15:15 | |
sc68cal | baoli: When you did that neutron subnet and router create | 15:19 |
sc68cal | was that before the force_gateway_on_subnet flag was introduced? | 15:19 |
*** Jabadia has quit IRC | 15:20 | |
*** Jabadia has joined #openstack-neutron | 15:20 | |
*** vkozhukalov_ has quit IRC | 15:22 | |
*** networkstatic has quit IRC | 15:22 | |
*** saju_m has quit IRC | 15:23 | |
xuhanp | sc68cal, thanks for letting me know about the email. I will have a look | 15:23 |
*** Jabadia has quit IRC | 15:25 | |
*** xuhanp has quit IRC | 15:25 | |
ram5391 | Hi, 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.conf | 15:26 |
*** networkstatic has joined #openstack-neutron | 15:26 | |
markmcclain | orion195: openstack-db is not used by Neutron | 15:30 |
markmcclain | ram5391: depends where is the neutronclient being invoked? | 15:31 |
*** vkozhukalov_ has joined #openstack-neutron | 15:33 | |
ram5391 | markmcclain: 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 https | 15:33 |
*** markwash has joined #openstack-neutron | 15:36 | |
*** devlaps has joined #openstack-neutron | 15:37 | |
*** yamahata has quit IRC | 15:39 | |
*** yamahata has joined #openstack-neutron | 15:39 | |
markmcclain | ram5391 not sure I'm understanding | 15:39 |
markmcclain | last response is what? | 15:40 |
*** saju_m has joined #openstack-neutron | 15:40 | |
*** yfried has quit IRC | 15:43 | |
openstackgerrit | Sean M. Collins proposed a change to openstack/neutron: Advanced Services documentation https://review.openstack.org/80825 | 15:45 |
ram5391 | markmcclain: sorry, I'm in the process of rebuilding all my relationships. That may fix the problem | 15:47 |
*** overlayer has quit IRC | 15:47 | |
*** overlayer has joined #openstack-neutron | 15:47 | |
*** markwash has quit IRC | 15:51 | |
*** thuc has joined #openstack-neutron | 15:56 | |
*** rupsky has joined #openstack-neutron | 15:56 | |
*** saju_m has quit IRC | 15:57 | |
*** thuc_ has joined #openstack-neutron | 15:57 | |
*** rupsky has quit IRC | 15:58 | |
*** sfox has joined #openstack-neutron | 15:59 | |
*** jorgem has joined #openstack-neutron | 16:00 | |
*** cgoncalves has quit IRC | 16:00 | |
*** thuc has quit IRC | 16:01 | |
*** cgoncalves has joined #openstack-neutron | 16:02 | |
*** cgoncalves has quit IRC | 16:03 | |
*** cgoncalves has joined #openstack-neutron | 16:04 | |
*** krtaylor has quit IRC | 16:04 | |
*** sbadia has quit IRC | 16:05 | |
*** dvorkinista has joined #openstack-neutron | 16:05 | |
*** blogan has joined #openstack-neutron | 16:07 | |
*** sbalukoff has quit IRC | 16:08 | |
*** SumitNaiksatam has quit IRC | 16:11 | |
*** _cjones_ has joined #openstack-neutron | 16:13 | |
*** sfox has quit IRC | 16:16 | |
*** arosen1 has joined #openstack-neutron | 16:17 | |
*** spandhe has joined #openstack-neutron | 16:20 | |
*** sweston has joined #openstack-neutron | 16:22 | |
*** sfox has joined #openstack-neutron | 16:23 | |
*** vkozhukalov_ has quit IRC | 16:25 | |
sdague | so what's up with this in every single neutron log fie? | 16:29 |
sdague | 2014-03-18 13:59:27.701 3242 ERROR neutron.common.legacy [-] Skipping unknown group key: firewall_driver | 16:29 |
sdague | is that something that devstack is setting incorrectly? | 16:29 |
*** dvorkinista has quit IRC | 16:31 | |
*** chandankumar_ has joined #openstack-neutron | 16:34 | |
*** chandan_kumar has quit IRC | 16:37 | |
*** pete5 has quit IRC | 16:38 | |
anteaya | https://review.openstack.org/#/c/80989/ is failing in the gate | 16:42 |
*** dguitarbite has quit IRC | 16:42 | |
openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Add update binding:profile with physical_network https://review.openstack.org/81281 | 16:42 |
*** ijw has joined #openstack-neutron | 16:43 | |
*** networkstatic has quit IRC | 16:44 | |
*** networkstatic has joined #openstack-neutron | 16:44 | |
*** ijw has quit IRC | 16:45 | |
*** bada__ has joined #openstack-neutron | 16:45 | |
*** ijw has joined #openstack-neutron | 16:46 | |
*** bada_ has quit IRC | 16:48 | |
*** overlayer has quit IRC | 16:49 | |
*** overlayer has joined #openstack-neutron | 16:51 | |
openstackgerrit | Dane 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/81304 | 16:51 |
*** spandhe has quit IRC | 16:52 | |
*** harlowja_away is now known as harlowja | 16:53 | |
openstackgerrit | Édouard Thuleau proposed a change to openstack/neutron: OVS lib defer apply doesn't handle concurrency https://review.openstack.org/63917 | 16:54 |
openstackgerrit | Yves-Gwenael Bourhis proposed a change to openstack/neutron: Make dnsmasq aware of all names https://review.openstack.org/52930 | 16:54 |
*** arosen1 has quit IRC | 16:55 | |
*** spandhe has joined #openstack-neutron | 16:56 | |
*** arosen1 has joined #openstack-neutron | 16:56 | |
*** markwash has joined #openstack-neutron | 16:57 | |
*** overlayer has quit IRC | 16:58 | |
*** overlayer has joined #openstack-neutron | 16:59 | |
*** markwash has quit IRC | 17:01 | |
*** morganfainberg_Z is now known as morganfainberg | 17:01 | |
openstackgerrit | Dane 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/81304 | 17:02 |
*** SumitNaiksatam has joined #openstack-neutron | 17:03 | |
*** markwash has joined #openstack-neutron | 17:07 | |
openstackgerrit | Darragh O'Reilly proposed a change to openstack/neutron: lb-agent: reduce delete_vlan_bridge() logging severity https://review.openstack.org/75993 | 17:12 |
pba1 | Hello, 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 to | 17: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 IRC | 17:16 | |
*** amotoki has quit IRC | 17:17 | |
*** otherwiseguy has quit IRC | 17:17 | |
*** networkstatic has quit IRC | 17:17 | |
_cjones_ | pba1: You shouldn't need to restack. Just restart the Neutron service. | 17:18 |
pba1 | but even restacking isn't working | 17:20 |
*** roeyc has quit IRC | 17:21 | |
*** alagalah_ has joined #openstack-neutron | 17:22 | |
*** alagalah has quit IRC | 17:22 | |
*** rotbeard has quit IRC | 17:22 | |
*** devlaps1 has joined #openstack-neutron | 17: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 10 | 17:23 |
*** devlaps has quit IRC | 17:24 | |
_cjones_ | pba1: It should. Perhaps that code isn't being executed. | 17:24 |
_cjones_ | pba1: Bust out the debugger. | 17:24 |
pba1 | I 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 IRC | 17:24 | |
_cjones_ | pba1: I'm not that intimately familiar. I'd have to look. | 17:25 |
pba1 | or should I change also sowhere else | 17: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 |
pba1 | i did ;p | 17:26 |
pba1 | all of them ^^ | 17:26 |
pba1 | and new are not beeing generated | 17:26 |
pba1 | and everytinhg works | 17:27 |
_cjones_ | pba1: If they're not being generated, they're not being executed. | 17:27 |
pba1 | yes, so when exactly extensions are executed | 17:27 |
pba1 | should I load them manualy after changing them ? | 17:28 |
pba1 | somehow | 17:28 |
*** sbalukoff has joined #openstack-neutron | 17:28 | |
*** leseb has quit IRC | 17:30 | |
*** leseb has joined #openstack-neutron | 17:30 | |
*** thedodd has quit IRC | 17:31 | |
*** kashyap has left #openstack-neutron | 17:32 | |
cgoncalves | SumitNaiksatam: 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-neutron | 17:33 | |
cgoncalves | SumitNaiksatam: I can comment on the presentation doc if you prefer | 17:33 |
*** leseb has quit IRC | 17:34 | |
SumitNaiksatam | cgoncalves: hi | 17:34 |
SumitNaiksatam | cgoncalves: am in a meeting | 17:34 |
*** bada__ has quit IRC | 17:35 | |
*** aveiga has quit IRC | 17:35 | |
SumitNaiksatam | cgoncalves: good question | 17:36 |
*** ram5391 has quit IRC | 17:37 | |
cgoncalves | SumitNaiksatam: oh, sorry. we can chat a bit later or bring this question to the presentation doc | 17:37 |
*** arosen1 has joined #openstack-neutron | 17:37 | |
*** arosen1 is now known as arosen-home | 17:37 | |
SumitNaiksatam | cgoncalves: yeah sure | 17:37 |
*** morganfainberg is now known as morganfainberg_Z | 17:38 | |
SumitNaiksatam | cgoncalves: give me about 90 mins | 17:38 |
cgoncalves | SumitNaiksatam: ok, thanks | 17:38 |
*** geekinutah has quit IRC | 17:38 | |
*** aveiga has joined #openstack-neutron | 17:39 | |
*** geekinutah has joined #openstack-neutron | 17:40 | |
*** otherwiseguy has joined #openstack-neutron | 17:43 | |
jlibosva | markmcclain: hi, do you have a minute to discuss https://review.openstack.org/#/c/75924/ ? | 17:44 |
*** ygbo has quit IRC | 17:44 | |
*** crc32 has joined #openstack-neutron | 17:50 | |
*** sbadia has joined #openstack-neutron | 17:55 | |
*** jp_at_hp has quit IRC | 17:56 | |
*** morganfainberg_Z is now known as morganfainberg | 17:58 | |
*** aveiga has quit IRC | 18:00 | |
*** harlowja is now known as harlowja_away | 18:01 | |
openstackgerrit | Evgeny Fedoruk proposed a change to openstack/neutron: Cancelling thread start while unit tests running. https://review.openstack.org/81323 | 18:01 |
*** harlowja_away is now known as harlowja | 18:02 | |
*** devvesa has joined #openstack-neutron | 18:03 | |
sballe | I 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-neutron | 18:04 | |
geekinutah | sballe: join #openstack and I'll help you in there | 18:06 |
sballe | ok thx | 18:06 |
openstackgerrit | A change was merged to openstack/neutron: NSX: Ensure gateway devices are usable after upgrade https://review.openstack.org/80989 | 18:07 |
*** ijw has quit IRC | 18:07 | |
*** aveiga has joined #openstack-neutron | 18:10 | |
*** aveiga has quit IRC | 18:10 | |
markmcclain | jlibosva: yes | 18:15 |
*** suresh12 has joined #openstack-neutron | 18:15 | |
jlibosva | markmcclain: oh, I'm on a meeting right now, can I ping you later? (50 mins) | 18:16 |
markmcclain | sure | 18:17 |
jlibosva | thx | 18:17 |
*** shakamunyi has quit IRC | 18:18 | |
*** devvesa has quit IRC | 18:23 | |
*** D7 has quit IRC | 18:24 | |
*** shakamunyi has joined #openstack-neutron | 18:27 | |
*** amuller has quit IRC | 18:28 | |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup https://review.openstack.org/80261 | 18:28 |
*** ijw has joined #openstack-neutron | 18:30 | |
*** jlibosva has quit IRC | 18:31 | |
*** yfried1 has joined #openstack-neutron | 18:31 | |
*** _sweston_ has joined #openstack-neutron | 18:31 | |
*** jgallard has quit IRC | 18:33 | |
*** sweston has quit IRC | 18:33 | |
*** alagalah_ has quit IRC | 18:34 | |
*** ijw has quit IRC | 18:34 | |
*** ijw has joined #openstack-neutron | 18:36 | |
*** _sweston_ has quit IRC | 18:36 | |
*** luqas has quit IRC | 18:37 | |
openstackgerrit | John Eckersberg proposed a change to openstack/neutron: Use InnoDB storage engine when creating tables in migrations https://review.openstack.org/81334 | 18:42 |
*** thuc_ has quit IRC | 18:43 | |
*** thuc has joined #openstack-neutron | 18:44 | |
*** BuSerD has joined #openstack-neutron | 18:44 | |
*** thuc_ has joined #openstack-neutron | 18:45 | |
*** networkstatic has joined #openstack-neutron | 18:45 | |
*** overlayer has quit IRC | 18:47 | |
*** vkozhukalov_ has joined #openstack-neutron | 18:48 | |
*** thuc has quit IRC | 18:48 | |
*** shakamunyi has quit IRC | 18:49 | |
*** spandhe has quit IRC | 18:52 | |
*** spandhe has joined #openstack-neutron | 18:52 | |
*** alagalah has joined #openstack-neutron | 18:52 | |
*** jecarey has quit IRC | 18:55 | |
*** jecarey has joined #openstack-neutron | 18:55 | |
*** jlibosva has joined #openstack-neutron | 18:59 | |
sc68cal | pba1: what plugin are you running | 19:00 |
sc68cal | there should be a log line for each extension that the plugin loads | 19:00 |
sc68cal | when neutron server starts | 19:00 |
sc68cal | so look for a line that says the plugin loaded the extension you are modifying | 19:00 |
*** gdubreui has joined #openstack-neutron | 19:01 | |
*** krtaylor has joined #openstack-neutron | 19:03 | |
*** otherwiseguy has quit IRC | 19:11 | |
*** carl_baldwin has joined #openstack-neutron | 19:14 | |
*** arosen-home has quit IRC | 19:18 | |
*** saju_m has joined #openstack-neutron | 19:18 | |
*** arosen1 has joined #openstack-neutron | 19:18 | |
*** djoreilly has quit IRC | 19:21 | |
*** overlayer has joined #openstack-neutron | 19:24 | |
*** zzelle has joined #openstack-neutron | 19:25 | |
*** _sweston_ has joined #openstack-neutron | 19:30 | |
*** spandhe has quit IRC | 19:34 | |
*** samuelbercovici has joined #openstack-neutron | 19:34 | |
*** spandhe has joined #openstack-neutron | 19:35 | |
*** alagalah has quit IRC | 19:37 | |
blogan | markmcclain: when you have time can you look at https://review.openstack.org/#/c/80816/ its a really simple one | 19:39 |
*** salv-orlando has joined #openstack-neutron | 19:42 | |
*** nati_ueno has joined #openstack-neutron | 19:43 | |
*** dguitarbite has joined #openstack-neutron | 19:52 | |
*** julim has quit IRC | 20:02 | |
*** salv-orlando has quit IRC | 20:03 | |
*** sfox has quit IRC | 20:06 | |
*** ianw has joined #openstack-neutron | 20:11 | |
ianw | obondarev: re https://review.openstack.org/#/c/81173/ do you mean file a bug describing the debug output? | 20:12 |
openstackgerrit | Dane LeBlanc proposed a change to openstack/neutron: Cisco plugin fails with ParseError no elem found https://review.openstack.org/81304 | 20:13 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: ML2 BigSwitch: Don't modify parent context https://review.openstack.org/81360 | 20:14 |
*** dvorkinista has joined #openstack-neutron | 20:16 | |
*** salv-orlando has joined #openstack-neutron | 20:16 | |
*** leseb has joined #openstack-neutron | 20:16 | |
*** peristeri has quit IRC | 20:16 | |
*** jlibosva has quit IRC | 20:20 | |
*** singhs has joined #openstack-neutron | 20:31 | |
marun | rkukura: ping | 20:34 |
rkukura | marun: hi | 20:34 |
marun | rkukura: hi Bob | 20:34 |
marun | rkukura: I wanted to make sure I understood some of your comments re: migration script | 20:35 |
rkukura | marun: OK | 20:35 |
marun | rkukura: so re: port bindings, migrate port id and host, and set vif type to UNBOUND? | 20:36 |
marun | rkukura: 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 |
rkukura | marun: That may be the way to go, provided that the patch I'm working on gets merged. | 20:37 |
marun | rkukura: what to do if it doesn't? | 20:37 |
marun | rkukura: or should the migration patch also not merge? | 20:37 |
rkukura | If 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 |
marun | rkukura: erg | 20:39 |
marun | markmcclain: ^^ | 20:39 |
*** alagalah has joined #openstack-neutron | 20:40 | |
rkukura | marun: 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 IRC | 20:41 | |
marun | rkukura: fair enough, I'll trust you on that. | 20:41 |
marun | rkukura: is there a reason that your patch won't land in icehouse? | 20:42 |
*** marun has quit IRC | 20:42 | |
rkukura | I don't see any reason, other than running out of time. | 20:42 |
*** marun has joined #openstack-neutron | 20:45 | |
*** dave_tucker is now known as dave_tucker_zzz | 20:46 | |
marun | rkukura: sorry, lost my connection | 20:46 |
rkukura | marun: Just replied that I don't see any reason my fix won't make it, unless we run out of time. | 20:46 |
marun | rkukura: ok. | 20:47 |
rkukura | Trying to complete it today. | 20:47 |
marun | rkukura: so I should rebase on your patch to ensure an explicit dependency | 20:47 |
*** tvardeman has joined #openstack-neutron | 20:48 | |
*** tvardeman_ has joined #openstack-neutron | 20:48 | |
*** ijw has quit IRC | 20:48 | |
*** dave_tucker_zzz is now known as dave_tucker | 20:48 | |
*** BuSerD has quit IRC | 20:49 | |
markmcclain | marun: -1 on that | 20:50 |
rkukura | marun: Probably. I'd suggest making the other updates and letting the review continue on yours in the mean time. | 20:50 |
marun | markmcclain: ok | 20:50 |
markmcclain | we should keep them independent as one is a bug and the other is a blueprint | 20:50 |
markmcclain | I'd rather merge an imperfect migration that we can clean up with bob's patch | 20:51 |
marun | markmcclain: so is there another way to ensure correct migration of port bindings? | 20:51 |
markmcclain | rkukura knows that stuff better than me | 20:51 |
*** jlibosva has joined #openstack-neutron | 20:51 | |
marun | markmcclain: ok | 20:53 |
marun | rkukura: 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-neutron | 20:53 | |
*** BuSerD has joined #openstack-neutron | 20:54 | |
rkukura | marun, 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 |
rkukura | Without that, the get_device_details RPC from the L2 agent won't return the info needed. | 20:55 |
jlibosva | markmcclain: hi, I accidentally wrote you on openstack-meeting | 20:56 |
marun | rkukura: what do you mean by '(only)'? | 20:56 |
*** salv-orlando has quit IRC | 20:56 | |
rkukura | So things would continue to work, but if an L2 agent was restarted, they would break. | 20:56 |
* jlibosva is not used on this irc client | 20:56 | |
rkukura | marun: 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 IRC | 20:57 | |
*** ijw has joined #openstack-neutron | 21:00 | |
*** ijw_ has joined #openstack-neutron | 21:01 | |
markmcclain | jlibosva: I'll be multitasking in the project meeting.. what's up? | 21:01 |
jlibosva | markmcclain: I just wanted to ask about https://review.openstack.org/#/c/75924/ but we can discuss this tomorrow (again :)) ) if you want | 21:03 |
openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Add update binding:profile with physical_network https://review.openstack.org/81281 | 21:03 |
*** spandhe has quit IRC | 21:03 | |
marun | rkukura: ah, so it should be easy enough to ensure that the bindings have the required segment id | 21:04 |
markmcclain | jlibosva: ok so what did you want to discuss | 21:04 |
*** ijw has quit IRC | 21:04 | |
*** manishg_ has joined #openstack-neutron | 21:05 | |
jlibosva | markmcclain: 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 lbaas | 21:05 |
jlibosva | markmcclain: the only difference is that the one between folsom and grizzly is never used because is missing plugin dependency | 21:05 |
*** spandhe has joined #openstack-neutron | 21:05 | |
*** spandhe has quit IRC | 21:05 | |
markmcclain | yeah we can't modify the timeline because those tracking trunk | 21:05 |
markmcclain | or on a release version do not get the modifications | 21:06 |
jlibosva | markmcclain: 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 it | 21:06 |
rkukura | marun: right, so maybe go with that for now, and then we can revisit later | 21:06 |
*** zhipeng has joined #openstack-neutron | 21:07 | |
markmcclain | but it also doesn't cause any benefit for those stuck on other points of the timeline | 21:07 |
*** spandhe has joined #openstack-neutron | 21:07 | |
*** singhs has quit IRC | 21:07 | |
markmcclain | I was considering spending sometime discussing in Atlanta on how we can clarify our migration editing policy | 21:08 |
*** manishg has quit IRC | 21:08 | |
*** manishg_ is now known as manishg | 21:08 | |
marun | rkukura: ok | 21:08 |
jlibosva | markmcclain: 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 again | 21:08 |
jlibosva | markmcclain: +1 on discussing it | 21:09 |
rkukura | marun: thanks | 21:09 |
*** banix_ has joined #openstack-neutron | 21:11 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BigSwitch ML2: Include bound_segment in port https://review.openstack.org/81367 | 21:12 |
*** dims has quit IRC | 21:12 | |
markmcclain | jlibosva: right.. need to revisit the implications of offline migrating | 21:13 |
*** ajo has joined #openstack-neutron | 21:16 | |
*** banix has quit IRC | 21:16 | |
*** banix_ is now known as banix | 21:16 | |
*** banix has quit IRC | 21:17 | |
jlibosva | I 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_zzz | 21:21 | |
*** zhipeng has quit IRC | 21:23 | |
*** dims has joined #openstack-neutron | 21:26 | |
*** blogan has quit IRC | 21:30 | |
*** krtaylor has quit IRC | 21:31 | |
*** jlibosva has quit IRC | 21:35 | |
*** leseb has quit IRC | 21:38 | |
*** zhipeng has joined #openstack-neutron | 21:41 | |
*** saju_m has quit IRC | 21:42 | |
*** _cjones_ has quit IRC | 21:45 | |
*** _cjones_ has joined #openstack-neutron | 21:46 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BigSwitch ML2: Include bound_segment in port https://review.openstack.org/81367 | 21:47 |
*** thuc_ has quit IRC | 21:48 | |
*** ijw_ has quit IRC | 21:50 | |
*** _cjones_ has quit IRC | 21:50 | |
*** spandhe has quit IRC | 21:51 | |
*** alagalah has joined #openstack-neutron | 21:51 | |
*** dave_tucker_zzz is now known as dave_tucker | 21:51 | |
*** markwash_ has joined #openstack-neutron | 21:52 | |
openstackgerrit | Nachi Ueno proposed a change to openstack/neutron: Add enable_security_group option https://review.openstack.org/67281 | 21:54 |
*** markwash has quit IRC | 21:54 | |
*** markwash_ is now known as markwash | 21:54 | |
*** salv-orlando has joined #openstack-neutron | 21:55 | |
*** alagalah has quit IRC | 21:57 | |
*** jgrimm has quit IRC | 21:57 | |
openstackgerrit | Claudiu Belu proposed a change to openstack/neutron: Fixes Hyper-V agent security groups enable issue https://review.openstack.org/81382 | 21:58 |
*** spandhe has joined #openstack-neutron | 22:00 | |
*** networkstatic has quit IRC | 22:04 | |
*** _sweston_ is now known as sweston | 22:06 | |
oda-g | Hi core reviewers, would you pick up to review https://review.openstack.org/#/c/79858/ which needs one more +2 ? | 22:07 |
*** jecarey has quit IRC | 22:08 | |
*** claudiub has joined #openstack-neutron | 22:09 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow CIDRs with non-zero masked portions https://review.openstack.org/81137 | 22:09 |
openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: NEC plugin: Rename quantum_id column to neutron_id https://review.openstack.org/78854 | 22:12 |
*** ajo has quit IRC | 22:12 | |
*** pcm_ has quit IRC | 22:14 | |
*** markmcclain1 has joined #openstack-neutron | 22:17 | |
*** markmcclain has quit IRC | 22:17 | |
*** leseb has joined #openstack-neutron | 22:17 | |
*** leseb has quit IRC | 22:19 | |
*** leseb has joined #openstack-neutron | 22:19 | |
*** mwagner_lap has quit IRC | 22:21 | |
*** markmcclain1 has quit IRC | 22:24 | |
*** jobewan has quit IRC | 22:28 | |
*** tvardeman_ has quit IRC | 22:30 | |
*** tvardeman has joined #openstack-neutron | 22:30 | |
*** banix has joined #openstack-neutron | 22:31 | |
*** tvardeman has quit IRC | 22:32 | |
*** BuSerD has quit IRC | 22:35 | |
*** _cjones_ has joined #openstack-neutron | 22:39 | |
*** thuc has joined #openstack-neutron | 22:40 | |
*** banix has quit IRC | 22:40 | |
*** BuSerD has joined #openstack-neutron | 22:52 | |
*** alagalah has joined #openstack-neutron | 22:53 | |
*** vkozhukalov_ has quit IRC | 22:56 | |
*** zhipeng has quit IRC | 22:58 | |
oda-g | Hi, is there somebody core reviewer ? | 22:59 |
*** dvorkini_ has joined #openstack-neutron | 22:59 | |
*** devlaps has joined #openstack-neutron | 23:00 | |
*** dvorkinista has quit IRC | 23:02 | |
*** devlaps1 has quit IRC | 23:02 | |
*** nati_ueno has quit IRC | 23:07 | |
oda-g | nati_ueno: ping | 23:07 |
*** claudiub has quit IRC | 23:08 | |
*** leseb has quit IRC | 23:11 | |
openstackgerrit | Claudiu Belu proposed a change to openstack/neutron: Fixes the Hyper-V agent individual ports metrics https://review.openstack.org/78795 | 23:11 |
*** leseb has joined #openstack-neutron | 23:12 | |
*** crc32 has quit IRC | 23:13 | |
*** leseb has quit IRC | 23:16 | |
*** alagalah has quit IRC | 23:19 | |
*** zhipeng has joined #openstack-neutron | 23:22 | |
*** alagalah has joined #openstack-neutron | 23:22 | |
*** changbl has quit IRC | 23:24 | |
openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Don't use root to list namespaces https://review.openstack.org/81098 | 23:26 |
*** blogan has joined #openstack-neutron | 23:27 | |
*** dvorkinista has joined #openstack-neutron | 23:28 | |
*** BuSerD has quit IRC | 23:30 | |
*** dvorkini_ has quit IRC | 23:31 | |
*** spandhe has quit IRC | 23:31 | |
*** krtaylor has joined #openstack-neutron | 23:31 | |
*** spandhe has joined #openstack-neutron | 23:33 | |
*** blogan has quit IRC | 23:37 | |
*** dvorkini_ has joined #openstack-neutron | 23:47 | |
*** carl_baldwin has quit IRC | 23:48 | |
*** dvorkinista has quit IRC | 23:48 | |
*** networkstatic has joined #openstack-neutron | 23:51 | |
*** alexpilotti has quit IRC | 23:52 | |
*** zzelle has quit IRC | 23:53 | |
openstackgerrit | Brandon Logan proposed a change to openstack/neutron: Added config value help text in ns metadata proxy https://review.openstack.org/80816 | 23:54 |
*** blogan has joined #openstack-neutron | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!