| *** baoli has quit IRC | 00:00 | |
| *** natarajk1 has quit IRC | 00:00 | |
| *** liusheng has quit IRC | 00:00 | |
| *** natarajk has joined #openstack-neutron | 00:00 | |
| *** lukasa_ has joined #openstack-neutron | 00:00 | |
| *** baoli has joined #openstack-neutron | 00:00 | |
| *** liusheng has joined #openstack-neutron | 00:00 | |
| *** natarajk has left #openstack-neutron | 00:00 | |
| *** natarajk has joined #openstack-neutron | 00:00 | |
| *** fredlhsu has left #openstack-neutron | 00:01 | |
| *** yalie has quit IRC | 00:01 | |
| *** r-daneel has joined #openstack-neutron | 00:02 | |
| *** dane_leblanc has quit IRC | 00:02 | |
| *** lukasa_work has quit IRC | 00:03 | |
| *** natarajk has quit IRC | 00:05 | |
| *** nati_ueno has joined #openstack-neutron | 00:08 | |
| flwang1 | garyk: any luck you around? | 00:10 |
|---|---|---|
| *** igordcard_ has quit IRC | 00:10 | |
| *** padkrish has quit IRC | 00:11 | |
| *** padkrish has joined #openstack-neutron | 00:11 | |
| *** Rajeev has quit IRC | 00:12 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 00:12 |
| *** aix has joined #openstack-neutron | 00:12 | |
| *** padkrish has joined #openstack-neutron | 00:12 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron-fwaas: Decouple L3 and Firewall during DVR router migration https://review.openstack.org/162380 | 00:12 |
| *** banix has joined #openstack-neutron | 00:14 | |
| *** natarajk1 has joined #openstack-neutron | 00:16 | |
| armax | HenryG: ping | 00:16 |
| *** vivek-ebay has quit IRC | 00:16 | |
| armax | dougwig: ping | 00:16 |
| *** vivek-ebay has joined #openstack-neutron | 00:16 | |
| dougwig | armax: ack ish | 00:17 |
| armax | dougwig: how can one ack-ish? | 00:17 |
| armax | :) | 00:17 |
| *** yamamoto has quit IRC | 00:18 | |
| armax | dougwig: was wondering what we should do with https://review.openstack.org/#/c/164363/ ? | 00:18 |
| dougwig | On phone, at restaurant. :) | 00:18 |
| *** yamamoto has joined #openstack-neutron | 00:18 | |
| armax | oh, don’t wanna intrude then | 00:18 |
| armax | I’ll talk to you once your tummy is full | 00:18 |
| *** YorikSar has quit IRC | 00:18 | |
| dougwig | Ok. :) | 00:19 |
| *** amotoki has joined #openstack-neutron | 00:19 | |
| *** YorikSar has joined #openstack-neutron | 00:19 | |
| *** pavel_bondar has quit IRC | 00:20 | |
| *** pavel_bondar has joined #openstack-neutron | 00:20 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation https://review.openstack.org/155470 | 00:22 |
| *** Swami has quit IRC | 00:22 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Revert "Set default of api_workers to number of CPUs" https://review.openstack.org/164363 | 00:23 |
| *** dane_leblanc has joined #openstack-neutron | 00:25 | |
| *** emagana has quit IRC | 00:26 | |
| *** banix has quit IRC | 00:26 | |
| *** ijw has quit IRC | 00:27 | |
| *** yamamoto has quit IRC | 00:28 | |
| *** vmtrooper has joined #openstack-neutron | 00:28 | |
| *** xgerman has quit IRC | 00:29 | |
| openstackgerrit | Miguel Lavalle proposed openstack/neutron: Add tests for the l3 agent namespaces manager https://review.openstack.org/162491 | 00:33 |
| *** vmtrooper has quit IRC | 00:33 | |
| *** natarajk1 has left #openstack-neutron | 00:35 | |
| *** claudiub has quit IRC | 00:36 | |
| *** shashankhegde has joined #openstack-neutron | 00:36 | |
| *** r-daneel has quit IRC | 00:37 | |
| *** marrusl has quit IRC | 00:37 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 00:37 | |
| *** mlavalle has quit IRC | 00:39 | |
| *** reed has quit IRC | 00:39 | |
| *** shashankhegde has quit IRC | 00:39 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 00:40 |
| *** YorikSar has quit IRC | 00:41 | |
| *** Longgeek has joined #openstack-neutron | 00:41 | |
| *** tflynn has quit IRC | 00:41 | |
| *** tidwellr has quit IRC | 00:44 | |
| HenryG | armax:pong | 00:46 |
| *** Marga_ has quit IRC | 00:46 | |
| *** Marga_ has joined #openstack-neutron | 00:47 | |
| *** achanda_ has quit IRC | 00:48 | |
| *** salv-orlando has joined #openstack-neutron | 00:50 | |
| HenryG | armax: It feels like the gate is trying to prevent us from reverting that api_workers patch | 00:51 |
| *** shaohe_feng1 has joined #openstack-neutron | 00:52 | |
| HenryG | armax: or fate is | 00:52 |
| *** yeungp has joined #openstack-neutron | 00:52 | |
| *** yalie has joined #openstack-neutron | 00:53 | |
| *** aveiga has joined #openstack-neutron | 00:54 | |
| *** banix has joined #openstack-neutron | 00:56 | |
| *** topshare has joined #openstack-neutron | 00:57 | |
| *** yeungp has quit IRC | 00:58 | |
| *** topshare has quit IRC | 00:58 | |
| *** yamamoto has joined #openstack-neutron | 00:59 | |
| *** aranjan has quit IRC | 01:00 | |
| *** Longgeek has quit IRC | 01:01 | |
| *** Longgeek has joined #openstack-neutron | 01:01 | |
| *** topshare has joined #openstack-neutron | 01:03 | |
| *** dmellado has quit IRC | 01:03 | |
| *** britthouser has quit IRC | 01:04 | |
| armax | HenryG: ya | 01:04 |
| *** YorikSar has joined #openstack-neutron | 01:04 | |
| *** vivek-ebay has quit IRC | 01:05 | |
| *** vivek-ebay has joined #openstack-neutron | 01:05 | |
| *** oda-g has quit IRC | 01:06 | |
| *** britthouser has joined #openstack-neutron | 01:07 | |
| *** claired has joined #openstack-neutron | 01:08 | |
| *** nati_ueno has quit IRC | 01:08 | |
| *** shwetaap has joined #openstack-neutron | 01:09 | |
| HenryG | armax: is there anything I can do? | 01:10 |
| *** nlahouti has quit IRC | 01:10 | |
| *** tsg has joined #openstack-neutron | 01:11 | |
| *** itzikb has joined #openstack-neutron | 01:14 | |
| *** shwetaap1 has joined #openstack-neutron | 01:14 | |
| *** oda-g has joined #openstack-neutron | 01:17 | |
| *** shwetaap has quit IRC | 01:17 | |
| *** tsg has quit IRC | 01:18 | |
| *** leenheer has joined #openstack-neutron | 01:19 | |
| *** tiswanso has joined #openstack-neutron | 01:19 | |
| *** wverdugo5002 has joined #openstack-neutron | 01:19 | |
| *** wverdugo5002 has left #openstack-neutron | 01:21 | |
| *** claired has quit IRC | 01:23 | |
| *** leenheer has quit IRC | 01:23 | |
| *** lykinsbd has joined #openstack-neutron | 01:24 | |
| *** pritesh has joined #openstack-neutron | 01:25 | |
| *** tiswanso_ has joined #openstack-neutron | 01:25 | |
| *** dmellado has joined #openstack-neutron | 01:26 | |
| *** dims has quit IRC | 01:26 | |
| *** pritesh has quit IRC | 01:28 | |
| *** tiswanso has quit IRC | 01:28 | |
| *** lykinsbd has quit IRC | 01:28 | |
| *** pritesh has joined #openstack-neutron | 01:30 | |
| *** banix has quit IRC | 01:31 | |
| *** liusheng has quit IRC | 01:31 | |
| *** liusheng has joined #openstack-neutron | 01:32 | |
| *** shashankhegde has joined #openstack-neutron | 01:34 | |
| *** dmellado has quit IRC | 01:35 | |
| *** stanzgy has joined #openstack-neutron | 01:36 | |
| *** stanzgy_ has joined #openstack-neutron | 01:36 | |
| *** garyduan has joined #openstack-neutron | 01:37 | |
| *** cference has joined #openstack-neutron | 01:39 | |
| *** anish_ has joined #openstack-neutron | 01:40 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create https://review.openstack.org/163672 | 01:40 |
| openstackgerrit | Merged openstack/neutron: Move Unix domain socket helpers to a common place https://review.openstack.org/164032 | 01:40 |
| *** vishwana_ has quit IRC | 01:40 | |
| *** Longgeek has quit IRC | 01:41 | |
| *** banix has joined #openstack-neutron | 01:42 | |
| amuller | marun: ping | 01:43 |
| *** popw has joined #openstack-neutron | 01:44 | |
| *** itzikb has quit IRC | 01:46 | |
| *** _cjones_ has joined #openstack-neutron | 01:47 | |
| *** _cjones_ has quit IRC | 01:48 | |
| *** s3wong has quit IRC | 01:48 | |
| *** _cjones_ has joined #openstack-neutron | 01:48 | |
| *** topshare has quit IRC | 01:48 | |
| *** nati_ueno has joined #openstack-neutron | 01:48 | |
| *** Longgeek has joined #openstack-neutron | 01:49 | |
| *** YorikSar has quit IRC | 01:49 | |
| *** topshare has joined #openstack-neutron | 01:52 | |
| *** natarajk1 has joined #openstack-neutron | 01:54 | |
| *** dims has joined #openstack-neutron | 01:54 | |
| *** Networkn3rd has joined #openstack-neutron | 01:54 | |
| natarajk1 | armax: hi | 01:54 |
| *** Marga_ has quit IRC | 01:55 | |
| *** JohnT has joined #openstack-neutron | 01:58 | |
| *** Marga_ has joined #openstack-neutron | 01:58 | |
| *** _cjones_ has quit IRC | 01:58 | |
| *** baoli has quit IRC | 01:58 | |
| *** pal has joined #openstack-neutron | 01:59 | |
| *** baoli has joined #openstack-neutron | 01:59 | |
| *** natarajk1 has quit IRC | 01:59 | |
| *** dmellado has joined #openstack-neutron | 02:01 | |
| *** dims has quit IRC | 02:03 | |
| *** topshare has quit IRC | 02:03 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit https://review.openstack.org/162840 | 02:04 |
| *** padkrish has quit IRC | 02:04 | |
| openstackgerrit | Merged openstack/neutron: Revert "fix check_ports_exist_on_l3agent in no subnet case" https://review.openstack.org/164852 | 02:04 |
| *** topshare has joined #openstack-neutron | 02:04 | |
| *** padkrish has joined #openstack-neutron | 02:05 | |
| *** ivar-laz_ has joined #openstack-neutron | 02:06 | |
| *** yamahata has quit IRC | 02:07 | |
| *** Networkn3rd has quit IRC | 02:07 | |
| *** Marga_ has quit IRC | 02:08 | |
| *** padkrish has quit IRC | 02:09 | |
| *** zzzeek has quit IRC | 02:09 | |
| dougwig | armax: here now. did you get it sorted? | 02:09 |
| *** ivar-lazzaro has quit IRC | 02:09 | |
| *** Marga_ has joined #openstack-neutron | 02:10 | |
| dougwig | i'd be tempted to manually fix that and rage merge it. | 02:10 |
| dougwig | manually fix the rebase, i mean. | 02:10 |
| *** ivar-laz_ has quit IRC | 02:10 | |
| *** salv-orlando has quit IRC | 02:11 | |
| *** Marga_ has quit IRC | 02:13 | |
| *** anish_ has quit IRC | 02:13 | |
| HenryG | dougwig: it passed the check queue, in the gate now | 02:13 |
| dougwig | ok. so you're saying about 8 more tries. | 02:13 |
| HenryG | pessimist | 02:14 |
| *** gongysh has joined #openstack-neutron | 02:15 | |
| *** absubram has joined #openstack-neutron | 02:16 | |
| *** JohnT is now known as HomeLate | 02:16 | |
| *** leenheer has joined #openstack-neutron | 02:16 | |
| *** leenheer has quit IRC | 02:16 | |
| *** HomeLate is now known as cjh- | 02:17 | |
| *** vmtrooper has joined #openstack-neutron | 02:17 | |
| *** cjh- is now known as cjh_ | 02:19 | |
| *** cjh_ is now known as cjha | 02:19 | |
| *** absubram has quit IRC | 02:20 | |
| *** absubram has joined #openstack-neutron | 02:20 | |
| *** catohornet has quit IRC | 02:21 | |
| *** jyuso1 has joined #openstack-neutron | 02:21 | |
| *** vmtrooper has quit IRC | 02:22 | |
| *** erkules_ has joined #openstack-neutron | 02:22 | |
| *** tsg has joined #openstack-neutron | 02:23 | |
| *** baoli has quit IRC | 02:23 | |
| *** baoli has joined #openstack-neutron | 02:24 | |
| *** erkules has quit IRC | 02:25 | |
| *** dane_leblanc has quit IRC | 02:25 | |
| *** coolsvap|afk is now known as coolsvap | 02:27 | |
| *** achanda has joined #openstack-neutron | 02:27 | |
| *** vivek-ebay has quit IRC | 02:28 | |
| *** cference has quit IRC | 02:29 | |
| *** lykinsbd has joined #openstack-neutron | 02:31 | |
| *** cjha is now known as kaneeni | 02:32 | |
| *** dane_leblanc has joined #openstack-neutron | 02:33 | |
| *** stanzgy has quit IRC | 02:34 | |
| *** stanzgy has joined #openstack-neutron | 02:34 | |
| *** kaneeni is now known as somefag | 02:35 | |
| *** somefag has quit IRC | 02:35 | |
| *** JohnT has joined #openstack-neutron | 02:36 | |
| *** lykinsbd has quit IRC | 02:36 | |
| *** amuller has quit IRC | 02:36 | |
| *** vishwanathj has joined #openstack-neutron | 02:37 | |
| *** krtaylor has quit IRC | 02:38 | |
| *** tsg has quit IRC | 02:40 | |
| *** tflynn has joined #openstack-neutron | 02:40 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 02:44 |
| *** tflynn has quit IRC | 02:45 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: abcd | 02:46 |
| *** pritesh has quit IRC | 02:47 | |
| *** shashankhegde has quit IRC | 02:49 | |
| *** krtaylor has joined #openstack-neutron | 02:50 | |
| *** cference has joined #openstack-neutron | 02:51 | |
| *** chandankumar has joined #openstack-neutron | 02:51 | |
| *** JohnT is now known as p1np01nt | 02:51 | |
| *** achanda has quit IRC | 02:52 | |
| *** pal has quit IRC | 02:53 | |
| *** shashankhegde has joined #openstack-neutron | 02:54 | |
| *** yuanying has quit IRC | 02:54 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: update the wrapping lines | 02:55 |
| *** natarajk has joined #openstack-neutron | 02:58 | |
| *** p1np01nt is now known as Exp1r3dP0ps1cle | 02:58 | |
| *** natarajk has left #openstack-neutron | 02:58 | |
| *** Exp1r3dP0ps1cle is now known as d[-_-]b | 02:58 | |
| *** natarajk has joined #openstack-neutron | 02:58 | |
| *** melwitt has quit IRC | 02:58 | |
| *** topshare has quit IRC | 02:59 | |
| natarajk | mestery: hi | 03:00 |
| *** d[-_-]b has quit IRC | 03:00 | |
| *** sbalukoff has quit IRC | 03:02 | |
| *** dims has joined #openstack-neutron | 03:04 | |
| *** topshare has joined #openstack-neutron | 03:06 | |
| *** leenheer has joined #openstack-neutron | 03:06 | |
| *** cference has quit IRC | 03:06 | |
| *** tflynn has joined #openstack-neutron | 03:06 | |
| *** tflynn has quit IRC | 03:07 | |
| *** tflynn has joined #openstack-neutron | 03:08 | |
| *** RalphWiggum has joined #openstack-neutron | 03:08 | |
| *** RalphWiggum is now known as Guest80749 | 03:08 | |
| *** topshare has quit IRC | 03:08 | |
| openstackgerrit | Merged openstack/neutron: Add MTU selection & advertisement settings to Neutron config https://review.openstack.org/153733 | 03:09 |
| *** topshare has joined #openstack-neutron | 03:09 | |
| *** popw has quit IRC | 03:10 | |
| *** salv-orlando has joined #openstack-neutron | 03:11 | |
| *** dims has quit IRC | 03:11 | |
| *** boris-42 has quit IRC | 03:12 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: Remove the extra blank lines https://review.openstack.org/164935 | 03:13 |
| openstackgerrit | Merged openstack/neutron: Prevent updating mac address of bound port https://review.openstack.org/162500 | 03:13 |
| openstackgerrit | Merged openstack/neutron: Revert "Set default of api_workers to number of CPUs" https://review.openstack.org/164363 | 03:14 |
| *** padkrish has joined #openstack-neutron | 03:15 | |
| HenryG | finally | 03:15 |
| *** harlowja is now known as harlowja_away | 03:15 | |
| *** tiswanso_ has quit IRC | 03:17 | |
| *** Murali has joined #openstack-neutron | 03:17 | |
| *** banix has quit IRC | 03:18 | |
| *** topshare has quit IRC | 03:18 | |
| *** banix has joined #openstack-neutron | 03:19 | |
| *** popw has joined #openstack-neutron | 03:19 | |
| *** padkrish has quit IRC | 03:20 | |
| HenryG | armax: ping | 03:20 |
| *** zzzeek has joined #openstack-neutron | 03:22 | |
| *** zzzeek has quit IRC | 03:22 | |
| *** signed8b_ has joined #openstack-neutron | 03:23 | |
| *** topshare has joined #openstack-neutron | 03:25 | |
| HenryG | dougwig: ping | 03:25 |
| *** signed8bit has quit IRC | 03:26 | |
| dougwig | HenryG: ack | 03:26 |
| *** tflynn has quit IRC | 03:26 | |
| HenryG | dougwig: is there a new grenade breakage? | 03:26 |
| dougwig | omg, you are kidding me?!? | 03:27 |
| HenryG | dougwig: Have you seen this before? http://logs.openstack.org/93/162693/5/check/check-grenade-dsvm-neutron/d913004/logs/new/screen-q-svc.txt.gz#_2015-03-16_23_09_15_083 | 03:27 |
| *** sluo_wfh is now known as sluo_laptop | 03:28 | |
| dougwig | no, i haven't. and that's lbaas v1. | 03:28 |
| HenryG | When great swaths of code get merged all in a rush, things break | 03:28 |
| dougwig | no joke. | 03:28 |
| dougwig | has that failed in more than one? | 03:28 |
| HenryG | dougwig: still looking | 03:28 |
| *** markvoelker has quit IRC | 03:29 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: fix some wrapping errors https://review.openstack.org/164937 | 03:31 |
| HenryG | dougwig: phew, the more recent jobs (last 3 hours or so) seem to be passing grenade | 03:32 |
| *** dmellado has quit IRC | 03:33 | |
| *** shashankhegde has quit IRC | 03:34 | |
| *** david-lyle is now known as david-lyle_afk | 03:35 | |
| *** trad511 has joined #openstack-neutron | 03:36 | |
| *** jckasper has joined #openstack-neutron | 03:37 | |
| *** padkrish has joined #openstack-neutron | 03:39 | |
| dougwig | HenryG: oh good, i'm not sure my sanity could handle another full breakage. | 03:39 |
| *** padkrish has quit IRC | 03:39 | |
| *** yuanying has joined #openstack-neutron | 03:39 | |
| *** padkrish has joined #openstack-neutron | 03:39 | |
| *** mitz has quit IRC | 03:40 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 03:40 |
| *** dmellado has joined #openstack-neutron | 03:41 | |
| *** tsg has joined #openstack-neutron | 03:42 | |
| *** Murali has quit IRC | 03:42 | |
| *** chandankumar has quit IRC | 03:42 | |
| *** tflynn has joined #openstack-neutron | 03:45 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 03:46 |
| *** coolsvap is now known as coolsvap|afk | 03:48 | |
| *** aranjan has joined #openstack-neutron | 03:48 | |
| *** nlahouti has joined #openstack-neutron | 03:48 | |
| *** shashankhegde has joined #openstack-neutron | 03:48 | |
| *** trad511 has quit IRC | 03:49 | |
| *** vmtrooper has joined #openstack-neutron | 03:49 | |
| *** pal has joined #openstack-neutron | 03:50 | |
| *** Networkn3rd has joined #openstack-neutron | 03:50 | |
| *** aranjan has quit IRC | 03:53 | |
| *** nlahouti has quit IRC | 03:55 | |
| *** coolsvap|afk is now known as coolsvap | 03:55 | |
| *** nlahouti has joined #openstack-neutron | 03:56 | |
| *** chandankumar has joined #openstack-neutron | 03:57 | |
| *** Longgeek has quit IRC | 03:59 | |
| *** chandankumar has quit IRC | 03:59 | |
| *** chandankumar has joined #openstack-neutron | 04:00 | |
| *** padkrish has quit IRC | 04:00 | |
| *** jckasper has quit IRC | 04:00 | |
| *** achanda has joined #openstack-neutron | 04:01 | |
| *** jckasper has joined #openstack-neutron | 04:01 | |
| *** JoeJulian has quit IRC | 04:02 | |
| *** baoli has quit IRC | 04:02 | |
| *** jckasper has quit IRC | 04:02 | |
| *** britthouser has quit IRC | 04:02 | |
| *** Guest80749 has quit IRC | 04:04 | |
| *** _cjones_ has joined #openstack-neutron | 04:04 | |
| *** _cjones_ has quit IRC | 04:04 | |
| *** _cjones_ has joined #openstack-neutron | 04:05 | |
| *** RalphWiggum has joined #openstack-neutron | 04:05 | |
| *** RalphWiggum is now known as Guest22117 | 04:05 | |
| *** tflynn has quit IRC | 04:05 | |
| *** tflynn has joined #openstack-neutron | 04:05 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Functional tests for OVS-agent OpenFlow drivers https://review.openstack.org/164584 | 04:07 |
| *** jckasper has joined #openstack-neutron | 04:08 | |
| *** jckasper has quit IRC | 04:10 | |
| *** banix has quit IRC | 04:11 | |
| *** topshare_ has joined #openstack-neutron | 04:14 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Functional tests for OVS-agent OpenFlow drivers https://review.openstack.org/164584 | 04:16 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 04:16 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 04:16 |
| *** topshare has quit IRC | 04:16 | |
| *** tflynn has quit IRC | 04:17 | |
| openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 04:18 |
| *** drjones has joined #openstack-neutron | 04:19 | |
| *** lykinsbd has joined #openstack-neutron | 04:20 | |
| *** _cjones_ has quit IRC | 04:21 | |
| *** padkrish has joined #openstack-neutron | 04:21 | |
| *** Bhavya has joined #openstack-neutron | 04:22 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create https://review.openstack.org/163672 | 04:22 |
| *** haleyb has quit IRC | 04:23 | |
| *** aranjan_ has joined #openstack-neutron | 04:23 | |
| *** salv-orlando has quit IRC | 04:23 | |
| *** haleyb has joined #openstack-neutron | 04:23 | |
| *** shwetaap1 has quit IRC | 04:24 | |
| Guest22117 | DCC SEND STARTKEYLOGGER 0 0 0 | 04:24 |
| *** haleyb has quit IRC | 04:24 | |
| *** Guest22117 has left #openstack-neutron | 04:24 | |
| *** haleyb has joined #openstack-neutron | 04:24 | |
| *** lykinsbd has quit IRC | 04:25 | |
| *** shashankhegde has quit IRC | 04:26 | |
| *** zhhuabj_ has quit IRC | 04:28 | |
| *** zhhuabj_ has joined #openstack-neutron | 04:28 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment https://review.openstack.org/164943 | 04:28 |
| *** padkrish has quit IRC | 04:31 | |
| *** padkrish has joined #openstack-neutron | 04:33 | |
| *** mitz has joined #openstack-neutron | 04:34 | |
| *** tsg has quit IRC | 04:35 | |
| *** baoli has joined #openstack-neutron | 04:35 | |
| *** JoeJulian has joined #openstack-neutron | 04:35 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment https://review.openstack.org/164943 | 04:36 |
| *** natarajk has left #openstack-neutron | 04:36 | |
| *** stanzgy has quit IRC | 04:37 | |
| *** vmtrooper has quit IRC | 04:37 | |
| *** stanzgy has joined #openstack-neutron | 04:38 | |
| *** sbalukoff has joined #openstack-neutron | 04:39 | |
| *** yatin has joined #openstack-neutron | 04:39 | |
| *** Tetris- has joined #openstack-neutron | 04:39 | |
| *** baoli has quit IRC | 04:40 | |
| *** Tetris- has left #openstack-neutron | 04:40 | |
| *** baoli has joined #openstack-neutron | 04:40 | |
| *** baoli has quit IRC | 04:40 | |
| *** oomichi has joined #openstack-neutron | 04:41 | |
| *** oomichi has quit IRC | 04:41 | |
| *** padkrish_ has joined #openstack-neutron | 04:42 | |
| *** enikanorov has quit IRC | 04:43 | |
| *** padkrish has quit IRC | 04:43 | |
| *** enikanorov has joined #openstack-neutron | 04:43 | |
| *** achanda has quit IRC | 04:46 | |
| *** trinaths has joined #openstack-neutron | 04:47 | |
| *** achanda has joined #openstack-neutron | 04:49 | |
| *** leenheer has quit IRC | 04:49 | |
| *** achanda has quit IRC | 04:49 | |
| *** dboik_ has quit IRC | 04:49 | |
| *** angela-s has left #openstack-neutron | 04:53 | |
| openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers https://review.openstack.org/152697 | 04:54 |
| *** anilvenkata has joined #openstack-neutron | 04:57 | |
| *** padkrish_ has quit IRC | 04:57 | |
| *** pradipta has joined #openstack-neutron | 04:57 | |
| *** lykinsbd has joined #openstack-neutron | 04:57 | |
| *** padkrish has joined #openstack-neutron | 04:58 | |
| *** Marga_ has joined #openstack-neutron | 04:58 | |
| *** Longgeek has joined #openstack-neutron | 04:59 | |
| *** baoli has joined #openstack-neutron | 05:01 | |
| *** baoli has quit IRC | 05:01 | |
| *** lykinsbd has quit IRC | 05:02 | |
| *** padkrish has quit IRC | 05:02 | |
| *** tflynn has joined #openstack-neutron | 05:02 | |
| *** Marga_ has quit IRC | 05:04 | |
| *** Longgeek has quit IRC | 05:04 | |
| *** drjones has quit IRC | 05:05 | |
| *** _cjones_ has joined #openstack-neutron | 05:05 | |
| *** emagana has joined #openstack-neutron | 05:10 | |
| *** Networkn3rd has quit IRC | 05:14 | |
| *** leenheer has joined #openstack-neutron | 05:14 | |
| *** leenheer has quit IRC | 05:16 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 05:17 |
| *** dane_leblanc has quit IRC | 05:17 | |
| *** salv-orlando has joined #openstack-neutron | 05:20 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 05:21 |
| openstackgerrit | Merged openstack/python-neutronclient: Change Creates to Create in help text https://review.openstack.org/137481 | 05:23 |
| *** tflynn has quit IRC | 05:34 | |
| *** pritesh has joined #openstack-neutron | 05:35 | |
| *** Bhavya has quit IRC | 05:36 | |
| *** tflynn has joined #openstack-neutron | 05:37 | |
| *** Bhavya has joined #openstack-neutron | 05:39 | |
| *** nlahouti has quit IRC | 05:43 | |
| *** nlahouti has joined #openstack-neutron | 05:43 | |
| *** pritesh has quit IRC | 05:44 | |
| *** yamahata has joined #openstack-neutron | 05:49 | |
| openstackgerrit | Aman Kumar proposed openstack/neutron: Added config variable for External Network type in ML2 https://review.openstack.org/149546 | 05:51 |
| *** aranjan_ has quit IRC | 05:51 | |
| *** aranjan has joined #openstack-neutron | 05:52 | |
| *** ishant|2 has joined #openstack-neutron | 05:53 | |
| *** ishant|2 has quit IRC | 05:54 | |
| *** ishant has joined #openstack-neutron | 05:54 | |
| *** drjones has joined #openstack-neutron | 05:56 | |
| *** _cjones_ has quit IRC | 05:57 | |
| *** ajo has joined #openstack-neutron | 05:57 | |
| *** yamamoto has quit IRC | 06:01 | |
| *** Longgeek has joined #openstack-neutron | 06:01 | |
| *** baoli has joined #openstack-neutron | 06:02 | |
| *** SridharG has joined #openstack-neutron | 06:04 | |
| *** baoli has quit IRC | 06:06 | |
| *** tflynn has quit IRC | 06:07 | |
| *** tflynn has joined #openstack-neutron | 06:07 | |
| *** SridharG has left #openstack-neutron | 06:08 | |
| *** hichihara has quit IRC | 06:08 | |
| *** padkrish has joined #openstack-neutron | 06:08 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 06:09 |
| *** nyechiel__ has joined #openstack-neutron | 06:10 | |
| *** yfried_ has joined #openstack-neutron | 06:11 | |
| *** lpetrut has joined #openstack-neutron | 06:11 | |
| *** Marga_ has joined #openstack-neutron | 06:12 | |
| *** absubram has quit IRC | 06:12 | |
| *** padkrish has quit IRC | 06:13 | |
| *** sudipto has joined #openstack-neutron | 06:13 | |
| openstackgerrit | Merged openstack/neutron: Fix l3_agentschedulers_db for consistency of code https://review.openstack.org/163740 | 06:14 |
| *** numan has joined #openstack-neutron | 06:15 | |
| *** Marga_ has quit IRC | 06:15 | |
| *** Marga_ has joined #openstack-neutron | 06:15 | |
| *** yfauser has joined #openstack-neutron | 06:16 | |
| *** yfauser has left #openstack-neutron | 06:16 | |
| *** Murali has joined #openstack-neutron | 06:21 | |
| *** saju_m has joined #openstack-neutron | 06:22 | |
| *** yog__ has joined #openstack-neutron | 06:24 | |
| *** vmtrooper has joined #openstack-neutron | 06:26 | |
| *** achanda has joined #openstack-neutron | 06:26 | |
| *** drjones has quit IRC | 06:28 | |
| *** nyechiel__ has quit IRC | 06:29 | |
| *** dboik has joined #openstack-neutron | 06:29 | |
| *** vmtrooper has quit IRC | 06:31 | |
| *** baoli has joined #openstack-neutron | 06:32 | |
| *** baoli has quit IRC | 06:32 | |
| *** baoli has joined #openstack-neutron | 06:33 | |
| *** baoli has quit IRC | 06:33 | |
| vishwanathj | amotoki, FYI , I have responded to your comments and uploaded a new patch addressing some of your comments at https://review.openstack.org/#/c/162552/ | 06:34 |
| *** signed8b_ has quit IRC | 06:34 | |
| *** Longgeek has quit IRC | 06:35 | |
| vishwanathj | oops, I should have posted that in the openstack-horizon channel, sorry about that | 06:35 |
| *** vishwanathj has quit IRC | 06:35 | |
| *** Longgeek has joined #openstack-neutron | 06:35 | |
| *** numan has quit IRC | 06:38 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 06:39 |
| *** nyechiel__ has joined #openstack-neutron | 06:39 | |
| *** amotoki has quit IRC | 06:43 | |
| *** gildub has quit IRC | 06:43 | |
| *** tflynn has quit IRC | 06:44 | |
| *** tflynn has joined #openstack-neutron | 06:44 | |
| *** hanzhang has quit IRC | 06:44 | |
| *** salv-orlando has quit IRC | 06:44 | |
| *** hanzhang has joined #openstack-neutron | 06:45 | |
| *** lykinsbd has joined #openstack-neutron | 06:46 | |
| *** tflynn has quit IRC | 06:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/163708 | 06:47 |
| *** tflynn has joined #openstack-neutron | 06:47 | |
| *** oda-g has quit IRC | 06:48 | |
| *** abehl has joined #openstack-neutron | 06:49 | |
| *** amotoki has joined #openstack-neutron | 06:49 | |
| *** lykinsbd has quit IRC | 06:50 | |
| *** killer_prince is now known as lazy_prince | 06:52 | |
| *** SridharG has joined #openstack-neutron | 06:52 | |
| *** yamahata has quit IRC | 06:53 | |
| *** yamahata has joined #openstack-neutron | 06:53 | |
| *** haleyb has quit IRC | 06:56 | |
| *** haleyb has joined #openstack-neutron | 06:56 | |
| *** saju_m has quit IRC | 06:56 | |
| *** haleyb has quit IRC | 06:57 | |
| *** haleyb has joined #openstack-neutron | 06:57 | |
| *** ferest has joined #openstack-neutron | 06:57 | |
| *** saju_m has joined #openstack-neutron | 06:58 | |
| *** saju_m has quit IRC | 07:00 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 07:03 |
| *** markvoelker has joined #openstack-neutron | 07:04 | |
| *** yalie has quit IRC | 07:06 | |
| *** markvoelker has quit IRC | 07:08 | |
| *** ferest has quit IRC | 07:09 | |
| openstackgerrit | Merged openstack/neutron-fwaas: FWaaS Insertion Model on Routers https://review.openstack.org/152697 | 07:11 |
| *** nlahouti has quit IRC | 07:12 | |
| *** tflynn has quit IRC | 07:13 | |
| *** emagana has quit IRC | 07:14 | |
| *** numan has joined #openstack-neutron | 07:15 | |
| *** kickinz1|afk is now known as kickinz1 | 07:15 | |
| *** scheuran has joined #openstack-neutron | 07:16 | |
| *** yamahata has quit IRC | 07:20 | |
| *** yfried_ is now known as yfried|afk | 07:22 | |
| *** lpetrut has left #openstack-neutron | 07:24 | |
| *** FingerPistol has joined #openstack-neutron | 07:25 | |
| *** afazekas has joined #openstack-neutron | 07:25 | |
| *** nati_ueno has quit IRC | 07:25 | |
| *** nati_ueno has joined #openstack-neutron | 07:26 | |
| *** neeti has joined #openstack-neutron | 07:27 | |
| openstackgerrit | Merged openstack/neutron: Schedule net to a DHCP agt on subnet create https://review.openstack.org/163672 | 07:29 |
| *** yalie has joined #openstack-neutron | 07:29 | |
| *** FingerPistol has quit IRC | 07:29 | |
| *** yfried|afk is now known as yfried_ | 07:30 | |
| *** nati_ueno has quit IRC | 07:30 | |
| *** achanda has quit IRC | 07:31 | |
| *** nyechiel__ has quit IRC | 07:31 | |
| *** nyechiel__ has joined #openstack-neutron | 07:31 | |
| *** baoli has joined #openstack-neutron | 07:34 | |
| *** yog__ has quit IRC | 07:34 | |
| *** Marga_ has quit IRC | 07:34 | |
| *** nyechiel__ has quit IRC | 07:37 | |
| *** baoli has quit IRC | 07:38 | |
| *** yog__ has joined #openstack-neutron | 07:38 | |
| *** yfried_ is now known as yfried|afk | 07:41 | |
| *** aranjan has quit IRC | 07:43 | |
| *** aranjan_ has joined #openstack-neutron | 07:45 | |
| *** rkukura_ has joined #openstack-neutron | 07:45 | |
| *** ajo has quit IRC | 07:46 | |
| *** aranjan_ has quit IRC | 07:46 | |
| *** aranjan__ has joined #openstack-neutron | 07:46 | |
| *** ajo has joined #openstack-neutron | 07:46 | |
| *** rkukura has quit IRC | 07:47 | |
| *** rkukura_ is now known as rkukura | 07:47 | |
| *** armax has quit IRC | 07:48 | |
| *** aranjan__ has quit IRC | 07:51 | |
| *** kawa2014 has joined #openstack-neutron | 07:51 | |
| *** yatin has quit IRC | 07:56 | |
| *** nati_ueno has joined #openstack-neutron | 07:56 | |
| *** nati_uen_ has joined #openstack-neutron | 07:58 | |
| *** nati_ueno has quit IRC | 07:58 | |
| *** safchain has joined #openstack-neutron | 07:59 | |
| *** yfried|afk is now known as yfried_ | 08:01 | |
| *** oomichi_ has joined #openstack-neutron | 08:02 | |
| *** nati_uen_ has quit IRC | 08:02 | |
| *** sridhar_ram has joined #openstack-neutron | 08:04 | |
| *** erkules_ is now known as erkuls | 08:04 | |
| *** bobmel has joined #openstack-neutron | 08:04 | |
| *** moha_hunt has joined #openstack-neutron | 08:05 | |
| *** pask81 has joined #openstack-neutron | 08:05 | |
| *** markvoelker has joined #openstack-neutron | 08:05 | |
| *** erkuls is now known as erkules | 08:05 | |
| *** bobmel has quit IRC | 08:05 | |
| *** erkules has quit IRC | 08:05 | |
| *** erkules has joined #openstack-neutron | 08:05 | |
| *** yatin has joined #openstack-neutron | 08:07 | |
| *** aepifanov has joined #openstack-neutron | 08:08 | |
| *** markvoelker has quit IRC | 08:10 | |
| *** yfried_ is now known as yfried|afk | 08:11 | |
| *** lukasa_work has joined #openstack-neutron | 08:14 | |
| *** irenab_ has joined #openstack-neutron | 08:15 | |
| *** vmtrooper has joined #openstack-neutron | 08:15 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 08:15 |
| *** topshare_ has quit IRC | 08:15 | |
| *** irenab has quit IRC | 08:16 | |
| *** irenab_ is now known as irenab | 08:16 | |
| *** kickinz1 is now known as kickinz1|afk | 08:18 | |
| *** lukasa_ has quit IRC | 08:18 | |
| *** kickinz1|afk is now known as kickinz1 | 08:19 | |
| *** dsneddon has quit IRC | 08:19 | |
| *** dsneddon has joined #openstack-neutron | 08:20 | |
| *** vmtrooper has quit IRC | 08:21 | |
| *** ronis has joined #openstack-neutron | 08:21 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 08:24 |
| *** ajo has quit IRC | 08:25 | |
| *** yfried|afk is now known as yfried_ | 08:27 | |
| *** lizk has joined #openstack-neutron | 08:27 | |
| *** ivar-lazzaro has joined #openstack-neutron | 08:28 | |
| *** topshare has joined #openstack-neutron | 08:28 | |
| *** salv-orlando has joined #openstack-neutron | 08:31 | |
| *** ivar-lazzaro has quit IRC | 08:32 | |
| *** baoli has joined #openstack-neutron | 08:35 | |
| *** lykinsbd has joined #openstack-neutron | 08:35 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 08:37 |
| *** hanzhang has quit IRC | 08:37 | |
| *** baoli has quit IRC | 08:39 | |
| *** lykinsbd has quit IRC | 08:39 | |
| *** apuimedo has joined #openstack-neutron | 08:41 | |
| *** yfried_ is now known as yfried|afk | 08:42 | |
| *** dims has joined #openstack-neutron | 08:42 | |
| *** jlanoux has joined #openstack-neutron | 08:46 | |
| *** aranjan has joined #openstack-neutron | 08:47 | |
| *** bobmel has joined #openstack-neutron | 08:48 | |
| *** dims has quit IRC | 08:48 | |
| *** matrohon has joined #openstack-neutron | 08:49 | |
| *** criptoverso has joined #openstack-neutron | 08:51 | |
| *** aranjan has quit IRC | 08:52 | |
| lizk | mestery ++ please, :) | 08:55 |
| lizk | @ https://review.openstack.org/#/c/163454/ | 08:55 |
| *** yfried|afk is now known as yfried_ | 08:55 | |
| *** yalie has quit IRC | 08:56 | |
| *** vthapar has joined #openstack-neutron | 08:58 | |
| *** jpena has joined #openstack-neutron | 08:59 | |
| *** boris-42 has joined #openstack-neutron | 08:59 | |
| *** pradipta has quit IRC | 08:59 | |
| *** rotbeard has joined #openstack-neutron | 09:02 | |
| *** jistr has joined #openstack-neutron | 09:04 | |
| *** topshare has quit IRC | 09:04 | |
| *** topshare has joined #openstack-neutron | 09:05 | |
| *** yfried_ is now known as yfried|afk | 09:05 | |
| *** markvoelker has joined #openstack-neutron | 09:06 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 09:07 |
| *** topshare has quit IRC | 09:09 | |
| *** lykinsbd has joined #openstack-neutron | 09:10 | |
| *** ygbo has joined #openstack-neutron | 09:11 | |
| *** markvoelker has quit IRC | 09:11 | |
| *** lykinsbd has quit IRC | 09:14 | |
| *** blues-man has joined #openstack-neutron | 09:18 | |
| *** apuimedo has quit IRC | 09:19 | |
| *** salv-orlando has quit IRC | 09:21 | |
| *** salv-orlando has joined #openstack-neutron | 09:21 | |
| enikanorov__ | kevinbenton: ping | 09:22 |
| kevinbenton | enikanorov__: pong | 09:22 |
| enikanorov__ | hope it's not tooooo late for you :) | 09:22 |
| enikanorov__ | kevinbenton: do you know any method how automatically initiate a nested transaction? | 09:22 |
| enikanorov__ | by 'automatically' i mean that depending on whether it is indeed wrapped by other transaction or not - it creates a proper savepoint | 09:23 |
| *** lykinsbd has joined #openstack-neutron | 09:24 | |
| kevinbenton | enikanorov__: no, i don't | 09:24 |
| enikanorov__ | ok... i'm trying to avoid inventing a wheel | 09:25 |
| *** spradeepv has quit IRC | 09:25 | |
| kevinbenton | enikanorov__: definitely makes sense | 09:25 |
| kevinbenton | enikanorov__: well let me make sure i understand | 09:25 |
| kevinbenton | what would be the difference if one savepoint was already created? | 09:25 |
| *** yfried|afk is now known as yfried_ | 09:26 | |
| kevinbenton | wouldn't you want the second call to create a savepoint either way? | 09:26 |
| *** apuimedo has joined #openstack-neutron | 09:27 | |
| enikanorov__ | ok, let me explain the motivation. so when you're working with a transaction and you trying to commit something and you get an exception (integrity or something) - if that transaction is rapped by some outer-transaction, then whole thing will roll back. so to avoid this rollback, you need inner one to be nested to not have this negative effect | 09:27 |
| enikanorov__ | next thing, if this inner tx is is separate method - you don't know whether it is really inner (nested), or just a regular transaction | 09:28 |
| *** xchat has joined #openstack-neutron | 09:28 | |
| enikanorov__ | sqla doesn't seem to automatically create such transaction | 09:28 |
| *** blues-man has quit IRC | 09:28 | |
| enikanorov__ | so right now it's solved by providing a flag to a method, whether it is called from within a tx, or not | 09:29 |
| enikanorov__ | not a good way... | 09:29 |
| kevinbenton | i see | 09:29 |
| *** xchat is now known as blues-man | 09:31 | |
| *** blues-man has quit IRC | 09:31 | |
| *** blues-man has joined #openstack-neutron | 09:31 | |
| enikanorov__ | so i'm thinking about writing my own tx maker to be used as 'with make_tx(session):' but wanted to know if there is something available already. or maybe i'm just missing some sqla capability | 09:31 |
| *** lykinsbd has quit IRC | 09:32 | |
| openstackgerrit | Mathieu Rohon proposed openstack/neutron: ML2: manage wrong host in get_device_details RPC call https://review.openstack.org/163178 | 09:32 |
| kevinbenton | enikanorov__: check out the second code example in this section http://docs.sqlalchemy.org/en/rel_0_9/orm/session_transaction.html#using-savepoint | 09:32 |
| kevinbenton | enikanorov__: isn't that already the desired behavior? | 09:32 |
| kevinbenton | enikanorov__: the inner transaction exception only rollsback the inner transaction | 09:33 |
| enikanorov__ | kevinbenton: let me read through | 09:33 |
| kevinbenton | enikanorov__: ok, i'm referring to the "for record in records:" code sample | 09:34 |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 09:34 |
| enikanorov__ | kevinbenton: yes, so this begin_nested() could only be used withing parent transaction | 09:36 |
| *** yfried_ is now known as yfried|afk | 09:36 | |
| enikanorov__ | basically i need a method which would either start that 'parent' OR create nested (e.g. savepoint) | 09:36 |
| kevinbenton | enikanorov__: oooooh | 09:36 |
| kevinbenton | enikanorov__: i see now :) | 09:36 |
| kevinbenton | enikanorov__: i didn't realized begin_nested() didn't work without a parent one | 09:37 |
| kevinbenton | enikanorov__: yeah, i'm not aware of a method to do that | 09:38 |
| enikanorov__ | ok, thanks :) | 09:39 |
| *** ZZelle has joined #openstack-neutron | 09:43 | |
| *** kawa2014 has quit IRC | 09:46 | |
| *** scheuran has quit IRC | 09:47 | |
| *** aranjan has joined #openstack-neutron | 09:48 | |
| *** rotbeard has quit IRC | 09:50 | |
| *** dims has joined #openstack-neutron | 09:50 | |
| *** gildub has joined #openstack-neutron | 09:50 | |
| *** aranjan has quit IRC | 09:52 | |
| *** scheuran has joined #openstack-neutron | 09:59 | |
| *** igordcard_ has joined #openstack-neutron | 09:59 | |
| *** moha_hunt has quit IRC | 10:00 | |
| *** pradip_vedams has quit IRC | 10:01 | |
| *** john-davidge has joined #openstack-neutron | 10:02 | |
| openstackgerrit | Merged openstack/neutron: Fix netns-cleanup broken by ProcessMonitor refactor https://review.openstack.org/163401 | 10:03 |
| openstackgerrit | enikanorov proposed openstack/neutron: Handle DBDuplicateError exception properly when creating default sg https://review.openstack.org/164463 | 10:03 |
| *** vmtrooper has joined #openstack-neutron | 10:04 | |
| *** ajo has joined #openstack-neutron | 10:05 | |
| *** markvoelker has joined #openstack-neutron | 10:07 | |
| *** claudiub has joined #openstack-neutron | 10:08 | |
| enikanorov__ | kevinbenton: if for some wierd reason you're still not sleeping, you may take a look at ^^^ | 10:09 |
| *** vmtrooper has quit IRC | 10:09 | |
| enikanorov__ | :) | 10:09 |
| *** markvoelker has quit IRC | 10:12 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: IPsec strongSwan driver implemention https://review.openstack.org/144391 | 10:12 |
| *** yfried|afk is now known as yfried_ | 10:16 | |
| *** sinh has quit IRC | 10:20 | |
| *** yalie has joined #openstack-neutron | 10:22 | |
| *** lazy_prince has quit IRC | 10:25 | |
| *** yfried_ is now known as yfried|afk | 10:26 | |
| *** YorikSar has joined #openstack-neutron | 10:28 | |
| *** killer_prince has joined #openstack-neutron | 10:29 | |
| *** killer_prince is now known as lazy_prince | 10:29 | |
| *** popw has quit IRC | 10:29 | |
| *** achanda has joined #openstack-neutron | 10:31 | |
| *** yuanying has quit IRC | 10:31 | |
| *** claudiub has quit IRC | 10:32 | |
| ZZelle | enikanorov__, i propose to use a less broad try/except in https://review.openstack.org/164463 | 10:33 |
| *** topshare has joined #openstack-neutron | 10:33 | |
| *** pc_m has joined #openstack-neutron | 10:33 | |
| *** hanzhang has joined #openstack-neutron | 10:34 | |
| enikanorov__ | ZZelle: replied | 10:35 |
| *** topshare has quit IRC | 10:35 | |
| *** pask81 has quit IRC | 10:35 | |
| *** achanda has quit IRC | 10:36 | |
| *** pask81 has joined #openstack-neutron | 10:38 | |
| *** yalie has quit IRC | 10:42 | |
| openstackgerrit | Aman Kumar proposed openstack/neutron: ovs agent resync may miss port remove event https://review.openstack.org/165023 | 10:45 |
| openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 10:46 |
| *** aranjan has joined #openstack-neutron | 10:48 | |
| *** blues-man has quit IRC | 10:52 | |
| *** coolsvap is now known as coolsvap|afk | 10:53 | |
| *** aranjan has quit IRC | 10:53 | |
| *** kawa2014 has joined #openstack-neutron | 10:54 | |
| *** stanzgy has quit IRC | 10:56 | |
| *** blues-man has joined #openstack-neutron | 10:56 | |
| *** stanzgy_ has quit IRC | 10:56 | |
| *** blues-man has quit IRC | 10:57 | |
| *** blues-man has joined #openstack-neutron | 10:57 | |
| *** shadyabh1 has quit IRC | 11:00 | |
| *** Bsony has joined #openstack-neutron | 11:00 | |
| *** Bsony has quit IRC | 11:00 | |
| *** Bsony has joined #openstack-neutron | 11:01 | |
| *** markvoelker has joined #openstack-neutron | 11:08 | |
| *** blues-man has quit IRC | 11:11 | |
| *** imcsk8 has quit IRC | 11:14 | |
| *** markvoelker has quit IRC | 11:14 | |
| *** imcsk8 has joined #openstack-neutron | 11:14 | |
| *** claudiub has joined #openstack-neutron | 11:17 | |
| openstackgerrit | enikanorov proposed openstack/neutron: Handle DBDuplicateError exception properly when creating default sg https://review.openstack.org/164463 | 11:17 |
| *** blues-man has joined #openstack-neutron | 11:22 | |
| *** britthouser has joined #openstack-neutron | 11:24 | |
| *** yalie has joined #openstack-neutron | 11:24 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: WIP:Avoid assigned port as a second gateway to an existing router https://review.openstack.org/165031 | 11:26 |
| *** blues-man has quit IRC | 11:28 | |
| *** Marga_ has joined #openstack-neutron | 11:29 | |
| *** blues-man has joined #openstack-neutron | 11:35 | |
| *** trinaths has quit IRC | 11:36 | |
| *** trinaths has joined #openstack-neutron | 11:37 | |
| openstackgerrit | Aman Kumar proposed openstack/neutron: ovs agent resync may miss port remove event https://review.openstack.org/165023 | 11:38 |
| *** markvoelker has joined #openstack-neutron | 11:38 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: WIP:Avoid assigned port as a second gateway to an existing router https://review.openstack.org/165031 | 11:39 |
| *** SridharG has left #openstack-neutron | 11:41 | |
| *** baoli has joined #openstack-neutron | 11:45 | |
| *** blues-man has quit IRC | 11:46 | |
| *** nmagnezi has joined #openstack-neutron | 11:46 | |
| *** baoli has quit IRC | 11:47 | |
| *** baoli has joined #openstack-neutron | 11:47 | |
| *** trinaths has quit IRC | 11:47 | |
| *** blues-man has joined #openstack-neutron | 11:48 | |
| *** blues-man has quit IRC | 11:48 | |
| *** blues-man has joined #openstack-neutron | 11:48 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 11:48 |
| *** igordcard has quit IRC | 11:48 | |
| *** igordcard has joined #openstack-neutron | 11:49 | |
| *** aranjan has joined #openstack-neutron | 11:49 | |
| *** adelinatuvenie has joined #openstack-neutron | 11:50 | |
| *** pal has quit IRC | 11:50 | |
| *** gildub has quit IRC | 11:50 | |
| *** baoli has quit IRC | 11:50 | |
| *** Bhavya has quit IRC | 11:51 | |
| *** nmagnezi has quit IRC | 11:51 | |
| *** puck has quit IRC | 11:52 | |
| *** vmtrooper has joined #openstack-neutron | 11:53 | |
| *** flwang1 has quit IRC | 11:53 | |
| *** aranjan has quit IRC | 11:54 | |
| *** puck has joined #openstack-neutron | 11:56 | |
| *** flwang has joined #openstack-neutron | 11:56 | |
| *** Longgeek has quit IRC | 11:58 | |
| *** vmtrooper has quit IRC | 11:58 | |
| openstackgerrit | Merged openstack/neutron: OVS UT: Remove useless return_value for setup_integration_br https://review.openstack.org/161100 | 11:59 |
| *** amotoki has quit IRC | 11:59 | |
| adelinatuvenie | I have a quick question. I have a module that only has from ... import * in it and this causes pep8 gate to fail, even tough I added #noqa at the end of the line, I still get [34munused-wildcard-import]. What can I do about this? | 12:03 |
| *** yamamoto has joined #openstack-neutron | 12:04 | |
| *** coolsvap|afk is now known as coolsvap | 12:04 | |
| *** oomichi_ has quit IRC | 12:06 | |
| *** topshare has joined #openstack-neutron | 12:07 | |
| openstackgerrit | Merged openstack/neutron: Move mlnx agent to be under ml2/drivers/mlnx https://review.openstack.org/162693 | 12:08 |
| *** topshare has quit IRC | 12:09 | |
| pc_m | adelinatuvenie: The expectation is that we don't use wildcard (*) imports. You can modify your code to import only the items needed. | 12:09 |
| *** topshare has joined #openstack-neutron | 12:09 | |
| pc_m | from the module. | 12:09 |
| *** topshare has quit IRC | 12:09 | |
| *** ihrachyshka has joined #openstack-neutron | 12:10 | |
| *** apuimedo has quit IRC | 12:13 | |
| *** topshare has joined #openstack-neutron | 12:13 | |
| *** radez_g0n3 is now known as radez | 12:13 | |
| *** sudipto has quit IRC | 12:13 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Functional tests for OVS-agent OpenFlow drivers https://review.openstack.org/164584 | 12:16 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 12:16 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 12:16 |
| *** dims has quit IRC | 12:16 | |
| *** amuller has joined #openstack-neutron | 12:17 | |
| *** dims has joined #openstack-neutron | 12:17 | |
| *** jaypipes has joined #openstack-neutron | 12:20 | |
| *** SridharG has joined #openstack-neutron | 12:26 | |
| *** jistr is now known as jistr|class | 12:26 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 12:27 |
| *** salv-orlando has quit IRC | 12:30 | |
| *** chandankumar has quit IRC | 12:33 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Ml2 Mechanism Driver for OVSvApp Solution https://review.openstack.org/160247 | 12:33 |
| *** blues-man has quit IRC | 12:33 | |
| *** shaohe_feng1 has quit IRC | 12:34 | |
| *** vthapar has quit IRC | 12:40 | |
| *** Marga_ has quit IRC | 12:42 | |
| *** dane_leblanc has joined #openstack-neutron | 12:43 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS DVR UT: Remove an inappropriate str() conversion https://review.openstack.org/161524 | 12:44 |
| *** puck has quit IRC | 12:44 | |
| *** flwang has quit IRC | 12:45 | |
| *** Marga_ has joined #openstack-neutron | 12:46 | |
| openstackgerrit | shihanzhang proposed openstack/neutron: Use iptables zone to seperate different conntrack https://review.openstack.org/118274 | 12:46 |
| *** banix has joined #openstack-neutron | 12:48 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Functional tests for OVS-agent OpenFlow drivers https://review.openstack.org/164584 | 12:48 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 12:48 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 12:48 |
| *** flwang has joined #openstack-neutron | 12:48 | |
| *** puck has joined #openstack-neutron | 12:49 | |
| *** topshare has quit IRC | 12:49 | |
| *** aranjan has joined #openstack-neutron | 12:50 | |
| *** jckasper has joined #openstack-neutron | 12:54 | |
| *** aranjan has quit IRC | 12:55 | |
| mestery | otherwiseguy: Looks like a legit failure in in the functional job for https://review.openstack.org/#/c/152761/ | 12:57 |
| *** jpena is now known as jpena|lunch | 13:00 | |
| *** salv-orlando has joined #openstack-neutron | 13:00 | |
| *** lykinsbd has joined #openstack-neutron | 13:02 | |
| *** topshare has joined #openstack-neutron | 13:04 | |
| *** ChuckC has quit IRC | 13:04 | |
| *** ChuckC_ has joined #openstack-neutron | 13:04 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: L2 agent RPC add bulk calls for device up and device down https://review.openstack.org/147435 | 13:04 |
| *** shwetaap has joined #openstack-neutron | 13:05 | |
| mestery | HenryG: The situation is growing dire for these: https://review.openstack.org/#/q/topic:bp/mtu-selection-and-advertisement,n,z | 13:06 |
| mestery | HenryG: Does rob plan to re-spin the one at the top of the pile ASAP? | 13:06 |
| mestery | HenryG: I'd really like to see those and the VLAN NFV one land today if possible :) | 13:06 |
| *** ChuckC has joined #openstack-neutron | 13:07 | |
| *** ChuckC_ has quit IRC | 13:07 | |
| HenryG | mestery: Let me check with him | 13:08 |
| mestery | HenryG: thank you! | 13:08 |
| *** julim has joined #openstack-neutron | 13:09 | |
| *** amotoki has joined #openstack-neutron | 13:09 | |
| *** rohara1 has quit IRC | 13:09 | |
| mestery | salv-orlando: What's the status of https://blueprints.launchpad.net/neutron/+spec/better-quotas? I see the reservation patch is WIP now, are you still working on this one? | 13:09 |
| *** kbringard has joined #openstack-neutron | 13:10 | |
| mestery | salv-orlando: Also, for some weird reason, the subnet pool patch I tried to merge depends on this one (https://review.openstack.org/#/c/134339/), can you merge that one if you feel it's ok? | 13:10 |
| *** igordcard_ has quit IRC | 13:10 | |
| *** ijw has joined #openstack-neutron | 13:11 | |
| *** ijw has quit IRC | 13:11 | |
| *** ijw has joined #openstack-neutron | 13:12 | |
| *** ChuckC has quit IRC | 13:12 | |
| *** topshare has quit IRC | 13:13 | |
| *** jckasper has quit IRC | 13:13 | |
| *** kickinz1 is now known as kickinz1|afk | 13:14 | |
| *** dboik has quit IRC | 13:14 | |
| *** ijw_ has joined #openstack-neutron | 13:15 | |
| *** yatin has quit IRC | 13:15 | |
| *** yamamoto has quit IRC | 13:15 | |
| *** ijw has quit IRC | 13:16 | |
| *** puck has quit IRC | 13:16 | |
| *** changbl has quit IRC | 13:16 | |
| *** flwang has quit IRC | 13:16 | |
| *** topshare has joined #openstack-neutron | 13:17 | |
| *** topshare has quit IRC | 13:17 | |
| *** apuimedo has joined #openstack-neutron | 13:18 | |
| *** flwang has joined #openstack-neutron | 13:18 | |
| *** liusheng has quit IRC | 13:19 | |
| *** liusheng has joined #openstack-neutron | 13:19 | |
| *** busterswt has joined #openstack-neutron | 13:19 | |
| *** topshare has joined #openstack-neutron | 13:20 | |
| *** topshare has quit IRC | 13:20 | |
| *** john-davidge_ has joined #openstack-neutron | 13:22 | |
| *** banix has quit IRC | 13:22 | |
| *** kickinz1|afk is now known as kickinz1 | 13:22 | |
| *** yamamoto has joined #openstack-neutron | 13:22 | |
| *** rook-tmp has quit IRC | 13:22 | |
| *** john-davidge has quit IRC | 13:23 | |
| *** puck has joined #openstack-neutron | 13:24 | |
| *** cference has joined #openstack-neutron | 13:25 | |
| *** topshare has joined #openstack-neutron | 13:25 | |
| *** topshare has quit IRC | 13:26 | |
| *** alexpilotti has joined #openstack-neutron | 13:27 | |
| amotoki | mestery: just FYI, please keep this bug in neutornclient in your mind https://bugs.launchpad.net/python-neutronclient/+bug/1431449 . | 13:30 |
| openstack | Launchpad bug 1431449 in python-neutronclient "list operations fail using Python binding with object of type 'NoneType' has no len()" [Critical,In progress] - Assigned to Anand Shanmugam (anand1712) | 13:30 |
| mestery | amotoki: looking | 13:31 |
| *** salv-orlando has quit IRC | 13:31 | |
| amotoki | mestery: We must fix this bug before the next client release. I will investigate the detail after Kilo-3 cut. | 13:31 |
| mestery | amotoki: Thanks for looking into this! | 13:31 |
| *** alexpilotti has quit IRC | 13:32 | |
| *** geaaru has joined #openstack-neutron | 13:32 | |
| *** alexpilotti has joined #openstack-neutron | 13:32 | |
| amotoki | This breaks a normal library usage of neutronclient completely. | 13:32 |
| *** ChuckC has joined #openstack-neutron | 13:33 | |
| geaaru | hi, when a new neutron-openvswitch-agent is added to a platform where is save mapping between bridge_mapping and hostname (with ml2 plugin) ? is there a table that contains this data ? thanks in advance | 13:33 |
| *** tiswanso has joined #openstack-neutron | 13:35 | |
| *** dboik has joined #openstack-neutron | 13:36 | |
| *** banix has joined #openstack-neutron | 13:36 | |
| *** Murali has quit IRC | 13:36 | |
| amotoki | geaaru: the information is stored in Agents table. See the output of neutron agent-show <agent-id>. configuration attribute contains the information you asked. | 13:36 |
| *** flwang has quit IRC | 13:37 | |
| *** padkrish has joined #openstack-neutron | 13:37 | |
| *** rohara has joined #openstack-neutron | 13:37 | |
| *** flwang has joined #openstack-neutron | 13:38 | |
| *** tiswanso has quit IRC | 13:38 | |
| *** topshare has joined #openstack-neutron | 13:38 | |
| *** topshare has quit IRC | 13:39 | |
| *** tiswanso_ has joined #openstack-neutron | 13:39 | |
| *** moha_hunt has joined #openstack-neutron | 13:41 | |
| *** prad has joined #openstack-neutron | 13:42 | |
| *** vmtrooper has joined #openstack-neutron | 13:42 | |
| *** abehl has quit IRC | 13:42 | |
| *** yamamoto has quit IRC | 13:43 | |
| *** claudiub has quit IRC | 13:44 | |
| *** shwetaap has quit IRC | 13:45 | |
| *** pal has joined #openstack-neutron | 13:46 | |
| *** shwetaap has joined #openstack-neutron | 13:47 | |
| *** vmtrooper has quit IRC | 13:47 | |
| *** shwetaap1 has joined #openstack-neutron | 13:48 | |
| *** zzzeek has joined #openstack-neutron | 13:50 | |
| geaaru | amotoki: ah ok, thanks. I have a strange error on use an a physnet that is available only on a particular host but it seems that this physnet is not usable. If i try to create a new instance this instance is create on wrong host where physnet that i configured is not present. And then if I try to create a router (dvr or not) for an external network on the same physnet i receive an error that said that l3 | 13:50 |
| geaaru | agent is not available. I'm trying to investigate. thanks again. | 13:50 |
| *** aranjan has joined #openstack-neutron | 13:51 | |
| *** shwetaap has quit IRC | 13:51 | |
| openstackgerrit | Brian Bowen proposed openstack/neutron: Fix missing spaces in strings split across lines. https://review.openstack.org/163859 | 13:52 |
| *** banix has quit IRC | 13:52 | |
| *** kickinz1 is now known as kickinz1|afk | 13:53 | |
| openstackgerrit | Mamta Prabhu proposed openstack/neutron: IBM SDN-VE Plugin decomposition https://review.openstack.org/130471 | 13:54 |
| *** kickinz1|afk is now known as kickinz1 | 13:54 | |
| *** FJB has joined #openstack-neutron | 13:55 | |
| *** aranjan has quit IRC | 13:55 | |
| *** yamamoto has joined #openstack-neutron | 13:56 | |
| *** shaohe_feng1 has joined #openstack-neutron | 13:56 | |
| ajo | ping mestery , amuller was reminding me that process monitoring is disabled by default, may be we want to enable it by default? | 13:56 |
| ajo | marun requested to keep it off until we had dhcp functional tests, which is taking longer than expected, it's functionally tested everywhere else, and itself, and the dhcp/pm pieces may be covered by tempest, ... | 13:57 |
| pc_m | amuller: hi | 13:57 |
| ZZelle | mestery, hi | 13:57 |
| ajo | amuller was commenting to do it now, we can loop marun in the patch... to make sure he's aware/could give his opinion, | 13:58 |
| ajo | my question actually is... can we do it yet? (almost the end of Kilo-3) | 13:58 |
| *** topshare has joined #openstack-neutron | 13:59 | |
| openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: Adds FWaaS service plugin for Cisco CSR1kv https://review.openstack.org/152282 | 14:00 |
| *** topshare has quit IRC | 14:00 | |
| *** dboik_ has joined #openstack-neutron | 14:00 | |
| amuller | mestery: ajo: marun: I think it's well tested via functional tests, and if we don't enable it for Kilo we won't get feedback | 14:01 |
| *** kickinz1 is now known as kickinz1|afk | 14:01 | |
| *** kickinz1|afk is now known as kickinz1 | 14:01 | |
| *** topshare has joined #openstack-neutron | 14:02 | |
| *** topshare has quit IRC | 14:02 | |
| *** dboik has quit IRC | 14:03 | |
| *** jckasper has joined #openstack-neutron | 14:03 | |
| *** r-daneel has joined #openstack-neutron | 14:03 | |
| *** topshare has joined #openstack-neutron | 14:04 | |
| *** topshare has quit IRC | 14:04 | |
| *** jistr|class is now known as jistr | 14:05 | |
| ajo | +1 | 14:05 |
| *** jprovazn has joined #openstack-neutron | 14:05 | |
| *** blues-man has joined #openstack-neutron | 14:05 | |
| *** rook_run has quit IRC | 14:07 | |
| *** garyk1 has joined #openstack-neutron | 14:07 | |
| *** garyk has quit IRC | 14:07 | |
| *** rook has joined #openstack-neutron | 14:08 | |
| *** jobewan has joined #openstack-neutron | 14:09 | |
| *** jpena|lunch is now known as jpena | 14:10 | |
| pc_m | amuller: Can you look at https://review.openstack.org/#/c/164466? I'm trying to switch over to the neutron/callback mechanism and used metatdata proxy as example. I think you did some work on the use of previous mechanism. | 14:10 |
| *** padkrish has quit IRC | 14:11 | |
| amuller | pc_m: Maybe you can help me - What do you want to get from the new system that the old one is not providing you? I mean what is the motivation here? | 14:11 |
| *** rushil has joined #openstack-neutron | 14:11 | |
| *** salv-orlando has joined #openstack-neutron | 14:11 | |
| *** salv-orlando has quit IRC | 14:11 | |
| *** lpetrut has joined #openstack-neutron | 14:11 | |
| *** salv-orlando has joined #openstack-neutron | 14:11 | |
| *** padkrish has joined #openstack-neutron | 14:11 | |
| *** topshare has joined #openstack-neutron | 14:12 | |
| *** topshare has quit IRC | 14:12 | |
| pc_m | amuller: Sure. One is that we can eliminate the AdvancedService base class. Another is that we won't have two mechanisms that do the same thing (with the old one very specific). A third is that the service derived classes will be simplified. | 14:13 |
| amuller | pc_m: Can you expand on the third point? Or can I just look at the code | 14:13 |
| *** topshare has joined #openstack-neutron | 14:14 | |
| *** topshare has quit IRC | 14:14 | |
| pc_m | amuller: Right now, the service classes, do three things. Loading of device drivers, implementation of the callback methods, and act as a forwarder for device driver methods that need access to the router. | 14:14 |
| *** tflynn has joined #openstack-neutron | 14:14 | |
| *** ishant has quit IRC | 14:15 | |
| *** thomasem has joined #openstack-neutron | 14:15 | |
| pc_m | amuller: I've eliminated the last of those (for VPN), whereas the device driver just talks to the router directly. | 14:15 |
| *** padkrish_ has joined #openstack-neutron | 14:15 | |
| *** thomasem has quit IRC | 14:15 | |
| *** padkrish has quit IRC | 14:15 | |
| pc_m | amuller: If we remove the callback, then the class can just do the driver loading, and we won't need the class hierarchy at all. | 14:16 |
| *** thomasem has joined #openstack-neutron | 14:16 | |
| pc_m | amuller: I'm also trying to get rid of the AdvanceService classes squirreling away the agent and agent config. | 14:16 |
| *** Rajeev has joined #openstack-neutron | 14:17 | |
| pc_m | amuller: I hope to push out a change to the VPN code today that is another "client" of this AdvancedService mechanism. | 14:17 |
| *** topshare has joined #openstack-neutron | 14:18 | |
| *** topshare has quit IRC | 14:18 | |
| mestery | ajo: It is? We should look into that | 14:21 |
| *** tflynn has quit IRC | 14:21 | |
| mestery | ZZelle: Yo | 14:21 |
| ZZelle | mestery, about https://review.openstack.org/161494, should we target to run metadata proxy with neutron user/group by default or with nobody user/group? | 14:21 |
| pc_m | amuller: Does that make sense? | 14:21 |
| *** tflynn has joined #openstack-neutron | 14:22 | |
| amuller | pc_m: I think so | 14:22 |
| amuller | pc_m: I'm dealing with urgent keepalived related issues today I hope to be able to review again as soon as possible | 14:23 |
| mestery | ZZelle: Looking | 14:23 |
| pc_m | amuller: As long as this new mechanism works for metadata_proxy, it'll be better to use one mechanism for all these callbacks. | 14:23 |
| *** moha_hunt has quit IRC | 14:23 | |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 14:23 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: IPAM reference driver https://review.openstack.org/150485 | 14:23 |
| ajo | mestery, amuller , I can send the patch to enable by default, and we can loop in marun to see if he's ok now | 14:23 |
| pc_m | amuller: Sure. Thanks! I jsut want to make sure this is OK for metadata_proxy, as the commit uses it as the guinea pig :) | 14:23 |
| amuller | ajo: ok | 14:23 |
| ajo | thanks for bringing this up amuller ;) | 14:24 |
| mestery | ajo: Ack | 14:25 |
| *** tflynn has quit IRC | 14:25 | |
| mestery | ZZelle: I'd be inclined to leave it as neutron by default | 14:25 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Introduce get_devices_details_list_and_failed_devices https://review.openstack.org/157839 | 14:27 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: L2 agent RPC add bulk calls for device up and device down https://review.openstack.org/147435 | 14:27 |
| amuller | ajo: yeah I'm just playing around with keepalived and I killed it, smiling to myself that it will magically come back up in a second, and it didn't! | 14:27 |
| amuller | I was surprised :) | 14:27 |
| ajo | :-) amuller | 14:27 |
| *** ronis_ has joined #openstack-neutron | 14:28 | |
| *** topshare has joined #openstack-neutron | 14:29 | |
| *** topshare has quit IRC | 14:29 | |
| *** abehl has joined #openstack-neutron | 14:30 | |
| *** tsg has joined #openstack-neutron | 14:30 | |
| ZZelle | mestery, ok | 14:30 |
| *** ronis has quit IRC | 14:31 | |
| amuller | pc_m: can't you register class methods as callbacks, instead of module level functions? | 14:31 |
| amuller | in https://review.openstack.org/#/c/164466/1/neutron/agent/metadata/driver.py | 14:31 |
| *** rook has quit IRC | 14:31 | |
| *** rotbeard has joined #openstack-neutron | 14:32 | |
| *** garyk1 has quit IRC | 14:32 | |
| *** zz_jgrimm is now known as jgrimm | 14:33 | |
| *** rook-tmp has joined #openstack-neutron | 14:33 | |
| *** achanda has joined #openstack-neutron | 14:34 | |
| *** abehl has quit IRC | 14:34 | |
| *** topshare has joined #openstack-neutron | 14:36 | |
| *** topshare has quit IRC | 14:36 | |
| *** emagana has joined #openstack-neutron | 14:37 | |
| *** tflynn has joined #openstack-neutron | 14:38 | |
| *** radez has left #openstack-neutron | 14:38 | |
| *** Marga_ has quit IRC | 14:38 | |
| *** tflynn has quit IRC | 14:38 | |
| *** topshare has joined #openstack-neutron | 14:39 | |
| pc_m | amuller: Unfortunately, no, not at this time. That is what I tried initially. There is a problem with the weakref dict used and class methods. | 14:39 |
| openstackgerrit | Merged openstack/neutron: Remove router binding with router-interface-delete https://review.openstack.org/163454 | 14:39 |
| *** tflynn has joined #openstack-neutron | 14:39 | |
| amuller | pc_m: that sucks | 14:40 |
| pc_m | amuller: Yeah, I was disappointed. | 14:40 |
| openstackgerrit | Ryan O'Hara proposed openstack/neutron: Fix spelling error in neutron.conf https://review.openstack.org/165091 | 14:41 |
| pc_m | amuller: Able to work around, as the info is passed into the methods. | 14:41 |
| amuller | yeah it's just a bit weird as it's currently proposed | 14:41 |
| *** leenheer has joined #openstack-neutron | 14:41 | |
| pc_m | amuller: Using a class method threw an exception. | 14:41 |
| *** julien_ZTE has joined #openstack-neutron | 14:42 | |
| *** yog__ has quit IRC | 14:42 | |
| openstackgerrit | Merged openstack/neutron: portsecurity_db: Use is_attr_set instead of a home-grown equivalent https://review.openstack.org/162565 | 14:42 |
| *** vishwanathj has joined #openstack-neutron | 14:43 | |
| *** achanda has quit IRC | 14:43 | |
| *** rook has joined #openstack-neutron | 14:44 | |
| *** baoli has joined #openstack-neutron | 14:44 | |
| *** neeti has quit IRC | 14:44 | |
| *** ronis has joined #openstack-neutron | 14:44 | |
| HenryG | mestery: These can go in, IMO: https://review.openstack.org/113339 https://review.openstack.org/161033 https://review.openstack.org/164943 | 14:45 |
| mestery | HenryG: They are failing jenkins and the DVR job, can you look into those? | 14:46 |
| *** absubram has joined #openstack-neutron | 14:46 | |
| HenryG | mestery: yes, obviously I mean if they don't break any jobs they can go in :) | 14:47 |
| *** ronis_ has quit IRC | 14:47 | |
| *** tflynn has quit IRC | 14:47 | |
| mestery | HenryG: lol | 14:47 |
| mestery | HenryG: ebtables is out, FYI | 14:47 |
| mestery | HenryG: It hasn't had reviews and with precious little time I'm going to make some removals now, just FYI | 14:47 |
| amuller | pc_m: reviewed the patch | 14:48 |
| amuller | pc_m: just the testing thing | 14:48 |
| HenryG | mestery: understood | 14:48 |
| *** ronis_ has joined #openstack-neutron | 14:48 | |
| pc_m | amuller: Thank you! | 14:48 |
| amuller | pc_m: ping me when you upload a new revision I'll +1 | 14:48 |
| pc_m | will do! | 14:48 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: OVS agent react to events instead of polling https://review.openstack.org/164880 | 14:49 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add get_events to OVSDB monitor https://review.openstack.org/161806 | 14:49 |
| mestery | marun: Did we decide to remove this from kilo and merge it once RC is cut? https://blueprints.launchpad.net/neutron/+spec/integration-tests | 14:50 |
| *** abehl has joined #openstack-neutron | 14:51 | |
| *** baoli has quit IRC | 14:51 | |
| *** aranjan has joined #openstack-neutron | 14:52 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: WIP:Avoid assigned port as a second gateway to an existing router https://review.openstack.org/165031 | 14:52 |
| *** ronis__ has joined #openstack-neutron | 14:52 | |
| *** ronis has quit IRC | 14:52 | |
| *** tflynn has joined #openstack-neutron | 14:53 | |
| *** baoli has joined #openstack-neutron | 14:53 | |
| mestery | Folks: I'm now moving things out of Kilo with temporary -2s, please be cauitious on what you merge going forward as we hope to cut kilo-3 tomorrow or thursday morning | 14:53 |
| *** vishwana_ has joined #openstack-neutron | 14:54 | |
| *** rook-tmp has quit IRC | 14:55 | |
| *** ronis_ has quit IRC | 14:55 | |
| *** aranjan has quit IRC | 14:56 | |
| *** vishwanathj has quit IRC | 14:56 | |
| *** david-lyle_afk is now known as david-lyle | 14:57 | |
| mestery | marun: On a similar note, this one won't make Kilo either: https://blueprints.launchpad.net/neutron/+spec/retargetable-functional-testing | 14:58 |
| mestery | marun: I think all these testing ones we can merge once we cut the RC branch if they're ready, so I've marked them as targeting "neutron-master" for now. | 14:58 |
| *** tflynn has quit IRC | 14:58 | |
| *** SumitNaiksatam has joined #openstack-neutron | 15:00 | |
| mestery | HenryG: Is this one still on target for today? https://review.openstack.org/#/c/156318/ | 15:00 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 15:00 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 15:00 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: IPAM reference driver https://review.openstack.org/150485 | 15:00 |
| *** yeungp has joined #openstack-neutron | 15:00 | |
| mestery | HenryG: I'd really, really like to see MTU and VLAN NFV make it | 15:00 |
| HenryG | mestery: Yes it is, owner is verifying some tests | 15:01 |
| mestery | HenryG: Sweet! Ping me ASAP once you see it land, lets get it in the queue assuming it passes Jenkins. Thanks! | 15:01 |
| *** leenheer has quit IRC | 15:02 | |
| HenryG | mestery: IMO the ipv6 multiple prefix is more than ready too. It's a bunch of related patches though. And it shouldn't be affecting the dvr job, I am trying to figure out what the failures are there. | 15:03 |
| mestery | HenryG: Please do. | 15:03 |
| *** rook-tmp has joined #openstack-neutron | 15:03 | |
| *** crose has joined #openstack-neutron | 15:04 | |
| *** yalie has quit IRC | 15:04 | |
| *** chandankumar has joined #openstack-neutron | 15:04 | |
| *** julim has quit IRC | 15:04 | |
| *** padkrish_ has quit IRC | 15:05 | |
| *** topshare has quit IRC | 15:05 | |
| *** padkrish has joined #openstack-neutron | 15:05 | |
| *** jaypipes has quit IRC | 15:06 | |
| *** numan has quit IRC | 15:06 | |
| openstackgerrit | Rob Pothier proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 15:07 |
| *** tiswanso_ has quit IRC | 15:07 | |
| *** julim has joined #openstack-neutron | 15:07 | |
| *** rook-tmp has quit IRC | 15:08 | |
| *** padkrish has quit IRC | 15:09 | |
| *** tflynn has joined #openstack-neutron | 15:11 | |
| *** leenheer has joined #openstack-neutron | 15:13 | |
| *** banix has joined #openstack-neutron | 15:13 | |
| *** yamamoto has quit IRC | 15:14 | |
| *** tiswanso has joined #openstack-neutron | 15:14 | |
| *** tiswanso_ has joined #openstack-neutron | 15:15 | |
| *** xgerman has joined #openstack-neutron | 15:16 | |
| *** crc32 has joined #openstack-neutron | 15:17 | |
| *** tiswanso has quit IRC | 15:18 | |
| *** crc32 has quit IRC | 15:18 | |
| *** crc32 has joined #openstack-neutron | 15:19 | |
| *** vivek-ebay has joined #openstack-neutron | 15:20 | |
| *** gongysh has quit IRC | 15:20 | |
| zigo_ | How should I fill bugs against neutron-lbaas? | 15:21 |
| zigo_ | Should I use the neutron project in launchpad? | 15:21 |
| mestery | rossella_s: Hi! Is there only a single patch left for this BP: https://blueprints.launchpad.net/neutron/+spec/restructure-l2-agent | 15:21 |
| mestery | zigo_: Yes, please use neutron in LP | 15:21 |
| zigo_ | neutron-lbaas is trying to read neutron.conf from /usr/lib/python2.7/dist-packages/etc/neutron.conf | 15:21 |
| zigo_ | Which of course fails when I try to build the package ... :/ | 15:22 |
| zigo_ | mestery: Thanks. | 15:22 |
| *** tflynn has quit IRC | 15:22 | |
| mestery | dougwig blogan: ^^^^^ | 15:22 |
| *** padkrish has joined #openstack-neutron | 15:22 | |
| pc_m | amuller: hi | 15:23 |
| amuller | pc_m: yo | 15:23 |
| *** dannywilson has joined #openstack-neutron | 15:24 | |
| pc_m | On 164466 review, do you think there is an issue possible with the singleton? | 15:24 |
| rossella_s | mestery: hi! the patches are upstream but not merged, I am going to push the last one today | 15:24 |
| *** tidwellr has joined #openstack-neutron | 15:24 | |
| openstackgerrit | Rob Pothier proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 15:24 |
| mestery | rossella_s: OK, we have until today to get it merged or Ill move it out of Kilo, thanks for all your hard work! :) | 15:24 |
| amuller | pc_m: so you added those unregisters because something was not passing I'm assuming | 15:24 |
| pc_m | amuller: no | 15:25 |
| *** anilvenkata has quit IRC | 15:25 | |
| pc_m | amuller: Sorry. Yea, I the two tests conflict. | 15:25 |
| zigo_ | https://bugs.launchpad.net/neutron/+bug/1433146 | 15:26 |
| openstack | Launchpad bug 1433146 in neutron "neutron-lbaas tries to read neutron.conf from /usr" [Undecided,New] | 15:26 |
| amuller | pc_m: So I think it's better to be pre-emptive here (We had issues with the drivers being singletons with the previous notifications system as well), and just call .clear() in the test setUp, both in the l3 agent unit and functonal tests | 15:26 |
| zigo_ | It'd be nice to have this fixed "soon", so that my Debian package can run the unit tests ... | 15:26 |
| pc_m | amuller: One sets the callback, and the other expects there not to be one. So, I resolved by explicitly clearing right befure using. | 15:26 |
| *** chandankumar has quit IRC | 15:26 | |
| openstackgerrit | Phillip Toohill proposed openstack/python-neutronclient: Updates pool session persistence options https://review.openstack.org/163682 | 15:26 |
| pc_m | amuller: Can you point me to where the functional test is? | 15:27 |
| amuller | pc_m: neutron/tests/functional/agent/test_l3_agent | 15:27 |
| rossella_s | mestery: It's hard to get it merged today, the patches didn't get many reviews. Can this blueprint get an exception maybe? | 15:27 |
| pc_m | amuller: thanks. | 15:27 |
| rossella_s | mestery: sorry but I am not very familiar with the procedure | 15:27 |
| amuller | pc_m: I'd put it in L3AgentTestFramework.setUp | 15:27 |
| mestery | rossella_s: I'll look into it and see if we can get some folks reviewign in more detaiul, thanks! | 15:28 |
| *** crc32 has quit IRC | 15:28 | |
| amuller | pc_m: because there's a lot of different test clases that inherit from that one | 15:28 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 15:28 |
| openstackgerrit | Mohammad Banikazemi proposed openstack/neutron: IBM SDN-VE Plugin decomposition https://review.openstack.org/130471 | 15:29 |
| pc_m | amuller: good to know. thanks! | 15:29 |
| mestery | otherwiseguy: I'm getting nervous about https://blueprints.launchpad.net/neutron/+spec/vsctl-to-ovsdb | 15:29 |
| mestery | amuller: I don't think this one will make it in Kilo, thoughts: https://blueprints.launchpad.net/neutron/+spec/report-ha-router-master | 15:30 |
| amuller | mestery: It will with an FEE *assuming* that Maru and Carl are able to dedicate time to this. I'ts more up to them than me. | 15:31 |
| *** vmtrooper has joined #openstack-neutron | 15:31 | |
| amuller | FFE* | 15:31 |
| *** Marga_ has joined #openstack-neutron | 15:31 | |
| mestery | amuller: OK, let me see what we can do, I know some of those patches have landed yet. Thanks! | 15:31 |
| *** changbl has joined #openstack-neutron | 15:32 | |
| *** gongysh has joined #openstack-neutron | 15:35 | |
| *** sridhar_ram1 has joined #openstack-neutron | 15:35 | |
| *** padkrish has quit IRC | 15:35 | |
| *** vmtrooper has quit IRC | 15:35 | |
| *** padkrish has joined #openstack-neutron | 15:36 | |
| *** shwetaap1 has quit IRC | 15:36 | |
| *** sridhar_ram has quit IRC | 15:36 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy to log with nobody user/group https://review.openstack.org/161494 | 15:37 |
| *** vivek-ebay has quit IRC | 15:37 | |
| *** sripriya has joined #openstack-neutron | 15:38 | |
| *** tiswanso_ has quit IRC | 15:38 | |
| *** tiswanso has joined #openstack-neutron | 15:39 | |
| *** tiswanso has quit IRC | 15:39 | |
| *** gongysh has quit IRC | 15:39 | |
| *** yfried|afk is now known as yfried_ | 15:40 | |
| *** ronis_ has joined #openstack-neutron | 15:40 | |
| *** padkrish has quit IRC | 15:40 | |
| openstackgerrit | Sripriya Seetharam proposed openstack/neutron: Brocade Vyatta vrouter shim plugin for vendor decomposition https://review.openstack.org/159056 | 15:42 |
| *** _cjones_ has joined #openstack-neutron | 15:42 | |
| *** adelinatuvenie has quit IRC | 15:42 | |
| *** ronis__ has quit IRC | 15:43 | |
| *** tsg has quit IRC | 15:43 | |
| *** dane_leblanc_ has joined #openstack-neutron | 15:43 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy running with nobody user/group https://review.openstack.org/165115 | 15:44 |
| *** dane_leblanc has quit IRC | 15:45 | |
| dougwig | mestery: ty | 15:45 |
| mestery | dougwig: ??? | 15:46 |
| dougwig | the bug shout out above | 15:46 |
| mestery | dougwig: ah, yes, yw | 15:46 |
| *** mlavalle has joined #openstack-neutron | 15:46 | |
| *** yamahata has joined #openstack-neutron | 15:49 | |
| *** yfried_ is now known as yfried|afk | 15:50 | |
| *** prad has quit IRC | 15:51 | |
| *** prad has joined #openstack-neutron | 15:51 | |
| *** ajmiller has joined #openstack-neutron | 15:52 | |
| *** tiswanso has joined #openstack-neutron | 15:53 | |
| *** tiswanso has quit IRC | 15:53 | |
| *** tiswanso has joined #openstack-neutron | 15:53 | |
| openstackgerrit | Kyle Mestery proposed openstack/neutron-specs: Add liberty directory for neutron-specs https://review.openstack.org/165116 | 15:54 |
| otherwiseguy | mestery: There is an issue with the ovsdb stuff where things return before vswitchd actually creates interfaces out-of-band, so when we then try to do something with the interface immediately after calling add_bridge/add_port, it can fail. Trying to fix now. | 15:54 |
| mestery | openstackgerrit: Ack. thanks for the update! | 15:54 |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Enable Process Monitor by default. https://review.openstack.org/165117 | 15:56 |
| ajo | amuller, mestery ^ | 15:56 |
| *** ronis has joined #openstack-neutron | 15:56 | |
| *** Marga_ has quit IRC | 15:56 | |
| *** liusheng has quit IRC | 15:57 | |
| *** yfried|afk is now known as yfried_ | 15:57 | |
| amuller | ajo: do you need a docimpact when you change the default of a value? I think so | 15:57 |
| amuller | or is that picked up automatically | 15:57 |
| ajo | amuller, no if it's a newly introduced one for this version? | 15:57 |
| zigo_ | Where may I find the Cisco plugin code for Neutron? | 15:57 |
| ajo | not sure, it could be | 15:57 |
| ajo | amuller, point it out in the review, and we can check with others, thanks for looking at it ;) | 15:58 |
| *** Marga_ has joined #openstack-neutron | 15:58 | |
| *** gongysh has joined #openstack-neutron | 15:58 | |
| *** liusheng has joined #openstack-neutron | 15:58 | |
| amuller | I wrote it in the review | 15:58 |
| *** matrohon has quit IRC | 15:58 | |
| *** ronis_ has quit IRC | 15:59 | |
| *** ijw_ has quit IRC | 15:59 | |
| anteaya | mestery: can you review https://review.openstack.org/#/c/164390/4 this adds dougwig's non-voting job to neutron check | 16:00 |
| *** jorgem has joined #openstack-neutron | 16:01 | |
| *** pask811 has joined #openstack-neutron | 16:01 | |
| *** shwetaap has joined #openstack-neutron | 16:02 | |
| lpetrut | Hi guys. We're planning on adding network QoS support on Hyper-V. Do you think it would be ok to use nova flavor extra specs for this? | 16:02 |
| amuller | emagana: Ping about DocImpact | 16:02 |
| *** nlahouti has joined #openstack-neutron | 16:02 | |
| *** localloop127 has joined #openstack-neutron | 16:03 | |
| *** Marga_ has quit IRC | 16:03 | |
| *** pask81 has quit IRC | 16:03 | |
| *** rook-tmp has joined #openstack-neutron | 16:04 | |
| *** pask811 has quit IRC | 16:04 | |
| *** leenheer has quit IRC | 16:05 | |
| *** Longgeek has joined #openstack-neutron | 16:06 | |
| *** padkrish has joined #openstack-neutron | 16:06 | |
| *** amotoki has quit IRC | 16:06 | |
| *** yamamoto has joined #openstack-neutron | 16:07 | |
| *** fredlhsu has joined #openstack-neutron | 16:07 | |
| *** dane_leblanc_ has quit IRC | 16:07 | |
| openstackgerrit | Mamta Prabhu proposed openstack/neutron: IBM SDN-VE Plugin decomposition https://review.openstack.org/130471 | 16:08 |
| mestery | atn: done | 16:08 |
| mestery | anteaya: done | 16:09 |
| *** baoli has quit IRC | 16:09 | |
| *** pask81 has joined #openstack-neutron | 16:09 | |
| *** dboik_ has quit IRC | 16:09 | |
| anteaya | mestery: good timing | 16:10 |
| anteaya | mestery: and thanks | 16:10 |
| *** Longgeek has quit IRC | 16:10 | |
| *** chandankumar has joined #openstack-neutron | 16:11 | |
| HenryG | zigo_: which cisco plugin? There are many. Or are you looking for the splitted-out code? | 16:11 |
| anteaya | mestery: actually can you +1 again, dougwig submitted patchset 6 while you +1'd patchset 5 | 16:11 |
| anteaya | https://review.openstack.org/#/c/164390/ | 16:11 |
| mestery | done | 16:12 |
| zigo_ | HenryG: Yeah, I'm looking for the splitted-out code which now produces some unit tests failures when I build the Debian Neutron package which I maintain. | 16:12 |
| *** padkrish has quit IRC | 16:12 | |
| *** absubram has quit IRC | 16:13 | |
| HenryG | zigo_: https://git.openstack.org/stackforge/networking-cisco mirrored at https://github.com/stackforge/networking-cisco | 16:14 |
| *** yfried_ is now known as yfried|afk | 16:14 | |
| zigo_ | HenryG: Cheers! | 16:14 |
| HenryG | zigo_: if you want to file bugs, https://bugs.launchpad.net/networking-cisco | 16:15 |
| zigo_ | Ok. | 16:15 |
| *** sridhar_ram1 has left #openstack-neutron | 16:15 | |
| *** julien_ZTE has quit IRC | 16:15 | |
| *** julien_ZTE has joined #openstack-neutron | 16:15 | |
| zigo_ | HenryG: I may start filling bugs after b3 is released! :) | 16:15 |
| zigo_ | I'm currently packaging all this week from trunk, waiting for the b3 tag. | 16:16 |
| *** criptoverso has quit IRC | 16:16 | |
| zigo_ | I've done keystone, nova, and ceilometer so far. | 16:16 |
| zigo_ | Neutron is next... | 16:17 |
| *** Murali has joined #openstack-neutron | 16:17 | |
| HenryG | Neutron will be "interesting" I am sure | 16:17 |
| zigo_ | Right, because of so many changes. | 16:18 |
| zigo_ | On Juno, clearly, Horizon was the project which gave me the more work, because of splitting out all the javascript. | 16:18 |
| *** ronis has quit IRC | 16:18 | |
| zigo_ | HenryG: Do I need to only package a python-networking-cisco and that's it? Or is there a daemon or something for the Cisco stuff? | 16:19 |
| HenryG | zigo_: Unfortunately because it contains so many drivers from different teams, I personally don't actually know. :( | 16:21 |
| *** salv-orlando has quit IRC | 16:21 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: DVR qrouters not created on compute Node when vm added https://review.openstack.org/149143 | 16:22 |
| *** gongysh has quit IRC | 16:22 | |
| HenryG | zigo_: Can you give an example of a package that also needs a separate daemon included? | 16:22 |
| *** vivek-ebay has joined #openstack-neutron | 16:22 | |
| zigo_ | HenryG: neutron-lbaas needs to have a neutron-lbaas-agent daemon running... | 16:22 |
| zigo_ | Hum... | 16:23 |
| *** crc32 has joined #openstack-neutron | 16:23 | |
| zigo_ | It has imported all the tags from Neutron, but it's looking like it's in fact version 0.0.1 | 16:24 |
| zigo_ | Weird... | 16:24 |
| *** yfried|afk is now known as yfried_ | 16:24 | |
| * zigo_ shoots a mail to openstack-networking@cisco.com | 16:24 | |
| dougwig | zigo_: anything i can help with on the lbaas front? | 16:24 |
| HenryG | zigo_: yes, that is best | 16:25 |
| zigo_ | dougwig: Well, yes! I have just filed a bug against neutron, because neutron-lbaas is trying to read the neutron.conf from /usr/lib/python2.7/dist-packages/etc/neutron/neutron.conf | 16:25 |
| zigo_ | dougwig: I've tried to grep in the code, and didn't find where to fix that ... | 16:25 |
| openstackgerrit | Brian Bowen proposed openstack/neutron: Fix missing spaces in strings split across lines. https://review.openstack.org/163859 | 16:25 |
| zigo_ | dougwig: So I disabled unit tests, which is obviously not what I want to do. | 16:26 |
| zigo_ | dougwig: Also, having neutron-lbaas build-depend on python-neutron, and neutron build-depends on python-neutron-lbaas is far from best. It's the chicken and egg here ... | 16:26 |
| dougwig | zigo_: weird. (CC blogan) | 16:26 |
| *** julien_ZTE has quit IRC | 16:27 | |
| dougwig | yes, the implicit dependency thing sucks, but why does neutron depend on neutron-lbaas? | 16:27 |
| *** julien_ZTE has joined #openstack-neutron | 16:27 | |
| *** jaypipes has joined #openstack-neutron | 16:27 | |
| dougwig | i was hoping blogan would chime in on the unit tests, but the unit base class should be setting up oslo.config to look in the right spot. what environment are you doing this in? | 16:28 |
| openstackgerrit | Claudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 16:29 |
| blogan | dougwig, zigo_: reading backlog | 16:29 |
| zigo_ | dougwig: It needs it for unit tests. | 16:29 |
| dougwig | zigo_: right, but jenkins and tox run those without issue, so i'm wondering where the difference is. | 16:30 |
| zigo_ | dougwig: I'm building my package using sbuild, no tox involved, just subunit + testr. | 16:30 |
| zigo_ | No pip ... | 16:30 |
| zigo_ | Just my build-depends and build-depends-indep. | 16:30 |
| *** Swami has joined #openstack-neutron | 16:31 | |
| zigo_ | dougwig: The "it works for me" or "it works in the gate" usually doesn't apply to the issues I report! :) | 16:31 |
| *** garyk has joined #openstack-neutron | 16:31 | |
| *** pal has quit IRC | 16:31 | |
| dougwig | right, just trying to narrow it down. you might be the first to use raw setup.py here. | 16:31 |
| blogan | zigo_: the neutron-lbaas unit tests inhert from neutron unit tests so a lot of the setup didn't have to be duplicated, including the config setup | 16:31 |
| *** julien_ZTE has quit IRC | 16:31 | |
| *** absubram has joined #openstack-neutron | 16:32 | |
| ihrachyshka | zigo, why do you have the file packaged there at all? | 16:32 |
| dougwig | neutron-lbaas *should* have neutron in its requirements.txt file, but doesn't for a variety of sane and crazy reasons. are you accounting for that? | 16:32 |
| zigo_ | blogan: I tried to build-depends-indep: on neutron-server, just to make sure that /etc/neutron/neutron.conf is there, but it was the same, it didn't fix the issue, and neutron-lbaas tried to read neutron.conf from /usr | 16:32 |
| ihrachyshka | zigo, oslo.config has some heuristics to find config files, maybe that's why it tries to read from there. | 16:32 |
| zigo_ | ihrachyshka: Which file? | 16:33 |
| ihrachyshka | zigo, /usr/lib/python2.7/dist-packages/etc/neutron/neutron.conf | 16:33 |
| openstackgerrit | Michael Smith proposed openstack/neutron: delay fip namespace cleanup/destroy https://review.openstack.org/151758 | 16:33 |
| zigo_ | ihrachyshka: Of course I don't have such file, neutron.conf is in /etc, not in /usr ! | 16:33 |
| zigo_ | Which is the issue here... | 16:33 |
| *** packet has joined #openstack-neutron | 16:33 | |
| ihrachyshka | zigo, you haven't posted a bug number (?) | 16:33 |
| *** Bsony has quit IRC | 16:33 | |
| zigo_ | I just did above. | 16:34 |
| zigo_ | Let me find it ... | 16:34 |
| * ihrachyshka is late for the dinner... | 16:34 | |
| *** scheuran has quit IRC | 16:34 | |
| zigo_ | https://bugs.launchpad.net/neutron/+bug/1433146 | 16:34 |
| openstack | Launchpad bug 1433146 in neutron "neutron-lbaas tries to read neutron.conf from /usr" [Undecided,New] | 16:34 |
| zigo_ | dougwig: Got my log? :) | 16:35 |
| *** yamamoto has quit IRC | 16:36 | |
| *** absubram has quit IRC | 16:36 | |
| dougwig | zigo_: i'm still not clear on why you need a circular dependency, though? | 16:36 |
| dougwig | i don't think neutron requires neutron-lbaas. | 16:36 |
| zigo_ | dougwig: Because otherwise, it spits "plugin not found" ... | 16:36 |
| zigo_ | (lots of errors) | 16:37 |
| dougwig | really...?? | 16:37 |
| dougwig | that's a bug. | 16:37 |
| zigo_ | Yeah... | 16:37 |
| zigo_ | Ah. | 16:37 |
| zigo_ | Good to know. | 16:37 |
| zigo_ | I can file it then. | 16:37 |
| dougwig | can you file that? | 16:37 |
| dougwig | ok, thanks. | 16:37 |
| zigo_ | Yup. | 16:37 |
| zigo_ | Will do. | 16:37 |
| ihrachyshka | dougwig, well, you still have server bits in *lbaas? | 16:37 |
| dougwig | yes, but they're not required to run neutron proper. | 16:37 |
| openstackgerrit | Darragh O'Reilly proposed openstack/neutron: lb-agent: use 'replace' instead of 'add' with 'bridge fdb' https://review.openstack.org/165137 | 16:37 |
| zigo_ | Just let me finish the networking-cisco package, re-run a neutron build, then I'll try removing the python-neutron-lbaas build-depends and see again... | 16:38 |
| dougwig | i think that dependency arrow can go one direction. | 16:38 |
| ihrachyshka | dougwig, but if a user ran lbaas on Juno, they NEED to get lbaas on upgrade, otherwise they are broken | 16:38 |
| blogan | dougwig: doesn't it depend on it to import it? | 16:38 |
| ihrachyshka | and since packagers don't know who used which setup, we are safer to just always install it | 16:38 |
| zigo_ | ihrachyshka: a build-depends is not the same at all as a simple Depends: | 16:38 |
| blogan | dougwig: in the NeutronMods package? | 16:38 |
| dougwig | ihrachyshka: that is definitely true. though i'd separate that from the technical dependency. | 16:38 |
| *** melwitt has joined #openstack-neutron | 16:39 | |
| zigo_ | ihrachyshka: I'll make something like: Package: python-neutron-lbaas Depends: python-neutron | 16:39 |
| dougwig | i.e. there is a definite upgrade dependency. there should not be a technical dependency. if that makes sense. i am pre-caffeine. | 16:39 |
| ihrachyshka | zigo, "neutron must build-depend on neutron-lbaas" ?? why | 16:39 |
| ihrachyshka | dougwig, that's correct. so RDO will probably have circular deps for Kilo only | 16:40 |
| zigo_ | ihrachyshka: Re-read above, I just explained. There's lots of unit tests errors otherwise. | 16:40 |
| dougwig | blogan: no, it handles the not installed case fine. i think the plugin thing might be because the service plugin list is hard-coded? | 16:40 |
| blogan | dougwig: nvm i see why that doesnt matter | 16:40 |
| *** SumitNaiksatam has quit IRC | 16:40 | |
| ihrachyshka | zigo, unit tests for neutron failing without lbaas installed? that's just wrong. | 16:40 |
| *** enikanorov has quit IRC | 16:40 | |
| ihrachyshka | zigo, every neutron dev successfully runs them without lbaas | 16:40 |
| *** fredlhsu has quit IRC | 16:40 | |
| *** enikanorov has joined #openstack-neutron | 16:41 | |
| blogan | yeah that sounds not right | 16:41 |
| ihrachyshka | zigo, for etc/neutron.conf, I guess it's because of a bug in test_db_plugin that makes it require /etc/neutron/neutron.conf to be installed | 16:42 |
| tidwellr | rosella_s: carl_baldwin mentioned you might be able to point me at a good example of locking DB query, any chance he's right :) | 16:42 |
| ihrachyshka | that should not be the case, and probably can be fixed in neutron | 16:42 |
| *** baoli has joined #openstack-neutron | 16:43 | |
| ihrachyshka | zigo, or maybe it should be fixed in the lbaas test suite to avoid the requirement. in any case, there is no big reason to make it a build dep. | 16:43 |
| zigo_ | The networking-cisco is doing the same thing: ConfigFilesNotFoundError: Failed to find some config files: /usr/lib/python2.7/dist-packages/etc/neutron.conf | 16:44 |
| zigo_ | :( | 16:45 |
| ihrachyshka | zigo, so you should install neutron.conf as part of your build env for *aas packages. what's the problem with that? | 16:45 |
| zigo_ | ihrachyshka: I'm *not* going to install a configuration file, as root, into /usr. | 16:46 |
| *** rotbeard has quit IRC | 16:46 | |
| zigo_ | ihrachyshka: That's simply not possible. | 16:46 |
| *** aranjan has joined #openstack-neutron | 16:46 | |
| ihrachyshka | zigo_, don't you build packages in mocking env? | 16:46 |
| zigo_ | ihrachyshka: Packages are built as user, and they don't have root access, they can only build-depends. | 16:46 |
| zigo_ | ihrachyshka: I'm using sbuild, which does a clean chroot, but this doesn't mean that I can do whatever I want. | 16:47 |
| *** thomasem has quit IRC | 16:47 | |
| *** yamamoto has joined #openstack-neutron | 16:47 | |
| zigo_ | The package should respect the Debian policy and such. | 16:47 |
| zigo_ | Clearly, installing neutron.conf in /usr/lib/python2.7/dist-packages/etc is *not* a good idea ! :) | 16:47 |
| ihrachyshka | zigo, it is NOT searching for the file there if you install it in proper location | 16:48 |
| ihrachyshka | zigo, it's just that oslo.config, failing to find it in proper location, checks some more | 16:48 |
| zigo_ | ihrachyshka: Well, the file is there, in /etc/neutron/neutron.conf, I assure you. | 16:48 |
| *** absubram has joined #openstack-neutron | 16:48 | |
| ihrachyshka | I guess dist-packages one is the last one checked | 16:48 |
| zigo_ | Hum... | 16:48 |
| zigo_ | Let me check then. | 16:48 |
| *** chandankumar has quit IRC | 16:48 | |
| zigo_ | I'll try to do this in a schroot session, not using sbuild. | 16:49 |
| *** FJB has quit IRC | 16:49 | |
| *** sparky43786 has quit IRC | 16:49 | |
| ihrachyshka | building packages against system is just wrong. I'm surprised debian requires it to work. | 16:49 |
| *** haleyb has quit IRC | 16:49 | |
| *** harlowja_away is now known as harlowja | 16:49 | |
| *** haleyb has joined #openstack-neutron | 16:50 | |
| *** padkrish has joined #openstack-neutron | 16:50 | |
| *** yfried_ is now known as yfried|afk | 16:50 | |
| *** yamamoto has quit IRC | 16:50 | |
| *** haleyb has quit IRC | 16:50 | |
| ihrachyshka | can be relevant to what you experience: https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L521 | 16:50 |
| *** absubram has quit IRC | 16:50 | |
| *** haleyb has joined #openstack-neutron | 16:51 | |
| *** absubram has joined #openstack-neutron | 16:51 | |
| *** rook-tmp has quit IRC | 16:52 | |
| *** yatin has joined #openstack-neutron | 16:52 | |
| *** alexpilotti has quit IRC | 16:53 | |
| *** salv-orlando has joined #openstack-neutron | 16:53 | |
| tidwellr | salv-orlando: ping | 16:54 |
| *** lpetrut has quit IRC | 16:59 | |
| openstackgerrit | Merged openstack/neutron: If providers exist in neutron.conf, don't look in services conf https://review.openstack.org/157967 | 17:00 |
| openstackgerrit | Merged openstack/neutron-fwaas: Adds FWaaS service plugin for Cisco CSR1kv https://review.openstack.org/152282 | 17:00 |
| *** xgerman has quit IRC | 17:01 | |
| *** xgerman has joined #openstack-neutron | 17:01 | |
| *** abehl has quit IRC | 17:01 | |
| *** ParsectiX has joined #openstack-neutron | 17:01 | |
| zigo_ | ilyashakhat: I've been packaging OpenStack since Cactus, and doing Debian packages since maybe 2008, so I'm quite sure I'm doing things right, and that "building packages against system" as you say, is the right thing to do ! :) | 17:02 |
| zigo_ | ilyashakhat: We just use some throwable system using sbuild or cowbuilder (the generic name is "pbuilder"). | 17:03 |
| *** Bsony has joined #openstack-neutron | 17:03 | |
| *** yfried|afk is now known as yfried_ | 17:03 | |
| *** ygbo has quit IRC | 17:03 | |
| zigo_ | sbuild is what is used in the buildd network in Debian. | 17:03 |
| *** sbfox has joined #openstack-neutron | 17:03 | |
| *** vthapar has joined #openstack-neutron | 17:03 | |
| *** safchain has quit IRC | 17:04 | |
| *** jlanoux has quit IRC | 17:04 | |
| *** _cjones_ has quit IRC | 17:05 | |
| mestery | rossella_s: I'm testing out your entire stack of L2 agent changes now with devstack. | 17:05 |
| *** dane_leblanc has joined #openstack-neutron | 17:06 | |
| *** ajmiller_ has joined #openstack-neutron | 17:06 | |
| *** ajmiller has quit IRC | 17:07 | |
| *** jorgem has quit IRC | 17:08 | |
| salv-orlando | hello tidwellr | 17:08 |
| *** chandankumar has joined #openstack-neutron | 17:08 | |
| tidwellr | hi | 17:09 |
| *** GumBall has joined #openstack-neutron | 17:10 | |
| tidwellr | so, subnet allocation...... | 17:10 |
| *** dboik has joined #openstack-neutron | 17:10 | |
| *** GumBall has left #openstack-neutron | 17:10 | |
| *** coolsvap is now known as coolsvap|afk | 17:11 | |
| *** _cjones_ has joined #openstack-neutron | 17:11 | |
| salv-orlando | tidwellr: yes... | 17:11 |
| ihrachyshka | zigo, well, I don't know. I probably still miss some context here, specifically, how you avoid installing anything (like python deps) into your system before running unit tests. | 17:11 |
| tidwellr | I'm exploring options for doing some DB-level locking, any ideas or insight? | 17:11 |
| ihrachyshka | I will check the bug again tomorrow, now need to leave | 17:11 |
| *** yeungp has quit IRC | 17:11 | |
| salv-orlando | in the data model prefixes are just a string attribute of the subnetpool | 17:11 |
| zigo_ | ilyashakhat: I do install all python deps, just not using pip / tox. | 17:11 |
| *** nati_ueno has joined #openstack-neutron | 17:12 | |
| salv-orlando | perhaps we can have a row-level write intent lock on that table. | 17:12 |
| *** gsagie_ has joined #openstack-neutron | 17:12 | |
| zigo_ | Just using what's defined in debian/control for a given package. | 17:12 |
| ihrachyshka | zigo, you trigger wrong nickname | 17:12 |
| *** arosen1 has joined #openstack-neutron | 17:12 | |
| *** _cjones_ has quit IRC | 17:12 | |
| *** leenheer has joined #openstack-neutron | 17:12 | |
| salv-orlando | tidwellr: I would like to use memcached or any other backend for doing some basic distributed lock. But as you know it's too late in the cycle to introduce new things | 17:12 |
| *** gsagie_ is now known as gsagie | 17:12 | |
| ihrachyshka | zigo, right. so if you install python deps, then there is no problem installing another package that would ship config files. | 17:12 |
| salv-orlando | so we have to put up with the techniques we've used so far. Hence, SELECT... FOR UPDATE | 17:13 |
| ihrachyshka | (though I agree it's not optimal and maybe should be fixed in neutron too) | 17:13 |
| *** yeungp has joined #openstack-neutron | 17:13 | |
| zigo_ | ilyashakhat: Which is why I made neutron-lbaas build-depends on neutron-common. | 17:13 |
| tidwellr | salv-orland: right, I'm thinking that if we can just lock in the allocations table, we should be protected | 17:13 |
| ihrachyshka | zigo, you still trigger a wrong nickname. whatever, I'll get back to the bug tomorrow. | 17:13 |
| *** Murali has quit IRC | 17:13 | |
| zigo_ | Woops, sorry ! :( | 17:14 |
| ihrachyshka | zigo_, ok, and that's the correct thing to do. with config file installed, it should work. | 17:14 |
| zigo_ | I just use [tab] too early. | 17:14 |
| *** jistr has quit IRC | 17:14 | |
| zigo_ | ihrachyshka: I'm trying again now, in a cowbuilder, I'll be able to tell in a minute. | 17:14 |
| *** _cjones_ has joined #openstack-neutron | 17:15 | |
| ihrachyshka | zigo, ack, thanks. please report in the bug. | 17:15 |
| *** yeungp has quit IRC | 17:15 | |
| *** s3wong has joined #openstack-neutron | 17:15 | |
| zigo_ | ihrachyshka: Same stuff, so I confirm what I reported. | 17:15 |
| zigo_ | ihrachyshka: Nothing more to add to the bug report, it's accurate. | 17:16 |
| *** yeungp has joined #openstack-neutron | 17:16 | |
| zigo_ | I do have /etc/neutron/neutron.conf, but for some reason, something tries to get it from /usr/lib/python2.7/dist-packages/etc/neutron | 17:16 |
| *** kawa2014 has quit IRC | 17:16 | |
| *** yeungp has quit IRC | 17:16 | |
| sripriya | mestery: Hi | 17:16 |
| *** yeungp has joined #openstack-neutron | 17:16 | |
| mestery | sripriya: yo | 17:16 |
| zigo_ | Also, I get some stuff like this: | 17:17 |
| zigo_ | File "neutron_lbaas/db/loadbalancer/loadbalancer_db.py", line 23, in <module> | 17:17 |
| zigo_ | from neutron.openstack.common import log as logging | 17:17 |
| zigo_ | ImportError: cannot import name log | 17:17 |
| *** abehl has joined #openstack-neutron | 17:17 | |
| salv-orlando | tidwellr: I think this is the only option we have at this stage, honestly. there are concerns over use of write-intent locks | 17:17 |
| salv-orlando | but it's better to have that rather than not having anything | 17:17 |
| zigo_ | I guess this wouldn't work since Neutron switched to oslo.log ... | 17:17 |
| sripriya | mestery: Brocade Vyatta CI logs are ready for the patchset at: https://review.openstack.org/#/c/159056/ | 17:17 |
| *** chandankumar has quit IRC | 17:18 | |
| tidwellr | salv-orlando: agreed, any chance you have a pointer to a retry on deadlock decorator? | 17:19 |
| zigo_ | from oslo_log import log as logging is the correct thing to do these days. | 17:19 |
| *** ihrachyshka has quit IRC | 17:19 | |
| *** natarajk has joined #openstack-neutron | 17:19 | |
| mestery | sripriya: Cool! Lets let armax or HenryG look at it too and get it in the merge queue. | 17:19 |
| *** vmtrooper has joined #openstack-neutron | 17:20 | |
| HenryG | sripriya: mestery: looking at it now | 17:20 |
| sripriya | mestery: HenryG: Thanks very much! | 17:20 |
| mestery | sripriya HenryG: Thanks! | 17:20 |
| amuller | wow, just got a python stack trace while running the dvr functional tests, fun | 17:21 |
| *** aepifanov has quit IRC | 17:21 | |
| tidwellr | salv-orlando: btw I'll refactor the usage of the session in the SubnetAllocator | 17:21 |
| *** armax has joined #openstack-neutron | 17:21 | |
| *** salv-orl_ has joined #openstack-neutron | 17:21 | |
| mestery | amuller: Buy a lottery ticket, it's your lucky day :) | 17:22 |
| salv-orl_ | tidwellr: there is one in oslo_db | 17:22 |
| salv-orl_ | I think it's called wrap_db_retry | 17:22 |
| salv-orl_ | oslo_db.api if I'm not wrong | 17:23 |
| tidwellr | cool, I'll take a look | 17:23 |
| *** dane_leblanc has quit IRC | 17:23 | |
| *** salv-orlando has quit IRC | 17:25 | |
| *** vmtrooper has quit IRC | 17:25 | |
| sripriya | HenryG: Thank you very much for the quick review. | 17:25 |
| *** abehl has quit IRC | 17:26 | |
| HenryG | sripriya: np, I was already tracking it | 17:26 |
| *** jbonjean has quit IRC | 17:26 | |
| amuller | the DVR functional tests used to take 50 seconds each, now they take 5 seconds on the same machine | 17:26 |
| enikanorov | salv-orl_: tidwellr: i don't think using wrap_db_retry is a good idea to hangle deadlocks | 17:26 |
| amuller | I don't know if this is because of the rootwrap work or maybe something is horribly wrong | 17:27 |
| *** localloo1 has joined #openstack-neutron | 17:27 | |
| openstackgerrit | Bradley Jones proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 17:29 |
| openstackgerrit | Bradley Jones proposed openstack/neutron: Advertise mtu over dhcp https://review.openstack.org/155874 | 17:29 |
| tidwellr | enikanorov: any thoughts on an alternative approach? | 17:30 |
| enikanorov | tidwellr: retries, but without locking. update failure should be detected differently | 17:30 |
| *** leenheer has quit IRC | 17:30 | |
| *** localloop127 has quit IRC | 17:31 | |
| tidwellr | enikanorov: well, we're seeing not seeing an alternative to locking in this situation | 17:31 |
| tidwellr | enikanorov: at least given the time constraints | 17:32 |
| enikanorov | tidwellr: why is that? try-and-fail is a generic alternative to serialization of operations | 17:32 |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes https://review.openstack.org/154921 | 17:32 |
| enikanorov | given time constraints we may just skip commititng multiple api/rpc workers patch | 17:32 |
| *** packet has quit IRC | 17:33 | |
| *** achanda has joined #openstack-neutron | 17:34 | |
| *** Networkn3rd has joined #openstack-neutron | 17:34 | |
| *** localloo1 is now known as localloop127 | 17:34 | |
| *** natarajk has left #openstack-neutron | 17:34 | |
| *** chandankumar has joined #openstack-neutron | 17:35 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Avoid assigned port as a second gateway to an existing router https://review.openstack.org/165031 | 17:35 |
| *** ronis has joined #openstack-neutron | 17:36 | |
| tidwellr | enikanorov: I've got my wires crossed too much happening in my brain at the moment, yes you're right and I believe you've done some work in this area? | 17:36 |
| enikanorov | tidwellr: i've done some work that may be considered adjacent :) | 17:37 |
| tidwellr | :) | 17:37 |
| *** pask81 has quit IRC | 17:38 | |
| *** nati_ueno has quit IRC | 17:38 | |
| *** nati_uen_ has joined #openstack-neutron | 17:38 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes https://review.openstack.org/154921 | 17:38 |
| openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 17:38 |
| openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Add some unit tests for strongswan driver https://review.openstack.org/165159 | 17:38 |
| tidwellr | enikanorov: the problem we're trying to avoid is concurrent updates to the subnetallocations table | 17:39 |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 17:39 |
| openstackgerrit | Rob Pothier proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 17:39 |
| tidwellr | enikanorov: as written, I have some patches that don't protect that table and could result in duplicate allocations of subnets, very bad | 17:40 |
| *** natarajk has joined #openstack-neutron | 17:41 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 17:41 |
| enikanorov | tidwellr: that may be handled by DB constraint, no? | 17:41 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 17:42 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Don't delete HA router primary VIP on agent restarts https://review.openstack.org/164841 | 17:42 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Enable pre-emption for L3 HA https://review.openstack.org/165162 | 17:42 |
| *** reed has joined #openstack-neutron | 17:42 | |
| salv-orl_ | enikanorov: is this about "write intent locks considered harmful" | 17:42 |
| salv-orl_ | because they don't work on multimasters? | 17:42 |
| amuller | marun: kevinbenton: New patches up | 17:43 |
| *** afazekas has quit IRC | 17:43 | |
| *** tflynn has joined #openstack-neutron | 17:43 | |
| openstackgerrit | Merged openstack/neutron: Fix spelling error in neutron.conf https://review.openstack.org/165091 | 17:43 |
| *** yatin has quit IRC | 17:44 | |
| tidwellr | enikanorov: DB constraint won't help here since we can optionally allow overlapping subnets in a pool | 17:44 |
| enikanorov | tidwellr: i think ip allocation could be easily contrained (against a subnet, for instance) | 17:45 |
| salv-orl_ | tidwellr: perhaps we can drop that requirement. I suggested that and carl_baldwin is onboard with it. But still do not forget that it is not simple to enforce a constraint about two cidrs not being overlapping | 17:45 |
| enikanorov | salv-orl_: right, but i'm just trying to not be religious | 17:45 |
| tidwellr | salv-orlando: agreed, i was just about to suggest that this is our trigger for pulling support of allow_overlap | 17:46 |
| *** angela-s has joined #openstack-neutron | 17:46 | |
| salv-orl_ | enikanorov: aaaah you'll go to hell if you're not religious ;) | 17:46 |
| enikanorov | salv-orl_: i know, I'll find nice ppl there :D | 17:46 |
| salv-orl_ | tidwellr: that's no trigger imho.. even with a primary key constraint how would you enforce non overlapping cidrs? | 17:47 |
| salv-orl_ | you can enforce unique cidrs... but 10.0.0.0/24 and 10.0.0.0/25 will be ok for the constraint | 17:47 |
| tidwellr | salv-orlando: right, got a little excited there | 17:48 |
| *** rook-tmp has joined #openstack-neutron | 17:48 | |
| *** chandankumar has quit IRC | 17:50 | |
| *** ParsectiX has quit IRC | 17:51 | |
| *** rook-tmp has quit IRC | 17:52 | |
| *** blues-man has quit IRC | 17:52 | |
| *** lykinsbd has quit IRC | 17:52 | |
| *** ParsectiX has joined #openstack-neutron | 17:52 | |
| *** ParsectiX has quit IRC | 17:53 | |
| *** ParsectiX has joined #openstack-neutron | 17:53 | |
| openstackgerrit | Paul Michali proposed openstack/neutron: Refactoring of L3 agent notifications for router https://review.openstack.org/164466 | 17:54 |
| pc_m | amuller: ^^^ | 17:54 |
| *** lykinsbd has joined #openstack-neutron | 17:55 | |
| *** csoukup has joined #openstack-neutron | 17:55 | |
| amuller | pc_m: added a comment | 17:56 |
| pc_m | amuller: checking... | 17:56 |
| *** natarajk has left #openstack-neutron | 17:57 | |
| *** nati_uen_ has quit IRC | 17:57 | |
| *** jbonjean has joined #openstack-neutron | 17:57 | |
| *** nati_ueno has joined #openstack-neutron | 17:57 | |
| *** baoli has quit IRC | 18:01 | |
| *** nati_ueno has quit IRC | 18:02 | |
| *** alexpilotti has joined #openstack-neutron | 18:02 | |
| *** adelinatuvenie has joined #openstack-neutron | 18:03 | |
| openstackgerrit | Rob Pothier proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 18:03 |
| *** thomasem has joined #openstack-neutron | 18:03 | |
| *** banix has quit IRC | 18:04 | |
| *** shwetaap has quit IRC | 18:04 | |
| *** chandankumar has joined #openstack-neutron | 18:04 | |
| *** sripriya has quit IRC | 18:05 | |
| *** sripriya has joined #openstack-neutron | 18:05 | |
| *** baoli_ has joined #openstack-neutron | 18:05 | |
| *** vivek-ebay has quit IRC | 18:06 | |
| *** aix has quit IRC | 18:06 | |
| *** vivek-ebay has joined #openstack-neutron | 18:06 | |
| openstackgerrit | Bradley Jones proposed openstack/neutron: Advertise mtu over dhcp https://review.openstack.org/155874 | 18:06 |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes https://review.openstack.org/154921 | 18:07 |
| *** vishwana_ has quit IRC | 18:08 | |
| *** tiswanso has quit IRC | 18:08 | |
| *** vishwanathj has joined #openstack-neutron | 18:08 | |
| *** jprovazn has quit IRC | 18:10 | |
| *** nati_ueno has joined #openstack-neutron | 18:10 | |
| *** jpena is now known as jpena|away | 18:11 | |
| *** yfried_ is now known as yfried|afk | 18:12 | |
| *** vishwanathj has quit IRC | 18:13 | |
| *** pritesh has joined #openstack-neutron | 18:13 | |
| *** sbfox has quit IRC | 18:13 | |
| *** dboik has quit IRC | 18:14 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Enable pre-emption for L3 HA https://review.openstack.org/165162 | 18:16 |
| *** achanda has quit IRC | 18:16 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Send notification to controller about HA router state change https://review.openstack.org/126188 | 18:16 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 18:16 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 18:16 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 18:16 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 18:16 |
| *** baoli_ has quit IRC | 18:16 | |
| *** chandankumar has quit IRC | 18:16 | |
| *** ParsectiX has quit IRC | 18:16 | |
| *** shwetaap has joined #openstack-neutron | 18:16 | |
| amuller | marun: I pushed the pre-emption patch out of the series | 18:16 |
| marun | amuller: danke | 18:16 |
| *** ParsectiX has joined #openstack-neutron | 18:17 | |
| *** absubram has quit IRC | 18:17 | |
| *** geaaru has quit IRC | 18:18 | |
| amuller | haleyb: Hey Brian | 18:18 |
| amuller | haleyb: How are you? | 18:18 |
| *** yeungp has quit IRC | 18:19 | |
| *** fredlhsu has joined #openstack-neutron | 18:20 | |
| *** yfried|afk is now known as yfried_ | 18:21 | |
| *** achanda has joined #openstack-neutron | 18:21 | |
| *** fredlhsu has quit IRC | 18:21 | |
| *** fredlhsu has joined #openstack-neutron | 18:22 | |
| *** sripriya has quit IRC | 18:23 | |
| *** ijw has joined #openstack-neutron | 18:23 | |
| *** jorgem has joined #openstack-neutron | 18:23 | |
| *** ParsectiX has quit IRC | 18:24 | |
| *** sripriya has joined #openstack-neutron | 18:24 | |
| *** ParsectiX has joined #openstack-neutron | 18:24 | |
| *** tiswanso has joined #openstack-neutron | 18:24 | |
| openstackgerrit | Paul Michali proposed openstack/neutron: Refactoring of L3 agent notifications for router https://review.openstack.org/164466 | 18:27 |
| pc_m | amuller: ^^^ | 18:27 |
| amuller | pc_m: +1 | 18:28 |
| pc_m | Thanks! | 18:28 |
| *** tiswanso has quit IRC | 18:29 | |
| *** aranjan has quit IRC | 18:29 | |
| *** leenheer has joined #openstack-neutron | 18:30 | |
| *** leenheer has quit IRC | 18:31 | |
| rossella_s | mestery: nice, let me know how the tests go! | 18:31 |
| *** absubram has joined #openstack-neutron | 18:31 | |
| *** tiswanso_ has joined #openstack-neutron | 18:31 | |
| *** Marga_ has joined #openstack-neutron | 18:31 | |
| *** tiswanso_ has quit IRC | 18:31 | |
| *** leenheer has joined #openstack-neutron | 18:31 | |
| *** SridharG has left #openstack-neutron | 18:32 | |
| *** vthapar has quit IRC | 18:32 | |
| *** aranjan has joined #openstack-neutron | 18:32 | |
| absubram | hi.. anyone from NEC here? | 18:32 |
| *** yapeng has joined #openstack-neutron | 18:32 | |
| *** tiswanso_ has joined #openstack-neutron | 18:33 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Send notification to controller about HA router state change https://review.openstack.org/126188 | 18:33 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 18:33 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 18:33 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 18:33 |
| absubram | had a quick comment about a change needed for the NEC CI check | 18:33 |
| amuller | marun: addressed some nits in the later patches in the series | 18:33 |
| marun | amuller: ok | 18:34 |
| *** dboik has joined #openstack-neutron | 18:34 | |
| *** baoli has joined #openstack-neutron | 18:34 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 18:35 |
| *** sbfox has joined #openstack-neutron | 18:36 | |
| *** dboik has quit IRC | 18:36 | |
| *** dboik has joined #openstack-neutron | 18:36 | |
| *** baoli_ has joined #openstack-neutron | 18:37 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron-fwaas: Adds FWaaS driver for Cisco CSR1kv https://review.openstack.org/161019 | 18:37 |
| *** lazy_prince is now known as killer_prince | 18:37 | |
| *** gsagie has quit IRC | 18:38 | |
| *** packet has joined #openstack-neutron | 18:39 | |
| *** leenheer has quit IRC | 18:39 | |
| *** baoli has quit IRC | 18:39 | |
| *** dboik has quit IRC | 18:41 | |
| *** dboik has joined #openstack-neutron | 18:41 | |
| *** chandankumar has joined #openstack-neutron | 18:41 | |
| *** vivek-ebay has quit IRC | 18:41 | |
| *** dboik has quit IRC | 18:41 | |
| *** vivek-ebay has joined #openstack-neutron | 18:42 | |
| *** SumitNaiksatam has joined #openstack-neutron | 18:43 | |
| *** dboik has joined #openstack-neutron | 18:43 | |
| *** dboik has quit IRC | 18:45 | |
| *** leenheer has joined #openstack-neutron | 18:45 | |
| *** yfried_ is now known as yfried|afk | 18:45 | |
| *** dboik has joined #openstack-neutron | 18:45 | |
| *** Swami has quit IRC | 18:45 | |
| *** Swami has joined #openstack-neutron | 18:46 | |
| HenryG | haleyb: ping | 18:47 |
| *** jorgem has quit IRC | 18:48 | |
| *** rook-tmp has joined #openstack-neutron | 18:48 | |
| SumitNaiksatam | mestery: dougwig: hi, can I request your blessings on this one: https://review.openstack.org/#/c/161479 (it was held up earlier by the grenade failures, but not related to this patch)? | 18:49 |
| mestery | SumitNaiksatam: Will look today | 18:50 |
| SumitNaiksatam | mestery: thanks, you had +2’ed earlier | 18:50 |
| dougwig | SumitNaiksatam: done | 18:51 |
| SumitNaiksatam | dougwig: thanks! | 18:51 |
| pc_m | SumitNaiksatam: hi | 18:51 |
| SumitNaiksatam | pc_m: hi | 18:51 |
| pc_m | SumitNaiksatam: Two questions for you... | 18:51 |
| SumitNaiksatam | pc_m: sure | 18:51 |
| pc_m | SumitNaiksatam: Did Anne create a doc bug for FWaaS for you? | 18:51 |
| SumitNaiksatam | pc_m: yes | 18:52 |
| pc_m | SumitNaiksatam: If so, what's the number. I'll need to create one for VPN and would like to see what info they want. | 18:52 |
| SumitNaiksatam | pc_m: sure, let me send you the link - | 18:52 |
| *** rook-tmp has quit IRC | 18:53 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 18:54 |
| *** matrohon has joined #openstack-neutron | 18:54 | |
| *** aepifanov has joined #openstack-neutron | 18:55 | |
| pc_m | SumitNaiksatam: Another one is a VPN question, but wondering if you have any thoughts from a FWaaS POV... | 18:55 |
| armax | HenryG: ping | 18:55 |
| *** chandankumar has quit IRC | 18:55 | |
| armax | pc_m: ping | 18:55 |
| HenryG | armax: pong | 18:56 |
| armax | pc_m: any chance you know how to get hold of the author for https://review.openstack.org/#/c/161019/ | 18:56 |
| *** trad511 has joined #openstack-neutron | 18:56 | |
| pc_m | SumitNaiksatam: I see in the VPN agent, they take the L3 agent's conf setting and pass it down to the device driver. Any idea why we'd want to do that, instead of jsut accessing cfg.CONF? | 18:56 |
| armax | the patch looks ready to merge, but I am a confused on a couple of lose ends | 18:56 |
| pc_m | armax: pong. | 18:56 |
| armax | pc_m: can you have a look at https://review.openstack.org/#/c/161019/ PS6 | 18:57 |
| pc_m | armax: YEs, I can IM her. | 18:57 |
| armax | ? | 18:57 |
| armax | pc_m: thanks | 18:57 |
| pc_m | armax: sure | 18:57 |
| yamahata | mestery: ping? | 18:58 |
| mestery | yamahata: pong | 18:58 |
| yamahata | mestery: regarding to https://review.openstack.org/#/c/160051/ you thought it's out of kilo. But it is approved for Kilo. | 18:59 |
| yamahata | as https://blueprints.launchpad.net/neutron/+spec/ml2-ovs-portsecurity | 18:59 |
| pc_m | armax: It's up to PS7. Are there unaddressed comments in PS6 still? | 18:59 |
| armax | yes | 18:59 |
| SumitNaiksatam | pc_m: okay | 18:59 |
| armax | pc_m: I forced PS7 to get it out of the queue | 18:59 |
| yamahata | mestery: Can you please reconsider your -2? | 19:00 |
| mestery | yamahata: No, I mean it won't merge for Kilo, FF is Thursday, and we have way too many things to merge, so I've been taking some things out. | 19:00 |
| SumitNaiksatam | pc_m: you mean in the VPN agent or the FWaaS agent? | 19:00 |
| pc_m | armax: OK. I'll look at it too (not really familiar with FWaaS, but SumitNaiksatam is :) | 19:00 |
| armax | pc_m: I was in the middle of the review when the patch got in the gate | 19:00 |
| armax | then, SumitNaiksatam approved it, but maybe he can shed some light on the question I had | 19:00 |
| *** lykinsbd has quit IRC | 19:00 | |
| SumitNaiksatam | armax: sorry, i did not see that you were reviewing it | 19:00 |
| armax | all the fw drivers available seem to honor the driver signatures, but the cisco driver doesn't | 19:01 |
| armax | SumitNaiksatam: you couldn’t have, no worries | 19:01 |
| yamahata | mestery: okay, where can I see the discussion? neturon-driver? | 19:01 |
| *** lykinsbd has joined #openstack-neutron | 19:01 | |
| *** bobmel has quit IRC | 19:01 | |
| ijw | mestery: post security is out? | 19:01 |
| SumitNaiksatam | armax: the cisco driver actually works with a Cisco FWaaS plugin | 19:01 |
| ijw | port | 19:01 |
| SumitNaiksatam | armax: which is different from the FWaaS plugin which other drivers use | 19:01 |
| pc_m | SumitNaiksatam: I'm speaking of VPN, but was wondering for a service in general. Wondering if there is a need to pass in config vs just using cfg.CONF | 19:02 |
| SumitNaiksatam | armax: hence it works with a different signature | 19:02 |
| mestery | yamahata: We had 37 BPs open this morning which needed merging by Thursday, that one hadn't had many reviews, so it's out. | 19:02 |
| mestery | ijw: yes | 19:02 |
| armax | SumitNaiksatam: so why does in inherit from the same base driver? | 19:02 |
| armax | SumitNaiksatam: it looks suspicious | 19:02 |
| mestery | yamahata: The discussion was me going through the list and making hard choices on what was in and out, I had to make that today as I have another meeting with release manaagement tomorrow to make sure K3 is in shape | 19:02 |
| SumitNaiksatam | armax: i understood as an intention to override | 19:02 |
| yamahata | mestery: I see. thanks. | 19:03 |
| armax | SumitNaiksatam: but it won't | 19:03 |
| mestery | yamahata: I do appreciate your work, and I know this won't make you happy, but part of being PTL is making the hard choices :( | 19:03 |
| yamahata | mestery: thanks for clarification. I really appreciate your hard work for PTL. | 19:04 |
| pc_m | armax: I got ahold of Yanping and asked her to look at the review | 19:04 |
| armax | pc_m: thanks, I’ll eyeball the review | 19:04 |
| mestery | yamahata: Thank you! I do appreciate all your work, and we're going to change the spec/approval process for Liberty to make landing things much easier in neutron | 19:04 |
| mestery | yamahata: Lets connect in Vancouver and make sure we get this stuff in for Liberty :) | 19:05 |
| *** john-davidge_ has quit IRC | 19:05 | |
| *** yfried|afk is now known as yfried_ | 19:05 | |
| *** lykinsbd has quit IRC | 19:05 | |
| *** ivar-lazzaro has joined #openstack-neutron | 19:06 | |
| *** jorgem has joined #openstack-neutron | 19:06 | |
| *** lykinsbd has joined #openstack-neutron | 19:07 | |
| *** shivharis has joined #openstack-neutron | 19:07 | |
| SumitNaiksatam | armax: i believe your point is that the two methods with same names and different signatures represent two different methods? | 19:08 |
| *** Marga_ has quit IRC | 19:08 | |
| *** vmtrooper has joined #openstack-neutron | 19:08 | |
| armax | SumitNaiksatam: yes | 19:09 |
| ijw | mestery: shit. What's the problem with it? I thought it was almost done. | 19:10 |
| SumitNaiksatam | armax: so the agent mode is only relevant for the DVR, and probably the thinking here was that it is never going to exercised with that argument in the case of the cisco plugin/agent/driver configuration | 19:10 |
| mestery | ijw: FF is Thursday and we're backlogged as usual, and it's lower priority than many other things. | 19:10 |
| *** shivharis has quit IRC | 19:10 | |
| *** shivharis has joined #openstack-neutron | 19:10 | |
| mestery | ijw: We're going to get MTU and VLAN NFV in though | 19:11 |
| armax | SumitNaiksatam: so is the caller is going to call the method with one signature vs the other depending on the router mode? | 19:11 |
| SumitNaiksatam | armax: that said, i believe this is an oversight | 19:11 |
| *** signed8bit has joined #openstack-neutron | 19:11 | |
| *** killer_prince has quit IRC | 19:11 | |
| SumitNaiksatam | armax: the signature of the fwaas base driver was updated recently | 19:11 |
| armax | SumitNaiksatam: I think so | 19:12 |
| SumitNaiksatam | armax: and i believe this particular driver has not kept up with that | 19:12 |
| armax | SumitNaiksatam: so it could be that we just need to update the signature on the cisco driver | 19:12 |
| armax | SumitNaiksatam: that’s what I think too | 19:12 |
| armax | SumitNaiksatam: a bad rebase if you will | 19:12 |
| SumitNaiksatam | armax: so yeah, either they should not have inherited, or udated the signature and ignored the agent_mode argument | 19:12 |
| armax | SumitNaiksatam: hence I was seeking confirmation from the author | 19:12 |
| SumitNaiksatam | *updated | 19:12 |
| SumitNaiksatam | armax: yes, good catch on that | 19:12 |
| SumitNaiksatam | armax: i will let the author confirm, but i suspect that is the reason | 19:13 |
| *** ihrachyshka has joined #openstack-neutron | 19:13 | |
| *** vmtrooper has quit IRC | 19:13 | |
| *** killer_prince has joined #openstack-neutron | 19:13 | |
| *** killer_prince is now known as lazy_prince | 19:14 | |
| emagana | amuller: ping! | 19:14 |
| amuller | emagana: yo | 19:14 |
| emagana | amuller: Did you look for me before? | 19:14 |
| *** s3wong has quit IRC | 19:14 | |
| amuller | emagana: I wanted to ask about the 'DocImpact' tag. If you change the default of a config option, do you need that tag? | 19:15 |
| *** yfried_ is now known as yfried|afk | 19:15 | |
| pc_m | armax: SumitNaiksatam: I had a question... in the VPN code, the device driver copies the L3 agent conf (https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/device_drivers/ipsec.py#L516). | 19:15 |
| emagana | Yes, it will impact the config files | 19:15 |
| *** s3wong has joined #openstack-neutron | 19:15 | |
| *** sudipto has joined #openstack-neutron | 19:16 | |
| *** sbfox has quit IRC | 19:16 | |
| emagana | amuller: please, do it. Although you wont need to fix anything, it will automagically updated | 19:16 |
| SumitNaiksatam | pc_m: sorry, i am still hunting that bug in launchpad, i swear i had it open in a tab | 19:16 |
| pc_m | armax: SumitNaiksatam: Am I missing something or can we just use cfg.CONF? Trying to see why it would take the passed in value. | 19:16 |
| pc_m | SumitNaiksatam: np | 19:16 |
| *** lpetrut has joined #openstack-neutron | 19:16 | |
| *** packet has quit IRC | 19:16 | |
| amuller | emagana: I thought the doc people have a script that gathers all of the config options and their default values, regardless of any patches or their DocImpact tags | 19:16 |
| amuller | or the trigger to run that script is a DocImpact patch getting merged? | 19:17 |
| emagana | amuller: They do.. that is what I mean whit automagically | 19:17 |
| amuller | so what's the DocImpact for? | 19:17 |
| emagana | DocImpact will create the bug in launchpad to track the change | 19:17 |
| amuller | I see | 19:17 |
| emagana | amuller: we just make sure the script matches the config updated with the already created bug | 19:18 |
| *** mleroy_ has joined #openstack-neutron | 19:18 | |
| amuller | emagana: gotcha | 19:19 |
| amuller | emagana: Thanks | 19:19 |
| emagana | amuller: n.p. | 19:19 |
| mestery | HenryG: For the MTU and VLAN NFV ones, I told pritesh and company to make sure to post the tempest API tests (in-tree) | 19:19 |
| mestery | HenryG: I'm fine with merging those as long as the tempest test patches are posted, and we can merge those post-freeze. | 19:19 |
| mestery | marun: ^^^^ | 19:19 |
| pritesh | mestery: thanks. | 19:20 |
| *** dboik has quit IRC | 19:20 | |
| HenryG | mestery: ack, thanks | 19:20 |
| mestery | The queue is super deep and getting longer | 19:20 |
| * mestery thinks he'll be up all night merging code. | 19:20 | |
| mestery | At least I'll have armax to virtually commiserate with ;) | 19:20 |
| *** dboik has joined #openstack-neutron | 19:20 | |
| pritesh | mestery: sending in new patch for tempest test and updating older one to remove it now. | 19:20 |
| armax | mestery: today is St Patrick day, right? | 19:21 |
| ihrachyshka | tidwellr, hi! | 19:21 |
| armax | mestery: at least we can get drunk whilst doing that | 19:21 |
| mestery | armax: That means we can virtually share green beers! | 19:21 |
| mestery | armax: Drunk code merging hangout, sweet! :D | 19:21 |
| mestery | pritesh: Awesome! | 19:21 |
| armax | mestery: oh, my we’re gonna break Neutron for good | 19:21 |
| *** aix has joined #openstack-neutron | 19:21 | |
| *** leenheer has quit IRC | 19:21 | |
| armax | pritesh: wait | 19:21 |
| armax | :) | 19:21 |
| armax | pritesh: the VLAN transparent one? | 19:21 |
| *** yamahata has quit IRC | 19:22 | |
| HenryG | mestery: Can you look at the dvr job results through beer googles please? ;) | 19:22 |
| pritesh | armax: yep | 19:22 |
| armax | posted | 19:22 |
| mestery | HenryG: lol | 19:22 |
| armax | pritesh: in a nutshell, I think we should try and see if we can get rid of the unecessary changes made to the tests | 19:22 |
| armax | pritesh: plz have a look and tell me what you think | 19:23 |
| pritesh | armax: yep just saw you posted comments on the changes, checking them out now. | 19:23 |
| armax | pritesh: thanks | 19:23 |
| pritesh | armax: thanks for the review. | 19:23 |
| openstackgerrit | Merged openstack/neutron: Brocade Vyatta vrouter shim plugin for vendor decomposition https://review.openstack.org/159056 | 19:24 |
| *** vivek-ebay has quit IRC | 19:26 | |
| mestery | ijw: port security is back IN for Kilo | 19:26 |
| *** vivek-ebay has joined #openstack-neutron | 19:26 | |
| Swami | HenryG: yes I think we reverted a patch that caused the DVR job to spike up yesterday. | 19:27 |
| Swami | HenryG: is that what you are talking about. | 19:27 |
| *** marun has quit IRC | 19:28 | |
| *** sudipto has quit IRC | 19:28 | |
| HenryG | Swami: I was looking at the dvr job for a slaac change: http://logs.openstack.org/39/113339/26/check/check-tempest-dsvm-neutron-dvr/cb08575/console.html#_2015-03-17_15_39_31_843 | 19:29 |
| *** marun has joined #openstack-neutron | 19:29 | |
| HenryG | armax: ping | 19:30 |
| armax | HenryG: pong | 19:30 |
| *** _cjones_ has quit IRC | 19:30 | |
| HenryG | armax: For new shims, like https://review.openstack.org/155436 , the deadline is not 3/19 is it? | 19:30 |
| *** dboik has quit IRC | 19:31 | |
| HenryG | cc: mestery ^^ | 19:31 |
| armax | HenryG: it depends | 19:31 |
| *** dboik has joined #openstack-neutron | 19:31 | |
| mestery | HenryG: Correct, though the further into RC we get the less likely to land them. | 19:31 |
| armax | that one involve a db migration, so ideally sooner the better | 19:31 |
| armax | HenryG: I’ll review it by the end of today | 19:31 |
| *** tsg_ has joined #openstack-neutron | 19:32 | |
| HenryG | mestery, armax: OK, we need to stage two of them, both involving DB changes :( | 19:32 |
| haleyb | HenryG: pong | 19:32 |
| ihrachyshka | mestery, hey | 19:32 |
| mestery | HenryG: icky | 19:32 |
| mestery | ihrachyshka: yo! | 19:32 |
| ihrachyshka | mestery, regarding ipam interface. since you +2'd the patch, I guess you will be able to answer | 19:33 |
| *** achanda has quit IRC | 19:33 | |
| pc_m | armax: Have a moment for a Q? | 19:33 |
| *** ParsectiX has quit IRC | 19:33 | |
| armax | pc_m: I can try | 19:33 |
| ihrachyshka | mestery, I fail to understand how AnySubnetRequest is expected to be used there. | 19:33 |
| mestery | ihrachyshka: Please go ahead. | 19:33 |
| ihrachyshka | mestery, I came up with a test case http://paste.fedoraproject.org/199013/08960142 | 19:33 |
| ihrachyshka | mestery, that I would expect to work | 19:33 |
| armax | HenryG: so what about the CI on that patch? | 19:33 |
| armax | HenryG: I remember it was pretty close | 19:34 |
| *** ParsectiX has joined #openstack-neutron | 19:34 | |
| pc_m | I posted it above, but there's been a bunch of discussions, so it was prob lost. | 19:34 |
| HenryG | haleyb: was hoping you might be able to help with figuring out the dvr failure for this https://review.openstack.org/113339 | 19:34 |
| ihrachyshka | but tidwellr said it's wrong. though I still fail why it's wrong. can you check it too? | 19:34 |
| mestery | ihrachyshka: Looking now | 19:34 |
| pc_m | armax: In VPN device driver, the save the agent's conf attribute in self.conf. Wondering why we'd need to do that, and not just use cfg.CONF? | 19:34 |
| pc_m | armax: https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/device_drivers/ipsec.py#L516 | 19:34 |
| ihrachyshka | mestery, (comments are in line 143) | 19:34 |
| haleyb | HenryG: i can take a look - consistent failure? | 19:35 |
| HenryG | armax: yes, it works, but it is playing musical chairs with the one for https://review.openstack.org/157243 | 19:35 |
| armax | pc_m: I think that is there to help the functional testing? amuller might be able to tell you more perhaps? | 19:35 |
| *** tgohad has joined #openstack-neutron | 19:36 | |
| pc_m | amuller: any thoughts on ^^^ | 19:36 |
| amuller | sorry I can't find the original question or patch in question | 19:36 |
| ihrachyshka | mestery, it seems weird that we need to pass smth like [netaddr.IPRange('0.0.0.4', '0.0.0.5')] as a pool. why not having a 'any' request from a specific pool? | 19:36 |
| mestery | ihrachyshka: I'm still digesting all the comments on PS40 now, bear with me. | 19:36 |
| armax | pc_m: what’s that in regard? | 19:37 |
| amuller | HenryG: What is Pradeep's nick in here? | 19:37 |
| pc_m | armax: From what I can see, it's only used for things like path to template files and intervals, which don't need to be tweaked. | 19:37 |
| *** matrohon has quit IRC | 19:37 | |
| pc_m | armax: Refactoring and trying to get rid of passing the agent into the device driver (don't need it). Just have these references. | 19:37 |
| armax | pc_m: if you can root where it’s come from, it’ll tell you if you can safely switch to cfg.CONF | 19:38 |
| mestery | ihrachyshka: My understanding according to tidwellr is that the way you wrote the test doesn't match what the API is providing. | 19:38 |
| armax | HenryG: I am not familiar with the expression | 19:38 |
| pc_m | armax: The VPN agent, a child of L3 agent, passes it into the device driver when it is loaded. | 19:39 |
| mestery | ihrachyshka: From your test, you want to specify a range larger than your requesting CIDR, and have the API allocate something in there, right? | 19:39 |
| *** tsg_ has quit IRC | 19:39 | |
| *** yfried|afk is now known as yfried_ | 19:39 | |
| ihrachyshka | mestery, right, that's the intent. 'give me any subnet of specified length from the ranges' | 19:39 |
| amuller | ijw: HenryG: Are Bradley and Pradeep on IRC? | 19:40 |
| * ihrachyshka is not even sure what it would mean to request 'any' subnet from wildcard range | 19:40 | |
| pc_m | armax: It used to be passing the L3 agent, now it passes a VPNService object, which has a reference to the l3-agent, if needed. https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/vpn_service.py#L48 | 19:40 |
| mestery | ihrachyshka: I'm with you there, that would seem to make sense to me. But from what tidwellr has indicated, to get that behavior, you have to specify things differently, which is slightly awkward I think. | 19:40 |
| HenryG | armax: Co-ordinating the changes and the CIs so they work for both shims is hard, until they are both merged | 19:40 |
| ijw | amuller: they may be | 19:40 |
| armax | HenryG: ok | 19:40 |
| armax | HenryG: so, what’s the plan? | 19:41 |
| *** sbfox has joined #openstack-neutron | 19:41 | |
| pc_m | armax: I'm trying to get rid of all that. Looks like the Vyatta VPN device driver needs the VPNService object so that they can call a method on their agent, but I'm hoping they can change it later and we can remove the arg entirely. | 19:41 |
| *** yamahata has joined #openstack-neutron | 19:41 | |
| *** kutija has joined #openstack-neutron | 19:41 | |
| pc_m | armax: I should have a WIP patch that I can post in an hour or so, just rebasing right now. | 19:41 |
| HenryG | armax: getting the patches updated and the CIs rerun to see where things stand | 19:42 |
| *** padkrish has quit IRC | 19:42 | |
| armax | HenryG: ok cool | 19:42 |
| armax | pc_m: ok | 19:42 |
| HenryG | armax: I will ping you, trust me :) | 19:42 |
| amuller | prad: ping | 19:42 |
| *** padkrish has joined #openstack-neutron | 19:42 | |
| pc_m | may make more sense with actual proposed changes. | 19:42 |
| prad | amuller, pong | 19:42 |
| amuller | prad: why did you remove the test here? https://review.openstack.org/#/c/154921/15..19/neutron/tests/unit/test_db_plugin.py | 19:43 |
| prad | amuller, because the config got moved into ml2 and that dint apply here anymore | 19:43 |
| *** jorgem has quit IRC | 19:43 | |
| *** jorgem has joined #openstack-neutron | 19:44 | |
| *** banix has joined #openstack-neutron | 19:44 | |
| prad | amuller, there is no more segement_mtu in neutron.conf its part of the ml2 con and covered in ml2 patch ahead of this | 19:44 |
| *** padkrish_ has joined #openstack-neutron | 19:45 | |
| amuller | prad: I don't see any patch merged or open that covers that? Can you point me to it please? | 19:45 |
| *** nmagnezi has joined #openstack-neutron | 19:45 | |
| amuller | looking here: https://review.openstack.org/#/q/project:openstack/neutron+branch:master+topic:bp/mtu-selection-and-advertisement,n,z | 19:45 |
| *** ParsectiX has quit IRC | 19:46 | |
| *** dane_leblanc has joined #openstack-neutron | 19:46 | |
| *** ParsectiX has joined #openstack-neutron | 19:46 | |
| prad | amuller, https://review.openstack.org/#/c/153733/ and https://review.openstack.org/#/c/156318/ | 19:46 |
| amuller | ok I didn't see https://review.openstack.org/#/c/156318/, it's under the wrong topic =/ | 19:47 |
| *** padkrish has quit IRC | 19:47 | |
| prad | amuller, np | 19:48 |
| *** moha_hunt has joined #openstack-neutron | 19:48 | |
| ihrachyshka | zigo, I think I know what's the reason for your issue. that's actually one of my patches for test suite. I'll look into fixing it this week. thanks for reporting. | 19:48 |
| *** dboik has quit IRC | 19:48 | |
| *** dboik has joined #openstack-neutron | 19:49 | |
| *** rook-tmp has joined #openstack-neutron | 19:49 | |
| *** aranjan has quit IRC | 19:50 | |
| *** pritesh_ has joined #openstack-neutron | 19:50 | |
| *** aranjan has joined #openstack-neutron | 19:51 | |
| *** aranjan has quit IRC | 19:53 | |
| *** aranjan has joined #openstack-neutron | 19:53 | |
| pc_m | mestery: hi | 19:53 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy running with nobody user/group https://review.openstack.org/165115 | 19:53 |
| amuller | prad: I can't find Rob, maybe you know... Here: https://review.openstack.org/#/c/156318/23/neutron/plugins/ml2/plugin.py | 19:53 |
| *** aranjan has quit IRC | 19:53 | |
| *** rook-tmp has quit IRC | 19:54 | |
| amuller | Why is it necessary for create network to call update network? | 19:54 |
| *** pritesh has quit IRC | 19:54 | |
| *** dboik has quit IRC | 19:54 | |
| *** ZZelle_ has joined #openstack-neutron | 19:55 | |
| *** dboik has joined #openstack-neutron | 19:55 | |
| *** trad511 has left #openstack-neutron | 19:55 | |
| *** vishwanathj has joined #openstack-neutron | 19:55 | |
| prad | amuller, hmm perhaps for the default logic to kick in, he might be the best to answer.. might ask on the review? I can poke him to reply | 19:56 |
| *** overlayer has joined #openstack-neutron | 19:56 | |
| amuller | prad: OK I see create_network in the DB base plugin calls _make_network_dict with process_extensions=False | 19:56 |
| amuller | that it would return a network dict without the MTU extension attribute | 19:56 |
| amuller | hrmph, having create_network call update_network might introduce unneeded DB friction | 19:56 |
| *** tiswanso_ has quit IRC | 19:58 | |
| *** gildub has joined #openstack-neutron | 19:58 | |
| *** matrohon has joined #openstack-neutron | 19:59 | |
| *** overlayer has quit IRC | 19:59 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 19:59 |
| *** dboik_ has joined #openstack-neutron | 20:00 | |
| *** igordcard_ has joined #openstack-neutron | 20:00 | |
| *** jpena|away is now known as jpena | 20:00 | |
| pritesh_ | armax: made the change according to your comment, except the long kwargs one, doing it now in next patch set. | 20:01 |
| *** tiswanso_ has joined #openstack-neutron | 20:01 | |
| pritesh_ | mestery: revived the old tempest api patch, will send new patch there, moving the tests to new api directory. | 20:01 |
| *** tsg_ has joined #openstack-neutron | 20:01 | |
| *** dboik has quit IRC | 20:02 | |
| *** tgohad has quit IRC | 20:02 | |
| mestery | pritesh_: thanks! | 20:02 |
| openstackgerrit | pritesh proposed openstack/neutron: Adding Tempest API tests for VLAN Transparency support https://review.openstack.org/164887 | 20:03 |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Api tests to cover network mtu attribute https://review.openstack.org/165201 | 20:03 |
| *** aepifanov has quit IRC | 20:03 | |
| tidwellr | ihrachyshka: See comments https://review.openstack.org/#/c/134339/ in PS40. I think there's a misunderstanding as to what is meant by 'allocation_pools' | 20:04 |
| *** vishwanathj has quit IRC | 20:04 | |
| *** cference has quit IRC | 20:04 | |
| *** melwitt_ has joined #openstack-neutron | 20:04 | |
| *** melwitt has quit IRC | 20:05 | |
| *** sripriya has quit IRC | 20:05 | |
| *** sripriya has joined #openstack-neutron | 20:05 | |
| *** melwitt_ is now known as melwitt | 20:05 | |
| *** absubram has quit IRC | 20:05 | |
| ihrachyshka | tidwellr, aha. so the argument names are indeed misleading me | 20:05 |
| *** carl_baldwin has joined #openstack-neutron | 20:06 | |
| *** vishwanathj has joined #openstack-neutron | 20:06 | |
| *** jorgem has quit IRC | 20:06 | |
| tidwellr | ihrachyshka: sorry for the back-and-forth there, it didn't dawn on me that we were talking past each other until I got some food in my stomach :) | 20:07 |
| ihrachyshka | tidwellr, ok, now I see my failure. I will re-read the patch bearing in mind the proper meaning of the argument. oops. | 20:07 |
| *** fifieldt has quit IRC | 20:08 | |
| *** yfried_ is now known as yfried|afk | 20:08 | |
| pc_m | mestery: ping | 20:09 |
| *** aix has quit IRC | 20:09 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Api tests to cover network mtu attribute https://review.openstack.org/165201 | 20:09 |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Api tests to cover network mtu attribute https://review.openstack.org/165201 | 20:09 |
| openstackgerrit | pritesh proposed openstack/neutron: Adding Tempest API tests for VLAN Transparency support https://review.openstack.org/164887 | 20:10 |
| *** jorgem has joined #openstack-neutron | 20:11 | |
| pc_m | ihrachyshka: ping | 20:11 |
| ihrachyshka | pc_m, pong | 20:11 |
| openstackgerrit | pritesh proposed openstack/neutron: Adding Tempest API tests for VLAN Transparency support https://review.openstack.org/164887 | 20:12 |
| pc_m | ihrachyshka: I'm tryint ot UT some neutron-vpnaas code, and I need, neutron changes that are out for review. To do this, I did a pip install -U /opt/stack/neutron in the .tox/py27 env. to use my local repo's neutron area. | 20:13 |
| *** vishwanathj has quit IRC | 20:13 | |
| *** vishwana_ has joined #openstack-neutron | 20:13 | |
| ihrachyshka | pc_m, right. doesn't it work? | 20:14 |
| pc_m | ihrachyshka: When I run UT, it is trying to load neutron.conf from /opt/stack/neutron-vpnaas/.tox/py27/local/lib/python2.7/site-packages/etc | 20:14 |
| haleyb | HenryG: that failure seems to be due to my ip_lib change, trying to see why my change in the _snat_redirect_remove() calls broke things | 20:14 |
| pc_m | ihrachyshka: Used to work. | 20:14 |
| *** jorgem1 has joined #openstack-neutron | 20:14 | |
| ihrachyshka | pc_m, ok, got the problem. zigo reported it several hours ago. | 20:14 |
| pc_m | ihrachyshka: Saw some talk about that earlier... any idea? | 20:15 |
| ihrachyshka | pc_m, I have a bug assigned to me, and I know what resulted in regression | 20:15 |
| *** ParsectiX has quit IRC | 20:15 | |
| ihrachyshka | pc_m, the bug is https://bugs.launchpad.net/neutron/+bug/1433146 | 20:15 |
| openstack | Launchpad bug 1433146 in neutron "neutron-lbaas tries to read neutron.conf from /usr" [Undecided,New] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 20:15 |
| HenryG | haleyb: thanks for looking. Please keep dane_leblanc and dboik_ in the loop. | 20:15 |
| *** yfried|afk is now known as yfried_ | 20:16 | |
| *** jorgem has quit IRC | 20:16 | |
| pc_m | ihrachyshka: I did this in VPN recently (last week?) and it worked fine. Wondering what changed. Also, know of any workaround? | 20:16 |
| pc_m | ihrachyshka: Could I use the gerrit UUID in requirements.txt? | 20:17 |
| *** rotbeard has joined #openstack-neutron | 20:17 | |
| *** diegows has joined #openstack-neutron | 20:17 | |
| ihrachyshka | pc_m, the patch merged a week ago, so... | 20:17 |
| ihrachyshka | pc_m, re UUID not sure | 20:17 |
| pc_m | ihrachyshka: Ah | 20:18 |
| ihrachyshka | pc_m, the easy workaround is to revert the patch; I'm thinking about better fix | 20:18 |
| pc_m | ihrachyshka: thanks | 20:18 |
| *** achanda has joined #openstack-neutron | 20:19 | |
| *** fifieldt has joined #openstack-neutron | 20:21 | |
| *** vivek-ebay has quit IRC | 20:21 | |
| *** dboik_ has quit IRC | 20:22 | |
| *** dboik has joined #openstack-neutron | 20:22 | |
| openstackgerrit | pritesh proposed openstack/python-neutronclient: Adding VLAN Transparency support to neutronclient https://review.openstack.org/162040 | 20:23 |
| *** r-daneel has quit IRC | 20:25 | |
| *** aix has joined #openstack-neutron | 20:25 | |
| *** dboik has quit IRC | 20:26 | |
| *** dboik has joined #openstack-neutron | 20:26 | |
| *** julim has quit IRC | 20:27 | |
| *** nmagnezi has quit IRC | 20:29 | |
| *** amuller has quit IRC | 20:30 | |
| *** _cjones_ has joined #openstack-neutron | 20:31 | |
| *** vishwana_ has quit IRC | 20:33 | |
| *** vishwanathj has joined #openstack-neutron | 20:34 | |
| *** yalie has joined #openstack-neutron | 20:34 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 20:34 |
| *** dboik has quit IRC | 20:35 | |
| *** dboik has joined #openstack-neutron | 20:36 | |
| ihrachyshka | pc_m, so the problem is that we assume that conf files are available in the directory where neutron code is installed, which is not the case when actual pip install is called (though it's the case when either we run tests from neutron root tree, or we fetch neutron via -e git directive that checks out the whole tree and stores all neutron code there. | 20:36 |
| ihrachyshka | pc_m, I guess the proper response would be not to rely on any conf files in the tree (generate them in temp dir?) | 20:36 |
| *** padkrish_ has quit IRC | 20:36 | |
| *** padkrish has joined #openstack-neutron | 20:37 | |
| *** aranjan has joined #openstack-neutron | 20:37 | |
| *** raildo is now known as raildo_away | 20:37 | |
| *** imcsk8 is now known as imcsk8|lunch | 20:38 | |
| *** jpena has quit IRC | 20:38 | |
| *** _cjones_ has quit IRC | 20:38 | |
| *** _cjones_ has joined #openstack-neutron | 20:38 | |
| pc_m | ihrachyshka: I thought it always looks for conf files in /etc/neutron. I also thought when running DevStack, it copies (and possibly modifies) conf files from in tree etc/ area. | 20:39 |
| pc_m | copies them into /etc/neutron | 20:39 |
| ihrachyshka | pc_m, actually, no, in that particular case, the problem is not about /etc/neutron/ but that neutron/tests/etc/ is not present. | 20:40 |
| pc_m | ihrachyshka: I don't ever remember it using conf files from tree. | 20:40 |
| ihrachyshka | BaseTestCase does it | 20:40 |
| pc_m | Oh for tests, yeah it uses the tree. | 20:40 |
| *** ajo has quit IRC | 20:40 | |
| ihrachyshka | pc_m, https://github.com/openstack/neutron/blob/master/neutron/tests/base.py#L71 | 20:40 |
| pc_m | ihrachyshka: yeah that's right. | 20:41 |
| *** padkrish has quit IRC | 20:41 | |
| *** Bsony has quit IRC | 20:42 | |
| *** boris-42 has quit IRC | 20:42 | |
| pc_m | ihrachyshka: I reversed that patch and my UTs are giving all sorts of odd errors (200+) | 20:43 |
| *** achanda has quit IRC | 20:44 | |
| *** achanda has joined #openstack-neutron | 20:45 | |
| pc_m | ihrachyshka: Looks like reverse patch didn't apply correctly.... :( | 20:48 |
| *** rook-tmp has joined #openstack-neutron | 20:50 | |
| *** cference has joined #openstack-neutron | 20:52 | |
| ihrachyshka | pc_m, I'm trying to come up with a patch that would fix the issue without revert | 20:53 |
| *** padkrish has joined #openstack-neutron | 20:53 | |
| pc_m | ihrachyshka: cool | 20:53 |
| *** vivek-ebay has joined #openstack-neutron | 20:54 | |
| *** mestery has quit IRC | 20:55 | |
| *** rook-tmp has quit IRC | 20:55 | |
| *** _cjones_ has quit IRC | 20:55 | |
| *** carl_baldwin has quit IRC | 20:57 | |
| *** vmtrooper has joined #openstack-neutron | 20:58 | |
| *** thomasem has quit IRC | 20:58 | |
| *** vivek-ebay has quit IRC | 20:58 | |
| *** parvez has joined #openstack-neutron | 20:58 | |
| parvez | hi | 20:58 |
| *** mestery has joined #openstack-neutron | 20:59 | |
| *** vivek-ebay has joined #openstack-neutron | 20:59 | |
| *** padkrish has quit IRC | 21:00 | |
| *** alexpilotti has quit IRC | 21:00 | |
| *** nlahouti has quit IRC | 21:00 | |
| *** jorgem1 is now known as jorgem | 21:01 | |
| *** padkrish has joined #openstack-neutron | 21:01 | |
| *** alexpilotti has joined #openstack-neutron | 21:02 | |
| *** sripriya has quit IRC | 21:02 | |
| *** vmtrooper has quit IRC | 21:02 | |
| *** cference has quit IRC | 21:02 | |
| pc_m | ihrachyshka: Once reversed patch, and repaired some missing changes (imports), I ran, and tests failed. I manually copied /opt/stack/neutron/etc/ to .tox/.../site-packages/etc/ and then the tests worked. | 21:03 |
| *** harlowja has quit IRC | 21:03 | |
| *** alexpilotti has quit IRC | 21:03 | |
| pc_m | ihrachyshka: So is pip install missing copying that directory? | 21:03 |
| ihrachyshka | pc_m, how does they fail with the patch reverted? | 21:03 |
| *** nlahouti has joined #openstack-neutron | 21:03 | |
| *** harlowja_ has joined #openstack-neutron | 21:03 | |
| pc_m | Many not finding config files in /opt/stack/neutron-vpnaas/.tox/py27/local/lib/python2.7/site-packages/etc/ | 21:04 |
| *** carl_baldwin has joined #openstack-neutron | 21:04 | |
| *** skraynev has quit IRC | 21:04 | |
| pc_m | ihrachyshka: Like policy.json | 21:04 |
| ihrachyshka | pc_m, which config file? | 21:04 |
| salv-orl_ | carl_baldwin: aloha? | 21:04 |
| pc_m | Some others had server failures in tests. 282 failures. | 21:04 |
| ihrachyshka | pc_m, hm, then it probably means that patch is not reverted in lbaas .tox evn | 21:04 |
| ihrachyshka | pc_m, have you reinstalled neutron there with pip? | 21:05 |
| carl_baldwin | salv-orl_: hi | 21:05 |
| *** padkrish_ has joined #openstack-neutron | 21:05 | |
| *** padkrish has quit IRC | 21:05 | |
| parvez | I am using RHEL 7 IceHouse OpenStack , with VLAN provider network | 21:05 |
| *** ajo has joined #openstack-neutron | 21:05 | |
| parvez | ovs-vsctl commands are timing out | 21:05 |
| pc_m | ihrachyshka: I'm not working in lbaas, I'm working in neutron-vpnaas repo | 21:05 |
| salv-orl_ | I am discussing with armax the question about the 2 ids for get_subnet | 21:06 |
| *** alexpilotti has joined #openstack-neutron | 21:06 | |
| *** skraynev has joined #openstack-neutron | 21:06 | |
| pc_m | ihrachyshka: In neutron repo, I have one patch of my change that is out for review, then I reverse patched 159862. | 21:06 |
| *** alexpilotti has quit IRC | 21:06 | |
| salv-orl_ | carl_baldwin: the reason for having a CIDR is that in the design of the pool feature it is supposed to be unique within a pool. | 21:07 |
| pc_m | ihrachyshka: I then went to neutron-vpnaas, and pip installed neutron, so it is in tox. It was all there, including the reverse patch, but missing the contents of the etc dir | 21:07 |
| *** sripriya has joined #openstack-neutron | 21:07 | |
| salv-orl_ | carl_baldwin: I think we agree that if people use a subnet pool, cidr uniqueness withing the pool makes sense. | 21:07 |
| pc_m | ihrachyshka: from /opt/stack/neutron (not the one from /opt/stack/neutron/neutron/tests/) | 21:07 |
| pc_m | ihrachyshka: So the pip install stuff must not be installing that dir? | 21:08 |
| salv-orl_ | carl_baldwin: but we were wondering if it will be too ugly to retrieve subnets anyway by id - to simplify the interface when one is not using pools | 21:08 |
| carl_baldwin | salv-orl_: At this point, whatever people will agree to, I’m fine with. | 21:08 |
| ihrachyshka | pc_m, probably not. | 21:08 |
| *** carl_baldwin has quit IRC | 21:09 | |
| *** adelinatuvenie has quit IRC | 21:10 | |
| mestery | yamahata: port security is back in for Kilo. :) | 21:11 |
| mestery | yamahata: After disucssion with some folks, we think we can get this merged (it may need an FFE). /cc marios | 21:11 |
| mestery | Sorry, /cc marun ^^^^ | 21:11 |
| mestery | marios: Wrong cc sorry | 21:11 |
| *** carl_baldwin has joined #openstack-neutron | 21:11 | |
| mestery | carl_baldwin: Aren't you on vacation this week? :) | 21:12 |
| yamahata | mestery: wow, thanks. pleasant surprise. | 21:12 |
| mestery | yamahata: Thank you, lets see if we can land it now :) | 21:12 |
| *** carl_baldwin has quit IRC | 21:12 | |
| *** mlavalle has quit IRC | 21:12 | |
| marun | pc_m: https://review.openstack.org/#/c/145535/ | 21:12 |
| marun | pc_m: is this related? | 21:13 |
| *** imcsk8|lunch is now known as imcsk8 | 21:14 | |
| Swami | :q! | 21:14 |
| *** lpetrut has quit IRC | 21:15 | |
| pc_m | marun: not sure. Looks like there are several things. ihrachyshka mentioned 159862 is causing the issue in UTs. I reversed that, but it looks like the etc dir is not copied, when I pip install neutron for use by neutron-vpnaas. | 21:15 |
| *** lpetrut has joined #openstack-neutron | 21:16 | |
| *** lori is now known as lori|away | 21:16 | |
| pc_m | marun: When I manually copy, I can get things to pass. Is there something that pip install uses (setup.cfg?) to cause the install to include etc/? | 21:16 |
| ihrachyshka | pc_m, probably data_files are not triggered sometimes | 21:17 |
| *** _cjones_ has joined #openstack-neutron | 21:17 | |
| *** pritesh has joined #openstack-neutron | 21:18 | |
| *** amuller has joined #openstack-neutron | 21:18 | |
| *** leenheer has joined #openstack-neutron | 21:18 | |
| *** ajo has quit IRC | 21:18 | |
| *** lpetrut has quit IRC | 21:20 | |
| *** gildub has quit IRC | 21:21 | |
| *** pritesh_ has quit IRC | 21:21 | |
| *** gildub has joined #openstack-neutron | 21:21 | |
| amuller | armax: py27 failure with no such table: brocadenetworks | 21:22 |
| amuller | Did you see this? | 21:22 |
| amuller | http://logs.openstack.org/84/125384/41/check/gate-neutron-python27/c867829/testr_results.html.gz | 21:22 |
| armax | no | 21:22 |
| armax | any idea how did sneak in? | 21:22 |
| *** s3wong has quit IRC | 21:23 | |
| armax | amuller: this looks confined still | 21:23 |
| marun | it looks like db creation simply failed | 21:23 |
| armax | or not? | 21:23 |
| *** s3wong has joined #openstack-neutron | 21:23 | |
| *** ajo has joined #openstack-neutron | 21:23 | |
| *** ajo has quit IRC | 21:23 | |
| salv-orl_ | marun: it's a unit test... no migrations are run | 21:24 |
| salv-orl_ | I mean they are | 21:24 |
| salv-orl_ | no they aren't sorry ;) | 21:24 |
| amuller | some other tests in the same patch are failing because some brocade module is being imported but it's not there | 21:24 |
| salv-orl_ | I've seen some traffic regarding brocade vyatta decomp earlier on today | 21:24 |
| marun | amuller: link to change? | 21:24 |
| amuller | looks like my entire patch series is failing on that stuff | 21:25 |
| *** tiswanso_ has quit IRC | 21:25 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Refactoring to use callback mechanism (WIP) https://review.openstack.org/165226 | 21:25 |
| amuller | marun: 125384 for example | 21:25 |
| amuller | marun: basically all the red stuff on zuul :) | 21:25 |
| armax | amuller: looking to see what’s going on | 21:26 |
| marun | amuller: can you repro locally? | 21:26 |
| amuller | marun: let me try | 21:26 |
| *** carl_baldwin has joined #openstack-neutron | 21:27 | |
| haleyb | HenryG: ping | 21:27 |
| salv-orl_ | amuller, marun: this is rather disturbing in unit tests logs "Loading core plugin: neutron.plugins.brocade.NeutronPlugin.BrocadePluginV2" :) | 21:28 |
| armax | amuller: see if by backing out https://review.openstack.org/#/c/159056/ things improve, but I doubt | 21:28 |
| *** arif-ali has quit IRC | 21:28 | |
| salv-orl_ | I thought brocade did decompose | 21:28 |
| amuller | hmm tox -e py27 neutron.tests.unit.brocade works locally on the tip of the branch that failed at the gate | 21:28 |
| salv-orl_ | what are those unit tests doung there | 21:28 |
| marun | salv-orl_: someone pulled the plugin but not the tests? | 21:28 |
| amuller | likes like they did not decompose enough =D | 21:28 |
| armax | salv-orl_: indeed it did | 21:28 |
| carl_baldwin | salv-orl_: Just chatted with armax. He’s going to try to tie up the loose ends. Sounds like they’re going to pass id and then will look up the cidr in the driver when needed or something like that. | 21:28 |
| armax | amuller: but I could that be? | 21:29 |
| salv-orl_ | carl_baldwin: that sounds good to me. | 21:29 |
| marun | when exactly did brocade exit the tree | 21:29 |
| salv-orl_ | carl_baldwin: we'll keep this interface as experimetal during the L release cycle, so if we want to come back to it we can | 21:29 |
| marun | ? | 21:29 |
| marun | https://github.com/openstack/neutron/tree/master/neutron/plugins/brocade | 21:29 |
| salv-orl_ | using the id will simply development of the ref driver | 21:29 |
| carl_baldwin | salv-orl_: It is settled then. And before I got booted off of IRC yet again due to unreliable internet. | 21:29 |
| *** busterswt has quit IRC | 21:29 | |
| carl_baldwin | :) | 21:29 |
| marun | also, who the heck merged unit tests under the code tree :( | 21:30 |
| carl_baldwin | mestery: yes, can’t you tell? | 21:30 |
| amuller | armax: this branch is rebased on 'Move mlnx agent to be under ml2/drivers/mlnx' | 21:30 |
| mestery | carl_baldwin: lol | 21:30 |
| *** Marga_ has joined #openstack-neutron | 21:31 | |
| *** carl_baldwin has quit IRC | 21:31 | |
| amuller | armax: which is after the brocade decomposition and before the other brocade patch you linked | 21:31 |
| armax | amuller: hang on…I am trying to follow the thread | 21:31 |
| salv-orl_ | armax: 159056 did sound like a candidate for me too but I do not see how it can break stuff | 21:32 |
| HenryG | haleyb: pong | 21:33 |
| haleyb | HenryG: this seems interesting, http://logs.openstack.org/39/113339/26/check/check-tempest-dsvm-neutron-dvr/cb08575/logs/screen-q-agt.txt.gz at 2015-03-17 15:20:33.457 | 21:33 |
| salv-orl_ | amuller, armax: I think a failure like that could hardly pass gate checks | 21:33 |
| haleyb | OVS pulling the port out from under us? | 21:33 |
| salv-orl_ | perhaps we should just rebase amuller's patch series? | 21:34 |
| armax | salv-orl_: indeed\ | 21:34 |
| amuller | I guess | 21:34 |
| haleyb | HenryG: it's nearby the l3-agent failure (about 20 seconds after), but none the less, interesting | 21:34 |
| amuller | it failed anyway no reason not to rebase and send it again | 21:34 |
| *** arif-ali has joined #openstack-neutron | 21:34 | |
| armax | I see nothing pop up in logstash yet | 21:35 |
| haleyb | HenryG: because from what Rajeev and I found, the port was there, then it was gone, and the l3-agent didn't seem to touch it | 21:35 |
| armax | the other failures I see on the gate seem to be unrelated | 21:35 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Send notification to controller about HA router state change https://review.openstack.org/126188 | 21:35 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 21:35 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 21:35 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Don't delete HA router primary VIP on agent restarts https://review.openstack.org/164841 | 21:35 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 21:35 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 21:35 |
| armax | but I haven’t scanned through all of them yet | 21:35 |
| amuller | bam | 21:35 |
| salv-orl_ | other paches are succeeding | 21:35 |
| armax | amuller: you’re definitely a great community citizen | 21:35 |
| armax | :) | 21:35 |
| amuller | *cough* review that crap *cough* | 21:36 |
| amuller | :) | 21:36 |
| salv-orl_ | is this the place where we insult amuller for sending 7 patches through gate checks? | 21:36 |
| armax | instead I see these types of failures: RetryError: RetryError[Attempts: 16, Value: None] | 21:36 |
| armax | salv-orl_: yes, it is | 21:36 |
| armax | salv-orl_: and no, I did not insult anyone | 21:36 |
| salv-orl_ | armax: is that the vlan allocation table | 21:36 |
| *** pc_m has quit IRC | 21:37 | |
| armax | from what I can tell, the gate looks fine | 21:38 |
| *** jckasper has quit IRC | 21:39 | |
| armax | but I hope I just haven’t spoken too soon | 21:39 |
| *** _cjones_ has quit IRC | 21:40 | |
| armax | amuller: so you can’t repro locally? | 21:40 |
| openstackgerrit | Merged openstack/neutron: IBM SDN-VE Plugin decomposition https://review.openstack.org/130471 | 21:40 |
| amuller | armax: no, not even before I rebased | 21:40 |
| openstackgerrit | Merged openstack/neutron: OVS DVR UT: Remove an inappropriate str() conversion https://review.openstack.org/161524 | 21:40 |
| armax | ok | 21:40 |
| openstackgerrit | Merged openstack/neutron: replaces enumeration method used to get a list of interfaces https://review.openstack.org/154128 | 21:41 |
| armax | let’s hold on before give me a hearthattack next time? | 21:41 |
| armax | :) | 21:41 |
| *** signed8bit has quit IRC | 21:41 | |
| armax | *giving | 21:41 |
| amuller | well... | 21:42 |
| * armax flat lines | 21:43 | |
| HenryG | haleyb: I can't make any sense of it :( | 21:43 |
| *** crc32 has quit IRC | 21:43 | |
| *** fredlhsu has quit IRC | 21:44 | |
| *** _cjones_ has joined #openstack-neutron | 21:44 | |
| amuller | armax: something is still wrong but it looks like it's some patch in the series | 21:45 |
| armax | amuller: which series, yours? | 21:45 |
| amuller | yeah | 21:45 |
| *** banix has quit IRC | 21:45 | |
| *** yfried_ is now known as yfried|afk | 21:45 | |
| armax | ok, so tip of trunk is good, but something is funky in your patches? | 21:45 |
| amuller | yes | 21:46 |
| amuller | some patch in my series is screwing up the unit tests horribly =D | 21:46 |
| armax | um…ok | 21:46 |
| salv-orl_ | amuller: meh shall we rule out a PEBCAK case? | 21:46 |
| *** dboik has quit IRC | 21:46 | |
| armax | so the gate is fine, then? | 21:46 |
| amuller | gate's fine | 21:46 |
| armax | amuller: O.K. | 21:46 |
| *** dboik has joined #openstack-neutron | 21:46 | |
| armax | amuller: sorry, I misurstood you there for a sec | 21:47 |
| armax | amuller: I am susceptible more than usual closer fo FF | 21:47 |
| armax | :) | 21:47 |
| *** nlahouti has quit IRC | 21:47 | |
| salv-orl_ | armax: I couldn't care less as usual instead :0 | 21:48 |
| armax | salv-orl_: we all know that | 21:48 |
| haleyb | HenryG: I don't know what else to look at, tempted to just recheck to see what happens | 21:49 |
| armax | amuller: ok, give me a few, and I’ll look into it | 21:51 |
| amuller | armax: it's the 2nd patch in the series, trying to figure this out... | 21:51 |
| kevinbenton | ooh, what's happening? | 21:51 |
| kevinbenton | amuller: an unreproducable bug? | 21:51 |
| *** rook-tmp has joined #openstack-neutron | 21:51 | |
| amuller | no it's just me being stupid somehow | 21:52 |
| amuller | this patch: https://review.openstack.org/#/c/125384/42 | 21:52 |
| amuller | is being weird when you run tox -e py27 | 21:52 |
| kevinbenton | what's it doing? | 21:52 |
| amuller | http://logs.openstack.org/84/125384/41/check/gate-neutron-python27/c867829/testr_results.html.gz | 21:53 |
| kevinbenton | well duh | 21:54 |
| kevinbenton | you can't iterate over None | 21:54 |
| kevinbenton | :) | 21:54 |
| amuller | That is 100% true | 21:54 |
| armax | kevinbenton: who says that? | 21:54 |
| armax | of course you can | 21:54 |
| kevinbenton | armax: i think the fix is to modify the None type with an __iter__ method | 21:55 |
| armax | kevinbenton: right | 21:55 |
| kevinbenton | amuller: i bet you hadn't considered that :) | 21:56 |
| *** rook-tmp has quit IRC | 21:56 | |
| openstackgerrit | Claudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 21:56 |
| *** ChuckC has quit IRC | 21:57 | |
| *** julien_ZTE has joined #openstack-neutron | 21:59 | |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 21:59 |
| *** fredlhsu has joined #openstack-neutron | 21:59 | |
| *** kickinz1 is now known as kickinz1|afk | 22:00 | |
| *** ipolyzos has left #openstack-neutron | 22:01 | |
| *** dboik has quit IRC | 22:03 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 22:03 |
| *** padkrish_ has quit IRC | 22:03 | |
| armax | salv-orl_: lot | 22:03 |
| armax | salv-orl_: ^^ | 22:03 |
| *** dboik has joined #openstack-neutron | 22:03 | |
| *** ijw has quit IRC | 22:03 | |
| *** padkrish has joined #openstack-neutron | 22:04 | |
| amuller | trying out tox -e py27 -- --analyze-isolation | 22:04 |
| *** oda-g has joined #openstack-neutron | 22:04 | |
| *** vivek-ebay has quit IRC | 22:04 | |
| amuller | let's see if it can figure it out for me :) | 22:04 |
| *** ijw has joined #openstack-neutron | 22:05 | |
| *** vivek-ebay has joined #openstack-neutron | 22:05 | |
| amuller | marun: I do have an issue locally btw, after I rebased | 22:05 |
| amuller | marun: not on master, only on my branch | 22:05 |
| *** nlahouti has joined #openstack-neutron | 22:06 | |
| *** yfried|afk is now known as yfried_ | 22:06 | |
| *** nlahouti has quit IRC | 22:06 | |
| *** ijw has quit IRC | 22:06 | |
| *** ijw has joined #openstack-neutron | 22:07 | |
| *** julien_ZTE has quit IRC | 22:07 | |
| *** julien_ZTE has joined #openstack-neutron | 22:07 | |
| HenryG | marun: where are unit tests in the code tree? | 22:07 |
| *** padkrish has quit IRC | 22:08 | |
| *** harlowja_ has quit IRC | 22:09 | |
| *** localloop127 has quit IRC | 22:09 | |
| parvez | can anybody help ,e | 22:09 |
| parvez | can anybody help me figuring out why ovs-vsctl command is timing out? | 22:10 |
| *** csoukup has quit IRC | 22:11 | |
| *** harlowja has joined #openstack-neutron | 22:11 | |
| HenryG | amuller: is it only your patch(es) seeing the issues? | 22:14 |
| amuller | HenryG: So far, that I noticed. master is doing fine. | 22:14 |
| amuller | locally | 22:14 |
| HenryG | amuller: rebase rebase rebase ;) | 22:14 |
| amuller | I did, it's failing when rebased on master | 22:14 |
| amuller | same master that's working fine | 22:14 |
| kevinbenton | i'm looking into it too | 22:15 |
| amuller | trying to figure out now what is the minimal set of unit tests that I run that fail... it's some interaction between tests | 22:15 |
| kevinbenton | something is leaving a booby trap | 22:15 |
| amuller | right | 22:15 |
| amuller | tests that pass alone fail when run concurrently with another test | 22:15 |
| *** ronis has quit IRC | 22:15 | |
| *** yfried_ is now known as yfried|afk | 22:16 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Stop relying on configuration files being available for tests https://review.openstack.org/165237 | 22:17 |
| *** s3wong has quit IRC | 22:17 | |
| *** Marga_ has quit IRC | 22:17 | |
| openstackgerrit | Merged openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 22:17 |
| *** s3wong has joined #openstack-neutron | 22:18 | |
| dougwig | marun: are the neutron tempest/api tests in a state where i can migrate the lbaas tests into neutron-lbaas now? | 22:18 |
| *** Marga_ has joined #openstack-neutron | 22:18 | |
| *** Marga_ has quit IRC | 22:18 | |
| *** tidwellr has left #openstack-neutron | 22:18 | |
| kevinbenton | amuller: neutron.tests.unit.agent.metadata.test_driver.TestMetadataDriverProcess.test_spawn_metadata_proxy_with_user | 22:18 |
| amuller | yeah? looking | 22:18 |
| kevinbenton | that seems to be the cause of issues | 22:19 |
| *** jorgem has quit IRC | 22:19 | |
| *** Marga_ has joined #openstack-neutron | 22:19 | |
| *** jorgem has joined #openstack-neutron | 22:19 | |
| *** yalie has quit IRC | 22:19 | |
| *** moha_hunt has quit IRC | 22:20 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 22:20 |
| amuller | kevinbenton: I think you're right, thanks | 22:20 |
| armax | otherwiseguy: there’s a femto-typo :) | 22:21 |
| armax | https://review.openstack.org/#/c/152761/15/etc/neutron/plugins/openvswitch/ovs_neutron_plugin.ini | 22:21 |
| *** s3wong has quit IRC | 22:21 | |
| *** s3wong_ has joined #openstack-neutron | 22:21 | |
| *** TrevorV|Home has joined #openstack-neutron | 22:21 | |
| *** jgrimm is now known as zz_jgrimm | 22:21 | |
| otherwiseguy | marun: mestery: ^ should hopefully fix both the 'patch_idl is ugly' and 'we don't wait on interface creation' issues. | 22:21 |
| otherwiseguy | armax: figures | 22:21 |
| armax | otherwiseguy: be honest, that was there on purpose, to see if people paid attention? | 22:22 |
| otherwiseguy | armax: yeah! that was it! :) | 22:22 |
| *** s3wong_ has quit IRC | 22:22 | |
| * otherwiseguy does a quick fix | 22:23 | |
| armax | otherwiseguy: see? I paid attention! | 22:23 |
| otherwiseguy | armax: apparently more than I did. :) | 22:23 |
| *** s3wong_ has joined #openstack-neutron | 22:23 | |
| armax | otherwiseguy: I am still going through the bits | 22:23 |
| otherwiseguy | armax: so...many...bits | 22:24 |
| *** rotbeard has quit IRC | 22:24 | |
| armax | otherwiseguy: looks good so far, I’ll sync up with Maru | 22:24 |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 22:24 |
| armax | otherwiseguy: too many bytes | 22:24 |
| *** yalie has joined #openstack-neutron | 22:24 | |
| *** harlowja has quit IRC | 22:24 | |
| armax | otherwiseguy: oh my, I need a break | 22:24 |
| *** harlowja_ has joined #openstack-neutron | 22:24 | |
| otherwiseguy | armax: :) | 22:25 |
| otherwiseguy | See what happens when you point out spelling issues? You get new patches to comment on. | 22:25 |
| armax | soo true | 22:25 |
| *** TrevorV|Home has quit IRC | 22:25 | |
| *** mleroy_ has quit IRC | 22:26 | |
| openstackgerrit | Merged openstack/neutron: Advertise mtu over dhcp https://review.openstack.org/155874 | 22:26 |
| openstackgerrit | Merged openstack/neutron: Expose Rest Api access to mtu attributes https://review.openstack.org/154921 | 22:26 |
| *** nati_ueno has quit IRC | 22:27 | |
| kevinbenton | amuller: you're leaking eventlet | 22:27 |
| kevinbenton | spawn | 22:27 |
| kevinbenton | amuller: eventlet.spawn(self._start_keepalived_notifications_server) | 22:28 |
| kevinbenton | amuller: L94 | 22:28 |
| kevinbenton | amuller: neutron/agent/l3/ha.py | 22:28 |
| kevinbenton | amuller: comment that out and the tests are good to go! | 22:29 |
| kevinbenton | amuller: i assume it does something important though so i would suggest mocking it :) | 22:29 |
| amuller | kevinbenton: I guess the solution is to mock out eventlet during those tests | 22:29 |
| *** vishwanathj has quit IRC | 22:29 | |
| amuller | I don't understand why this has the effect it does though | 22:29 |
| amuller | it looks like once you run that code, the next test fails to initialize its core plugin | 22:29 |
| *** crc32 has joined #openstack-neutron | 22:29 | |
| *** abehl has joined #openstack-neutron | 22:30 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 22:30 |
| amuller | why would running eventlet.spawn cause the following test to fail to import a brocade module | 22:31 |
| *** tidwellr has joined #openstack-neutron | 22:31 | |
| amuller | meh | 22:31 |
| amuller | kevinbenton: Thanks for your assistance | 22:31 |
| amuller | now I'll fix this and pretend to know WTF happened | 22:31 |
| kevinbenton | amuller: yeah, it's interesting | 22:31 |
| amuller | well, I'll go through all initializations to L3NATAgent and see that they all mock eventlet | 22:32 |
| *** ijw has quit IRC | 22:32 | |
| otherwiseguy | Is this the part where we start bashing on eventlet? | 22:32 |
| kevinbenton | otherwiseguy: nah, we shouldn't be leaving eventlet servers running as a side effect of unit tests anyway | 22:32 |
| ZZelle_ | otherwiseguy, the trouble is not eventlet but how we use it ... | 22:32 |
| * otherwiseguy sadly walks away then | 22:33 | |
| otherwiseguy | ZZelle_: It's funny, because I *hate* monkey_patching as a concept. And then I did it to the ovs lib with my ovsdb patch (but ended up having to remove that part). So I realize I'm a hypocrite. :) | 22:34 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 22:35 |
| kevinbenton | amuller: if you change the number of workers to 1 for the keepalivestatechangehanlder, it fixes the failure too :) | 22:36 |
| *** sambetts has quit IRC | 22:36 | |
| amuller | kevinbenton: uhh, wouldn't that fork the process that runs the tests | 22:36 |
| amuller | double the testing? | 22:36 |
| kevinbenton | amuller: double our test coverage! | 22:37 |
| kevinbenton | amuller: one weird trick QA testers don't want you to know! :) | 22:37 |
| amuller | lol | 22:37 |
| *** sambetts has joined #openstack-neutron | 22:38 | |
| *** ijw has joined #openstack-neutron | 22:38 | |
| *** shwetaap has quit IRC | 22:39 | |
| *** marun has quit IRC | 22:40 | |
| *** ijw has quit IRC | 22:40 | |
| *** parvez has quit IRC | 22:41 | |
| *** ijw has joined #openstack-neutron | 22:42 | |
| *** natarajk has joined #openstack-neutron | 22:42 | |
| *** Trevor_V has joined #openstack-neutron | 22:44 | |
| *** matrohon has quit IRC | 22:45 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Send notification to controller about HA router state change https://review.openstack.org/126188 | 22:45 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 22:45 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 22:45 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Don't delete HA router primary VIP on agent restarts https://review.openstack.org/164841 | 22:45 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 22:45 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 22:45 |
| *** vmtrooper has joined #openstack-neutron | 22:46 | |
| *** Trevor_V has quit IRC | 22:47 | |
| *** Rajeev has quit IRC | 22:47 | |
| *** s3wong_ has quit IRC | 22:47 | |
| *** s3wong has joined #openstack-neutron | 22:47 | |
| *** wverdugo500 has joined #openstack-neutron | 22:48 | |
| *** jobewan has quit IRC | 22:49 | |
| *** wverdugo5001 has joined #openstack-neutron | 22:49 | |
| *** s3wong_ has joined #openstack-neutron | 22:50 | |
| *** s3wong has quit IRC | 22:51 | |
| *** kbringard has quit IRC | 22:51 | |
| *** vmtrooper has quit IRC | 22:51 | |
| *** rook-tmp has joined #openstack-neutron | 22:52 | |
| *** reed has quit IRC | 22:53 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 22:53 |
| *** zzzeek has quit IRC | 22:54 | |
| *** ajmiller__ has joined #openstack-neutron | 22:55 | |
| *** nati_ueno has joined #openstack-neutron | 22:55 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches. https://review.openstack.org/154709 | 22:56 |
| *** sgordon has quit IRC | 22:56 | |
| tidwellr | armax: https://review.openstack.org/#/c/148698/ should have a dependency on https://review.openstack.org/#/c/134339/, gerrit seems to have lost that dependency. Could this be why https://review.openstack.org/#/c/148698/ is stuck? | 22:57 |
| *** thomasem has joined #openstack-neutron | 22:57 | |
| *** thomasem has quit IRC | 22:57 | |
| HenryG | yalie: ping | 22:57 |
| *** ajmiller_ has quit IRC | 22:58 | |
| *** changbl has quit IRC | 22:59 | |
| *** yalie has quit IRC | 23:00 | |
| *** aranjan has quit IRC | 23:02 | |
| *** yalie has joined #openstack-neutron | 23:03 | |
| *** aranjan has joined #openstack-neutron | 23:03 | |
| *** nlahouti has joined #openstack-neutron | 23:05 | |
| *** padkrish has joined #openstack-neutron | 23:05 | |
| *** smcquaid has joined #openstack-neutron | 23:07 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: WIP: stop relying on configuration files being available for tests https://review.openstack.org/165237 | 23:08 |
| *** emagana has quit IRC | 23:08 | |
| *** s3wong_ has quit IRC | 23:08 | |
| *** gongysh has joined #openstack-neutron | 23:08 | |
| *** dims has quit IRC | 23:09 | |
| *** smcquaid has quit IRC | 23:10 | |
| *** s3wong has joined #openstack-neutron | 23:10 | |
| *** s3wong_ has joined #openstack-neutron | 23:13 | |
| *** s3wong has quit IRC | 23:13 | |
| *** enikanorov has quit IRC | 23:14 | |
| *** enikanorov has joined #openstack-neutron | 23:15 | |
| *** YorikSar has quit IRC | 23:15 | |
| *** nati_ueno has quit IRC | 23:17 | |
| armax | tidwellr: it might as well be | 23:17 |
| *** nati_ueno has joined #openstack-neutron | 23:17 | |
| openstackgerrit | Merged openstack/neutron: Handle DBDuplicateError exception properly when creating default sg https://review.openstack.org/164463 | 23:18 |
| tidwellr | armx: not sure why this happens, best I can tell it seems to happen when running rechecks but that's just a wild theory. All I know is several times I've logged in and found some of my reviews detached from the appropriate parent. Looks like it happened again | 23:19 |
| armax | tidwellr: bad rebase? | 23:19 |
| *** yfauser has joined #openstack-neutron | 23:19 | |
| armax | tidwellr: what’s the depedent patch? I can’t seem to find 79e5ce4e70a86563a04e3a16ef216ba5023bae29 | 23:20 |
| *** Marga_ has quit IRC | 23:20 | |
| *** yfauser has left #openstack-neutron | 23:20 | |
| tidwellr | https://review.openstack.org/#/c/134339 | 23:20 |
| armax | got it | 23:20 |
| *** jorgem has quit IRC | 23:20 | |
| *** dims has joined #openstack-neutron | 23:20 | |
| armax | tidwellr: have you tried to rebase again? | 23:20 |
| *** yuanying has joined #openstack-neutron | 23:21 | |
| *** yfauser1 has joined #openstack-neutron | 23:21 | |
| *** ihrachyshka has quit IRC | 23:21 | |
| *** yfauser1 has left #openstack-neutron | 23:21 | |
| tidwellr | I can, the check jobs are failing due to some import failures on neutron.ipam, which means it probably hasn't been rebased on https://review.openstack.org/#/c/134339 anywhere along the way | 23:22 |
| armax | indeed | 23:22 |
| *** Marga_ has joined #openstack-neutron | 23:23 | |
| *** rook-tmp has quit IRC | 23:26 | |
| *** ChuckC has joined #openstack-neutron | 23:26 | |
| *** nlahouti has quit IRC | 23:26 | |
| *** Marga_ has quit IRC | 23:26 | |
| *** Marga_ has joined #openstack-neutron | 23:27 | |
| *** nlahouti has joined #openstack-neutron | 23:28 | |
| *** YorikSar has joined #openstack-neutron | 23:28 | |
| *** Marga_ has quit IRC | 23:31 | |
| *** Marga_ has joined #openstack-neutron | 23:31 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 23:31 |
| openstackgerrit | Yanping Qu proposed openstack/neutron-fwaas: Adds FWaaS driver for Cisco CSR1kv https://review.openstack.org/161019 | 23:32 |
| *** Marga_ has quit IRC | 23:32 | |
| *** Marga_ has joined #openstack-neutron | 23:32 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 23:34 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: DVR qrouters not created on compute Node when vm added https://review.openstack.org/149143 | 23:34 |
| *** sbfox has quit IRC | 23:35 | |
| armax | prad: | 23:37 |
| armax | oops wrong nick | 23:37 |
| armax | pritesh: ping | 23:37 |
| *** crc32 has quit IRC | 23:39 | |
| *** nlahouti has quit IRC | 23:40 | |
| *** padkrish has quit IRC | 23:41 | |
| *** padkrish has joined #openstack-neutron | 23:42 | |
| *** Swami has quit IRC | 23:42 | |
| openstackgerrit | Baodong (Robert) Li proposed openstack/neutron: Include IPv6 SLAAC addresses implicitly for port create https://review.openstack.org/113339 | 23:42 |
| *** nlahouti has joined #openstack-neutron | 23:44 | |
| *** crc32 has joined #openstack-neutron | 23:44 | |
| *** padkrish_ has joined #openstack-neutron | 23:44 | |
| *** fredlhsu has quit IRC | 23:45 | |
| *** Marga_ has quit IRC | 23:45 | |
| *** padkrish has quit IRC | 23:46 | |
| *** banix has joined #openstack-neutron | 23:46 | |
| kevinbenton | amuller: still around? | 23:46 |
| amuller | kevinbenton: yeesh | 23:47 |
| kevinbenton | amuller: so when that extra thread hangs out, it causes the plugin to get a different db engine than the unit test setup methods | 23:48 |
| amuller | why though? | 23:48 |
| *** tsg_ has quit IRC | 23:49 | |
| kevinbenton | amuller: not sure yet :) | 23:49 |
| *** ajmiller__ has quit IRC | 23:50 | |
| *** david-lyle is now known as david-lyle_afk | 23:50 | |
| shivharis | kevinbenton, amuller: could the problem be due to the api_worker default being ncpu/2 ? | 23:50 |
| *** oomichi has joined #openstack-neutron | 23:51 | |
| kevinbenton | shivharis: i'm not sure. didn't that patch get reverted? | 23:51 |
| shivharis | the reversal did go in, but there are patch-sets that use an older snapshot, not yet rebased to bring that reversal in | 23:51 |
| *** julien_ZTE has quit IRC | 23:52 | |
| kevinbenton | shivharis: i don't think this is related to api workers | 23:52 |
| shivharis | if it is rebased then there should not be a problem | 23:52 |
| kevinbenton | shivharis: its a side effect of him leaving a spawned process | 23:52 |
| shivharis | kevinbenton: ok | 23:52 |
| kevinbenton | shivharis: but its a weird side effect | 23:52 |
| *** mlavalle has joined #openstack-neutron | 23:52 | |
| kevinbenton | shivharis: he was still getting failures rebased on master, right amuller? | 23:53 |
| *** yamamoto has joined #openstack-neutron | 23:53 | |
| amuller | yes | 23:53 |
| kevinbenton | amuller: do you want to hear something even cooler? | 23:53 |
| *** ZZelle_ has quit IRC | 23:54 | |
| amuller | always | 23:54 |
| shivharis | amuller: np | 23:54 |
| kevinbenton | amuller: if i call db api.get_engine() before you spawn your server, everything works just fine :) | 23:54 |
| *** Marga_ has joined #openstack-neutron | 23:56 | |
| *** SumitNaiksatam has quit IRC | 23:56 | |
| *** pritesh_ has joined #openstack-neutron | 23:56 | |
| amuller | kevinbenton: looking at what L3AgentKeepalivedStateChangeServer does, I don't see how it would cause any DB interaction | 23:56 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Functional tests for OVS-agent OpenFlow drivers https://review.openstack.org/164584 | 23:57 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 23:57 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 23:57 |
| *** sgordon has joined #openstack-neutron | 23:57 | |
| kevinbenton | amuller: oooooh, one sec | 23:58 |
| *** dboik_ has joined #openstack-neutron | 23:58 | |
| * amuller fell off the egde of his seat | 23:58 | |
| *** yalie has quit IRC | 23:58 | |
| kevinbenton | amuller: new service starts | 23:59 |
| kevinbenton | amuller: calls api.get_engine().pool.dispose() | 23:59 |
| *** pritesh has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!