| *** jbell8 has quit IRC | 00:01 | |
| *** eckelcu has quit IRC | 00:01 | |
| *** barra204 has quit IRC | 00:03 | |
| *** nlahouti has joined #openstack-neutron | 00:05 | |
| *** topshare has quit IRC | 00:07 | |
| openstackgerrit | Merged openstack/neutron: Register extraroute extension https://review.openstack.org/196996 | 00:08 |
|---|---|---|
| *** MentalRay has joined #openstack-neutron | 00:09 | |
| openstackgerrit | Merged openstack/neutron: Cleanup unused method get_plugin_version https://review.openstack.org/201034 | 00:10 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/201803 | 00:11 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/200680 | 00:11 |
| *** mickeys has quit IRC | 00:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/200682 | 00:12 |
| *** jagter has quit IRC | 00:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/201837 | 00:14 |
| *** johnbelamaric has joined #openstack-neutron | 00:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/201803 | 00:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/200682 | 00:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/201837 | 00:21 |
| *** sridhar_ram1 is now known as sridhar_ram | 00:22 | |
| *** padkrish_ has quit IRC | 00:24 | |
| openstackgerrit | Merged openstack/neutron: Abstract sync_allocations https://review.openstack.org/191971 | 00:24 |
| *** david-lyle has quit IRC | 00:25 | |
| *** abhiraut has quit IRC | 00:25 | |
| *** padkrish has joined #openstack-neutron | 00:25 | |
| *** hichihara has joined #openstack-neutron | 00:25 | |
| *** padkrish has quit IRC | 00:25 | |
| *** padkrish has joined #openstack-neutron | 00:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/201803 | 00:26 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/200680 | 00:26 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/200682 | 00:26 |
| *** abhiraut has joined #openstack-neutron | 00:27 | |
| *** sdake_ has joined #openstack-neutron | 00:28 | |
| *** Swami has quit IRC | 00:28 | |
| *** Kennan2 is now known as Kennan | 00:29 | |
| *** hitalia has quit IRC | 00:29 | |
| *** achanda has quit IRC | 00:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/201837 | 00:29 |
| *** jagter has joined #openstack-neutron | 00:29 | |
| *** MentalRay has quit IRC | 00:30 | |
| *** abhiraut has quit IRC | 00:31 | |
| *** sdake has quit IRC | 00:31 | |
| *** tflynn has quit IRC | 00:31 | |
| openstackgerrit | enikanorov proposed openstack/neutron: Graceful ovs-agent restart https://review.openstack.org/182920 | 00:31 |
| *** tflynn has joined #openstack-neutron | 00:32 | |
| *** MentalRay has joined #openstack-neutron | 00:32 | |
| *** signed8b_ has joined #openstack-neutron | 00:33 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/201803 | 00:33 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/200680 | 00:33 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/200682 | 00:34 |
| *** sripriya has quit IRC | 00:34 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/201837 | 00:36 |
| *** tflynn_ has joined #openstack-neutron | 00:37 | |
| *** tflynn_ has quit IRC | 00:37 | |
| *** padkrish has quit IRC | 00:37 | |
| *** tflynn_ has joined #openstack-neutron | 00:38 | |
| *** padkrish has joined #openstack-neutron | 00:38 | |
| *** tflynn has quit IRC | 00:38 | |
| *** thorst has joined #openstack-neutron | 00:38 | |
| *** signed8b_ is now known as signed8bit_ZZZzz | 00:40 | |
| *** signed8bit_ZZZzz has quit IRC | 00:41 | |
| *** padkrish has quit IRC | 00:42 | |
| *** bitblt has quit IRC | 00:44 | |
| *** watanabe_isao has quit IRC | 00:45 | |
| *** baoli has joined #openstack-neutron | 00:45 | |
| *** shivrao has quit IRC | 00:46 | |
| *** shivrao has joined #openstack-neutron | 00:46 | |
| *** nlahouti has quit IRC | 00:47 | |
| *** ivar-lazzaro has quit IRC | 00:50 | |
| *** ivar-lazzaro has joined #openstack-neutron | 00:51 | |
| *** topshare has joined #openstack-neutron | 00:53 | |
| *** dims__ has quit IRC | 00:54 | |
| *** fitoduarte has quit IRC | 00:55 | |
| *** padkrish has joined #openstack-neutron | 00:56 | |
| *** regXboi has quit IRC | 00:56 | |
| *** abhiraut has joined #openstack-neutron | 00:59 | |
| *** wei-wang has quit IRC | 01:04 | |
| *** sthillma has quit IRC | 01:04 | |
| *** srilatha has quit IRC | 01:05 | |
| *** sridhar_ram has quit IRC | 01:06 | |
| *** shashank_hegde_ has quit IRC | 01:08 | |
| *** yalie has joined #openstack-neutron | 01:09 | |
| *** johnbelamaric has quit IRC | 01:09 | |
| *** johnbelamaric has joined #openstack-neutron | 01:10 | |
| *** padkrish has quit IRC | 01:14 | |
| *** punter has joined #openstack-neutron | 01:15 | |
| *** padkrish has joined #openstack-neutron | 01:15 | |
| punter | Hi all | 01:15 |
| *** erkules_ has joined #openstack-neutron | 01:15 | |
| punter | Hi .. Could any one let me know if there is a way to throw the neutron exception raised by a vendor plugin when attempting hot plug command .. for example nova interface-attach vm --net ecc95d03-2033-4b20-8c10-27dad1bf5299 [18:13] ... I get the following exception --> ERROR (ClientException): The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-758cbcdf-4722-4849-8cf | 01:15 |
| *** topshare has quit IRC | 01:18 | |
| *** erkules has quit IRC | 01:18 | |
| *** padkrish has quit IRC | 01:19 | |
| *** topshare has joined #openstack-neutron | 01:23 | |
| *** jwy has quit IRC | 01:23 | |
| *** changbl has joined #openstack-neutron | 01:24 | |
| *** bradjones has quit IRC | 01:26 | |
| *** dims_ has joined #openstack-neutron | 01:28 | |
| *** arajagopal has joined #openstack-neutron | 01:29 | |
| *** bradjones has joined #openstack-neutron | 01:30 | |
| *** bradjones has quit IRC | 01:30 | |
| *** bradjones has joined #openstack-neutron | 01:30 | |
| *** signed8b_ has joined #openstack-neutron | 01:30 | |
| *** signed8b_ is now known as signed8bit_ZZZzz | 01:31 | |
| *** iwamoto has joined #openstack-neutron | 01:31 | |
| *** aranjan has quit IRC | 01:32 | |
| *** stanzgy has joined #openstack-neutron | 01:33 | |
| *** signed8bit_ZZZzz has quit IRC | 01:33 | |
| *** abhiraut has quit IRC | 01:35 | |
| *** padkrish has joined #openstack-neutron | 01:36 | |
| *** shashank_hegde_ has joined #openstack-neutron | 01:40 | |
| *** thorst has quit IRC | 01:41 | |
| *** shivrao has quit IRC | 01:49 | |
| *** shashank_hegde_ has quit IRC | 01:51 | |
| *** boris-42 has quit IRC | 01:52 | |
| *** signed8bit has joined #openstack-neutron | 01:53 | |
| *** s3wong has quit IRC | 01:53 | |
| *** markvoelker has quit IRC | 01:54 | |
| *** jwy has joined #openstack-neutron | 01:54 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 01:54 | |
| *** signed8bit_ZZZzz has quit IRC | 01:56 | |
| *** markvoelker has joined #openstack-neutron | 02:00 | |
| *** signed8b_ has joined #openstack-neutron | 02:01 | |
| *** ivar-lazzaro has quit IRC | 02:01 | |
| *** Kennan2 has joined #openstack-neutron | 02:02 | |
| *** signed8b_ is now known as signed8bit_ZZZzz | 02:02 | |
| *** Kennan has quit IRC | 02:03 | |
| *** signed8bit_ZZZzz has quit IRC | 02:04 | |
| *** shashank_hegde_ has joined #openstack-neutron | 02:04 | |
| *** zhenguo has joined #openstack-neutron | 02:09 | |
| *** patrickeast_ has quit IRC | 02:10 | |
| *** coolsvap|away has quit IRC | 02:10 | |
| *** yamahata has quit IRC | 02:10 | |
| *** coolsvap|away has joined #openstack-neutron | 02:12 | |
| *** david-lyle has joined #openstack-neutron | 02:12 | |
| *** abregman has quit IRC | 02:13 | |
| *** punter has quit IRC | 02:17 | |
| *** shivrao has joined #openstack-neutron | 02:17 | |
| *** dims_ has quit IRC | 02:17 | |
| *** markvoelker has quit IRC | 02:18 | |
| *** yamamoto has joined #openstack-neutron | 02:18 | |
| *** dave-mccowan has quit IRC | 02:18 | |
| *** mutoulbj has joined #openstack-neutron | 02:22 | |
| openstackgerrit | Zhi Chang proposed openstack/neutron: Keep dns nameserver order consistency https://review.openstack.org/199588 | 02:25 |
| *** mmasaki has quit IRC | 02:26 | |
| *** mmasaki has joined #openstack-neutron | 02:26 | |
| *** padkrish has quit IRC | 02:28 | |
| *** padkrish has joined #openstack-neutron | 02:28 | |
| *** padkrish has quit IRC | 02:29 | |
| *** shashank_hegde_ has quit IRC | 02:31 | |
| *** sdake_ is now known as sdake | 02:34 | |
| *** david-lyle has quit IRC | 02:35 | |
| *** julien_ZTE has joined #openstack-neutron | 02:35 | |
| *** MentalRay has quit IRC | 02:37 | |
| *** gberginc has joined #openstack-neutron | 02:41 | |
| *** tflynn has joined #openstack-neutron | 02:41 | |
| *** sdake_ has joined #openstack-neutron | 02:43 | |
| *** tflynn_ has quit IRC | 02:44 | |
| *** coolsvap has quit IRC | 02:45 | |
| *** tflynn has quit IRC | 02:45 | |
| *** johnbelamaric has quit IRC | 02:46 | |
| *** sdake has quit IRC | 02:46 | |
| *** sdake__ has joined #openstack-neutron | 02:46 | |
| *** baoli has quit IRC | 02:49 | |
| *** sdake_ has quit IRC | 02:50 | |
| *** hakimo has joined #openstack-neutron | 02:52 | |
| *** arajagopal has quit IRC | 02:52 | |
| *** baoli has joined #openstack-neutron | 02:52 | |
| *** patrickeast has joined #openstack-neutron | 02:53 | |
| *** mchestr has joined #openstack-neutron | 02:53 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** shivrao_ has joined #openstack-neutron | 02:54 | |
| *** shivrao has quit IRC | 02:58 | |
| *** shivrao_ is now known as shivrao | 02:58 | |
| *** mchestr has quit IRC | 02:58 | |
| *** sdake__ is now known as sdake | 02:58 | |
| *** signed8b_ has joined #openstack-neutron | 03:04 | |
| *** jwy has quit IRC | 03:08 | |
| *** ParsectiX has joined #openstack-neutron | 03:09 | |
| *** achanda has joined #openstack-neutron | 03:10 | |
| *** vivek-ebay has quit IRC | 03:18 | |
| *** dims_ has joined #openstack-neutron | 03:19 | |
| *** wei-wang has joined #openstack-neutron | 03:20 | |
| *** julien_ZTE has quit IRC | 03:21 | |
| *** dims_ has quit IRC | 03:23 | |
| *** yog__ has joined #openstack-neutron | 03:24 | |
| *** pal has joined #openstack-neutron | 03:28 | |
| *** topshare has quit IRC | 03:28 | |
| *** topshare has joined #openstack-neutron | 03:30 | |
| *** markvoelker has joined #openstack-neutron | 03:35 | |
| *** y_sawai has joined #openstack-neutron | 03:36 | |
| *** mmasaki has quit IRC | 03:36 | |
| *** mmasaki has joined #openstack-neutron | 03:37 | |
| *** y_sawai has quit IRC | 03:38 | |
| *** david-lyle has joined #openstack-neutron | 03:38 | |
| *** markvoelker_ has joined #openstack-neutron | 03:38 | |
| *** shashank_hegde_ has joined #openstack-neutron | 03:40 | |
| *** markvoelker has quit IRC | 03:40 | |
| *** _cjones_ has joined #openstack-neutron | 03:41 | |
| *** _cjones_ has quit IRC | 03:41 | |
| *** hitalia has joined #openstack-neutron | 03:42 | |
| *** yog__ has quit IRC | 03:43 | |
| *** arosen has quit IRC | 03:43 | |
| *** arajagopal has joined #openstack-neutron | 03:45 | |
| *** zhhuabj_ has quit IRC | 03:45 | |
| openstackgerrit | Wei Wang proposed openstack/neutron-specs: Enable setting default rules for default security group https://review.openstack.org/200847 | 03:55 |
| *** achanda has quit IRC | 03:56 | |
| *** breitz has quit IRC | 03:56 | |
| *** breitz has joined #openstack-neutron | 03:56 | |
| *** arosen has joined #openstack-neutron | 03:56 | |
| *** achanda has joined #openstack-neutron | 03:57 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Fix a regression in a recent IPAM change https://review.openstack.org/201907 | 03:58 |
| *** signed8__ has joined #openstack-neutron | 04:03 | |
| *** markvoelker has joined #openstack-neutron | 04:04 | |
| *** signed8b_ has quit IRC | 04:04 | |
| *** sthillma has joined #openstack-neutron | 04:05 | |
| *** sthillma_ has joined #openstack-neutron | 04:06 | |
| *** crc32 has quit IRC | 04:06 | |
| *** markvoelker_ has quit IRC | 04:07 | |
| *** markvoelker_ has joined #openstack-neutron | 04:08 | |
| *** dannywilson has joined #openstack-neutron | 04:08 | |
| *** dannywilson has quit IRC | 04:08 | |
| *** dannywilson has joined #openstack-neutron | 04:09 | |
| *** dannywilson has quit IRC | 04:09 | |
| *** markvoelker has quit IRC | 04:09 | |
| *** sthillma has quit IRC | 04:09 | |
| *** sthillma_ is now known as sthillma | 04:09 | |
| *** carl_baldwin has quit IRC | 04:10 | |
| *** gildub has quit IRC | 04:14 | |
| *** tflynn has joined #openstack-neutron | 04:17 | |
| *** signed8bit has joined #openstack-neutron | 04:17 | |
| *** achanda has quit IRC | 04:18 | |
| *** tflynn_ has joined #openstack-neutron | 04:18 | |
| *** signed8__ has quit IRC | 04:20 | |
| *** mmasaki has quit IRC | 04:20 | |
| *** coolsvap|away is now known as coolsvap | 04:20 | |
| *** mmasaki has joined #openstack-neutron | 04:21 | |
| *** tflynn has quit IRC | 04:21 | |
| *** signed8bit has quit IRC | 04:21 | |
| *** hichihara has quit IRC | 04:22 | |
| *** dannywilson has joined #openstack-neutron | 04:24 | |
| *** sridhar_ram has joined #openstack-neutron | 04:24 | |
| *** tflynn_ has quit IRC | 04:26 | |
| *** topshare has quit IRC | 04:26 | |
| *** tflynn has joined #openstack-neutron | 04:28 | |
| *** m0h4_hun7 has quit IRC | 04:28 | |
| *** fifieldt has joined #openstack-neutron | 04:28 | |
| *** topshare has joined #openstack-neutron | 04:31 | |
| *** Aish has joined #openstack-neutron | 04:32 | |
| *** lazy_prince has joined #openstack-neutron | 04:34 | |
| *** ajo has quit IRC | 04:35 | |
| *** ajo has joined #openstack-neutron | 04:37 | |
| *** pradipta has joined #openstack-neutron | 04:37 | |
| *** tflynn has quit IRC | 04:37 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/neutron: Switch to the oslo_utils.fileutils https://review.openstack.org/199550 | 04:38 |
| *** baoli has quit IRC | 04:39 | |
| *** yog__ has joined #openstack-neutron | 04:40 | |
| *** oomichi has quit IRC | 04:40 | |
| *** baoli_ has joined #openstack-neutron | 04:41 | |
| *** gildub has joined #openstack-neutron | 04:41 | |
| *** boris-42 has joined #openstack-neutron | 04:43 | |
| *** baoli_ has quit IRC | 04:44 | |
| *** padkrish has joined #openstack-neutron | 04:48 | |
| *** padkrish has quit IRC | 04:48 | |
| *** padkrish has joined #openstack-neutron | 04:48 | |
| *** tflynn has joined #openstack-neutron | 04:50 | |
| *** Aish has quit IRC | 04:51 | |
| *** patrickeast has quit IRC | 04:51 | |
| *** fitoduarte has joined #openstack-neutron | 04:51 | |
| *** yfried_ has quit IRC | 04:53 | |
| *** shashank_hegde_ has quit IRC | 04:56 | |
| *** shashank_hegde_ has joined #openstack-neutron | 04:56 | |
| *** tflynn has quit IRC | 04:57 | |
| *** shashank_hegde_ has quit IRC | 04:58 | |
| *** ekuris has quit IRC | 05:01 | |
| *** markvoelker has joined #openstack-neutron | 05:06 | |
| *** markvoelker_ has quit IRC | 05:06 | |
| *** sridhar_ram1 has joined #openstack-neutron | 05:07 | |
| *** achanda has joined #openstack-neutron | 05:07 | |
| *** sridhar_ram has quit IRC | 05:08 | |
| *** markvoelker_ has joined #openstack-neutron | 05:08 | |
| *** sridhar_ram2 has joined #openstack-neutron | 05:09 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/neutron: Switch to the oslo_utils.fileutils https://review.openstack.org/199550 | 05:10 |
| *** sridhar_ram1 has quit IRC | 05:11 | |
| *** markvoelker has quit IRC | 05:11 | |
| *** gberginc has joined #openstack-neutron | 05:11 | |
| *** adduarte has joined #openstack-neutron | 05:13 | |
| *** padkrish has quit IRC | 05:13 | |
| *** srilatha has joined #openstack-neutron | 05:13 | |
| *** padkrish has joined #openstack-neutron | 05:14 | |
| *** lajos-katona has joined #openstack-neutron | 05:14 | |
| *** gberginc has quit IRC | 05:14 | |
| *** fitoduarte has quit IRC | 05:16 | |
| *** mkolesni__ has joined #openstack-neutron | 05:21 | |
| *** sudipto has joined #openstack-neutron | 05:24 | |
| *** irenab has joined #openstack-neutron | 05:24 | |
| *** jbell8 has joined #openstack-neutron | 05:27 | |
| *** minwang2 has joined #openstack-neutron | 05:27 | |
| *** moshele has joined #openstack-neutron | 05:28 | |
| *** Longgeek has joined #openstack-neutron | 05:28 | |
| *** padkrish has quit IRC | 05:30 | |
| *** aranjan has joined #openstack-neutron | 05:30 | |
| *** ekuris has joined #openstack-neutron | 05:31 | |
| *** mutoulbj_ has joined #openstack-neutron | 05:31 | |
| *** yog__ has quit IRC | 05:33 | |
| *** mutoulb__ has joined #openstack-neutron | 05:34 | |
| *** mutoulbj has quit IRC | 05:34 | |
| *** sudipto has quit IRC | 05:35 | |
| *** ekuris has quit IRC | 05:36 | |
| *** pradipta has quit IRC | 05:37 | |
| *** mutoulbj_ has quit IRC | 05:37 | |
| *** Sukhdev has joined #openstack-neutron | 05:38 | |
| *** ByPasS has joined #openstack-neutron | 05:39 | |
| *** kiran-r has joined #openstack-neutron | 05:39 | |
| *** jyuso1 has quit IRC | 05:43 | |
| *** padkrish has joined #openstack-neutron | 05:47 | |
| *** topshare has quit IRC | 05:48 | |
| *** ekuris has joined #openstack-neutron | 05:49 | |
| *** topshare has joined #openstack-neutron | 05:51 | |
| *** sudipto has joined #openstack-neutron | 05:51 | |
| *** amirosh has joined #openstack-neutron | 05:52 | |
| *** yog__ has joined #openstack-neutron | 05:52 | |
| *** dboik has joined #openstack-neutron | 05:53 | |
| *** Murali has joined #openstack-neutron | 05:53 | |
| *** fawadkhaliq has joined #openstack-neutron | 05:54 | |
| *** norman has joined #openstack-neutron | 05:55 | |
| *** sridhar_ram2 has quit IRC | 05:56 | |
| openstackgerrit | gongysh proposed openstack/neutron: Replace internal calls of create_{network, subnet, port} https://review.openstack.org/129773 | 05:57 |
| *** dboik has quit IRC | 05:57 | |
| *** itzikb_ has joined #openstack-neutron | 06:01 | |
| *** adreznec has quit IRC | 06:01 | |
| *** changzhi has joined #openstack-neutron | 06:02 | |
| *** sudipto has quit IRC | 06:03 | |
| *** adreznec has joined #openstack-neutron | 06:03 | |
| *** cing has joined #openstack-neutron | 06:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/201928 | 06:09 |
| *** anilvenkata has joined #openstack-neutron | 06:09 | |
| *** shivrao has quit IRC | 06:10 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: WIP: deco syntax for registry https://review.openstack.org/192081 | 06:13 |
| *** bzhao6 has quit IRC | 06:14 | |
| *** shashank_hegde_ has joined #openstack-neutron | 06:14 | |
| openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB https://review.openstack.org/183369 | 06:15 |
| *** padkrish has quit IRC | 06:17 | |
| *** yfried_ has joined #openstack-neutron | 06:18 | |
| *** sthillma has quit IRC | 06:20 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: WIP: deco syntax for registry https://review.openstack.org/192081 | 06:20 |
| openstackgerrit | shihanzhang proposed openstack/neutron: Disable DHCP service if dhcp port is deleted https://review.openstack.org/196927 | 06:20 |
| *** shashank_hegde_ has quit IRC | 06:21 | |
| *** yfried__ has joined #openstack-neutron | 06:22 | |
| *** yfried_ has quit IRC | 06:22 | |
| *** trinaths has joined #openstack-neutron | 06:24 | |
| *** ParsectiX has quit IRC | 06:24 | |
| *** abregman has joined #openstack-neutron | 06:25 | |
| *** padkrish has joined #openstack-neutron | 06:26 | |
| *** shashank_hegde_ has joined #openstack-neutron | 06:27 | |
| *** minwang2 has quit IRC | 06:28 | |
| *** gberginc has joined #openstack-neutron | 06:28 | |
| *** wei-wang has quit IRC | 06:28 | |
| *** topshare has quit IRC | 06:29 | |
| *** afazekas has joined #openstack-neutron | 06:31 | |
| *** fawadkhaliq has quit IRC | 06:33 | |
| *** topshare has joined #openstack-neutron | 06:33 | |
| openstackgerrit | ZhaoBo proposed openstack/neutron: Fixed pass wrong cidr check during create subnet https://review.openstack.org/201942 | 06:36 |
| *** pal has quit IRC | 06:38 | |
| trinaths | Hi, Good Morning All | 06:39 |
| *** kawa2014 has joined #openstack-neutron | 06:41 | |
| *** pal has joined #openstack-neutron | 06:41 | |
| *** nmagnezi has joined #openstack-neutron | 06:44 | |
| *** lori|away is now known as lori | 06:45 | |
| *** shashank_hegde_ has quit IRC | 06:46 | |
| *** neeti has joined #openstack-neutron | 06:46 | |
| *** Miouge has quit IRC | 06:49 | |
| *** Miouge has joined #openstack-neutron | 06:50 | |
| *** amirosh has quit IRC | 06:51 | |
| *** ildikov has quit IRC | 06:53 | |
| *** fawadkhaliq has joined #openstack-neutron | 06:54 | |
| *** srilatha has quit IRC | 06:54 | |
| *** abehl has joined #openstack-neutron | 06:54 | |
| *** fawadk has joined #openstack-neutron | 06:55 | |
| *** jbell8 has quit IRC | 06:58 | |
| *** adduarte has quit IRC | 06:58 | |
| *** jyuso1 has joined #openstack-neutron | 06:58 | |
| *** fawadkhaliq has quit IRC | 06:58 | |
| *** fawadkhaliq has joined #openstack-neutron | 06:59 | |
| *** fawadk has quit IRC | 07:00 | |
| *** vthapar has joined #openstack-neutron | 07:03 | |
| *** yamamoto has quit IRC | 07:03 | |
| *** HoloIRCUser4 has joined #openstack-neutron | 07:03 | |
| *** evgenyf has joined #openstack-neutron | 07:03 | |
| *** HoloIRCUser4 is now known as apuimedo-2 | 07:03 | |
| *** dannywilson has quit IRC | 07:05 | |
| *** sdake has quit IRC | 07:07 | |
| openstackgerrit | Zhi Chang proposed openstack/neutron: Keep dns nameserver order consistency https://review.openstack.org/199588 | 07:07 |
| *** josecastroleon has quit IRC | 07:07 | |
| *** topshare_ has joined #openstack-neutron | 07:07 | |
| *** rook has quit IRC | 07:07 | |
| *** yamamoto has joined #openstack-neutron | 07:09 | |
| *** lukasa has joined #openstack-neutron | 07:09 | |
| *** topshare has quit IRC | 07:09 | |
| *** arajagopal has quit IRC | 07:10 | |
| *** changzhi has quit IRC | 07:10 | |
| *** sthillma has joined #openstack-neutron | 07:10 | |
| *** mutoulb__ has quit IRC | 07:10 | |
| *** alex_klimov has joined #openstack-neutron | 07:10 | |
| *** mutoulbj has joined #openstack-neutron | 07:11 | |
| openstackgerrit | Zhi Chang proposed openstack/neutron: Keep dns nameserver order consistency https://review.openstack.org/199588 | 07:11 |
| *** ihrachyshka has joined #openstack-neutron | 07:12 | |
| ihrachyshka | ajo, ping re service plugin for qos | 07:13 |
| *** ushkalim has joined #openstack-neutron | 07:18 | |
| *** yog__ has quit IRC | 07:19 | |
| *** jlibosva has joined #openstack-neutron | 07:20 | |
| *** dims_ has joined #openstack-neutron | 07:20 | |
| *** yog__ has joined #openstack-neutron | 07:22 | |
| *** Longgeek has quit IRC | 07:23 | |
| *** fzdarsky has joined #openstack-neutron | 07:24 | |
| *** zigo has quit IRC | 07:24 | |
| *** zigo has joined #openstack-neutron | 07:25 | |
| *** sbelous has quit IRC | 07:25 | |
| *** dims_ has quit IRC | 07:26 | |
| *** sbelous has joined #openstack-neutron | 07:26 | |
| *** Kiall has quit IRC | 07:26 | |
| *** rook has joined #openstack-neutron | 07:27 | |
| *** Kiall has joined #openstack-neutron | 07:28 | |
| *** topshare_ has quit IRC | 07:28 | |
| *** q3k has quit IRC | 07:29 | |
| *** jschwarz has joined #openstack-neutron | 07:30 | |
| *** sthillma has quit IRC | 07:30 | |
| jschwarz | irenab, ping | 07:30 |
| *** aix has joined #openstack-neutron | 07:31 | |
| *** rossella_s has joined #openstack-neutron | 07:32 | |
| *** yfried has joined #openstack-neutron | 07:34 | |
| *** lazy_prince has quit IRC | 07:34 | |
| *** lazy_prince has joined #openstack-neutron | 07:34 | |
| *** yfried__ has quit IRC | 07:35 | |
| *** padkrish has quit IRC | 07:36 | |
| *** jschwarz has quit IRC | 07:36 | |
| *** topshare has joined #openstack-neutron | 07:37 | |
| *** Sukhdev has quit IRC | 07:37 | |
| *** jlanoux has joined #openstack-neutron | 07:40 | |
| *** Miouge has quit IRC | 07:40 | |
| *** yushiro has joined #openstack-neutron | 07:41 | |
| openstackgerrit | Aviram Bar-Haim proposed openstack/neutron: Fix SR-IOV mechanism driver tests directory https://review.openstack.org/201965 | 07:43 |
| moshele | ihrachyshka: can you review https://review.openstack.org/201965 it trivial fix | 07:46 |
| *** ildikov has joined #openstack-neutron | 07:47 | |
| *** dsc_ has joined #openstack-neutron | 07:47 | |
| moshele | garyk:^ | 07:47 |
| *** safchain has joined #openstack-neutron | 07:49 | |
| *** rotbeard has joined #openstack-neutron | 07:50 | |
| *** topshare has quit IRC | 07:50 | |
| *** dsc_ has quit IRC | 07:51 | |
| *** obondarev__ has joined #openstack-neutron | 07:55 | |
| *** trinaths has quit IRC | 07:55 | |
| *** yuanying_ has joined #openstack-neutron | 07:55 | |
| *** trinaths has joined #openstack-neutron | 07:55 | |
| *** erkules_ is now known as erkules | 07:55 | |
| *** erkules has joined #openstack-neutron | 07:55 | |
| *** topshare has joined #openstack-neutron | 07:56 | |
| *** yuanying has quit IRC | 07:57 | |
| openstackgerrit | Joe Mills proposed openstack/neutron: midonet: define reference to midonet mech driver https://review.openstack.org/201969 | 07:58 |
| *** ronis has joined #openstack-neutron | 08:00 | |
| *** tflynn has joined #openstack-neutron | 08:00 | |
| *** park_ has joined #openstack-neutron | 08:00 | |
| *** Longgeek has joined #openstack-neutron | 08:00 | |
| *** vikram has joined #openstack-neutron | 08:01 | |
| *** yamamoto has quit IRC | 08:01 | |
| *** achanda has quit IRC | 08:02 | |
| *** ZZelle has quit IRC | 08:03 | |
| *** fawadkhaliq has quit IRC | 08:05 | |
| *** yfried has quit IRC | 08:07 | |
| ihrachyshka | ajo, mkolesni__: what's the plan for service plugin patch? anyone working on it? | 08:07 |
| ihrachyshka | I don't see motion there | 08:08 |
| ajo | ihrachyshka: mkolesni__ and I were just talking about it | 08:08 |
| ajo | on it's way in a few minutes | 08:08 |
| *** dmellado has left #openstack-neutron | 08:08 | |
| ajo | yesterday we spent some time looking at some iterations which were necessary plugin<-> qos service plugin | 08:09 |
| ihrachyshka | ok, it's a critical piece and we should make sure it's going in today. | 08:09 |
| ajo | yes, it's simple, it should not take more than a few minutes to fix now | 08:09 |
| ajo | expect green jenkins and responding APIs in the next hour ;) | 08:09 |
| *** tflynn has quit IRC | 08:09 | |
| *** yfried_ has joined #openstack-neutron | 08:10 | |
| *** ccard has quit IRC | 08:11 | |
| *** norman has quit IRC | 08:12 | |
| *** jistr has joined #openstack-neutron | 08:12 | |
| *** boris-42 has quit IRC | 08:12 | |
| *** aranjan has quit IRC | 08:12 | |
| *** yushiro has quit IRC | 08:14 | |
| *** adjohn has quit IRC | 08:16 | |
| *** adjohn has joined #openstack-neutron | 08:16 | |
| moshele | gsagie, ajo the missing piece in ovs agent https://review.openstack.org/#/c/201975/1 | 08:17 |
| mkolesni__ | ajo, ihrachyshka: im sending a very preliminary patch set just to see that the direction is good | 08:17 |
| *** mkolesni__ is now known as mkolesni | 08:17 | |
| ajo | thanks mkolesni , ping me when it's jup | 08:18 |
| ajo | up | 08:18 |
| ajo | moshele, let me look | 08:18 |
| *** jbonjean has quit IRC | 08:18 | |
| *** gildub has quit IRC | 08:18 | |
| moshele | ajo: with this you can test the full til the ovs agent and apply rules | 08:18 |
| ajo | moshele, I didn't realize this before in the sriov agent | 08:19 |
| ajo | but I believe inheritance is not needed here | 08:19 |
| mkolesni | ajo: https://review.openstack.org/201979 | 08:19 |
| ajo | we can just compose with the l2agent methods in an attribute | 08:19 |
| ajo | or have an extension manager singleton | 08:19 |
| mkolesni | ajo: thats the ML2 alterations | 08:19 |
| *** yamamoto has joined #openstack-neutron | 08:20 | |
| *** ygbo has joined #openstack-neutron | 08:20 | |
| ajo | mkolesni, I know, I just think L2Agent doesn't need to compose the agent just to provide a few methods | 08:20 |
| moshele | ajo: the l2 agent can be a place holder to agent code reuse | 08:20 |
| ajo | inheritance and mixins is very hard to follow for later code readers | 08:20 |
| ajo | yes | 08:20 |
| ajo | but you don't need to use inheritance for that | 08:20 |
| ajo | later, | 08:20 |
| mkolesni | ajo: i think you meant moshele | 08:20 |
| ajo | mkolesni, sorry, yes | 08:21 |
| ajo | later, when people come to read the code | 08:21 |
| *** sudipto has joined #openstack-neutron | 08:21 | |
| ajo | it's hard for them to find where process_port_extensions(self.context, details) goes | 08:21 |
| ajo | instead if you did | 08:21 |
| ajo | self.ext_manager.process_port_extensions(self.context, details) | 08:21 |
| ajo | it's super clear | 08:21 |
| *** markvoelker_ has quit IRC | 08:21 | |
| ajo | that's why inheritance is not preferred unless you have a good reason | 08:21 |
| *** yfried_ is now known as yfried|afk | 08:21 | |
| ajo | and neutron has used it a lot, without good reasons | 08:21 |
| ajo | we're trying to step back from that | 08:22 |
| ajo | ihrachyshka ^ | 08:22 |
| moshele | ajo: this is becasue of many mixing but if it will be single inheritance I think it fine | 08:22 |
| ajo | (looping in ihrachyshka to check that I'm not saying nonsense, I sometimes do) | 08:22 |
| *** fawadkhaliq has joined #openstack-neutron | 08:22 | |
| gsagie | ajo: i agree | 08:22 |
| *** yfried|afk is now known as yfried_ | 08:22 | |
| ajo | but here we're inheriting already from 3 mixings + your new base class | 08:23 |
| ajo | gsagie, moshele , ok, let me think about it more calmly may be you're right and it's really a "base" for L2 agents | 08:23 |
| ajo | but | 08:23 |
| gsagie | i would prefer it to be a composition | 08:23 |
| moshele | ajo: it legacy I assume it will fix some day :) | 08:23 |
| gsagie | not inheritance | 08:24 |
| gsagie | but thats just me | 08:24 |
| ajo | gsagie, that's my first thinking too | 08:24 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Galera multi-writers compliant sync_allocations https://review.openstack.org/192099 | 08:24 |
| moshele | gsagie: don't you think l2 agent need some base class for common funcinality | 08:24 |
| irenab | ajo: I am with you on this | 08:24 |
| ajo | I want to sit down, and read L2 agent again, and think if we can extend it to be a real base class in the future, or if it would be better as composition | 08:24 |
| ajo | it's on a fine line | 08:24 |
| ajo | moshele: common functionalities also are in libraries | 08:25 |
| ajo | but | 08:25 |
| ajo | I see your point too | 08:25 |
| irenab | adding functionality => composition, implementing interface=>inheritance | 08:25 |
| ajo | sometimes | 08:25 |
| ajo | modifying functionality also is something good for inheritance | 08:26 |
| moshele | ireanb there is also abstract class | 08:26 |
| ajo | when you just modify a big class, and change a processing method to do something different | 08:26 |
| ajo | for example IptablesFirewallDriver is inherited by OVSHybridFirewallDriver (or something like that) can't remember the exact methods now | 08:26 |
| ajo | the second modifies how the tap interfaces and so are named. | 08:27 |
| ajo | that's another fine example of inheritance use | 08:27 |
| ajo | moshele, for example | 08:27 |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 08:27 |
| ajo | if your l2agent provided main... | 08:27 |
| irenab | moshele: abstract class is sort of interface | 08:27 |
| ajo | and general l2 agent initialitzation | 08:27 |
| ajo | I'd agree it was a good class to inherit | 08:27 |
| ajo | but only providing methods... | 08:27 |
| ajo | hmm | 08:27 |
| ajo | may be we can refactor all this, in the future to an l2-agent | 08:28 |
| ajo | class | 08:28 |
| ajo | which initializes basic stuff, provides main.. and loops over a function implemented on the inheriting class | 08:28 |
| moshele | ajo: that was the intention, but I did do it because it major change for the l2 agnet | 08:29 |
| ajo | irenab, moshele , gsagie , and l2_agent, I'd remove that class currrently and create a singleton and get_manager()/get_instance() in the extension manager module | 08:29 |
| irenab | I guess L2 agent will have to go under major refactoring to be trully modular one, so the current approach can be either one | 08:30 |
| ajo | moshele, then may be we can do it in follow up, it's a good idea, I agree | 08:30 |
| *** topshare has quit IRC | 08:30 | |
| ajo | if that's the long term plan, and we can mark that with a TODO(moshele) | 08:30 |
| ajo | it could be fine | 08:30 |
| *** obondarev__ has quit IRC | 08:30 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: WIP: deco syntax for registry https://review.openstack.org/192081 | 08:31 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Ensure non-overlapping cidrs in subnetpools with galera https://review.openstack.org/187985 | 08:31 |
| *** miyagishi_t has joined #openstack-neutron | 08:31 | |
| gsagie | ajo : btw, https://review.openstack.org/#/c/201583/ has some fixes that are required for the end-to-end tests | 08:32 |
| *** q3k has joined #openstack-neutron | 08:32 | |
| ajo | gsagie, thanks, I will check now | 08:33 |
| *** hanzhang has quit IRC | 08:33 | |
| *** hitalia has quit IRC | 08:33 | |
| *** topshare has joined #openstack-neutron | 08:34 | |
| *** iwamoto has quit IRC | 08:35 | |
| *** markvoelker has joined #openstack-neutron | 08:36 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart https://review.openstack.org/182920 | 08:37 |
| *** yatin has joined #openstack-neutron | 08:40 | |
| openstackgerrit | Merged openstack/neutron: Tighten exception handler for import_object https://review.openstack.org/201061 | 08:42 |
| openstackgerrit | shihanzhang proposed openstack/neutron: Avoid using logging in signal handler https://review.openstack.org/201474 | 08:42 |
| openstackgerrit | shihanzhang proposed openstack/neutron: Unplug the VIF if dhcp port is deleted https://review.openstack.org/196927 | 08:44 |
| *** ronis_ has joined #openstack-neutron | 08:44 | |
| *** ronis has quit IRC | 08:45 | |
| *** markvoelker has quit IRC | 08:45 | |
| *** gongysh has quit IRC | 08:46 | |
| *** mickeys has joined #openstack-neutron | 08:46 | |
| *** trinaths has quit IRC | 08:46 | |
| *** kawa2014 has quit IRC | 08:48 | |
| *** kawa2014 has joined #openstack-neutron | 08:49 | |
| *** hanzhang has joined #openstack-neutron | 08:49 | |
| *** trinaths has joined #openstack-neutron | 08:50 | |
| *** markvoelker has joined #openstack-neutron | 08:51 | |
| *** DaveJ__ has joined #openstack-neutron | 08:54 | |
| *** markvoelker has quit IRC | 08:56 | |
| *** mickeys has quit IRC | 09:01 | |
| *** markvoelker has joined #openstack-neutron | 09:06 | |
| *** aranjan has joined #openstack-neutron | 09:09 | |
| *** markvoelker has quit IRC | 09:10 | |
| jlibosva | moshele: ajo ihrachyshka anyone planning to prepare fullstack test for qos? I'd like to do that as I'm familiar with amuller's work | 09:12 |
| jlibosva | gsagie: ^^ | 09:12 |
| ajo | jlibosva, we will need something once fullstack is merged | 09:12 |
| ihrachyshka | jlibosva, I planned to start looking into amuller's patches today as he said we'll require them in master first | 09:12 |
| ihrachyshka | so we need review for amuller's work and then we are able to implement fullstack | 09:13 |
| jlibosva | ihrachyshka: ajo I think we can work in parallel even though it's not merged | 09:13 |
| ajo | may be jlibosva could be good at that | 09:13 |
| ajo | jlibosva, yeah, may be you could do it by locally cherry picking fullstack stuff | 09:13 |
| ihrachyshka | ajo, totally agree on jlibosva as a candidate :) | 09:13 |
| ajo | jlibosva++ | 09:13 |
| jlibosva | ajo: are you reading my mind? | 09:13 |
| ihrachyshka | jlibosva, yeah, it may be worth it | 09:13 |
| jlibosva | ack, I'll add it to etherpad | 09:13 |
| ihrachyshka | jlibosva, no, he only owns a time machine | 09:13 |
| *** aranjan has quit IRC | 09:14 | |
| ajo | jlibosva, lol, it must be this monster ripper thing I'm drinking this morning.. | 09:14 |
| ajo | nothing to do too often ;D | 09:14 |
| ajo | unless you want to become diabetic, destroy your stomach, and probably more stuff | 09:15 |
| ihrachyshka | ajo, btw, could you please travel to +3 months from now and bring us the whole feature please? | 09:15 |
| *** yamahata has joined #openstack-neutron | 09:15 | |
| ajo | ihrachyshka, a future-cherry-pick | 09:15 |
| ajo | that'd be awesome ;) | 09:15 |
| ajo | but then we'd create a loopwhole, as we cherry pick, we stop working | 09:15 |
| ajo | and we create another parallel universe | 09:15 |
| jlibosva | ajo: I think there was a xkcd comics that if you drink a lot of caffeine, you'll see the future :D | 09:16 |
| ajo | where we're less smart because we didn't have to do it | 09:16 |
| ajo | jlibosva lol | 09:16 |
| *** yfried_ is now known as yfried|afk | 09:16 | |
| *** apuimedo-2 has quit IRC | 09:17 | |
| ihrachyshka | ajo, mkolesni: enlighten me on why we couple qos plugin into ml2. | 09:18 |
| *** ZZelle has joined #openstack-neutron | 09:18 | |
| *** yamahata has quit IRC | 09:20 | |
| ihrachyshka | ajo, if you give me 10 mins, I want to walk thru the devref once more before you respin | 09:20 |
| *** markvoelker has joined #openstack-neutron | 09:20 | |
| *** kiran-r has quit IRC | 09:21 | |
| *** zhhuabj has joined #openstack-neutron | 09:23 | |
| *** ccard has joined #openstack-neutron | 09:23 | |
| *** zhhuabj has quit IRC | 09:23 | |
| *** zhhuabj_ has joined #openstack-neutron | 09:23 | |
| *** ekuris has quit IRC | 09:24 | |
| *** ekuris has joined #openstack-neutron | 09:24 | |
| *** markvoelker has quit IRC | 09:25 | |
| *** yfried|afk is now known as yfried_ | 09:25 | |
| *** gongysh has joined #openstack-neutron | 09:29 | |
| openstackgerrit | shihanzhang proposed openstack/neutron: Avoid using logging in signal handler https://review.openstack.org/201474 | 09:29 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: DO NOT MERGE: RBAC PATCH WITH SPLIT MIGRATIONS https://review.openstack.org/202013 | 09:31 |
| *** doude has quit IRC | 09:31 | |
| ihrachyshka | kevinbenton, I've played with splitting your migration rule into pieces ^^ hopefully it will work in gate and not just for me locally | 09:32 |
| ihrachyshka | if it does, you are welcome to reuse once we get split merged | 09:32 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Move db_list to OVSBridge https://review.openstack.org/200279 | 09:34 |
| *** markvoelker has joined #openstack-neutron | 09:35 | |
| *** dguitarbite has quit IRC | 09:36 | |
| openstackgerrit | Ross Annetts proposed openstack/python-neutronclient: Testing. https://review.openstack.org/202017 | 09:37 |
| moshele | ajo: I did the change https://review.openstack.org/#/c/201975/ | 09:37 |
| ihrachyshka | ajo, may I get response? | 09:37 |
| ZZelle | ihrachyshka, hi | 09:40 |
| *** markvoelker has quit IRC | 09:40 | |
| ajo | ihrachyshka, mkolesni , irenab , can you re-read comments here: https://review.openstack.org/#/c/201979/ | 09:40 |
| ajo | I believe irenab is right, and this can be put into an extension, also I'd move most of the logic to the service plugin itself, as global methods of the module, so the integration with other plugins is just adding a couple of lines only | 09:41 |
| ajo | ihrachyshka, the ml2 design needs to know if the profile on the port changed to really notify mech_drivers (agents) down the line | 09:42 |
| ihrachyshka | ajo, I agree. and we need tests, I'm not convinced the fields are there. | 09:42 |
| ihrachyshka | tests would prove it | 09:42 |
| ajo | such design makes ml2 necessarily coupled to any extension, sadly | 09:42 |
| ajo | this is why mkolesni was proposing callbacks to return results about how they handled the value, to be able to identify such thing happening | 09:42 |
| ihrachyshka | ajo, but shouldn't extension only report whether notify is needed, without details on why? | 09:42 |
| ajo | ihrachyshka, yes, | 09:43 |
| ajo | only whether notify is needed, that's all we need | 09:43 |
| ihrachyshka | ok. so mkolesni we need it as an extension, we need tests, and overall as little coupling | 09:43 |
| ihrachyshka | ajo, so on more general note, why is it all in ml2? | 09:43 |
| ajo | ihrachyshka: if we used callbacks, we'd have the same issue, | 09:44 |
| ajo | since, the AFTER_UPDATE/CREATE happens from ml2/plugin now | 09:44 |
| ajo | the specific plugins implementing it, would have to notify the callback, or.... | 09:45 |
| ajo | now... call on us to handle the qos_policy_id | 09:45 |
| ajo | we can move that to callbacks later, but we believe we need larger discussion on how to use callbacks | 09:45 |
| ajo | in such case | 09:45 |
| ajo | and sanitize interfaces for that | 09:45 |
| ihrachyshka | ajo, so callbacks are ml2 only now in master? | 09:46 |
| ajo | ihrachyshka: AFTER_CREATE/UPDATE for ports, only ml2 for master, correct | 09:46 |
| *** SridharG has joined #openstack-neutron | 09:47 | |
| ajo | ihrachyshka, for it to be complete, I sense we may need to move those callbacks after plugin call (plugin independent) , | 09:47 |
| ajo | and... add another callback in the middle | 09:47 |
| ajo | to let "core resources extenders" do their thing | 09:47 |
| ihrachyshka | ajo, ok, that's fine for me. if we don't couple stuff more than it's now | 09:47 |
| ajo | but may be there was a reason why this was moved in the middle of ml2, it was armax, so he may know | 09:47 |
| ajo | after we do this | 09:47 |
| ihrachyshka | he is not avail these days | 09:47 |
| ajo | I think we will be on a good position to talk all together | 09:48 |
| ihrachyshka | ajo, I'm still walking thru devref | 09:48 |
| ihrachyshka | comments in 5 mins | 09:48 |
| ajo | and find the best way to do core resource extension properly | 09:48 |
| ajo | thanks ihrachyshka | 09:48 |
| ajo | in your's I'll keep bulletpoints to the modules you're talking about | 09:48 |
| ajo | even if you give the longer/detailed explanation, which in that case I think it's necessary | 09:48 |
| *** dguitarbite has joined #openstack-neutron | 09:48 | |
| ajo | to keep it consistent, then somebody reviewing the code can very easily identify where's exaclty what we're talking about (cross ref) | 09:49 |
| ajo | I guess mkolesni is @lunch ;) | 09:49 |
| *** markvoelker has joined #openstack-neutron | 09:49 | |
| moshele | ajo,ihrachyshka: why we have https://github.com/openstack/neutron/blob/feature/qos/neutron/api/rpc/handlers/resources_rpc.py#L46 is not it done automatically by http://docs.openstack.org/developer/oslo.versionedobjects/api/base.html#oslo_versionedobjects.base.VersionedObjectSerializer | 09:49 |
| ajo | yes moshele | 09:50 |
| ajo | we're going to use that | 09:50 |
| *** amotoki has quit IRC | 09:50 | |
| *** trinaths has quit IRC | 09:50 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Improve check_migration command error message https://review.openstack.org/199075 | 09:50 |
| ajo | versioned objects have two methods to serialize/deserialize | 09:50 |
| ajo | we just need to call on those :) | 09:50 |
| ajo | but we didn't have them in full until now, only dicts :) | 09:50 |
| ajo | I can point you to them if you want to add a patch for it. | 09:50 |
| *** trinaths has joined #openstack-neutron | 09:51 | |
| moshele | ajo: sure | 09:51 |
| ajo | moshele, you could do: | 09:52 |
| ajo | QosPolicy.obj_from_primitive(serialized_dict) | 09:52 |
| ajo | and you'd get a Qos Policy | 09:52 |
| ajo | or | 09:52 |
| ajo | you could do a | 09:52 |
| *** doude has joined #openstack-neutron | 09:53 | |
| ajo | qos_policy.obj_to_primitive() | 09:53 |
| ajo | and get a serialized dict | 09:53 |
| moshele | ok cool | 09:53 |
| ajo | I'm not sure if we have a general obj_from_primitive that detects the object | 09:53 |
| ajo | that'd be convenient | 09:53 |
| ajo | otherwise we need to detect the object type from resource_type (may be a mapping in rpc_callbacks) | 09:54 |
| ajo | thanks moshele | 09:54 |
| *** markvoelker has quit IRC | 09:54 | |
| ajo | going straight to read in your new changes | 09:54 |
| moshele | ajo: thanks :) | 09:54 |
| ajo | and then, I'll go back to service plugin and cut out any port ext handling so we can merge, and follow up with mkolesni patch | 09:54 |
| *** RA_ has joined #openstack-neutron | 09:54 | |
| ajo | now that it seems we're on the good direction, even if we need to do some uncoupling | 09:54 |
| ihrachyshka | ajo, ok, some initial comments for devref posted | 09:55 |
| ajo | thanks ihrachyshka , I'll read | 09:55 |
| *** RA_ has quit IRC | 09:55 | |
| ajo | in a few minutes | 09:55 |
| ajo | thanks | 09:55 |
| ajo | ahh moshele : stevedore.named.NamedExtensionManager) ... | 09:56 |
| ajo | this is how we use the "qos" extension | 09:56 |
| ajo | defined in setup.cfg | 09:57 |
| ajo | :) | 09:57 |
| ajo | I had totally forgotten | 09:57 |
| moshele | yep | 09:57 |
| moshele | to many features in the qos feature :) | 09:57 |
| openstackgerrit | shihanzhang proposed openstack/neutron: Check the validity of security group rule port range https://review.openstack.org/201524 | 10:01 |
| *** Kennan has joined #openstack-neutron | 10:03 | |
| *** Kennan2 has quit IRC | 10:03 | |
| *** dims_ has joined #openstack-neutron | 10:03 | |
| *** park_ has quit IRC | 10:04 | |
| *** gampel has joined #openstack-neutron | 10:04 | |
| *** markvoelker has joined #openstack-neutron | 10:04 | |
| *** pal has quit IRC | 10:05 | |
| *** dims_ has quit IRC | 10:05 | |
| ihrachyshka | ajo, may I ask you to utilize your review rights in feature/qos more actively? ;) | 10:05 |
| ihrachyshka | moshele, abandon? https://review.openstack.org/#/c/199996/ | 10:06 |
| *** txomon|fon has quit IRC | 10:06 | |
| *** mutoulbj has quit IRC | 10:07 | |
| ajo | ihrachyshka, yes, anything which could be ready for merge? | 10:07 |
| ajo | sorry, I'm doing lots of round robing today | 10:07 |
| ihrachyshka | robing? :) | 10:08 |
| ihrachyshka | like that? http://2.bp.blogspot.com/_sK5JKhMZm00/TR9vvM64CnI/AAAAAAAAAKs/G8OcUai8kZQ/s1600/stealing.jpg | 10:08 |
| ihrachyshka | ajo, not merge, but review | 10:08 |
| *** markvoelker has quit IRC | 10:08 | |
| moshele | ihrachyshka: if I will add serialize deserialize then can we use the tests https://review.openstack.org/#/c/199996/ ? | 10:09 |
| ihrachyshka | ajo, I have rules in policies: https://review.openstack.org/200608 | 10:09 |
| ihrachyshka | moshele, can be. ok, let's extend them | 10:09 |
| ajo | I've been reviewing all morning ;) | 10:09 |
| *** yfried_ is now known as yfried|afk | 10:09 | |
| ajo | but I will review that one, of course | 10:09 |
| ihrachyshka | ajo, I don't see a vote there though | 10:09 |
| ihrachyshka | ajo, being a core, it's all about reviews | 10:10 |
| ajo | just ping me with anything I'm missing and that's a priority, like that | 10:10 |
| ihrachyshka | ajo, ack | 10:10 |
| ihrachyshka | I'm reviewing myself now and will get back to you | 10:10 |
| *** aranjan has joined #openstack-neutron | 10:10 | |
| ajo | ihrachyshka: if you ask moshele , he will complain about me making too many change requests/reviews ;Dç | 10:11 |
| ajo | may be you were lucky in some sense.. | 10:11 |
| ihrachyshka | woot! I'm happy now! | 10:11 |
| ihrachyshka | but no I am not | 10:11 |
| ajo | ;) | 10:12 |
| ihrachyshka | I will be happy once it's in master, I guess | 10:12 |
| ajo | we will all be | 10:12 |
| moshele | me too | 10:12 |
| ihrachyshka | till that time, bare with me grumbling | 10:12 |
| ajo | thanks ihrachyshka , we appreciate your grumbling, it push us forward ;) | 10:12 |
| *** gildub has joined #openstack-neutron | 10:12 | |
| ihrachyshka | am I the bad policeman? | 10:12 |
| *** jbonjean has joined #openstack-neutron | 10:18 | |
| ajo | lol | 10:18 |
| ajo | ihrachyshka, you're the bad polic*y*man | 10:18 |
| *** yamamoto has quit IRC | 10:18 | |
| *** markvoelker has joined #openstack-neutron | 10:19 | |
| ajo | 5) | 10:19 |
| ajo | %) | 10:19 |
| openstackgerrit | sean mooney proposed openstack/neutron: adds configurable agent type https://review.openstack.org/197210 | 10:22 |
| *** markvoelker has quit IRC | 10:23 | |
| *** txomon|fon has joined #openstack-neutron | 10:24 | |
| *** HoloIRCUser1 has joined #openstack-neutron | 10:25 | |
| *** dedery has joined #openstack-neutron | 10:27 | |
| *** pc_m has joined #openstack-neutron | 10:28 | |
| *** yfried|afk is now known as yfried_ | 10:29 | |
| *** dedery has quit IRC | 10:30 | |
| *** markvoelker has joined #openstack-neutron | 10:33 | |
| *** HoloIRCUser1 has quit IRC | 10:35 | |
| *** HoloIRCUser1 has joined #openstack-neutron | 10:35 | |
| *** HoloIRCUser4 has joined #openstack-neutron | 10:37 | |
| *** HoloIRCUser1 has quit IRC | 10:37 | |
| *** mkolesni_ has joined #openstack-neutron | 10:37 | |
| *** markvoelker has quit IRC | 10:38 | |
| *** wei-wang has joined #openstack-neutron | 10:38 | |
| *** yog__ has quit IRC | 10:39 | |
| *** mkolesni has quit IRC | 10:40 | |
| *** Qlawy has joined #openstack-neutron | 10:41 | |
| Qlawy | Hi guys | 10:41 |
| *** jamielennox is now known as jamielennox|away | 10:41 | |
| Qlawy | can anyone tell me if GRE tunnelling is supported over IPv6 in neutron? | 10:41 |
| *** coolsvap is now known as coolsvap|away | 10:41 | |
| *** HoloIRCUser4 has quit IRC | 10:41 | |
| Qlawy | When I set local_ip to v6 IP it fails with "this is not ipv4 address" | 10:41 |
| kashyap | ihrachyshka: Well said on the upstream list: "it's a pity that some people see chats on summits as something defining the future" | 10:43 |
| kashyap | People can't make it to Summits for a bazillion reasons. | 10:44 |
| *** HoloIRCUser4 has joined #openstack-neutron | 10:44 | |
| kashyap | "If it's not on the list, it doesn't exist." | 10:45 |
| *** aranjan has quit IRC | 10:45 | |
| ihrachyshka | kashyap, or, people are on summits but are dragged into other sessions | 10:45 |
| kashyap | Yeah, too many parallel sessions. It's okay to discussion things at Summit, a summary or actions/decisions ought to be verbalized. | 10:45 |
| ihrachyshka | I haven't missed any of two latest, but at the same time, I failed to visit pretty any stable/releng sessions | 10:45 |
| ihrachyshka | why? because I actually do work | 10:45 |
| kashyap | It's not unalloyed fun, but need to be done. | 10:46 |
| kashyap | ihrachyshka: Yep, I understand. | 10:46 |
| ihrachyshka | gsagie, I've commented on unit tests: https://review.openstack.org/#/c/201583/ | 10:46 |
| *** markvoelker has joined #openstack-neutron | 10:48 | |
| *** HoloIRCUser4 has quit IRC | 10:48 | |
| miyagishi_t | hanzhang: ping | 10:48 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Add Pluggable IPAM Backend Part 2 https://review.openstack.org/183099 | 10:49 |
| *** stanzgy has quit IRC | 10:52 | |
| *** markvoelker has quit IRC | 10:54 | |
| *** topshare has quit IRC | 10:54 | |
| *** gberginc has quit IRC | 10:54 | |
| *** mutoulbj has joined #openstack-neutron | 10:54 | |
| *** pcm_ has joined #openstack-neutron | 10:54 | |
| openstackgerrit | Roman Bogorodskiy proposed openstack/neutron: mech_sriov: vf cleanup on delete https://review.openstack.org/202044 | 10:55 |
| *** yfried_ is now known as yfried|afk | 10:56 | |
| ihrachyshka | ajo, I see lots of DBDeadlock and failure of a job in service plugin | 10:57 |
| *** pc_m has quit IRC | 10:57 | |
| *** nmagnezi has quit IRC | 10:58 | |
| *** kutija has joined #openstack-neutron | 10:59 | |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Enable pluggable IPAM implementation by default https://review.openstack.org/181023 | 11:00 |
| *** kutija_ has quit IRC | 11:01 | |
| *** markvoelker has joined #openstack-neutron | 11:02 | |
| *** gberginc has joined #openstack-neutron | 11:02 | |
| *** miyagishi_t has quit IRC | 11:04 | |
| *** afazekas has quit IRC | 11:05 | |
| *** nmagnezi has joined #openstack-neutron | 11:05 | |
| *** wei-wang has quit IRC | 11:06 | |
| *** markvoelker has quit IRC | 11:07 | |
| ihrachyshka | mkolesni_, ping re service plugin | 11:11 |
| mkolesni_ | ihrachyshka: pong | 11:12 |
| *** gildub has quit IRC | 11:13 | |
| ajo | ihrachyshka, where, how? (deadlocks I mean) | 11:14 |
| ajo | mkolesni_ is going to resubmit current (afterplugin) patch with some TODO's and will start working on a cleanup | 11:15 |
| ajo | he made a nice broader plan, but we need to unblock this for now. | 11:15 |
| *** mkolesni__ has joined #openstack-neutron | 11:15 | |
| ajo | ihrachyshka, can you point me to the deadlock things? | 11:15 |
| ihrachyshka | ajo, http://logs.openstack.org/64/197564/16/check/check-tempest-dsvm-neutron-full/dbf2d90/logs/screen-q-svc.txt.gz?level=WARNING | 11:16 |
| ihrachyshka | ajo, it's in latest results from jenkins for the patch | 11:16 |
| ihrachyshka | mkolesni_, ^^ | 11:16 |
| *** yfried|afk is now known as yfried_ | 11:16 | |
| ajo | hmm | 11:16 |
| ajo | but we don't do anything about subnets | 11:16 |
| *** markvoelker has joined #openstack-neutron | 11:17 | |
| ihrachyshka | on first sight, yeah, can be irrelevant. but I wanted you guys to check. | 11:17 |
| *** nmagnezi has quit IRC | 11:17 | |
| ihrachyshka | maybe we yield somewhere in wrong place? | 11:17 |
| ajo | and that insert doesn't relate to ports/nets wither | 11:17 |
| ihrachyshka | I don't see anything IO in the patch, but who knows | 11:17 |
| *** kutija_ has joined #openstack-neutron | 11:17 | |
| ajo | I will check | 11:17 |
| *** baoli has joined #openstack-neutron | 11:17 | |
| ajo | ihrachyshka, I'm stopping the review of your versioned object rules, to cleanup the plugin and resubmit, and then I'll jump back to it | 11:18 |
| ajo | I'm finding it good | 11:18 |
| ihrachyshka | ajo, cool | 11:18 |
| *** mkolesni_ has quit IRC | 11:18 | |
| ajo | but It's a big patch, I want to make sure I fully understand everything | 11:18 |
| *** breitz has quit IRC | 11:18 | |
| *** kutija__ has joined #openstack-neutron | 11:19 | |
| mkolesni__ | ihrachyshka: i saw failures to find host.. | 11:19 |
| *** kutija has quit IRC | 11:19 | |
| mkolesni__ | ihrachyshka: i think the deadlocks are unrelated and are resulting from the move to pymysql | 11:20 |
| *** mkolesni__ is now known as mkolesni | 11:20 | |
| ihrachyshka | mkolesni, I will check master, there were some patches that we may miss re pymysql | 11:20 |
| *** breitz has joined #openstack-neutron | 11:21 | |
| *** markvoelker has quit IRC | 11:21 | |
| ihrachyshka | mkolesni, the only patch I know for pymysql is in the branch | 11:21 |
| openstackgerrit | Neil Jerram proposed openstack/neutron: Correct two spelling mistakes in Neutron devrefs https://review.openstack.org/202052 | 11:22 |
| mkolesni | ihrachyshka: amuller informed me about some patch that's adding retries to the api methods to aviod the deadlocks | 11:22 |
| *** kutija has joined #openstack-neutron | 11:22 | |
| mkolesni | ihrachyshka: but i think they will still be in the log, it will just retry the operation | 11:22 |
| *** kutija_ has quit IRC | 11:22 | |
| ihrachyshka | mkolesni, ok, can you check the test failures then? they may be other than those traces, but still relevant | 11:23 |
| *** fawadkhaliq has quit IRC | 11:23 | |
| mkolesni | ihrachyshka: yeah as i said i saw complaints about 'no host found' | 11:23 |
| mkolesni | ihrachyshka: "No hosts matched" | 11:24 |
| *** kutija__ has quit IRC | 11:24 | |
| mkolesni | ihrachyshka: for example in http://logs.openstack.org/79/201979/1/check/check-tempest-dsvm-neutron-full/a3e9837/console.html | 11:24 |
| *** markvoelker has joined #openstack-neutron | 11:24 | |
| *** cing has quit IRC | 11:26 | |
| *** MentalRay has joined #openstack-neutron | 11:26 | |
| *** openstack has joined #openstack-neutron | 11:38 | |
| *** openstackstatus has joined #openstack-neutron | 11:38 | |
| *** ChanServ sets mode: +v openstackstatus | 11:38 | |
| *** yfried_ is now known as yfried|afk | 11:39 | |
| *** markvoelker has joined #openstack-neutron | 11:39 | |
| *** aranjan has joined #openstack-neutron | 11:41 | |
| mkolesni | ihrachyshka: PEP8 recommand checking is not none | 11:41 |
| mkolesni | ihrachyshka: https://www.python.org/dev/peps/pep-0008/#programming-recommendations | 11:41 |
| *** zhhuabj_ has joined #openstack-neutron | 11:41 | |
| *** yfried|afk is now known as yfried_ | 11:41 | |
| mkolesni | ihrachyshka: if it's very important for neutron code not to be explicit, and risk some funky behavior then i dont mind using just "if x:" | 11:42 |
| mkolesni | ihrachyshka: what say you? | 11:42 |
| *** markvoelker has quit IRC | 11:44 | |
| *** fawadkhaliq has quit IRC | 11:45 | |
| ihrachyshka | mkolesni, I don't think there is reason to be explicit there. do you expect containers there? really? | 11:46 |
| mkolesni | ihrachyshka: i dont know what to expect in python, its not type safe :) | 11:46 |
| ihrachyshka | or empty string? it seems like you try to be java strict in python context. | 11:47 |
| *** dims_ has joined #openstack-neutron | 11:47 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Ensure non-overlapping cidrs in subnetpools with galera https://review.openstack.org/187985 | 11:48 |
| mkolesni | ihrachyshka: im following PEP8 guidelines | 11:48 |
| mkolesni | ihrachyshka: i didn't know it's a bad thing to do in python.. | 11:49 |
| *** vikram has quit IRC | 11:49 | |
| ihrachyshka | mkolesni, do whatever you want. if that's the only issue with the patch, I am glad to allow it as-is | 11:50 |
| *** nmagnezi has quit IRC | 11:50 | |
| mkolesni | ihrachyshka: just trying to understand whats the policy to aviod future comments on this or that | 11:51 |
| irenab | Ramanjaneya: hi | 11:51 |
| Ramanjaneya | Hi irenab | 11:51 |
| *** zhhuabj_ has quit IRC | 11:51 | |
| irenab | Ramanjaneya: I have question regarding comenct on https://review.openstack.org/#/c/199112/4/specs/liberty/qos-api-extension.rst | 11:52 |
| irenab | line 405 | 11:52 |
| openstackgerrit | yalei wang proposed openstack/neutron: pass the extention driver exception to plugin https://review.openstack.org/202061 | 11:52 |
| ihrachyshka | mkolesni, if you ask me, I think we are usually more liberal, and it seems like proper approach to me. if you want otherwise, fine with me, I didn't think there is any real concern around it from your side. | 11:52 |
| *** Avin has joined #openstack-neutron | 11:53 | |
| irenab | Ramanjaneya: Shall we remove —type? How are you managing it today? | 11:53 |
| *** MentalRay_ has joined #openstack-neutron | 11:53 | |
| *** pradipta has joined #openstack-neutron | 11:53 | |
| *** markvoelker has joined #openstack-neutron | 11:54 | |
| Ramanjaneya | Regarding: neutron qos-rule-create <policy-name-or-id> <type> bandwidth_limit | 11:54 |
| *** MentalRay has quit IRC | 11:54 | |
| ihrachyshka | HenryG, ping re migrations | 11:54 |
| irenab | Ramanjaneya: so actually <type> is bandwidth-limit in this case | 11:55 |
| Ramanjaneya | we are manage it by type | 11:55 |
| ihrachyshka | gsagie, we have objects already | 11:55 |
| irenab | Ramanjaneya: the command will look like qos-rule-create MyPolicy bandwidth_limit —etc. ? | 11:56 |
| *** vikram has joined #openstack-neutron | 11:56 | |
| *** ihrachyshka has left #openstack-neutron | 11:57 | |
| *** ihrachyshka has joined #openstack-neutron | 11:57 | |
| *** txomon|fon has quit IRC | 11:57 | |
| ajo | irenab, doing it like that, I'm not sure we can separate different parameters / help per type | 11:57 |
| openstackgerrit | Roman Bogorodskiy proposed openstack/neutron: mech_sriov: vf cleanup on delete https://review.openstack.org/202044 | 11:57 |
| ajo | I looked at neutron client, and argparser, and not sure how to do it | 11:57 |
| *** lindycoder has quit IRC | 11:57 | |
| *** yamamoto has joined #openstack-neutron | 11:58 | |
| *** markvoelker has quit IRC | 11:58 | |
| *** lindycoder has joined #openstack-neutron | 11:59 | |
| moshele | ihrachyshka: we need first the serialize/deserialize in rpc and then gsagie can do the change | 11:59 |
| irenab | ajo: initially we had —type , but vikram does not like the — meaning not mandatory parameter | 11:59 |
| ihrachyshka | moshele, are agent pieces already tested integrated? | 11:59 |
| *** topshare has joined #openstack-neutron | 11:59 | |
| vikram | irenab: "--type" cannot be optional | 12:00 |
| pcm_ | akamyshnikova: ping | 12:00 |
| *** topshare has quit IRC | 12:00 | |
| ajo | irenab, vikram , whatever we do, --help for a type needs to talk only about the specific type parameters | 12:00 |
| ajo | having 100 optional parameters is a mess | 12:00 |
| vikram | i agree with ajo here as a common cli for rule-type is confusing | 12:00 |
| moshele | ihrachyshka: it was with a mock on the server returning a dict | 12:00 |
| *** dguitarbite has quit IRC | 12:00 | |
| ajo | (100 = sum of all different rule-type parameters) | 12:00 |
| akamyshnikova | pcm_, hi | 12:01 |
| ihrachyshka | moshele, ack, good | 12:01 |
| ajo | I'd step back, and provide a separate command set for every rule type | 12:01 |
| irenab | vikram: I trying to understand your comment on the spec | 12:01 |
| ajo | as there is a separate api end | 12:01 |
| pcm_ | akamyshnikova: On https://review.openstack.org/#/c/199670/3, I created local tables for migration, but when I run the upgrade, I get an error saying that one of the tables does not have an attribute. | 12:01 |
| Ramanjaneya | irenab: we are not able to differentiate by rule-type when showing help, different cli per rule-type is good | 12:01 |
| * ajo digs his head back to service plugin | 12:01 | |
| pcm_ | addresses = session.query(IPAllocation.ip_address).filter( | 12:01 |
| pcm_ | AttributeError: 'Table' object has no attribute 'ip_address' | 12:01 |
| akamyshnikova | pcm_, OK, I'll look at that later today | 12:02 |
| pcm_ | akamyshnikova: Wondering if you have any thoughts on why it fails? | 12:02 |
| irenab | vikram: so command for rule creation will look like qos-rule-create MyPolicy bandwidth_limit —etc. | 12:02 |
| *** yamamoto has quit IRC | 12:02 | |
| vikram | irenab: Currently, we are take "--type" from qos-rule cli. My concern was if we have this optional then we must allow to take the command without "--type" parameter | 12:02 |
| pcm_ | akamyshnikova: I haven't pushed up the latest, should I? | 12:02 |
| akamyshnikova | pcm_, what attribute? | 12:03 |
| pcm_ | ip_address in the IPAllocation table | 12:03 |
| *** amuller has joined #openstack-neutron | 12:03 | |
| irenab | or like vikram: agree that without type it does not make semse. Let just clarify how cli will look like | 12:04 |
| vikram | irenab: now coming to " qos-rule-create MyPolicy bandwidth_limit —etc" the issue is how to provide help for specific type only? Actually, we add arguments irrespective of types | 12:04 |
| pcm_ | akamyshnikova: See above in the log at 08:01 (my time) | 12:04 |
| vikram | so it the user do " qos-rule-create MyPolicy bandwidth_limit —help" then arguments for all the types will be displayed and will be confusing | 12:05 |
| pcm_ | akamyshnikova: I query the IPAllocation table for ip_address. I also see it complaining about Router.id attribute too, and I have that defined in a table as well. | 12:05 |
| vikram | irenab : so it the user do " qos-rule-create MyPolicy bandwidth_limit —help" then arguments for all the types will be displayed and will be confusing | 12:05 |
| pcm_ | akamyshnikova: Let me push up the latest code. | 12:05 |
| moshele | ihrachyshka: do you have a good example how to create QosPolicy, I failed to understand how you create it here https://github.com/openstack/neutron/blob/feature/qos/neutron/tests/unit/objects/qos/test_policy.py#L43-46 | 12:05 |
| irenab | vikram: I would expect for something like this be possible : qos-rule-create my_policy —help => see different rule types | 12:05 |
| vikram | irenab: we can't control this. At the max we can have a validation in the code and restrict user | 12:06 |
| akamyshnikova | pcm_, this is strange.. | 12:06 |
| irenab | vikram: and qos-rule-create my_policy bandwidth-limit —help, see BW limit parameters | 12:06 |
| ihrachyshka | moshele, what do you mean? you just pass the class __init__ a context and kwargs that define properties for the objec | 12:06 |
| ihrachyshka | *object | 12:06 |
| vikram | irenab: yes this is possible | 12:06 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Store local side's tunnel IP for later retrieval https://review.openstack.org/199670 | 12:06 |
| pcm_ | akamyshnikova: ^^ | 12:06 |
| akamyshnikova | pcm_, yes, thanks, I'll look at it | 12:07 |
| ihrachyshka | akamyshnikova, I see garyk pushed the migration patch in gate; do we want to block it before HenryG and amotoki gets to it? | 12:07 |
| *** kbringard has joined #openstack-neutron | 12:07 | |
| vikram | irenab: AFAIK, positional help is not supported | 12:07 |
| vikram | amotoki: ping | 12:07 |
| ihrachyshka | amuller, hi. I would like to sync with you on fullstack today. | 12:07 |
| pcm_ | akamyshnikova: When in the debugger, if I display the IPAllocation object, it shows the column, but the query fails. If I remove .ip_address and query the whole table, it fails on Router.id. | 12:07 |
| vikram | ajo: ping | 12:07 |
| moshele | ihrachyshka: you example of the kwargs | 12:07 |
| ajo | hi, vikram | 12:07 |
| *** ushkalim has quit IRC | 12:08 | |
| vikram | ajo: do you have any idea about positional arg help? | 12:08 |
| amuller | ihrachyshka: me too! My last meetings for today ends in 3 hours, will you still be on? | 12:08 |
| ihrachyshka | moshele, self.db_obj there is a dict of values, like {'id': 'xxx', 'type': YYY, ...} | 12:08 |
| akamyshnikova | ihrachyshka, HenryG is still on vacations | 12:08 |
| irenab | vikram: if it will be different cli command per rule type creation, will this be possible then? | 12:08 |
| moshele | ihrachyshka: thanks | 12:08 |
| vikram | irenab: ++ | 12:08 |
| ihrachyshka | amuller, I can try. can you point me to patches I need to look at? I failed to find the start of the series | 12:08 |
| *** markvoelker has joined #openstack-neutron | 12:08 | |
| ajo | vikram, no, this is what I'm saying all the time :) | 12:08 |
| vikram | ajo:+1 | 12:08 |
| amuller | ihrachyshka: https://review.openstack.org/#/c/188221/ | 12:09 |
| irenab | ajo: yes or no? | 12:09 |
| ajo | irenab, with a separate command per rule type, it should work | 12:09 |
| *** zhhuabj_ has joined #openstack-neutron | 12:09 | |
| ajo | this is how it works for all parameters | 12:09 |
| ihrachyshka | moshele, but if you mean how to get an object on agent endpoint of rpc, then you need to call those *_primitive() methods to instantiate it | 12:09 |
| vikram | ajo: i agree, but will require server changes | 12:09 |
| amuller | ihrachyshka: Kuba pointed out that the latest PS failed the fullstack job once, I didn't have time to look in to it yet | 12:09 |
| ajo | but, I don't know (may be it's possible, but I haven't found the way) how to make later arg parsing dependent on type | 12:09 |
| irenab | ajo, vikram, I am OK with having rule creation command per rule type | 12:09 |
| amuller | ihrachyshka: so I put it in WIP, but you can definitely review it | 12:10 |
| irenab | At least to start | 12:10 |
| ajo | irenab, creation, update, delete | 12:10 |
| ihrachyshka | amuller, it's based on some outdated patch | 12:10 |
| ajo | a full set by rule type | 12:10 |
| ihrachyshka | amuller, anyway, will start from there | 12:10 |
| amuller | ihrachyshka: doesn't really matter | 12:10 |
| irenab | ajo: delete is common | 12:10 |
| irenab | ajo: only create and update | 12:10 |
| ihrachyshka | amuller, ping me when you are ready for the chat | 12:11 |
| *** dasp has joined #openstack-neutron | 12:11 | |
| ajo | hmm | 12:11 |
| ajo | we could do that too | 12:11 |
| vikram | irenab: only create and update is enough | 12:11 |
| vikram | detele, list, show could be same | 12:11 |
| ajo | less uniform, but less repeated code | 12:11 |
| irenab | vikram : +1 | 12:11 |
| vikram | *delete | 12:11 |
| ajo | vikram, may be not show | 12:11 |
| ajo | well, if you can handle different dict values and show them | 12:12 |
| ajo | independently , that's fine | 12:12 |
| ajo | probably you can | 12:12 |
| vikram | ajo: show will only take policy_id and rule_id.. right? no type is involved | 12:12 |
| akamyshnikova | pcm_, sorry I mentioned incorrect you should use Router.c.id and IPAllocation.c.ip_address | 12:12 |
| *** topshare has joined #openstack-neutron | 12:12 | |
| ajo | vikram, but in show you need to expose the attributes | 12:12 |
| ajo | max_kbps, max_burst_kbps | 12:12 |
| ajo | or the specific of every rule | 12:12 |
| *** topshare has quit IRC | 12:12 | |
| ajo | that's what I mean | 12:12 |
| vikram | ajo: that will be just string :-) | 12:12 |
| ajo | but probably you can get that from the dict | 12:12 |
| *** markvoelker has quit IRC | 12:13 | |
| akamyshnikova | pcm_, like this is done here https://github.com/openstack/neutron/blob/master/neutron/db/migration/alembic_migrations/versions/14be42f3d0a5_default_sec_group_table.py#L59 | 12:13 |
| ajo | ok vikram , go ahead, and show us :D | 12:13 |
| *** mkolesni_ has joined #openstack-neutron | 12:13 | |
| ajo | +1 | 12:13 |
| *** topshare has joined #openstack-neutron | 12:13 | |
| *** crose has joined #openstack-neutron | 12:13 | |
| vikram | ajo: i may be wrong :-) | 12:13 |
| pcm_ | akamyshnikova: How does the .c fit in? Is it specifying that the field is a column? needed for all? | 12:13 |
| irenab | vikram: ajo : when new rule type will be added, it will be easy to add new cli command to manage its creation, makes sense to me what we get here | 12:14 |
| *** aranjan has quit IRC | 12:14 | |
| *** mkolesni has quit IRC | 12:14 | |
| *** txomon|fon has joined #openstack-neutron | 12:14 | |
| akamyshnikova | pcm_, yes | 12:14 |
| Ramanjaneya | irenab: +1 | 12:14 |
| ajo | irenab, vikram , Ramanjaneya +1 | 12:14 |
| irenab | vikram: jo: as far as spec is concerned, I will remove —type and make it explicit bandwidth-limit rule create command for now | 12:14 |
| vikram | Ramanjaneya: ping.. ok with this? | 12:15 |
| pcm_ | akamyshnikova: Giving it a try... | 12:15 |
| *** johnbelamaric has joined #openstack-neutron | 12:15 | |
| Ramanjaneya | +1, per rule type seperate command ok | 12:15 |
| vikram | irenab: i think we need to update the spec for separate cli as well | 12:15 |
| ihrachyshka | akamyshnikova, that's a pity. should we wait for amotoki? | 12:16 |
| irenab | vikram: I will add some note that when new rule type will be added, new cli command should be added to handle it | 12:16 |
| vikram | irenab: As discussed, we can't use qos-rule-create anymore | 12:16 |
| akamyshnikova | ihrachyshka, it will be good, I won't expect that it will be merged so quickly, although it is OK for me. | 12:17 |
| irenab | vikram: so we want qos-bandwidth-limit-rule-create? | 12:17 |
| vikram | there will be no cli called "qos-rule-create" now | 12:17 |
| vikram | irenab: +++++++++++ | 12:17 |
| vikram | ajo:? | 12:17 |
| pcm_ | akamyshnikova: Is the .c only needed for certain fields? I'm getting an error saying 'result' object has no attribute 'c' for the line: | 12:17 |
| irenab | and later qos-dscp-marking-rule-create? | 12:17 |
| ihrachyshka | akamyshnikova, ok, if you don't have strong opinion, I let it merge and unleash the dragons :) | 12:18 |
| pcm_ | akamyshnikova: service.c.router_id == Router.c.id, | 12:18 |
| Ramanjaneya | irenab: i thik this fine | 12:18 |
| vikram | ajo: i believe this was there in our initial spec :-) | 12:18 |
| ajo | yes | 12:18 |
| ajo | :) | 12:18 |
| ajo | correct | 12:18 |
| vikram | ajo, irenab: world is too small | 12:18 |
| Ramanjaneya | irenab: please share the updated spec, so will correct neutron client changes | 12:18 |
| vikram | we are again back to the place where we satrted | 12:18 |
| irenab | Ramanjaneya: I am going to update it right now. | 12:19 |
| vikram | irenab, ajo, Ramanjaneya: thanks for the input | 12:19 |
| irenab | ajo: vikram : I must say, I hate it that we cannot make it generic one, but given the limitations, let do our best :-) | 12:20 |
| pcm_ | akamyshnikova: I see the issue. The service object, a result of the query, doesn't have that field. | 12:20 |
| vikram | irenab: we can but need to re-visit the limitation for python-neutronclient :-) | 12:20 |
| pcm_ | akamyshnikova: (or attribute, the .c) | 12:21 |
| vikram | irenab: probably we can have requirement for this | 12:21 |
| ajo | irenab: me too | 12:21 |
| irenab | vikram: I guesstrying to keep spec updated is impossible mission :-) | 12:21 |
| akamyshnikova | pcm_, c is for tables, we need to check what service expect to have | 12:21 |
| vikram | irenab: hat's off to Ramanjaneya for posting CLI patches/.... | 12:22 |
| akamyshnikova | pcm_, I'll check that later | 12:22 |
| vikram | irenab, Ramanjaneya: hat's off | 12:22 |
| mkolesni_ | ajo: i gtg, i see theres some failures on the second patch so i'll check them out later | 12:22 |
| pcm_ | akamyshnikova: Looks like there is an error in my logic in attempting to update the service object. | 12:22 |
| pcm_ | akamyshnikova: OK. I'm going to keep plugging at this to see if I can figure it out. | 12:23 |
| *** markvoelker has joined #openstack-neutron | 12:23 | |
| *** ushkalim has joined #openstack-neutron | 12:23 | |
| ajo | mkolesni_, I can try to check in the meanwhile | 12:23 |
| ajo | I will ping you via comments | 12:23 |
| mkolesni_ | ajo: i see the port that is sent doesn't have 'id' | 12:24 |
| *** topshare has quit IRC | 12:24 | |
| mkolesni_ | ajo: not sure why, need to check out what's this attrs really contains | 12:24 |
| mkolesni_ | ajo: if you find something drop me a note | 12:24 |
| mkolesni_ | ajo: cya! | 12:24 |
| *** markvoelker has quit IRC | 12:28 | |
| *** mkolesni_ has quit IRC | 12:30 | |
| gsagie | ihrachyshka: https://review.openstack.org/#/c/201583/ , addressed your comments | 12:30 |
| ihrachyshka | gsagie, that's what I look into right now | 12:30 |
| *** mikeymeitbual has quit IRC | 12:30 | |
| *** dasp has left #openstack-neutron | 12:31 | |
| ihrachyshka | gsagie, +2 | 12:31 |
| *** sudipto_ has quit IRC | 12:32 | |
| *** wei-wang has joined #openstack-neutron | 12:33 | |
| *** wznoinsk has quit IRC | 12:33 | |
| jlibosva | ihrachyshka: https://review.openstack.org/#/c/201544/3/neutron/agent/l2/extensions/qos_agent.py - about qos_driver_cls - you mean we should use stevedore there as well? Like we did with backward compatibility with adv services? | 12:34 |
| jlibosva | ihrachyshka: e.g. using https://github.com/openstack/neutron/blob/master/neutron/services/provider_configuration.py#L44 | 12:35 |
| openstackgerrit | yalei wang proposed openstack/neutron: Add support for unaddressed port https://review.openstack.org/200809 | 12:35 |
| *** edmondsw has joined #openstack-neutron | 12:35 | |
| jlibosva | ajo: ^^ :) | 12:35 |
| ihrachyshka | jlibosva, yeah, similar. we have it for everything configurable (unless we failed to add it) | 12:35 |
| jlibosva | ihrachyshka: ok, I'll move the function out of service package and reuse | 12:35 |
| ajo | yes, we have something similar already in qos too... | 12:36 |
| ihrachyshka | jlibosva, we have it in lots of places, ideally we would reuse the same code to load different drivers. but that's out of scope for qos. | 12:36 |
| ajo | for the qos "agent extension" | 12:36 |
| ihrachyshka | jlibosva++ man, if you reuse it, that's coll | 12:36 |
| ihrachyshka | *cool | 12:36 |
| *** flaviof has joined #openstack-neutron | 12:36 | |
| ajo | then that's the qos-agent-extension driver | 12:36 |
| ihrachyshka | if not, just add TODO | 12:36 |
| ajo | neutron is an onion | 12:37 |
| ihrachyshka | a pile of onions | 12:37 |
| ihrachyshka | all similar but with slight differences | 12:37 |
| ajo | the meta-onion | 12:37 |
| *** markvoelker has joined #openstack-neutron | 12:37 | |
| jlibosva | ihrachyshka: ajo just to make sure - aren't you missing loading extensions vs. loading drivers? | 12:37 |
| ajo | we should change the project name to onion | 12:37 |
| *** shardy has joined #openstack-neutron | 12:38 | |
| jlibosva | s/missing/mixing/ | 12:38 |
| ajo | jlibosva, what do you mean? | 12:38 |
| ajo | extension is loaded, here: (ajo looks) | 12:38 |
| jlibosva | ajo: IIUC agent extension is loaded by constructor. driver doesn't inherit from stevedore | 12:38 |
| ihrachyshka | jlibosva, is there really a difference in terms of communicating with stevedore? It just returns a class, right? ideally, all the communication with the library would be isolated somewhere | 12:38 |
| jlibosva | ihrachyshka: maybe I don't get that. But I think we have extension manager - inherited from NamedExtensionsManager | 12:39 |
| jlibosva | that is capable of loading extensions | 12:39 |
| shardy | Hi all, I have a question about conflict errors when deleting subnets | 12:39 |
| jlibosva | for agent driver we don't create driver manager | 12:39 |
| jlibosva | so we must explicitly create a driver and then possibly import class that is defined in driver object | 12:39 |
| shardy | if I delete all ports for a subnet, then wait until I get 404 from the API for each port, why would I get a confict error when deleting the subnet? | 12:40 |
| jlibosva | I hope that makes sense :) | 12:40 |
| shardy | I get "One or more ports have an IP allocation from this subnet" | 12:40 |
| pcm_ | akamyshnikova: ping | 12:40 |
| ihrachyshka | jlibosva, not sure I follow. I don't mind about implementation as long as in the end we don't put import paths in config files | 12:40 |
| ajo | correct | 12:40 |
| ajo | ihrachyshka +1 | 12:40 |
| shardy | Is there anything else I can do to be really sure the ports are deleted enough for subnet deletion to work? | 12:40 |
| ajo | I was about to say the same thing | 12:40 |
| *** dims_ has quit IRC | 12:41 | |
| akamyshnikova | pcm_, pong | 12:42 |
| Qlawy | can anyone tell me if GRE tunnelling is supported over IPv6 in neutron? | 12:42 |
| pcm_ | akamyshnikova: So I have the query working and get the needed information, but I'm not sure how to update the table (the result of the VPNService table query does not accept update() method). | 12:42 |
| Qlawy | When I set local_ip to v6 IP it fails with "this is not ipv4 address" | 12:42 |
| *** markvoelker has quit IRC | 12:42 | |
| jlibosva | ihrachyshka: ajo I mean if you used only extension loader than we would still need to function used in adv services for loading drivers | 12:42 |
| ajo | ihrachyshka: https://review.openstack.org/#/c/201975/3/neutron/agent/l2/agent_extensions_manager.py | 12:42 |
| moshele | jlibosva: I am with you on this :) | 12:42 |
| *** dasp has joined #openstack-neutron | 12:42 | |
| *** yatin has quit IRC | 12:42 | |
| ajo | sorry | 12:42 |
| ajo | I'm trying to concentrate in to many things at the same time | 12:43 |
| jlibosva | ok, I'll pull the function out and if we find a way to re-use it on other parts that make sense we can do it | 12:43 |
| ihrachyshka | jlibosva, you know more about it than me | 12:43 |
| pcm_ | akamyshnikova: IOW, I have the values for the new fields, but am not sure how to update the table (given it is specified locally and not using a model) | 12:43 |
| * ajo 's mental bandwidth limit reached | 12:43 | |
| jlibosva | ajo: I hope you still have headphones listening to some music and singing along | 12:43 |
| wei-wang | ;-D | 12:43 |
| openstackgerrit | Salvatore Orlando proposed openstack/python-neutronclient: NSX gateway extension: allow more transport type values https://review.openstack.org/202089 | 12:44 |
| *** afazekas has joined #openstack-neutron | 12:44 | |
| *** topshare has joined #openstack-neutron | 12:44 | |
| ajo | jlibosva, I trust you to figure out the best way to do it with stevedore, | 12:45 |
| ajo | I believe it makes sense since, otherwise if people reference that in configuration directly, and we move the driver around (out of tree, refactor, whatever), config references become broken | 12:46 |
| *** cing has joined #openstack-neutron | 12:46 | |
| akamyshnikova | pcm_, what errors do you have? | 12:47 |
| pcm_ | akamyshnikova: Would I do some form of op.execute(tablename.update()...) operation? | 12:47 |
| pcm_ | akamyshnikova: I query for all the VPN services, giving a list of 'result' objects. I then, use that to get to the router's GW IP and obtain that. | 12:48 |
| *** signed8bit has joined #openstack-neutron | 12:48 | |
| akamyshnikova | pcm_, I'm sure that op.execute() could work so | 12:48 |
| pcm_ | akamyshnikova: Then, I want to store that GW IP in the service entry. | 12:48 |
| *** trinaths has quit IRC | 12:48 | |
| *** nmagnezi has joined #openstack-neutron | 12:49 | |
| pcm_ | So, as I have it now, there is a service object (result of query). I cannot do a service.udpate() call on that, as it is not supported. | 12:49 |
| *** hanzhang has quit IRC | 12:49 | |
| pcm_ | akamyshnikova: https://review.openstack.org/#/c/199670/4/neutron_vpnaas/db/migration/alembic_migrations/versions/24f28869838b_add_fields_to_vpn_service_table.py line 72 | 12:49 |
| pcm_ | akamyshnikova: Where I could before, when using the model. | 12:50 |
| openstackgerrit | Berezovsky Irena proposed openstack/neutron-specs: Update spec to reflect current QoS API https://review.openstack.org/199112 | 12:50 |
| *** pal has joined #openstack-neutron | 12:50 | |
| *** hanzhang has joined #openstack-neutron | 12:50 | |
| *** markvoelker has joined #openstack-neutron | 12:52 | |
| akamyshnikova | pcm_, you still use the model, but now you create it here. | 12:52 |
| pcm_ | akamyshnikova: Not sure I follow. On L72 'service' is an SQLAlchemy result object and doesn't support an update() method. | 12:53 |
| *** ferest has joined #openstack-neutron | 12:54 | |
| openstackgerrit | Merged openstack/neutron: Switch to the oslo_utils.fileutils https://review.openstack.org/199550 | 12:54 |
| akamyshnikova | pcm_, SQLAlchemy query should support update() http://docs.sqlalchemy.org/en/rel_1_0/orm/query.html#sqlalchemy.orm.query.Query.update. Can you give 15 minutes that I finish some activities and will be able to check your change on my computer? | 12:55 |
| pcm_ | akamyshnikova: sure | 12:55 |
| pcm_ | n | 12:55 |
| *** markvoelker has quit IRC | 12:56 | |
| *** yamamoto has joined #openstack-neutron | 12:58 | |
| openstackgerrit | Merged openstack/neutron: [neutron-db-manage] support separate migration branches https://review.openstack.org/194198 | 12:59 |
| *** jaypipes has joined #openstack-neutron | 12:59 | |
| *** Aish has joined #openstack-neutron | 13:00 | |
| *** julim has joined #openstack-neutron | 13:00 | |
| ajo | irenab: about this: https://review.openstack.org/#/c/201979/ | 13:01 |
| ajo | we should not extend dicts in ml2 extensions driver | 13:01 |
| ajo | that's already done by the after_read callback | 13:01 |
| *** markvoelker has joined #openstack-neutron | 13:01 | |
| ajo | for all the plugins | 13:02 |
| ajo | what makes the callbacks incomplete is that we also need to update bindings/unbindings of resources to the policies | 13:02 |
| ajo | mike will make an abstraction of that which could be reused in general, and interfaced to the ml2 extension drivers | 13:02 |
| *** neeti has quit IRC | 13:02 | |
| *** yamamoto has quit IRC | 13:03 | |
| wei-wang | request a favor for reviewing this: https://review.openstack.org/#/c/172875/ :-D | 13:03 |
| ihrachyshka | ajo, I see service plugin patch failing the same way. I don't see the same for other patches, so there is a bug in the patch itself. | 13:04 |
| ajo | ihrachyshka the deadlock? | 13:05 |
| *** Aish has quit IRC | 13:05 | |
| irenab | ajo: yes | 13:05 |
| ajo | ihrachyshka, that could happen by yielding inside a transaction, right? | 13:05 |
| ihrachyshka | ajo, I think yes, but I'm not sure. we may need to talk to kevinbenton maybe. | 13:06 |
| *** dims_ has joined #openstack-neutron | 13:06 | |
| ihrachyshka | kevinbenton, btw split migrations merged | 13:08 |
| irenab | ajo: I am not sure I follow you | 13:08 |
| *** Avin has quit IRC | 13:08 | |
| *** Avin has joined #openstack-neutron | 13:09 | |
| ajo | ihrachyshka, do we call logs inside neutron objects? | 13:09 |
| *** markvoelker has quit IRC | 13:09 | |
| *** markvoelker has joined #openstack-neutron | 13:09 | |
| ihrachyshka | ajo, we should not, but that's a good idea to check. I'll do | 13:09 |
| *** signed8bit is now known as signed8bit_ZZZzz | 13:10 | |
| *** markvoelker_ has joined #openstack-neutron | 13:10 | |
| *** aranjan has joined #openstack-neutron | 13:10 | |
| *** kiran-r has joined #openstack-neutron | 13:10 | |
| *** signed8bit_ZZZzz has quit IRC | 13:10 | |
| *** Avin has quit IRC | 13:11 | |
| *** Avin has joined #openstack-neutron | 13:11 | |
| *** markvoelker has quit IRC | 13:14 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: OVS agent react to events instead of polling https://review.openstack.org/164880 | 13:15 |
| *** aranjan has quit IRC | 13:15 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: L2 agent RPC add new RPC calls https://review.openstack.org/147435 | 13:15 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Don't sync all devices when some fail https://review.openstack.org/202098 | 13:15 |
| *** Avin has quit IRC | 13:15 | |
| *** Avin has joined #openstack-neutron | 13:15 | |
| ajo | ihrachyshka, you don't seem to be calling logs | 13:15 |
| ihrachyshka | ajo, yeah, I don't see any | 13:16 |
| *** krtaylor has joined #openstack-neutron | 13:16 | |
| ajo | qos/models.py:LOG = logging.getLogger(__name__) | 13:16 |
| ajo | I see this, but I think it's not used afterwards | 13:16 |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: VMware: enable two more values in validators for gw devices https://review.openstack.org/202099 | 13:17 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Store local side's tunnel IP for later retrieval https://review.openstack.org/199670 | 13:17 |
| *** kiran-r has quit IRC | 13:18 | |
| neiljerram | Afternoon all. I have some questions about the flavor framework; is there someone who could discuss that? | 13:18 |
| openstackgerrit | Berezovsky Irena proposed openstack/neutron-specs: Update spec to reflect current QoS API https://review.openstack.org/199112 | 13:19 |
| *** dane_leblanc has joined #openstack-neutron | 13:20 | |
| *** baoli has quit IRC | 13:20 | |
| *** ferest has quit IRC | 13:23 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Ensure non-overlapping cidrs in subnetpools with galera https://review.openstack.org/187985 | 13:24 |
| *** topshare has quit IRC | 13:25 | |
| ihrachyshka | pcm_, http://lists.openstack.org/pipermail/openstack-dev/2015-July/069582.html | 13:27 |
| akamyshnikova | pcm_, you've already fixed problem as I see :) | 13:27 |
| *** shwetaap has joined #openstack-neutron | 13:27 | |
| pcm_ | akamyshnikova: Yeah, I figured it out. Thanks for helping. | 13:28 |
| akamyshnikova | pcm_, np, sorry for confusion with usage c for tables and op.execute | 13:29 |
| pcm_ | ihrachyshka: Thanks. So, I updated the original VPN migration so that it is at least mechanically correct. Will need to look that the above description to modify it for the new format. | 13:29 |
| *** shardy_ has joined #openstack-neutron | 13:29 | |
| ihrachyshka | pcm_, I planned to allocate some time to rearrange migration trees in *aas | 13:30 |
| pcm_ | akamyshnikova: np. It was all new for me, only seen it once before and didn't know how it all worked. Learned a bit. | 13:30 |
| ajo | ha! | 13:30 |
| ajo | ihar | 13:30 |
| ajo | I think I catched it | 13:30 |
| ihrachyshka | pcm_, that would go as the first patch, and then your patch reworked | 13:30 |
| ajo | ihrachyshka ^ | 13:30 |
| ihrachyshka | ajo, shoot | 13:30 |
| ajo | 2015-07-15 12:31:55.596 DEBUG neutron.callbacks.manager [req-5dab16cf-e187-4cc7-bbb3-8d26b2e132a4 None None] Notify callbacks for network, after_read _notify_loop /opt/stack/new/neutron/neutron/callbacks/manager.py:135 | 13:30 |
| ajo | 2015-07-15 12:31:55.596 DEBUG neutron.callbacks.manager [req-5dab16cf-e187-4cc7-bbb3-8d26b2e132a4 None None] Calling callback neutron.services.qos.qos_plugin.QoSPlugin._extend_network_policy_data _notify_loop /opt/stack/new/neutron/neutron/callbacks/manager.py:141 | 13:30 |
| ajo | sorry, too long for IRC :( | 13:30 |
| pcm_ | ihrachyshka: Cool. Would like to try to get the commit in soon. | 13:30 |
| ajo | it's the callback interface itself | 13:30 |
| ajo | emiting logs | 13:30 |
| ihrachyshka | pcm_, ack. I'll look at it tomorrow in the morning. | 13:30 |
| *** jprovazn has joined #openstack-neutron | 13:30 | |
| pcm_ | ihrachyshka: ok. | 13:31 |
| ihrachyshka | pcm_, I'll pick lbaas as a guinea pig | 13:31 |
| *** shardy has quit IRC | 13:31 | |
| ihrachyshka | ajo, that's bad. will we kill callbacks? | 13:31 |
| ihrachyshka | sorry, logs | 13:31 |
| pcm_ | ihrachyshka: I'm fine with doing VPN (as I want to get this change in asap) | 13:31 |
| ajo | poor callbacks | 13:31 |
| ihrachyshka | but yeah, killing callbacks should be fun too! | 13:31 |
| ajo | yeah, I'd kill logs | 13:31 |
| ihrachyshka | pcm_, oh, I meant vpn and told lb | 13:31 |
| ajo | ihrachyshka, I will update the patch to do it all at once, or shall we prefer a prepended patch? | 13:32 |
| *** zzzeek has joined #openstack-neutron | 13:32 | |
| * ajo thinks of vote conservation ;) | 13:32 | |
| ihrachyshka | ajo, prepend please | 13:32 |
| ajo | ack | 13:32 |
| pcm_ | ihrachyshka: squeal! squeal! :) | 13:32 |
| ihrachyshka | ajo, votes to the trash, it's just the right thing to split | 13:32 |
| ajo | ack | 13:33 |
| ihrachyshka | ajo, good, then I finally get to fullstack reviews | 13:33 |
| ajo | '':D | 13:33 |
| ajo | ETOOMANYTHINGS | 13:34 |
| ajo | sorry | 13:34 |
| *** dguitarbite has joined #openstack-neutron | 13:34 | |
| *** dguitarbite is now known as Guest4286 | 13:34 | |
| *** Guest4286 has quit IRC | 13:35 | |
| openstackgerrit | Merged openstack/neutron: Galera multi-writers compliant sync_allocations https://review.openstack.org/192099 | 13:35 |
| *** dbite has joined #openstack-neutron | 13:35 | |
| *** shardy_ has quit IRC | 13:35 | |
| *** dbite has quit IRC | 13:35 | |
| *** shardy has joined #openstack-neutron | 13:35 | |
| openstackgerrit | Merged openstack/neutron: Fix SR-IOV mechanism driver tests directory https://review.openstack.org/201965 | 13:36 |
| *** Piet has quit IRC | 13:38 | |
| *** signed8bit has joined #openstack-neutron | 13:41 | |
| neiljerram | Re flavors... does anyone have a handy example of how a flavor might be used for a network? I'm wondering if I should be using flavors to describe the kind of routing-based network that I've proposed at https://review.openstack.org/#/c/198439/ | 13:42 |
| *** agireud has joined #openstack-neutron | 13:42 | |
| openstackgerrit | Merged openstack/neutron: bugs: Update info about current bug czar https://review.openstack.org/201556 | 13:42 |
| *** kawa2014 has quit IRC | 13:43 | |
| *** kawa2014 has joined #openstack-neutron | 13:43 | |
| *** signed8b_ has joined #openstack-neutron | 13:43 | |
| *** arajagopal has joined #openstack-neutron | 13:44 | |
| *** cing has quit IRC | 13:44 | |
| *** signed8bit has quit IRC | 13:45 | |
| ajo | ihrachyshka: //review.openstack.org/202114 | 13:45 |
| ajo | kevinbenton, salv-orlando , do you know if LOG.<...> can yield? | 13:46 |
| ajo | https://review.openstack.org/202114 | 13:46 |
| ajo | I guess write's are more unlikely to IO block, I guess they can (full buffers, etc), but.. not sure how eventlet handles it | 13:46 |
| *** prad has joined #openstack-neutron | 13:47 | |
| pcm_ | mestery: ping | 13:47 |
| salv-orlando | ajo: it does | 13:47 |
| ajo | ha! | 13:48 |
| ajo | that's it then | 13:48 |
| mestery | pcm_: pong | 13:48 |
| salv-orlando | it won't be the first time a lock wait condition disappears or is introduced by moving a log statement. I think I and other spent a while discussing this with comstud a few years ago | 13:48 |
| pcm_ | mestery: Morning! Can you review https://review.openstack.org/#/c/200592/ (adding devstack plugin for VPN)? | 13:48 |
| *** jistr is now known as jistr|mtg | 13:49 | |
| ajo | salv-orlando, we found some DBDeadLocks probably quite related to those LOG events, but not fully sure yet, we found them more when extending resources via AFTER_READ callback | 13:49 |
| ajo | like two times in a row, which made us suspicious | 13:49 |
| mestery | pcm_: sure! | 13:49 |
| pcm_ | mestery: thanks! | 13:49 |
| salv-orlando | ajo, for this analysis to be correct, the deadlock reason should be a "lock wait timeout" error, and you should see a 50 sec interruption in the logs | 13:50 |
| ajo | salv-orlando, let me check | 13:50 |
| ajo | salv-orlando: http://logs.openstack.org/64/197564/16/check/check-tempest-dsvm-neutron-full/dbf2d90/logs/screen-q-svc.txt.gz?level=WARNING#_2015-07-15_09_12_35_430 | 13:50 |
| ajo | looks like that | 13:50 |
| ihrachyshka | LOG. can definitely yield | 13:52 |
| ajo | lock wait timeout, and a ~55 between the LOG and the deadlock | 13:53 |
| ihrachyshka | ajo, yeah, that's exactly the behaviour | 13:53 |
| ajo | yup | 13:53 |
| ajo | thanks salv-orlando for the confirmation | 13:53 |
| ajo | ihrachyshka, may be I should change the comments and remove the TODO | 13:54 |
| ihrachyshka | ajo, now, the question is: are we using rpc callbacks api in some unintended way? maybe it's not guaranteed to be yield safe. | 13:54 |
| ajo | to totally kill the logs without TODO | 13:54 |
| ajo | ihrachyshka, that's a good reason for a separate API | 13:54 |
| ajo | since we're now the only users/providers of AFTER_READ and that will go away for the new API | 13:54 |
| ihrachyshka | ajo, yeah, we can kill them assuming we use api correctly | 13:54 |
| ajo | the new API will have to guarantee that | 13:54 |
| ajo | let's leave the TODO for revisit then | 13:55 |
| ihrachyshka | ok, let's just kill that then. update the patch | 13:55 |
| ihrachyshka | well, ok, I don't mind | 13:55 |
| ajo | lol :) | 13:55 |
| *** ushkalim has quit IRC | 13:55 | |
| ihrachyshka | as long as we get that bloody plugin in today | 13:55 |
| ihrachyshka | it's long overdue | 13:55 |
| ajo | yeessss | 13:55 |
| ihrachyshka | and I'm pissed off and grumble because of that | 13:55 |
| openstackgerrit | Cyril Roelandt proposed openstack/neutron: Python 3: do not use types.ClassType https://review.openstack.org/200553 | 13:56 |
| Ramanjaneya | \join openstack-meeting-3 | 13:56 |
| wei-wang | jckasper: hi | 13:57 |
| *** LukeHinds has joined #openstack-neutron | 13:57 | |
| *** lukasa has quit IRC | 13:58 | |
| *** jckasper has quit IRC | 13:59 | |
| ihrachyshka | ajo, you're my hero of the day for getting that LOG fixed | 13:59 |
| *** jckasper has joined #openstack-neutron | 13:59 | |
| ajo | you're my hero everyday ;) | 13:59 |
| *** pmcfadden has joined #openstack-neutron | 13:59 | |
| ajo | I really hope it's that | 14:00 |
| ajo | it really looks like | 14:00 |
| *** lajos-katona1 has left #openstack-neutron | 14:00 | |
| *** Piet has joined #openstack-neutron | 14:00 | |
| ihrachyshka | mestery, we need a +2 for https://review.openstack.org/#/c/202114/ to unblock another patch (service plugin) in qos | 14:00 |
| ihrachyshka | ajo, I will be surprised if it's not | 14:00 |
| ihrachyshka | ajo, the pattern is so clear | 14:00 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Ensure non-overlapping cidrs in subnetpools with galera https://review.openstack.org/187985 | 14:00 |
| ihrachyshka | ajo, meeting? | 14:01 |
| ajo | ihrachyshka, yikes!, yes | 14:01 |
| mestery | ihrachyshka: Done! | 14:01 |
| ajo | thanks mestery | 14:01 |
| mestery | ihrachyshka: Sounds nasty, the logging yield issue :( | 14:01 |
| ihrachyshka | ajo, so speaking of api usage, can it be some option to make us call it from outside txn? | 14:02 |
| ihrachyshka | (I don't know exact trace that leads us to that LOG, so bear with me) | 14:02 |
| gsagie | pc_m: for VPNaaS you enable q-vpn both in the controller node and in the network-node ? there is no agent specific service? | 14:03 |
| *** Avin has quit IRC | 14:03 | |
| gsagie | pcm_ | 14:03 |
| ajo | jlibosva, do you want #openstack-meeting-3 ? | 14:03 |
| *** dboik has joined #openstack-neutron | 14:03 | |
| ihrachyshka | ajo, he was probably not notified about that meeting. he just left the room. | 14:03 |
| *** njohnston has joined #openstack-neutron | 14:03 | |
| ajo | ah, np | 14:04 |
| ajo | I understand | 14:04 |
| *** kutija_ has joined #openstack-neutron | 14:04 | |
| pcm_ | gsagie: q-vpn service would be enabled on the network node. | 14:05 |
| gsagie | pcm_: ok thanks | 14:06 |
| *** annegentle has joined #openstack-neutron | 14:06 | |
| *** jckasper has quit IRC | 14:07 | |
| pcm_ | gsagie: VPN is a plugin (part of q-svc), service driver (called by neutron server), and device driver (part of agent). | 14:07 |
| *** ushkalim has joined #openstack-neutron | 14:07 | |
| *** kutija has quit IRC | 14:07 | |
| pcm_ | gsagie: It overrides the L3 agent, if enabled (IOW a VPN agent IS-A L3 agent). | 14:07 |
| *** changzhi has joined #openstack-neutron | 14:07 | |
| gsagie | pcm_ : thanks for the explanation! | 14:08 |
| pcm_ | gsagie: sure | 14:08 |
| changzhi | amuller: hi | 14:10 |
| jlibosva | ajo: joinign | 14:10 |
| *** aranjan has joined #openstack-neutron | 14:11 | |
| *** obondarev_ has joined #openstack-neutron | 14:11 | |
| amuller | changzhi: heya | 14:13 |
| changzhi | amuller: please review https://review.openstack.org/#/c/199588/ :-) | 14:14 |
| amuller | changzhi: indeed, it's on my list | 14:14 |
| changzhi | amuller: Thx :-O | 14:15 |
| *** shardy_ has joined #openstack-neutron | 14:16 | |
| *** shardy has quit IRC | 14:17 | |
| *** yamahata has joined #openstack-neutron | 14:18 | |
| *** shardy_ has quit IRC | 14:21 | |
| *** shardy has joined #openstack-neutron | 14:22 | |
| *** sdake has joined #openstack-neutron | 14:23 | |
| *** fzdarsky_ has quit IRC | 14:23 | |
| *** arajagopal has quit IRC | 14:23 | |
| *** topshare has joined #openstack-neutron | 14:25 | |
| *** topshare has quit IRC | 14:25 | |
| *** SridharG has left #openstack-neutron | 14:25 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart https://review.openstack.org/182920 | 14:27 |
| *** fzdarsky has joined #openstack-neutron | 14:28 | |
| *** markvoelker has joined #openstack-neutron | 14:28 | |
| *** absubram has joined #openstack-neutron | 14:29 | |
| *** john-davidge has joined #openstack-neutron | 14:30 | |
| *** topshare has joined #openstack-neutron | 14:30 | |
| *** topshare has quit IRC | 14:31 | |
| *** absubram has quit IRC | 14:31 | |
| *** markvoel_ has joined #openstack-neutron | 14:31 | |
| *** markvoelker has quit IRC | 14:31 | |
| *** abehl has quit IRC | 14:31 | |
| *** topshare has joined #openstack-neutron | 14:31 | |
| *** absubram has joined #openstack-neutron | 14:31 | |
| *** markvoelker_ has quit IRC | 14:32 | |
| *** eckelcu has joined #openstack-neutron | 14:33 | |
| *** jistr|mtg is now known as jistr | 14:33 | |
| *** markvoel_ has quit IRC | 14:33 | |
| *** pal has quit IRC | 14:37 | |
| *** minwang2 has joined #openstack-neutron | 14:38 | |
| *** minwang2 has quit IRC | 14:38 | |
| *** mlavalle has joined #openstack-neutron | 14:39 | |
| *** rushil has joined #openstack-neutron | 14:40 | |
| *** arajagopal has joined #openstack-neutron | 14:40 | |
| *** mickeys has joined #openstack-neutron | 14:41 | |
| *** signed8b_ is now known as signed8bit_ZZZzz | 14:42 | |
| *** signed8bit_ZZZzz is now known as signed8b_ | 14:42 | |
| *** mutoulbj has quit IRC | 14:45 | |
| *** aranjan has quit IRC | 14:45 | |
| *** mutoulbj has joined #openstack-neutron | 14:47 | |
| *** Murali has quit IRC | 14:47 | |
| *** absubram has quit IRC | 14:48 | |
| *** jasondotstar has joined #openstack-neutron | 14:49 | |
| *** yfried_ is now known as yfried|afk | 14:49 | |
| *** sbalukoff has quit IRC | 14:49 | |
| *** jobewan has joined #openstack-neutron | 14:51 | |
| *** markvoelker has joined #openstack-neutron | 14:53 | |
| *** yfried|afk is now known as yfried_ | 14:54 | |
| *** markvoelker_ has joined #openstack-neutron | 14:54 | |
| *** pal has joined #openstack-neutron | 14:55 | |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 14:55 |
| *** topshare has quit IRC | 14:56 | |
| *** markvoelker has quit IRC | 14:58 | |
| *** ushkalim has quit IRC | 14:58 | |
| *** nmagnezi has quit IRC | 14:59 | |
| *** jckasper has joined #openstack-neutron | 14:59 | |
| *** topshare has joined #openstack-neutron | 14:59 | |
| ajo | moshele | 14:59 |
| ajo | let me check the object/dicts thing | 14:59 |
| moshele | ajo: ok | 15:00 |
| ajo | moshele, yes | 15:00 |
| ajo | for now | 15:00 |
| ajo | we're using VersionedObjectsDictCompat | 15:00 |
| ajo | that means | 15:00 |
| ajo | you can do | 15:00 |
| ajo | policy['rules'] | 15:00 |
| ajo | to get the rules | 15:00 |
| ajo | for example | 15:00 |
| ajo | or to get the description | 15:00 |
| ajo | we may want to change that later | 15:00 |
| ajo | and remove the dictcompat | 15:00 |
| *** yfried__ has joined #openstack-neutron | 15:01 | |
| ajo | obj_to_primitive | 15:01 |
| ajo | and obj_from_privimitve | 15:01 |
| ajo | primitive | 15:01 |
| *** yamahata has quit IRC | 15:02 | |
| *** changzhi has quit IRC | 15:02 | |
| *** yfried_ has quit IRC | 15:02 | |
| *** mlavalle has quit IRC | 15:02 | |
| johnbelamaric | carl_baldwin, haleyb: I think the last ipam patch is ready for core reviews: #link https://review.openstack.org/#/c/183099/ | 15:02 |
| *** yamahata has joined #openstack-neutron | 15:03 | |
| *** abregman has quit IRC | 15:03 | |
| *** shardy_ has joined #openstack-neutron | 15:03 | |
| openstackgerrit | Merged openstack/neutron: Move pylint dep from tox.ini to test-requirements https://review.openstack.org/197227 | 15:03 |
| moshele | ajo: do we have ovs test for end to end? | 15:03 |
| moshele | ajo: I mean from service plugin to ovs agnet | 15:04 |
| ajo | not yet, we depend on the merge of the rules in policies patch from ihar | 15:04 |
| ajo | to start | 15:04 |
| *** shardy has quit IRC | 15:04 | |
| ajo | and also finishing rule creation/adding to policy in plugin. | 15:05 |
| ajo | but we're striving to merge the initial plugin yet. | 15:05 |
| moshele | ajo: but this should be working by the end of the month? | 15:06 |
| ajo | moshele: ideally, yes | 15:06 |
| ajo | or by the start of the month | 15:06 |
| jlibosva | ajo: do you think it would make sense to put all the tasks, todos and what is going on to the etherpad? | 15:06 |
| jlibosva | ajo: and who is working on what. I think it would make overview easier | 15:07 |
| ajo | jlibosva, we had one like this, but I don't find time to update it... | 15:07 |
| jlibosva | and I'm not fan of etherpads :) | 15:07 |
| *** ronis_ has quit IRC | 15:07 | |
| ajo | I generally do just search for branch:feature/qos | 15:07 |
| ajo | and topic:neutron-qos | 15:07 |
| ajo | in gerrit | 15:07 |
| ajo | which is more auto-updated than etherpads | 15:07 |
| ajo | people forget to update etherpad, and I don't have time ':D | 15:07 |
| *** regXboi has joined #openstack-neutron | 15:07 | |
| jlibosva | ajo: I see | 15:07 |
| ajo | jlibosva, still, there are tasks not in review | 15:07 |
| ajo | probably a guide for what's missing, would be good | 15:08 |
| ajo | I'll put one with the gaps on friday. | 15:08 |
| ajo | so you have something to look at to fill the gaps while I ran away :/ | 15:08 |
| *** shardy_ has quit IRC | 15:08 | |
| *** sudipto has joined #openstack-neutron | 15:09 | |
| haleyb | johnbelamaric: i'll take a look today | 15:09 |
| johnbelamaric | haleyb: thanks Brian | 15:09 |
| *** gampel has quit IRC | 15:09 | |
| *** shardy has joined #openstack-neutron | 15:09 | |
| johnbelamaric | it's a relatively large one | 15:09 |
| johnbelamaric | but logically all goes together | 15:10 |
| *** yamahata has quit IRC | 15:12 | |
| *** flaviof has quit IRC | 15:12 | |
| *** yamahata has joined #openstack-neutron | 15:12 | |
| openstackgerrit | Eric Larese proposed openstack/neutron: Update allocation_pool validation https://review.openstack.org/202150 | 15:13 |
| *** fzdarsky has quit IRC | 15:14 | |
| *** itzikb_ has quit IRC | 15:16 | |
| *** padkrish has joined #openstack-neutron | 15:16 | |
| *** rm_work|away is now known as rm_work | 15:16 | |
| *** padkrish has quit IRC | 15:16 | |
| *** padkrish has joined #openstack-neutron | 15:17 | |
| *** mlavalle has joined #openstack-neutron | 15:17 | |
| *** crc32 has joined #openstack-neutron | 15:17 | |
| *** carl_baldwin has joined #openstack-neutron | 15:17 | |
| *** ParsectiX has joined #openstack-neutron | 15:18 | |
| ajo | ihrachyshka: https://review.openstack.org/#/c/200608/ | 15:19 |
| ajo | jlibosva comments make sense | 15:19 |
| ajo | otherwise it looks good to me | 15:19 |
| *** topshare has quit IRC | 15:19 | |
| *** ekuris has quit IRC | 15:19 | |
| ihrachyshka | ajo, yeah, metaclasses. I will need to learn that crap, finally | 15:20 |
| *** topshare has joined #openstack-neutron | 15:20 | |
| ihrachyshka | I will update later | 15:20 |
| ajo | ihrachyshka I proposed a simpler solution | 15:20 |
| ajo | may be it works | 15:20 |
| ihrachyshka | maybe we may go with TODO | 15:20 |
| ajo | ihrachyshka, that works for me | 15:20 |
| ajo | check my simpler solution idea in comments | 15:21 |
| *** mkolesni_ has joined #openstack-neutron | 15:21 | |
| ajo | and may be that + TODO if metaclass is somehow cleaner | 15:21 |
| *** wwriverrat has quit IRC | 15:21 | |
| ihrachyshka | ajo, moving what? | 15:21 |
| ajo | ihrachyshka the for loop to a class method | 15:22 |
| ajo | or to a static method where you receive the rule field references | 15:22 |
| ihrachyshka | ajo, who will call it? | 15:22 |
| ajo | then at class level you call it | 15:22 |
| mkolesni_ | ajo: hey | 15:22 |
| ihrachyshka | no I cannot call class method from inside class definition I think | 15:22 |
| ajo | and it won't polute your class with new attributes, because those will be scopped to the function | 15:22 |
| mkolesni_ | ajo: did u have time to look at it? | 15:22 |
| ajo | mkolesni_, yes | 15:22 |
| ajo | attr receives only the changed fields | 15:22 |
| *** dannywilson has joined #openstack-neutron | 15:22 | |
| ajo | mkolesni_, I didn't have time to fix it though.. | 15:23 |
| mkolesni_ | ajo: ah ok ill check it out | 15:23 |
| ajo | thanks mkolesni_ | 15:23 |
| mkolesni_ | ajo: btw how did it even get called, do we have test for qos? | 15:23 |
| *** zhenguo has quit IRC | 15:23 | |
| ajo | I used neutronclient from review | 15:23 |
| *** aix has quit IRC | 15:23 | |
| ajo | let me point you to it | 15:23 |
| openstackgerrit | Eric Larese proposed openstack/neutron: Update allocation_pool validation https://review.openstack.org/202150 | 15:24 |
| ajo | it's working for port modification/disociation | 15:24 |
| *** dannywilson has quit IRC | 15:24 | |
| ajo | just make sure to install service_plugins = qos | 15:24 |
| ajo | locally | 15:24 |
| *** moshele has quit IRC | 15:24 | |
| *** dannywilson has joined #openstack-neutron | 15:24 | |
| openstackgerrit | Sean M. Collins proposed openstack/neutron: lb: avoid doing nova VIF work plumbing tap to qbr https://review.openstack.org/193485 | 15:24 |
| *** agireud has quit IRC | 15:24 | |
| *** srilatha has joined #openstack-neutron | 15:24 | |
| ajo | mkolesni_: https://review.openstack.org/#/c/189655/ | 15:25 |
| ajo | this one may work | 15:25 |
| ajo | then use port-create port-update | 15:25 |
| ajo | with the modified things | 15:25 |
| openstackgerrit | Merged openstack/neutron: Correct two spelling mistakes in Neutron devrefs https://review.openstack.org/202052 | 15:25 |
| ajo | also create profile may work | 15:25 |
| mkolesni_ | ajo: ok | 15:25 |
| ajo | mkolesni_, if you see explosions in other parts, ping me | 15:27 |
| ihrachyshka | ajo, pep... | 15:27 |
| *** flaviof has joined #openstack-neutron | 15:27 | |
| mkolesni_ | ajo: ok | 15:27 |
| ajo | mestery, ihrachyshka : https://review.openstack.org/#/c/202114/ | 15:27 |
| ajo | could you please? | 15:27 |
| ajo | now pep8 is passing | 15:27 |
| ajo | I was quite confident because it was all comments, but it seems pep8 will check the TODO(xxx) | 15:28 |
| ajo | shame on me... | 15:28 |
| ihrachyshka | that check for pep is so 'useful' | 15:28 |
| mkolesni_ | ajo: doh it's of course a logic fail, when u send request to create port ud never send an ID | 15:29 |
| mkolesni_ | ajo: shame on me 2 | 15:29 |
| ihrachyshka | shame party! | 15:29 |
| *** minwang2 has joined #openstack-neutron | 15:30 | |
| mkolesni_ | woo | 15:30 |
| ajo | mkolesni_, I think I also saw that on update | 15:30 |
| ajo | although I'm not sure now | 15:30 |
| ajo | it was while debugging | 15:31 |
| mkolesni_ | ajo: anyhow it's weird, why does it even reach the qos plugin..? | 15:32 |
| *** sdake has quit IRC | 15:32 | |
| mkolesni_ | ajo: it would only reach if the qos_profile_id is set | 15:32 |
| mestery | ajo ihrachyshka: +2/+A | 15:32 |
| mestery | second time is the charm ;) | 15:32 |
| *** sdake has joined #openstack-neutron | 15:32 | |
| *** dmellado has joined #openstack-neutron | 15:32 | |
| mkolesni_ | ajo: im sending the slightly updated version that should fix the functional tests | 15:33 |
| ajo | ihrachyshka, you're right you cannot call a method while you're defining the class, because you won't find your own class name... | 15:33 |
| *** yfried__ is now known as yfried|afk | 15:33 | |
| ajo | ihrachyshka: http://ur1.ca/n4r9v | 15:33 |
| ajo | you could do that ^ | 15:33 |
| ihrachyshka | ajo, I need metaclasses, I agree with jlibosva | 15:33 |
| ajo | ihrachyshka ack | 15:33 |
| ajo | show me that magic | 15:33 |
| ajo | I will be looking | 15:33 |
| ihrachyshka | we may postpone it with TODO, but that's the goal | 15:33 |
| ajo | ok | 15:33 |
| * mkolesni_ crosses fingers that it will pass | 15:34 | |
| ihrachyshka | and I finally learn that stuff | 15:34 |
| *** mkolesni_ is now known as mkolesni | 15:34 | |
| mkolesni | ajo: do you think neutron.services.qos is the proper place to put the adapter class? | 15:35 |
| mkolesni | ajo: the one that's responsible to process the core plugin's requests to update/extend the resources | 15:36 |
| jlibosva | ihrachyshka: woot, metaclasses :) | 15:36 |
| ihrachyshka | jlibosva, we may want to postpone it to after the patch is merged, but yeah, woot | 15:36 |
| ajo | mkolesni, I think for now that's correct | 15:37 |
| ajo | then we can show the world the broader plan, and may be move to a different module | 15:37 |
| mkolesni | ajo: ok cool, going with that for start | 15:37 |
| mkolesni | :) | 15:37 |
| ajo | :) | 15:37 |
| ajo | thanks mkolesni | 15:37 |
| * mkolesni starts working on the master plan exemplar | 15:37 | |
| mkolesni | :) | 15:37 |
| *** yfried|afk has quit IRC | 15:37 | |
| ajo | mkolesni, we need to merge the service plugin first | 15:37 |
| *** jlibosva has quit IRC | 15:37 | |
| ajo | if the fix is up, then go! ;) | 15:38 |
| mkolesni | ajo: sure, i'm doing it post those 2 patches already up | 15:38 |
| ajo | wow | 15:38 |
| mkolesni | ajo: yeah i sent the fix, jenkins will take care of testing it :) | 15:38 |
| mkolesni | ajo: i have confidence in it, if it found the first problem it can surely test if it's solved :) | 15:38 |
| *** crc32 is now known as crc32_soda_break | 15:39 | |
| ihrachyshka | mkolesni, I hope at least pep won't fail ;) | 15:39 |
| ajo | awesome :) | 15:39 |
| ajo | yeah | 15:39 |
| *** ig0r__ has quit IRC | 15:39 | |
| ajo | mkolesni: there's people not doing local pep8, can you believe it | 15:39 |
| ajo | ? | 15:39 |
| *** padkrish has quit IRC | 15:39 | |
| mkolesni | ihrachyshka: u insult me dear sir, i already ran pep8 locally :) | 15:39 |
| ihrachyshka | :) | 15:40 |
| mkolesni | the audacity of those people.. | 15:40 |
| * ajo runs and hides | 15:40 | |
| ihrachyshka | ajo did not! | 15:40 |
| ihrachyshka | :P | 15:40 |
| mkolesni | heh | 15:40 |
| wei-wang | :-D | 15:40 |
| mkolesni | we need to install amuller's gerrit hook.. | 15:40 |
| wei-wang | that's cool | 15:40 |
| *** ig0r_ has joined #openstack-neutron | 15:40 | |
| ajo | ihrachyshka I should probably rebase the qos plugin on top of the no-log-callbacks | 15:40 |
| mkolesni | if i werent as lazy i'd do it a long time ago | 15:40 |
| ajo | and also mkolesni's one that goes aftewards | 15:40 |
| *** ajmiller has joined #openstack-neutron | 15:40 | |
| ihrachyshka | ajo, yes, sure, you MUST! :) | 15:40 |
| ajo | ok, doing | 15:40 |
| ajo | mkolesni, I'm going to rebase your patch to cut out a bug we have | 15:41 |
| ihrachyshka | as I said, we should get plugin in today! no excuses! | 15:41 |
| ihrachyshka | :) | 15:41 |
| *** kawa2014 has quit IRC | 15:41 | |
| ajo | yes sir! | 15:41 |
| ajo | o/ | 15:41 |
| *** aranjan has joined #openstack-neutron | 15:41 | |
| ajo | o\ | 15:42 |
| *** nlahouti has joined #openstack-neutron | 15:42 | |
| *** sridhar_ram has joined #openstack-neutron | 15:43 | |
| ajo | mkolesni: beware, I rebased this: https://review.openstack.org/#/c/201979/ | 15:44 |
| ajo | if you're doing the prepended patch, you may want to rebase on top of PS4 | 15:45 |
| wei-wang | the git hook should hook at pre-commit or post-commit? | 15:45 |
| *** rm_work is now known as rm_work|away | 15:45 | |
| mkolesni | ajo: ok sure, i'll keep that in mind | 15:45 |
| ajo | ok, | 15:45 |
| ajo | I need to make a pause, I'll be back later at night | 15:45 |
| ajo | drop me an email if there's somewhere I can help | 15:45 |
| ajo | I'll be back at now +5h or so | 15:46 |
| *** padkrish has joined #openstack-neutron | 15:46 | |
| ihrachyshka | ajo, +2 for service plugin? | 15:46 |
| ajo | mkolesni, ihrachyshka ^ | 15:46 |
| *** vikram has quit IRC | 15:46 | |
| ajo | ihrachyshka, link? | 15:46 |
| mkolesni | ajo, ihrachyshka - yeah the first one is harmless | 15:46 |
| ihrachyshka | ajo, ah sorry, it has votes | 15:46 |
| ajo | yes I only did +A | 15:46 |
| ihrachyshka | I thought rebase will reset them | 15:46 |
| ajo | it was a rebase | 15:46 |
| ihrachyshka | ok. you're free ajo ;) | 15:47 |
| ihrachyshka | for next 5h | 15:47 |
| ajo | I have family work to do ;) | 15:47 |
| ihrachyshka | ;) | 15:47 |
| ihrachyshka | enjoy | 15:47 |
| ajo | not exactly free, but... QoS-free | 15:47 |
| ajo | thanks | 15:47 |
| *** ajmiller has quit IRC | 15:52 | |
| *** ajmiller has joined #openstack-neutron | 15:52 | |
| *** Aish has joined #openstack-neutron | 15:53 | |
| *** JonB has joined #openstack-neutron | 15:54 | |
| *** wwriverrat has joined #openstack-neutron | 15:54 | |
| *** kiran-r has joined #openstack-neutron | 15:54 | |
| *** patrickeast has joined #openstack-neutron | 15:56 | |
| *** armax has joined #openstack-neutron | 15:56 | |
| *** kawa2014 has joined #openstack-neutron | 15:56 | |
| *** kiran-r has quit IRC | 15:56 | |
| *** scheuran has joined #openstack-neutron | 15:57 | |
| *** lazy_prince has quit IRC | 15:57 | |
| *** alex_klimov has quit IRC | 15:58 | |
| *** jaypipes has quit IRC | 15:58 | |
| *** rm_work|away is now known as rm_work | 16:00 | |
| *** logan2 has quit IRC | 16:00 | |
| *** sbelous_ has quit IRC | 16:00 | |
| *** shashank_hegde_ has joined #openstack-neutron | 16:01 | |
| *** topshare has quit IRC | 16:01 | |
| *** liudong has quit IRC | 16:01 | |
| *** madhu_ak_ has joined #openstack-neutron | 16:01 | |
| *** agireud has joined #openstack-neutron | 16:01 | |
| *** salv-orl_ has joined #openstack-neutron | 16:01 | |
| *** amotoki has joined #openstack-neutron | 16:02 | |
| *** sbelous has joined #openstack-neutron | 16:02 | |
| *** juliancash has joined #openstack-neutron | 16:02 | |
| *** TrevorV has joined #openstack-neutron | 16:02 | |
| *** dlundquist has joined #openstack-neutron | 16:03 | |
| juliancash | The etherpad for the LBaaS MidCycle meetup at HP office: https://etherpad.openstack.org/p/LBaaS-FWaaS-VPNaaS_Summer_Midcycle_meetup | 16:03 |
| *** Sukhdev has joined #openstack-neutron | 16:03 | |
| *** ajmiller has quit IRC | 16:03 | |
| *** ygbo has quit IRC | 16:04 | |
| *** padkrish has quit IRC | 16:04 | |
| *** johnsom has joined #openstack-neutron | 16:04 | |
| *** salv-orlando has quit IRC | 16:05 | |
| *** madhu_ak_ is now known as madhu_ak | 16:05 | |
| *** shihanzhang has joined #openstack-neutron | 16:07 | |
| juliancash | http://auroralightportraits.com/v/hpcloud/ | 16:08 |
| *** sdake has quit IRC | 16:08 | |
| mkolesni | ajo: u know once we plug it properly to ml2, this can be reverted: https://review.openstack.org/#/c/202114/2 | 16:08 |
| *** yfried|afk has joined #openstack-neutron | 16:09 | |
| *** sudipto has quit IRC | 16:09 | |
| *** kiran-r has joined #openstack-neutron | 16:09 | |
| *** evgenyf has quit IRC | 16:09 | |
| *** hanzhang has quit IRC | 16:10 | |
| *** jlanoux has quit IRC | 16:11 | |
| *** sdake has joined #openstack-neutron | 16:11 | |
| *** rm_work is now known as rm_work|away | 16:11 | |
| *** rm_work|away is now known as rm_work | 16:13 | |
| *** yfried has joined #openstack-neutron | 16:13 | |
| *** _cjones_ has joined #openstack-neutron | 16:14 | |
| *** yfried|afk has quit IRC | 16:14 | |
| *** signed8b_ is now known as signed8bit_ZZZzz | 16:14 | |
| *** ildikov has quit IRC | 16:14 | |
| *** crose has quit IRC | 16:15 | |
| *** krtaylor has quit IRC | 16:15 | |
| *** aranjan has quit IRC | 16:16 | |
| *** padkrish has joined #openstack-neutron | 16:16 | |
| *** rotbeard has quit IRC | 16:19 | |
| *** zhhuabj_ has quit IRC | 16:20 | |
| *** bradjones has quit IRC | 16:20 | |
| *** ZZelle has quit IRC | 16:20 | |
| *** verdurin has quit IRC | 16:20 | |
| *** stefanb has quit IRC | 16:20 | |
| *** crc32_soda_break is now known as crc32 | 16:20 | |
| *** bradjones has joined #openstack-neutron | 16:20 | |
| *** bradjones has quit IRC | 16:20 | |
| *** bradjones has joined #openstack-neutron | 16:20 | |
| *** zhhuabj_ has joined #openstack-neutron | 16:20 | |
| *** ZZelle has joined #openstack-neutron | 16:20 | |
| *** stefanb has joined #openstack-neutron | 16:20 | |
| *** verdurin has joined #openstack-neutron | 16:20 | |
| *** pal has quit IRC | 16:23 | |
| scheuran | mestery: Kyle, regarding the discussion if the macvtap support should land in it's own repo or somewhere else: can I put this on next weeks neutron meeting agenda? Is that the right place to get things forward? | 16:25 |
| scheuran | mestery: refering to this one: https://review.openstack.org/195907 | 16:25 |
| *** vivek-ebay has joined #openstack-neutron | 16:25 | |
| *** yfried has quit IRC | 16:27 | |
| *** afazekas has quit IRC | 16:27 | |
| *** krtaylor has joined #openstack-neutron | 16:28 | |
| openstackgerrit | Merged openstack/neutron: Improve check_migration command error message https://review.openstack.org/199075 | 16:28 |
| *** mlavalle has quit IRC | 16:29 | |
| *** amuller is now known as amuller_adfk | 16:29 | |
| *** amuller_adfk is now known as amuller_afk | 16:29 | |
| *** mlavalle has joined #openstack-neutron | 16:29 | |
| *** padkrish has quit IRC | 16:31 | |
| *** scheuran has quit IRC | 16:32 | |
| *** crc32 has quit IRC | 16:32 | |
| *** shashank_hegde_ has quit IRC | 16:33 | |
| *** scheuran has joined #openstack-neutron | 16:34 | |
| *** shashank_hegde_ has joined #openstack-neutron | 16:34 | |
| *** scheuran has quit IRC | 16:35 | |
| *** minwang2 has quit IRC | 16:35 | |
| *** ijw has joined #openstack-neutron | 16:35 | |
| *** crc32 has joined #openstack-neutron | 16:36 | |
| *** minwang2 has joined #openstack-neutron | 16:36 | |
| *** jckasper has quit IRC | 16:36 | |
| *** ronis_ has joined #openstack-neutron | 16:37 | |
| *** sripriya has joined #openstack-neutron | 16:38 | |
| *** markvoelker_ has quit IRC | 16:40 | |
| pavel_bondar | carl_baldwin: Hi | 16:41 |
| *** markvoelker has joined #openstack-neutron | 16:42 | |
| *** sudipto has joined #openstack-neutron | 16:42 | |
| pavel_bondar | carl_baldwin: only one review left for IPAM, https://review.openstack.org/#/c/183099/ | 16:42 |
| *** gsagie_ has joined #openstack-neutron | 16:42 | |
| carl_baldwin | pavel_bondar: I will get to it today. :) | 16:43 |
| pavel_bondar | carl_baldwin: nice, thanks | 16:43 |
| pavel_bondar | carl_baldwin: if you think it is still big I can decompose it more, just ping me in this case | 16:44 |
| *** spligak has joined #openstack-neutron | 16:45 | |
| *** s3wong has joined #openstack-neutron | 16:45 | |
| *** padkrish has joined #openstack-neutron | 16:46 | |
| carl_baldwin | pavel_bondar: I’ll see how the review goes. I think the nature of this patch set allows for a little bit larger change. At first glance, it looks like it is just adding to an already refactored code base and not mixing that with various refactoring efforts and moving things around. | 16:47 |
| carl_baldwin | pavel_bondar: I personally have the most trouble when there is just too much of the existing code base changing in various ways to fit the new stuff in. | 16:48 |
| sc68cal | pavel_bondar: congrats :) | 16:49 |
| *** jistr has quit IRC | 16:50 | |
| *** minwang2 has quit IRC | 16:50 | |
| *** amotoki has quit IRC | 16:50 | |
| *** sbalukoff has joined #openstack-neutron | 16:51 | |
| *** minwang2 has joined #openstack-neutron | 16:52 | |
| *** hichihara has joined #openstack-neutron | 16:52 | |
| *** padkrish has quit IRC | 16:53 | |
| *** ajmiller has joined #openstack-neutron | 16:53 | |
| pavel_bondar | sc68cal: thanks:) | 16:53 |
| *** ajmiller_ has joined #openstack-neutron | 16:54 | |
| pavel_bondar | carl_baldwin: ack | 16:54 |
| *** ijw has quit IRC | 16:54 | |
| *** ijw has joined #openstack-neutron | 16:55 | |
| *** ajmiller__ has joined #openstack-neutron | 16:55 | |
| *** yamahata has quit IRC | 16:57 | |
| *** ptoohill is now known as pothole | 16:57 | |
| *** ajmiller__ has quit IRC | 16:57 | |
| *** ajmiller__ has joined #openstack-neutron | 16:57 | |
| *** pcm_ is now known as pc_m | 16:58 | |
| *** yamahata has joined #openstack-neutron | 16:58 | |
| *** pal has joined #openstack-neutron | 16:58 | |
| *** patrickeast has quit IRC | 16:59 | |
| *** mutoulbj has quit IRC | 16:59 | |
| *** ajmiller_ has quit IRC | 16:59 | |
| *** ajmiller has quit IRC | 16:59 | |
| *** kbringard has quit IRC | 17:00 | |
| *** lori is now known as lori|away | 17:01 | |
| *** minwang2 has quit IRC | 17:01 | |
| *** Vikram has joined #openstack-neutron | 17:02 | |
| *** kiran-r has quit IRC | 17:02 | |
| *** Vikram is now known as Vikram_ | 17:03 | |
| pc_m | ihrachyshka: hi | 17:04 |
| *** minwang2 has joined #openstack-neutron | 17:06 | |
| *** juliancash has quit IRC | 17:07 | |
| *** Vikram_ has quit IRC | 17:07 | |
| *** hichihara has quit IRC | 17:07 | |
| *** ihrachyshka has quit IRC | 17:07 | |
| *** crose has joined #openstack-neutron | 17:09 | |
| *** yamahata has quit IRC | 17:09 | |
| *** ajmiller__ is now known as ajmiller | 17:10 | |
| *** anilvenkata has left #openstack-neutron | 17:10 | |
| *** logan2 has joined #openstack-neutron | 17:10 | |
| *** sballe has joined #openstack-neutron | 17:10 | |
| *** aranjan has joined #openstack-neutron | 17:12 | |
| *** s3wong has quit IRC | 17:12 | |
| *** moshele has joined #openstack-neutron | 17:14 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Temporarily disable check_migration https://review.openstack.org/202197 | 17:14 |
| pc_m | HenryG: ping | 17:14 |
| *** ronis_ has quit IRC | 17:15 | |
| *** shwetaap1 has joined #openstack-neutron | 17:15 | |
| *** amuller_afk is now known as amuller | 17:16 | |
| *** shwetaap has quit IRC | 17:16 | |
| carl_baldwin | pavel_bondar: I guess I never grew out of needing to be spoon fed. ;) | 17:17 |
| *** kawa2014 has quit IRC | 17:17 | |
| Sukhdev | vivek: Ping | 17:17 |
| *** banix has joined #openstack-neutron | 17:18 | |
| *** patrickeast has joined #openstack-neutron | 17:18 | |
| *** Aish has quit IRC | 17:18 | |
| pc_m | akamyshnikova: ping? | 17:20 |
| *** sripriya has quit IRC | 17:21 | |
| *** shivrao has joined #openstack-neutron | 17:21 | |
| *** sripriya has joined #openstack-neutron | 17:21 | |
| *** agireud has quit IRC | 17:22 | |
| *** moshele has quit IRC | 17:22 | |
| *** padkrish has joined #openstack-neutron | 17:25 | |
| *** ajmiller_ has joined #openstack-neutron | 17:25 | |
| *** ajmiller__ has joined #openstack-neutron | 17:26 | |
| *** TrevorV has quit IRC | 17:26 | |
| *** ajmiller has quit IRC | 17:26 | |
| *** TrevorV has joined #openstack-neutron | 17:26 | |
| *** minwang2 has quit IRC | 17:28 | |
| *** sthillma has joined #openstack-neutron | 17:28 | |
| *** mkolesni has quit IRC | 17:28 | |
| *** shashank_hegde_ has quit IRC | 17:29 | |
| *** achanda has joined #openstack-neutron | 17:29 | |
| *** ajmiller_ has quit IRC | 17:29 | |
| *** padkrish has quit IRC | 17:30 | |
| *** padkrish has joined #openstack-neutron | 17:31 | |
| wei-wang | Hi, since test_db_base_plugin_v2 is really to large, I supposed to split it to "test_db_base_plugin_v2" and "test_db_base_plugin_resources_v2", any suggestions? | 17:32 |
| *** gsagie_ has quit IRC | 17:32 | |
| *** mgarza_ has joined #openstack-neutron | 17:32 | |
| *** john-davidge has quit IRC | 17:32 | |
| *** padkrish_ has joined #openstack-neutron | 17:32 | |
| *** yamahata has joined #openstack-neutron | 17:33 | |
| wei-wang | and the latter one covers tests to ports, nets and subnets | 17:33 |
| *** padkrish has quit IRC | 17:33 | |
| wei-wang | amuller: ping | 17:33 |
| amuller | wei-wang: pong | 17:34 |
| wei-wang | amuller: ping for propose above :-D | 17:34 |
| wei-wang | amuller: do you think it is necessary | 17:34 |
| *** dkehn has quit IRC | 17:34 | |
| *** jaypipes has joined #openstack-neutron | 17:35 | |
| *** gampel has joined #openstack-neutron | 17:35 | |
| *** dkehn has joined #openstack-neutron | 17:36 | |
| *** shashank_hegde_ has joined #openstack-neutron | 17:37 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Allow external services to define their providers https://review.openstack.org/202207 | 17:38 |
| *** achanda has quit IRC | 17:39 | |
| *** hitalia has joined #openstack-neutron | 17:39 | |
| amuller | wei-wang: looking | 17:39 |
| wei-wang | amuller: thx! | 17:40 |
| madhu_ak | hi, I am looking for Ihar Hrachyshka in this channel. can somebody help me with his nick name ? | 17:40 |
| amuller | madhu_ak: he's offline | 17:40 |
| *** ivar-lazzaro has joined #openstack-neutron | 17:40 | |
| amuller | madhu_ak: but his nick is ihrachyshka | 17:41 |
| amuller | madhu_ak: he'll be online in european hours tomorrow | 17:41 |
| wei-wang | iharmadhu_ak: you can find here: https://launchpad.net/~ihar-hrachyshka | 17:41 |
| *** aranjan has quit IRC | 17:41 | |
| madhu_ak | amuller. aah, thanks for your info. anyone around can help with db migrations? I had some questions and posted email on ML | 17:41 |
| amuller | madhu_ak: you're better off waiting for him until tomorrow IMO | 17:42 |
| amuller | so you don't get second hand information | 17:42 |
| openstackgerrit | Merged openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/201837 | 17:42 |
| amuller | madhu_ak: but search for the patch by him that was recently merged and figure out what it's about from the commit message | 17:42 |
| madhu_ak | hmm, sounds good amuller. thanks much! :) | 17:42 |
| amuller | that's a possibility | 17:42 |
| *** hitalia has quit IRC | 17:42 | |
| *** agireud has joined #openstack-neutron | 17:43 | |
| *** pradipta has quit IRC | 17:43 | |
| wei-wang | madhu_ak: you can read this patch first :-D https://review.openstack.org/194198 | 17:43 |
| amuller | wei-wang: What did you want to put in the first file? | 17:43 |
| *** ivar-laz_ has joined #openstack-neutron | 17:44 | |
| madhu_ak | wei-wang: looking :) thanks | 17:44 |
| wei-wang | amuller: eh, basic test which excludes the core resources we defined in plugin | 17:44 |
| amuller | wei-wang: for example? | 17:46 |
| wei-wang | amuller: like helpers, HTTP responders | 17:46 |
| amuller | wei-wang: isn't that like a 5% 95% split? | 17:46 |
| *** ivar-lazzaro has quit IRC | 17:47 | |
| wei-wang | amuller: >_< , yes... | 17:47 |
| wei-wang | amuller: what about split different resources to different file? | 17:47 |
| amuller | wei-wang: then we don't stand to gain much by that. BTW a practical issue is that we have a check that runs as part of tox -e pep8 that every test file correctly correlates correctly to a file of the same path in the actual code tree | 17:47 |
| *** pritesh has joined #openstack-neutron | 17:47 | |
| wei-wang | amuller: yes | 17:48 |
| amuller | wei-wang: so because so much crap is in db_base_plugin_v2 and that's defined in neutron/db/X, basically anything that tests that has to be in tests/neutron/db/X | 17:48 |
| *** achanda has joined #openstack-neutron | 17:48 | |
| amuller | the paths have to match up | 17:49 |
| amuller | so you can't split up difference resources in separate files unless you also split out the actual DB class by resource | 17:49 |
| amuller | and put it in separate files | 17:49 |
| *** ajmiller__ is now known as ajmiller | 17:49 | |
| wei-wang | you are right | 17:50 |
| *** hitalia has joined #openstack-neutron | 17:50 | |
| *** sripriya has quit IRC | 17:50 | |
| *** salv-orl_ has quit IRC | 17:50 | |
| *** crose has quit IRC | 17:51 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Stop reading service providers from neutron.conf https://review.openstack.org/202212 | 17:51 |
| pc_m | mestery: ping | 17:52 |
| *** dlundquist has quit IRC | 17:54 | |
| *** dlundquist has joined #openstack-neutron | 17:54 | |
| *** sridhar_ram1 has joined #openstack-neutron | 17:54 | |
| *** sridhar_ram has quit IRC | 17:55 | |
| *** Swami has joined #openstack-neutron | 17:58 | |
| *** mutoulbj has joined #openstack-neutron | 17:59 | |
| madhu_ak | amuller, wei-wang: okay, thanks for link to the patch. got it what it is doing | 18:00 |
| *** nlahouti has quit IRC | 18:00 | |
| *** padkrish_ has quit IRC | 18:00 | |
| pc_m | carl_baldwin: dougwig: mestery: Looks like the online-schema-migration changes for neutron are breaking the (old) check-migration test used in VPN for PEP8. I created https://review.openstack.org/#/c/202197/ to disable the check. Can we get this pushed through quickly? It is log-jamming VPN commits/merges. | 18:01 |
| *** agireud has quit IRC | 18:01 | |
| *** mickeys has quit IRC | 18:01 | |
| carl_baldwin | pc_m: +2 | 18:02 |
| pc_m | carl_baldwin: thanks! | 18:02 |
| *** juliancash has joined #openstack-neutron | 18:02 | |
| *** Kennan has quit IRC | 18:02 | |
| mestery | pc_m: done | 18:03 |
| *** nlahouti has joined #openstack-neutron | 18:03 | |
| pc_m | mestery: Thanks! | 18:03 |
| *** nlahouti has quit IRC | 18:03 | |
| *** Sukhdev has quit IRC | 18:03 | |
| *** padkrish has joined #openstack-neutron | 18:03 | |
| *** Kennan has joined #openstack-neutron | 18:03 | |
| *** signed8bit_ZZZzz is now known as signed8b_ | 18:04 | |
| *** signed8b_ is now known as signed8bit_ZZZzz | 18:04 | |
| juliancash | For the Seattle MidCycle, the info on what we're doing for lunch + evening events for each day has been added to the top of the etherpad: https://etherpad.openstack.org/p/LBaaS-FWaaS-VPNaaS_Summer_Midcycle_meetup | 18:04 |
| mestery | juliancash: Hey man! Long time no see, hope all is well, cool to see you pop in channel :) | 18:06 |
| *** nlahouti has joined #openstack-neutron | 18:06 | |
| pc_m | mestery: Is the gate having issues (that commit is failing grenade, tempest, and functional tests)? | 18:06 |
| mestery | pc_m: It could be, yes | 18:06 |
| *** thurloat_isgone is now known as thurloat | 18:06 | |
| *** annegentle has quit IRC | 18:06 | |
| anteaya | yes something something configdriver something | 18:06 |
| anteaya | that's what I got from a quick skim | 18:07 |
| anteaya | config drive | 18:07 |
| *** mickeys has joined #openstack-neutron | 18:07 | |
| *** mutoulbj has quit IRC | 18:07 | |
| *** jbell8 has joined #openstack-neutron | 18:08 | |
| juliancash | Thanks grand mestery. :-) Life is good! Life highlights... My daughter is 6 months old and has 2 cute little teeth. I just finished replacing the pillars that hold up the back of my house. | 18:12 |
| juliancash | And it's fun to be here with so many fab OpenStack folks in Seattle. | 18:12 |
| *** pmcfadden has quit IRC | 18:12 | |
| mestery | juliancash: Awesome indeed! Congrats on the daughter, my son (fourth) is 9 months old but only has one tooth ;) | 18:12 |
| juliancash | Ha. Power to the one tooth! And congrads to you also. | 18:14 |
| *** jwarendt_ has joined #openstack-neutron | 18:16 | |
| sc68cal | jwarendt_: https://etherpad.openstack.org/p/fwaas-api-evolution-spec | 18:17 |
| *** jckasper has joined #openstack-neutron | 18:18 | |
| *** pmcfadden has joined #openstack-neutron | 18:18 | |
| Swami | carl_baldwin: ping | 18:18 |
| *** gberginc has quit IRC | 18:20 | |
| carl_baldwin | Swami: pong | 18:21 |
| Swami | carl_baldwin: I have confirmed that the "FloatingIPJSON" tests are failing with DVR from yesterday noon | 18:21 |
| Swami | carl_baldwin: I will look into it and see what got introduced in tempest/neutron that breaks those two tests. | 18:22 |
| carl_baldwin | Just in time to get the other problem fixed finally. :( | 18:22 |
| Swami | carl_baldwin: yes carl we are going on circles. | 18:22 |
| carl_baldwin | Swami: Thanks, Swami. | 18:22 |
| *** boris-42 has joined #openstack-neutron | 18:23 | |
| *** ajmiller_ has joined #openstack-neutron | 18:25 | |
| *** ajmiller_ has quit IRC | 18:25 | |
| *** vishwanathj has joined #openstack-neutron | 18:25 | |
| *** TrevorV has quit IRC | 18:27 | |
| *** sridhar_ram1 is now known as sridhar_ram | 18:27 | |
| *** akamyshnikova_ has joined #openstack-neutron | 18:28 | |
| *** ivar-laz_ has quit IRC | 18:29 | |
| *** ivar-lazzaro has joined #openstack-neutron | 18:29 | |
| *** jprovazn has quit IRC | 18:30 | |
| *** pritesh_ has joined #openstack-neutron | 18:30 | |
| *** ivar-lazzaro has quit IRC | 18:31 | |
| *** moha_hunt has joined #openstack-neutron | 18:32 | |
| *** ivar-lazzaro has joined #openstack-neutron | 18:32 | |
| *** dlundquist has quit IRC | 18:33 | |
| *** dlundquist has joined #openstack-neutron | 18:34 | |
| *** rpothier has joined #openstack-neutron | 18:34 | |
| *** pritesh has quit IRC | 18:34 | |
| *** shashank_hegde_ has quit IRC | 18:34 | |
| *** moha_hunt has quit IRC | 18:34 | |
| *** annegentle has joined #openstack-neutron | 18:35 | |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Create dvr base class and stop passing around snat_ports https://review.openstack.org/200294 | 18:36 |
| *** eckelcu has quit IRC | 18:37 | |
| *** jwy has joined #openstack-neutron | 18:37 | |
| *** aranjan has joined #openstack-neutron | 18:38 | |
| *** aranjan has quit IRC | 18:43 | |
| *** agireud has joined #openstack-neutron | 18:43 | |
| *** achanda has quit IRC | 18:43 | |
| openstackgerrit | Ivar Lazzaro proposed openstack/neutron: Decompose Apic ML2 mechanism driver https://review.openstack.org/193365 | 18:44 |
| *** agireud has quit IRC | 18:48 | |
| *** minwang2 has joined #openstack-neutron | 18:48 | |
| *** tidwellr has quit IRC | 18:49 | |
| *** ParsectiX has quit IRC | 18:50 | |
| *** redbeard has joined #openstack-neutron | 18:52 | |
| *** pal has quit IRC | 18:53 | |
| *** TrevorV has joined #openstack-neutron | 18:56 | |
| *** flwang has quit IRC | 18:58 | |
| *** markvoelker has quit IRC | 18:58 | |
| *** ZZelle_ has joined #openstack-neutron | 18:58 | |
| *** shwetaap has joined #openstack-neutron | 19:00 | |
| *** mlavalle has quit IRC | 19:00 | |
| *** dlundquist has quit IRC | 19:00 | |
| *** DaveJ__ has quit IRC | 19:01 | |
| *** shwetaap1 has quit IRC | 19:03 | |
| *** minwang2 has quit IRC | 19:04 | |
| *** sripriya has joined #openstack-neutron | 19:04 | |
| *** sudipto has quit IRC | 19:05 | |
| *** shashank_hegde_ has joined #openstack-neutron | 19:05 | |
| *** crose has joined #openstack-neutron | 19:06 | |
| *** Longgeek has quit IRC | 19:08 | |
| *** mutoulbj has joined #openstack-neutron | 19:08 | |
| *** fifieldt_ has joined #openstack-neutron | 19:09 | |
| *** mgarza_ has quit IRC | 19:11 | |
| *** tongli has joined #openstack-neutron | 19:11 | |
| *** mgarza has joined #openstack-neutron | 19:11 | |
| *** achanda has joined #openstack-neutron | 19:11 | |
| *** fitoduarte has joined #openstack-neutron | 19:11 | |
| *** spandhe has joined #openstack-neutron | 19:12 | |
| *** thurloat is now known as thurloat_isgone | 19:12 | |
| *** scheuran has joined #openstack-neutron | 19:13 | |
| *** fifieldt has quit IRC | 19:13 | |
| *** hakimo has quit IRC | 19:13 | |
| *** TrevorV has quit IRC | 19:13 | |
| *** mutoulbj has quit IRC | 19:14 | |
| *** hakimo has joined #openstack-neutron | 19:14 | |
| spandhe | kevinbenton: hey, yt? | 19:15 |
| spandhe | kevinbenton: regarding bug https://bugs.launchpad.net/neutron/+bug/1470612 , do you have some time to discuss? | 19:15 |
| openstack | Launchpad bug 1470612 in neutron "neutron-dhcp-agent does not recover known ports cache after restart" [High,Confirmed] - Assigned to Shraddha Pandhe (shraddha-pandhe) | 19:15 |
| *** yfried has joined #openstack-neutron | 19:15 | |
| *** yfried is now known as yfried|afk | 19:15 | |
| *** asadoughi has joined #openstack-neutron | 19:16 | |
| *** srilatha has quit IRC | 19:17 | |
| *** pritesh_ has quit IRC | 19:19 | |
| *** prad has quit IRC | 19:19 | |
| *** pritesh has joined #openstack-neutron | 19:20 | |
| *** nlahouti has quit IRC | 19:20 | |
| *** hitalia has quit IRC | 19:21 | |
| *** padkrish has quit IRC | 19:21 | |
| *** padkrish has joined #openstack-neutron | 19:22 | |
| *** prad has joined #openstack-neutron | 19:22 | |
| *** padkrish_ has joined #openstack-neutron | 19:24 | |
| *** padkrish has quit IRC | 19:24 | |
| *** nlahouti has joined #openstack-neutron | 19:24 | |
| *** ijw has quit IRC | 19:26 | |
| *** sridhar_ram1 has joined #openstack-neutron | 19:27 | |
| *** mlavalle has joined #openstack-neutron | 19:27 | |
| *** padkrish_ has quit IRC | 19:28 | |
| *** prad has quit IRC | 19:28 | |
| *** clawft has joined #openstack-neutron | 19:29 | |
| *** sridhar_ram has quit IRC | 19:30 | |
| *** jwarendt_ has quit IRC | 19:34 | |
| *** scheuran has quit IRC | 19:35 | |
| *** tflynn has joined #openstack-neutron | 19:36 | |
| *** moha_hunt has joined #openstack-neutron | 19:36 | |
| *** prad has joined #openstack-neutron | 19:40 | |
| *** sdake_ has joined #openstack-neutron | 19:41 | |
| openstackgerrit | Cyril Roelandt proposed openstack/neutron: Python 3: do not use types.ClassType https://review.openstack.org/200553 | 19:41 |
| openstackgerrit | Merged openstack/neutron: Fix update_port_postcommit and port not found with DVR https://review.openstack.org/196908 | 19:42 |
| openstackgerrit | Merged openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/200680 | 19:43 |
| *** markvoelker has joined #openstack-neutron | 19:43 | |
| *** mickeys has quit IRC | 19:45 | |
| *** sdake has quit IRC | 19:45 | |
| *** pritesh has quit IRC | 19:45 | |
| *** markvoelker has quit IRC | 19:45 | |
| *** markvoelker has joined #openstack-neutron | 19:46 | |
| *** madhu_ak has quit IRC | 19:46 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/201803 | 19:46 |
| *** tidwellr has joined #openstack-neutron | 19:47 | |
| *** dave-mccowan has joined #openstack-neutron | 19:48 | |
| *** eckelcu has joined #openstack-neutron | 19:49 | |
| *** mgarza has quit IRC | 19:50 | |
| *** mgarza has joined #openstack-neutron | 19:50 | |
| *** ihrachyshka has joined #openstack-neutron | 19:50 | |
| *** ijw has joined #openstack-neutron | 19:51 | |
| *** ParsectiX has joined #openstack-neutron | 19:51 | |
| *** y_sawai has joined #openstack-neutron | 19:51 | |
| *** salv-orlando has joined #openstack-neutron | 19:51 | |
| ihrachyshka | ajo, hey man. still around? | 19:52 |
| *** sdake_ has quit IRC | 19:52 | |
| *** srilatha has joined #openstack-neutron | 19:53 | |
| *** annegentle has quit IRC | 19:53 | |
| *** eckelcu has quit IRC | 19:53 | |
| *** annegentle has joined #openstack-neutron | 19:53 | |
| *** asadoughi has quit IRC | 19:54 | |
| *** eckelcu has joined #openstack-neutron | 19:55 | |
| *** dave-mccowan has quit IRC | 19:55 | |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Use only the lower 16 bits of iptables mark for marking https://review.openstack.org/201727 | 19:56 |
| *** salv-orlando has quit IRC | 19:57 | |
| *** dave-mccowan has joined #openstack-neutron | 19:58 | |
| *** sdake has joined #openstack-neutron | 20:00 | |
| *** padkrish has joined #openstack-neutron | 20:00 | |
| *** mgarza has quit IRC | 20:01 | |
| *** mgarza has joined #openstack-neutron | 20:02 | |
| *** crose has quit IRC | 20:04 | |
| *** hitalia has joined #openstack-neutron | 20:04 | |
| *** madhu_ak has joined #openstack-neutron | 20:05 | |
| *** minwang2 has joined #openstack-neutron | 20:05 | |
| *** Longgeek has joined #openstack-neutron | 20:08 | |
| *** shwetaap has quit IRC | 20:08 | |
| *** mgarza has quit IRC | 20:10 | |
| *** mgarza has joined #openstack-neutron | 20:10 | |
| *** abregman has joined #openstack-neutron | 20:11 | |
| *** mutoulbj has joined #openstack-neutron | 20:12 | |
| *** shwetaap has joined #openstack-neutron | 20:12 | |
| *** dlundquist has joined #openstack-neutron | 20:13 | |
| *** Longgeek has quit IRC | 20:13 | |
| *** redbeard has quit IRC | 20:14 | |
| *** shashank_hegde_ has quit IRC | 20:15 | |
| *** akamyshnikova_ has quit IRC | 20:17 | |
| *** annegentle has quit IRC | 20:18 | |
| *** pmcfadden has quit IRC | 20:19 | |
| *** fzdarsky has joined #openstack-neutron | 20:20 | |
| *** crose has joined #openstack-neutron | 20:21 | |
| openstackgerrit | Lynn Li proposed openstack/neutron: Use 1st SNAT DVR host name for FIP assoc w/unbound https://review.openstack.org/179439 | 20:22 |
| openstackgerrit | Lynn Li proposed openstack/neutron: Add predefined DVR device owner for FIP w/unbound https://review.openstack.org/181599 | 20:22 |
| *** vivek-ebay has quit IRC | 20:22 | |
| *** padkrish has quit IRC | 20:24 | |
| *** y_sawai has quit IRC | 20:24 | |
| *** y_sawai has joined #openstack-neutron | 20:25 | |
| *** shashank_hegde_ has joined #openstack-neutron | 20:25 | |
| *** padkrish has joined #openstack-neutron | 20:25 | |
| *** dave-mccowan has quit IRC | 20:26 | |
| *** padkrish has quit IRC | 20:26 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Stop reading service providers from neutron.conf https://review.openstack.org/202212 | 20:26 |
| *** padkrish has joined #openstack-neutron | 20:27 | |
| *** TrevorV has joined #openstack-neutron | 20:27 | |
| *** dave-mccowan has joined #openstack-neutron | 20:28 | |
| *** ijw has quit IRC | 20:28 | |
| *** mutoulbj has quit IRC | 20:28 | |
| *** ijw has joined #openstack-neutron | 20:29 | |
| *** pritesh has joined #openstack-neutron | 20:30 | |
| *** dlundquist has quit IRC | 20:31 | |
| *** minwang2 has quit IRC | 20:32 | |
| *** jwy has quit IRC | 20:34 | |
| openstackgerrit | Lynn Li proposed openstack/neutron: Use 1st SNAT DVR host name for FIP assoc w/unbound https://review.openstack.org/179439 | 20:35 |
| *** minwang2 has joined #openstack-neutron | 20:35 | |
| *** annegentle has joined #openstack-neutron | 20:36 | |
| *** aranjan has joined #openstack-neutron | 20:36 | |
| *** jasondotstar has quit IRC | 20:37 | |
| *** dane_leblanc has quit IRC | 20:38 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Fixes Floatingip_agent_gw_port delete failure with DVR https://review.openstack.org/202274 | 20:39 |
| *** jwy has joined #openstack-neutron | 20:40 | |
| *** krtaylor has quit IRC | 20:40 | |
| Swami | carl_baldwin: I found out the problem with the DVR tests failing in tempest. DVR is calling internal private functions and so ML2Plugin throws an error. | 20:40 |
| sc68cal | markmcclain: https://etherpad.openstack.org/p/fwaas-api-evolution-spec | 20:40 |
| Swami | carl_baldwin: this is for delete_port | 20:40 |
| Swami | carl_baldwin: I posted a patch. Can you take a look at it. #link https://review.openstack.org/202274 | 20:41 |
| haleyb | Swami: i think someone posted a patch for that earlier, let me look | 20:41 |
| carl_baldwin | Swami: Great. Looking... | 20:41 |
| *** vivek-ebay has joined #openstack-neutron | 20:41 | |
| Swami | haleyb: ok, I did not notice. | 20:42 |
| haleyb | https://review.openstack.org/201907 | 20:42 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/201803 | 20:42 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/202284 | 20:42 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/200682 | 20:43 |
| *** vivek-ebay has quit IRC | 20:43 | |
| carl_baldwin | Swami: By chance, do you know which change merged to Neutron to start the failing? | 20:43 |
| haleyb | carl_baldwin: https://review.openstack.org/#/c/153236/ caused the failure | 20:43 |
| haleyb | ipam | 20:43 |
| Swami | carl_baldwin: I could not figure out | 20:43 |
| Swami | haleyb: I don't see any related patch to the fix. | 20:44 |
| *** y_sawai has quit IRC | 20:45 | |
| haleyb | Swami: it got lost above, https://review.openstack.org/#/c/201907/ | 20:45 |
| *** padkrish has quit IRC | 20:45 | |
| *** ihrachyshka has quit IRC | 20:46 | |
| *** padkrish has joined #openstack-neutron | 20:46 | |
| *** jasondotstar has joined #openstack-neutron | 20:46 | |
| Swami | haleyb: thanks that helps. | 20:46 |
| *** dlundquist has joined #openstack-neutron | 20:46 | |
| *** mlavalle has quit IRC | 20:47 | |
| *** dboik has quit IRC | 20:47 | |
| *** mlavalle has joined #openstack-neutron | 20:47 | |
| *** dboik has joined #openstack-neutron | 20:47 | |
| *** dboik has quit IRC | 20:48 | |
| *** mgarza has quit IRC | 20:48 | |
| *** mlavalle has quit IRC | 20:48 | |
| *** dboik has joined #openstack-neutron | 20:49 | |
| *** mgarza has joined #openstack-neutron | 20:49 | |
| johnbelamaric | logically swami's patch makes more sense to me | 20:49 |
| *** dboik has quit IRC | 20:49 | |
| johnbelamaric | carl_baldwin, swami, haleyb ^^ | 20:50 |
| Swami | johnbelamaric: yes I was little bit confused why should I use "ipam" object to delete a port | 20:50 |
| carl_baldwin | johnbelamaric: I think I agree. Looking... | 20:51 |
| johnbelamaric | swami: some day we would hope to separate it out better (but it's really a major rework) so that the IPAM piece is not done within the transaction. So, when (if) we get there, the other patch will probably break | 20:51 |
| *** agireud has joined #openstack-neutron | 20:52 | |
| *** padkrish has quit IRC | 20:52 | |
| *** krtaylor has joined #openstack-neutron | 20:53 | |
| *** pritesh_ has joined #openstack-neutron | 20:53 | |
| *** padkrish has joined #openstack-neutron | 20:53 | |
| *** dboik has joined #openstack-neutron | 20:53 | |
| *** padkrish has quit IRC | 20:54 | |
| *** padkrish has joined #openstack-neutron | 20:54 | |
| *** pritesh has quit IRC | 20:56 | |
| *** pc_m is now known as pcm_m_AWAY | 20:56 | |
| *** abhiraut has joined #openstack-neutron | 20:57 | |
| *** ParsectiX has quit IRC | 20:59 | |
| *** ildikov has joined #openstack-neutron | 21:00 | |
| *** mickeys has joined #openstack-neutron | 21:01 | |
| *** mgarza has quit IRC | 21:01 | |
| *** dave-mccowan has quit IRC | 21:01 | |
| *** mgarza has joined #openstack-neutron | 21:02 | |
| *** minwang2 has quit IRC | 21:03 | |
| *** thorst has quit IRC | 21:03 | |
| *** mickeys has quit IRC | 21:03 | |
| *** mickeys has joined #openstack-neutron | 21:04 | |
| *** Piet has quit IRC | 21:04 | |
| *** rpothier has quit IRC | 21:06 | |
| carl_baldwin | johnbelamaric: Swami: haleyb: Seems this broke it: https://review.openstack.org/#/c/153236/97/neutron/db/ipam_backend_mixin.py | 21:06 |
| carl_baldwin | Ignoring the issues around why delete_port was moved in to ipam and whether it belongs there, it seems 201907 is the logical and easiest fix for the regression. | 21:07 |
| *** dims_ has quit IRC | 21:07 | |
| Swami | carl_baldwin: thanks got it. | 21:07 |
| * carl_baldwin still thinking out loud... | 21:07 | |
| carl_baldwin | Swami: johnbelamaric: 202274 might be the right way to go but it will take some more care in the review because it doesn’t make such a trivial swap. | 21:09 |
| *** jasondotstar has quit IRC | 21:09 | |
| *** mlavalle has joined #openstack-neutron | 21:09 | |
| carl_baldwin | haleyb: ^ | 21:09 |
| Swami | carl_baldwin: there are other places in the code 'l3_dvr_db.py" file that we call "_core_plugin.delete_port". | 21:09 |
| Swami | carl_baldwin: this fix would be odd in the file. Do you think that any call to "delete_port" should be called through "ipam". | 21:10 |
| *** julim has quit IRC | 21:10 | |
| *** shashank_hegde_ has quit IRC | 21:10 | |
| *** jbell8 has quit IRC | 21:11 | |
| *** tongli has quit IRC | 21:11 | |
| *** jschwarz has joined #openstack-neutron | 21:12 | |
| jschwarz | amuller, ping | 21:12 |
| amuller | jschwarz: hello sir | 21:12 |
| amuller | jschwarz: How are you? | 21:12 |
| jschwarz | amuller, good. a bit tipsy, but that's a story for some other time | 21:12 |
| *** mgarza has quit IRC | 21:13 | |
| *** mgarza_ has joined #openstack-neutron | 21:13 | |
| carl_baldwin | Swami: I think we should separate that question from fixing this regression. 201907 acknowledges that DVR was already calling the method out of IPAM and simply makes a change that 153236 should’ve made in the first place. I think it should merge as is to fix the regression and then yours should be rebased after it to house the discussion of what DVR really should be calling. | 21:13 |
| jschwarz | amuller, lets have that meeting tomorrow? will probably return to work next week for a few days | 21:13 |
| *** pritesh_ has quit IRC | 21:13 | |
| Swami | carl_baldwin: Ok makes sense. | 21:14 |
| *** rm_work is now known as rm_work|away | 21:14 | |
| *** madhu_ak has quit IRC | 21:14 | |
| amuller | carl_baldwin: with this regression fixed, are there other known bugs that would cause the DVR job to be higher than the non-DVR job? I understand the create/update-port post-commit bug was fixed (Thinking how soon can we make the DVR job voting to avoid this stuff in the future) | 21:15 |
| *** fawadkhaliq has joined #openstack-neutron | 21:15 | |
| *** madhu_ak has joined #openstack-neutron | 21:15 | |
| *** rossella_s has quit IRC | 21:15 | |
| *** safchain has quit IRC | 21:15 | |
| *** jschwarz has quit IRC | 21:16 | |
| carl_baldwin | amuller: /me thinking the same thing. Before these regressions, the two jobs were getting pretty close to the same failure rate. We were *about* to pull the trigger and make it voting. | 21:16 |
| amuller | carl_baldwin: *excited*, let's merge this regression then and monitor the failure rare closely over the next 2-3 days | 21:16 |
| amuller | if it's similar to the job, make it voting! | 21:16 |
| amuller | to the non-DVR job that is :) | 21:17 |
| carl_baldwin | +1 | 21:17 |
| carl_baldwin | johnbelamaric: What are your thoughts on how I propose to handle these two patches? Am I being too pedantic? | 21:18 |
| Swami | amuller: I would prefer we make this job voting as quick as possible. | 21:19 |
| amuller | Swami: That's well understood :) | 21:19 |
| Swami | amuller: let us have the deadline till this week end | 21:19 |
| Swami | I will rebase the voting job patch and push it in. | 21:19 |
| *** rm_work|away is now known as rm_work | 21:20 | |
| johnbelamaric | carl_baldwin: it sounds like we may want to let the one go through just to get it working. we can let the other patch go through after discussion. although in the end it's simple enough difference it probably doesn't matter | 21:20 |
| johnbelamaric | carl_baldwin: if it were my call I would probably just go with swami's patch though | 21:20 |
| *** dave-mccowan has joined #openstack-neutron | 21:21 | |
| *** pcm_m_AWAY is now known as pc_m | 21:21 | |
| carl_baldwin | johnbelamaric: If we combine the two, I’m going to insist on taking my time reviewing to convince myself that it is the right switch to make. That holds up fixing the regression or it makes me hold up other work. The other one is trivial given the current context, it has already been approved and just needs to get through the gate. | 21:23 |
| johnbelamaric | carl_baldwin: let's just do the fix that is ready to go then | 21:23 |
| *** srilatha has quit IRC | 21:24 | |
| Swami | carl_baldwin: yes I agree with merging the existing patch and then we can revisit and make necessary changes. | 21:24 |
| *** dboik_ has joined #openstack-neutron | 21:26 | |
| *** mutoulbj has joined #openstack-neutron | 21:27 | |
| amuller | Swami: carl_baldwin: I added a little remark to Yamamoto's patch. It's pretty obvious to me that a unit test should have caught the KeyError value in the original patch that introduced the regression. It's alarming to me that you are happy with just changing the UT that did not catch the error like this. | 21:27 |
| carl_baldwin | Swami: johnbelamaric: With that agreed, Swami’s time wasn’t wasted. I think that his patch raises an important question that should be reviewed and probably fixed. | 21:28 |
| *** crose has quit IRC | 21:28 | |
| amuller | So, I'm raising a testing red flag here, although I don't intend to delay the fix. For your consideration... | 21:28 |
| *** eckelcu has quit IRC | 21:28 | |
| * carl_baldwin has alarms going off in his head all the time. Getting hard to know which ones to pay attention to. | 21:29 | |
| carl_baldwin | amuller: Thanks | 21:29 |
| *** dboik has quit IRC | 21:30 | |
| *** sdake_ has joined #openstack-neutron | 21:30 | |
| *** sdake_ has quit IRC | 21:30 | |
| *** Sukhdev has joined #openstack-neutron | 21:30 | |
| *** sdake_ has joined #openstack-neutron | 21:30 | |
| amuller | I guess it's an example of a unit test with so many mocks so that it's basically engineered to pass, and it doesn't really verify much | 21:31 |
| *** dboik_ has quit IRC | 21:31 | |
| carl_baldwin | amuller: Good point. I’ll admit that since the patch was already approved, I didn’t review the tests. Sorry. | 21:31 |
| *** LukeHinds has quit IRC | 21:32 | |
| *** sdake has quit IRC | 21:33 | |
| *** Piet has joined #openstack-neutron | 21:33 | |
| * carl_baldwin points out it was a test that caught the problem. Just not a voting one. Still, could be better. | 21:33 | |
| amuller | yeah but you *really* don't need a full-on cloud (i.e. Tempest) to catch what is essentially a syntax error | 21:36 |
| amuller | it points to a problem in DVR's unit test coverage | 21:36 |
| *** crose has joined #openstack-neutron | 21:36 | |
| *** salv-orlando has joined #openstack-neutron | 21:36 | |
| *** Swami has quit IRC | 21:37 | |
| *** minwang2 has joined #openstack-neutron | 21:38 | |
| *** shashank_hegde_ has joined #openstack-neutron | 21:38 | |
| *** dave-mccowan has quit IRC | 21:39 | |
| *** mutoulbj has quit IRC | 21:41 | |
| *** dsneddon is now known as dsneddon_afk | 21:41 | |
| *** amuller has quit IRC | 21:41 | |
| *** sdake has joined #openstack-neutron | 21:43 | |
| *** slaweq has joined #openstack-neutron | 21:43 | |
| *** mgarza_ has quit IRC | 21:43 | |
| *** annegentle has quit IRC | 21:44 | |
| *** mgarza has joined #openstack-neutron | 21:44 | |
| *** crc32 has quit IRC | 21:44 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use oslo_log.helpers.log_method_call https://review.openstack.org/202308 | 21:44 |
| *** crc32 has joined #openstack-neutron | 21:44 | |
| *** annegentle has joined #openstack-neutron | 21:44 | |
| *** sdake_ has quit IRC | 21:46 | |
| *** sripriya has quit IRC | 21:47 | |
| *** shardy has quit IRC | 21:47 | |
| haleyb | carl_baldwin: sorry, I had to step out, but feel free to stop that patch if there is a better fix. | 21:47 |
| carl_baldwin | haleyb: tl;dr I want both patches. | 21:48 |
| *** sripriya has joined #openstack-neutron | 21:48 | |
| haleyb | carl_baldwin: ok, can't say i've looked at the second yet, and i'm at end of day here | 21:49 |
| *** jamielennox|away is now known as jamielennox | 21:49 | |
| carl_baldwin | haleyb: I put notes in both patches if you decide to catch up later. | 21:49 |
| *** annegentle has quit IRC | 21:49 | |
| *** rm_work is now known as rm_work|away | 21:50 | |
| *** madhu_ak has quit IRC | 21:50 | |
| *** madhu_ak has joined #openstack-neutron | 21:50 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: VPNaaS: Temporarily disable check_migration https://review.openstack.org/202197 | 21:51 |
| *** Swami has joined #openstack-neutron | 21:52 | |
| *** rm_work|away is now known as rm_work | 21:52 | |
| Swami | amuller:ping | 21:53 |
| *** slaweq has quit IRC | 21:54 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron-fwaas: Use oslo_log.helpers.log_method_call https://review.openstack.org/202314 | 21:55 |
| *** fawadkhaliq has quit IRC | 21:55 | |
| *** lilynn has joined #openstack-neutron | 21:56 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS DevStack Plugin support https://review.openstack.org/200592 | 21:56 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron-fwaas: Use oslo_log.helpers.log_method_call https://review.openstack.org/202314 | 21:56 |
| *** jaypipes has quit IRC | 21:57 | |
| *** crc32 has quit IRC | 21:57 | |
| *** Longgeek has joined #openstack-neutron | 21:57 | |
| openstackgerrit | Merged openstack/neutron: Remove db-access semaphore in ML2 https://review.openstack.org/192084 | 21:58 |
| *** crc32 has joined #openstack-neutron | 22:00 | |
| *** kbringard has joined #openstack-neutron | 22:02 | |
| *** patrickeast has quit IRC | 22:03 | |
| *** krtaylor has quit IRC | 22:03 | |
| *** shwetaap1 has joined #openstack-neutron | 22:04 | |
| *** edmondsw has quit IRC | 22:05 | |
| *** achanda has quit IRC | 22:06 | |
| *** achanda has joined #openstack-neutron | 22:06 | |
| *** breitz has quit IRC | 22:06 | |
| *** breitz has joined #openstack-neutron | 22:06 | |
| *** signed8bit has joined #openstack-neutron | 22:06 | |
| *** shivrao has quit IRC | 22:07 | |
| *** hakimo has quit IRC | 22:07 | |
| *** shivrao has joined #openstack-neutron | 22:07 | |
| *** britthouser has quit IRC | 22:07 | |
| *** hakimo has joined #openstack-neutron | 22:08 | |
| *** shwetaap has quit IRC | 22:08 | |
| *** crc32 has quit IRC | 22:09 | |
| *** signed8bit_ZZZzz has quit IRC | 22:10 | |
| *** mgarza has quit IRC | 22:11 | |
| *** mgarza has joined #openstack-neutron | 22:12 | |
| *** mlavalle has quit IRC | 22:14 | |
| *** banix has quit IRC | 22:15 | |
| *** agireud has quit IRC | 22:15 | |
| *** patrickeast has joined #openstack-neutron | 22:15 | |
| *** krtaylor has joined #openstack-neutron | 22:16 | |
| *** Longgeek has quit IRC | 22:16 | |
| *** johnbelamaric has quit IRC | 22:17 | |
| *** ijw has quit IRC | 22:18 | |
| *** sripriya has quit IRC | 22:19 | |
| *** sripriya has joined #openstack-neutron | 22:20 | |
| *** arajagopal has quit IRC | 22:21 | |
| *** EmilienM is now known as EmilienM|off | 22:22 | |
| *** doude has quit IRC | 22:23 | |
| *** crc32 has joined #openstack-neutron | 22:23 | |
| *** abhiraut has quit IRC | 22:24 | |
| spandhe | mestery: Hi Kyle, I have a question about your comment on bug 1464793 : https://bugs.launchpad.net/neutron/+bug/1464793/comments/4 | 22:24 |
| openstack | Launchpad bug 1464793 in neutron "Add a driver for isc-dhcpd" [Undecided,Confirmed] - Assigned to Shraddha Pandhe (shraddha-pandhe) | 22:24 |
| *** topshare has joined #openstack-neutron | 22:25 | |
| *** doude has joined #openstack-neutron | 22:26 | |
| *** patrickeast has quit IRC | 22:27 | |
| *** crc32 has quit IRC | 22:29 | |
| *** ivar-laz_ has joined #openstack-neutron | 22:30 | |
| *** igordcard_ has quit IRC | 22:30 | |
| *** crc32 has joined #openstack-neutron | 22:31 | |
| *** crc32 has quit IRC | 22:31 | |
| *** patrickeast has joined #openstack-neutron | 22:32 | |
| *** crc32 has joined #openstack-neutron | 22:33 | |
| *** shwetaap1 has quit IRC | 22:33 | |
| *** ivar-lazzaro has quit IRC | 22:33 | |
| openstackgerrit | Paul Michali proposed openstack/neutron: DO NOT MERGE - test change https://review.openstack.org/196149 | 22:35 |
| *** ArchiFleKs has quit IRC | 22:36 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 22:37 | |
| *** rushil has quit IRC | 22:37 | |
| *** EmilienM|off is now known as EmilienM | 22:37 | |
| *** cing has joined #openstack-neutron | 22:37 | |
| *** ijw has joined #openstack-neutron | 22:37 | |
| *** mitchjameson has joined #openstack-neutron | 22:38 | |
| *** rdo_ has quit IRC | 22:39 | |
| *** ijw_ has joined #openstack-neutron | 22:41 | |
| *** rdo has joined #openstack-neutron | 22:41 | |
| *** pritesh has joined #openstack-neutron | 22:41 | |
| *** mutoulbj has joined #openstack-neutron | 22:41 | |
| *** ijw has quit IRC | 22:42 | |
| *** yamahata has quit IRC | 22:42 | |
| regXboi | carl_baldwin: ping? | 22:43 |
| *** shashank_hegde_ has quit IRC | 22:43 | |
| *** minwang2 has quit IRC | 22:43 | |
| *** shwetaap has joined #openstack-neutron | 22:43 | |
| carl_baldwin | regXboi: pong | 22:43 |
| regXboi | carl_baldwin: two questions | 22:43 |
| *** vivek-ebay has joined #openstack-neutron | 22:43 | |
| regXboi | (1) since the bgp dyanmic routing is bgpspeaker only, am I correct in implying that there is no bgp listener or RIB and these may come later in a separate BP? | 22:44 |
| *** pc_m has quit IRC | 22:44 | |
| carl_baldwin | regXboi: Yes | 22:44 |
| *** prad has quit IRC | 22:45 | |
| HenryG | armax: ping | 22:45 |
| armax | HenryG: pong | 22:45 |
| regXboi | (2) a question came up internally about enable_snat on routers - if I turn that off, then I had better be using nonRFC 1918 addresses in the networks behind the router | 22:45 |
| *** minwang2 has joined #openstack-neutron | 22:45 | |
| HenryG | armax: I believe the first full-decomposition is ready: https://review.openstack.org/193365 | 22:46 |
| regXboi | is that a correct interpretation of the enable_snat code? | 22:46 |
| *** mickeys has quit IRC | 22:46 | |
| *** mickeys has joined #openstack-neutron | 22:46 | |
| armax | HenryG: looking | 22:47 |
| HenryG | armax: give it a good scrutiny and let me know what you think. I'll check in again tomorrow some time. | 22:47 |
| armax | will o | 22:47 |
| armax | will do | 22:47 |
| HenryG | armax: thanks, ttyl | 22:48 |
| armax | bye | 22:48 |
| carl_baldwin | regXboi: iiuc the router port will get an IP on the external network regardless of the enable_snat setting. | 22:48 |
| *** vivek-ebay has quit IRC | 22:48 | |
| armax | I hope you’re enjoying Sweden | 22:48 |
| regXboi | carl_baldwin: that's not what I'm asking | 22:48 |
| carl_baldwin | regXboi: You’re right, I read it too hastily. | 22:49 |
| carl_baldwin | regXboi: Okay, read it again... | 22:49 |
| *** dsneddon_afk is now known as dsneddon | 22:49 | |
| carl_baldwin | regXboi: RFC 1918 doesn’t really have anything to do with it but I see what you’re getting at. | 22:49 |
| regXboi | carl_baldwin: point | 22:50 |
| carl_baldwin | regXboi: You’d better be using routable addresses. Routable in the context where you’re operating. | 22:50 |
| *** shashank_hegde_ has joined #openstack-neutron | 22:50 | |
| regXboi | carl_baldwin: yes, that's what I was trying to say | 22:50 |
| carl_baldwin | Those *might* be rfc1918 addresses. | 22:50 |
| regXboi | carl_baldwin: yes | 22:51 |
| *** dims_ has joined #openstack-neutron | 22:51 | |
| carl_baldwin | We’re going to require they come from the same address scope. | 22:51 |
| regXboi | carl_baldwin: thx, my reading of the code is correct | 22:51 |
| *** mutoulbj has quit IRC | 22:51 | |
| *** abhiraut has joined #openstack-neutron | 22:52 | |
| *** dlundquist has quit IRC | 22:52 | |
| *** dlundquist has joined #openstack-neutron | 22:53 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use oslo_log.helpers.log_method_call https://review.openstack.org/202308 | 22:55 |
| *** jobewan has quit IRC | 22:55 | |
| *** kbringard1 has joined #openstack-neutron | 22:56 | |
| *** kbringard has quit IRC | 22:57 | |
| *** kbringard1 has quit IRC | 22:57 | |
| *** minwang2 has quit IRC | 22:58 | |
| openstackgerrit | Cyril Roelandt proposed openstack/neutron: Python 3: do not use types.ClassType https://review.openstack.org/200553 | 22:59 |
| *** mestery_ has joined #openstack-neutron | 23:00 | |
| openstackgerrit | Merged openstack/python-neutronclient: Support resource plurals not ending in 's' https://review.openstack.org/201610 | 23:02 |
| *** achanda_ has joined #openstack-neutron | 23:02 | |
| *** achanda has quit IRC | 23:02 | |
| *** EmilienM is now known as EmilienM|off | 23:03 | |
| *** minwang2 has joined #openstack-neutron | 23:03 | |
| *** mestery has quit IRC | 23:03 | |
| *** ijw_ has quit IRC | 23:04 | |
| *** padkrish has quit IRC | 23:04 | |
| *** mgarza has quit IRC | 23:04 | |
| *** ijw has joined #openstack-neutron | 23:04 | |
| *** shashank_hegde_ has quit IRC | 23:04 | |
| *** padkrish has joined #openstack-neutron | 23:04 | |
| *** topshare has quit IRC | 23:06 | |
| *** jasondotstar has joined #openstack-neutron | 23:07 | |
| *** shashank_hegde_ has joined #openstack-neutron | 23:07 | |
| *** devlaps has joined #openstack-neutron | 23:08 | |
| *** devlaps has quit IRC | 23:08 | |
| *** padkrish has quit IRC | 23:09 | |
| *** topshare has joined #openstack-neutron | 23:09 | |
| *** jasondotstar has quit IRC | 23:10 | |
| *** annegentle has joined #openstack-neutron | 23:10 | |
| *** mickeys has quit IRC | 23:12 | |
| *** zzzeek has quit IRC | 23:13 | |
| *** topshare has quit IRC | 23:14 | |
| *** padkrish has joined #openstack-neutron | 23:14 | |
| *** annegentle has quit IRC | 23:15 | |
| *** topshare has joined #openstack-neutron | 23:17 | |
| *** fitoduarte has quit IRC | 23:18 | |
| *** MentalRay__ has joined #openstack-neutron | 23:19 | |
| *** carl_baldwin has quit IRC | 23:20 | |
| *** MentalRay__ has quit IRC | 23:20 | |
| *** MentalRay_ has quit IRC | 23:20 | |
| *** ZZelle_ has quit IRC | 23:24 | |
| *** yuanying_ has quit IRC | 23:24 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Network RBAC DB setup and legacy migration https://review.openstack.org/191707 | 23:25 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Neutron RBAC API and network support https://review.openstack.org/192555 | 23:25 |
| *** annegentle has joined #openstack-neutron | 23:26 | |
| *** zhenguo has joined #openstack-neutron | 23:27 | |
| *** minwang2 has quit IRC | 23:29 | |
| *** pritesh has quit IRC | 23:29 | |
| *** mchestr has joined #openstack-neutron | 23:30 | |
| *** Sukhdev has quit IRC | 23:30 | |
| *** minwang2 has joined #openstack-neutron | 23:30 | |
| *** mchestr has quit IRC | 23:30 | |
| *** gildub has joined #openstack-neutron | 23:34 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 23:34 |
| *** shivrao has quit IRC | 23:35 | |
| *** ijw_ has joined #openstack-neutron | 23:37 | |
| *** dave-mccowan has joined #openstack-neutron | 23:37 | |
| *** ijw__ has joined #openstack-neutron | 23:39 | |
| *** doude has quit IRC | 23:40 | |
| *** doude has joined #openstack-neutron | 23:40 | |
| *** doude has quit IRC | 23:40 | |
| *** doude has joined #openstack-neutron | 23:40 | |
| *** oomichi has joined #openstack-neutron | 23:40 | |
| *** ijw has quit IRC | 23:40 | |
| *** thorst has joined #openstack-neutron | 23:41 | |
| *** ijw_ has quit IRC | 23:42 | |
| *** ijw has joined #openstack-neutron | 23:42 | |
| *** yuanying has joined #openstack-neutron | 23:42 | |
| *** cing has quit IRC | 23:43 | |
| *** topshare has quit IRC | 23:43 | |
| *** topshare has joined #openstack-neutron | 23:44 | |
| *** dlundquist has quit IRC | 23:45 | |
| *** ijw__ has quit IRC | 23:46 | |
| *** aranjan has quit IRC | 23:46 | |
| *** topshare_ has joined #openstack-neutron | 23:46 | |
| *** dave-mccowan has quit IRC | 23:46 | |
| *** shwetaap has quit IRC | 23:47 | |
| *** topshare has quit IRC | 23:49 | |
| *** wei-wang has quit IRC | 23:49 | |
| *** topshare_ has quit IRC | 23:51 | |
| *** mutoulbj has joined #openstack-neutron | 23:51 | |
| *** aranjan has joined #openstack-neutron | 23:51 | |
| *** thorst has quit IRC | 23:52 | |
| *** aranjan has quit IRC | 23:53 | |
| *** padkrish has quit IRC | 23:57 | |
| *** padkrish has joined #openstack-neutron | 23:57 | |
| *** gampel has quit IRC | 23:58 | |
| *** crose has quit IRC | 23:58 | |
| *** aranjan has joined #openstack-neutron | 23:58 | |
| *** Sukhdev has joined #openstack-neutron | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
