| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Minor refactoring of auto_schedule_routers https://review.openstack.org/114691 | 00:02 |
|---|---|---|
| *** baoli has joined #openstack-neutron | 00:02 | |
| *** sbfox has quit IRC | 00:04 | |
| *** tomoe_ has joined #openstack-neutron | 00:05 | |
| *** rudrarugge has joined #openstack-neutron | 00:06 | |
| *** baoli has quit IRC | 00:07 | |
| *** networkstatic has quit IRC | 00:09 | |
| *** sgordon has quit IRC | 00:09 | |
| *** rudrarugge has quit IRC | 00:12 | |
| *** ZZelle has quit IRC | 00:13 | |
| *** harlowja has quit IRC | 00:15 | |
| *** harlowja_ has joined #openstack-neutron | 00:15 | |
| *** popw has quit IRC | 00:21 | |
| *** yamahata has joined #openstack-neutron | 00:28 | |
| *** yamahata has quit IRC | 00:33 | |
| *** yamahata has joined #openstack-neutron | 00:34 | |
| *** diegows has quit IRC | 00:37 | |
| openstackgerrit | shihanzhang proposed a change to openstack/neutron: Refactor security group rpc call https://review.openstack.org/111876 | 00:40 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Minor refactoring for add_router_to_l3_agent https://review.openstack.org/115829 | 00:40 |
| *** SumitNaiksatam has joined #openstack-neutron | 00:43 | |
| *** ijw has quit IRC | 00:47 | |
| *** flwang has quit IRC | 00:47 | |
| *** banix has quit IRC | 00:48 | |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements-3: Name Mapping https://review.openstack.org/111432 | 00:48 |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements-4: Sync, Gateway https://review.openstack.org/111433 | 00:48 |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements-5: Topology https://review.openstack.org/111434 | 00:48 |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements-1: Client https://review.openstack.org/109782 | 00:48 |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements-2.2: Driver and Manager refactor (L3) https://review.openstack.org/115786 | 00:48 |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements-2.1: Driver and Manager refactor (L2) https://review.openstack.org/111431 | 00:48 |
| *** zhhuabj has quit IRC | 00:51 | |
| *** zhhuabj has joined #openstack-neutron | 00:51 | |
| *** zhipeng has quit IRC | 00:53 | |
| *** moha_hunt has quit IRC | 00:53 | |
| *** nlahouti has quit IRC | 00:53 | |
| *** xgerman has quit IRC | 01:00 | |
| *** celttechie_ has quit IRC | 01:01 | |
| *** flwang has joined #openstack-neutron | 01:01 | |
| openstackgerrit | Divya ChanneGowda proposed a change to openstack/neutron: Add support for provider-network extension in nuage Plugin https://review.openstack.org/114402 | 01:02 |
| *** tomoe_ has quit IRC | 01:02 | |
| *** tomoe_ has joined #openstack-neutron | 01:02 | |
| *** baoli has joined #openstack-neutron | 01:03 | |
| *** topshare has joined #openstack-neutron | 01:04 | |
| *** ryu25 has joined #openstack-neutron | 01:06 | |
| gus | huh. so opendaylight-ci is spreading vicious -1 lies about my change, but when I mail odl-openstack@opendaylight.org I get a vacation responder. | 01:07 |
| *** topshare_ has joined #openstack-neutron | 01:07 | |
| *** ijw has joined #openstack-neutron | 01:07 | |
| gus | and they have no detailed logs nor recheck command afaics. I should start a ranty os-dev thread. | 01:07 |
| kevinbenton | armax: found the SQL issue | 01:07 |
| kevinbenton | armax: was pretty stupid :-) | 01:07 |
| *** baoli has quit IRC | 01:07 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Do not use auto_schedule_routers to add router to agent https://review.openstack.org/115837 | 01:08 |
| kevinbenton | armax: wasn’t actually joining the tables based on the agent IDs, so it was returning whenever there was any agent down | 01:08 |
| *** topshare has quit IRC | 01:09 | |
| kevinbenton | armax: in all of my testing all of the agents except for the ones i forced down ones were up so it pretended like it was doing what i thought | 01:09 |
| *** yamahata has quit IRC | 01:09 | |
| *** diegows has joined #openstack-neutron | 01:09 | |
| *** yamahata has joined #openstack-neutron | 01:09 | |
| *** tgohad has joined #openstack-neutron | 01:09 | |
| *** VijayB has quit IRC | 01:10 | |
| *** tsg has quit IRC | 01:10 | |
| *** ijw has quit IRC | 01:11 | |
| *** otherwiseguy has quit IRC | 01:13 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Avoid auto-scheduling for distributed routers https://review.openstack.org/115754 | 01:15 |
| *** sungju has joined #openstack-neutron | 01:15 | |
| *** shakamunyi has joined #openstack-neutron | 01:17 | |
| *** ijw has joined #openstack-neutron | 01:20 | |
| *** stanzgy has joined #openstack-neutron | 01:21 | |
| openstackgerrit | gongysh proposed a change to openstack/neutron: Deletes floating ip related connection states https://review.openstack.org/103475 | 01:21 |
| openstackgerrit | gongysh proposed a change to openstack/neutron: Deletes floating ip related connection states https://review.openstack.org/103475 | 01:24 |
| *** zhipeng has joined #openstack-neutron | 01:25 | |
| ryu25 | hi all, if there are core devs with free time, could you review this patch? https://review.openstack.org/#/c/105355/ It's fairly straight forward, and it would be tremendously helpful for us to move forward with our plugin. Thanks in advance!! | 01:26 |
| *** otherwiseguy has joined #openstack-neutron | 01:26 | |
| *** ijw has quit IRC | 01:28 | |
| *** mwagner_lap has joined #openstack-neutron | 01:29 | |
| *** WackoRobie has joined #openstack-neutron | 01:29 | |
| *** britthouser has joined #openstack-neutron | 01:33 | |
| *** topshare_ has quit IRC | 01:33 | |
| *** amotoki has joined #openstack-neutron | 01:34 | |
| *** britthou_ has quit IRC | 01:35 | |
| *** topshare has joined #openstack-neutron | 01:36 | |
| *** tflynn has quit IRC | 01:37 | |
| *** diegows has quit IRC | 01:39 | |
| *** seizadi has quit IRC | 01:42 | |
| *** topshare has quit IRC | 01:42 | |
| *** topshare has joined #openstack-neutron | 01:46 | |
| *** vivek-ebay has quit IRC | 01:47 | |
| *** zhipeng has quit IRC | 01:47 | |
| *** spandhe has quit IRC | 01:48 | |
| *** topshare has quit IRC | 01:50 | |
| *** topshare has joined #openstack-neutron | 01:51 | |
| *** prasoon has joined #openstack-neutron | 01:52 | |
| openstackgerrit | shihanzhang proposed a change to openstack/neutron: Refactor security group rpc call https://review.openstack.org/111876 | 01:53 |
| *** nlahouti has joined #openstack-neutron | 01:55 | |
| *** sbfox has joined #openstack-neutron | 01:55 | |
| *** topshare_ has joined #openstack-neutron | 01:55 | |
| *** baohua has joined #openstack-neutron | 01:56 | |
| *** topshare has quit IRC | 01:58 | |
| *** nlahouti has quit IRC | 02:02 | |
| *** nlahouti has joined #openstack-neutron | 02:03 | |
| *** baoli has joined #openstack-neutron | 02:04 | |
| *** baoli has quit IRC | 02:08 | |
| nlahouti | salv-orlando: Hi, just as reminder, would you please review the patches that waiting for your reply | 02:09 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Minor refactoring of auto_schedule_routers https://review.openstack.org/114691 | 02:11 |
| *** tongli has quit IRC | 02:12 | |
| *** popw has joined #openstack-neutron | 02:13 | |
| *** tgohad has quit IRC | 02:16 | |
| *** SridharRamaswamy has quit IRC | 02:17 | |
| openstackgerrit | Mandeep Dhami proposed a change to openstack/neutron: Apic drivers enhancements-6: VPC support https://review.openstack.org/115847 | 02:23 |
| *** shakayumi has joined #openstack-neutron | 02:24 | |
| *** shakamunyi has quit IRC | 02:28 | |
| *** alexiz has joined #openstack-neutron | 02:30 | |
| *** jeffrey4l has joined #openstack-neutron | 02:32 | |
| *** iwamoto has joined #openstack-neutron | 02:34 | |
| *** shakayumi has quit IRC | 02:34 | |
| *** topshare_ has quit IRC | 02:36 | |
| *** topshare has joined #openstack-neutron | 02:36 | |
| openstackgerrit | A change was merged to openstack/python-neutronclient: Rename --timeout param to --http-timeout https://review.openstack.org/115109 | 02:37 |
| *** sbfox has quit IRC | 02:39 | |
| *** jckasper has joined #openstack-neutron | 02:40 | |
| *** tsg has joined #openstack-neutron | 02:41 | |
| *** harlowja_ is now known as harlowja_away | 02:42 | |
| *** jeffrey4l has left #openstack-neutron | 02:42 | |
| *** xuhanp has joined #openstack-neutron | 02:44 | |
| *** ramishra has joined #openstack-neutron | 02:48 | |
| *** ramishra_ has joined #openstack-neutron | 02:49 | |
| *** ramishra has quit IRC | 02:52 | |
| *** padkrish has quit IRC | 02:55 | |
| *** baoli has joined #openstack-neutron | 03:05 | |
| *** k_stev has quit IRC | 03:05 | |
| *** britthouser has quit IRC | 03:07 | |
| *** baoli has quit IRC | 03:09 | |
| *** ramishra_ has quit IRC | 03:10 | |
| *** ramishra has joined #openstack-neutron | 03:11 | |
| *** Sukhdev has joined #openstack-neutron | 03:17 | |
| *** ramishra has quit IRC | 03:18 | |
| *** ramishra has joined #openstack-neutron | 03:19 | |
| *** britthouser has joined #openstack-neutron | 03:22 | |
| *** tflynn has joined #openstack-neutron | 03:22 | |
| openstackgerrit | nlahouti proposed a change to openstack/neutron: Cisco DFA ML2 Mechanism Driver - part 2 https://review.openstack.org/110065 | 03:22 |
| *** prasoon has quit IRC | 03:23 | |
| *** tflynn has quit IRC | 03:27 | |
| *** chandankumar has joined #openstack-neutron | 03:27 | |
| *** tflynn has joined #openstack-neutron | 03:27 | |
| *** WackoRobie has quit IRC | 03:32 | |
| *** WackoRobie has joined #openstack-neutron | 03:33 | |
| *** tflynn has quit IRC | 03:37 | |
| *** zz_satyag is now known as satyag | 03:38 | |
| *** WackoRobie has quit IRC | 03:38 | |
| *** satyag has left #openstack-neutron | 03:41 | |
| *** tflynn has joined #openstack-neutron | 03:44 | |
| *** chandankumar has quit IRC | 03:44 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Minor refactoring for add_router_to_l3_agent https://review.openstack.org/115829 | 03:47 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Do not use auto_schedule_routers to add router to agent https://review.openstack.org/115837 | 03:47 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Avoid auto-scheduling for distributed routers https://review.openstack.org/115754 | 03:47 |
| *** geekinutah has joined #openstack-neutron | 03:50 | |
| *** tflynn has quit IRC | 03:50 | |
| *** pcm_ has quit IRC | 03:50 | |
| *** tgohad has joined #openstack-neutron | 03:51 | |
| *** tsg has quit IRC | 03:54 | |
| *** shakamunyi has joined #openstack-neutron | 03:54 | |
| *** alexiz has quit IRC | 03:57 | |
| *** prasoon has joined #openstack-neutron | 04:00 | |
| *** jckasper has quit IRC | 04:01 | |
| *** ijw has joined #openstack-neutron | 04:02 | |
| *** SridharG has joined #openstack-neutron | 04:03 | |
| *** baoli has joined #openstack-neutron | 04:05 | |
| *** ijw has quit IRC | 04:07 | |
| *** cristov_mac has quit IRC | 04:09 | |
| *** baoli has quit IRC | 04:10 | |
| *** otherwiseguy has quit IRC | 04:18 | |
| *** dfarrell07 has joined #openstack-neutron | 04:18 | |
| *** otherwiseguy has joined #openstack-neutron | 04:21 | |
| *** sbfox has joined #openstack-neutron | 04:24 | |
| openstackgerrit | A change was merged to openstack/neutron: Expand arp_responder help text https://review.openstack.org/106860 | 04:41 |
| *** tflynn has joined #openstack-neutron | 04:42 | |
| *** tgohad has quit IRC | 04:43 | |
| *** seizadi has joined #openstack-neutron | 04:56 | |
| *** otherwiseguy has quit IRC | 05:01 | |
| *** nlahouti_ has joined #openstack-neutron | 05:01 | |
| *** nlahouti has quit IRC | 05:03 | |
| *** baoli has joined #openstack-neutron | 05:06 | |
| *** tsg has joined #openstack-neutron | 05:09 | |
| *** baoli has quit IRC | 05:10 | |
| *** topshare has quit IRC | 05:13 | |
| *** topshare has joined #openstack-neutron | 05:17 | |
| *** Sukhdev has quit IRC | 05:17 | |
| *** dfarrell07 has quit IRC | 05:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/114069 | 05:21 |
| *** trinaths has joined #openstack-neutron | 05:23 | |
| *** topshare has quit IRC | 05:24 | |
| *** chandankumar has joined #openstack-neutron | 05:25 | |
| *** tflynn has quit IRC | 05:26 | |
| *** tflynn has joined #openstack-neutron | 05:27 | |
| *** nlahouti_ has quit IRC | 05:27 | |
| *** nlahouti has joined #openstack-neutron | 05:27 | |
| *** padkrish has joined #openstack-neutron | 05:31 | |
| *** padkrish has joined #openstack-neutron | 05:32 | |
| *** topshare has joined #openstack-neutron | 05:34 | |
| *** reed has quit IRC | 05:36 | |
| *** shakamunyi has quit IRC | 05:37 | |
| *** ramishra has quit IRC | 05:41 | |
| *** ramishra has joined #openstack-neutron | 05:42 | |
| *** SridharG has quit IRC | 05:42 | |
| *** tflynn has quit IRC | 05:42 | |
| *** pradipta_away is now known as pradipta | 05:45 | |
| *** otherwiseguy has joined #openstack-neutron | 05:46 | |
| *** ramishra has quit IRC | 05:47 | |
| *** ramishra has joined #openstack-neutron | 05:48 | |
| *** cnesa7 has joined #openstack-neutron | 05:48 | |
| *** teju has joined #openstack-neutron | 05:50 | |
| *** irenab has joined #openstack-neutron | 05:50 | |
| openstackgerrit | Preeti Mirji proposed a change to openstack/neutron: Adm policy can't use other tenants unshared rules https://review.openstack.org/108952 | 05:52 |
| *** otherwiseguy has quit IRC | 05:55 | |
| *** Sukhdev has joined #openstack-neutron | 05:56 | |
| *** lordd_ has joined #openstack-neutron | 05:58 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/115505 | 06:04 |
| *** ijw has joined #openstack-neutron | 06:04 | |
| *** baoli has joined #openstack-neutron | 06:07 | |
| *** ijw has quit IRC | 06:08 | |
| *** baoli has quit IRC | 06:12 | |
| *** rotbeard has joined #openstack-neutron | 06:13 | |
| *** jlibosva has joined #openstack-neutron | 06:15 | |
| openstackgerrit | Paddu Krishnan proposed a change to openstack/neutron: VDP Support: DFA Overlay Type Driver - Part 3 https://review.openstack.org/115452 | 06:15 |
| *** SridharG has joined #openstack-neutron | 06:15 | |
| *** saju_m has joined #openstack-neutron | 06:19 | |
| *** ramishra has quit IRC | 06:21 | |
| *** ramishra has joined #openstack-neutron | 06:22 | |
| *** k4n0 has joined #openstack-neutron | 06:25 | |
| *** seizadi has quit IRC | 06:25 | |
| *** simon-AS559 has joined #openstack-neutron | 06:27 | |
| *** nlahouti has quit IRC | 06:28 | |
| *** SumitNaiksatam_ has joined #openstack-neutron | 06:33 | |
| *** SumitNaiksatam has quit IRC | 06:36 | |
| *** SumitNaiksatam_ is now known as SumitNaiksatam | 06:36 | |
| *** afazekas is now known as __afazekas | 06:55 | |
| *** jschwarz has joined #openstack-neutron | 06:57 | |
| *** nlahouti has joined #openstack-neutron | 06:58 | |
| *** jckasper has joined #openstack-neutron | 06:58 | |
| openstackgerrit | A change was merged to openstack/neutron: corrects the typos in l3_router_plugin's comments https://review.openstack.org/115558 | 06:58 |
| *** saju_m has quit IRC | 06:58 | |
| *** saju_m has joined #openstack-neutron | 06:59 | |
| *** Sukhdev has quit IRC | 07:01 | |
| *** jckasper has quit IRC | 07:03 | |
| *** shakamunyi has joined #openstack-neutron | 07:03 | |
| *** Altran has joined #openstack-neutron | 07:04 | |
| *** nlahouti has quit IRC | 07:05 | |
| *** ihrachyshka has joined #openstack-neutron | 07:06 | |
| *** yongli has joined #openstack-neutron | 07:06 | |
| *** roeyc has joined #openstack-neutron | 07:07 | |
| *** padkrish has quit IRC | 07:07 | |
| *** baoli has joined #openstack-neutron | 07:08 | |
| *** zhipeng has joined #openstack-neutron | 07:08 | |
| *** padkrish has joined #openstack-neutron | 07:08 | |
| *** shakamunyi has quit IRC | 07:08 | |
| *** Altran has quit IRC | 07:12 | |
| *** baoli has quit IRC | 07:12 | |
| *** pasquier-s has joined #openstack-neutron | 07:15 | |
| *** Altran has joined #openstack-neutron | 07:18 | |
| *** afazekas has joined #openstack-neutron | 07:18 | |
| *** afazekas has quit IRC | 07:18 | |
| *** ihrachyshka has quit IRC | 07:19 | |
| *** karimb has joined #openstack-neutron | 07:21 | |
| *** padkrish has quit IRC | 07:22 | |
| *** skolekonov has joined #openstack-neutron | 07:23 | |
| *** prasoon has quit IRC | 07:27 | |
| *** nlahouti has joined #openstack-neutron | 07:31 | |
| *** sungju has quit IRC | 07:32 | |
| *** nlahouti has quit IRC | 07:36 | |
| *** doude has joined #openstack-neutron | 07:38 | |
| *** otherwiseguy has joined #openstack-neutron | 07:38 | |
| *** banix has joined #openstack-neutron | 07:38 | |
| *** lordd_ has quit IRC | 07:39 | |
| *** afazekas has joined #openstack-neutron | 07:43 | |
| *** otherwiseguy has quit IRC | 07:44 | |
| *** jpich has joined #openstack-neutron | 07:51 | |
| *** aix has joined #openstack-neutron | 07:53 | |
| *** matrohon has joined #openstack-neutron | 07:53 | |
| *** sergeysh has joined #openstack-neutron | 07:56 | |
| *** prasoon has joined #openstack-neutron | 07:58 | |
| *** rms_13 has joined #openstack-neutron | 08:00 | |
| *** prasoon has quit IRC | 08:03 | |
| *** shakamunyi has joined #openstack-neutron | 08:04 | |
| *** lori|away is now known as lori | 08:04 | |
| *** amaretskiy has joined #openstack-neutron | 08:06 | |
| *** ijw has joined #openstack-neutron | 08:06 | |
| *** rms_13 has quit IRC | 08:06 | |
| *** prasoon has joined #openstack-neutron | 08:07 | |
| *** ramishra has quit IRC | 08:08 | |
| *** baoli has joined #openstack-neutron | 08:08 | |
| *** shakamunyi has quit IRC | 08:09 | |
| *** devvesa has joined #openstack-neutron | 08:10 | |
| *** ijw has quit IRC | 08:10 | |
| *** Longgeek has joined #openstack-neutron | 08:10 | |
| *** Longgeek has quit IRC | 08:11 | |
| *** weiw has joined #openstack-neutron | 08:11 | |
| *** Longgeek has joined #openstack-neutron | 08:12 | |
| *** baoli has quit IRC | 08:13 | |
| *** ollivier has joined #openstack-neutron | 08:16 | |
| *** ramishra has joined #openstack-neutron | 08:18 | |
| *** Altran has quit IRC | 08:27 | |
| *** amuller has joined #openstack-neutron | 08:27 | |
| *** sbfox has quit IRC | 08:27 | |
| *** samuelbercovici has quit IRC | 08:28 | |
| *** Altran has joined #openstack-neutron | 08:28 | |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Rename range to avoid shadowing the builtin https://review.openstack.org/115175 | 08:30 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Remove SELECT FOR UPDATE from db_base_plugin https://review.openstack.org/100963 | 08:30 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Add _store_ip_allocation method https://review.openstack.org/115598 | 08:30 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Add port id and network id to IPAM related methods https://review.openstack.org/115597 | 08:30 |
| *** nlahouti has joined #openstack-neutron | 08:32 | |
| *** otherwiseguy has joined #openstack-neutron | 08:33 | |
| *** dave_tucker is now known as dave_tucker_zzz | 08:34 | |
| *** nlahouti has quit IRC | 08:36 | |
| *** amaretskiy has quit IRC | 08:36 | |
| *** otherwiseguy has quit IRC | 08:37 | |
| *** ramishra_ has joined #openstack-neutron | 08:38 | |
| *** ramishra has quit IRC | 08:39 | |
| *** ihrachyshka has joined #openstack-neutron | 08:39 | |
| *** amaretskiy has joined #openstack-neutron | 08:41 | |
| *** sdmonov has joined #openstack-neutron | 08:43 | |
| *** sambetts has joined #openstack-neutron | 08:45 | |
| *** obondarev has quit IRC | 08:45 | |
| trinaths | amotoki: Hi | 08:46 |
| *** aix has quit IRC | 08:46 | |
| *** igordcard has joined #openstack-neutron | 08:47 | |
| *** sdmonov has quit IRC | 08:55 | |
| *** mrmoje has joined #openstack-neutron | 08:55 | |
| *** lordd_ has joined #openstack-neutron | 08:59 | |
| *** nlahouti has joined #openstack-neutron | 09:03 | |
| *** lordd_ has quit IRC | 09:03 | |
| *** shakamunyi has joined #openstack-neutron | 09:05 | |
| *** sungju has joined #openstack-neutron | 09:06 | |
| *** trinaths_afk has joined #openstack-neutron | 09:06 | |
| *** ijw has joined #openstack-neutron | 09:06 | |
| *** jp_at_hp has joined #openstack-neutron | 09:07 | |
| *** nlahouti has quit IRC | 09:07 | |
| *** trinaths has quit IRC | 09:08 | |
| *** trinaths_afk is now known as trinaths | 09:08 | |
| *** baoli has joined #openstack-neutron | 09:09 | |
| *** shakamunyi has quit IRC | 09:09 | |
| *** dsneddon has quit IRC | 09:10 | |
| *** ijw has quit IRC | 09:11 | |
| *** baoli has quit IRC | 09:14 | |
| openstackgerrit | Romil Gupta proposed a change to openstack/neutron: Tunnel ID range validation for VXLAN/GRE networks https://review.openstack.org/89982 | 09:14 |
| *** obondarev has joined #openstack-neutron | 09:20 | |
| *** igordcard has quit IRC | 09:20 | |
| *** roeyc has quit IRC | 09:20 | |
| *** claudiub has joined #openstack-neutron | 09:23 | |
| *** aix has joined #openstack-neutron | 09:24 | |
| *** igordcard has joined #openstack-neutron | 09:25 | |
| *** sdmonov has joined #openstack-neutron | 09:25 | |
| *** obondarev_ has joined #openstack-neutron | 09:26 | |
| *** otherwiseguy has joined #openstack-neutron | 09:27 | |
| *** obondarev has quit IRC | 09:28 | |
| *** obondarev_ is now known as obondarev | 09:28 | |
| *** sdmonov has quit IRC | 09:30 | |
| *** otherwiseguy has quit IRC | 09:31 | |
| *** topshare has quit IRC | 09:32 | |
| *** otherwiseguy has joined #openstack-neutron | 09:32 | |
| *** topshare has joined #openstack-neutron | 09:33 | |
| *** nlahouti has joined #openstack-neutron | 09:33 | |
| *** tsg has quit IRC | 09:34 | |
| *** banix has quit IRC | 09:35 | |
| *** otherwiseguy has quit IRC | 09:37 | |
| *** nlahouti has quit IRC | 09:37 | |
| *** sungju has quit IRC | 09:39 | |
| *** pradipta is now known as pradipta_away | 09:41 | |
| *** topshare has quit IRC | 09:44 | |
| openstackgerrit | veena proposed a change to openstack/neutron: Removes duplicate checks for device existance https://review.openstack.org/113897 | 09:44 |
| *** veena has joined #openstack-neutron | 09:45 | |
| openstackgerrit | Koteswara Rao Kelam proposed a change to openstack/neutron: shared policy shouldn't have unshared rules https://review.openstack.org/115912 | 09:45 |
| *** sungju has joined #openstack-neutron | 09:47 | |
| *** iwamoto has quit IRC | 09:47 | |
| *** topshare has joined #openstack-neutron | 09:49 | |
| *** mrmoje_ has joined #openstack-neutron | 09:54 | |
| *** mrmoje has quit IRC | 09:56 | |
| *** ryu25 has quit IRC | 09:57 | |
| *** yamahata has quit IRC | 09:58 | |
| openstackgerrit | Romil Gupta proposed a change to openstack/neutron: Schema enhancement to support MultiSegment Network https://review.openstack.org/103546 | 09:58 |
| *** baohua has quit IRC | 10:01 | |
| *** xuhanp has quit IRC | 10:03 | |
| *** nlahouti has joined #openstack-neutron | 10:04 | |
| *** matrohon has quit IRC | 10:06 | |
| *** prasoon has quit IRC | 10:07 | |
| *** nlahouti has quit IRC | 10:08 | |
| *** baoli has joined #openstack-neutron | 10:10 | |
| *** aswadr has quit IRC | 10:12 | |
| *** baoli has quit IRC | 10:14 | |
| *** flwang1 has joined #openstack-neutron | 10:18 | |
| *** pasquier-s has quit IRC | 10:22 | |
| *** roeyc has joined #openstack-neutron | 10:23 | |
| *** aepifanov has joined #openstack-neutron | 10:24 | |
| *** aepifanov has quit IRC | 10:25 | |
| *** aepifanov has joined #openstack-neutron | 10:26 | |
| openstackgerrit | enikanorov proposed a change to openstack/neutron: Refactor l3 agent scheduling logic to avoid unwanted failures https://review.openstack.org/115383 | 10:28 |
| *** topshare has quit IRC | 10:28 | |
| *** eezhova has quit IRC | 10:33 | |
| *** eezhova has joined #openstack-neutron | 10:34 | |
| *** flwang1 has quit IRC | 10:34 | |
| *** Longgeek has quit IRC | 10:35 | |
| *** nlahouti has joined #openstack-neutron | 10:35 | |
| *** Longgeek has joined #openstack-neutron | 10:36 | |
| *** flwang1 has joined #openstack-neutron | 10:36 | |
| *** ramishra_ has quit IRC | 10:36 | |
| *** nlahouti has quit IRC | 10:40 | |
| *** Longgeek has quit IRC | 10:40 | |
| *** topshare has joined #openstack-neutron | 10:42 | |
| *** topshare has quit IRC | 10:44 | |
| *** NunoLoureiro has joined #openstack-neutron | 10:44 | |
| NunoLoureiro | Hello | 10:44 |
| NunoLoureiro | I'm new to openstack and I was wondering if anyone could help me | 10:45 |
| *** ramishra has joined #openstack-neutron | 10:45 | |
| NunoLoureiro | I'm having troubles establishing an ipsec connection between two openstack installations | 10:46 |
| jschwarz | NunoLoureiro, what's your question? | 10:46 |
| NunoLoureiro | I was able to get it working one time, but when I changed the subnets it stopped working | 10:46 |
| NunoLoureiro | The VPN services are Active in both locations | 10:47 |
| jschwarz | Unfortunately I don't have any VPN experience, but mayhaps someone else here will be able to help | 10:47 |
| NunoLoureiro | But i keep getting troubles on the ipsec connection negotiation | 10:47 |
| openstackgerrit | veena proposed a change to openstack/neutron: Removes duplicate checks for device existance https://review.openstack.org/113897 | 10:48 |
| NunoLoureiro | In the tcpdump i keep getting an udp port isakmp unreachable | 10:48 |
| NunoLoureiro | So it must be aan iptables issue | 10:48 |
| openstackgerrit | veena proposed a change to openstack/neutron: Removes duplicate checks for device existance https://review.openstack.org/113897 | 10:50 |
| jschwarz | veena, hi | 10:54 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessesManager in the dhcp_agent and dnsmasq driver https://review.openstack.org/115935 | 10:54 |
| *** ZZelle has joined #openstack-neutron | 10:55 | |
| veena | Hi jschwarz | 10:55 |
| jschwarz | veena, regarding your latest patch - what about the plug function defined in MetaInterfaceDriver? | 10:55 |
| *** jaypipes has quit IRC | 10:55 | |
| jschwarz | that one doesn't check for not ip_lib.device_exists | 10:55 |
| jschwarz | the inner driver probably does but it still has a call to _set_device_plugin_tag which might do harm | 10:56 |
| veena | yeah, I didn't check for the plug() function in MetaInterfaceDriver, since in bug description only LinuxInterfaceDriver was mentioned. | 10:58 |
| *** lordd_ has joined #openstack-neutron | 10:59 | |
| *** teju has quit IRC | 11:00 | |
| *** prasoon has joined #openstack-neutron | 11:00 | |
| veena | So, please suggest me jschwarz on how I should make the changes | 11:00 |
| *** ramishra_ has joined #openstack-neutron | 11:01 | |
| *** ramishra_ has quit IRC | 11:01 | |
| *** lazy_prince has joined #openstack-neutron | 11:01 | |
| *** ramishra_ has joined #openstack-neutron | 11:02 | |
| *** prasoon has quit IRC | 11:03 | |
| ZZelle | obondarev, hi | 11:03 |
| *** lordd_ has quit IRC | 11:04 | |
| obondarev | ZZelle: hi | 11:05 |
| *** ramishra has quit IRC | 11:05 | |
| jschwarz | veena, I don't know metaplugin very well, but if I were you i'd look into whether or not a call to ip_lib.device_exists should be added to plug()/unplug() | 11:05 |
| openstackgerrit | Jaume Devesa proposed a change to openstack/neutron: BGP Dynamic Routing: introduce routing instance https://review.openstack.org/115554 | 11:05 |
| *** nlahouti has joined #openstack-neutron | 11:06 | |
| *** Longgeek has joined #openstack-neutron | 11:06 | |
| ZZelle | obondarev, if you have some time could you also have a look to https://review.openstack.org/111194 and https://review.openstack.org/111198 which are linked to https://review.openstack.org/115351 ? | 11:06 |
| ZZelle | obondarev, it's about code share between gre/vxlan type drivers | 11:07 |
| obondarev | ZZelle: sure, I will | 11:07 |
| *** prasoon has joined #openstack-neutron | 11:07 | |
| veena | sure jschwarz, I'll look into it | 11:08 |
| *** Longgeek has quit IRC | 11:09 | |
| ZZelle | obondarev, thanks | 11:09 |
| *** Longgeek has joined #openstack-neutron | 11:09 | |
| *** nlahouti has quit IRC | 11:10 | |
| openstackgerrit | Elena Ezhova proposed a change to openstack/neutron: Forbid regular users to reset admin-only attrs to default values https://review.openstack.org/114531 | 11:10 |
| openstackgerrit | Jaume Devesa proposed a change to openstack/neutron: BGP Dynamic Routing: adding agent and scheduling https://review.openstack.org/115667 | 11:10 |
| obondarev | ZZelle: np, thanks for getting rid of duplicated code! | 11:10 |
| *** baoli has joined #openstack-neutron | 11:11 | |
| openstackgerrit | Jaume Devesa proposed a change to openstack/neutron: BGP Dynamic Routing: adding BGP driver https://review.openstack.org/115938 | 11:12 |
| *** baoli has quit IRC | 11:15 | |
| openstackgerrit | veena proposed a change to openstack/neutron: Removes duplicate checks for device existance https://review.openstack.org/113897 | 11:17 |
| *** mikedillion has joined #openstack-neutron | 11:18 | |
| devvesa | hi amuller: i've just read your post about L3 HA. It is great | 11:20 |
| amuller | devvesa: Glad it helped :) | 11:20 |
| devvesa | i feel ready for review it :) | 11:20 |
| amuller | Please do! | 11:20 |
| devvesa | i have 'finished' my work on bgp stuff, so meanwhile i wait for reviews, I'll find time to do it | 11:21 |
| amuller | Great | 11:21 |
| amuller | After this freeze today I should have more time, I think your BGP work is interesting, I should review it | 11:22 |
| *** rromans has joined #openstack-neutron | 11:23 | |
| devvesa | good to know | 11:24 |
| *** pcm_ has joined #openstack-neutron | 11:27 | |
| *** amotoki has quit IRC | 11:28 | |
| *** diegows has joined #openstack-neutron | 11:31 | |
| *** Zemeio has quit IRC | 11:33 | |
| jschwarz | salv-orlando, thanks a lot for the review! :) | 11:35 |
| *** mikedillion has quit IRC | 11:35 | |
| salv-orlando | jschwarz: np. I need to be afk for a little now. I’ll be back later if you want to discuss details | 11:35 |
| jschwarz | I will respond asap to the comments you wrong regarding the select statement. Also, can you look at what Maru wrote about a week ago? | 11:35 |
| *** nlahouti has joined #openstack-neutron | 11:37 | |
| *** ramishra_ has quit IRC | 11:37 | |
| *** seizadi has joined #openstack-neutron | 11:38 | |
| *** nlahouti has quit IRC | 11:41 | |
| openstackgerrit | A change was merged to openstack/neutron: Rename range to avoid shadowing the builtin https://review.openstack.org/115175 | 11:46 |
| *** sambetts has quit IRC | 11:47 | |
| *** diego__ has joined #openstack-neutron | 11:47 | |
| *** diego__ is now known as snowgust | 11:47 | |
| *** sambetts has joined #openstack-neutron | 11:48 | |
| *** ramishra has joined #openstack-neutron | 11:49 | |
| openstackgerrit | Stanislav Kudriashev proposed a change to openstack/python-neutronclient: Clean-up shell run and run_subcommand methods https://review.openstack.org/115955 | 11:49 |
| *** bashok has joined #openstack-neutron | 11:51 | |
| openstackgerrit | enikanorov proposed a change to openstack/neutron: Refactor l3 agent scheduling logic to avoid unwanted failures https://review.openstack.org/115383 | 11:53 |
| *** ramishra has quit IRC | 12:01 | |
| openstackgerrit | Cedric Brandily proposed a change to openstack/neutron: Refactor type_tunnel/gre/vxlan to reduce duplicate code https://review.openstack.org/111198 | 12:01 |
| *** Longgeek has quit IRC | 12:03 | |
| openstackgerrit | Cedric Brandily proposed a change to openstack/neutron: Refactor test_type_gre/vxlan to reduce duplicate code https://review.openstack.org/115351 | 12:05 |
| *** ramishra has joined #openstack-neutron | 12:07 | |
| *** nlahouti has joined #openstack-neutron | 12:07 | |
| *** sungju has quit IRC | 12:08 | |
| *** ijw has joined #openstack-neutron | 12:08 | |
| *** baoli has joined #openstack-neutron | 12:11 | |
| *** nlahouti has quit IRC | 12:11 | |
| *** Longgeek has joined #openstack-neutron | 12:12 | |
| *** Longgeek has quit IRC | 12:13 | |
| *** ijw has quit IRC | 12:13 | |
| *** Longgeek has joined #openstack-neutron | 12:14 | |
| *** aepifanov has quit IRC | 12:14 | |
| *** aepifanov has joined #openstack-neutron | 12:14 | |
| *** diegows has quit IRC | 12:16 | |
| *** baoli has quit IRC | 12:16 | |
| openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Detect if iproute2 support SR-IOV commands https://review.openstack.org/114968 | 12:18 |
| *** pasquier-s has joined #openstack-neutron | 12:18 | |
| *** WackoRobie has joined #openstack-neutron | 12:19 | |
| *** flaviof is now known as flaviof_zzz | 12:23 | |
| *** ramishra has quit IRC | 12:24 | |
| *** dave_tucker_zzz is now known as dave_tucker | 12:24 | |
| *** zhipeng has quit IRC | 12:26 | |
| *** trinaths has quit IRC | 12:28 | |
| *** wanghong has quit IRC | 12:29 | |
| *** sambetts has quit IRC | 12:29 | |
| *** sambetts has joined #openstack-neutron | 12:29 | |
| *** ollivier has quit IRC | 12:31 | |
| *** sambetts has quit IRC | 12:31 | |
| *** dfarrell07 has joined #openstack-neutron | 12:31 | |
| *** sambetts has joined #openstack-neutron | 12:32 | |
| *** sambetts has quit IRC | 12:33 | |
| *** wanghong has joined #openstack-neutron | 12:33 | |
| *** lordd_ has joined #openstack-neutron | 12:33 | |
| *** sambetts has joined #openstack-neutron | 12:34 | |
| *** roeyc has quit IRC | 12:34 | |
| *** aepifanov has quit IRC | 12:37 | |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [2/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/92600 | 12:38 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [1/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/92599 | 12:38 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [3/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115964 | 12:38 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [4/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115965 | 12:38 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [5/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115966 | 12:38 |
| *** nlahouti has joined #openstack-neutron | 12:38 | |
| jschwarz | amuller, isn't there a law against spamming here? | 12:39 |
| *** dfarrell07 has quit IRC | 12:39 | |
| amuller | jschwarz: I don't think it's the best code but calling it spam is just insulting =D | 12:39 |
| *** sgordon has joined #openstack-neutron | 12:39 | |
| *** aepifanov has joined #openstack-neutron | 12:39 | |
| *** weiw has quit IRC | 12:41 | |
| *** salv-orlando has quit IRC | 12:42 | |
| *** nlahouti has quit IRC | 12:43 | |
| openstackgerrit | Vivekanandan Narasimhan proposed a change to openstack/neutron: Fix DVR to service LBaaS VIP Ports https://review.openstack.org/114141 | 12:44 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [5/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115966 | 12:45 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [4/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115965 | 12:45 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [3/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115964 | 12:45 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [2/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/92600 | 12:45 |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [1/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/92599 | 12:45 |
| openstackgerrit | Stanislav Kudriashev proposed a change to openstack/python-neutronclient: Unify doc-strings format https://review.openstack.org/115970 | 12:45 |
| *** saju_m has quit IRC | 12:46 | |
| *** radez_g0n3 is now known as radez | 12:48 | |
| amuller | irenab: Hey Irena, about your patch, is there a command that you can just try executing, instead of parsing the help? | 12:50 |
| amuller | irenab: Possibly in a namespace if isolation is a concern | 12:51 |
| *** syedawaisali has joined #openstack-neutron | 12:53 | |
| amuller | irenab: Theoretically the check itself can be 5~ lines of code | 12:53 |
| *** flaviof_zzz is now known as flaviof | 12:53 | |
| *** veena has quit IRC | 12:53 | |
| *** pradk has joined #openstack-neutron | 12:54 | |
| *** syedawaisali has left #openstack-neutron | 12:54 | |
| *** aix has quit IRC | 12:57 | |
| *** yfauser has joined #openstack-neutron | 12:58 | |
| *** jckasper has joined #openstack-neutron | 12:58 | |
| *** WackoRobie has quit IRC | 12:58 | |
| *** yfauser has left #openstack-neutron | 12:58 | |
| *** WackoRobie has joined #openstack-neutron | 12:58 | |
| *** jckasper has quit IRC | 13:01 | |
| *** jckasper_ has joined #openstack-neutron | 13:01 | |
| *** thomasem has joined #openstack-neutron | 13:02 | |
| *** jckasper_ has quit IRC | 13:02 | |
| *** jckasper_ has joined #openstack-neutron | 13:02 | |
| *** WackoRobie has quit IRC | 13:03 | |
| rossella_s | jschwarz veena: just read your discussion. MetaInterfaceDriver will call another driver, which will have to take care of checking for the device existence. All the drivers in tree do that. BTW no harm can be done, if the device doesn't exist it will be created. I think it's responsibility of driver to implement the check for the device existence. There's no need of checking twice | 13:04 |
| *** julim has joined #openstack-neutron | 13:05 | |
| *** xianghuihui has joined #openstack-neutron | 13:06 | |
| openstackgerrit | Stanislav Kudriashev proposed a change to openstack/python-neutronclient: Small improve of str2dict function https://review.openstack.org/115975 | 13:06 |
| jschwarz | rossella_s, but MetaInterfaceDriver.plug() doesn't only serve as a proxy to the other driver - it also calls "_set_device_plugin_tag", which looks like it runs an actual "ip link" command. | 13:07 |
| rossella_s | jschwarz: that line is executed after the plug() | 13:07 |
| *** tty57 has joined #openstack-neutron | 13:07 | |
| *** jaypipes has joined #openstack-neutron | 13:07 | |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [4/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115965 | 13:08 |
| rossella_s | jschwarz: at that point the device must exist or plug() would have raised an exception | 13:08 |
| jschwarz | rossella_s, yes, but prior to the patchset veena offered it wasn't call when the device didn't exist | 13:08 |
| jschwarz | (also veena is not logged on...) | 13:08 |
| *** roeyc has joined #openstack-neutron | 13:09 | |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [5/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115966 | 13:09 |
| jschwarz | rossella_s, but plug() doesn't raise an exception - all implementations check if the device exists, and if not simply return (no exceptions) | 13:09 |
| jschwarz | thus MetaInterfaceDriver has no way of knowing of the device in question exists or not | 13:09 |
| *** xianghui has quit IRC | 13:09 | |
| jschwarz | Unless I understood you incorrectly (which I may have) :) | 13:10 |
| *** xianghui has joined #openstack-neutron | 13:10 | |
| rossella_s | jschwarz: I sent veena an email to read the chat history. I think I understood it incorrectly | 13:11 |
| *** aepifanov has quit IRC | 13:11 | |
| *** aepifanov has joined #openstack-neutron | 13:11 | |
| rossella_s | jschwarz: your point is that set_device_plugin_tag is not executed now if we call plug() only after checking that the device doesn't exist | 13:11 |
| *** xianghui has quit IRC | 13:12 | |
| jschwarz | rossella_s, exactly | 13:12 |
| *** baoli has joined #openstack-neutron | 13:12 | |
| rossella_s | jschwarz: MetaInterfaceDriver.plug() should make no assumption on how it will be called. If it's a problem that the device already exists than this should be checked inside MetaInterfaceDriver.plug() | 13:12 |
| *** xianghui has joined #openstack-neutron | 13:12 | |
| *** xianghuihui has quit IRC | 13:13 | |
| rossella_s | jschwarz: and avoid the call _set_device_plugin_tag if the device already exists | 13:13 |
| rossella_s | jschwarz: anyway I don't think it's a problem...that method just sets an alias. Anyway it's good that you raised this point | 13:13 |
| jschwarz | rossella_s, I disagree | 13:15 |
| jschwarz | rossella_s, If I have a function that's supposed to add a new subnet with a specified CIDR, the function better make sure there will be no conflicts | 13:16 |
| *** baoli has quit IRC | 13:17 | |
| rossella_s | jschwarz: what kind of conflict? | 13:17 |
| jschwarz | rossella_s, similarly, if plug() can't work as intended if the device doesn't exist, then it should check that first | 13:17 |
| jschwarz | rossella_s, for example 2 subnets connected to the same network shouldn't have the same cidr | 13:17 |
| rossella_s | jschwarz: that's exactly my point, plug() should check, not the caller of plug() | 13:18 |
| jschwarz | rossella_s, my bad then - we are both agreeing :) I misread you | 13:18 |
| rossella_s | jschwarz: maybe I should explain my though more clearly. :)ç | 13:19 |
| jschwarz | rossella_s, nah, I misread that entire sentence. When I read it again it made perfect sense | 13:19 |
| jschwarz | rossella_s, so in short MetaInterfaceDriver.plug() should check if the device exists or not, even if it's only an alias (garbage is garbage) | 13:19 |
| rossella_s | jschwarz: yes. If it's a problem that the device already exists, that should be checked in MetaInterfaceDriver.plug() | 13:20 |
| jschwarz | rossella_s, I'll write a comment on this subject on her review | 13:21 |
| *** openstackgerrit has quit IRC | 13:21 | |
| rossella_s | jschwarz: thanks a lot! | 13:21 |
| jschwarz | rossella_s, thank you :) | 13:21 |
| *** amuller_ has joined #openstack-neutron | 13:21 | |
| *** salv-orlando has joined #openstack-neutron | 13:22 | |
| *** dfarrell07 has joined #openstack-neutron | 13:23 | |
| *** amuller has quit IRC | 13:23 | |
| *** ramishra has joined #openstack-neutron | 13:25 | |
| *** Austin_ has joined #openstack-neutron | 13:25 | |
| *** tty57 has quit IRC | 13:27 | |
| jschwarz | rossella_s, done :) | 13:27 |
| *** jobewan has joined #openstack-neutron | 13:27 | |
| jschwarz | salv-orlando, hey, do you have some time to discuss by refactor patch a bit? :) | 13:28 |
| *** saju_m has joined #openstack-neutron | 13:28 | |
| *** salv-orlando_ has joined #openstack-neutron | 13:29 | |
| *** ramishra has quit IRC | 13:29 | |
| *** snowgust has quit IRC | 13:30 | |
| jschwarz | salv-orlando_, ping? | 13:30 |
| *** diego_ has joined #openstack-neutron | 13:30 | |
| *** salv-orlando has quit IRC | 13:31 | |
| *** salv-orlando_ is now known as salv-orlando | 13:31 | |
| *** tomoe__ has joined #openstack-neutron | 13:32 | |
| *** tty57 has joined #openstack-neutron | 13:34 | |
| *** tomoe_ has quit IRC | 13:34 | |
| *** absubram_ has joined #openstack-neutron | 13:35 | |
| *** lazy_prince is now known as killer_prince | 13:38 | |
| *** nlahouti has joined #openstack-neutron | 13:40 | |
| HenryG | jlibosva: ping | 13:41 |
| *** krtaylor has quit IRC | 13:41 | |
| jlibosva | HenryG: hi | 13:41 |
| HenryG | jlibosva: I figured out why the first unit test is failing in the import all models | 13:42 |
| jlibosva | HenryG: please tell me, I was about to run with my head against wall :) | 13:42 |
| HenryG | jlibosva: sure, one sec, need to bring up the env again | 13:43 |
| *** radez is now known as radez_g0n3 | 13:43 | |
| *** nlahouti has quit IRC | 13:44 | |
| *** dfarrell07 has quit IRC | 13:44 | |
| HenryG | jlibosva: This one: BigSwitch.ServerManagerTests.test_no_cert_error | 13:45 |
| HenryG | jlibosva: It patches 'os.path.exists' with side_effect of 5 return values | 13:46 |
| *** mestery has joined #openstack-neutron | 13:47 | |
| HenryG | jlibosva: Turns out that importing models uses pkgutils which uses os.path.exists for each module, so it quickly consumes the 5 return values and bails out with StopIteration. | 13:48 |
| jlibosva | HenryG: I think you are not right. | 13:48 |
| jlibosva | HenryG: because importing happens before the test, not inside of context that patches os.path.exists | 13:49 |
| jlibosva | HenryG: I think it's because once "some" module is imported, it overwrites "ssl" attribute of ServerPool | 13:49 |
| jlibosva | HenryG: if you take a look on the implementation of _get_combined_cert_for_server() | 13:50 |
| *** irenab has quit IRC | 13:50 | |
| jlibosva | HenryG: it starts with "if self.ssl" which is by the time of execution "False" - meaning it doesn't even reach the place where exception is raised | 13:50 |
| jlibosva | but I need to confirm that | 13:50 |
| *** baohua has joined #openstack-neutron | 13:50 | |
| jlibosva | HenryG: I'm talking about file neutron/plugins/bigswitch/servermanager.py | 13:51 |
| *** xgerman has joined #openstack-neutron | 13:51 | |
| *** diegows has joined #openstack-neutron | 13:52 | |
| jlibosva | HenryG: what do you think about that? | 13:52 |
| * jlibosva is going for coffee | 13:52 | |
| *** devvesa has quit IRC | 13:55 | |
| *** k4n0 has quit IRC | 13:55 | |
| *** WackoRobie has joined #openstack-neutron | 13:56 | |
| *** devvesa has joined #openstack-neutron | 13:57 | |
| *** lordd_ has quit IRC | 13:57 | |
| *** nlahouti_ has joined #openstack-neutron | 13:57 | |
| *** armax has joined #openstack-neutron | 13:58 | |
| *** dfarrell07 has joined #openstack-neutron | 13:59 | |
| *** nlahouti_ has quit IRC | 13:59 | |
| jlibosva | HenryG: that's probably a trash what I said :) There is server.ssl = True before calling that function. I'll keep digging | 13:59 |
| *** openstackgerrit has joined #openstack-neutron | 14:00 | |
| *** jorgem has joined #openstack-neutron | 14:00 | |
| HenryG | jlibosva: Well I also had some coffee and I agree I was wrong. | 14:00 |
| *** samuelbercovici has joined #openstack-neutron | 14:00 | |
| *** mestery has quit IRC | 14:00 | |
| *** ollivier has joined #openstack-neutron | 14:00 | |
| HenryG | jlibosva: But definitely os.path.exists is being called more than 5 times | 14:01 |
| *** mestery has joined #openstack-neutron | 14:01 | |
| *** salv-orlando has quit IRC | 14:01 | |
| HenryG | jlibosva: Seems to be called by translation _() now, while it wasn't before | 14:02 |
| HenryG | jlibosva: that might be a clue | 14:03 |
| *** alexpilotti has joined #openstack-neutron | 14:03 | |
| HenryG | jlibosva: Maybe importing everything is enabling translation? | 14:04 |
| alexpilotti | hi guys, I have a 1 line patch with 2 x +2, but no +a: https://review.openstack.org/#/c/115812/ | 14:05 |
| alexpilotti | can a core reviewer please hit the magic +a? thanks!:-) | 14:06 |
| jlibosva | HenryG: yeah, you're right. The StopIteration jumps out from L339 neutron/plugins/bigswitch/servermanager.py | 14:07 |
| *** crc32 has joined #openstack-neutron | 14:08 | |
| *** alexpilotti has quit IRC | 14:08 | |
| HenryG | jlibosva: I guess somehow you need to exclude the translation monkeypatcher module from imports? | 14:09 |
| armax | enikanorov_: ping | 14:09 |
| *** rpothier has joined #openstack-neutron | 14:09 | |
| * jlibosva should start reading the whole backtraces | 14:10 | |
| *** mestery has quit IRC | 14:10 | |
| *** mestery_ has joined #openstack-neutron | 14:10 | |
| *** ijw has joined #openstack-neutron | 14:10 | |
| jlibosva | HenryG: I thought the whole time the exception is from assertRaises when comparing some results | 14:10 |
| *** tongli has joined #openstack-neutron | 14:11 | |
| *** chuckC_ has quit IRC | 14:11 | |
| *** mestery_ is now known as mestery | 14:11 | |
| *** vjardin_ has joined #openstack-neutron | 14:12 | |
| *** baoli has joined #openstack-neutron | 14:13 | |
| *** alexpilotti has joined #openstack-neutron | 14:14 | |
| HenryG | jlibosva: Do you know how/where the translation is patched in? | 14:14 |
| *** ijw has quit IRC | 14:15 | |
| jlibosva | HenryG: what do you mean by patched in? | 14:16 |
| *** alexpilotti has quit IRC | 14:17 | |
| *** AbsinthMind has joined #openstack-neutron | 14:17 | |
| HenryG | jlibosva: I assumed it was patched in somehow. Looking now, it seems to use some lazy evaluation? | 14:17 |
| *** mrmoje_ has quit IRC | 14:17 | |
| *** krtaylor has joined #openstack-neutron | 14:17 | |
| *** baoli has quit IRC | 14:17 | |
| HenryG | jlibosva: gettextutils.enable_lazy() ? | 14:18 |
| jlibosva | HenryG: ah, I don't know. | 14:18 |
| *** mrmoje has joined #openstack-neutron | 14:19 | |
| *** crc32 has quit IRC | 14:19 | |
| *** ChuckC has quit IRC | 14:20 | |
| *** aix has joined #openstack-neutron | 14:20 | |
| HenryG | jlibosva: neutron/server/__init__.py line 33 | 14:20 |
| HenryG | jlibosva: Have you tried skipping __init__.py files when importing models? | 14:21 |
| jlibosva | HenryG: nope | 14:21 |
| ajo_ | Having lots of fun here, trying to cleanup the test_linux_dhcp tests, after refactoring the dhcp agent to use "external_process" instead of lots of duplicated code.. | 14:22 |
| *** alexpilotti has joined #openstack-neutron | 14:22 | |
| ajo_ | because the tests mix both, coverage already done in external_process, and it's own tests | 14:22 |
| *** ajo_ is now known as ajo | 14:22 | |
| *** vivek-ebay has joined #openstack-neutron | 14:22 | |
| *** yeungp has joined #openstack-neutron | 14:23 | |
| *** ramishra has joined #openstack-neutron | 14:24 | |
| *** amuller__ has joined #openstack-neutron | 14:25 | |
| *** alexpilotti has quit IRC | 14:26 | |
| *** pradipta_away is now known as pradipta | 14:26 | |
| *** lordd_ has joined #openstack-neutron | 14:28 | |
| *** amuller_ has quit IRC | 14:28 | |
| *** mrmoje has quit IRC | 14:29 | |
| jlibosva | HenryG: it must be cause by some recent change. It wasn't failing on Aug 15. By that time we still had create_all though. | 14:29 |
| openstackgerrit | Andrey Epifanov proposed a change to openstack/neutron: Add ipv6 forwarding for router namespaces https://review.openstack.org/115993 | 14:29 |
| jlibosva | s/cause/caused/ | 14:30 |
| *** pradipta is now known as pradipta_away | 14:30 | |
| *** tsg has joined #openstack-neutron | 14:30 | |
| openstackgerrit | Andrey Epifanov proposed a change to openstack/neutron: Add ipv6 forwarding for router namespaces https://review.openstack.org/115993 | 14:32 |
| *** AbsinthMind has quit IRC | 14:33 | |
| *** lordd_ has quit IRC | 14:34 | |
| *** yfauser has joined #openstack-neutron | 14:35 | |
| *** alexpilotti has joined #openstack-neutron | 14:35 | |
| *** yfauser has left #openstack-neutron | 14:35 | |
| *** otherwiseguy has joined #openstack-neutron | 14:35 | |
| *** kbringard has joined #openstack-neutron | 14:35 | |
| *** dfarrell07 has quit IRC | 14:37 | |
| *** jgrimm has joined #openstack-neutron | 14:37 | |
| *** ramishra_ has joined #openstack-neutron | 14:39 | |
| *** samuelbercovici has quit IRC | 14:41 | |
| *** ramishra has quit IRC | 14:42 | |
| *** samuelbercovici has joined #openstack-neutron | 14:42 | |
| *** amuller__ is now known as amuller | 14:43 | |
| openstackgerrit | Elena Ezhova proposed a change to openstack/neutron: Fix policy rules for adding and removing router interfaces https://review.openstack.org/115997 | 14:44 |
| *** lordd_ has joined #openstack-neutron | 14:45 | |
| *** amuller has quit IRC | 14:47 | |
| *** WackoRobie has quit IRC | 14:48 | |
| rkukura | armax: have a moment for a question? | 14:48 |
| openstackgerrit | Elena Ezhova proposed a change to openstack/neutron: Fix policy rules for adding and removing router interfaces https://review.openstack.org/115997 | 14:49 |
| *** zilberstein has joined #openstack-neutron | 14:50 | |
| zilberstein | good day #openstack-neutron | 14:50 |
| *** diego_ has quit IRC | 14:50 | |
| zilberstein | could someone throw a hint on why neutron, configured to use ML2 with linux bridge plugin, gives warning: "Failed to bind port N on host localhost" | 14:50 |
| armax | rkukura: sure | 14:50 |
| *** diego_ has joined #openstack-neutron | 14:51 | |
| *** ramishra_ has quit IRC | 14:52 | |
| *** mestery has quit IRC | 14:52 | |
| zilberstein | http://pastie.org/pastes/9491784/text | 14:53 |
| jlibosva | HenryG: I'm starting to regret we didn't separated models into several files dedicated to models... | 14:53 |
| *** NunoLoureiro has quit IRC | 14:54 | |
| HenryG | jlibosva: I would prefer too. Or some variation of it. | 14:55 |
| *** alexpilotti has quit IRC | 14:55 | |
| *** dave_tucker is now known as dave_tucker_zzz | 14:55 | |
| zilberstein | nova raises exception: RescheduledException: Build of instance 1ddcfeb7-716d-4fb8-bc8c-329237cb59f2 was re-scheduled: Unexpected vif_type=binding_failed | 14:56 |
| rkukura | armax, or anyone else: Have you ever run into intermittent issues running UTs where DB queries return stale data? I’m seeing this about 20-30% of the time from queries in get_bound_port_context() in ML2 in my hierarchcal port binding branch in the test_l2population.test_host_changed[_twice] tests. | 14:56 |
| zilberstein | NovaException: Unexpected vif_type=binding_failed | 14:56 |
| *** alexpilotti has joined #openstack-neutron | 14:56 | |
| *** padkrish has joined #openstack-neutron | 14:56 | |
| *** carl_baldwin has joined #openstack-neutron | 14:56 | |
| rkukura | armax: What’s odd is it happened consistently, but as I’ve added more temporary logging to track it down, the frequency has decreased. | 14:57 |
| armax | rkukura: um, how are you running the tests? | 14:57 |
| * ajo hates code duplication | 14:57 | |
| armax | rkukura: I never experienced this behavior myself, the UTs either fail consistently or don’t | 14:58 |
| rkukura | armax: “tox -e py27 neutron.tests.unit.ml2.drivers.test_l2population.TestL2PopulationRpcTestCase.test_host_changed” | 14:58 |
| armax | rkukura: this leads me to believe that the DB is not cleaned up properly across tests | 14:58 |
| armax | rkukura: so you’re running a single test? | 14:58 |
| *** coolsvap has quit IRC | 14:58 | |
| rkukura | armax: It happens even if I run “time tox -e py27 neutron.tests.unit.ml2.drivers.test_l2population.TestL2PopulationRpcTestCase.test_host_changed_twice”, which just runs a single test. | 14:59 |
| HenryG | jlibosva: It may not be needed to have them all in neutron/db/models/* but instead just a common naming convention like foo_model_cfg.py | 14:59 |
| *** ChuckC has joined #openstack-neutron | 14:59 | |
| armax | rkukura: so the test fails intermittently? | 14:59 |
| *** alexpilotti has quit IRC | 15:00 | |
| jlibosva | HenryG: like keystone? | 15:00 |
| *** chuckC_ has joined #openstack-neutron | 15:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/114069 | 15:00 |
| rkukura | armax: Correct - about 20% of the time with the logging I’ve added. | 15:00 |
| *** roeyc has quit IRC | 15:00 | |
| armax | rkukura: what’s the failure mode? | 15:01 |
| *** sbfox has joined #openstack-neutron | 15:01 | |
| *** Altran has left #openstack-neutron | 15:01 | |
| *** salv-orlando has joined #openstack-neutron | 15:02 | |
| *** saju_m has quit IRC | 15:02 | |
| rkukura | armax: Both these tests create a port with one host_id, then do an update that changes the host_id. The result from the update has a the new host_id. I added a show after the update that also always returns the new host_id. But the tests hen do a get_device_details RPC, and the query inside this returns a binding record with the old host_id. | 15:02 |
| armax | rkukura: so can you reproduce this on master too? I can try to have a look at it locally here | 15:03 |
| rkukura | armax: This could have been happening all along (I haven’t checked). | 15:03 |
| *** AbsinthMind has joined #openstack-neutron | 15:03 | |
| *** rudrarugge has joined #openstack-neutron | 15:04 | |
| *** padkrish has quit IRC | 15:04 | |
| armax | rkukura: so you’re saying that if I run tox -e py27 neutron.tests.unit.ml2.drivers.test_l2population.TestL2PopulationRpcTestCase.test_host_change | 15:04 |
| armax | rkukura: I should be able to see the intermittent failures myself? | 15:04 |
| *** jorgem has quit IRC | 15:05 | |
| *** padkrish has joined #openstack-neutron | 15:05 | |
| rkukura | My branch now has a new port_binding_levels table where the host_id is part of the key (so it works with DVR and normal bindings), and getting the wrong host_id from the port_bindings quey cause my new query of the port_binding_levels table to fail. | 15:05 |
| *** mikedillion has joined #openstack-neutron | 15:05 | |
| rkukura | armax: So, without my branch, I don’t thing we’d have ever noticed if this host_id was wrong. | 15:05 |
| armax | rkukura: ic | 15:06 |
| *** dkehn_ has joined #openstack-neutron | 15:07 | |
| armax | rkukura: so you’ve made changes to the code and the model, but not the tests themselves? | 15:07 |
| enikanorov__ | armax: pong | 15:08 |
| armax | enikanorov__: ping | 15:08 |
| armax | enikanorov__: :) | 15:08 |
| rkukura | armax: I’ve added all kinds of logging to debug this, and added the show to the test, but otherwise the test is unchanged | 15:08 |
| enikanorov__ | armax: if've replied in my review, i'll try to rebase the patch | 15:08 |
| jlibosva | HenryG: hm, I don't know what I did but now I'm not even able to start UT. Even if I checkout the version from review. cleared .tox and still import errors | 15:08 |
| armax | enikanorov__: oh, cool, I’ll have a look | 15:08 |
| *** amuller has joined #openstack-neutron | 15:08 | |
| *** marun has joined #openstack-neutron | 15:08 | |
| armax | enikanorov__: thanks | 15:08 |
| *** skolekonov has quit IRC | 15:08 | |
| *** ihrachyshka has quit IRC | 15:09 | |
| HenryG | jlibosva: remove .pyc files? | 15:09 |
| *** dkehnx has quit IRC | 15:09 | |
| *** dkehn has quit IRC | 15:09 | |
| jlibosva | HenryG: oh yeah! | 15:09 |
| jlibosva | HenryG: thanks!! | 15:09 |
| jlibosva | HenryG++ | 15:09 |
| *** yamahata has joined #openstack-neutron | 15:10 | |
| armax | rkukura: hang on let me have a look at the test | 15:10 |
| *** coolsvap has joined #openstack-neutron | 15:10 | |
| *** dkehn has joined #openstack-neutron | 15:11 | |
| *** ijw has joined #openstack-neutron | 15:11 | |
| *** nlahouti has joined #openstack-neutron | 15:11 | |
| armax | rkukura: so you’re saying this one fails? https://github.com/openstack/neutron/blob/master/neutron/tests/unit/ml2/drivers/test_l2population.py#L634 | 15:12 |
| *** samuelbercovici has quit IRC | 15:12 | |
| rkukura | armax: Yes, that and the test_host_changed_twice test in the same file. | 15:13 |
| armax | rkukura: what’s the stacktrace like? Can you paste it? | 15:13 |
| rkukura | armax: The symptom of the test failure is that assert on line 683 fails. | 15:14 |
| *** baoli has joined #openstack-neutron | 15:14 | |
| rkukura | armax: But the wrong host_id, if it is occuring, will not cause the test to fail on master. | 15:14 |
| rossella_s | carl_baldwin: ping | 15:15 |
| armax | rkukura: you mean the assert fails because the actual call does not match the expected one? | 15:15 |
| *** ijw has quit IRC | 15:15 | |
| *** tobybot11 has joined #openstack-neutron | 15:16 | |
| armax | rkukura: a host mismatch shouldn’t affect the test as far as I can tell, unless something else is happening under the covers | 15:17 |
| armax | rkukura: without a trace it’s very difficult for me to visualize | 15:17 |
| carl_baldwin | rossella_s: In the L3 meeting at the moment. | 15:17 |
| rkukura | armax: No, I’m pretty sure the expected call does not occur at all. With my changes, the wrong host_id means that the port_binding_level table entry/entries aren’t found, so there is no segment, and a warning about this is logged within get_device_details(). | 15:18 |
| *** baohua_ has joined #openstack-neutron | 15:18 | |
| armax | rkukura: ic, so the failure is about no call at all | 15:18 |
| *** baoli has quit IRC | 15:19 | |
| *** dkehn_ is now known as dkehnx | 15:19 | |
| rossella_s | carl_baldwin: sorry, will ping you later | 15:19 |
| *** Austin_ has quit IRC | 15:19 | |
| rkukura | armax: Yes, and I’ve worked out why the expected call doesn’t happen. Its because get_device_details does not find a segment for the binding, which is because the value of host_id in the port_binding record is wrong. | 15:19 |
| armax | rkukura: what if if you registered only one agent in the test? | 15:19 |
| carl_baldwin | rossella_s: I’ll ping you when the meeting is through. Hopefully it is not getting too late for you. | 15:19 |
| *** vjardin_ has quit IRC | 15:19 | |
| rkukura | But show() calls from the test between these return the right value for host_id | 15:19 |
| *** VijayB has joined #openstack-neutron | 15:20 | |
| armax | rkukura: maybe it’s not worth trying it out, but I wonder if more than one agent throws you off | 15:20 |
| *** NunoLoureiro has joined #openstack-neutron | 15:20 | |
| rkukura | armax: It might be worth seeing if the underlying DB issue is happening on master, but adding some code that logs an error if the binding.host_id does not match the host from the RPC request. | 15:21 |
| *** baohua has quit IRC | 15:21 | |
| rkukura | armax: Do you know if the RPC in the UT could possibly be getting handled by a different thread than the REST calls? | 15:22 |
| *** ollivier has quit IRC | 15:22 | |
| *** ramishra has joined #openstack-neutron | 15:23 | |
| *** rwsu has quit IRC | 15:23 | |
| armax | rkukura: good question, I am not sure, but I’d be tempted to say no | 15:23 |
| zilberstein | neutron server log: INFO keystoneclient.middleware.auth_token [-] Invalid user token - rejecting request | 15:23 |
| zilberstein | WTF ? | 15:23 |
| rkukura | armax: I’m suspecting that sqlite might not make transactions commited in one thread visible immediately from other threads. Or maybe somehow sqlalchemy is reusing an out-of-date cached instance of the model class or something. | 15:23 |
| * zilberstein tries to force it to work for two weeks already | 15:24 | |
| armax | rkukura: I don’t think there’s contention in accessing the DB from a single test | 15:25 |
| openstackgerrit | Sridar Kandaswamy proposed a change to openstack/neutron: Adds FWaaS service plugin for Cisco CSR1kv https://review.openstack.org/115308 | 15:26 |
| openstackgerrit | Sridar Kandaswamy proposed a change to openstack/neutron: Adds router service plugin for CSR1kv https://review.openstack.org/101002 | 15:26 |
| armax | rkukura: I’d suggest to post the code and see if we can see the failure in action, I am lurking in the dark at the moment | 15:27 |
| *** ramishra has quit IRC | 15:27 | |
| armax | rkukura: tangentially, do you think that your changes are going to supersede this one? https://review.openstack.org/#/c/109046/ | 15:27 |
| rkukura | armax: I’m going to switch to master and see if I can find evidence of the wrong host_id getting fetched there. | 15:28 |
| *** afazekas has quit IRC | 15:29 | |
| rkukura | armax: I think we may want to move the port binding logic to the manager at some point, but doing so right now while lots of work is going on on that code would make things difficult. | 15:29 |
| armax | rkukura: agreed…so you think we should try and do that after your hierarchy patch? | 15:29 |
| armax | rkukura: or I could rebase it on top of yours | 15:30 |
| rkukura | armax: I also think we will be able to unify the DVR and non-DVR schema and logic before Juno ships. | 15:30 |
| armax | rkukura: yup, that was the plan all along | 15:30 |
| rkukura | armax: I’m assuming that kind of cleanup work can happen as bug fixes after feature freeze. | 15:30 |
| armax | rkukura: in theory yes, we’d need to figure out the extent of the patch | 15:31 |
| rkukura | armax: Hopefully a series of incremental improvements will get the job done. | 15:32 |
| *** otherwiseguy has quit IRC | 15:32 | |
| armax | rkukura: yup | 15:32 |
| *** otherwiseguy has joined #openstack-neutron | 15:32 | |
| armax | rkukura: so back to your failure, let me know if there’s anything I can do to help you troubleshoot this | 15:33 |
| *** rohara has quit IRC | 15:33 | |
| *** chandankumar has quit IRC | 15:36 | |
| rkukura | armax: thanks | 15:37 |
| armax | rkukura: sorry I couldn’t be more helpful, but if you want me to run some tests feel free to poke me later | 15:39 |
| *** rwsu has joined #openstack-neutron | 15:39 | |
| openstackgerrit | Paul Michali proposed a change to openstack/neutron: Rework and enable VPNaaS UT for Cisco CSR REST https://review.openstack.org/116018 | 15:39 |
| HenryG | jlibosva: I managed to fix it! At least for the first test case. | 15:39 |
| *** annegentle has joined #openstack-neutron | 15:40 | |
| *** trinaths has joined #openstack-neutron | 15:40 | |
| rkukura | armax: If/when I manage to get this patch into shape, I’d appreciate help making sure it didn’t break DVR. | 15:40 |
| armax | rkukura: you got it | 15:40 |
| *** baohua_ has quit IRC | 15:40 | |
| *** jaypipes has quit IRC | 15:41 | |
| *** trinaths_afk has joined #openstack-neutron | 15:41 | |
| rkukura | armax: My current thinking is that we can unify the schema by splitting what was the original port_binding table into two tables. One keeps the binding-related attributes that are set through the REST API, including host_id. The second has host_id as part of the key, and stores the results of the binding, which in the DVR case are per-host, but in the non-DVR case, uses the host_id from the first table as part of its key. In the pa | 15:42 |
| rkukura | I’m working on, I add what would be a third table, where the key is made up of the port_id, the host_id, and level, and stores the driver and segment for each level of a hierarchical bindings. | 15:42 |
| armax | rkukura: that’s in line with my thinking as well, I think we talked about this very split when you were in town | 15:43 |
| rkukura | armax: Right, but it started making more sense as I tried to figure out how to add the hierarchical binding support. | 15:44 |
| *** trinaths has quit IRC | 15:44 | |
| *** trinaths_afk is now known as trinaths | 15:44 | |
| armax | rkukura: gotcha | 15:44 |
| *** Madkiss has joined #openstack-neutron | 15:45 | |
| Madkiss | hello everybody | 15:45 |
| Madkiss | folks, I've just heard somebody talking about a successor for Neutron | 15:45 |
| Madkiss | referring to some 2-weeks old mailing list post | 15:45 |
| Madkiss | could somebody point me to that post kindly? thanks in advance. | 15:45 |
| HenryG | jlibosva: I added comments to the review | 15:45 |
| *** doude has quit IRC | 15:45 | |
| rkukura | armax: Using master with no changes, I’m seeing the same UT failure, but only about 3% of the time!!! | 15:46 |
| armax | rkukura: interesting... | 15:46 |
| armax | rkukura: let me see if I can do the same | 15:46 |
| rkukura | armax: Never mind - I was looking at some of the older results mixed in with the new ones | 15:46 |
| armax | rkukura: oh, ic | 15:47 |
| *** otherwiseguy has quit IRC | 15:47 | |
| rkukura | armax: Its only on the 60th run or so, and there were failures left from before in the remaining 40 runs | 15:47 |
| rkukura | armax: I would have been surprised, because I don’t think a wrong host_id would cause the test to fail on master. | 15:48 |
| *** aepifanov has quit IRC | 15:48 | |
| *** yamahata has quit IRC | 15:48 | |
| armax | rkukura: that’s weird…the db is cleared up every time…there’s shouldn’t be any aging of sort | 15:49 |
| *** reed has joined #openstack-neutron | 15:49 | |
| rkukura | armax: I’m just running a single UT, so its not an interaction across tests. | 15:49 |
| *** zslot has joined #openstack-neutron | 15:51 | |
| armax | rkukura: right, that’s what I am saying | 15:51 |
| *** jaypipes has joined #openstack-neutron | 15:54 | |
| *** padkrish has quit IRC | 15:55 | |
| *** SridharG has quit IRC | 15:56 | |
| *** mikedillion has quit IRC | 15:58 | |
| *** ijw has joined #openstack-neutron | 15:58 | |
| *** otherwiseguy has joined #openstack-neutron | 15:59 | |
| *** pasquier-s has quit IRC | 15:59 | |
| *** SridharG has joined #openstack-neutron | 15:59 | |
| *** mikedillion has joined #openstack-neutron | 16:00 | |
| *** krtaylor has quit IRC | 16:00 | |
| *** padkrish has joined #openstack-neutron | 16:00 | |
| *** aix has quit IRC | 16:00 | |
| *** vjardin_ has joined #openstack-neutron | 16:01 | |
| carl_baldwin | rossella_s: ping | 16:01 |
| *** dguitarbite has quit IRC | 16:01 | |
| *** ihrachyshka has joined #openstack-neutron | 16:01 | |
| rossella_s | carl_baldwin: Hi Carl. Just wanted to ask you to have a look at https://review.openstack.org/#/c/115597/ when you have some time. I'd like to be sure I didn't misinterpret your suggestion | 16:02 |
| carl_baldwin | rossella_s: Yes, I can do that. | 16:02 |
| rossella_s | carl_baldwin: thanks a lot | 16:02 |
| *** amuller_ has joined #openstack-neutron | 16:03 | |
| *** jschwarz has quit IRC | 16:03 | |
| rkukura | armax: I am now fairly certain I’m seeing the wrong host_id intermittently on master, with a couple lines of code I’ve added to check for this! | 16:03 |
| *** amuller has quit IRC | 16:04 | |
| carl_baldwin | rossella_s: Oleg raises a good point. What do you think? Do you think breaking this out is worth it? | 16:04 |
| rossella_s | carl_baldwin: I think it's a fair point, I will merge that patch with the main one | 16:05 |
| rossella_s | carl_baldwin: as regards bug 1358554 , unfortunately I have to go in 5 minutes. Can we talk tomorrow about it? I was wondering, maybe I can catch you early in my morning. | 16:05 |
| *** rudrarugge has quit IRC | 16:05 | |
| carl_baldwin | rossella_s: You correctly interpreted my suggestion. But, my suggestion may not have been fully baked. | 16:05 |
| carl_baldwin | rossella_s: Yes, we can plan to talk in the morning. What time would be good? | 16:05 |
| *** jschwarz has joined #openstack-neutron | 16:06 | |
| rossella_s | carl_baldwin: what's your timezone? I am UTC+2 | 16:06 |
| rossella_s | carl_baldwin: it's 10 am for you now I guess :) | 16:06 |
| carl_baldwin | MDT. So, UTC - 6 | 16:07 |
| carl_baldwin | rossella_s: Yes, it is just after 10am. | 16:07 |
| rossella_s | carl_baldwin: I will try to connect in a few hours if I can...otherwise I will wait till tomorrow when you start | 16:08 |
| carl_baldwin | So, 7:30 or 8:00 MDT would be good (that is 3:30 for you?). I’ve got an 8:30am meeting. | 16:08 |
| carl_baldwin | rossella_s: Okay. | 16:08 |
| rossella_s | carl_baldwin:thanks a lot! ttyl | 16:08 |
| *** jobewan has quit IRC | 16:08 | |
| carl_baldwin | rossella_s: take care | 16:09 |
| rossella_s | carl_baldwin: you too | 16:09 |
| *** padkrish has quit IRC | 16:11 | |
| *** aix has joined #openstack-neutron | 16:13 | |
| openstackgerrit | pritesh proposed a change to openstack/neutron: Remove the Cisco Nexus monolithic plugin as ml2 driver takes it place https://review.openstack.org/116028 | 16:13 |
| *** baoli has joined #openstack-neutron | 16:14 | |
| *** vivek-ebay has quit IRC | 16:15 | |
| rkukura | armax: Are you still around? | 16:15 |
| *** devvesa has quit IRC | 16:18 | |
| *** diego_ is now known as snowgust | 16:19 | |
| *** baoli has quit IRC | 16:19 | |
| *** vishwanathj has joined #openstack-neutron | 16:20 | |
| *** aix has quit IRC | 16:21 | |
| *** padkrish has joined #openstack-neutron | 16:26 | |
| *** emagana has joined #openstack-neutron | 16:27 | |
| *** aepifanov has joined #openstack-neutron | 16:30 | |
| *** tty57 has quit IRC | 16:33 | |
| *** tomoe__ has quit IRC | 16:33 | |
| *** sbfox has quit IRC | 16:34 | |
| *** jpich has quit IRC | 16:36 | |
| *** padkrish has quit IRC | 16:36 | |
| *** bashok has quit IRC | 16:38 | |
| *** simon-AS559 has quit IRC | 16:40 | |
| *** mageshgv has joined #openstack-neutron | 16:40 | |
| *** zhhuabj has quit IRC | 16:43 | |
| *** networkstatic has joined #openstack-neutron | 16:44 | |
| *** sbfox has joined #openstack-neutron | 16:44 | |
| rkukura | armax: Are you around? | 16:44 |
| *** krtaylor has joined #openstack-neutron | 16:45 | |
| *** amaretskiy has quit IRC | 16:46 | |
| sbfox | Hey all, is anybody here running an openstack cloud with vlans instead of gre/vxlan overlays? I specifically looking for help with how the metadata server works with a non-neutron default route | 16:48 |
| *** dsneddon has joined #openstack-neutron | 16:48 | |
| *** vjardin_ has quit IRC | 16:49 | |
| *** jaypipes has quit IRC | 16:49 | |
| *** rudrarugge has joined #openstack-neutron | 16:49 | |
| *** pballand has joined #openstack-neutron | 16:51 | |
| armax | rkukura: pong | 16:55 |
| *** jaypipes has joined #openstack-neutron | 16:55 | |
| *** jschwarz has quit IRC | 16:56 | |
| rkukura | armax: I’m about to post a patch to gerrit that will cause the test_host_changed_twice testcase to fail if the query returns a ml2_port_binding record with a stale host_id. This fails about 35% of the time for me. | 16:56 |
| openstackgerrit | Robert Kukura proposed a change to openstack/neutron: DO NOT MERGE - detect stale host_id in get_device_details RPC https://review.openstack.org/116039 | 16:57 |
| armax | rkukura: cool…I’ll have a look | 16:57 |
| armax | rkukura: I’ll pull it and play with it locally | 16:57 |
| rkukura | armax: thanks! | 16:58 |
| armax | rkukura: np…I’ll try to get back to you in the next hour or so | 16:58 |
| *** jlibosva has quit IRC | 16:59 | |
| *** saju_m has joined #openstack-neutron | 16:59 | |
| rkukura | armax: Great. I’m going to grab lunch. Back soon. | 16:59 |
| *** k_stev has joined #openstack-neutron | 17:00 | |
| openstackgerrit | Magesh GV proposed a change to openstack/python-neutronclient: Group Policy CLI-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/104013 | 17:00 |
| *** dguitarbite has joined #openstack-neutron | 17:00 | |
| *** stanzgy has quit IRC | 17:00 | |
| *** amuller__ has joined #openstack-neutron | 17:00 | |
| *** amuller__ is now known as amuller | 17:00 | |
| *** zhhuabj has joined #openstack-neutron | 17:01 | |
| *** AbsinthMind has quit IRC | 17:01 | |
| trinaths | can some review this patchset https://review.openstack.org/#/c/109659/30. | 17:04 |
| *** amuller_ has quit IRC | 17:04 | |
| *** dkehn has quit IRC | 17:08 | |
| *** dkehnx has quit IRC | 17:10 | |
| *** gduan has joined #openstack-neutron | 17:10 | |
| *** dkehn has joined #openstack-neutron | 17:10 | |
| *** garyduan has quit IRC | 17:11 | |
| *** padkrish has joined #openstack-neutron | 17:11 | |
| *** sdmonov has joined #openstack-neutron | 17:11 | |
| openstackgerrit | Magesh GV proposed a change to openstack/python-neutronclient: Group Policy CLI-2: Classifiers, Actions, and Rules https://review.openstack.org/110798 | 17:11 |
| openstackgerrit | Magesh GV proposed a change to openstack/python-neutronclient: Group Policy CLI-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/104013 | 17:11 |
| *** dkehn_ has joined #openstack-neutron | 17:12 | |
| *** dkehn__ has joined #openstack-neutron | 17:13 | |
| *** dkehn has quit IRC | 17:14 | |
| *** dkehn__ has quit IRC | 17:14 | |
| absubram_ | could I please get some core reviews for https://review.openstack.org/#/c/105514/.. It's been sitting with a few+1s for a while in review now.. thanks and sorry for the shout out.. | 17:14 |
| *** dkehn_ is now known as dkehnx | 17:14 | |
| *** dkehn has joined #openstack-neutron | 17:14 | |
| *** baoli has joined #openstack-neutron | 17:15 | |
| *** ijw has quit IRC | 17:18 | |
| *** ijw has joined #openstack-neutron | 17:19 | |
| *** baoli has quit IRC | 17:20 | |
| *** dkehn has quit IRC | 17:20 | |
| *** dkehn has joined #openstack-neutron | 17:20 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Minor refactoring of auto_schedule_routers https://review.openstack.org/114691 | 17:21 |
| *** karimb has quit IRC | 17:21 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Minor refactoring for add_router_to_l3_agent https://review.openstack.org/115829 | 17:23 |
| *** pcm_ has quit IRC | 17:23 | |
| *** ijw has quit IRC | 17:23 | |
| *** tong_ has joined #openstack-neutron | 17:23 | |
| *** ihrachyshka has quit IRC | 17:23 | |
| *** annegentle has quit IRC | 17:24 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Do not use auto_schedule_routers to add router to agent https://review.openstack.org/115837 | 17:24 |
| *** mestery has joined #openstack-neutron | 17:24 | |
| *** geekinutah has quit IRC | 17:25 | |
| *** harlowja_away is now known as harlowja_ | 17:25 | |
| *** spandhe has joined #openstack-neutron | 17:26 | |
| *** jp_at_hp has quit IRC | 17:26 | |
| *** tongli has quit IRC | 17:26 | |
| *** roeyc has joined #openstack-neutron | 17:26 | |
| *** abrimer__ has quit IRC | 17:27 | |
| *** saju_m has quit IRC | 17:27 | |
| openstackgerrit | Paddu Krishnan proposed a change to openstack/neutron: VDP Support: DFA Overlay Type Driver - Part 3 https://review.openstack.org/115452 | 17:28 |
| *** pcm_ has joined #openstack-neutron | 17:28 | |
| *** tflynn has joined #openstack-neutron | 17:29 | |
| *** tobybot11 has quit IRC | 17:30 | |
| *** roeyc has quit IRC | 17:30 | |
| *** roeyc has joined #openstack-neutron | 17:30 | |
| *** mestery has quit IRC | 17:31 | |
| *** mestery has joined #openstack-neutron | 17:32 | |
| *** mestery has quit IRC | 17:32 | |
| *** mestery has joined #openstack-neutron | 17:32 | |
| *** chandankumar has joined #openstack-neutron | 17:34 | |
| *** chandankumar has quit IRC | 17:37 | |
| *** markmcclain has joined #openstack-neutron | 17:38 | |
| *** flwang1 has quit IRC | 17:38 | |
| *** otherwiseguy has quit IRC | 17:39 | |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [4/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115965 | 17:41 |
| *** diegows has quit IRC | 17:41 | |
| openstackgerrit | Assaf Muller proposed a change to openstack/neutron: [5/5] Clean up Neutron resources upon tenant deletion https://review.openstack.org/115966 | 17:41 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessesManager in the dhcp_agent and dnsmasq driver https://review.openstack.org/115935 | 17:41 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessesManager to watch over external processes https://review.openstack.org/112798 | 17:41 |
| *** lordd_ has quit IRC | 17:42 | |
| *** lordd_ has joined #openstack-neutron | 17:42 | |
| *** Sukhdev has joined #openstack-neutron | 17:43 | |
| *** mestery has quit IRC | 17:43 | |
| trinaths | can some review this patchset https://review.openstack.org/#/c/109659/30. Its having a couple of +1s and need a +2. Thanks in advance and very sorry to ask it now. | 17:46 |
| *** VijayB_ has joined #openstack-neutron | 17:46 | |
| *** SumitNaiksatam has quit IRC | 17:47 | |
| *** SridharRamaswamy has joined #openstack-neutron | 17:48 | |
| *** otherwiseguy has joined #openstack-neutron | 17:48 | |
| *** tongli has joined #openstack-neutron | 17:48 | |
| *** VijayB_ has quit IRC | 17:49 | |
| *** jprovazn has joined #openstack-neutron | 17:49 | |
| *** VijayB has quit IRC | 17:49 | |
| *** tong_ has quit IRC | 17:52 | |
| *** sbfox has quit IRC | 17:52 | |
| *** sbfox has joined #openstack-neutron | 17:52 | |
| *** sbfox has quit IRC | 17:52 | |
| *** claudiub_ has joined #openstack-neutron | 17:53 | |
| *** roeyc has quit IRC | 17:55 | |
| *** claudiub has quit IRC | 17:57 | |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: Fixing URL for stats collection Closes-Bug: #1359869 https://review.openstack.org/116049 | 17:58 |
| *** yeungp has quit IRC | 17:58 | |
| *** annegentle has joined #openstack-neutron | 18:00 | |
| *** SumitNaiksatam has joined #openstack-neutron | 18:00 | |
| *** NunoLoureiro has left #openstack-neutron | 18:00 | |
| *** sambetts has quit IRC | 18:00 | |
| *** spandhe has quit IRC | 18:01 | |
| *** prasoon has quit IRC | 18:03 | |
| *** mikedillion has quit IRC | 18:05 | |
| *** igordcard has quit IRC | 18:05 | |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: NetScaler LBaaS driver does not need to create SNAT port, it is now done by the middleware NetScaler Control Center. Closes-Bug: #1359889 https://review.openstack.org/116052 | 18:05 |
| *** wanghong has quit IRC | 18:06 | |
| *** annegentle has quit IRC | 18:06 | |
| *** markmcclain has quit IRC | 18:06 | |
| *** AbsinthMind has joined #openstack-neutron | 18:07 | |
| *** pradk has quit IRC | 18:07 | |
| *** AbsinthMind has quit IRC | 18:08 | |
| *** geekinutah has joined #openstack-neutron | 18:09 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Avoid auto-scheduling for distributed routers https://review.openstack.org/115754 | 18:09 |
| *** pcm__ has joined #openstack-neutron | 18:10 | |
| *** pcm_ has quit IRC | 18:10 | |
| *** ijw has joined #openstack-neutron | 18:10 | |
| *** annegentle has joined #openstack-neutron | 18:11 | |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: NetScaler LBaaS driver does not need to create SNAT port, it is now done by the middleware NetScaler Control Center. Closes-Bug: #1359889 https://review.openstack.org/116052 | 18:11 |
| *** spandhe has joined #openstack-neutron | 18:11 | |
| *** morganfainberg is now known as morganfainberg_Z | 18:13 | |
| *** mandeep has joined #openstack-neutron | 18:15 | |
| *** diegows has joined #openstack-neutron | 18:15 | |
| *** baoli has joined #openstack-neutron | 18:16 | |
| *** lordd_ has quit IRC | 18:16 | |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: Pool Members status will be ACTIVE/UP when the monitor in the NetScaler backend has detected it is UP and will be INACTIVE/Down when the NetScaler backend has detected it is DOWN. Closes-Bug: #1359890 https://review.openstack.org/116058 | 18:17 |
| *** radez_g0n3 is now known as radez | 18:17 | |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: Pool Members status will be ACTIVE/UP when the monitor in the NetScaler backend has detected it is UP and will be INACTIVE/Down when the NetScaler backend has detected it is DOWN. https://review.openstack.org/116058 | 18:18 |
| *** dfarrell07 has joined #openstack-neutron | 18:19 | |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: NetScaler LBaaS Driver: enable pool member status refresh https://review.openstack.org/116058 | 18:20 |
| kevinbenton | enikanorov_: ping | 18:20 |
| enikanorov_ | kevinbenton: pong | 18:20 |
| *** baoli has quit IRC | 18:21 | |
| kevinbenton | enikanorov_: do the fuel people trust you? they don’t seem to believe me when I say that running multiple l3 and dhcp agents is right for neutron | 18:21 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessesManager in the l3_agent https://review.openstack.org/114931 | 18:21 |
| kevinbenton | enikanorov_: maybe you can comment on this bug. https://bugs.launchpad.net/fuel/+bug/1359082 | 18:21 |
| ajo | enikanorov_, they don't believe you? :) | 18:21 |
| enikanorov_ | kevinbenton: i will, let me look | 18:21 |
| kevinbenton | ajo: they don’t believe me :-) | 18:22 |
| enikanorov_ | kevinbenton: i totally understand what you're trying to do there | 18:23 |
| ajo | ah, probably they rely on A/P configuration | 18:23 |
| kevinbenton | ajo: it’s just a misconfiguration. there is nothing actually relying on it | 18:23 |
| kevinbenton | ajo: we encounted this using fuel | 18:23 |
| *** moha_hunt has joined #openstack-neutron | 18:23 | |
| kevinbenton | ajo: they don’t actually have something to sync the routers over or anything | 18:24 |
| openstackgerrit | Assaf Muller proposed a change to openstack/python-neutronclient: Refactor CreateRouter to use update_dict https://review.openstack.org/116060 | 18:24 |
| ajo | yes, in RH we use A/P like this too (two nodes, one active, other passive, same host_id for the l3 agent) | 18:24 |
| ajo | to avoid a bunch of "headless" l3 qrouters... | 18:25 |
| ajo | but... | 18:25 |
| ajo | you could have two A/P pairs, for example | 18:25 |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: NetScaler LBaaS Driver: Enable async creation of VIP in the NetScaler backend https://review.openstack.org/116061 | 18:25 |
| ajo | if you want to load share, and yet prevent a headless issue | 18:25 |
| ajo | kevinbenton: which would be in good combination with your rescheduling patch | 18:26 |
| *** mrmoje has joined #openstack-neutron | 18:26 | |
| ajo | not sure, it'd require some experimentation | 18:26 |
| ajo | but.. | 18:26 |
| ajo | I wonder if it's their point.. | 18:27 |
| kevinbenton | ajo: i don’t know. it seems like they are worried that multiple l3 agents causes conflicting addresses | 18:27 |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: NetScaler LBaaS Driver: Enable async creation of VIP https://review.openstack.org/116061 | 18:27 |
| *** mikedillion has joined #openstack-neutron | 18:28 | |
| ajo | kevinbenton, I think I know what they mean | 18:28 |
| ajo | you will have an IP conflict on the tenant networks | 18:28 |
| ajo | and the external network | 18:28 |
| openstackgerrit | Assaf Muller proposed a change to openstack/python-neutronclient: Fix CLI support for DVR, take 2 https://review.openstack.org/114263 | 18:28 |
| kevinbenton | ajo: not by letting multiple l3 agents run | 18:28 |
| ajo | if one of your l3-agent nodes get's disconnected from the AMQP network | 18:28 |
| kevinbenton | ajo: only if my rescheduling patch is enabled | 18:28 |
| ajo | then, all qrouters are rescheduling to the other | 18:29 |
| ajo | kevinbenton, sure | 18:29 |
| kevinbenton | ajo: if not, they won’t be | 18:29 |
| openstackgerrit | Assaf Muller proposed a change to openstack/python-neutronclient: Add L3 HA / VRRP support to CLI https://review.openstack.org/108378 | 18:29 |
| ajo | ah | 18:29 |
| ajo | ok | 18:29 |
| ajo | kevinbenton, you're right | 18:29 |
| ajo | but | 18:29 |
| ajo | kevinbenton, if they're deploying the l3_agents with the same host="....." | 18:29 |
| ajo | in the l3_agent.ini | 18:29 |
| ajo | to do active/passive | 18:29 |
| ajo | the qrouters will get scheduled to both hosts | 18:29 |
| kevinbenton | ajo: they’re not. all of the agents show up in neutron agent-list | 18:29 |
| ajo | aha | 18:29 |
| ajo | kevinbenton, so they're using A/A ? | 18:30 |
| *** AbsinthMind has joined #openstack-neutron | 18:30 | |
| ajo | with different host=... | 18:30 |
| ajo | then your patch is ok | 18:30 |
| kevinbenton | ajo: maybe they meant to, but if they did it’s not working | 18:30 |
| ajo | kevinbenton, may be they think they're doing and they don't... | 18:30 |
| ajo | think->believe | 18:30 |
| ajo | I have to AFK for a while | 18:31 |
| kevinbenton | ajo: that’s possible. that would make all of the comments make a lot more sense | 18:31 |
| kevinbenton | ajo: i’ll clarify on the bug report in case that’s what’s happening | 18:31 |
| ajo | kevinbenton, could be, ask them about that, and post the agent listing to clarify... :) | 18:31 |
| ajo | good luck :) | 18:31 |
| *** cnesa7 has quit IRC | 18:31 | |
| *** mikedill_ has joined #openstack-neutron | 18:32 | |
| *** mikedillion has quit IRC | 18:33 | |
| ajo | kevinbenton, do you know how to recheck nec-ci? | 18:34 |
| ajo | "recheck nec-ci" ? | 18:34 |
| *** mikedill_ has quit IRC | 18:34 | |
| kevinbenton | ajo: i don’t remember. it used to say in the nec comment body though | 18:35 |
| kevinbenton | ajo: if not you might try pinging amotoki when he’s online because i think he runs it | 18:36 |
| *** mikedillion has joined #openstack-neutron | 18:36 | |
| *** seizadi has quit IRC | 18:36 | |
| openstackgerrit | Vijay Kumar Venkatachalam proposed a change to openstack/neutron: NetScaler driver for LBaaS V2 extension https://review.openstack.org/114173 | 18:36 |
| *** sbfox has joined #openstack-neutron | 18:36 | |
| *** aepifanov has quit IRC | 18:37 | |
| ajo | thanks kevinbenton :) | 18:37 |
| *** banix has joined #openstack-neutron | 18:37 | |
| *** rotbeard has quit IRC | 18:38 | |
| *** a_le has joined #openstack-neutron | 18:39 | |
| enikanorov_ | kevinbenton: so i understand what you're trying to do - deploy l3/dhcp on each node | 18:39 |
| enikanorov_ | however i think it's just not that easy | 18:39 |
| enikanorov_ | other parts of the system need to be cosidered also | 18:40 |
| openstackgerrit | Assaf Muller proposed a change to openstack/python-neutronclient: Fix CLI support for DVR, take 2 https://review.openstack.org/114263 | 18:40 |
| openstackgerrit | Assaf Muller proposed a change to openstack/python-neutronclient: Add L3 HA / VRRP support to CLI https://review.openstack.org/108378 | 18:40 |
| kevinbenton | enikanorov_: this is how neutron is intended to work. what do you mean? | 18:42 |
| enikanorov_ | kevinbenton: i mean from fuel perspective | 18:42 |
| enikanorov_ | kevinbenton: i'm not an expert in how it is implemented, i know that such services like agents are monitored | 18:43 |
| enikanorov_ | so i'm just not sure that simple cloning will instantly solve the problem | 18:43 |
| enikanorov_ | otherwise i'm totally for this change | 18:43 |
| *** Longgeek has quit IRC | 18:44 | |
| *** AbsinthMind has quit IRC | 18:44 | |
| *** aepifanov has joined #openstack-neutron | 18:44 | |
| *** radez is now known as radez_g0n3 | 18:45 | |
| *** emagana has quit IRC | 18:45 | |
| *** amuller has quit IRC | 18:45 | |
| kevinbenton | enikanorov_: right, they are monitored to make sure there is only one active agent right now | 18:46 |
| *** emagana has joined #openstack-neutron | 18:46 | |
| kevinbenton | enikanorov_: but they aren’t configured to all simulate the same host or anything | 18:46 |
| kevinbenton | enikanorov_: so it makes no sense to block multiple from running | 18:46 |
| enikanorov_ | kevinbenton: so speaking about your patch, i'm not sure that monitoring will work correctly if you simply clone agents | 18:46 |
| kevinbenton | enikanorov_: the clone change i pushed just ensures that they are running on all nodes | 18:46 |
| *** morganfainberg_Z is now known as morganfainberg | 18:47 | |
| *** AbsinthMind has joined #openstack-neutron | 18:48 | |
| *** emagana_ has joined #openstack-neutron | 18:48 | |
| kevinbenton | enikanorov_: i tested this manually by changing the mode | 18:48 |
| *** emagana has quit IRC | 18:49 | |
| kevinbenton | enikanorov_: i just added the commands to the bug report that i used to test | 18:49 |
| armax | rkukura: I managed to reproduce the transient issue locally | 18:50 |
| armax | rkukura: looking... | 18:50 |
| *** spandhe has quit IRC | 18:51 | |
| *** radez_g0n3 is now known as radez | 18:53 | |
| rkukura | armax: Thanks! | 18:53 |
| armax | rkukura: that is baffling indeed | 18:53 |
| armax | rkukura: bear with me | 18:53 |
| armax | rkukura: collecting data points to root cause this | 18:53 |
| enikanorov_ | kevinbenton: thanks. I'll speak with Sergey on the bug tomorrow. However the patch will hardly land for 5.1 | 18:54 |
| *** roeyc has joined #openstack-neutron | 18:54 | |
| kevinbenton | enikanorov_: ugh, but this makes neutron basically useless :-( | 18:55 |
| kevinbenton | enikanorov_: one l3 agent for an entire deployment grinds all of the traffic to a halt | 18:55 |
| kevinbenton | enikanorov_: if you have anything more than a few compute nodes | 18:56 |
| enikanorov_ | kevinbenton: hate to say that, but you can make your conclusions about the scale of deployments fuel supprots | 18:56 |
| enikanorov_ | kevinbenton: we're working on that... | 18:56 |
| *** rudrarug_ has joined #openstack-neutron | 18:56 | |
| kevinbenton | enikanorov_: does fuel not support large neutron deployments? | 18:56 |
| kevinbenton | enikanorov_: or what do you mean? | 18:56 |
| enikanorov_ | kevinbenton: that's what i mean, right | 18:58 |
| rkukura | armax: Fresh eyes may be what’s needed. I’m ignoring the issue for the moment and trying to finish up that patch, then have another one to get ready by midnight somewhere. | 18:58 |
| armax | rkukura: ok | 18:59 |
| *** ijw_ has joined #openstack-neutron | 18:59 | |
| *** rudrarugge has quit IRC | 18:59 | |
| sbfox | hey kevinbenton, im having the same scalability problems with neutron. we've dropped overlays and gone to traditional vlan with external routers and load balancers | 19:00 |
| kevinbenton | enikanorov_: oh okay. well this single change would really buy us a lot in that direction | 19:00 |
| kevinbenton | sbfox: you ran into the conntrack issue, right? | 19:01 |
| sbfox | yeah, killed us in minutes | 19:01 |
| *** tsg has quit IRC | 19:01 | |
| sbfox | no realistic way around it in neutron | 19:01 |
| sbfox | a single l3 agent choked at ~90k conntracks | 19:02 |
| *** lori is now known as lori|away | 19:02 | |
| sbfox | ack'd kernel limit | 19:02 |
| kevinbenton | sbfox: interesting. how many compute nodes did you use to generate that? | 19:03 |
| sbfox | 25 big machines | 19:03 |
| *** ijw_ has quit IRC | 19:03 | |
| sbfox | ~200 vms, 80 app servers | 19:04 |
| kevinbenton | sbfox: how many virtual routers? | 19:04 |
| sbfox | 1, the vms were in a single subnet | 19:05 |
| *** mikedillion has quit IRC | 19:05 | |
| kevinbenton | sbfox: ok. we are about to do some scale testing too with the L3 agent. i’ll see if we can run into the same issue | 19:05 |
| kevinbenton | sbfox: what OS was running the L3 agent? | 19:06 |
| *** julim has quit IRC | 19:06 | |
| sbfox | we toyed with the idea of splitting the subnets, multiple routers etc but that introduces lots of EW traffic to the databases/redis/etc etc | 19:06 |
| kevinbenton | sbfox: right, that might have just made it worse since EW traffic crosses the same router... | 19:06 |
| sbfox | ubuntu 14.04 and kernels 3.13 - 3.16 | 19:06 |
| kevinbenton | sbfox: ok. thanks. can you ping me start of next week if there isn’t any more activity on the mail thread you started? | 19:07 |
| *** mikedillion has joined #openstack-neutron | 19:08 | |
| sbfox | moral of our story, drop to vlans and hw routers | 19:08 |
| klindgren | sbfox, out of curiosity - did you increase ip_conntrack_max? | 19:08 |
| sbfox | Will do kevinbenton | 19:08 |
| *** harlowja_ has quit IRC | 19:08 | |
| sbfox | Yes klindgren, we played with that figure al lot. Matching the max and hash limits 1:1 etc | 19:09 |
| sbfox | we aggresively tuned the network sysctl stuff for mice style traffic | 19:09 |
| *** AbsinthMind has quit IRC | 19:09 | |
| *** tongli has quit IRC | 19:10 | |
| klindgren | sbfox, - kk, we use vlans and real gateways in our implementation, but was just wondering | 19:10 |
| sbfox | im lucky we has an asr9k we are now using :) | 19:11 |
| *** snowgust has quit IRC | 19:12 | |
| sbfox | not to sound completely whiny, without the nat/snat/dnat/conntrack problems neutron streamed large file traffic pretty well | 19:13 |
| kevinbenton | armax: did you see my messages about the root cause of my sql woes? | 19:13 |
| *** spandhe has joined #openstack-neutron | 19:14 | |
| *** rms_13 has joined #openstack-neutron | 19:14 | |
| armax | kevinbenton: which ones? | 19:15 |
| *** harlowja has joined #openstack-neutron | 19:15 | |
| kevinbenton | armax: about me saying that i was just stupid and messed up the sql filter statement | 19:15 |
| kevinbenton | armax: in all of my testing during the patch development, i never had an agent down that i didn’t intentionally bring down for the test | 19:16 |
| armax | kevinbenton: :) well we should all take the blame | 19:16 |
| kevinbenton | armax: so i didn’t realize that the query was just essentially asking if any agent was down | 19:17 |
| *** baoli has joined #openstack-neutron | 19:17 | |
| *** nlahouti has quit IRC | 19:17 | |
| *** emagana_ has quit IRC | 19:18 | |
| *** emagana has joined #openstack-neutron | 19:19 | |
| *** harlowja has quit IRC | 19:19 | |
| *** smcquaid has joined #openstack-neutron | 19:19 | |
| *** julim has joined #openstack-neutron | 19:20 | |
| kevinbenton | armax: had to enable logging on the SQL server to find that one :-) | 19:20 |
| *** nlahouti has joined #openstack-neutron | 19:20 | |
| *** radez is now known as radez_g0n3 | 19:20 | |
| *** geekinutah has quit IRC | 19:21 | |
| *** baoli has quit IRC | 19:22 | |
| armax | kevinbenton: you did well | 19:22 |
| *** JayF has joined #openstack-neutron | 19:23 | |
| *** law has joined #openstack-neutron | 19:23 | |
| *** emagana has quit IRC | 19:23 | |
| law | hello all, n00b-type neutron question: why does one have to 'neutron net-create' a network just to have to 'neutron subnet-create' *another* network inside of it to be able to create routable VMs in there? | 19:23 |
| *** nlahouti has quit IRC | 19:24 | |
| *** morganfainberg has quit IRC | 19:25 | |
| *** padkrish has quit IRC | 19:26 | |
| dougwig | so this is what #openstack-neutron feels like without gerrit spam. | 19:27 |
| *** morganfainberg has joined #openstack-neutron | 19:27 | |
| *** alexpilotti has joined #openstack-neutron | 19:27 | |
| *** harlowja has joined #openstack-neutron | 19:28 | |
| *** dfarrell07 has quit IRC | 19:29 | |
| openstackgerrit | Thomas Bechtold proposed a change to openstack/neutron: Add specific docs build option to tox https://review.openstack.org/116077 | 19:31 |
| *** s3wong has joined #openstack-neutron | 19:33 | |
| *** harlowja has quit IRC | 19:34 | |
| *** harlowja has joined #openstack-neutron | 19:40 | |
| *** popw has quit IRC | 19:42 | |
| *** popw1 has joined #openstack-neutron | 19:42 | |
| *** Sukhdev has quit IRC | 19:42 | |
| *** seizadi has joined #openstack-neutron | 19:43 | |
| *** emagana has joined #openstack-neutron | 19:43 | |
| *** claudiub_ has quit IRC | 19:44 | |
| *** padkrish has joined #openstack-neutron | 19:45 | |
| *** rm_work is now known as rm_work|away | 19:46 | |
| *** thomasem has quit IRC | 19:46 | |
| *** jprovazn has quit IRC | 19:49 | |
| kevinbenton | law: neutron net-create creates the broadcast domain | 19:49 |
| kevinbenton | law: neutron subnet-create creates the IP allocation inside of it | 19:49 |
| kevinbenton | law: a network could have many subnets | 19:50 |
| kevinbenton | law: the subnet is required so the VM can be allocated an IP | 19:50 |
| *** seizadi has quit IRC | 19:50 | |
| *** SridharG has quit IRC | 19:53 | |
| law | ahh, most excellent, thank you! | 19:59 |
| law | so net-create is L2, subnet-create is L3? | 19:59 |
| *** ijw_ has joined #openstack-neutron | 20:00 | |
| *** ijw__ has joined #openstack-neutron | 20:01 | |
| *** AbsinthMind has joined #openstack-neutron | 20:02 | |
| *** jobewan has joined #openstack-neutron | 20:03 | |
| *** mrmoje has quit IRC | 20:03 | |
| *** AbsinthMind has quit IRC | 20:03 | |
| *** ijw_ has quit IRC | 20:04 | |
| *** ijw has quit IRC | 20:05 | |
| *** matrohon has joined #openstack-neutron | 20:06 | |
| *** wanghong has joined #openstack-neutron | 20:07 | |
| jamespd | ^^ yes | 20:08 |
| openstackgerrit | Kanzhe Jiang proposed a change to openstack/neutron: ServiceInstanceInfo DB table https://review.openstack.org/116090 | 20:08 |
| *** pcm__ has quit IRC | 20:08 | |
| *** ijw__ has quit IRC | 20:09 | |
| *** jorgem has joined #openstack-neutron | 20:11 | |
| openstackgerrit | Kanzhe Jiang proposed a change to openstack/neutron: ServiceInstanceInfo DB table https://review.openstack.org/116090 | 20:12 |
| *** flwang1 has joined #openstack-neutron | 20:13 | |
| *** otherwiseguy has quit IRC | 20:16 | |
| openstackgerrit | Weidong Shao proposed a change to openstack/neutron: Empty files should not contain copyright or license https://review.openstack.org/116094 | 20:16 |
| *** baoli has joined #openstack-neutron | 20:17 | |
| *** flwang1 has quit IRC | 20:18 | |
| *** matrohon has quit IRC | 20:18 | |
| openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy: Resource Mapping Driver (2) https://review.openstack.org/113775 | 20:20 |
| *** nlahouti has joined #openstack-neutron | 20:21 | |
| *** baoli has quit IRC | 20:22 | |
| *** emagana has quit IRC | 20:23 | |
| *** emagana has joined #openstack-neutron | 20:23 | |
| *** radez_g0n3 is now known as radez | 20:26 | |
| *** nlahouti has quit IRC | 20:26 | |
| *** emagana_ has joined #openstack-neutron | 20:27 | |
| *** emagana has quit IRC | 20:27 | |
| *** dfarrell07 has joined #openstack-neutron | 20:28 | |
| *** jordandh has quit IRC | 20:28 | |
| *** AbsinthMind has joined #openstack-neutron | 20:29 | |
| *** SridharG has joined #openstack-neutron | 20:29 | |
| *** marun has quit IRC | 20:29 | |
| *** AbsinthMind has quit IRC | 20:30 | |
| *** morganfainberg is now known as morganfainberg_Z | 20:31 | |
| *** nlahouti has joined #openstack-neutron | 20:33 | |
| *** law has left #openstack-neutron | 20:34 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Join tables in query for down L3 agents https://review.openstack.org/115821 | 20:35 |
| *** julim has quit IRC | 20:36 | |
| *** rm_work|away is now known as rm_work | 20:38 | |
| *** ijw has joined #openstack-neutron | 20:40 | |
| *** jordandh has joined #openstack-neutron | 20:40 | |
| *** ijw_ has joined #openstack-neutron | 20:41 | |
| *** chrisf has quit IRC | 20:42 | |
| *** ijw__ has joined #openstack-neutron | 20:43 | |
| *** rudrarug_ has quit IRC | 20:43 | |
| openstackgerrit | nlahouti proposed a change to openstack/neutron: Cisco DFA ML2 Mechanism Driver https://review.openstack.org/103281 | 20:43 |
| alexpilotti | rkukura: hi | 20:44 |
| *** radez is now known as radez_g0n3 | 20:44 | |
| rkukura | alexpilotti: hi | 20:44 |
| *** ijw has quit IRC | 20:44 | |
| *** rudrarugge has joined #openstack-neutron | 20:44 | |
| alexpilotti | rkukura: I have a small bug fix that got already 2 x +2, but is missing a +a: https://review.openstack.org/#/c/115812/ | 20:45 |
| *** wanghong has quit IRC | 20:45 | |
| *** thomasem has joined #openstack-neutron | 20:45 | |
| alexpilotti | rkukura: can I ask you to help with the +a? | 20:45 |
| *** ijw_ has quit IRC | 20:46 | |
| rkukura | alexpilotti: done | 20:46 |
| *** thomasem has joined #openstack-neutron | 20:46 | |
| alexpilotti | rkukura: tx! | 20:46 |
| alexpilotti | rkukura: btw we’re investigating some issues in the Hyper-V security groups that cause some tempest failures | 20:47 |
| *** SridharG has quit IRC | 20:47 | |
| *** chrisf has joined #openstack-neutron | 20:47 | |
| *** morganfainberg_Z is now known as morganfainberg | 20:47 | |
| alexpilotti | rkukura: we’ll report a bug and send the patch as soon as we got it clear | 20:48 |
| alexpilotti | rkukura: don’t think it’s related to the ML2 plugin, should be just an agent issue | 20:48 |
| *** ijw__ has quit IRC | 20:48 | |
| openstackgerrit | enikanorov proposed a change to openstack/neutron: Refactor l3 agent scheduling logic to avoid unwanted failures https://review.openstack.org/115383 | 20:52 |
| *** chrisf has quit IRC | 20:53 | |
| *** SumitNaiksatam has quit IRC | 20:54 | |
| *** chrisf has joined #openstack-neutron | 20:54 | |
| *** chrisf has quit IRC | 20:54 | |
| *** mandeep has quit IRC | 20:55 | |
| *** roeyc has left #openstack-neutron | 20:55 | |
| openstackgerrit | enikanorov proposed a change to openstack/neutron: Refactor l3 agent scheduling logic to avoid unwanted failures https://review.openstack.org/115383 | 20:56 |
| *** padkrish has quit IRC | 20:57 | |
| *** chrisf has joined #openstack-neutron | 20:58 | |
| *** mandeep has joined #openstack-neutron | 21:00 | |
| *** mandeep has quit IRC | 21:00 | |
| *** ijw has joined #openstack-neutron | 21:01 | |
| openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: Driver API changes for hierarchical port binding https://review.openstack.org/115110 | 21:03 |
| *** TrevorV_ has joined #openstack-neutron | 21:03 | |
| *** ijw has quit IRC | 21:05 | |
| *** chrisf has left #openstack-neutron | 21:05 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Do not log WARN messages about lack of L3 agents for DVR routers https://review.openstack.org/116100 | 21:06 |
| *** dfarrell07 has quit IRC | 21:06 | |
| *** manishg has joined #openstack-neutron | 21:08 | |
| *** s3wong has left #openstack-neutron | 21:10 | |
| *** ijw has joined #openstack-neutron | 21:11 | |
| *** ijw has quit IRC | 21:11 | |
| *** ijw has joined #openstack-neutron | 21:11 | |
| *** sdmonov has quit IRC | 21:11 | |
| openstackgerrit | nlahouti proposed a change to openstack/neutron: Cisco DFA ML2 Mechanism Driver - Part 3 https://review.openstack.org/111761 | 21:15 |
| *** emagana_ has quit IRC | 21:16 | |
| *** emagana has joined #openstack-neutron | 21:16 | |
| *** padkrish has joined #openstack-neutron | 21:17 | |
| *** baoli has joined #openstack-neutron | 21:18 | |
| *** beagles has quit IRC | 21:18 | |
| *** gugl2 has joined #openstack-neutron | 21:19 | |
| *** mestery has joined #openstack-neutron | 21:19 | |
| *** emagana has quit IRC | 21:21 | |
| *** jordandh has quit IRC | 21:22 | |
| *** baoli has quit IRC | 21:23 | |
| *** beagles has joined #openstack-neutron | 21:24 | |
| *** nlahouti has quit IRC | 21:24 | |
| *** beagles is now known as Guest5993 | 21:24 | |
| *** nlahouti has joined #openstack-neutron | 21:24 | |
| *** mestery has quit IRC | 21:25 | |
| *** banix_ has joined #openstack-neutron | 21:26 | |
| openstackgerrit | A change was merged to openstack/neutron: Ensure ip6tables are used only if ipv6 is enabled in kernel https://review.openstack.org/112253 | 21:28 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Do not log WARN messages about lack of L3 agents for DVR routers https://review.openstack.org/116100 | 21:28 |
| *** SumitNaiksatam has joined #openstack-neutron | 21:28 | |
| *** banix has quit IRC | 21:30 | |
| *** banix_ is now known as banix | 21:30 | |
| *** tsg has joined #openstack-neutron | 21:32 | |
| openstackgerrit | A change was merged to openstack/neutron: A10 Networks LBaaS v1 Driver https://review.openstack.org/113702 | 21:32 |
| openstackgerrit | AARON ZHANG proposed a change to openstack/neutron: Implement full sync for cisco n1kv plugin https://review.openstack.org/113037 | 21:32 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessesManager in the l3_agent https://review.openstack.org/114931 | 21:33 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessesManager to watch over external processes https://review.openstack.org/112798 | 21:33 |
| *** emagana has joined #openstack-neutron | 21:35 | |
| *** otherwiseguy has joined #openstack-neutron | 21:38 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Add test to compare security_group_info_for_devices with old rpc https://review.openstack.org/115575 | 21:38 |
| *** yfauser has joined #openstack-neutron | 21:40 | |
| *** yfauser has left #openstack-neutron | 21:40 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Add test to compare security_group_info_for_devices with old rpc https://review.openstack.org/115575 | 21:40 |
| *** krtaylor has quit IRC | 21:41 | |
| *** armax has quit IRC | 21:47 | |
| *** sgordon has quit IRC | 21:47 | |
| *** AlexF has joined #openstack-neutron | 21:48 | |
| *** seizadi has joined #openstack-neutron | 21:48 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessesManager in the dhcp_agent and dnsmasq driver https://review.openstack.org/115935 | 21:50 |
| *** JayF has left #openstack-neutron | 21:51 | |
| *** emagana has quit IRC | 21:51 | |
| *** emagana has joined #openstack-neutron | 21:52 | |
| annegentle | hey emagana or salv-orlando, can one of you help triage https://bugs.launchpad.net/openstack-api-site/+bug/1359987? Basically need to put the info from http://docs.openstack.org/api/openstack-network/2.0/content/floatingip_create.html into WADL so that the params can be documented. | 21:53 |
| *** TrevorV_ has quit IRC | 21:53 | |
| annegentle | optional vs required as a comment in the doc bug would be enough for us to fix | 21:53 |
| salv-orlando | annegentle: ah so it’s easy ;) | 21:54 |
| *** a_le has quit IRC | 21:54 | |
| *** rpothier has quit IRC | 21:55 | |
| *** a_le has joined #openstack-neutron | 21:55 | |
| *** geekinutah has joined #openstack-neutron | 21:55 | |
| salv-orlando | I am about to go to bed, but the sun is still high on emagana’s head. If he doesn’t provide these infiormation, I’ll take care of that in the morning. | 21:55 |
| ajo | good night salv-orlando ;) | 21:55 |
| *** aepifanov has quit IRC | 21:55 | |
| ajo | it's a bit late here too, I should be going to sleep instead of fixing tests :) | 21:56 |
| salv-orlando | ajo: have a good one. | 21:56 |
| *** emagana has quit IRC | 21:56 | |
| *** Sukhdev has joined #openstack-neutron | 21:57 | |
| *** emagana has joined #openstack-neutron | 21:57 | |
| *** bradjones has quit IRC | 21:58 | |
| *** kbringard has quit IRC | 22:01 | |
| *** Guest5993 is now known as b3nt_pin | 22:01 | |
| *** sgordon has joined #openstack-neutron | 22:02 | |
| *** banix has quit IRC | 22:03 | |
| *** aepifanov has joined #openstack-neutron | 22:03 | |
| *** annegentle has quit IRC | 22:04 | |
| *** AlexF has quit IRC | 22:04 | |
| *** flaviof is now known as flaviof_zzz | 22:07 | |
| *** dfarrell07 has joined #openstack-neutron | 22:10 | |
| *** a_le has quit IRC | 22:13 | |
| *** networkstatic has quit IRC | 22:13 | |
| *** emagana has quit IRC | 22:15 | |
| *** emagana has joined #openstack-neutron | 22:15 | |
| *** jaypipes has quit IRC | 22:16 | |
| *** armax has joined #openstack-neutron | 22:16 | |
| *** marun has joined #openstack-neutron | 22:17 | |
| *** gduan has quit IRC | 22:17 | |
| rkukura | armax: Any ideas on the stale data issue we were discussing? | 22:18 |
| armax | rkukura: I had to run a few errands, I’ll be looking at it shortly | 22:19 |
| *** baoli has joined #openstack-neutron | 22:19 | |
| rkukura | armax: Its getting even weirder | 22:19 |
| armax | rkukura: that said, you do introduce a change in the plugin | 22:20 |
| *** emagana has quit IRC | 22:20 | |
| armax | in get_bound_port_context | 22:20 |
| *** dfarrell07 has quit IRC | 22:20 | |
| rkukura | armax: In my hierarachical port binding DB branch, I removed all the extra logging and printing, committed everything, and rebased on another patch that I’ll (dynamic segments), and now I can’t get the failure to occur! | 22:21 |
| armax | rkukura: granted it should fail persistently | 22:21 |
| armax | rkukura: that’s bizzarre indeed | 22:21 |
| rkukura | wait - just failed on the 84th run! | 22:21 |
| armax | rkukura: :) | 22:21 |
| armax | rkukura 84-th the charm | 22:22 |
| *** jgrimm has quit IRC | 22:22 | |
| *** emagana has joined #openstack-neutron | 22:22 | |
| *** baoli has quit IRC | 22:23 | |
| rkukura | armax: So frequency went from ~30% to ~1% | 22:23 |
| armax | with the patch you posted upstream | 22:23 |
| *** thomasem has quit IRC | 22:24 | |
| rkukura | This is with the real branch | 22:24 |
| armax | rkukura: in my case it’s 1 in 6 | 22:24 |
| rkukura | I’ll post that in a minute | 22:24 |
| *** absubram_ has quit IRC | 22:25 | |
| *** bradjones has joined #openstack-neutron | 22:25 | |
| *** bradjones has quit IRC | 22:25 | |
| *** bradjones has joined #openstack-neutron | 22:25 | |
| *** lordd_ has joined #openstack-neutron | 22:28 | |
| *** aepifanov has quit IRC | 22:28 | |
| *** mageshgv has quit IRC | 22:29 | |
| openstackgerrit | Bogun Dmitriy proposed a change to openstack/neutron: Remove potential extra work in IpRouteCommand https://review.openstack.org/115725 | 22:29 |
| armax | rkukura: I think I got a theory | 22:31 |
| armax | rkukura: let me see if I can confirm it | 22:31 |
| rkukura | armax: OK | 22:31 |
| *** AlexF has joined #openstack-neutron | 22:32 | |
| rkukura | armax: My most latest untested guess is that the enable_eagerloads(False) in the queries might be responsible. | 22:33 |
| armax | rkukura: yup | 22:33 |
| armax | rkukura: I think there’s something quirky about sqlalchemy and backref | 22:33 |
| rkukura | armax: could be. I think we could eliminate them, and may need to as part of the schema cleanup we were discussing | 22:34 |
| *** annegentle has joined #openstack-neutron | 22:34 | |
| armax | rkukura: that shouldn’t be necessary though | 22:34 |
| armax | rkukura: ORM relationships either work or don’t | 22:35 |
| openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy: Resource Mapping Driver (2) https://review.openstack.org/113775 | 22:35 |
| *** emagana has quit IRC | 22:35 | |
| *** emagana has joined #openstack-neutron | 22:36 | |
| openstackgerrit | Sridar Kandaswamy proposed a change to openstack/neutron: Adds FWaaS service plugin for Cisco CSR1kv https://review.openstack.org/115308 | 22:36 |
| openstackgerrit | Sridar Kandaswamy proposed a change to openstack/neutron: Adds router service plugin for CSR1kv https://review.openstack.org/101002 | 22:36 |
| *** mrmoje has joined #openstack-neutron | 22:37 | |
| rkukura | armax: In the patch I’m about to post, you’ll see I didn’t use any sqlalchemy relationship between the ml2_port_bindings and ml2_port_binding_levels tables. | 22:38 |
| armax | rkukura: have a look at here: | 22:38 |
| armax | rkukura: http://paste.openstack.org/show/98399/ | 22:38 |
| *** sungju has joined #openstack-neutron | 22:38 | |
| armax | rkukura: wanna see if that helps? | 22:38 |
| armax | rkukura: with this tweak I can’t seem to reproduce the issue | 22:39 |
| armax | rkukura: as it keeps passing | 22:39 |
| armax | rkukura: the test I mean | 22:39 |
| *** annegentle has quit IRC | 22:39 | |
| rkukura | I’ll try | 22:39 |
| armax | rkukura: looking forward to learning the outcome of your experiment! | 22:40 |
| *** emagana has quit IRC | 22:40 | |
| rkukura | armax: I’m going to post my patch now, then add this change, and run 100 tests while having dinner | 22:40 |
| armax | rkukura: sounds like a plan to me! | 22:40 |
| armax | rkukura: ttyl | 22:41 |
| openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: DB changes for hierarchical port binding https://review.openstack.org/116122 | 22:41 |
| rkukura | armax: Take a look at ^^^ if/when you get a chance | 22:41 |
| armax | rkukura: will do | 22:42 |
| rkukura | armax: thanks!!! | 22:42 |
| *** emagana has joined #openstack-neutron | 22:42 | |
| *** stanzgy has joined #openstack-neutron | 22:43 | |
| armax | rkukura: what I meant earlier is that the test keeps passing without your induced change | 22:44 |
| openstackgerrit | Michael Smith proposed a change to openstack/neutron: Implement migration of legacy routers to distributed https://review.openstack.org/105855 | 22:44 |
| armax | rkukura: with the induced change, it keeps on failing | 22:44 |
| armax | rkukura: I’ll run a number of tests in sequence to see if I can see an intermittent failure | 22:45 |
| rkukura | armax: Are you saying that, with the test patch I posted early and your change, it fails reliably or it passes reliably? | 22:45 |
| armax | rkukura: with your patch, plus my lazy=joined tweak it fails reliably | 22:46 |
| rkukura | that doesn’t help! | 22:46 |
| rkukura | I’ll try it anyway | 22:46 |
| armax | rkukura: well, at it least we ruled out the intermittency | 22:46 |
| armax | rkukura: that’s gotta be a good thing! | 22:46 |
| rkukura | armax: It might help in that regard, but why is stale data getting read? | 22:47 |
| armax | rkukura: it means that there’s a logic error somewhere | 22:47 |
| *** _gk_ has joined #openstack-neutron | 22:47 | |
| rkukura | armax: logic error in sqlalchemy is possible, or in sqlite | 22:47 |
| rkukura | or my code | 22:47 |
| armax | rkukura: not some odd heisenbug | 22:47 |
| armax | rkukura: logic error in neutron’s code | 22:48 |
| armax | rkukura: not sure if it’s your fault or not | 22:48 |
| *** _gk_ has quit IRC | 22:48 | |
| armax | rkukura: let me play with this a little more anyway | 22:48 |
| rkukura | armax: I’m still pretty convinced that different reads of the same record from the same table are returning different results, with no write to the record in between | 22:49 |
| *** trinaths has quit IRC | 22:49 | |
| armax | rkukura: um, that would be weird | 22:49 |
| rkukura | armax: In an earlier patch, I had had a heck of a time trying to get a joined load working properly with lock_for_update applied to both retrieved records. | 22:50 |
| openstackgerrit | Karthik Natarajan proposed a change to openstack/neutron: Implementation of Brocade Vyatta vRouter L3 Plugin https://review.openstack.org/102336 | 22:51 |
| *** ijw has quit IRC | 22:51 | |
| *** shakamunyi has joined #openstack-neutron | 22:51 | |
| *** AlexF has quit IRC | 22:52 | |
| *** annegentle has joined #openstack-neutron | 22:52 | |
| *** ijw has joined #openstack-neutron | 22:52 | |
| rkukura | armax: Heading downstairs for a while. Thanks again for your help! | 22:52 |
| armax | rkukura: np | 22:52 |
| *** scott-millward has joined #openstack-neutron | 22:53 | |
| armax | rkukura: ping me back if you like | 22:53 |
| *** ijw_ has joined #openstack-neutron | 22:54 | |
| *** padkrish has quit IRC | 22:54 | |
| *** sbfox has quit IRC | 22:55 | |
| *** a_le has joined #openstack-neutron | 22:57 | |
| openstackgerrit | Magesh GV proposed a change to openstack/neutron: Group Policy: One Convergence NVSD Group Policy Driver https://review.openstack.org/113649 | 22:57 |
| *** ijw has quit IRC | 22:57 | |
| *** mageshgv has joined #openstack-neutron | 22:57 | |
| openstackgerrit | Swaminathan Vasudevan proposed a change to openstack/neutron: Problems with DVR-Router updates after ext-gw set https://review.openstack.org/114410 | 22:58 |
| openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Service Insertion: Service Interface API and DB https://review.openstack.org/113975 | 22:59 |
| *** sbfox has joined #openstack-neutron | 23:01 | |
| *** ijw_ has quit IRC | 23:01 | |
| *** ijw has joined #openstack-neutron | 23:01 | |
| openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Service Insertion: Service Interface API and DB https://review.openstack.org/113975 | 23:04 |
| *** mageshgv has quit IRC | 23:04 | |
| *** annegentle has quit IRC | 23:05 | |
| *** annegentle has joined #openstack-neutron | 23:05 | |
| *** mrmoje has quit IRC | 23:05 | |
| *** pballand has quit IRC | 23:11 | |
| armax | rkukura: the other theory I have is to pass the session along, like here: http://paste.openstack.org/show/98421/ | 23:11 |
| *** a_le has quit IRC | 23:11 | |
| *** otherwiseguy has quit IRC | 23:11 | |
| *** jorgem has quit IRC | 23:13 | |
| *** padkrish has joined #openstack-neutron | 23:13 | |
| *** otherwiseguy has joined #openstack-neutron | 23:15 | |
| *** a_le has joined #openstack-neutron | 23:16 | |
| *** Sukhdev has quit IRC | 23:17 | |
| openstackgerrit | Abhishek Raut proposed a change to openstack/neutron: ML2 mechanism driver for Cisco Nexus1000V https://review.openstack.org/114872 | 23:19 |
| *** baoli has joined #openstack-neutron | 23:20 | |
| *** annegentle has quit IRC | 23:22 | |
| *** mageshgv has joined #openstack-neutron | 23:22 | |
| *** Sukhdev has joined #openstack-neutron | 23:23 | |
| *** AlexF has joined #openstack-neutron | 23:23 | |
| *** sbfox has quit IRC | 23:24 | |
| *** baoli has quit IRC | 23:24 | |
| *** ijw_ has joined #openstack-neutron | 23:25 | |
| *** emagana has quit IRC | 23:28 | |
| *** ijw has quit IRC | 23:28 | |
| *** AlexF has quit IRC | 23:28 | |
| *** emagana has joined #openstack-neutron | 23:28 | |
| *** a_le has quit IRC | 23:30 | |
| *** emagana has quit IRC | 23:31 | |
| *** chuckC_ has quit IRC | 23:31 | |
| *** emagana has joined #openstack-neutron | 23:31 | |
| *** ijw_ has quit IRC | 23:35 | |
| *** alexpilotti has quit IRC | 23:35 | |
| *** ijw has joined #openstack-neutron | 23:35 | |
| openstackgerrit | nlahouti proposed a change to openstack/neutron: Cisco DFA ML2 Mechanism Driver - Part 3 https://review.openstack.org/111761 | 23:40 |
| *** ijw has quit IRC | 23:45 | |
| *** ijw has joined #openstack-neutron | 23:48 | |
| *** padkrish has quit IRC | 23:51 | |
| *** tsg has quit IRC | 23:51 | |
| *** annegentle has joined #openstack-neutron | 23:52 | |
| openstackgerrit | Kanzhe Jiang proposed a change to openstack/neutron: ServiceInstanceInfo DB table https://review.openstack.org/116090 | 23:55 |
| *** annegentle has quit IRC | 23:57 | |
| *** otherwiseguy has quit IRC | 23:59 | |
| *** tsg has joined #openstack-neutron | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!