| *** rossella_s has quit IRC | 00:00 | |
| *** rossella_s has joined #openstack-neutron | 00:00 | |
| *** lujinluo has joined #openstack-neutron | 00:04 | |
| *** nlahouti1 has joined #openstack-neutron | 00:04 | |
| *** nlahouti has quit IRC | 00:04 | |
| *** nlahouti1 has quit IRC | 00:06 | |
| *** nlahouti has joined #openstack-neutron | 00:06 | |
| *** singhj has quit IRC | 00:08 | |
| *** abhiraut has quit IRC | 00:10 | |
| *** nlahouti has quit IRC | 00:12 | |
| *** nlahouti has joined #openstack-neutron | 00:12 | |
| *** sbalukoff has quit IRC | 00:12 | |
| *** wolverineav has quit IRC | 00:12 | |
| *** wolverineav has joined #openstack-neutron | 00:13 | |
| *** iyamahat has quit IRC | 00:14 | |
| *** sputnik13 has quit IRC | 00:14 | |
| *** iyamahat has joined #openstack-neutron | 00:14 | |
| *** wolverineav has quit IRC | 00:14 | |
| openstackgerrit | Aradhana Singh proposed openstack/neutron: Refactoring config options for cmd https://review.openstack.org/323046 | 00:14 |
|---|---|---|
| *** wolverineav has joined #openstack-neutron | 00:15 | |
| *** fnaval has quit IRC | 00:15 | |
| *** yamahata has quit IRC | 00:17 | |
| openstackgerrit | Adolfo Duarte proposed openstack/neutron: Add RPC layer for Trunk Plugin and initial Open vSwitch driver https://review.openstack.org/347662 | 00:17 |
| *** zhhuabj has quit IRC | 00:18 | |
| *** mattgreene has quit IRC | 00:23 | |
| *** yamamoto_ has joined #openstack-neutron | 00:25 | |
| *** ircuser-1 has quit IRC | 00:28 | |
| *** yamamoto_ has quit IRC | 00:31 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Increment revision numbers on object changes https://review.openstack.org/303966 | 00:34 |
| *** fnaval has joined #openstack-neutron | 00:35 | |
| openstackgerrit | Lujin Luo proposed openstack/neutron: Add a unique key to port_id in routerports table https://review.openstack.org/285048 | 00:36 |
| *** zhhuabj has joined #openstack-neutron | 00:37 | |
| *** fnaval has quit IRC | 00:37 | |
| *** fnaval has joined #openstack-neutron | 00:37 | |
| *** jamesden_ has quit IRC | 00:38 | |
| openstackgerrit | Lujin Luo proposed openstack/neutron: Add a composite unique key to floatingip table in Neutron database https://review.openstack.org/276018 | 00:39 |
| *** Swami has quit IRC | 00:41 | |
| *** vijaykc4 has joined #openstack-neutron | 00:41 | |
| *** mfuruta has joined #openstack-neutron | 00:41 | |
| *** emagana has joined #openstack-neutron | 00:43 | |
| *** jckasper has quit IRC | 00:43 | |
| *** jckasper has joined #openstack-neutron | 00:44 | |
| *** hoangcx has joined #openstack-neutron | 00:44 | |
| *** andymaier has quit IRC | 00:44 | |
| *** yamahata has joined #openstack-neutron | 00:46 | |
| *** emagana has quit IRC | 00:47 | |
| *** sbalukoff has joined #openstack-neutron | 00:48 | |
| *** thorst has joined #openstack-neutron | 00:48 | |
| *** jckasper has quit IRC | 00:49 | |
| *** vijaykc4 has quit IRC | 00:49 | |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Refactor Router Db Models https://review.openstack.org/348562 | 00:49 |
| *** jamielennox is now known as jamielennox|away | 00:49 | |
| *** trananhkma has joined #openstack-neutron | 00:51 | |
| *** mlavalle has quit IRC | 00:53 | |
| *** tonytan4ever has joined #openstack-neutron | 00:53 | |
| *** nlahouti has quit IRC | 00:53 | |
| *** thorst has quit IRC | 00:54 | |
| *** emagana has joined #openstack-neutron | 00:54 | |
| *** thorst has joined #openstack-neutron | 00:54 | |
| *** yb has quit IRC | 00:55 | |
| *** abhiraut has joined #openstack-neutron | 00:56 | |
| *** fnaval has quit IRC | 00:57 | |
| *** tonytan4ever has quit IRC | 00:58 | |
| *** iyamahat has quit IRC | 00:58 | |
| *** iyamahat has joined #openstack-neutron | 00:58 | |
| *** hoangcx2 has quit IRC | 00:58 | |
| *** thorst has quit IRC | 00:59 | |
| *** elo has quit IRC | 00:59 | |
| openstackgerrit | Merged openstack/neutron: Wait for vswitchd to add interfaces in native ovsdb https://review.openstack.org/344859 | 00:59 |
| *** emagana has quit IRC | 01:01 | |
| *** abhiraut has quit IRC | 01:02 | |
| *** abhiraut has joined #openstack-neutron | 01:02 | |
| *** djan_ has joined #openstack-neutron | 01:03 | |
| *** djan has quit IRC | 01:04 | |
| openstackgerrit | gengchc2 proposed openstack/neutron: modify the home-page info with the developer documentation https://review.openstack.org/348598 | 01:06 |
| *** lujinluo has quit IRC | 01:06 | |
| *** lujinluo has joined #openstack-neutron | 01:06 | |
| *** wolverin_ has joined #openstack-neutron | 01:08 | |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Refactor Router Db Models https://review.openstack.org/348562 | 01:10 |
| *** yamahata has quit IRC | 01:10 | |
| *** itisha has quit IRC | 01:10 | |
| *** wolverineav has quit IRC | 01:11 | |
| *** elo has joined #openstack-neutron | 01:11 | |
| *** yb has joined #openstack-neutron | 01:11 | |
| *** s3wong has quit IRC | 01:12 | |
| *** wolverin_ has quit IRC | 01:12 | |
| *** vijaykc4 has joined #openstack-neutron | 01:15 | |
| *** namnh has joined #openstack-neutron | 01:17 | |
| *** johnbelamaric has joined #openstack-neutron | 01:25 | |
| openstackgerrit | li,chen proposed openstack/neutron-vpnaas: Fix API Tests https://review.openstack.org/335838 | 01:25 |
| *** yamamoto_ has joined #openstack-neutron | 01:27 | |
| *** bvandewa has quit IRC | 01:28 | |
| *** abhiraut has quit IRC | 01:28 | |
| *** banix has quit IRC | 01:28 | |
| *** iyamahat has quit IRC | 01:29 | |
| *** iyamahat has joined #openstack-neutron | 01:29 | |
| *** yamamoto_ has quit IRC | 01:32 | |
| *** tidwellr has left #openstack-neutron | 01:33 | |
| *** tonytan4ever has joined #openstack-neutron | 01:34 | |
| *** jamielennox|away is now known as jamielennox | 01:34 | |
| *** kevo has quit IRC | 01:36 | |
| *** vijaykc4 has quit IRC | 01:40 | |
| *** vijaykc4 has joined #openstack-neutron | 01:40 | |
| *** vijaykc4 has quit IRC | 01:41 | |
| *** abhiraut has joined #openstack-neutron | 01:42 | |
| *** EinstCrazy has joined #openstack-neutron | 01:44 | |
| *** yamahata has joined #openstack-neutron | 01:44 | |
| *** EinstCrazy has quit IRC | 01:45 | |
| *** EinstCrazy has joined #openstack-neutron | 01:45 | |
| *** janzian has quit IRC | 01:46 | |
| *** gvrangan has quit IRC | 01:51 | |
| *** vijaykc4 has joined #openstack-neutron | 01:56 | |
| *** thorst has joined #openstack-neutron | 02:04 | |
| *** thorst has quit IRC | 02:05 | |
| *** thorst has joined #openstack-neutron | 02:05 | |
| *** thorst has quit IRC | 02:10 | |
| namnh | zzzeek: Hello Mr.Mike, are you around? I have just answered your comment. Could you please take a look at it. | 02:10 |
| *** lujinluo has quit IRC | 02:11 | |
| *** lujinluo has joined #openstack-neutron | 02:11 | |
| *** yamahata has quit IRC | 02:15 | |
| *** Leo_ has joined #openstack-neutron | 02:15 | |
| *** iyamahat has quit IRC | 02:16 | |
| namnh | zzzeek: ^^ Here is our patch: https://review.openstack.org/#/c/314054/29 | 02:19 |
| *** Leom has joined #openstack-neutron | 02:20 | |
| *** Leo_ has quit IRC | 02:23 | |
| *** djan has joined #openstack-neutron | 02:23 | |
| *** xinliang has quit IRC | 02:24 | |
| *** jckasper has joined #openstack-neutron | 02:24 | |
| *** elopez has joined #openstack-neutron | 02:25 | |
| *** djan_ has quit IRC | 02:25 | |
| *** elopez is now known as Guest78543 | 02:25 | |
| *** Guest78543 has quit IRC | 02:26 | |
| *** yamamoto_ has joined #openstack-neutron | 02:28 | |
| *** abhiraut has quit IRC | 02:28 | |
| openstackgerrit | Anh Tran proposed openstack/neutron: Prevent duplicate SG rules in 'concurrent requests' case https://review.openstack.org/345978 | 02:31 |
| *** jamielennox is now known as jamielennox|away | 02:32 | |
| *** yamamoto_ has quit IRC | 02:34 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/348678 | 02:34 |
| openstackgerrit | Darek Smigiel proposed openstack/neutron: Rename DB columns: tenant -> project https://review.openstack.org/335786 | 02:38 |
| *** veena has joined #openstack-neutron | 02:38 | |
| *** veena has quit IRC | 02:40 | |
| *** tonytan4ever has quit IRC | 02:41 | |
| *** itzikb has joined #openstack-neutron | 02:42 | |
| openstackgerrit | Darek Smigiel proposed openstack/neutron: Added Deprecation Warning to models https://review.openstack.org/335792 | 02:42 |
| *** jhershbe has joined #openstack-neutron | 02:42 | |
| *** banix has joined #openstack-neutron | 02:46 | |
| *** claudiub|2 has quit IRC | 02:46 | |
| *** a_ta has joined #openstack-neutron | 02:47 | |
| *** gouthamr_ has quit IRC | 02:51 | |
| *** zhhuabj_ has joined #openstack-neutron | 02:52 | |
| *** zhhuabj has quit IRC | 02:53 | |
| *** xinliang has joined #openstack-neutron | 02:54 | |
| *** MadOtis has joined #openstack-neutron | 02:55 | |
| *** vijaykc4 has quit IRC | 02:55 | |
| *** jckasper has quit IRC | 02:56 | |
| *** elo has quit IRC | 02:56 | |
| *** jckasper has joined #openstack-neutron | 02:56 | |
| *** Leom has quit IRC | 02:56 | |
| *** yuanying_ has quit IRC | 02:57 | |
| *** elo has joined #openstack-neutron | 02:59 | |
| *** jckasper has quit IRC | 03:01 | |
| *** abhiraut has joined #openstack-neutron | 03:05 | |
| *** abhiraut has quit IRC | 03:06 | |
| *** jckasper has joined #openstack-neutron | 03:06 | |
| *** tbachman has quit IRC | 03:09 | |
| *** zhhuabj_ has quit IRC | 03:10 | |
| *** jckasper has quit IRC | 03:11 | |
| *** jckasper has joined #openstack-neutron | 03:11 | |
| *** emagana has joined #openstack-neutron | 03:13 | |
| *** jckasper has quit IRC | 03:13 | |
| openstackgerrit | huan proposed openstack/neutron: XenAPI: add support for conntrack with XenServer https://review.openstack.org/341304 | 03:13 |
| *** jckasper has joined #openstack-neutron | 03:14 | |
| *** tbachman has joined #openstack-neutron | 03:16 | |
| *** emagana has quit IRC | 03:17 | |
| *** jhershbe has quit IRC | 03:18 | |
| *** 1JTAAVE94 has joined #openstack-neutron | 03:19 | |
| *** 18VABF8Y2 has joined #openstack-neutron | 03:19 | |
| *** zhhuabj_ has joined #openstack-neutron | 03:22 | |
| *** 18VABF8Y2 has quit IRC | 03:24 | |
| *** 1JTAAVE94 has quit IRC | 03:24 | |
| *** lujinluo has quit IRC | 03:25 | |
| *** tflynn has joined #openstack-neutron | 03:27 | |
| *** zhhuabj_ has quit IRC | 03:28 | |
| openstackgerrit | Irina proposed openstack/neutron-vpnaas: Adding tests for endpoint-group api. https://review.openstack.org/338075 | 03:29 |
| *** yamamoto_ has joined #openstack-neutron | 03:31 | |
| *** jckasper has quit IRC | 03:31 | |
| *** jckasper has joined #openstack-neutron | 03:32 | |
| *** jckasper has quit IRC | 03:33 | |
| *** Guest78543 has joined #openstack-neutron | 03:36 | |
| *** yamamoto_ has quit IRC | 03:36 | |
| *** zhhuabj_ has joined #openstack-neutron | 03:36 | |
| *** Guest78543 has quit IRC | 03:38 | |
| *** MadOtis has quit IRC | 03:40 | |
| *** Sukhdev has joined #openstack-neutron | 03:40 | |
| *** tonytan4ever has joined #openstack-neutron | 03:42 | |
| *** yamamoto_ has joined #openstack-neutron | 03:44 | |
| *** liuyulong has joined #openstack-neutron | 03:45 | |
| *** zhhuabj_ has quit IRC | 03:45 | |
| *** tonytan4ever has quit IRC | 03:47 | |
| *** zhhuabj_ has joined #openstack-neutron | 03:49 | |
| *** yuanying has joined #openstack-neutron | 03:51 | |
| *** hieulq has joined #openstack-neutron | 03:51 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: Add Libreswan installation guide in devstack.rst https://review.openstack.org/344723 | 03:53 |
| *** tidwellr has joined #openstack-neutron | 03:53 | |
| *** zhhuabj has joined #openstack-neutron | 03:54 | |
| *** zhhuabj_ has quit IRC | 03:57 | |
| namnh | Hi everyone. Do you know so that Neutron use oslo_db which I cloned from Githup?. I don't want to use Oslo_db from .../python2.7/dist-packages/oslo_db. Thanks! | 03:58 |
| *** lujinluo has joined #openstack-neutron | 03:59 | |
| *** buttercup has joined #openstack-neutron | 03:59 | |
| *** zhhuabj has quit IRC | 04:01 | |
| *** emagana has joined #openstack-neutron | 04:01 | |
| *** zhhuabj has joined #openstack-neutron | 04:02 | |
| *** zhhuabj_ has joined #openstack-neutron | 04:03 | |
| *** emagana has quit IRC | 04:06 | |
| *** zhhuabj has quit IRC | 04:06 | |
| *** links has joined #openstack-neutron | 04:13 | |
| *** links has quit IRC | 04:13 | |
| *** links has joined #openstack-neutron | 04:13 | |
| *** djan has quit IRC | 04:13 | |
| *** djan has joined #openstack-neutron | 04:14 | |
| *** jckasper has joined #openstack-neutron | 04:14 | |
| *** zhhuabj_ has quit IRC | 04:19 | |
| *** jhershbe has joined #openstack-neutron | 04:21 | |
| *** itzikb has quit IRC | 04:21 | |
| *** Sukhdev has quit IRC | 04:24 | |
| *** Sukhdev has joined #openstack-neutron | 04:25 | |
| *** ircuser-1 has joined #openstack-neutron | 04:28 | |
| *** Sukhdev has quit IRC | 04:29 | |
| *** nlahouti has joined #openstack-neutron | 04:29 | |
| *** yamamoto_ has quit IRC | 04:34 | |
| *** krtaylor has joined #openstack-neutron | 04:35 | |
| openstackgerrit | Anh Tran proposed openstack/neutron: Rollback port after failed to add it to router https://review.openstack.org/341427 | 04:38 |
| *** chandankumar has joined #openstack-neutron | 04:38 | |
| *** zhhuabj_ has joined #openstack-neutron | 04:39 | |
| *** Leo_ has joined #openstack-neutron | 04:41 | |
| *** a_ta has quit IRC | 04:42 | |
| *** a_ta has joined #openstack-neutron | 04:43 | |
| *** Leo_ has quit IRC | 04:46 | |
| *** boden has joined #openstack-neutron | 04:46 | |
| *** elopez has joined #openstack-neutron | 04:46 | |
| *** elopez is now known as Guest31932 | 04:47 | |
| *** tonytan4ever has joined #openstack-neutron | 04:47 | |
| *** boden has quit IRC | 04:47 | |
| *** a_ta has quit IRC | 04:47 | |
| *** kevo has joined #openstack-neutron | 04:52 | |
| *** numans has joined #openstack-neutron | 04:53 | |
| *** banix has quit IRC | 04:54 | |
| *** elo has quit IRC | 04:56 | |
| *** elo has joined #openstack-neutron | 05:00 | |
| *** sindhu has quit IRC | 05:01 | |
| *** yfried has joined #openstack-neutron | 05:04 | |
| *** numans has quit IRC | 05:07 | |
| *** hynekm has joined #openstack-neutron | 05:10 | |
| *** pbandark has joined #openstack-neutron | 05:11 | |
| openstackgerrit | Irina proposed openstack/neutron-vpnaas: Adding tests for endpoint-group api. https://review.openstack.org/338075 | 05:15 |
| *** yamamoto_ has joined #openstack-neutron | 05:15 | |
| *** imcsk8 has quit IRC | 05:17 | |
| *** imcsk8 has joined #openstack-neutron | 05:17 | |
| *** jhershbe has quit IRC | 05:19 | |
| *** amitkqed has quit IRC | 05:19 | |
| *** amitkqed has joined #openstack-neutron | 05:19 | |
| *** vikram has joined #openstack-neutron | 05:20 | |
| *** mohankumar has joined #openstack-neutron | 05:22 | |
| *** tflynn has quit IRC | 05:26 | |
| *** vthapar has joined #openstack-neutron | 05:29 | |
| openstackgerrit | Merged openstack/neutron: Update testing coverage document https://review.openstack.org/292607 | 05:29 |
| *** shausy has joined #openstack-neutron | 05:31 | |
| *** yfried has quit IRC | 05:32 | |
| *** zhhuabj_ has quit IRC | 05:33 | |
| *** saggi has joined #openstack-neutron | 05:36 | |
| *** zhhuabj_ has joined #openstack-neutron | 05:36 | |
| *** Leo_ has joined #openstack-neutron | 05:36 | |
| *** saggi has quit IRC | 05:38 | |
| openstackgerrit | Reedip proposed openstack/neutron: Add name and desc to networksegments table https://review.openstack.org/293305 | 05:40 |
| *** mickeys has quit IRC | 05:40 | |
| *** Leo_ has quit IRC | 05:40 | |
| *** kawa2014 has joined #openstack-neutron | 05:42 | |
| *** mickeys has joined #openstack-neutron | 05:45 | |
| *** amotoki has joined #openstack-neutron | 05:46 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/neutron: [WIP] Testing latest u-c https://review.openstack.org/303346 | 05:49 |
| *** mickeys has quit IRC | 05:49 | |
| *** pgadiya has joined #openstack-neutron | 05:50 | |
| *** Fdaisuke has joined #openstack-neutron | 05:52 | |
| *** mfuruta has quit IRC | 05:52 | |
| *** mfuruta has joined #openstack-neutron | 05:52 | |
| *** _Fdaisuke_ has quit IRC | 05:54 | |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Relocate L3 DB Models https://review.openstack.org/348562 | 05:54 |
| *** sridharg has joined #openstack-neutron | 05:55 | |
| *** zhhuabj_ has quit IRC | 05:59 | |
| *** gouthamr has joined #openstack-neutron | 06:01 | |
| *** david-lyle has joined #openstack-neutron | 06:07 | |
| *** obondarev has joined #openstack-neutron | 06:08 | |
| *** gouthamr has quit IRC | 06:10 | |
| *** tidwellr has quit IRC | 06:13 | |
| *** tflynn has joined #openstack-neutron | 06:16 | |
| ajo | ping armax: I was thinking, now that I read your comments about bulk_push | 06:16 |
| armax | ajo: aye | 06:16 |
| ajo | that may be now we're still in time to change the subscriber callback function semantics | 06:16 |
| ajo | to receive always a list in all cases | 06:16 |
| armax | ajo: I have a local change | 06:16 |
| ajo | when it's a push we pass [resource] | 06:16 |
| ajo | and when it's a bulk, we pass resource_list | 06:16 |
| ajo | so we don't have two cases | 06:16 |
| armax | ajo: let me post what I have | 06:17 |
| ajo | ack | 06:17 |
| armax | ajo: then you can decide | 06:17 |
| armax | ajo: I think it can work as it is | 06:17 |
| ajo | thank's armax | 06:17 |
| armax | ajo: but we can see what’s better | 06:17 |
| armax | I won | 06:17 |
| armax | I won’t be long | 06:17 |
| armax | I just need to rebase | 06:17 |
| armax | 347662 on top of yours | 06:17 |
| armax | ajo: I also solved the sanity check issue | 06:18 |
| ajo | ack, I need to sort out the kids and do some bureaucracy this morning :/ | 06:18 |
| armax | ajo: actually I got rid of it altogether and solved the compat diffently :) | 06:18 |
| ajo | :) | 06:18 |
| armax | ajo: good luck! :) | 06:18 |
| armax | ajo: if Spanish bureucracy is like the Italian's | 06:18 |
| armax | ajo: you’re in trouble | 06:18 |
| *** zhhuabj_ has joined #openstack-neutron | 06:19 | |
| ajo | thanks :s, probably similar... at least they're no completely tracked and part of their salary comes from goals (forms/steps processed) | 06:19 |
| ajo | no->now | 06:20 |
| armax | ajo: that’s a bit step forward | 06:20 |
| ajo | metrics help :) | 06:20 |
| armax | ajo: can’t improve what you can’t measure! | 06:20 |
| *** shausy has quit IRC | 06:20 | |
| ajo | armax, I'll look at it all together when I'm back, and I'll also look at the payload sizes, etc | 06:21 |
| armax | ajo: ack | 06:21 |
| ajo | which I wanted to do yesterday and I didn't have enough time | 06:21 |
| armax | I’ll push this in the next 10 mins | 06:21 |
| armax | then I am going to bed | 06:21 |
| armax | I’ll catch up with you in a few hours | 06:21 |
| ajo | armax, yeah, rest is necessary | 06:21 |
| armax | ajo: indeed it is | 06:21 |
| armax | ajo: but this stuff is addictive! | 06:21 |
| armax | ajo: weird type of drug, let me tell you that! | 06:22 |
| ajo | hehehe, I know what you mean ;) | 06:22 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Change external_network_bridge default to '' https://review.openstack.org/298443 | 06:26 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add RPC layer for Trunk Plugin and initial Open vSwitch driver https://review.openstack.org/347662 | 06:27 |
| armax | ajo: ^ check it out and let me know what you reckon! | 06:27 |
| ajo | armax, ack :) | 06:27 |
| ajo | https://review.openstack.org/#/c/347662/8/neutron/api/rpc/callbacks/resources.py ahhh, I see | 06:28 |
| ajo | that could definitely work | 06:28 |
| ajo | I'll think of the possibilities | 06:28 |
| ajo | I need to afk now, but will get back to it in now+1.5h or so | 06:29 |
| *** jhershbe has joined #openstack-neutron | 06:29 | |
| *** Leo_ has joined #openstack-neutron | 06:30 | |
| *** tonytan4ever has quit IRC | 06:30 | |
| *** tflynn has quit IRC | 06:32 | |
| *** mickeys has joined #openstack-neutron | 06:34 | |
| *** kevo has quit IRC | 06:34 | |
| *** Leo_ has quit IRC | 06:34 | |
| *** mickeys has quit IRC | 06:34 | |
| *** jhershbe has quit IRC | 06:36 | |
| *** armax has quit IRC | 06:36 | |
| *** numans has joined #openstack-neutron | 06:36 | |
| *** tesseract- has joined #openstack-neutron | 06:37 | |
| *** gvrangan has joined #openstack-neutron | 06:38 | |
| *** sticker_ has joined #openstack-neutron | 06:41 | |
| *** andymaier has joined #openstack-neutron | 06:41 | |
| *** gongysh has joined #openstack-neutron | 06:42 | |
| *** hoangcx has quit IRC | 06:42 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: [DO NOT MERGE]: iptables diff canary https://review.openstack.org/301338 | 06:43 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add option to validate iptables rules https://review.openstack.org/348753 | 06:43 |
| *** sticker has quit IRC | 06:44 | |
| *** david-lyle has quit IRC | 06:45 | |
| *** pcaruana has joined #openstack-neutron | 06:45 | |
| *** obondarev has quit IRC | 06:47 | |
| openstackgerrit | Lewis proposed openstack/neutron-vpnaas: Add connection API cases https://review.openstack.org/338061 | 06:48 |
| *** sticker_ is now known as sticker | 06:49 | |
| *** manikanta_tadi has joined #openstack-neutron | 06:51 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: [DO NOT MERGE]: test short tokens https://review.openstack.org/300371 | 06:52 |
| *** matrohon has joined #openstack-neutron | 06:52 | |
| *** itisha has joined #openstack-neutron | 06:53 | |
| *** hoangcx has joined #openstack-neutron | 06:55 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Fix a race with auto_schedule of HA routers https://review.openstack.org/284400 | 06:55 |
| *** namnh has quit IRC | 06:58 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Use revision to discard stale DHCP updates https://review.openstack.org/304136 | 06:58 |
| *** Fdaisuke has quit IRC | 06:59 | |
| *** Fdaisuke has joined #openstack-neutron | 07:00 | |
| openstackgerrit | Andrey Shestakov proposed openstack/neutron: Consider baremetal device_owner as compute for nova notify https://review.openstack.org/346823 | 07:00 |
| *** anilvenkata has joined #openstack-neutron | 07:02 | |
| *** zhenguo has joined #openstack-neutron | 07:04 | |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Relocate Metering DB models https://review.openstack.org/348757 | 07:07 |
| *** nherciu has joined #openstack-neutron | 07:10 | |
| *** vijaykc4 has joined #openstack-neutron | 07:10 | |
| *** sticker has quit IRC | 07:12 | |
| *** vijaykc4 has quit IRC | 07:15 | |
| *** bvandewa has joined #openstack-neutron | 07:16 | |
| *** Fdaisuke has quit IRC | 07:17 | |
| *** Fdaisuke has joined #openstack-neutron | 07:17 | |
| *** vishwanathj has quit IRC | 07:21 | |
| *** davidlenwell has quit IRC | 07:22 | |
| *** Leo_ has joined #openstack-neutron | 07:24 | |
| *** rubasov has joined #openstack-neutron | 07:26 | |
| *** Leo_ has quit IRC | 07:28 | |
| *** matrohon has quit IRC | 07:29 | |
| *** tonytan4ever has joined #openstack-neutron | 07:31 | |
| *** davidlenwell has joined #openstack-neutron | 07:34 | |
| *** jlanoux has joined #openstack-neutron | 07:36 | |
| *** tonytan4ever has quit IRC | 07:36 | |
| *** zhhuabj_ has quit IRC | 07:36 | |
| *** zhhuabj_ has joined #openstack-neutron | 07:37 | |
| *** jlibosva has joined #openstack-neutron | 07:38 | |
| *** aqkhan has joined #openstack-neutron | 07:39 | |
| *** zhhuabj has joined #openstack-neutron | 07:41 | |
| *** mosulica has joined #openstack-neutron | 07:43 | |
| *** zhhuabj_ has quit IRC | 07:44 | |
| *** dkehn_ has quit IRC | 07:44 | |
| *** fzdarsky|afk has joined #openstack-neutron | 07:44 | |
| *** bvandewa has quit IRC | 07:45 | |
| *** dkehn has quit IRC | 07:45 | |
| *** hoangcx has quit IRC | 07:48 | |
| *** mohankumar has quit IRC | 07:49 | |
| *** kaisers has quit IRC | 07:50 | |
| *** hoangcx has joined #openstack-neutron | 07:50 | |
| *** obondarev has joined #openstack-neutron | 07:50 | |
| *** kaisers has joined #openstack-neutron | 07:51 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Use single priority for floating IP rules https://review.openstack.org/312790 | 07:52 |
| kevinbenton | garyk: hey, can you take a look at https://review.openstack.org/#/c/327996/ and weigh in? | 07:53 |
| *** gongysh has quit IRC | 07:53 | |
| *** jhershbe has joined #openstack-neutron | 07:56 | |
| *** dkehn has joined #openstack-neutron | 07:56 | |
| *** namnh has joined #openstack-neutron | 07:57 | |
| *** dkehn_ has joined #openstack-neutron | 07:58 | |
| openstackgerrit | LIU Yulong proposed openstack/neutron: Resolve infinite loops in L3 HA agent side https://review.openstack.org/265672 | 07:58 |
| *** zzzeek has quit IRC | 08:00 | |
| *** rossella_s has quit IRC | 08:00 | |
| *** johnbelamaric has quit IRC | 08:00 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Cleanup DB retry logic in ML2 https://review.openstack.org/332586 | 08:00 |
| *** rossella_s has joined #openstack-neutron | 08:00 | |
| *** johnbelamaric has joined #openstack-neutron | 08:00 | |
| *** zzzeek has joined #openstack-neutron | 08:00 | |
| *** gvrangan has quit IRC | 08:05 | |
| *** fragatina has quit IRC | 08:09 | |
| *** fzdarsky has joined #openstack-neutron | 08:11 | |
| *** fzdarsky|afk has quit IRC | 08:11 | |
| openstackgerrit | Lewis proposed openstack/neutron-vpnaas: Add connection API cases https://review.openstack.org/338061 | 08:16 |
| garyk | kevinbenton: there is an indentation and typo issue | 08:17 |
| kevinbenton | garyk: well it's a back-port | 08:17 |
| kevinbenton | garyk: should i fix there? | 08:17 |
| kevinbenton | garyk: we normally keep it the same | 08:17 |
| garyk | kevinbenton: its a backport so no need to change them | 08:19 |
| garyk | kevinbenton: i have given a +2 | 08:20 |
| garyk | kevinbenton: approved | 08:21 |
| *** jhershbe has quit IRC | 08:21 | |
| kevinbenton | garyk: thx | 08:21 |
| *** vijaykc4 has joined #openstack-neutron | 08:21 | |
| liuyulong | garyk, hi | 08:23 |
| liuyulong | garyk, how about this patch https://review.openstack.org/#/c/265680/ now ? Kevin has also explained some detail in comments. | 08:23 |
| garyk | git a guy a finger and he takes the whole hand | 08:23 |
| garyk | liuyulong: looks good. | 08:25 |
| garyk | liuyulong: i prefer that a core who is more familiar with the HA approve | 08:25 |
| liuyulong | garyk, thanks, that patch is staying there in a very long time, :) | 08:26 |
| *** ygbo has joined #openstack-neutron | 08:26 | |
| *** tflynn has joined #openstack-neutron | 08:27 | |
| liuyulong | jschwarz, hi, ping? | 08:29 |
| *** yamamoto_ has quit IRC | 08:30 | |
| *** iyamahat has joined #openstack-neutron | 08:32 | |
| *** bvandewa has joined #openstack-neutron | 08:32 | |
| *** yamahata has joined #openstack-neutron | 08:37 | |
| *** wolverineav has joined #openstack-neutron | 08:38 | |
| *** iranzo has joined #openstack-neutron | 08:42 | |
| *** iranzo has joined #openstack-neutron | 08:42 | |
| *** iyamahat has quit IRC | 08:42 | |
| *** wolverineav has quit IRC | 08:43 | |
| *** iyamahat has joined #openstack-neutron | 08:44 | |
| *** shausy has joined #openstack-neutron | 08:45 | |
| *** davidsha has joined #openstack-neutron | 08:47 | |
| *** ramishra has quit IRC | 08:48 | |
| *** mohankumar has joined #openstack-neutron | 08:49 | |
| *** gongysh has joined #openstack-neutron | 08:50 | |
| *** wolverineav has joined #openstack-neutron | 08:50 | |
| *** jhershbe has joined #openstack-neutron | 08:52 | |
| *** abregman has joined #openstack-neutron | 08:52 | |
| *** iyamahat has quit IRC | 08:54 | |
| *** wolverineav has quit IRC | 08:55 | |
| apuimedo | garyk: kevinbenton: we could use some merges in stable/mitaka for networking-midonet | 08:56 |
| *** mickeys has joined #openstack-neutron | 08:56 | |
| kevinbenton | apuimedo: links? | 08:56 |
| *** mhickey has joined #openstack-neutron | 08:56 | |
| *** amotoki has quit IRC | 08:56 | |
| apuimedo | https://review.openstack.org/#/c/348789/ | 08:57 |
| apuimedo | https://review.openstack.org/#/q/status:open+project:openstack/networking-midonet+branch:stable/mitaka+topic:bug/1598449 | 08:57 |
| *** mickeys has quit IRC | 08:58 | |
| *** mickeys has joined #openstack-neutron | 08:58 | |
| apuimedo | kevinbenton: garyk: ^^ | 08:59 |
| apuimedo | after getting these merged, pushing a new release tag would be ideal | 08:59 |
| *** amotoki has joined #openstack-neutron | 09:00 | |
| *** wolverineav has joined #openstack-neutron | 09:02 | |
| *** mickeys has quit IRC | 09:03 | |
| *** zhhuabj has quit IRC | 09:03 | |
| *** zhhuabj has joined #openstack-neutron | 09:03 | |
| *** wolverineav has quit IRC | 09:06 | |
| *** matrohon has joined #openstack-neutron | 09:07 | |
| *** jhershbe has quit IRC | 09:08 | |
| *** kawa2014 has quit IRC | 09:08 | |
| openstackgerrit | Lewis proposed openstack/neutron-vpnaas: Add connection API cases https://review.openstack.org/338061 | 09:11 |
| *** yb has quit IRC | 09:11 | |
| *** Leo_ has joined #openstack-neutron | 09:12 | |
| *** kawa2014 has joined #openstack-neutron | 09:12 | |
| *** djan has quit IRC | 09:13 | |
| openstackgerrit | Lujin Luo proposed openstack/neutron: Add a unique key to port_id in routerports table https://review.openstack.org/285048 | 09:14 |
| *** Leo_ has quit IRC | 09:17 | |
| *** mfuruta has quit IRC | 09:21 | |
| *** ihrachys has joined #openstack-neutron | 09:21 | |
| *** jhershbe has joined #openstack-neutron | 09:23 | |
| *** wolverineav has joined #openstack-neutron | 09:24 | |
| *** ociuhandu has quit IRC | 09:25 | |
| *** bvandewa has quit IRC | 09:27 | |
| *** wolverineav has quit IRC | 09:28 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Fix that api_workers=0 doesn't spawn any api workers https://review.openstack.org/348801 | 09:30 |
| *** tflynn has quit IRC | 09:31 | |
| apuimedo | jlibosva: good catch with the "not" -> "is None" | 09:32 |
| apuimedo | :-) | 09:32 |
| apuimedo | jlibosva: is there some fullstack tests that checks that it actually works without api workers? | 09:33 |
| jlibosva | apuimedo: :) I would be more glad if it will be actually tests that catch it | 09:33 |
| jlibosva | apuimedo: nope | 09:33 |
| *** permalac has joined #openstack-neutron | 09:33 | |
| apuimedo | :( | 09:33 |
| jlibosva | or not that I'm aware of | 09:34 |
| *** john-davidge has joined #openstack-neutron | 09:34 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add option to validate iptables rules https://review.openstack.org/348753 | 09:34 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: [DO NOT MERGE]: iptables diff canary https://review.openstack.org/301338 | 09:34 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Set prefix on floating_ip_mangle rules https://review.openstack.org/348805 | 09:34 |
| jlibosva | apuimedo: we use oslo library though for workers - so I guess it's fine to verify we call a correct parameter to the library | 09:35 |
| *** john-dav_ has joined #openstack-neutron | 09:35 | |
| jlibosva | s/call/pass/ | 09:35 |
| apuimedo | I guess so | 09:36 |
| *** john-davidge has quit IRC | 09:38 | |
| *** fragatina has joined #openstack-neutron | 09:38 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Fix that api_workers=0 doesn't spawn any api workers https://review.openstack.org/348801 | 09:38 |
| *** moshele has joined #openstack-neutron | 09:38 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Fix that api_workers=0 doesn't spawn any api workers https://review.openstack.org/348801 | 09:40 |
| *** zhhuabj has quit IRC | 09:41 | |
| *** moshele has quit IRC | 09:43 | |
| *** vijaykc4 has quit IRC | 09:44 | |
| *** wolverineav has joined #openstack-neutron | 09:47 | |
| *** abregman has quit IRC | 09:50 | |
| *** wolverineav has quit IRC | 09:52 | |
| *** vijaykc4 has joined #openstack-neutron | 09:52 | |
| *** wolverineav has joined #openstack-neutron | 09:55 | |
| jlibosva | kevinbenton: hi, still awake? quick Q: is a subnet that belongs to external network with 'access_as_external' rbac action and '*' target_tenant supposed to be seen by all tenants? | 09:57 |
| jlibosva | from the code it seems we limit it to access_as_shared action only | 09:57 |
| *** sambetts|afk is now known as sambetts | 09:58 | |
| jlibosva | ihrachys or any rbac experts: ^^ | 09:59 |
| *** ociuhandu has joined #openstack-neutron | 09:59 | |
| *** ramishra has joined #openstack-neutron | 10:00 | |
| kevinbenton | jlibosva: no | 10:01 |
| kevinbenton | jlibosva: i don't believe so | 10:01 |
| kevinbenton | jlibosva: which is why router-gateway-set takes a network instead of a subnet like the other router interface actions | 10:01 |
| *** zhhuabj has joined #openstack-neutron | 10:02 | |
| kevinbenton | jlibosva: even if you don't use rbac | 10:02 |
| jlibosva | kevinbenton: we have a bug report claiming shared subnets where seen in Mitaka | 10:02 |
| jlibosva | kevinbenton: https://bugs.launchpad.net/bugs/1607534 | 10:02 |
| openstack | Launchpad bug 1607534 in neutron "Subnets not visible for admin tenant external network for non-admin user" [Medium,Confirmed] | 10:02 |
| jlibosva | kevinbenton: if that is true, do you know if we have that documented somewhere? I can close the bug then as invalid, right? | 10:03 |
| kevinbenton | jlibosva: ah, no. it may have been a regression | 10:03 |
| kevinbenton | jlibosva: let me see if i can confirm | 10:03 |
| jlibosva | kevinbenton: thanks | 10:03 |
| liuyulong | cloud anyone have a look at https://bugs.launchpad.net/neutron/+bug/1578523 and its fix https://review.openstack.org/#/c/312849/ | 10:04 |
| openstack | Launchpad bug 1578523 in neutron "Floating IP updating procedure is inaccurate" [Low,In progress] - Assigned to LIU Yulong (dragon889) | 10:04 |
| *** wolverineav has quit IRC | 10:06 | |
| *** trananhkma has quit IRC | 10:08 | |
| *** diga has joined #openstack-neutron | 10:12 | |
| *** yamahata has quit IRC | 10:16 | |
| *** wolverineav has joined #openstack-neutron | 10:16 | |
| kevinbenton | jlibosva: left a comment. marked as incomplete because this is the behavior in stable/liberty as well | 10:17 |
| *** gongysh has quit IRC | 10:18 | |
| *** sdague has joined #openstack-neutron | 10:19 | |
| *** EinstCrazy has quit IRC | 10:19 | |
| *** namnh has quit IRC | 10:20 | |
| *** wolverineav has quit IRC | 10:20 | |
| *** liuyulong has quit IRC | 10:21 | |
| *** wolverineav has joined #openstack-neutron | 10:22 | |
| kevinbenton | ihrachys: yo | 10:27 |
| kevinbenton | ihrachys: what do you think about this? https://review.openstack.org/#/c/298443/ | 10:27 |
| ihrachys | kevinbenton: howdy! | 10:27 |
| ihrachys | looking | 10:27 |
| kevinbenton | ihrachys: slowly make it more painful for people refusing to move :) | 10:27 |
| *** liuyulong has joined #openstack-neutron | 10:29 | |
| ihrachys | kevinbenton: I am for the change. one dumb question since I am not too close to l3: so for those using flat external connectivity, no rewiring is generally needed? | 10:31 |
| ihrachys | is it only a matter of changing options and restarting the agent? | 10:31 |
| openstackgerrit | garyk proposed openstack/neutron: Add in missing translations for exceptions https://review.openstack.org/337630 | 10:31 |
| kevinbenton | ihrachys: yes, in fact the agent may not even rewire then unless the namespace is cleaned up or gateway is re-attached | 10:32 |
| kevinbenton | ihrachys: because i think it will see the interface already exists | 10:32 |
| kevinbenton | ihrachys: but yeah, no change if external network is flat | 10:32 |
| kevinbenton | ihrachys: because the ports will already be correctly bound to a segment | 10:32 |
| *** kawa2014 has quit IRC | 10:33 | |
| kevinbenton | ihrachys: even though i claimed we would remove in Ocata, i would like to wait until we have a method of making ports attempt to rebind on other segments | 10:33 |
| kevinbenton | ihrachys: so we have a migration path that is a little cleaner | 10:33 |
| *** hoangcx has quit IRC | 10:33 | |
| ihrachys | kevinbenton: is it possible to write a test that would validate this kind of migration? I want to make sure that the router still sees the port and does not attempt to wire another one. I think functional would do (?) | 10:34 |
| kevinbenton | ihrachys: if it decides to rewire it would delete it | 10:34 |
| *** hoangcx has joined #openstack-neutron | 10:34 | |
| kevinbenton | ihrachys: well i better check | 10:34 |
| kevinbenton | ihrachys: the delete may include the bridge name | 10:34 |
| ihrachys | kevinbenton: yeah, you said 'I think' which is not 100% sure :) | 10:35 |
| openstackgerrit | Kengo Hobo proposed openstack/neutron-vpnaas: Add router_id to query when check subnet is used by vpnservice https://review.openstack.org/348835 | 10:35 |
| kevinbenton | ihrachys: can't i just tell them to run the netns cleanup tool in between ? :) | 10:35 |
| *** claudiub|2 has joined #openstack-neutron | 10:35 | |
| ihrachys | kevinbenton: but I think we may still switch default, that validation would really be needed when we remove the rug completely from under operators? | 10:35 |
| ihrachys | kevinbenton: netns cleanup is data plane downtime. | 10:36 |
| *** wolverineav has quit IRC | 10:36 | |
| *** john-dav_ is now known as john-davidge | 10:37 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Use single priority for floating IP rules https://review.openstack.org/312790 | 10:38 |
| kevinbenton | ihrachys: https://github.com/openstack/neutron/blob/38d33363e3d5e1aea864259443379e0bcc9d9028/neutron/agent/linux/interface.py#L247 | 10:39 |
| *** moshele has joined #openstack-neutron | 10:39 | |
| ihrachys | kevinbenton: we should also have it covered with non dvr grenade multinode. | 10:39 |
| ihrachys | but a more granular check that backend have not done anything weird could still be of use. you decide if you think it's worth it. | 10:40 |
| ihrachys | I am ok to go as is. we may revisit validation later when we actually remove the option. | 10:40 |
| kevinbenton | ihrachys: let me check something first | 10:41 |
| kevinbenton | ihrachys: no, we likely won't be testing this with grenade unfortunately | 10:41 |
| kevinbenton | ihrachys: http://logs.openstack.org/43/298443/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/5acdcca/logs/etc/neutron/l3_agent.ini.txt.gz | 10:41 |
| kevinbenton | ihrachys: devstack plants that value in there | 10:42 |
| kevinbenton | ihrachys: with extreme prejudice | 10:42 |
| ihrachys | ah | 10:42 |
| ihrachys | do we have it cleaned up in devstack master? | 10:42 |
| kevinbenton | ihrachys: not yet | 10:42 |
| kevinbenton | ihrachys: had to revert that | 10:42 |
| kevinbenton | ihrachys: waiting for revert of the revert | 10:43 |
| kevinbenton | ihrachys: so plop a -1 on my patch an make me write a functional test | 10:43 |
| kevinbenton | ihrachys: i'll catch that tomorrow | 10:43 |
| kevinbenton | ihrachys: https://review.openstack.org/#/c/346282/ | 10:43 |
| kevinbenton | ihrachys: i'm worried that we scared Sean away by breaking the gate :) | 10:43 |
| ihrachys | kevinbenton: :) | 10:44 |
| ihrachys | we are still to land the first one anyway | 10:45 |
| ihrachys | kevinbenton: one question on sqla | 10:45 |
| ihrachys | kevinbenton: https://review.openstack.org/#/c/348279/3/neutron/objects/base.py | 10:46 |
| ihrachys | kevinbenton: see I needed to copy the model otherwise it seems like it always returned the same relationship on each access, even though more related models were pushed in the middle. | 10:46 |
| ihrachys | kevinbenton: I would hope that sqla would populate cached relationships with whatever related models are inserted, but it seems it's not the case (?) | 10:47 |
| kevinbenton | ihrachys: example? | 10:47 |
| kevinbenton | ihrachys: how are the related models being added? | 10:47 |
| ihrachys | just by adding new related models to the session | 10:48 |
| ihrachys | I still need to poke it myself, but I was thinking, maybe you have some ready to read doc in mind to understand how sqla tracks models | 10:49 |
| kevinbenton | ihrachys: nah, i've run into similar issues though with ports and ip addresses | 10:50 |
| kevinbenton | ihrachys: like if you create an IPallocation with a port ID and just jam it in the database | 10:50 |
| *** wolverineav has joined #openstack-neutron | 10:50 | |
| kevinbenton | ihrachys: an existing port in the session that has already had it's fixed_ips referenced will not see the update | 10:50 |
| ihrachys | I see. it's interesting that it really depends on me saving the reference to the model on the object. if I just comment out line 304, it all passes and properly loads eg routes thru the synthetic field for subnet. | 10:51 |
| kevinbenton | ihrachys: hang on | 10:53 |
| kevinbenton | ihrachys: i'm not sure i understand which model is breaking which object | 10:53 |
| *** moshele has quit IRC | 10:53 | |
| kevinbenton | ihrachys: can you elaborate? | 10:53 |
| ihrachys | kevinbenton: see fe. failures in http://logs.openstack.org/79/348279/2/check/gate-neutron-python27-ubuntu-xenial/776a25d/testr_results.html.gz | 10:54 |
| ihrachys | kevinbenton: that was without copying | 10:54 |
| *** wolverineav has quit IRC | 10:54 | |
| *** vsaienk0 has quit IRC | 10:55 | |
| *** tflynn has joined #openstack-neutron | 10:55 | |
| *** lujinluo has quit IRC | 10:56 | |
| kevinbenton | ihrachys: aye | 10:57 |
| kevinbenton | ihrachys: holding onto the reference is preventing garbage collection | 10:57 |
| kevinbenton | ihrachys: i suspect the result of copy.copy isn't know about by the session | 10:58 |
| kevinbenton | ihrachys: and by having it still in the session it is using that cached stuff | 10:58 |
| kevinbenton | ihrachys: https://github.com/openstack/neutron/blob/38d33363e3d5e1aea864259443379e0bcc9d9028/neutron/db/api.py#L119-L124 | 10:58 |
| kevinbenton | ihrachys: this is the root of all evils | 10:58 |
| *** vsaienk0 has joined #openstack-neutron | 10:58 | |
| kevinbenton | ihrachys: expire_on_commit=False | 10:58 |
| kevinbenton | ihrachys: can you see if you change expire_on_commit to true of that prevents the failures? | 10:59 |
| ihrachys | kevinbenton: it does! | 10:59 |
| kevinbenton | ihrachys: http://docs.sqlalchemy.org/en/latest/orm/session_api.html#sqlalchemy.orm.session.Session.params.expire_on_commit | 10:59 |
| kevinbenton | ihrachys: so that's the default from sqlalchemy | 11:00 |
| kevinbenton | ihrachys: i've considered changing it for us | 11:00 |
| kevinbenton | ihrachys: but i'm not sure why we went the other way with our default | 11:00 |
| ihrachys | kevinbenton: but them, do we have more sql statements issued on expire? | 11:00 |
| *** Leo_ has joined #openstack-neutron | 11:00 | |
| kevinbenton | ihrachys: yes | 11:00 |
| kevinbenton | ihrachys: expire will cause new lookups | 11:00 |
| *** fragatina has quit IRC | 11:00 | |
| ihrachys | so bad scaling wise | 11:00 |
| ihrachys | kevinbenton: problem is, if I copy, relationships do not reflect reality | 11:01 |
| *** obondarev has quit IRC | 11:01 | |
| kevinbenton | ihrachys: yeah, they're stuck in time | 11:01 |
| kevinbenton | ihrachys: because of this same reason | 11:01 |
| kevinbenton | ihrachys: are the subobjects you are creating part of the same OVO object? | 11:02 |
| ihrachys | kevinbenton: is that because I open subtransations agressively in objects base create/update/....? | 11:02 |
| *** tflynn has quit IRC | 11:02 | |
| ihrachys | kevinbenton: they are, yes | 11:02 |
| ihrachys | well, they are also their own objects | 11:02 |
| kevinbenton | ihrachys: well i see one failure is Trunk/Subport | 11:02 |
| ihrachys | but exposed thru list-of-objects fields too | 11:02 |
| apuimedo | kevinbenton: do you think my "recheck" message to https://review.openstack.org/#/c/336801/ will re-trigger? | 11:02 |
| ihrachys | apuimedo: it will | 11:03 |
| ihrachys | apuimedo: but you may check zuul | 11:03 |
| apuimedo | cool | 11:03 |
| ihrachys | apuimedo: anything that stats with 'recheck' will do | 11:03 |
| apuimedo | thanks Ihar :-) | 11:03 |
| ihrachys | `8starts | 11:03 |
| ihrachys | meh. fingers are so 90s, I need a brain-to-kbd interface | 11:03 |
| *** vijaykc4 has quit IRC | 11:04 | |
| apuimedo | ihrachys: you can repurpose a son to scribe | 11:04 |
| apuimedo | specially on summer holidays | 11:04 |
| *** Leo_ has quit IRC | 11:05 | |
| jlibosva | kevinbenton: thanks | 11:06 |
| kevinbenton | ihrachys: bleck. i need to revisit my notes from the ip allocation mess | 11:06 |
| openstackgerrit | John Davidge proposed openstack/neutron: Enable CRUD for Subnet Service Types https://review.openstack.org/337851 | 11:07 |
| *** vijaykc4 has joined #openstack-neutron | 11:07 | |
| *** hoangcx2 has joined #openstack-neutron | 11:09 | |
| kevinbenton | ihrachys: so one way i know does it for sure is when the new objects are constructed with a ref to the parent or by calling 'append' on the parent's relationship to the child | 11:10 |
| kevinbenton | ihrachys: but that might be difficult to do when they are separate OVO objects | 11:10 |
| *** hoangcx has quit IRC | 11:13 | |
| *** davidsha has quit IRC | 11:13 | |
| *** lucasagomes is now known as lucas-hungry | 11:14 | |
| *** rodrigods has quit IRC | 11:19 | |
| *** rodrigods has joined #openstack-neutron | 11:19 | |
| *** nlahouti has quit IRC | 11:20 | |
| *** rtheis has joined #openstack-neutron | 11:21 | |
| kevinbenton | ihrachys: whenever you build a dnsnameserver model. if you call 'print dns_db_obj.subnet' does that stop the failures? | 11:22 |
| kevinbenton | (for subnet at least) | 11:22 |
| kevinbenton | ihrachys: that should cause it to look for the subnet in the existing session | 11:22 |
| kevinbenton | ihrachys: so it would be interesting to see if it updates it when it finds it | 11:23 |
| ihrachys | kevinbenton: nope, it does not help. for some reason I get None then. | 11:28 |
| ihrachys | even though subnet_id is passed | 11:28 |
| *** ijw has joined #openstack-neutron | 11:28 | |
| *** jprovazn has joined #openstack-neutron | 11:29 | |
| ihrachys | ok, that shows up after I move it from under subtransaction | 11:29 |
| ihrachys | but still does not solve the failure | 11:29 |
| *** ijw has quit IRC | 11:30 | |
| *** ijw has joined #openstack-neutron | 11:31 | |
| *** tonytan4ever has joined #openstack-neutron | 11:32 | |
| *** abregman has joined #openstack-neutron | 11:32 | |
| kevinbenton | ihrachys: if you call session.expunge(self._db_obj...) on it does that stop the failure? | 11:33 |
| kevinbenton | ihrachys: i'm worried that the db object will have stale data in that case still though... | 11:34 |
| jlibosva | kevinbenton: http://paste.openstack.org/show/543866/ - don't you need to set the network to shared? | 11:34 |
| kevinbenton | jlibosva: sure, but that wouldn't be what the bug report is about :) | 11:35 |
| ihrachys | kevinbenton: it makes the test fail with: http://paste.fedoraproject.org/397252/46979209/ | 11:35 |
| kevinbenton | ihrachys: wait, don't detach until the object has constructed itself | 11:35 |
| jlibosva | kevinbenton: how come? the regression might have come due to creating access_as_external in rbac | 11:36 |
| kevinbenton | jlibosva: what? | 11:36 |
| kevinbenton | jlibosva: the poster is complaining they can't see external subnets | 11:36 |
| kevinbenton | jlibosva: unless they are shared | 11:36 |
| kevinbenton | jlibosva: so i showed that you can't see external subnets if they aren't shared in stable/liberty | 11:36 |
| kevinbenton | jlibosva: which is what that paste is | 11:36 |
| *** diga has quit IRC | 11:37 | |
| *** ijw has quit IRC | 11:37 | |
| kevinbenton | ihrachys: bleh, this isn't the right solution anyway, because an extension might reference a relationship and get that same error | 11:37 |
| *** tonytan4ever has quit IRC | 11:37 | |
| ihrachys | kevinbenton: huh. yeah, it fixes it. though the fix looks ugly atm | 11:38 |
| jlibosva | kevinbenton: aha, but in liberty, you had a way how to see shared subnets which now on master you don't. | 11:38 |
| kevinbenton | ihrachys: we may just impose a performance penaltly on people using out of tree extensions and call session.expire on the object each time before returning it in @property | 11:38 |
| jlibosva | kevinbenton: anyway, you're the expert here, I just wanted to make sure the 'shared' isn't missing in the paste :) Thanks for explanation | 11:39 |
| kevinbenton | jlibosva: no, in master you can see shared subnets | 11:39 |
| ihrachys | kevinbenton: but then, do they need to maintain the transaction to reload relationships? | 11:39 |
| kevinbenton | ihrachys: don't need a transaction for a relationship load, just a session | 11:40 |
| jlibosva | kevinbenton: those that belong to external network? | 11:40 |
| kevinbenton | jlibosva: yes | 11:40 |
| kevinbenton | ihrachys: and the thing is attached to the session | 11:40 |
| *** itisha has quit IRC | 11:40 | |
| *** thorst has joined #openstack-neutron | 11:40 | |
| kevinbenton | ihrachys: so just by nature of doing subnet.dnsnameservers or whatever, it will do the lookup | 11:40 |
| ihrachys | kevinbenton: the problem is that synthetic field loading depends on the relationship, not just users of exposed db_obj property. the former is broken by the caching, not the latter. | 11:41 |
| ihrachys | well, maybe the latter too (like in: relationships are obsolete), but still | 11:41 |
| kevinbenton | ihrachys: well if the object is expired, it has to do a db lookup anyway | 11:42 |
| *** obondarev has joined #openstack-neutron | 11:42 | |
| kevinbenton | ihrachys: there won't be a stale object hanging around in the session | 11:42 |
| jlibosva | kevinbenton: aha, I see now. | 11:42 |
| ihrachys | kevinbenton: I may miss something. how does sqla refresh relationships without a transaction? from some internal lib cache, not from db server? | 11:42 |
| kevinbenton | ihrachys: a session is a connection to a db | 11:42 |
| kevinbenton | ihrachys: it can issue select queries without a transaction | 11:42 |
| ihrachys | ok. but with no consistency guarantees. | 11:43 |
| kevinbenton | ihrachys: that's why it would be a performance hit for them | 11:43 |
| kevinbenton | ihrachys: right | 11:43 |
| ihrachys | so, save a copy, and expunge it on return? | 11:44 |
| ihrachys | I need to write some tests to validate that relationships are current, even if with a perf hit | 11:44 |
| kevinbenton | ihrachys: i think just change the @property decorator to call expire on the object before returing it | 11:45 |
| kevinbenton | ihrachys: not expunge | 11:45 |
| *** lilintan has joined #openstack-neutron | 11:45 | |
| kevinbenton | ihrachys: expunge will give them something detached | 11:45 |
| kevinbenton | ihrachys: which will be stale | 11:45 |
| ihrachys | kevinbenton: without copy, load_synthetic_... will see the model state as was when it was first time attached to an object | 11:46 |
| kevinbenton | ihrachys: what do you mean? | 11:46 |
| kevinbenton | ihrachys: it's stale? | 11:46 |
| *** jhershbe has quit IRC | 11:47 | |
| ihrachys | kevinbenton: the test does the following: create a subnet; create 3 dns servers; fetch subnet, which loads synthetic field for servers and is expected to load servers thru relationship; but the model that is returned from the sqla query for subnet does not contain servers probably because sqla returns the model we previously saved BEFORE servers were created. so the resulting subnet object misses servers on the field because | 11:48 |
| ihrachys | db_model.get('servers') returns [] | 11:48 |
| kevinbenton | right, but calling expire on db_model should cause its attributes to be reloaded | 11:49 |
| ihrachys | kevinbenton: right. but the test does not trigger db_obj property, so if we just put it there, it won't fix the test. we should also copy to make consequent fetches return a new model with proper relationship list. | 11:50 |
| *** obondarev has quit IRC | 11:50 | |
| *** wolverineav has joined #openstack-neutron | 11:51 | |
| kevinbenton | ihrachys: if we use copy.copy then all of the stuff is stale right? | 11:54 |
| *** jhershbe has joined #openstack-neutron | 11:54 | |
| *** pradk has joined #openstack-neutron | 11:55 | |
| *** wolverineav has quit IRC | 11:55 | |
| *** liyifeng has quit IRC | 11:55 | |
| *** moshele has joined #openstack-neutron | 11:58 | |
| ihrachys | kevinbenton: it's stale on the stored model only; but then the only place we'll use the stale model is in db_obj were we'll expire anyway, so it should not be an issue right? | 11:59 |
| *** imcsk8 has quit IRC | 11:59 | |
| *** rossella_s has quit IRC | 11:59 | |
| *** rossella_s has joined #openstack-neutron | 12:00 | |
| kevinbenton | ihrachys: no, if it's a copy i don't think calling expire will work anymore | 12:01 |
| kevinbenton | ihrachys: or did that work for you? | 12:02 |
| ihrachys | kevinbenton: I am writing a unit test to confirm | 12:02 |
| kevinbenton | ihrachys: can you try one more thing for me. at the end of https://github.com/openstack/neutron/blob/master/neutron/objects/db/api.py#L25 put .populate_existing() at the end of it and then don't use copy.copy() | 12:03 |
| kevinbenton | ihrachys: as i understand the docs http://docs.sqlalchemy.org/en/latest/orm/query.html#sqlalchemy.orm.query.Query.populate_existing | 12:03 |
| kevinbenton | ihrachys: that should update existing ones in the session rather than pull from them | 12:03 |
| kevinbenton | ihrachys: the issue is that getting by primary key makes it pull it out of the identity map and not even query the db | 12:04 |
| ihrachys | kevinbenton: you are right, copy results in sqlalchemy.exc.InvalidRequestError: Instance '<Trunk at 0x7fc950f48f00>' is not persistent within this Session | 12:04 |
| kevinbenton | ihrachys: so i'm hoping populate_existing() will get you the latest from the db without breaking the db reference in the old one | 12:04 |
| *** lenny__ has joined #openstack-neutron | 12:06 | |
| lenny__ | hi | 12:06 |
| *** johnbelamaric has quit IRC | 12:06 | |
| ihrachys | kevinbenton: yay | 12:07 |
| ihrachys | kevinbenton: at least it fixes the test | 12:07 |
| ihrachys | kevinbenton: now validating whether my NEW test for relationships on @db_obj to be current passes | 12:07 |
| *** liyifeng has joined #openstack-neutron | 12:08 | |
| lenny__ | hi | 12:08 |
| kevinbenton | lenny__: hi | 12:08 |
| lenny__ | how can i list the external ip adresses being used as external side on the gateway/router? | 12:08 |
| *** wolverineav has joined #openstack-neutron | 12:09 | |
| kevinbenton | lenny__: neutron router-show router-name | 12:09 |
| kevinbenton | lenny__: should have the gateway IP | 12:09 |
| lenny__ | kevinbenton: thx for your feedback. there is no general list command (so i do not have to create a loop) like floatingip-list? | 12:10 |
| kevinbenton | lenny__: are you an admin? | 12:11 |
| lenny__ | yes | 12:11 |
| kevinbenton | lenny__: if so, you can just list all of the ports on the external network | 12:11 |
| kevinbenton | lenny__: neutron port-list --network-id=external_net_id | 12:11 |
| lenny__ | kevinbenton: cool! :) thx | 12:11 |
| *** vijaykc4 has quit IRC | 12:12 | |
| kevinbenton | lenny__: you can add other columns to make it easy to see which routers they belong to | 12:12 |
| kevinbenton | lenny__: neutron port-list --network-id=external_net_id -c id -c fixed_ips -c device_id -c device_owner | 12:12 |
| kevinbenton | device_id is the id of the router | 12:13 |
| *** vijaykc4 has joined #openstack-neutron | 12:14 | |
| lenny__ | kevinbenton: cool, good tip, works a a charm :) | 12:14 |
| lenny__ | kevinbenton: thx! this is really appreciated | 12:14 |
| kevinbenton | lenny__: oh, you can also just list all routers as an admin as well | 12:14 |
| ihrachys | kevinbenton: ok, it seems to pass too. I have a single failure for qos policy to solve, but since it handles its rules field in a custom way, that's understandable. | 12:14 |
| kevinbenton | lenny__: neutron router-list -c id -c external_gateway_info | 12:14 |
| kevinbenton | lenny__: that will give you list of ids with IPs | 12:14 |
| ihrachys | kevinbenton: thanks a lot for pointers, I will upload a new revision once the test is fixed | 12:14 |
| kevinbenton | ihrachys: yeah, i suspect you should also add that statement to the get_objects query as well | 12:15 |
| *** boden has joined #openstack-neutron | 12:15 | |
| kevinbenton | ihrachys: because if it finds primary keys in a result in the identity map, it uses the version in the identity map | 12:15 |
| kevinbenton | ihrachys: http://docs.sqlalchemy.org/en/latest/orm/session_basics.html#is-the-session-a-cache | 12:15 |
| *** boden has quit IRC | 12:16 | |
| *** ijw has joined #openstack-neutron | 12:16 | |
| *** itisha has joined #openstack-neutron | 12:16 | |
| *** boden has joined #openstack-neutron | 12:16 | |
| *** boden has quit IRC | 12:16 | |
| *** jhershbe has quit IRC | 12:16 | |
| *** mohankumar has quit IRC | 12:18 | |
| lenny__ | kevinbenton: i will have a detailed look, thx again | 12:18 |
| kevinbenton | lenny__: no prob | 12:18 |
| *** boden has joined #openstack-neutron | 12:19 | |
| ihrachys | kevinbenton: right. in _get_collection, it's a tad harder because I don't have query available right away. | 12:19 |
| *** wolverineav has quit IRC | 12:20 | |
| kevinbenton | ihrachys: frankly i think this probably belongs in our general query builder | 12:20 |
| *** lucas-hungry is now known as lucasagomes | 12:20 | |
| kevinbenton | ihrachys: we've fought this issue before | 12:20 |
| kevinbenton | ihrachys: grep for context.session.expire in the code | 12:21 |
| kevinbenton | ihrachys: each one of those was likely associated with tears | 12:21 |
| ihrachys | :) | 12:21 |
| ihrachys | kevinbenton: do you suggest we kill all of them and apply populate_existing in common_db_mixin globally? | 12:21 |
| kevinbenton | ihrachys: yes | 12:22 |
| ihrachys | fine with me. I will craft something. any specific tests we would need to run/write for the change? | 12:22 |
| kevinbenton | ihrachys: actually just fix it for OVO for now if you can | 12:23 |
| kevinbenton | ihrachys: i want to do some more experimenting | 12:23 |
| ihrachys | fine with me. I will leave a TODO there to get back to it. | 12:23 |
| kevinbenton | ihrachys: sounds good | 12:24 |
| kevinbenton | ihrachys: keep an eye for wonky gate failures after this merges https://review.openstack.org/#/c/303966/ | 12:25 |
| kevinbenton | ihrachys: it's going to peel back the curtain on stuff that we were pretending was atomic but wasn't :) | 12:26 |
| *** moshele has quit IRC | 12:29 | |
| *** wolverineav has joined #openstack-neutron | 12:37 | |
| *** obondarev has joined #openstack-neutron | 12:41 | |
| *** wolverineav has quit IRC | 12:41 | |
| *** tpsilva has joined #openstack-neutron | 12:42 | |
| *** sindhu has joined #openstack-neutron | 12:44 | |
| *** kawa2014 has joined #openstack-neutron | 12:45 | |
| *** vijaykc4 has quit IRC | 12:46 | |
| *** ijw has quit IRC | 12:47 | |
| *** wolverineav has joined #openstack-neutron | 12:48 | |
| *** ijw has joined #openstack-neutron | 12:48 | |
| *** Leo_ has joined #openstack-neutron | 12:48 | |
| *** vijaykc4 has joined #openstack-neutron | 12:49 | |
| *** liyifeng has quit IRC | 12:52 | |
| *** wolverineav has quit IRC | 12:52 | |
| *** kawa2014 has quit IRC | 12:52 | |
| *** Leo_ has quit IRC | 12:53 | |
| *** edmondsw has joined #openstack-neutron | 12:54 | |
| *** amuller has joined #openstack-neutron | 12:55 | |
| *** sean-k-mooney has quit IRC | 12:59 | |
| *** pradk has quit IRC | 13:02 | |
| *** zkassab has joined #openstack-neutron | 13:03 | |
| *** manikanta_tadi has quit IRC | 13:04 | |
| *** tonytan4ever has joined #openstack-neutron | 13:04 | |
| *** links has quit IRC | 13:07 | |
| *** numans has quit IRC | 13:07 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Introduce bulk_push to rpc callback mechanism https://review.openstack.org/348476 | 13:07 |
| *** tonytan4ever has quit IRC | 13:08 | |
| john-davidge | carl_baldwin: haleyb: Hey guys, I've updated the subnet service types patch this morning https://review.openstack.org/#/c/337851 | 13:09 |
| *** ianw has quit IRC | 13:10 | |
| *** ianw has joined #openstack-neutron | 13:10 | |
| *** vijaykc4 has quit IRC | 13:10 | |
| *** ijw has quit IRC | 13:10 | |
| *** numans has joined #openstack-neutron | 13:10 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Introduce bulk push to rpc callback mechanism https://review.openstack.org/348476 | 13:11 |
| *** moshele has joined #openstack-neutron | 13:12 | |
| *** lilintan has quit IRC | 13:13 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: objects: expose database model for NeutronDbObject instances https://review.openstack.org/348279 | 13:13 |
| ajo | jlibosva, I posted a newer version ^ :) | 13:13 |
| ihrachys | kevinbenton: woohoo that should fix the problem ^ | 13:13 |
| jlibosva | ajo: saw, good one :) | 13:13 |
| jlibosva | ajo: is it still a wip? | 13:14 |
| ajo | jlibosva, not WIP now, I became serious about it :) | 13:14 |
| ajo | it's better to do it now that it's only used for one object, and only in-tree | 13:14 |
| ajo | :) | 13:14 |
| jlibosva | ajo: you have some todos there, those are ment for follow-ups? | 13:14 |
| ajo | jlibosva, 4 are for otaka, and I'm handlink the ValueError thing now | 13:15 |
| ajo | handling | 13:15 |
| ajo | (I forgot that one) | 13:15 |
| ajo | but feel free to check that what I'm writing makes sense, | 13:15 |
| *** gouthamr has joined #openstack-neutron | 13:18 | |
| *** boden has quit IRC | 13:20 | |
| *** wolverineav has joined #openstack-neutron | 13:20 | |
| *** buttercup has quit IRC | 13:21 | |
| *** cleong has joined #openstack-neutron | 13:22 | |
| jlibosva | ajo: some comments, I'm not sure about the backward compatibility | 13:24 |
| ihrachys | ajo: some comments from me too | 13:24 |
| jlibosva | ajo: I also think, we should somehow document that for other parties using callback mechanisms | 13:24 |
| ihrachys | I have concerns about compat | 13:24 |
| ajo | nobody else uses it yet | 13:24 |
| jlibosva | now it's two of us with concerns | 13:25 |
| ajo | concerners! | 13:25 |
| ajo | jlibosva, the callbacks mechanism is filtered so only QoSPolicies could be exposed over | 13:25 |
| ajo | (and that we exactly knew what was exposed), so far it's the only accepted resource to be pushed | 13:26 |
| ajo | so that scopes it all to qos | 13:26 |
| *** vijaykc4 has joined #openstack-neutron | 13:26 | |
| jlibosva | ajo: so you're saying there is no other project that is crazy enough to use this beautiful feature? | 13:26 |
| *** wolverineav has quit IRC | 13:27 | |
| ajo | considering no out of tree consumers of raw rpc callbacks, it should be fine, or if there is any undocumented one, the problem is non existing, since, we honour the old interface of having "resource" instead of resource_list for single items | 13:27 |
| ajo | jlibosva, we filtered the objects to have some control about what was being exposed over the wire with this :) | 13:27 |
| * ajo looks for the code | 13:27 | |
| jlibosva | ajo: I'm talking about other projects in the stadium for example | 13:28 |
| *** moshele has quit IRC | 13:28 | |
| ajo | I know OVN attempted that, and rolled back so they now use the qos notification driver | 13:28 |
| ajo | and same thing for nsx | 13:28 |
| *** wolverineav has joined #openstack-neutron | 13:29 | |
| ajo | jlibosva, didn't we have a code search thingy in openstack? | 13:29 |
| ajo | we could use that to verify | 13:29 |
| jlibosva | we do | 13:29 |
| russellb | http://codesearch.openstack.org/ | 13:29 |
| jlibosva | ajo: amuller knows | 13:29 |
| jlibosva | also russellb knows :) | 13:30 |
| jlibosva | thanks | 13:30 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Change external_network_bridge default to '' https://review.openstack.org/298443 | 13:30 |
| amuller | that's my favorite website :) | 13:30 |
| ajo | anyway, the scope of the danger is tiny, if broken, they change the callback to iterate over a list | 13:30 |
| russellb | i irc highlight on "OVN" heh | 13:30 |
| ajo | amuller, russellb if it had code xrefs links that'd be just plain amazing | 13:30 |
| ajo | ahh nsx http://codesearch.openstack.org/?q=neutron.api.rpc.callback&i=nope&files=&repos= | 13:31 |
| ajo | garyk, ^ | 13:31 |
| *** andymaier has quit IRC | 13:32 | |
| ajo | ok, nsx would have to patch this: http://git.openstack.org/cgit/openstack/vmware-nsx/tree/vmware_nsx/services/qos/nsx_v3/utils.py#n34 | 13:32 |
| ajo | and that's it | 13:32 |
| ajo | the damage is controlled, and we don't twist the code too much. | 13:33 |
| jlibosva | ajo: I think if we will be more paranoid about the push parameters and make it generic, it wouldn't hurt | 13:33 |
| jlibosva | maybe I'm too paranoid | 13:33 |
| *** wolverineav has quit IRC | 13:33 | |
| ajo | jlibosva, what do you mean exactly? | 13:33 |
| *** wolverineav has joined #openstack-neutron | 13:34 | |
| ajo | amuller, , russellb agreed, codesearch is cool :D | 13:34 |
| jlibosva | ajo: like introduce resource parameter that defaults to None, resource_list also defaults to None, and check that resource_list is a list type. | 13:34 |
| jlibosva | ajo: if resouce is used or resource_list is not a list type, it's someone that is using old code | 13:34 |
| ajo | jlibosva, yup, that's another option | 13:35 |
| ajo | I had a little ihrachys in my head telling me that lots of =None parameters in functions are hated | 13:35 |
| ajo | :D | 13:35 |
| ajo | lol | 13:35 |
| ajo | :) | 13:35 |
| jlibosva | :D | 13:35 |
| ajo | also **kwargs isn't the nicest thing, but ok :) | 13:35 |
| jlibosva | ajo: yeah, I think the bad thing is that you'll need to set all other parameters with default values :D | 13:36 |
| ajo | ah, like when calling the function with just the list of params, and no param names | 13:36 |
| russellb | fun codesearch game ... come up with a regex that finds all cases of an obvious bug / anti-pattern | 13:36 |
| ihrachys | ajo: I don't follow why not adding a new method and retiring the old one | 13:36 |
| ajo | ihrachys, that's a very good strategy when you have a lot of API users, and want to do a deprecation cycle | 13:37 |
| ajo | ihrachys, but if your external user is just one, and this is not neutron_lib but a neutron internal... | 13:38 |
| *** sridharg has quit IRC | 13:38 | |
| ihrachys | I don't buy it, sorry. we released a thing, we are supposed to maintain it. | 13:38 |
| *** tflynn has joined #openstack-neutron | 13:38 | |
| ajo | I think it's acceptable to tradeoff for a tiny breackage | 13:38 |
| ihrachys | anyhow, you break old agents, even in tree | 13:38 |
| *** wolverineav has quit IRC | 13:38 | |
| ajo | ihrachys, no I don't | 13:38 |
| ihrachys | you do. they will catch tracebacks. | 13:39 |
| ajo | or I think I don't may be I do, please explain :) | 13:39 |
| ihrachys | because their push method will fail to make sense of the payload | 13:39 |
| ihrachys | so for those 'new' messages, you will trigger logs errors. it's not like qos will be broken, but there are more facets of a 'breakage' than just 'crashes and burns' | 13:40 |
| ajo | ihrachys, hmm, no | 13:40 |
| ajo | ihrachys, no, because the scope of the push is per topic queue | 13:40 |
| *** jamesdenton has joined #openstack-neutron | 13:40 | |
| ajo | and the qos policy queue will only receive "resource" parameter | 13:40 |
| ajo | since we only push single items | 13:41 |
| ajo | and there's no plan to change that into lists | 13:41 |
| ajo | plan, or reason at all | 13:41 |
| ajo | so | 13:41 |
| *** janzian has joined #openstack-neutron | 13:41 | |
| ajo | on the neutron-object-qos_policy-<ver> topic, pushes will stay on the old "resource" parameter, | 13:41 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Use single priority for floating IP rules https://review.openstack.org/312790 | 13:42 |
| ihrachys | ? you have a TODO to switch to resource_lists in O | 13:42 |
| ajo | then, in newton, the agents will understand also resource_list even if it's not used | 13:42 |
| *** catintheroof has quit IRC | 13:42 | |
| ajo | anyeah | 13:42 |
| ajo | and yeah | 13:42 |
| *** Leo_ has joined #openstack-neutron | 13:42 | |
| ajo | in otaka, we kill the if branch, and | 13:42 |
| ajo | all goes via resource_lists | 13:42 |
| ajo | because agents on newton understand both | 13:42 |
| ihrachys | sec, digesting | 13:42 |
| *** buttercup has joined #openstack-neutron | 13:42 | |
| ajo | agents in otaka understand resource_list | 13:42 |
| *** Leom has joined #openstack-neutron | 13:43 | |
| ajo | yeah, I had to draw it a couple of times here to be sure, | 13:43 |
| *** tflynn has quit IRC | 13:43 | |
| ihrachys | ok, I think I see where you go | 13:43 |
| ajo | the only constraint to this | 13:44 |
| ajo | is that we won't start sending QoSPolicies pushes of more than one element during newton | 13:44 |
| ajo | and we won't | 13:44 |
| openstackgerrit | Hynek Mlnarik proposed openstack/neutron: Set secure fail mode for physical bridges https://review.openstack.org/348889 | 13:44 |
| jlibosva | hynekm: I love those unittests | 13:45 |
| ihrachys | ajo: I think you are right, updated the patch with comments | 13:46 |
| ihrachys | lemme read tests thru | 13:46 |
| ajo | if we find strong reasons to do a full deprecation cycle, I'm ok to do that, but may be a breakage document, and syncing with garyk to see it's fine with nsx would avoid burning time. | 13:46 |
| ajo | ihrachys, ack, thanks a lot for spending time trying to understand it :) | 13:46 |
| ajo | brb, | 13:47 |
| *** Leo_ has quit IRC | 13:47 | |
| ihrachys | ajo: I am not completely clear what we do with versions. I think that for new messages, we should set version 1.1 | 13:47 |
| ihrachys | ajo: and bump client versions to 1.1 so that they are ready to handle the messages. | 13:47 |
| ajo | ihrachys, no, for server>agent direction we don't bump | 13:47 |
| ajo | or I'm missing something , we must review that | 13:48 |
| ihrachys | ajo: why. we modified signature for some messages, so that would justify the bump, no? | 13:48 |
| ajo | but if we bump the server side to 1.1, agent stays 1.0 | 13:48 |
| hynekm | jlibosva, it's wonderful to become able to make sense of them | 13:48 |
| ajo | we upgrade server... | 13:48 |
| ihrachys | ajo: only for new messages | 13:48 |
| ajo | server is unable to call agent | 13:48 |
| ihrachys | ajo: meaning, pass version=... to casr | 13:48 |
| ihrachys | *cast | 13:48 |
| ajo | ahh | 13:48 |
| ajo | true | 13:48 |
| *** dansmith is now known as superdan | 13:49 | |
| ihrachys | then if there are for some reason some agents that listen on the new topic but don't specify the version of 1.1, they will have a nice log error instead of a traceback | 13:49 |
| jlibosva | hynekm: I'm also sure otherwiseguy will like those | 13:49 |
| ihrachys | I don't think it's too important, but makes the logic more clean. | 13:49 |
| *** akshai has joined #openstack-neutron | 13:50 | |
| *** wolverineav has joined #openstack-neutron | 13:51 | |
| jlibosva | hynekm: do you also plan to add the fullstack test we talked about yesterday? I think it should come with that patch | 13:53 |
| *** moshele has joined #openstack-neutron | 13:54 | |
| *** obondarev has quit IRC | 13:54 | |
| hynekm | jlibosva, it's already there - https://review.openstack.org/#/c/334926/ so I'd like it to be included once 348889 patch would get in | 13:54 |
| *** pgadiya has quit IRC | 13:54 | |
| hynekm | jlibosva, or you mean the one with blocking traffic for 15 secs? | 13:55 |
| jlibosva | hynekm: it doesn't cover the case where you don't restart agent | 13:55 |
| jlibosva | hynekm: yes, that one | 13:55 |
| *** jianghuaw has quit IRC | 13:55 | |
| *** jianghuaw has joined #openstack-neutron | 13:56 | |
| *** edmondsw has quit IRC | 13:56 | |
| hynekm | yes, that one will be added, I'll create it on Monday | 13:56 |
| hynekm | jlibosva, ^ | 13:56 |
| *** nlahouti has joined #openstack-neutron | 13:56 | |
| jlibosva | hynekm: acknowledged, you gonna mark it as WIP? | 13:57 |
| hynekm | jlibosva, either that or a separate patch, I'd prefer the latter but don't mind the former too | 13:57 |
| *** amotoki has quit IRC | 13:58 | |
| jlibosva | hynekm: I think it makes sense to have it altogether as it shows it actually fixes the issue | 13:58 |
| hynekm | jlibosva, ack | 13:59 |
| *** pradk has joined #openstack-neutron | 14:00 | |
| ihrachys | ajo: ok, more comments. you miss some pieces of the puzzle, like updating rpc callbacks subscribers in tree. | 14:00 |
| *** claudiub|2 has quit IRC | 14:01 | |
| *** zengine has joined #openstack-neutron | 14:03 | |
| *** wolverineav has quit IRC | 14:04 | |
| *** zengine has quit IRC | 14:04 | |
| *** sridharg has joined #openstack-neutron | 14:04 | |
| *** lenny__ has quit IRC | 14:05 | |
| *** akshai has quit IRC | 14:06 | |
| ajo | ihrachys, I thought I had updated those | 14:07 |
| *** akshai has joined #openstack-neutron | 14:07 | |
| ajo | but let's see what I missed.. | 14:07 |
| *** yb has joined #openstack-neutron | 14:07 | |
| *** a_ta has joined #openstack-neutron | 14:07 | |
| ihrachys | ajo: you touched server side only | 14:07 |
| ajo | hmmm, ahaa | 14:08 |
| *** wolverineav has joined #openstack-neutron | 14:08 | |
| ajo | ihrachys, about batching by resource type, I'd leave that to a later patch if necessary, I thought of that, but I wanted the thing to be atomic, I mean, if you send the pack, you expect the same thing at the other side... | 14:08 |
| ajo | if you break in batches, the callbacks get something different | 14:08 |
| ajo | but | 14:08 |
| *** obondarev has joined #openstack-neutron | 14:09 | |
| ajo | may be at some point we may need to batch big lists if necessary for payload size? | 14:09 |
| ihrachys | it should not be hard to test, I suppose. | 14:09 |
| ihrachys | ajo: that could be useful so that plugins could populate a storage with objects, then flush all of them at the very end. | 14:09 |
| ajo | hmm | 14:10 |
| ajo | yes | 14:10 |
| ihrachys | ajo: then you could trigger notify right from a transaction. | 14:10 |
| *** nlahouti has quit IRC | 14:10 | |
| ajo | that usage pattern would be very neat in fact | 14:10 |
| ajo | ihrachys, can I iterate a second patch for that? | 14:10 |
| ihrachys | you mean on top? | 14:10 |
| ajo | I don't want to make this one too complicated | 14:11 |
| ihrachys | why? you still need to respin | 14:11 |
| ajo | yup | 14:11 |
| ihrachys | it shouldn't be hard | 14:11 |
| ajo | ok, let me try | 14:11 |
| ajo | :) | 14:11 |
| ihrachys | :) | 14:11 |
| *** wolverineav has quit IRC | 14:13 | |
| amuller | ajo: I may have missed the conversation, but about https://review.openstack.org/#/c/348889/ | 14:13 |
| amuller | ajo: didn't you set the fail mode to a different value in the past? it was about broadcast storms? | 14:13 |
| ajo | amuller, yes, I did, but we had it only for br-int and br-tun | 14:14 |
| *** vijaykc4 has quit IRC | 14:14 | |
| ajo | br-tun was the responsible for generating the broadcast storms | 14:14 |
| ajo | I think all the br-tuns in diferent hypervisors | 14:14 |
| amuller | ajo: what value did you set it to? | 14:14 |
| ajo | started broadcasting to each other any multicast/broadcast packet | 14:14 |
| ajo | and boom | 14:14 |
| ajo | secure_mode=True | 14:14 |
| ajo | that's it :) | 14:14 |
| *** sridharg has quit IRC | 14:15 | |
| ajo | amuller, but for br-tun it was on a reset_bridge | 14:15 |
| ajo | or something like that | 14:15 |
| amuller | ajo: so now we'll be setting *all* bridges to secure=True | 14:15 |
| ajo | yes | 14:15 |
| ajo | that should be fine, we must verify that the br-<phys> tunnels don't relay on the default normal rule, | 14:15 |
| ajo | and if they do, we insert it | 14:16 |
| *** b_bezak has joined #openstack-neutron | 14:16 | |
| ajo | hynekm, ^ | 14:16 |
| ajo | do we test provider networks? | 14:16 |
| jlibosva | ajo: you mean with br-exes? | 14:17 |
| ajo | jlibosva, provider network bridges | 14:18 |
| ajo | br-eth1, br-eth0 br-whatever :) | 14:19 |
| *** hynekm has quit IRC | 14:19 | |
| ajo | I'd check that br-ex is treated equally | 14:19 |
| *** tbachman has quit IRC | 14:19 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: objects: expose database model for NeutronDbObject instances https://review.openstack.org/348279 | 14:20 |
| ihrachys | rossella_s: I hope ^ clarified | 14:20 |
| rossella_s | ihrachys, it looks great, thanks! | 14:21 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: objects: expose database model for NeutronDbObject instances https://review.openstack.org/348279 | 14:22 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron: Add QoS minimum bandwidth rule for instance egress traffic https://review.openstack.org/344145 | 14:22 |
| ihrachys | there was a small pep8 glitch.. | 14:23 |
| *** johnbelamaric has joined #openstack-neutron | 14:23 | |
| ihrachys | ajo: where should I start review for qos? from the validation patch? | 14:23 |
| ajo | 1 sec | 14:24 |
| ajo | ihrachys, https://review.openstack.org/#/c/319694/ | 14:24 |
| ihrachys | ok, will do | 14:24 |
| ajo | thanks ihrachys | 14:24 |
| *** wolverineav has joined #openstack-neutron | 14:26 | |
| *** moshele has quit IRC | 14:26 | |
| openstackgerrit | Sindhu Devale proposed openstack/neutron: Refactoring agent linux&ovsdb config https://review.openstack.org/347867 | 14:26 |
| *** yb has quit IRC | 14:28 | |
| *** banix has joined #openstack-neutron | 14:31 | |
| *** sindhu has quit IRC | 14:31 | |
| openstackgerrit | Richard Theis proposed openstack/neutron: Fix L3 NAT DB signature mismatch https://review.openstack.org/348558 | 14:31 |
| *** sindhu has joined #openstack-neutron | 14:32 | |
| *** claudiub|2 has joined #openstack-neutron | 14:34 | |
| *** edmondsw has joined #openstack-neutron | 14:34 | |
| *** tonytan4ever has joined #openstack-neutron | 14:35 | |
| *** wolverineav has quit IRC | 14:35 | |
| *** wolverineav has joined #openstack-neutron | 14:37 | |
| *** zhhuabj has quit IRC | 14:40 | |
| *** shausy has quit IRC | 14:40 | |
| *** Guest31932 has quit IRC | 14:42 | |
| *** dane_leblanc has joined #openstack-neutron | 14:42 | |
| *** david-lyle has joined #openstack-neutron | 14:42 | |
| *** banix has quit IRC | 14:45 | |
| *** wolverineav has quit IRC | 14:45 | |
| *** david-lyle_ has joined #openstack-neutron | 14:46 | |
| *** david-lyle has quit IRC | 14:47 | |
| *** banix has joined #openstack-neutron | 14:47 | |
| *** david-lyle__ has joined #openstack-neutron | 14:48 | |
| *** vijaykc4 has joined #openstack-neutron | 14:51 | |
| *** zhhuabj has joined #openstack-neutron | 14:51 | |
| *** david-lyle_ has quit IRC | 14:52 | |
| *** vhoward has joined #openstack-neutron | 14:52 | |
| *** vijaykc4 has quit IRC | 14:54 | |
| *** mosulica has quit IRC | 14:56 | |
| *** wolverineav has joined #openstack-neutron | 14:57 | |
| *** pbandark has quit IRC | 14:57 | |
| *** jlibosva has quit IRC | 14:59 | |
| *** garyk has quit IRC | 15:01 | |
| *** b_bezak has quit IRC | 15:02 | |
| *** gvrangan has joined #openstack-neutron | 15:02 | |
| *** mhickey has quit IRC | 15:05 | |
| *** dasanind has joined #openstack-neutron | 15:07 | |
| *** banix has quit IRC | 15:07 | |
| *** pbandark has joined #openstack-neutron | 15:09 | |
| *** elopez has joined #openstack-neutron | 15:09 | |
| *** elopez is now known as Guest21368 | 15:10 | |
| openstackgerrit | Aradhana Singh proposed openstack/neutron: Refactoring config options for cmd https://review.openstack.org/323046 | 15:10 |
| *** singhj has joined #openstack-neutron | 15:10 | |
| *** jistr is now known as jistr|call | 15:10 | |
| *** wolverineav has quit IRC | 15:12 | |
| *** armax has joined #openstack-neutron | 15:12 | |
| *** mickeys has joined #openstack-neutron | 15:16 | |
| *** jistr|call is now known as jistr | 15:17 | |
| *** vikram has quit IRC | 15:20 | |
| *** david-lyle has joined #openstack-neutron | 15:23 | |
| *** david-lyle__ has quit IRC | 15:23 | |
| *** wolverineav has joined #openstack-neutron | 15:24 | |
| *** dasanind has quit IRC | 15:25 | |
| *** dasanind has joined #openstack-neutron | 15:25 | |
| *** tflynn has joined #openstack-neutron | 15:26 | |
| *** wolverineav has quit IRC | 15:29 | |
| *** ijw has joined #openstack-neutron | 15:30 | |
| *** tflynn has quit IRC | 15:31 | |
| *** mlavalle has joined #openstack-neutron | 15:32 | |
| *** mlavalle has quit IRC | 15:32 | |
| *** marst has quit IRC | 15:33 | |
| *** mlavalle has joined #openstack-neutron | 15:33 | |
| *** vthapar has quit IRC | 15:34 | |
| *** yamamoto has joined #openstack-neutron | 15:34 | |
| *** jckasper has quit IRC | 15:35 | |
| *** mlavalle has quit IRC | 15:35 | |
| *** zhhuabj has quit IRC | 15:36 | |
| *** jckasper has joined #openstack-neutron | 15:36 | |
| *** jlanoux has quit IRC | 15:36 | |
| *** jckasper has quit IRC | 15:36 | |
| openstackgerrit | Richard Theis proposed openstack/neutron: Support callbacks for L3 plugins without an agent https://review.openstack.org/348933 | 15:37 |
| *** wolverineav has joined #openstack-neutron | 15:37 | |
| *** chandankumar has quit IRC | 15:38 | |
| *** mlavalle has joined #openstack-neutron | 15:38 | |
| *** zhhuabj has joined #openstack-neutron | 15:39 | |
| *** ijw has quit IRC | 15:40 | |
| openstackgerrit | John Davidge proposed openstack/neutron: Enable CRUD for Subnet Service Types https://review.openstack.org/337851 | 15:40 |
| *** namnh has joined #openstack-neutron | 15:43 | |
| ihrachys | ajo: ok posted some comments for the validation patch. I think we should tild design somewhat. | 15:43 |
| ajo | what do you mean by tild? | 15:44 |
| ajo | thanks for the review btw | 15:44 |
| ajo | will check as soon as I push another "push" :) | 15:44 |
| haleyb | john-davidge: i'm glad i'm slow and didn't get to PS13 :) I'll take a look and rebase the osc patch | 15:44 |
| *** wolverineav has quit IRC | 15:45 | |
| ajo | ihrachys, ^ | 15:45 |
| *** boden has joined #openstack-neutron | 15:45 | |
| ihrachys | ajo: sorry, tilt | 15:46 |
| john-davidge | haleyb: :) | 15:46 |
| *** jckasper has joined #openstack-neutron | 15:46 | |
| ajo | ihrachys, ack, I'll check it out :) | 15:46 |
| *** jckasper has quit IRC | 15:46 | |
| haleyb | john-davidge: thanks for keeping it updated btw | 15:47 |
| ajo | ihrachys, I disagree about the notification driver, in 1st place, it shouldn't be called notification driver, but qos_driver :/ | 15:47 |
| john-davidge | haleyb: Although since it's a cross-project dependency, does it actually need rebasing? Perhaps it only needs a recheck? | 15:47 |
| ajo | there's where confusion comes from :/ | 15:47 |
| ajo | in the current model, where qos service calls "the backend" using notification_driver, it's just a qos_driver (service side) | 15:48 |
| ihrachys | ajo: in the end, your core plugin is supposed to know what to do with ports, that's why it should own the validation. | 15:48 |
| *** liuyulong has quit IRC | 15:48 | |
| ajo | ihrachys, yeah, that makes sense | 15:48 |
| ajo | I'll read your comments and let it settle through the weekend ;) | 15:48 |
| ihrachys | sure | 15:48 |
| ajo | first, pushing push :] | 15:48 |
| haleyb | john-davidge: it needs an update i think anyways, but i do see some jenkins complaint | 15:49 |
| *** wolverineav has joined #openstack-neutron | 15:49 | |
| *** liuyulong has joined #openstack-neutron | 15:50 | |
| ajo | ihrachys, allowing mixed objects was easy to implement, PITA to test :P | 15:50 |
| ajo | :) | 15:50 |
| john-davidge | haleyb: I think that was from when the patch was in merge conflict, but I could be wrong | 15:50 |
| *** gvrangan has quit IRC | 15:50 | |
| ajo | ihrachys, still on it, good part is that test implementation is cleaning up the test code to group common stuff | 15:51 |
| *** marst has joined #openstack-neutron | 15:52 | |
| *** janzian has quit IRC | 15:52 | |
| *** Guest21368 has quit IRC | 15:55 | |
| *** jckasper has joined #openstack-neutron | 15:56 | |
| *** wolverineav has quit IRC | 15:57 | |
| *** vishwanathj has joined #openstack-neutron | 15:57 | |
| openstackgerrit | Merged openstack/neutron: Increment revision numbers on object changes https://review.openstack.org/303966 | 15:58 |
| *** dasanind has quit IRC | 15:58 | |
| ihrachys | kevinbenton: ^ bring on the apocalypse! | 15:59 |
| *** sdague has quit IRC | 15:59 | |
| *** rossella_s has quit IRC | 15:59 | |
| ihrachys | armax: ^ you should be aware of the patch landed. it may reveal some unknown issues in the code, we'll see how gate behaves | 16:00 |
| *** rossella_s has joined #openstack-neutron | 16:00 | |
| *** nlahouti has joined #openstack-neutron | 16:00 | |
| *** pcaruana has quit IRC | 16:01 | |
| *** john-davidge has quit IRC | 16:01 | |
| *** wolverineav has joined #openstack-neutron | 16:02 | |
| *** john-davidge has joined #openstack-neutron | 16:02 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/348648 | 16:03 |
| openstackgerrit | Sergey Belous proposed openstack/neutron-specs: Extend Quota API to report usage statistics https://review.openstack.org/348947 | 16:03 |
| openstackgerrit | Merged openstack/neutron-vpnaas: Fix API Tests https://review.openstack.org/335838 | 16:03 |
| *** nlahouti1 has joined #openstack-neutron | 16:03 | |
| *** pbandark has quit IRC | 16:05 | |
| *** nlahouti has quit IRC | 16:05 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: DVR: Clean stale snat-ns by checking its existence when agent restarts https://review.openstack.org/326729 | 16:08 |
| openstackgerrit | Merged openstack/neutron: Remove deprecated network_device_mtu option https://review.openstack.org/342958 | 16:09 |
| ihrachys | sc68cal: kevinbenton: Sam-I-Am: ^ die witch die | 16:09 |
| *** janzian has joined #openstack-neutron | 16:09 | |
| sc68cal | Woooo!!!! | 16:09 |
| *** wolverineav has quit IRC | 16:12 | |
| *** numans has quit IRC | 16:12 | |
| *** bvandewa has joined #openstack-neutron | 16:14 | |
| *** bvandewa has quit IRC | 16:15 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Introduce bulk push to rpc callback mechanism https://review.openstack.org/348476 | 16:16 |
| *** chandankumar has joined #openstack-neutron | 16:16 | |
| *** wolverineav has joined #openstack-neutron | 16:17 | |
| *** ygbo has quit IRC | 16:17 | |
| ajo | ihrachys, ^ there it is | 16:18 |
| *** dasanind has joined #openstack-neutron | 16:21 | |
| openstackgerrit | Merged openstack/neutron: Don't use exponential back-off for report_state https://review.openstack.org/347708 | 16:21 |
| Sam-I-Am | sc68cal: woohoo | 16:23 |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: WIP Tickle a problem with rollback trying to reuse the session https://review.openstack.org/348956 | 16:23 |
| *** namnh has quit IRC | 16:24 | |
| *** wolverineav has quit IRC | 16:25 | |
| openstackgerrit | Sreekumar S proposed openstack/neutron: Fix for Cookie Bridge not applying ext cookie to flow ops https://review.openstack.org/326637 | 16:26 |
| *** Leom has quit IRC | 16:26 | |
| *** sambetts is now known as sambetts|afk | 16:27 | |
| *** wolverineav has joined #openstack-neutron | 16:27 | |
| *** jckasper has quit IRC | 16:27 | |
| *** jckasper has joined #openstack-neutron | 16:28 | |
| *** jckasper has quit IRC | 16:29 | |
| *** tesseract- has quit IRC | 16:30 | |
| *** eezhova has quit IRC | 16:30 | |
| *** permalac has quit IRC | 16:31 | |
| armax | ihrachys: which one? | 16:31 |
| armax | https://review.openstack.org/#/c/303966/? | 16:31 |
| ihrachys | armax: the one that bumps revision numbers | 16:31 |
| ihrachys | yes, that one | 16:31 |
| armax | ihrachys: we’ll get kevinbenton to fix any issue | 16:31 |
| ihrachys | we'll do! | 16:31 |
| ihrachys | :) | 16:31 |
| ajo | armax, so 4096 subports serialized are 1255751 bytes (1.2MB) | 16:32 |
| ajo | big chunk of RPC, yes | 16:32 |
| *** jamielennox|away is now known as jamielennox | 16:32 | |
| armax | ajo: still manaageable though | 16:32 |
| armax | ajo: don’t you reckon? | 16:32 |
| ajo | yes, not super crazy, but it feels like it takes >10secs to handle it, let me check time | 16:33 |
| ajo | or may be it's because creation... | 16:33 |
| ajo | let's pdb | 16:33 |
| ajo | I hope rabbitmq doesn't handle fanouts by copying stuff into different queues instead of keeping some internal reference, | 16:34 |
| ajo | :) | 16:34 |
| ajo | that'd be 1.2MB * number of agents (boom) :) | 16:34 |
| *** kobis has quit IRC | 16:35 | |
| armax | ajo: I am sure we would have found out about this by now! | 16:35 |
| ajo | armax, they better did :D | 16:36 |
| armax | ajo: so are you gonna respin to expect list of heterogeneous objects? | 16:36 |
| ajo | armax, it's done already :) | 16:36 |
| *** wolverineav has quit IRC | 16:36 | |
| ajo | tests got a bit more complicated than the code to do it (which was trivial), but test code is sightly cleaner now too | 16:36 |
| *** wolverineav has joined #openstack-neutron | 16:37 | |
| *** wolverineav has quit IRC | 16:37 | |
| armax | ajo: I am not sure I get it though | 16:37 |
| openstackgerrit | Richard Theis proposed openstack/neutron: Support callbacks for L3 plugins without an agent https://review.openstack.org/348933 | 16:37 |
| ajo | nah, db fetch and serialization of the chunk is very tiny <2s in my system | 16:37 |
| *** iyamahat has joined #openstack-neutron | 16:37 | |
| armax | if the list is heterogeneous, what is the resource do we subscribe to? | 16:38 |
| *** wolverineav has joined #openstack-neutron | 16:38 | |
| *** wolverineav has quit IRC | 16:38 | |
| ajo | armax, all of the possible objects | 16:38 |
| ihrachys | ajo: -1 but nothing critical, please take a look | 16:38 |
| *** wolverineav has joined #openstack-neutron | 16:38 | |
| ajo | armax, as they go in different queue topics, they need to be sent per-type | 16:38 |
| *** wolverineav has quit IRC | 16:39 | |
| ajo | in different underlying push calls | 16:39 |
| ajo | ihrachys, ^ | 16:39 |
| ajo | thanks ihrachys | 16:39 |
| ajo | reading | 16:39 |
| *** wolverineav has joined #openstack-neutron | 16:39 | |
| *** wolverineav has quit IRC | 16:39 | |
| ajo | the non-heterogenous thing is a YAGNI, but ihrachys thinks could be valuable soon | 16:39 |
| *** jckasper has joined #openstack-neutron | 16:39 | |
| ihrachys | armax: from server perspective, we don't care if there are subscribers at all. we just push crap on the wire. | 16:39 |
| *** wolverineav has joined #openstack-neutron | 16:39 | |
| armax | ajo: ok | 16:39 |
| armax | ihrachys: I don’t disagree | 16:40 |
| *** wolverineav has quit IRC | 16:40 | |
| armax | ihrachys: from a consumer point of view | 16:40 |
| *** wolverineav has joined #openstack-neutron | 16:40 | |
| *** wolverineav has quit IRC | 16:40 | |
| ihrachys | armax: I still owe you an etherpad update. oh boy. | 16:40 |
| ihrachys | on it | 16:40 |
| armax | ihrachys: I think the resulting rpc callback registry API may not be super intuitive | 16:40 |
| ajo | armax, yes, you lose the atomicity of the list :/ | 16:40 |
| *** wolverineav has joined #openstack-neutron | 16:40 | |
| armax | ah | 16:40 |
| ajo | I guess it can be reasonable as soon as we document it | 16:40 |
| *** lucasagomes is now known as lucas-dinner | 16:41 | |
| *** wolverineav has quit IRC | 16:41 | |
| *** wolverineav has joined #openstack-neutron | 16:41 | |
| *** wolverineav has quit IRC | 16:41 | |
| armax | ajo: right, so what does a callback have to do to receive an heterogenous list? | 16:41 |
| ihrachys | armax: it won't | 16:41 |
| ajo | armax, it wont | 16:41 |
| ajo | imagine you have resource A and B | 16:41 |
| *** wolverineav has joined #openstack-neutron | 16:41 | |
| ihrachys | armax: it will always get the chunk that it cares | 16:41 |
| *** wolverineav has quit IRC | 16:42 | |
| ihrachys | if it wants both, it subscribes to both and get it in separate chunks | 16:42 |
| ajo | and you push [A(id=1), A(id=2), B(id=1), A(id=3)] | 16:42 |
| armax | ihrachys: but a callaback must subscribe to receive stuff | 16:42 |
| armax | no? | 16:42 |
| *** wolverineav has joined #openstack-neutron | 16:42 | |
| *** wolverineav has quit IRC | 16:42 | |
| ajo | your agent is subscribed to A and B | 16:42 |
| ajo | it get's two callback calls | 16:42 |
| ihrachys | yes, it will pick and choose what it cares about | 16:42 |
| armax | oh | 16:42 |
| ajo | one for A with [A(id=1), A(id=2), A(id=3)] | 16:42 |
| armax | so the callback always see the chunk for a given type? | 16:42 |
| *** wolverineav has joined #openstack-neutron | 16:42 | |
| ajo | another for B with [B(id=1)] | 16:42 |
| *** wolverineav has quit IRC | 16:43 | |
| ihrachys | so your server will eg be able to send both subports and trunks and maybe a flying saucer, and agents will receive just subports. along those lines. | 16:43 |
| *** wolverineav has joined #openstack-neutron | 16:43 | |
| *** wolverineav has quit IRC | 16:43 | |
| ihrachys | armax: yes, it's just a sugar on server side to allow to call registry once instead of multiple times | 16:43 |
| ajo | armax, yes, comes from the limitation of how we do things in the low level (one separate queue for resource type and resource version combination) | 16:43 |
| *** wolverineav has joined #openstack-neutron | 16:43 | |
| armax | ok so a bulky heternogenous push results into the receivers side into local bulky homogeneous pushes? | 16:43 |
| ihrachys | it could as well do N times where N is the number of resource types to update. | 16:43 |
| *** wolverineav has quit IRC | 16:44 | |
| *** wolverineav has joined #openstack-neutron | 16:44 | |
| *** wolverineav has quit IRC | 16:44 | |
| ajo | armax, that's correct | 16:44 |
| ihrachys | + | 16:44 |
| armax | ajo: well. if you’re ok with this level of complexity I am not gonna judge :) | 16:44 |
| *** wolverineav has joined #openstack-neutron | 16:44 | |
| *** wolverineav has quit IRC | 16:45 | |
| ajo | Yes, It's reasonable I think, the code complexity increase was very low | 16:45 |
| *** wolverineav has joined #openstack-neutron | 16:45 | |
| *** wolverineav has quit IRC | 16:45 | |
| ihrachys | in the future we could have a deferred transactional notifier that you could update from under transaction with objects, and when its __exit__ is triggered, we push it into registry in one call. | 16:45 |
| armax | the only reserve I have is the fact that this behavior is not clear from the registry API | 16:45 |
| armax | that’s all | 16:45 |
| ihrachys | ajo: it was either that, or validation for the same type | 16:45 |
| ajo | most pain came from the tests, which were assuming a single resource type in mocks (FakeResource), now we have Fake, and Faker resources... | 16:45 |
| armax | I can see that the code is only slightly more complex | 16:45 |
| *** wolverineav has joined #openstack-neutron | 16:45 | |
| ihrachys | armax: we may want to describe it in more details in devref | 16:45 |
| ihrachys | ajo: ^ | 16:45 |
| *** wolverineav has quit IRC | 16:45 | |
| ajo | yes, I can do that, and also in the push docstring | 16:46 |
| *** wolverineav has joined #openstack-neutron | 16:46 | |
| *** wolverineav has quit IRC | 16:46 | |
| armax | ultimately the receiver would always see a bulky homogenous list | 16:46 |
| ajo | if all this sounds reasonable | 16:46 |
| armax | but what’s not clear though | 16:46 |
| *** wolverineav has joined #openstack-neutron | 16:46 | |
| armax | is that the calls are a result of a single push from the producer side | 16:46 |
| *** wolverineav has quit IRC | 16:47 | |
| armax | I think this may be potentially confusing | 16:47 |
| *** wolverineav has joined #openstack-neutron | 16:47 | |
| *** wolverineav has quit IRC | 16:47 | |
| armax | so overall I can see how powerful this is | 16:47 |
| armax | but it’s a lot tricker to handle and understand | 16:47 |
| armax | so I wonder whether it’s gonna backfire | 16:47 |
| *** neiljerram has joined #openstack-neutron | 16:47 | |
| ajo | armax, I agree on that part | 16:47 |
| *** wolverineav has joined #openstack-neutron | 16:47 | |
| openstackgerrit | Richard Theis proposed openstack/neutron: Support callbacks for L3 plugins without an agent https://review.openstack.org/348933 | 16:48 |
| ajo | I wonder if there's any value in having a separate function for homogeneous push, and heterogeneous , where we explain clearly the difference | 16:48 |
| armax | ajo: in other words from a receiver point of view it’s gonnab be trickier to understand where the push is coming from from the server side | 16:48 |
| *** tonytan_brb has joined #openstack-neutron | 16:48 | |
| ajo | still the heterogeneous could be confusing if you don't read the docstring or the devref | 16:48 |
| armax | in there are multiple producer side emission points | 16:48 |
| armax | if* | 16:49 |
| armax | so in a nutshell I feel that the tool may be prone to misuse, now assumed you guys are not going anywhere… | 16:49 |
| ajo | well, I'm going to the swimming pool in 15 min :D | 16:50 |
| armax | we can keep it as is…if you were to be hit by the proverbial bus, I’d rather have a slightly less powerful but more usable tool | 16:50 |
| *** moshele has joined #openstack-neutron | 16:50 | |
| ajo | ihrachys, armax I let you both choose , I'm happy with any of both options | 16:50 |
| armax | the notifier side is not afffected if we change the semantic | 16:50 |
| *** wolverineav has quit IRC | 16:51 | |
| armax | sorry I mean the consumer | 16:51 |
| openstackgerrit | Sindhu Devale proposed openstack/neutron: Refactoring db config options https://review.openstack.org/344877 | 16:51 |
| armax | the producer one is | 16:51 |
| *** wolverineav has joined #openstack-neutron | 16:51 | |
| *** wolverineav has quit IRC | 16:51 | |
| ajo | yes, that's the only thing | 16:51 |
| *** tonytan4ever has quit IRC | 16:51 | |
| ajo | this patch introduced the semantic change on the receiver that it's always going to get a list on the callback | 16:51 |
| ajo | instead of a single resource | 16:51 |
| armax | because if we decide to complain about heterogenous lists the producer freaks out | 16:51 |
| ajo | it can be a single item list, or multiple | 16:51 |
| *** wolverineav has joined #openstack-neutron | 16:51 | |
| armax | ajo: right | 16:51 |
| *** wolverineav has quit IRC | 16:52 | |
| *** wolverineav has joined #openstack-neutron | 16:52 | |
| *** wolverineav has quit IRC | 16:52 | |
| armax | perhaps we can make absolutely clear in the docs to use heterogenous list only if it is a really GOOD GOOD use case! | 16:52 |
| *** wolverineav has joined #openstack-neutron | 16:52 | |
| *** wolverineav has quit IRC | 16:53 | |
| ihrachys | I think it's just describing that as default, then having an 'advanced usage' below mentioning that this actually may also classify objects. | 16:53 |
| *** wolverineav has joined #openstack-neutron | 16:53 | |
| *** wolverineav has quit IRC | 16:53 | |
| ajo | ihrachys, armax option 3 can be, leave push homogeneous, and provide a helper to split & send, so it's obvious to the caller | 16:53 |
| *** elopez has joined #openstack-neutron | 16:53 | |
| *** sputnik13 has joined #openstack-neutron | 16:53 | |
| *** wolverineav has joined #openstack-neutron | 16:53 | |
| *** elopez is now known as Guest10600 | 16:54 | |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Relocate L3 DB Models https://review.openstack.org/348562 | 16:54 |
| armax | let me mull over this a bit, I can see that the heregeneous list push is nice | 16:54 |
| ajo | I'm going to fix the comments Ihar left (I'm afraid stuff won't work without the receiver version '1.1') and I'll continue monday morning with whatever you tell me on gerrit :) | 16:54 |
| ihrachys | I don't know how long we want to spin on it. I find a slightly more powerful API a good thing if it does not complicate much, but someone may claim it's YAGNI. | 16:54 |
| armax | I only wish that it would result into a single consumer-side pull | 16:54 |
| armax | *push | 16:55 |
| *** wolverineav has quit IRC | 16:55 | |
| *** wolverineav has joined #openstack-neutron | 16:55 | |
| *** wolverineav has quit IRC | 16:55 | |
| armax | ihrachys: I don’t carea bout YAGNI | 16:55 |
| armax | ihrachys: I care if the API is well designed | 16:55 |
| *** yamahata has joined #openstack-neutron | 16:55 | |
| *** wolverineav has joined #openstack-neutron | 16:55 | |
| *** sridharg has joined #openstack-neutron | 16:55 | |
| ajo | sometimes providing functionality in advance makes your future less painful | 16:55 |
| *** wolverineav has quit IRC | 16:56 | |
| armax | ihrachys: and if not, it may result into unintentional misuse | 16:56 |
| ajo | for example, multiple item push was a YAGNI in the past :) | 16:56 |
| *** iyamahat has quit IRC | 16:56 | |
| *** wolverineav has joined #openstack-neutron | 16:56 | |
| armax | the only argument I have is that there’s nothing in the API that tells me that a hetero-list push is gonna result into multiple homo-list pushes | 16:56 |
| *** wolverineav has quit IRC | 16:57 | |
| *** wolverineav has joined #openstack-neutron | 16:57 | |
| *** wolverineav has quit IRC | 16:57 | |
| armax | that’s quite an implicit assumption that can be discovered through RTFM | 16:57 |
| armax | rtheis: ping | 16:57 |
| *** wolverineav has joined #openstack-neutron | 16:57 | |
| rtheis | armax: hi | 16:57 |
| ihrachys | armax: it's a matter of doc | 16:57 |
| *** wolverineav has quit IRC | 16:58 | |
| *** wolverineav has joined #openstack-neutron | 16:58 | |
| armax | ihrachys: well, it’d be nice if the doc would be the API itself, that’s one of the secret of good API design | 16:58 |
| *** wolverineav has quit IRC | 16:58 | |
| armax | rtheis: re 348897 | 16:58 |
| ihrachys | anyhow, whatever makes you happy. I am slightly happier with what we have now, but if it blocks progress for vlan-aware, let's revert.. | 16:58 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron: SR-IOV: add agent QoS driver to support egress minimum bandwidth https://review.openstack.org/347302 | 16:58 |
| armax | rtheis: we should not just drop the module subscribe() | 16:58 |
| *** wolverineav has joined #openstack-neutron | 16:58 | |
| ihrachys | armax: an argument allowing heterogenous? | 16:58 |
| ihrachys | that is off by default? | 16:58 |
| *** wolverineav has quit IRC | 16:59 | |
| *** wolverineav has joined #openstack-neutron | 16:59 | |
| *** wolverineav has quit IRC | 16:59 | |
| *** eilert has joined #openstack-neutron | 16:59 | |
| rtheis | armax: ack | 16:59 |
| ihrachys | then we'll raise if it's not off and multiple keys in the per-type list | 16:59 |
| ihrachys | that would require some more tests. | 16:59 |
| armax | rtheis: but we should move it into the L3 mixin | 16:59 |
| *** wolverineav has joined #openstack-neutron | 16:59 | |
| armax | rtheis: perhaps into a __new__() block | 16:59 |
| *** wolverineav has quit IRC | 17:00 | |
| armax | rtheis: and only then it can be removed | 17:00 |
| *** wolverineav has joined #openstack-neutron | 17:00 | |
| *** wolverineav has quit IRC | 17:00 | |
| armax | ihrachys: right, my reserve against allowing hetero-list | 17:00 |
| armax | ihrachys: I am ok with the existing approach | 17:00 |
| *** wolverineav has joined #openstack-neutron | 17:00 | |
| armax | ihrachys: I am simply pondering :) | 17:00 |
| *** wolverineav has quit IRC | 17:01 | |
| ihrachys | it's open source, we'll iterate! | 17:01 |
| ihrachys | :) | 17:01 |
| *** wolverineav has joined #openstack-neutron | 17:01 | |
| *** wolverineav has quit IRC | 17:01 | |
| armax | ihrachys: right, let’s see if we get stung by this | 17:01 |
| rtheis | armax: we still keep the subscribe() method, correct? | 17:01 |
| armax | ihrachys: and reconsider things if need be | 17:01 |
| armax | rtheis: yes | 17:01 |
| *** wolverineav has joined #openstack-neutron | 17:01 | |
| armax | rtheis: split into agent vs agentless subscribes | 17:01 |
| ajo | I also wonder | 17:01 |
| *** wolverineav has quit IRC | 17:02 | |
| ajo | how useful is if you receive the different items out of order, well | 17:02 |
| *** wolverineav has joined #openstack-neutron | 17:02 | |
| armax | rtheis: one goes into the notifier mixin, the other goes into the L3_db mixin | 17:02 |
| *** wolverineav has quit IRC | 17:02 | |
| ajo | but that can also happen if you do it individually | 17:02 |
| *** Guest10600 has quit IRC | 17:02 | |
| armax | rtheis: well | 17:02 |
| ajo | different resource types, different queues.... | 17:02 |
| armax | rtheis: forget about the notifier part | 17:02 |
| *** ijw has joined #openstack-neutron | 17:02 | |
| armax | rtheis: we’d keep the subscriber as is and move into the mixin definition | 17:02 |
| *** wolverineav has joined #openstack-neutron | 17:02 | |
| *** matrohon has quit IRC | 17:02 | |
| *** wolverineav has quit IRC | 17:03 | |
| *** wolverineav has joined #openstack-neutron | 17:03 | |
| *** wolverineav has quit IRC | 17:03 | |
| armax | ajo: right, we’ll have to cross that bridge when we get to it, I guess | 17:03 |
| armax | even if the tool allows me hetero lists | 17:03 |
| armax | I don’t think I am ever gonna is it | 17:03 |
| armax | but never say never | 17:03 |
| *** wolverineav has joined #openstack-neutron | 17:03 | |
| armax | rtheis: what do you reckon? | 17:03 |
| *** wolverineav has quit IRC | 17:04 | |
| ajo | we could re-implement later when we find the use case | 17:04 |
| ajo | the patch is up | 17:04 |
| *** wolverineav has joined #openstack-neutron | 17:04 | |
| *** wolverineav has quit IRC | 17:04 | |
| armax | ajo: did you respin? | 17:04 |
| rtheis | armax: I started splitting and moving the bits under https://review.openstack.org/#/c/348933/ ... but it still had the implicit subscription | 17:04 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron: Add QoS minimum bandwidth rule for instance egress traffic https://review.openstack.org/344145 | 17:04 |
| ajo | armax, about to respin | 17:04 |
| armax | ajo: ack | 17:04 |
| ajo | armax, give me 5 more minutes | 17:04 |
| *** wolverineav has joined #openstack-neutron | 17:04 | |
| *** claudiub|2 has quit IRC | 17:04 | |
| armax | ajo: don’t be late to your swimming session! | 17:04 |
| *** wolverineav has quit IRC | 17:05 | |
| *** wolverineav has joined #openstack-neutron | 17:05 | |
| *** wolverineav has quit IRC | 17:05 | |
| armax | rtheis: in hindsight, I think the module subsription is overkill | 17:05 |
| rtheis | armax: if __new__ can be used for the implicit subscription then I suppose I could pull the unsubscribe bits | 17:05 |
| armax | rtheis: yes, if we give it a try, that should do it I think | 17:05 |
| armax | rtheis: and your life is simpler | 17:05 |
| *** wolverineav has joined #openstack-neutron | 17:05 | |
| *** wolverineav has quit IRC | 17:06 | |
| rtheis | armax: I like simple. I'll give it a try | 17:06 |
| armax | rtheis: you’d still keep the subscribe method | 17:06 |
| *** wolverineav has joined #openstack-neutron | 17:06 | |
| *** wolverineav has quit IRC | 17:06 | |
| rtheis | armax: ack | 17:06 |
| rtheis | thanks | 17:06 |
| *** elopez_ has joined #openstack-neutron | 17:06 | |
| armax | rtheis: and move the agentless subscribe into the dbonly mixin | 17:06 |
| *** wolverineav has joined #openstack-neutron | 17:06 | |
| *** wolverineav has quit IRC | 17:07 | |
| armax | and then call subscribe() from the agent-based mixin | 17:07 |
| *** wolverineav has joined #openstack-neutron | 17:07 | |
| *** wolverineav has quit IRC | 17:07 | |
| armax | now subscribe() could be split into agentless subscriptions and agent-based subscriptions | 17:07 |
| *** sbalukoff has quit IRC | 17:07 | |
| *** wolverineav has joined #openstack-neutron | 17:07 | |
| *** wolverineav has quit IRC | 17:08 | |
| *** wolverineav has joined #openstack-neutron | 17:08 | |
| *** wolverineav has quit IRC | 17:08 | |
| armax | rtheis: but that’s only to avoid duplicating the subscription to _prevent_l3_port_delete_callback | 17:08 |
| *** wolverineav has joined #openstack-neutron | 17:08 | |
| *** wolverineav has quit IRC | 17:09 | |
| *** wolverineav has joined #openstack-neutron | 17:09 | |
| *** wolverineav has quit IRC | 17:09 | |
| rtheis | armax: sounds good, I'll start reworking the patch | 17:09 |
| armax | rtheis: ok, let’s see how it works out | 17:09 |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Introduce bulk push to rpc callback mechanism https://review.openstack.org/348476 | 17:09 |
| ajo | armax, there it goes ^ | 17:09 |
| *** wolverineav has joined #openstack-neutron | 17:09 | |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Relocate Metering DB models https://review.openstack.org/348757 | 17:10 |
| *** wolverineav has quit IRC | 17:10 | |
| *** wolverineav has joined #openstack-neutron | 17:10 | |
| *** wolverineav has quit IRC | 17:10 | |
| * armax reviews | 17:10 | |
| *** gvrangan has joined #openstack-neutron | 17:10 | |
| armax | ajo: thank you for pulling ths off so quickly | 17:10 |
| armax | I owe you a free/uncoditional +2 | 17:10 |
| armax | :P | 17:10 |
| armax | use it with care | 17:10 |
| *** wolverineav has joined #openstack-neutron | 17:10 | |
| *** wolverineav has quit IRC | 17:11 | |
| ajo | nah, you're welcome :D | 17:11 |
| ajo | I'm glad to be of help | 17:11 |
| *** wolverineav has joined #openstack-neutron | 17:11 | |
| *** wolverineav has quit IRC | 17:11 | |
| armax | ajo: you know I wasn’t going to give you a free/unconditonal +2 anyway… :) | 17:11 |
| ajo | of course :P :D | 17:11 |
| *** wolverineav has joined #openstack-neutron | 17:11 | |
| *** wolverineav has quit IRC | 17:12 | |
| *** wolverineav has joined #openstack-neutron | 17:12 | |
| *** wolverineav has quit IRC | 17:12 | |
| *** kevo has joined #openstack-neutron | 17:12 | |
| *** wolverineav has joined #openstack-neutron | 17:12 | |
| *** wolverineav has quit IRC | 17:13 | |
| *** wolverineav has joined #openstack-neutron | 17:13 | |
| *** wolverineav has quit IRC | 17:13 | |
| *** wolverineav has joined #openstack-neutron | 17:13 | |
| ihrachys | wolverineav: struggling with comcast? | 17:14 |
| *** tflynn has joined #openstack-neutron | 17:14 | |
| *** wolverineav has quit IRC | 17:15 | |
| *** wolverineav has joined #openstack-neutron | 17:15 | |
| *** wolverineav has quit IRC | 17:15 | |
| *** wolverineav has joined #openstack-neutron | 17:15 | |
| *** nherciu has quit IRC | 17:15 | |
| armax | rtheis: btw what do you think about my response on https://review.openstack.org/#/c/340624/14/setup.cfg@43 | 17:15 |
| *** wolverineav has quit IRC | 17:16 | |
| *** wolverineav has joined #openstack-neutron | 17:16 | |
| *** wolverineav has quit IRC | 17:16 | |
| armax | rtheis: I’d like to close in on this if I can help it | 17:16 |
| *** wolverineav has joined #openstack-neutron | 17:16 | |
| *** wolverineav has quit IRC | 17:17 | |
| *** wolverineav has joined #openstack-neutron | 17:17 | |
| *** sdague has joined #openstack-neutron | 17:17 | |
| *** wolverineav has quit IRC | 17:17 | |
| *** wolverineav has joined #openstack-neutron | 17:17 | |
| *** ijw has quit IRC | 17:18 | |
| *** wolverineav has quit IRC | 17:19 | |
| *** wolverineav has joined #openstack-neutron | 17:19 | |
| *** wolverineav has quit IRC | 17:19 | |
| *** tflynn has quit IRC | 17:19 | |
| openstackgerrit | Merged openstack/neutron-specs: QoS Improved validation mechanism for rules https://review.openstack.org/323474 | 17:19 |
| *** wolverineav has joined #openstack-neutron | 17:19 | |
| *** wolverineav has quit IRC | 17:20 | |
| *** wolverineav has joined #openstack-neutron | 17:20 | |
| *** wolverineav has quit IRC | 17:20 | |
| ihrachys | ajo: ^ validation spec landed, we just need to make the code polished | 17:20 |
| *** wolverineav has joined #openstack-neutron | 17:20 | |
| *** wolverineav has quit IRC | 17:21 | |
| *** wolverineav has joined #openstack-neutron | 17:21 | |
| *** wolverineav has quit IRC | 17:21 | |
| *** wolverineav has joined #openstack-neutron | 17:21 | |
| *** wolverineav has quit IRC | 17:22 | |
| *** wolverineav has joined #openstack-neutron | 17:22 | |
| *** wolverineav has quit IRC | 17:22 | |
| *** wolverineav has joined #openstack-neutron | 17:22 | |
| *** wolverineav has quit IRC | 17:23 | |
| *** wolverineav has joined #openstack-neutron | 17:23 | |
| *** wolverineav has quit IRC | 17:23 | |
| *** singhj has quit IRC | 17:23 | |
| *** wolverineav has joined #openstack-neutron | 17:23 | |
| *** wolverineav has quit IRC | 17:24 | |
| *** wolverineav has joined #openstack-neutron | 17:24 | |
| *** wolverineav has quit IRC | 17:24 | |
| *** wolverineav has joined #openstack-neutron | 17:24 | |
| *** wolverineav has quit IRC | 17:25 | |
| *** wolverineav has joined #openstack-neutron | 17:25 | |
| *** wolverineav has quit IRC | 17:25 | |
| *** elopez_ has quit IRC | 17:25 | |
| *** wolverineav has joined #openstack-neutron | 17:25 | |
| *** wolverineav has quit IRC | 17:26 | |
| *** wolverineav has joined #openstack-neutron | 17:26 | |
| *** wolverineav has quit IRC | 17:26 | |
| *** wolverineav has joined #openstack-neutron | 17:26 | |
| *** wolverineav has quit IRC | 17:27 | |
| *** wolverineav has joined #openstack-neutron | 17:27 | |
| *** wolverineav has quit IRC | 17:27 | |
| *** wolverineav has joined #openstack-neutron | 17:28 | |
| *** wolverineav has quit IRC | 17:28 | |
| *** wolverineav has joined #openstack-neutron | 17:28 | |
| *** wolverineav has quit IRC | 17:29 | |
| *** wolverineav has joined #openstack-neutron | 17:29 | |
| *** wolverineav has quit IRC | 17:29 | |
| *** dasanind has quit IRC | 17:29 | |
| *** sbalukoff has joined #openstack-neutron | 17:29 | |
| *** wolverineav has joined #openstack-neutron | 17:29 | |
| *** wolverineav has quit IRC | 17:30 | |
| *** wolverineav has joined #openstack-neutron | 17:30 | |
| *** wolverineav has quit IRC | 17:30 | |
| *** wolverineav has joined #openstack-neutron | 17:30 | |
| *** neiljerram has quit IRC | 17:30 | |
| *** wolverineav has quit IRC | 17:31 | |
| *** wolverineav has joined #openstack-neutron | 17:31 | |
| *** wolverineav has quit IRC | 17:31 | |
| *** wolverineav has joined #openstack-neutron | 17:31 | |
| *** wolverineav has quit IRC | 17:32 | |
| *** neiljerram has joined #openstack-neutron | 17:32 | |
| *** wolverineav has joined #openstack-neutron | 17:32 | |
| *** wolverineav has quit IRC | 17:32 | |
| *** wolverineav has joined #openstack-neutron | 17:32 | |
| *** wolverineav has quit IRC | 17:33 | |
| *** wolverineav has joined #openstack-neutron | 17:33 | |
| *** wolverineav has quit IRC | 17:33 | |
| *** wolverineav has joined #openstack-neutron | 17:33 | |
| *** fnaval has joined #openstack-neutron | 17:34 | |
| ihrachys | armax: do we have time till Monday with the patch, or we want to land it quicker? | 17:34 |
| armax | ihrachys: I think it can bake a little longer | 17:35 |
| *** wolverineav has quit IRC | 17:35 | |
| ihrachys | armax: if the former, I will let ajo to handle remaining comments, otherwise I can respin | 17:35 |
| *** wolverineav has joined #openstack-neutron | 17:35 | |
| ihrachys | ack | 17:35 |
| *** wolverineav has quit IRC | 17:35 | |
| *** itlinux has quit IRC | 17:35 | |
| armax | ihrachys: I think we agreed that we are ok with the existing approach | 17:35 |
| *** jamielennox is now known as jamielennox|away | 17:35 | |
| *** wolverineav has joined #openstack-neutron | 17:35 | |
| *** wolverineav has quit IRC | 17:36 | |
| *** wolverineav has joined #openstack-neutron | 17:36 | |
| *** wolverineav has quit IRC | 17:36 | |
| *** iyamahat has joined #openstack-neutron | 17:36 | |
| *** fnaval has quit IRC | 17:36 | |
| *** wolverineav has joined #openstack-neutron | 17:36 | |
| *** fnaval has joined #openstack-neutron | 17:37 | |
| ihrachys | armax: there are some remaining comments in tests that would be cool to handle before landing, so if we have time, let's give it another day. | 17:37 |
| armax | ihrachys: absolutely | 17:38 |
| *** wolverineav has quit IRC | 17:38 | |
| *** wolverineav has joined #openstack-neutron | 17:38 | |
| *** wolverineav has quit IRC | 17:38 | |
| *** singhj has joined #openstack-neutron | 17:38 | |
| *** wolverineav has joined #openstack-neutron | 17:38 | |
| *** wolverineav has quit IRC | 17:39 | |
| openstackgerrit | Joseph Pasque proposed openstack/neutron: Action status updating tag changed from 201 to 200 https://review.openstack.org/348977 | 17:39 |
| *** wolverineav has joined #openstack-neutron | 17:39 | |
| *** wolverineav has quit IRC | 17:41 | |
| *** wolverineav has joined #openstack-neutron | 17:41 | |
| *** wolverineav has quit IRC | 17:41 | |
| *** wolverineav has joined #openstack-neutron | 17:41 | |
| *** wolverineav has quit IRC | 17:42 | |
| *** wolverineav has joined #openstack-neutron | 17:42 | |
| *** wolverineav has quit IRC | 17:42 | |
| *** singhj has quit IRC | 17:42 | |
| *** abhiraut has joined #openstack-neutron | 17:42 | |
| *** wolverineav has joined #openstack-neutron | 17:42 | |
| *** wolverineav has quit IRC | 17:43 | |
| *** wolverineav has joined #openstack-neutron | 17:43 | |
| *** wolverineav has quit IRC | 17:43 | |
| *** wolverineav has joined #openstack-neutron | 17:44 | |
| wolverineav | ihrachys: oops, not sure what happened there, but last night laptop didn't go to sleep after shut lid. endless loop of connect/disconnect. woke up and laptop is a hot cake! haha | 17:46 |
| *** wolverineav has quit IRC | 17:48 | |
| *** wolverineav has joined #openstack-neutron | 17:49 | |
| *** kobis has joined #openstack-neutron | 17:52 | |
| *** obondarev has quit IRC | 17:52 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: objects: switched back to passing models into extension handlers https://review.openstack.org/348987 | 17:53 |
| *** Sarath has joined #openstack-neutron | 17:53 | |
| ihrachys | kevinbenton: ^ switching back to models for some objects for which we passed objects into extend_dict functions for some time | 17:53 |
| *** links has joined #openstack-neutron | 17:53 | |
| *** s3wong has joined #openstack-neutron | 17:54 | |
| *** wolverineav has quit IRC | 17:57 | |
| *** nlahouti1 has quit IRC | 17:57 | |
| *** nlahouti has joined #openstack-neutron | 17:57 | |
| rtheis | armax: sorry, I stepped away for a while. I'll take a look at the review again | 17:58 |
| armax | rtheis: thanks | 17:58 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Revisit the Stadium section of the developer guide https://review.openstack.org/335739 | 17:59 |
| *** jprovazn has quit IRC | 17:59 | |
| *** elopez_ has joined #openstack-neutron | 18:01 | |
| *** johnbelamaric has quit IRC | 18:01 | |
| *** elopez_ has quit IRC | 18:01 | |
| *** johnbelamaric has joined #openstack-neutron | 18:01 | |
| *** jprovazn has joined #openstack-neutron | 18:02 | |
| *** nlahouti1 has joined #openstack-neutron | 18:02 | |
| *** nlahouti has quit IRC | 18:02 | |
| *** iranzo has quit IRC | 18:04 | |
| *** singhj has joined #openstack-neutron | 18:08 | |
| *** nlahouti1 has quit IRC | 18:09 | |
| *** nlahouti has joined #openstack-neutron | 18:09 | |
| *** Captain_Murdoch has quit IRC | 18:11 | |
| *** singhj has quit IRC | 18:13 | |
| *** ijw has joined #openstack-neutron | 18:13 | |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Pass the complete info in sg/rules db into PRECOMMIT_XXX callback https://review.openstack.org/281693 | 18:15 |
| *** wolverineav has joined #openstack-neutron | 18:16 | |
| *** nlahouti has quit IRC | 18:19 | |
| *** wolverineav has quit IRC | 18:19 | |
| *** nlahouti has joined #openstack-neutron | 18:19 | |
| *** wolverineav has joined #openstack-neutron | 18:19 | |
| *** ijw has quit IRC | 18:20 | |
| *** nlahouti1 has joined #openstack-neutron | 18:23 | |
| *** nlahouti has quit IRC | 18:23 | |
| *** amuller has quit IRC | 18:25 | |
| *** singhj has joined #openstack-neutron | 18:26 | |
| *** anilvenkata has quit IRC | 18:26 | |
| *** wolverineav has quit IRC | 18:28 | |
| *** wolverineav has joined #openstack-neutron | 18:29 | |
| *** nlahouti1 has quit IRC | 18:30 | |
| *** nlahouti has joined #openstack-neutron | 18:30 | |
| *** john-davidge has quit IRC | 18:32 | |
| *** nlahouti has quit IRC | 18:34 | |
| *** thorst has quit IRC | 18:34 | |
| *** nlahouti has joined #openstack-neutron | 18:34 | |
| *** ociuhandu has quit IRC | 18:37 | |
| *** thorst has joined #openstack-neutron | 18:37 | |
| *** nlahouti has quit IRC | 18:38 | |
| *** nlahouti has joined #openstack-neutron | 18:38 | |
| *** dkehn has quit IRC | 18:39 | |
| *** dkehn_ has quit IRC | 18:40 | |
| *** thorst has quit IRC | 18:42 | |
| openstackgerrit | Ankur proposed openstack/neutron: [WIP] Introducing Network OVO https://review.openstack.org/349018 | 18:43 |
| *** elopez_ has joined #openstack-neutron | 18:44 | |
| *** johnbelamaric has quit IRC | 18:44 | |
| *** johnbelamaric has joined #openstack-neutron | 18:44 | |
| *** iyamahat has quit IRC | 18:46 | |
| *** yamahata has quit IRC | 18:46 | |
| *** dkehn has joined #openstack-neutron | 18:46 | |
| *** gvrangan has quit IRC | 18:52 | |
| *** dkehn_ has joined #openstack-neutron | 18:52 | |
| *** achanda has joined #openstack-neutron | 18:55 | |
| *** hoonetorg has quit IRC | 18:56 | |
| *** Leo_ has joined #openstack-neutron | 18:56 | |
| *** thorst has joined #openstack-neutron | 18:56 | |
| *** Leom has joined #openstack-neutron | 18:56 | |
| *** ijw has joined #openstack-neutron | 18:57 | |
| *** Leo_ has quit IRC | 19:00 | |
| *** thorst has quit IRC | 19:00 | |
| *** thorst has joined #openstack-neutron | 19:01 | |
| *** bks has joined #openstack-neutron | 19:02 | |
| *** tflynn has joined #openstack-neutron | 19:03 | |
| *** ijw has quit IRC | 19:03 | |
| *** elopez_ has quit IRC | 19:03 | |
| *** thorst_ has joined #openstack-neutron | 19:03 | |
| *** ihrachys has quit IRC | 19:04 | |
| *** nlahouti has left #openstack-neutron | 19:05 | |
| *** cleong has quit IRC | 19:05 | |
| *** elo has quit IRC | 19:06 | |
| *** yamamoto has quit IRC | 19:06 | |
| *** thorst has quit IRC | 19:06 | |
| *** fifieldt has quit IRC | 19:07 | |
| *** elo has joined #openstack-neutron | 19:07 | |
| *** tflynn has quit IRC | 19:07 | |
| *** hynekm has joined #openstack-neutron | 19:07 | |
| openstackgerrit | Merged openstack/neutron: Fix that api_workers=0 doesn't spawn any api workers https://review.openstack.org/348801 | 19:13 |
| *** buttercup has quit IRC | 19:13 | |
| *** hynekm has quit IRC | 19:14 | |
| *** vhoward has quit IRC | 19:15 | |
| *** ijw has joined #openstack-neutron | 19:15 | |
| *** johnbelamaric has quit IRC | 19:15 | |
| *** sridharg has quit IRC | 19:16 | |
| *** moshele has quit IRC | 19:17 | |
| *** johnbelamaric has joined #openstack-neutron | 19:17 | |
| openstackgerrit | Merged openstack/neutron: tests: enable test_get_objects_queries_constant for trunk ports https://review.openstack.org/348378 | 19:18 |
| *** fifieldt has joined #openstack-neutron | 19:18 | |
| *** fzdarsky is now known as fzdarsky|afk | 19:19 | |
| *** fzdarsky|afk has quit IRC | 19:19 | |
| *** johnbelamaric has quit IRC | 19:19 | |
| *** dkehn_ has quit IRC | 19:19 | |
| *** links has quit IRC | 19:20 | |
| *** dkehn has quit IRC | 19:20 | |
| *** mickeys has quit IRC | 19:21 | |
| *** bvandewa has joined #openstack-neutron | 19:22 | |
| *** dkehn has joined #openstack-neutron | 19:22 | |
| *** iyamahat has joined #openstack-neutron | 19:24 | |
| *** ijw has quit IRC | 19:25 | |
| *** dkehn has quit IRC | 19:28 | |
| *** johnbelamaric has joined #openstack-neutron | 19:28 | |
| openstackgerrit | Merged openstack/neutron: Added test cases for DVR L3 schedulers. https://review.openstack.org/188157 | 19:30 |
| openstackgerrit | Merged openstack/neutron: Fix for check_vlan_transparency on mech drivers not called https://review.openstack.org/341003 | 19:31 |
| *** dkehn_ has joined #openstack-neutron | 19:32 | |
| openstackgerrit | Merged openstack/neutron: Refactoring config options for common config opts https://review.openstack.org/344281 | 19:32 |
| *** dkehn has joined #openstack-neutron | 19:34 | |
| *** edmondsw has quit IRC | 19:35 | |
| *** bvandewa has quit IRC | 19:35 | |
| *** yamahata has joined #openstack-neutron | 19:43 | |
| *** banix has joined #openstack-neutron | 19:43 | |
| *** chandankumar has quit IRC | 19:45 | |
| *** tidwellr has joined #openstack-neutron | 19:51 | |
| *** krtaylor has quit IRC | 19:52 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Revisit the Stadium section of the developer guide https://review.openstack.org/335739 | 19:54 |
| *** jprovazn has quit IRC | 19:54 | |
| *** Sarath has quit IRC | 19:55 | |
| *** ijw has joined #openstack-neutron | 19:58 | |
| *** rahuls has joined #openstack-neutron | 19:59 | |
| *** rossella_s has quit IRC | 19:59 | |
| *** rossella_s has joined #openstack-neutron | 20:00 | |
| openstackgerrit | Anindita Das proposed openstack/neutron: Refactoring config options for extension opts https://review.openstack.org/345486 | 20:05 |
| *** ociuhandu has joined #openstack-neutron | 20:07 | |
| *** yamamoto has joined #openstack-neutron | 20:07 | |
| *** sindhu has quit IRC | 20:07 | |
| *** ijw has quit IRC | 20:07 | |
| *** slaweq has joined #openstack-neutron | 20:09 | |
| *** baoli has quit IRC | 20:11 | |
| *** yamamoto has quit IRC | 20:12 | |
| *** bvandewa has joined #openstack-neutron | 20:12 | |
| *** jckasper has quit IRC | 20:12 | |
| *** jckasper has joined #openstack-neutron | 20:13 | |
| *** sindhu has joined #openstack-neutron | 20:14 | |
| *** jckasper has quit IRC | 20:17 | |
| *** azbiswas has joined #openstack-neutron | 20:20 | |
| *** ijw has joined #openstack-neutron | 20:21 | |
| *** mickeys_ has joined #openstack-neutron | 20:23 | |
| *** kobis has quit IRC | 20:27 | |
| *** ijw has quit IRC | 20:27 | |
| *** sindhu has quit IRC | 20:27 | |
| *** dasanind has joined #openstack-neutron | 20:27 | |
| *** mickeys_ has quit IRC | 20:28 | |
| *** sindhu has joined #openstack-neutron | 20:29 | |
| *** tonytan_brb has quit IRC | 20:31 | |
| *** janzian has quit IRC | 20:32 | |
| *** achanda has quit IRC | 20:34 | |
| *** slaweq has quit IRC | 20:37 | |
| *** gouthamr has quit IRC | 20:41 | |
| *** akshai has quit IRC | 20:42 | |
| *** gvrangan has joined #openstack-neutron | 20:45 | |
| *** moshele has joined #openstack-neutron | 20:46 | |
| openstackgerrit | Victor Morales proposed openstack/neutron: Relocate Flavor and ServiceProfile DB models https://review.openstack.org/349071 | 20:47 |
| *** boden has quit IRC | 20:53 | |
| *** abhiraut has quit IRC | 20:57 | |
| openstackgerrit | Anindita Das proposed openstack/neutron: Refactoring config options for services opts https://review.openstack.org/347044 | 20:59 |
| *** wolverin_ has joined #openstack-neutron | 21:03 | |
| *** bvandewa has quit IRC | 21:04 | |
| *** marst has quit IRC | 21:04 | |
| *** wolverineav has quit IRC | 21:06 | |
| *** singhj has quit IRC | 21:07 | |
| *** yamamoto has joined #openstack-neutron | 21:08 | |
| *** banix has quit IRC | 21:10 | |
| *** singhj has joined #openstack-neutron | 21:11 | |
| *** jamesdenton has quit IRC | 21:13 | |
| *** yamamoto has quit IRC | 21:13 | |
| *** elo has quit IRC | 21:14 | |
| *** xinliang has quit IRC | 21:14 | |
| *** elo has joined #openstack-neutron | 21:22 | |
| *** singhj has quit IRC | 21:23 | |
| *** abhiraut has joined #openstack-neutron | 21:25 | |
| *** singhj has joined #openstack-neutron | 21:25 | |
| *** armax has quit IRC | 21:26 | |
| *** elo has quit IRC | 21:27 | |
| *** xinliang has joined #openstack-neutron | 21:29 | |
| *** tonytan4ever has joined #openstack-neutron | 21:31 | |
| openstackgerrit | Brandon Logan proposed openstack/neutron: WIP: Pecan: Handle member actions https://review.openstack.org/344581 | 21:33 |
| *** banix has joined #openstack-neutron | 21:34 | |
| *** bvandewa has joined #openstack-neutron | 21:35 | |
| *** tonytan4ever has quit IRC | 21:36 | |
| *** wolverin_ has quit IRC | 21:41 | |
| *** liuyulong_ has joined #openstack-neutron | 21:44 | |
| *** liuyulong has quit IRC | 21:44 | |
| *** dims_ has quit IRC | 21:44 | |
| openstackgerrit | Merged openstack/neutron: Filter HA router without HA port bindings after race conditions https://review.openstack.org/265680 | 21:44 |
| *** dims has joined #openstack-neutron | 21:46 | |
| *** liuyulong has joined #openstack-neutron | 21:47 | |
| *** singhj has quit IRC | 21:48 | |
| *** liuyulong_ has quit IRC | 21:48 | |
| *** liuyulong_ has joined #openstack-neutron | 21:49 | |
| *** rtheis has quit IRC | 21:49 | |
| *** singhj has joined #openstack-neutron | 21:50 | |
| *** moshele has quit IRC | 21:50 | |
| openstackgerrit | Merged openstack/neutron: Fix L3 NAT DB signature mismatch https://review.openstack.org/348558 | 21:50 |
| *** liuyulong has quit IRC | 21:52 | |
| *** elo has joined #openstack-neutron | 21:56 | |
| *** singhj has quit IRC | 21:56 | |
| *** bvandewa has quit IRC | 21:57 | |
| *** sindhu has quit IRC | 21:57 | |
| *** singhj has joined #openstack-neutron | 21:58 | |
| *** ajmiller has joined #openstack-neutron | 22:01 | |
| *** sbalukoff has quit IRC | 22:01 | |
| *** azbiswas has quit IRC | 22:02 | |
| *** zkassab has quit IRC | 22:02 | |
| *** azbiswas has joined #openstack-neutron | 22:02 | |
| *** azbiswas has quit IRC | 22:02 | |
| *** singhj has quit IRC | 22:03 | |
| *** liuyulong_ has quit IRC | 22:03 | |
| *** johnbelamaric has quit IRC | 22:03 | |
| *** ajmiller_ has joined #openstack-neutron | 22:03 | |
| *** liuyulong has joined #openstack-neutron | 22:04 | |
| *** pradk has quit IRC | 22:04 | |
| *** ajmiller has quit IRC | 22:07 | |
| *** superdan has quit IRC | 22:07 | |
| *** moshele has joined #openstack-neutron | 22:08 | |
| *** bks has quit IRC | 22:08 | |
| *** yamamoto has joined #openstack-neutron | 22:10 | |
| *** azbiswas has joined #openstack-neutron | 22:12 | |
| *** wolverineav has joined #openstack-neutron | 22:13 | |
| *** achanda has joined #openstack-neutron | 22:13 | |
| *** ijw has joined #openstack-neutron | 22:14 | |
| *** daneyon has quit IRC | 22:15 | |
| *** singhj has joined #openstack-neutron | 22:15 | |
| *** yamamoto has quit IRC | 22:15 | |
| *** wolverineav has quit IRC | 22:18 | |
| *** liuyulong_ has joined #openstack-neutron | 22:21 | |
| *** liuyulong has quit IRC | 22:22 | |
| *** ijw has quit IRC | 22:24 | |
| *** dane_leblanc has quit IRC | 22:31 | |
| *** moshele has quit IRC | 22:32 | |
| *** wolverineav has joined #openstack-neutron | 22:33 | |
| *** liuyulong has joined #openstack-neutron | 22:37 | |
| *** wolverineav has quit IRC | 22:37 | |
| *** tpsilva has quit IRC | 22:38 | |
| *** dave-mccowan has quit IRC | 22:38 | |
| *** liuyulong_ has quit IRC | 22:39 | |
| *** wolverineav has joined #openstack-neutron | 22:40 | |
| *** wolverineav has quit IRC | 22:40 | |
| *** wolverineav has joined #openstack-neutron | 22:41 | |
| *** liuyulong_ has joined #openstack-neutron | 22:41 | |
| *** liuyulong has quit IRC | 22:41 | |
| *** abregman has quit IRC | 22:45 | |
| *** gouthamr has joined #openstack-neutron | 22:45 | |
| openstackgerrit | Abhishek Raut proposed openstack/python-neutronclient: Add trunk commands to openstackclient https://review.openstack.org/340624 | 22:46 |
| *** liuyulong has joined #openstack-neutron | 22:47 | |
| *** Guest41489 has joined #openstack-neutron | 22:49 | |
| *** liuyulong_ has quit IRC | 22:50 | |
| *** singhj has quit IRC | 22:50 | |
| *** moshele has joined #openstack-neutron | 22:52 | |
| *** singhj has joined #openstack-neutron | 22:52 | |
| openstackgerrit | Rawlin Peters proposed openstack/neutron: Pass bridge_name in OVS port's vif_details https://review.openstack.org/343816 | 22:52 |
| *** minwang2 has joined #openstack-neutron | 22:57 | |
| *** minwang2 has left #openstack-neutron | 22:58 | |
| *** sdague has quit IRC | 22:59 | |
| *** a_ta has quit IRC | 22:59 | |
| *** a_ta has joined #openstack-neutron | 23:00 | |
| openstackgerrit | Isaku Yamahata proposed openstack/neutron: many tests modify global RESOURCE_ATTRIBUTES_MAP https://review.openstack.org/349111 | 23:00 |
| *** sindhu has joined #openstack-neutron | 23:01 | |
| *** Guest41489 has quit IRC | 23:02 | |
| openstackgerrit | Rawlin Peters proposed openstack/neutron: Pass bridge_name in OVS port's vif_details https://review.openstack.org/343816 | 23:03 |
| *** a_ta has quit IRC | 23:04 | |
| *** thorst_ has quit IRC | 23:04 | |
| *** thorst has joined #openstack-neutron | 23:05 | |
| *** singhj has quit IRC | 23:08 | |
| *** mickeys has joined #openstack-neutron | 23:08 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Revert "Don't use exponential back-off for report_state" https://review.openstack.org/349113 | 23:09 |
| *** mickeys_ has joined #openstack-neutron | 23:11 | |
| *** mickeys has quit IRC | 23:11 | |
| *** yamamoto has joined #openstack-neutron | 23:12 | |
| *** abhiraut has quit IRC | 23:13 | |
| *** thorst has quit IRC | 23:13 | |
| *** mickeys_ has quit IRC | 23:13 | |
| *** sindhu has quit IRC | 23:14 | |
| *** achanda has quit IRC | 23:17 | |
| *** yamamoto has quit IRC | 23:17 | |
| *** dasanind has quit IRC | 23:17 | |
| apuimedo | kevinbenton: all the patches got merged. Thanks | 23:22 |
| apuimedo | please, when you can, tag a release :-) | 23:23 |
| apuimedo | networking-midonet stable/mitaka | 23:23 |
| *** tonytan4ever has joined #openstack-neutron | 23:25 | |
| *** eilert has quit IRC | 23:26 | |
| kevinbenton | apuimedo: i can't to tags | 23:26 |
| *** Guest39119 has joined #openstack-neutron | 23:27 | |
| *** armax has joined #openstack-neutron | 23:28 | |
| kevinbenton | apuimedo: try pinging armax | 23:28 |
| *** armax is now known as Guest46019 | 23:28 | |
| *** Guest46019 has quit IRC | 23:29 | |
| apuimedo | thanks kevinbenton. I will | 23:32 |
| apuimedo | carl_baldwin: ping | 23:34 |
| openstackgerrit | LIU Yulong proposed openstack/neutron: Put HA router state change event into queue https://review.openstack.org/275614 | 23:34 |
| apuimedo | carl_baldwin: I saw that you are on neutron-release, could you please tag a networking-midonet stable/mitaka release? | 23:34 |
| apuimedo | dougwig: ^^ | 23:35 |
| *** banix has quit IRC | 23:35 | |
| * apuimedo trying to find a neutron-release member :P | 23:35 | |
| apuimedo | johnsom: ^^ | 23:38 |
| *** vishwanathj has quit IRC | 23:52 | |
| *** Guest39119 is now known as dansmith | 23:52 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Don't use file() to write object hashes https://review.openstack.org/349117 | 23:54 |
| *** abhiraut has joined #openstack-neutron | 23:55 | |
| *** ajmiller_ has quit IRC | 23:58 | |
| *** achanda has joined #openstack-neutron | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!