| *** baoli has joined #openstack-neutron | 00:10 | |
| *** gildub has joined #openstack-neutron | 00:11 | |
| *** ZZelle_ has quit IRC | 00:13 | |
| *** baoli has quit IRC | 00:15 | |
| *** markvoelker has joined #openstack-neutron | 00:17 | |
| *** julim has joined #openstack-neutron | 00:17 | |
| *** yamamoto has joined #openstack-neutron | 00:19 | |
| *** sbfox has joined #openstack-neutron | 00:19 | |
| *** markvoelker has quit IRC | 00:22 | |
| *** pavel_bondar has quit IRC | 00:22 | |
| *** achanda has quit IRC | 00:22 | |
| *** yalie1 has quit IRC | 00:23 | |
| *** pavel_bondar has joined #openstack-neutron | 00:23 | |
| *** yamamoto has quit IRC | 00:24 | |
| *** sbfox has quit IRC | 00:24 | |
| *** liusheng has quit IRC | 00:26 | |
| *** liusheng has joined #openstack-neutron | 00:26 | |
| *** yamamoto has joined #openstack-neutron | 00:26 | |
| *** dane_leblanc_ has joined #openstack-neutron | 00:26 | |
| *** dane_leblanc has quit IRC | 00:30 | |
| *** shwetaap has quit IRC | 00:32 | |
| *** baoli has joined #openstack-neutron | 00:32 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 00:32 |
|---|---|---|
| *** emagana has quit IRC | 00:34 | |
| *** overlayer has quit IRC | 00:36 | |
| *** ajmiller__ is now known as ajmiller | 00:40 | |
| *** nati_ueno has joined #openstack-neutron | 00:41 | |
| *** amrith has joined #openstack-neutron | 00:48 | |
| *** julim has quit IRC | 00:53 | |
| *** achanda has joined #openstack-neutron | 00:53 | |
| *** ferest has joined #openstack-neutron | 00:57 | |
| *** amrith is now known as _amrith_ | 00:58 | |
| *** liusheng has quit IRC | 01:00 | |
| *** dims has joined #openstack-neutron | 01:01 | |
| *** dims has quit IRC | 01:01 | |
| *** shaohe_feng has joined #openstack-neutron | 01:01 | |
| *** topshare has joined #openstack-neutron | 01:01 | |
| *** ferest has quit IRC | 01:02 | |
| *** ferest has joined #openstack-neutron | 01:02 | |
| *** yamamoto has quit IRC | 01:04 | |
| *** aranjan has quit IRC | 01:05 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 01:05 |
| *** ferest has quit IRC | 01:07 | |
| *** Longgeek has joined #openstack-neutron | 01:08 | |
| *** nati_ueno has quit IRC | 01:08 | |
| *** topshare has quit IRC | 01:15 | |
| *** _amrith_ is now known as amrith | 01:15 | |
| *** dims__ has joined #openstack-neutron | 01:15 | |
| *** aranjan has joined #openstack-neutron | 01:17 | |
| *** tiswanso has joined #openstack-neutron | 01:18 | |
| *** markvoelker has joined #openstack-neutron | 01:18 | |
| *** yalie has joined #openstack-neutron | 01:18 | |
| *** markvoelker has quit IRC | 01:23 | |
| *** ianw has quit IRC | 01:25 | |
| *** ianw has joined #openstack-neutron | 01:26 | |
| *** yamamoto has joined #openstack-neutron | 01:28 | |
| *** haleyb has quit IRC | 01:30 | |
| *** stanzgy has joined #openstack-neutron | 01:40 | |
| *** stanzgy_ has joined #openstack-neutron | 01:40 | |
| *** oomichi_ has joined #openstack-neutron | 01:44 | |
| *** watanabe_isao has joined #openstack-neutron | 01:51 | |
| mestery_ | yamamoto: Thanks for the reviews of some of those networking-odl patches, much appreciated! :) | 01:52 |
| *** mestery_ is now known as mestery | 01:53 | |
| yamamoto | mestery: no problem! | 01:53 |
| mestery | yamamoto: I'm hoping to land that long batch of patches and then cut the first release of networking-odl tomorrow, but lets see how it goes. | 01:53 |
| *** shaohe_feng has quit IRC | 01:55 | |
| *** achanda has quit IRC | 01:57 | |
| *** baoli has quit IRC | 01:59 | |
| *** armax has joined #openstack-neutron | 02:00 | |
| *** shaohe_feng has joined #openstack-neutron | 02:03 | |
| *** baoli has joined #openstack-neutron | 02:04 | |
| *** armax has quit IRC | 02:04 | |
| *** popw has joined #openstack-neutron | 02:06 | |
| *** armax has joined #openstack-neutron | 02:06 | |
| *** armax has quit IRC | 02:07 | |
| *** erkules_ has joined #openstack-neutron | 02:16 | |
| *** popw has quit IRC | 02:16 | |
| *** erkules has quit IRC | 02:18 | |
| *** popw has joined #openstack-neutron | 02:18 | |
| *** markvoelker has joined #openstack-neutron | 02:19 | |
| *** shwetaap has joined #openstack-neutron | 02:21 | |
| *** markvoelker has quit IRC | 02:23 | |
| openstackgerrit | Joe Jiang proposed openstack/python-neutronclient: Set specify IP address when use 'router-gateway-set' subcommand. https://review.openstack.org/163332 | 02:23 |
| *** gongysh has joined #openstack-neutron | 02:29 | |
| *** stanzgy_ has quit IRC | 02:30 | |
| *** stanzgy has quit IRC | 02:30 | |
| *** aranjan has quit IRC | 02:31 | |
| *** stanzgy has joined #openstack-neutron | 02:31 | |
| *** jyuso1 has joined #openstack-neutron | 02:35 | |
| *** aranjan has joined #openstack-neutron | 02:36 | |
| *** aranjan has quit IRC | 02:48 | |
| *** Networkn3rd has joined #openstack-neutron | 02:48 | |
| yamamoto | mestery: when L specs process will be open? | 02:52 |
| *** amrith is now known as _amrith_ | 02:52 | |
| *** dboik has joined #openstack-neutron | 02:53 | |
| *** tflynn has joined #openstack-neutron | 02:53 | |
| *** yuanying has quit IRC | 02:56 | |
| *** achanda has joined #openstack-neutron | 02:58 | |
| openstackgerrit | gongysh proposed openstack/neutron: Add extra subnet route to ha router https://review.openstack.org/159772 | 02:59 |
| *** dims__ has quit IRC | 03:01 | |
| *** aranjan has joined #openstack-neutron | 03:01 | |
| *** achanda has quit IRC | 03:03 | |
| openstackgerrit | gongysh proposed openstack/neutron: Simplify keepalived.virtual_routes https://review.openstack.org/160430 | 03:03 |
| *** tflynn has quit IRC | 03:04 | |
| *** tflynn has joined #openstack-neutron | 03:04 | |
| *** aranjan has quit IRC | 03:09 | |
| *** aranjan has joined #openstack-neutron | 03:09 | |
| *** yuanying has joined #openstack-neutron | 03:09 | |
| openstackgerrit | gongysh proposed openstack/neutron: Simplify keepalived.virtual_routes https://review.openstack.org/160430 | 03:15 |
| *** tflynn has quit IRC | 03:23 | |
| *** coolsvap|afk is now known as coolsvap | 03:24 | |
| *** marrusl has quit IRC | 03:24 | |
| *** tflynn has joined #openstack-neutron | 03:25 | |
| *** Longgeek has quit IRC | 03:30 | |
| *** watanabe_isao has quit IRC | 03:34 | |
| *** anilvenkata has joined #openstack-neutron | 03:50 | |
| *** shaohe_feng has quit IRC | 03:52 | |
| *** watanabe_isao has joined #openstack-neutron | 04:01 | |
| *** leenheer has joined #openstack-neutron | 04:02 | |
| *** dave-mccowan has quit IRC | 04:02 | |
| *** tflynn has quit IRC | 04:03 | |
| *** dims__ has joined #openstack-neutron | 04:06 | |
| *** baoli has quit IRC | 04:07 | |
| *** achanda has joined #openstack-neutron | 04:08 | |
| *** baoli has joined #openstack-neutron | 04:10 | |
| *** baoli has quit IRC | 04:15 | |
| *** dboik has quit IRC | 04:21 | |
| *** dboik has joined #openstack-neutron | 04:21 | |
| *** sudipto has joined #openstack-neutron | 04:24 | |
| *** shaohe_feng has joined #openstack-neutron | 04:25 | |
| *** Longgeek has joined #openstack-neutron | 04:30 | |
| *** achanda has quit IRC | 04:34 | |
| *** vthapar has joined #openstack-neutron | 04:35 | |
| *** dims__ has quit IRC | 04:36 | |
| *** aranjan has quit IRC | 04:41 | |
| *** yatin has joined #openstack-neutron | 04:42 | |
| *** aranjan has joined #openstack-neutron | 04:45 | |
| *** chandankumar has joined #openstack-neutron | 04:46 | |
| *** yamahata has joined #openstack-neutron | 04:54 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Remove allow_overlap from subnetpools API https://review.openstack.org/166397 | 04:54 |
| *** trinaths has joined #openstack-neutron | 04:57 | |
| *** Networkn3rd has quit IRC | 04:59 | |
| *** Marga_ has quit IRC | 05:01 | |
| *** ajmiller has quit IRC | 05:03 | |
| *** pcaruana has quit IRC | 05:07 | |
| *** achanda has joined #openstack-neutron | 05:08 | |
| *** leenheer has quit IRC | 05:17 | |
| *** chandankumar has quit IRC | 05:17 | |
| *** Marga_ has joined #openstack-neutron | 05:21 | |
| *** achanda has quit IRC | 05:24 | |
| *** achanda has joined #openstack-neutron | 05:26 | |
| *** pal has joined #openstack-neutron | 05:27 | |
| *** chandankumar has joined #openstack-neutron | 05:33 | |
| trinaths | Hi all, very Good Morning | 05:37 |
| *** pradipta has joined #openstack-neutron | 05:40 | |
| *** yamahata has quit IRC | 05:41 | |
| *** tflynn has joined #openstack-neutron | 05:44 | |
| *** pal has quit IRC | 05:48 | |
| *** lpetrut has joined #openstack-neutron | 05:49 | |
| *** ishant has joined #openstack-neutron | 05:54 | |
| *** tflynn has quit IRC | 05:55 | |
| *** Bhavya has joined #openstack-neutron | 05:55 | |
| *** tflynn has joined #openstack-neutron | 05:56 | |
| *** yatin has quit IRC | 05:57 | |
| *** killer_prince has quit IRC | 05:59 | |
| *** Networkn3rd has joined #openstack-neutron | 06:00 | |
| *** SridharG has joined #openstack-neutron | 06:01 | |
| *** killer_prince has joined #openstack-neutron | 06:01 | |
| *** killer_prince is now known as lazy_prince | 06:01 | |
| *** neeti has joined #openstack-neutron | 06:04 | |
| *** Networkn3rd has quit IRC | 06:06 | |
| *** anshul has joined #openstack-neutron | 06:09 | |
| *** yfried|afk has quit IRC | 06:12 | |
| *** yfried|afk has joined #openstack-neutron | 06:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/165705 | 06:13 |
| *** trinaths has quit IRC | 06:13 | |
| *** trinaths has joined #openstack-neutron | 06:18 | |
| *** yfried|afk has quit IRC | 06:21 | |
| *** Marga_ has quit IRC | 06:21 | |
| *** dims__ has joined #openstack-neutron | 06:22 | |
| *** shwetaap has quit IRC | 06:24 | |
| *** nmagnezi has joined #openstack-neutron | 06:25 | |
| *** numan has joined #openstack-neutron | 06:26 | |
| *** yfried|afk has joined #openstack-neutron | 06:27 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/166443 | 06:27 |
| *** ajo_ has joined #openstack-neutron | 06:28 | |
| *** jamielennox is now known as jamielennox|away | 06:28 | |
| openstackgerrit | Akihiro Motoki proposed openstack/python-neutronclient: Update hacking to 0.10 https://review.openstack.org/166702 | 06:29 |
| *** gsagie has joined #openstack-neutron | 06:30 | |
| *** yatin has joined #openstack-neutron | 06:31 | |
| *** dannywilson has joined #openstack-neutron | 06:31 | |
| *** yfried|afk has quit IRC | 06:32 | |
| *** irenab has quit IRC | 06:36 | |
| *** gildub has quit IRC | 06:38 | |
| *** dannywilson has quit IRC | 06:40 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Follow up patch for Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/166707 | 06:41 |
| *** Bhavya has quit IRC | 06:41 | |
| *** aranjan has quit IRC | 06:42 | |
| *** Bhavya has joined #openstack-neutron | 06:45 | |
| *** stanzgy has quit IRC | 06:46 | |
| *** stanzgy has joined #openstack-neutron | 06:47 | |
| openstackgerrit | Aman Kumar proposed openstack/neutron: ovs agent resync may miss port remove event https://review.openstack.org/165023 | 06:49 |
| gsagie | If anyone can comment on https://bugs.launchpad.net/neutron/+bug/1435012 it would be appreciated, the problem as i see it right now is that the L3 code doesnt recognize the action on the router inside the process_router() method, knowing the action can help solve that case (we dont need to delete the ports in case of router removed) | 06:49 |
| openstack | Launchpad bug 1435012 in neutron "Exception when removing last network port from external router" [Undecided,New] - Assigned to Gal Sagie (gal-sagie) | 06:49 |
| gsagie | so for example if we are in process_router due to router remove action, we dont need to remove the SNAT port | 06:50 |
| *** yfried|afk has joined #openstack-neutron | 06:53 | |
| *** dims__ has quit IRC | 06:54 | |
| openstackgerrit | Merged openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/166470 | 06:54 |
| *** chandankumar has quit IRC | 06:57 | |
| *** Marga_ has joined #openstack-neutron | 06:57 | |
| *** pcaruana has joined #openstack-neutron | 06:58 | |
| *** moshele has joined #openstack-neutron | 06:59 | |
| openstackgerrit | Aman Kumar proposed openstack/neutron: Added config variable for External Network type in ML2 https://review.openstack.org/149546 | 06:59 |
| *** Bsony has joined #openstack-neutron | 06:59 | |
| *** anshul has quit IRC | 07:00 | |
| *** anshul has joined #openstack-neutron | 07:00 | |
| *** jamiec has quit IRC | 07:00 | |
| openstackgerrit | Akihiro Motoki proposed openstack/python-neutronclient: Remove author tag https://review.openstack.org/166714 | 07:01 |
| openstackgerrit | Akihiro Motoki proposed openstack/python-neutronclient: Fix E265 block comment should start with '# ' https://review.openstack.org/166715 | 07:01 |
| openstackgerrit | Akihiro Motoki proposed openstack/python-neutronclient: Fix one remaining E125 error and remove it from ignore list https://review.openstack.org/166716 | 07:01 |
| *** rohara has quit IRC | 07:01 | |
| *** Networkn3rd has joined #openstack-neutron | 07:02 | |
| *** garyk has joined #openstack-neutron | 07:03 | |
| *** jamiec has joined #openstack-neutron | 07:03 | |
| *** anshul has quit IRC | 07:04 | |
| *** rohara has joined #openstack-neutron | 07:04 | |
| *** veena has joined #openstack-neutron | 07:04 | |
| *** anshul has joined #openstack-neutron | 07:05 | |
| *** anshul has quit IRC | 07:05 | |
| *** Networkn3rd has quit IRC | 07:06 | |
| *** saikrishna has joined #openstack-neutron | 07:07 | |
| *** anshul has joined #openstack-neutron | 07:07 | |
| ajo_ | gerrit is down, isn't it? | 07:10 |
| *** ajo_ is now known as ajo | 07:10 | |
| *** bishtsanddev has joined #openstack-neutron | 07:10 | |
| bishtsanddev | Hi | 07:10 |
| bishtsanddev | Please guide me on neutron development | 07:10 |
| *** yog__ has joined #openstack-neutron | 07:11 | |
| bishtsanddev | I am interested in individual contribution to neutron SDN-NFV | 07:11 |
| *** yfried has joined #openstack-neutron | 07:11 | |
| *** vks has joined #openstack-neutron | 07:11 | |
| *** rotbeard has joined #openstack-neutron | 07:12 | |
| *** yfried|afk has quit IRC | 07:12 | |
| *** irenab has joined #openstack-neutron | 07:15 | |
| *** yfried has quit IRC | 07:15 | |
| vks | hi, how can one comment on blueprint ? | 07:16 |
| *** lpetrut has quit IRC | 07:17 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 07:17 |
| *** yfried has joined #openstack-neutron | 07:18 | |
| *** yfried is now known as yfried|afk | 07:18 | |
| *** bobmel_ has joined #openstack-neutron | 07:19 | |
| *** bobmel_ has quit IRC | 07:20 | |
| *** bobmel_ has joined #openstack-neutron | 07:20 | |
| *** kickinz1|afk is now known as kickinz1 | 07:21 | |
| vks | need help in setup system to clone and review blueprint | 07:21 |
| *** apuimedo has joined #openstack-neutron | 07:22 | |
| *** bobmel has quit IRC | 07:23 | |
| *** scheuran has joined #openstack-neutron | 07:24 | |
| *** Marga_ has quit IRC | 07:29 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Follow up patch for Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/166707 | 07:29 |
| *** achanda has quit IRC | 07:29 | |
| *** irenab has quit IRC | 07:29 | |
| *** numan has quit IRC | 07:30 | |
| *** irenab has joined #openstack-neutron | 07:34 | |
| *** yamahata has joined #openstack-neutron | 07:36 | |
| *** topshare has joined #openstack-neutron | 07:38 | |
| *** yfried has joined #openstack-neutron | 07:39 | |
| *** ajo has quit IRC | 07:42 | |
| *** chandankumar has joined #openstack-neutron | 07:42 | |
| *** numan has joined #openstack-neutron | 07:43 | |
| *** yfried|afk has quit IRC | 07:43 | |
| *** aranjan has joined #openstack-neutron | 07:43 | |
| *** anshul has quit IRC | 07:44 | |
| *** killer_prince has joined #openstack-neutron | 07:45 | |
| *** lazy_prince has quit IRC | 07:46 | |
| *** killer_prince is now known as lazy_prince | 07:46 | |
| *** anshul has joined #openstack-neutron | 07:47 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 07:47 |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Remove downgrade from existing migrations https://review.openstack.org/165740 | 07:47 |
| *** aranjan has quit IRC | 07:48 | |
| *** lazy_prince has quit IRC | 07:52 | |
| *** yamahata has quit IRC | 07:53 | |
| *** killer_prince has joined #openstack-neutron | 07:54 | |
| *** killer_prince is now known as lazy_prince | 07:54 | |
| *** topshare_ has joined #openstack-neutron | 07:56 | |
| *** topshare_ has quit IRC | 07:56 | |
| *** yfried is now known as yfried|afk | 07:57 | |
| *** topshare_ has joined #openstack-neutron | 07:57 | |
| *** chlong has quit IRC | 07:58 | |
| *** YorikSar has quit IRC | 07:58 | |
| *** saikrishna_ has joined #openstack-neutron | 07:58 | |
| *** topshare has quit IRC | 08:00 | |
| *** tflynn has quit IRC | 08:01 | |
| *** saikrishna has quit IRC | 08:01 | |
| *** aepifanov has joined #openstack-neutron | 08:02 | |
| *** yfried|afk is now known as yfried | 08:03 | |
| *** numan has quit IRC | 08:03 | |
| *** oomichi_ has quit IRC | 08:04 | |
| *** veena has quit IRC | 08:05 | |
| *** akamyshnikova_ has joined #openstack-neutron | 08:06 | |
| *** itzikb has joined #openstack-neutron | 08:09 | |
| *** criptoverso has joined #openstack-neutron | 08:13 | |
| *** saikrishna__ has joined #openstack-neutron | 08:14 | |
| *** numan has joined #openstack-neutron | 08:16 | |
| *** saikrishna_ has quit IRC | 08:16 | |
| *** ronis has joined #openstack-neutron | 08:19 | |
| *** gsagie has quit IRC | 08:19 | |
| *** yfried is now known as yfried|afk | 08:21 | |
| *** yfried|afk is now known as yfried | 08:22 | |
| *** doude has joined #openstack-neutron | 08:23 | |
| *** markvoelker has joined #openstack-neutron | 08:24 | |
| *** absubram has quit IRC | 08:24 | |
| *** ihrachyshka has joined #openstack-neutron | 08:25 | |
| openstackgerrit | Kobi Samoray proposed openstack/neutron: VMWare NSXv: Metadata default gateway param https://review.openstack.org/160711 | 08:25 |
| *** markvoelker has quit IRC | 08:28 | |
| *** lpetrut has joined #openstack-neutron | 08:28 | |
| *** yfried_ has joined #openstack-neutron | 08:28 | |
| *** numan has quit IRC | 08:29 | |
| *** britthou_ has joined #openstack-neutron | 08:31 | |
| *** yfried has quit IRC | 08:32 | |
| openstackgerrit | Merged openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/166441 | 08:33 |
| *** britthouser has quit IRC | 08:33 | |
| *** YorikSar has joined #openstack-neutron | 08:34 | |
| *** yfried_ has quit IRC | 08:37 | |
| *** yfried_ has joined #openstack-neutron | 08:37 | |
| *** gtt116 has joined #openstack-neutron | 08:38 | |
| *** dims__ has joined #openstack-neutron | 08:39 | |
| *** numan has joined #openstack-neutron | 08:44 | |
| *** yfried_ has quit IRC | 08:44 | |
| *** yfried__ has joined #openstack-neutron | 08:44 | |
| *** aranjan has joined #openstack-neutron | 08:44 | |
| *** jpena has joined #openstack-neutron | 08:46 | |
| *** julien_ZTE has joined #openstack-neutron | 08:46 | |
| *** nyechiel has joined #openstack-neutron | 08:46 | |
| *** ihrachyshka has quit IRC | 08:48 | |
| *** SridharG has quit IRC | 08:49 | |
| *** Networkn3rd has joined #openstack-neutron | 08:51 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Revert removed 'default' parameter from Agent model https://review.openstack.org/166739 | 08:52 |
| *** ajo has joined #openstack-neutron | 08:52 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Follow up patch for Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/166707 | 08:53 |
| *** watanabe_isao has quit IRC | 08:53 | |
| *** matrohon has joined #openstack-neutron | 08:53 | |
| *** vthapar_ has joined #openstack-neutron | 08:54 | |
| *** yuanying has quit IRC | 08:54 | |
| *** Networkn3rd has quit IRC | 08:56 | |
| *** blues-man has joined #openstack-neutron | 08:56 | |
| *** yfauser has joined #openstack-neutron | 08:57 | |
| *** vthapar has quit IRC | 08:57 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Revert removed 'default' parameter from Agent model https://review.openstack.org/166739 | 08:57 |
| *** yfauser has left #openstack-neutron | 08:58 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Enable Process Monitor by default. https://review.openstack.org/165117 | 08:59 |
| *** gildub has joined #openstack-neutron | 09:00 | |
| *** anshul has quit IRC | 09:00 | |
| *** jistr has joined #openstack-neutron | 09:02 | |
| *** ondergetekende_ has quit IRC | 09:03 | |
| *** bobmel has joined #openstack-neutron | 09:04 | |
| *** anshul has joined #openstack-neutron | 09:04 | |
| *** safchain_ has joined #openstack-neutron | 09:05 | |
| *** bobmel_ has quit IRC | 09:06 | |
| *** bjornar_ has joined #openstack-neutron | 09:06 | |
| *** ygbo has joined #openstack-neutron | 09:10 | |
| openstackgerrit | Merged openstack/neutron: Imported Translations from Transifex https://review.openstack.org/165705 | 09:11 |
| *** dims__ has quit IRC | 09:12 | |
| *** erkules_ is now known as erkules | 09:14 | |
| *** erkules has quit IRC | 09:14 | |
| *** erkules has joined #openstack-neutron | 09:14 | |
| *** SridharG has joined #openstack-neutron | 09:16 | |
| *** gongysh has quit IRC | 09:17 | |
| *** lpetrut has quit IRC | 09:17 | |
| *** yamamoto has quit IRC | 09:17 | |
| *** shaohe_feng has quit IRC | 09:18 | |
| *** aranjan has quit IRC | 09:19 | |
| *** lpetrut has joined #openstack-neutron | 09:20 | |
| *** bishtsanddev has quit IRC | 09:21 | |
| *** markvoelker has joined #openstack-neutron | 09:24 | |
| *** numan has quit IRC | 09:26 | |
| *** jlanoux has joined #openstack-neutron | 09:26 | |
| *** pradipta has quit IRC | 09:26 | |
| *** julien_ZTE has quit IRC | 09:27 | |
| *** markvoelker has quit IRC | 09:29 | |
| *** afazekas_ has joined #openstack-neutron | 09:29 | |
| *** gongysh has joined #openstack-neutron | 09:32 | |
| *** vthapar__ has joined #openstack-neutron | 09:34 | |
| *** kbyrne has joined #openstack-neutron | 09:35 | |
| *** evgenyf has joined #openstack-neutron | 09:36 | |
| *** vthapar_ has quit IRC | 09:36 | |
| *** numan has joined #openstack-neutron | 09:40 | |
| *** liusheng has joined #openstack-neutron | 09:41 | |
| *** dannywilson has joined #openstack-neutron | 09:41 | |
| *** exed_ has quit IRC | 09:42 | |
| *** trinaths has left #openstack-neutron | 09:43 | |
| *** ZZelle has quit IRC | 09:44 | |
| *** ZZelle has joined #openstack-neutron | 09:44 | |
| *** gsagie has joined #openstack-neutron | 09:44 | |
| *** dannywilson has quit IRC | 09:46 | |
| *** jyuso1 has quit IRC | 09:51 | |
| *** dims__ has joined #openstack-neutron | 09:51 | |
| *** yfried__ is now known as yfried|afk | 09:53 | |
| *** gtt116_ has joined #openstack-neutron | 09:53 | |
| *** yamamoto has joined #openstack-neutron | 09:53 | |
| *** exed_ has joined #openstack-neutron | 09:54 | |
| *** yfried|afk is now known as yfried__ | 09:54 | |
| *** tiswanso has quit IRC | 09:55 | |
| *** gtt116 has quit IRC | 09:56 | |
| *** akamyshnikova_ has quit IRC | 09:57 | |
| openstackgerrit | Merged openstack/neutron: Improve validate of remove_router_interface https://review.openstack.org/151973 | 10:08 |
| *** yalie has quit IRC | 10:09 | |
| *** enikanorov_ has quit IRC | 10:11 | |
| *** yalie has joined #openstack-neutron | 10:11 | |
| *** yfauser has joined #openstack-neutron | 10:11 | |
| *** yfauser has left #openstack-neutron | 10:11 | |
| *** enikanorov_ has joined #openstack-neutron | 10:12 | |
| *** ondergetekende has joined #openstack-neutron | 10:13 | |
| *** sbelous has quit IRC | 10:14 | |
| *** kkuznetsova has quit IRC | 10:14 | |
| *** sbelous has joined #openstack-neutron | 10:15 | |
| *** kkuznetsova has joined #openstack-neutron | 10:15 | |
| *** aranjan has joined #openstack-neutron | 10:15 | |
| *** topshare_ has quit IRC | 10:21 | |
| *** gongysh has quit IRC | 10:21 | |
| neiljerram | vks: ping, are you still there? | 10:21 |
| vks | neiljerram, hi | 10:22 |
| openstackgerrit | Cedric Brandily proposed openstack/python-neutronclient: Prefer argparse mutual exclusion https://review.openstack.org/166614 | 10:22 |
| neiljerram | For reviewing a blueprint, there should be a corresponding review job at https://review.openstack.org/#/q/status:open+project:openstack/neutron-specs,n,z | 10:22 |
| *** bjornar_ has quit IRC | 10:23 | |
| *** ihrachyshka has joined #openstack-neutron | 10:23 | |
| *** markvoelker has joined #openstack-neutron | 10:25 | |
| vks | neiljerram, i was working on bp and needed to put review comment. From web ui I was able to do. I downloaded bp locally on my setup and wanted to put comment locally and then commit. Is it possible? or All the review comment is only done in gerrit UI. | 10:26 |
| vks | *downloaded == clone | 10:26 |
| neiljerram | vks: As far as I understand, review comments are done only in gerrit UI. | 10:27 |
| *** yamamoto has quit IRC | 10:27 | |
| vks | neiljerram, if thats the case , then its fine | 10:27 |
| neiljerram | vks: Well I'm afraid I'm not 100% sure, but I believe that is the case. | 10:28 |
| *** yamamoto has joined #openstack-neutron | 10:28 | |
| vks | neiljerram, one thing more in another bp i was not added as reviewer but I given review comment . I see the (cross) icon which shows "remove reviewer" while thats not the case with other reviewers? | 10:29 |
| *** markvoelker has quit IRC | 10:30 | |
| neiljerram | vks: I believe you only see the cross icon for yourself - because you should not be able to remove other people as reviewers. | 10:31 |
| vks | neiljerram, exactly | 10:31 |
| openstackgerrit | Cedric Brandily proposed openstack/python-neutronclient: Prefer argparse mutual exclusion https://review.openstack.org/166614 | 10:31 |
| ajo | garyk, I'm trying to reply to your process monitor nit :), but... gerrit is acting up... | 10:32 |
| ajo | garyk, the stop() method only sets a variable to True, so no risk of exception, and no need for exception capture | 10:33 |
| *** Networkn3rd has joined #openstack-neutron | 10:40 | |
| *** shaohe_feng has joined #openstack-neutron | 10:40 | |
| *** jbrendel has quit IRC | 10:41 | |
| openstackgerrit | Cedric Brandily proposed openstack/python-neutronclient: Support fwaasrouterinsertion extension https://review.openstack.org/166776 | 10:41 |
| *** jbrendel has joined #openstack-neutron | 10:41 | |
| *** john-davidge has joined #openstack-neutron | 10:42 | |
| *** yamamoto has quit IRC | 10:43 | |
| *** Networkn3rd has quit IRC | 10:44 | |
| *** ihrachyshka has quit IRC | 10:47 | |
| *** aranjan has quit IRC | 10:49 | |
| *** stanzgy has quit IRC | 10:49 | |
| *** saikrishna_ has joined #openstack-neutron | 10:52 | |
| *** saikrishna__ has quit IRC | 10:54 | |
| *** anshul has quit IRC | 10:55 | |
| *** saikrishna__ has joined #openstack-neutron | 10:59 | |
| *** Bsony_ has joined #openstack-neutron | 11:00 | |
| *** saikrishna_ has quit IRC | 11:02 | |
| *** nyechiel_ has joined #openstack-neutron | 11:03 | |
| *** nyechiel has quit IRC | 11:03 | |
| *** Bsony has quit IRC | 11:04 | |
| openstackgerrit | Sam Betts proposed openstack/neutron: Add an internal DHCPv6 client for Prefix Delegation https://review.openstack.org/161381 | 11:05 |
| *** matrohon has quit IRC | 11:09 | |
| garyk | ajo: ok, undetsood. please write on gerrit if and when it is ever up again :) | 11:09 |
| *** saikrishna__ has quit IRC | 11:09 | |
| garyk | ajo: please post me the review link and I will change my score | 11:09 |
| ajo | sure, thanks garyk | 11:10 |
| *** anshul has joined #openstack-neutron | 11:10 | |
| *** coolsvap is now known as coolsvap|afk | 11:11 | |
| *** _amrith_ is now known as amrith | 11:13 | |
| *** vthapar__ has quit IRC | 11:17 | |
| *** neiljerr` has joined #openstack-neutron | 11:19 | |
| *** lukasa has quit IRC | 11:20 | |
| *** neiljerram has quit IRC | 11:20 | |
| *** yfried__ is now known as yfried|afk | 11:20 | |
| *** markvoelker has joined #openstack-neutron | 11:26 | |
| *** saikrishna has joined #openstack-neutron | 11:26 | |
| *** lori|away is now known as lori | 11:27 | |
| *** gsagie has quit IRC | 11:30 | |
| *** mwagner_lap has quit IRC | 11:30 | |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 11:30 |
| *** markvoelker has quit IRC | 11:30 | |
| sudipto | garyk: I incorporated the review comments on https://review.openstack.org/165539 - could you please take a look when you can? :) | 11:31 |
| *** lukasa has joined #openstack-neutron | 11:32 | |
| *** saikrishna has quit IRC | 11:33 | |
| *** shaohe_feng has quit IRC | 11:33 | |
| *** lukasa has quit IRC | 11:35 | |
| *** lukasa has joined #openstack-neutron | 11:36 | |
| *** lukasa has quit IRC | 11:36 | |
| *** lukasa has joined #openstack-neutron | 11:37 | |
| *** lukasa has joined #openstack-neutron | 11:37 | |
| sudipto | obondarev: Incorporated your comment on https://review.openstack.org/165539 | 11:38 |
| *** gsagie has joined #openstack-neutron | 11:40 | |
| *** yfried|afk is now known as yfried__ | 11:40 | |
| *** pc_m has joined #openstack-neutron | 11:42 | |
| *** atuvenie has joined #openstack-neutron | 11:43 | |
| *** aranjan has joined #openstack-neutron | 11:45 | |
| *** Longgeek has quit IRC | 11:46 | |
| *** markvoelker has joined #openstack-neutron | 11:46 | |
| *** ferest has joined #openstack-neutron | 11:47 | |
| *** ferest has quit IRC | 11:50 | |
| *** vthapar has joined #openstack-neutron | 11:52 | |
| *** jckasper has joined #openstack-neutron | 11:52 | |
| *** yamamoto_ has joined #openstack-neutron | 11:53 | |
| *** claudiub has joined #openstack-neutron | 11:56 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 12:00 |
| *** ggnel has joined #openstack-neutron | 12:01 | |
| *** nyechiel_ has quit IRC | 12:01 | |
| *** nyechiel_ has joined #openstack-neutron | 12:01 | |
| *** vks has quit IRC | 12:02 | |
| *** bjornar_ has joined #openstack-neutron | 12:03 | |
| *** raildo has joined #openstack-neutron | 12:04 | |
| *** bradjones has quit IRC | 12:05 | |
| *** bradjones has joined #openstack-neutron | 12:06 | |
| *** bradjones has quit IRC | 12:06 | |
| *** bradjones has joined #openstack-neutron | 12:06 | |
| *** criptoverso has quit IRC | 12:06 | |
| *** nyechiel_ has quit IRC | 12:08 | |
| *** nyechiel_ has joined #openstack-neutron | 12:08 | |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 12:09 |
| *** bjornar_ has quit IRC | 12:10 | |
| *** stockpirate has joined #openstack-neutron | 12:11 | |
| garyk | sudipto: can you please look at my comment on https://review.openstack.org/#/c/165539/5/neutron/db/agents_db.py,cm it is a minor nit | 12:12 |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 12:13 |
| sudipto | garyk: Does the 2 comments mean the same thing? As in, you want a consistency in the way the lines are started with " right? | 12:14 |
| sudipto | garyk: Nah, i figured :) | 12:16 |
| garyk | sudipto: no. sorry there are 2 things: | 12:16 |
| garyk | 1. " under " - alignment | 12:16 |
| garyk | 2. space at end of sentence | 12:16 |
| sudipto | garyk: Yeah i figured...when i looked them in the editor. | 12:16 |
| garyk | sudipto: tx | 12:17 |
| *** aranjan has quit IRC | 12:19 | |
| *** dave-mccowan has joined #openstack-neutron | 12:21 | |
| stockpirate | Hi, could i ask some configuration questions here? | 12:23 |
| *** mwagner_lap has joined #openstack-neutron | 12:23 | |
| gsagie | i have encountered in the following bug: https://bugs.launchpad.net/neutron/+bug/1435012 , just wondering what do you suggest will be the best way to solve this issue? it seems like the snat redirect remove inside the l3 agent is redundant (as the port is being deleted by the L2 agent) unless i am missing something | 12:23 |
| openstack | Launchpad bug 1435012 in neutron "Exception when removing last network port from external router" [Undecided,New] - Assigned to Gal Sagie (gal-sagie) | 12:23 |
| *** bishtsanddev has joined #openstack-neutron | 12:24 | |
| bishtsanddev | hi | 12:24 |
| *** ishant has quit IRC | 12:27 | |
| *** amuller has joined #openstack-neutron | 12:28 | |
| *** Networkn3rd has joined #openstack-neutron | 12:28 | |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 12:29 |
| *** blues-man has quit IRC | 12:29 | |
| sudipto | garyk: done :) | 12:29 |
| garyk | sudipto: ok, thanks! | 12:29 |
| *** topshare has joined #openstack-neutron | 12:29 | |
| *** ihrachyshka has joined #openstack-neutron | 12:31 | |
| *** bjornar_ has joined #openstack-neutron | 12:32 | |
| sudipto | garyk: Thx for the review! | 12:32 |
| *** Networkn3rd has quit IRC | 12:33 | |
| *** dims__ has quit IRC | 12:34 | |
| *** dims__ has joined #openstack-neutron | 12:34 | |
| *** amrith is now known as _amrith_ | 12:35 | |
| *** vthapar has quit IRC | 12:36 | |
| *** gildub has quit IRC | 12:37 | |
| *** jpena is now known as jpena|lunch | 12:37 | |
| *** nmagnezi has quit IRC | 12:43 | |
| *** kickinz1 is now known as kickinz1|afk | 12:44 | |
| *** lazy_prince is now known as killer_prince | 12:45 | |
| *** kickinz1|afk is now known as kickinz1 | 12:46 | |
| garyk | sudipto: np. thanks for addressing the comments | 12:48 |
| openstackgerrit | Cedric Brandily proposed openstack/python-neutronclient: Support fwaasrouterinsertion extension https://review.openstack.org/166776 | 12:49 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 12:52 |
| *** dims__ is now known as dims | 12:52 | |
| *** shwetaap has joined #openstack-neutron | 12:53 | |
| *** neeti has quit IRC | 12:54 | |
| *** chandankumar has quit IRC | 12:55 | |
| *** tiswanso has joined #openstack-neutron | 12:55 | |
| *** nmagnezi has joined #openstack-neutron | 12:55 | |
| *** matrohon has joined #openstack-neutron | 12:57 | |
| *** dboik has quit IRC | 12:58 | |
| *** Marga_ has joined #openstack-neutron | 12:59 | |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 12:59 |
| *** shwetaap1 has joined #openstack-neutron | 12:59 | |
| *** tiswanso has quit IRC | 12:59 | |
| *** chandankumar has joined #openstack-neutron | 12:59 | |
| *** dave-mccowan has quit IRC | 13:00 | |
| *** yatin has quit IRC | 13:00 | |
| *** lykinsbd has joined #openstack-neutron | 13:00 | |
| *** dave-mccowan has joined #openstack-neutron | 13:00 | |
| *** amotoki has quit IRC | 13:01 | |
| *** Bsony has joined #openstack-neutron | 13:01 | |
| *** shwetaap has quit IRC | 13:02 | |
| HenryG | amuller: ping | 13:02 |
| amuller | HenryG: pong | 13:02 |
| HenryG | amuller: can you review the functional test additions for https://review.openstack.org/149068 | 13:03 |
| *** chandankumar_ has joined #openstack-neutron | 13:03 | |
| *** chandankumar has quit IRC | 13:03 | |
| amuller | HenryG: Will do | 13:04 |
| *** thomasem has joined #openstack-neutron | 13:04 | |
| *** Bsony_ has quit IRC | 13:04 | |
| *** dane_leblanc_ has quit IRC | 13:04 | |
| HenryG | amuller: Thanks. It looks fine to me, but you are more familiar with the details | 13:04 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Simplify keepalived.virtual_routes https://review.openstack.org/160430 | 13:05 |
| *** nyechiel_ has quit IRC | 13:06 | |
| *** amotoki has joined #openstack-neutron | 13:06 | |
| *** baoli has joined #openstack-neutron | 13:07 | |
| *** nyechiel_ has joined #openstack-neutron | 13:08 | |
| *** marrusl has joined #openstack-neutron | 13:08 | |
| *** sudipto has quit IRC | 13:09 | |
| *** Networkn3rd has joined #openstack-neutron | 13:09 | |
| *** akamyshnikova_ has joined #openstack-neutron | 13:12 | |
| *** yamamoto_ has quit IRC | 13:12 | |
| *** thomasem has quit IRC | 13:12 | |
| *** topshare has quit IRC | 13:13 | |
| *** kbringard has joined #openstack-neutron | 13:13 | |
| *** flaviof has joined #openstack-neutron | 13:13 | |
| *** leakypipes has quit IRC | 13:13 | |
| *** aranjan has joined #openstack-neutron | 13:15 | |
| *** anilvenkata has left #openstack-neutron | 13:16 | |
| *** annegentle has joined #openstack-neutron | 13:16 | |
| *** chandankumar_ has quit IRC | 13:16 | |
| *** thomasem has joined #openstack-neutron | 13:16 | |
| *** topshare has joined #openstack-neutron | 13:17 | |
| *** topshare has quit IRC | 13:18 | |
| *** dboik has joined #openstack-neutron | 13:18 | |
| pc_m | mestery: ping | 13:19 |
| *** yamamoto_ has joined #openstack-neutron | 13:19 | |
| mestery | pc_m: pong | 13:20 |
| pc_m | Can you review https://review.openstack.org/#/c/166366/2? It actually breaks Vyatta code. | 13:20 |
| *** topshare has joined #openstack-neutron | 13:22 | |
| *** yfauser has joined #openstack-neutron | 13:22 | |
| mestery | pc_m: Will do sir, give me about 20 minutes or so | 13:23 |
| pc_m | mestery: thanks | 13:23 |
| neiljerr` | Please could I just check, as https://wiki.openstack.org/wiki/Network/Meetings hasn't been updated yet: is the next Neutron meeting at 2pm UTC on Tuesday? | 13:23 |
| *** yfauser has left #openstack-neutron | 13:23 | |
| *** nmagnezi_ has joined #openstack-neutron | 13:23 | |
| *** yfried__ is now known as yfried|afk | 13:24 | |
| *** coolsvap|afk is now known as coolsvap | 13:25 | |
| amotoki | neiljerr`: yes. The next meeting start 1400UTC. | 13:26 |
| amotoki | neiljerr`: you can check the meeting logs to know the next meeting time. | 13:26 |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 13:26 |
| *** stockpirate has quit IRC | 13:26 | |
| *** nmagnezi has quit IRC | 13:26 | |
| *** jschwarz has joined #openstack-neutron | 13:27 | |
| ZZelle | jschwarz, hi | 13:28 |
| *** claudiub has quit IRC | 13:28 | |
| neiljerr` | amotoki: thanks! | 13:29 |
| *** numan has quit IRC | 13:29 | |
| *** popw has quit IRC | 13:30 | |
| *** alexpilotti has joined #openstack-neutron | 13:31 | |
| jschwarz | ZZelle, hello :) | 13:31 |
| *** shwetaap1 has quit IRC | 13:31 | |
| jschwarz | ZZelle, how are you? | 13:31 |
| *** tiswanso has joined #openstack-neutron | 13:31 | |
| ZZelle | jschwarz, fine | 13:31 |
| ZZelle | jschwarz, could you relook at https://review.openstack.org/#/c/143412/22/neutron/tests/common/net_helpers.py? | 13:32 |
| jschwarz | ZZelle, sure thing, will do it in a few :) | 13:32 |
| *** yfried|afk is now known as yfried__ | 13:35 | |
| *** zzzeek has joined #openstack-neutron | 13:36 | |
| jschwarz | ZZelle, no place in the PEP8 mentions you can raise an exception class | 13:39 |
| *** superdan is now known as dansmith | 13:39 | |
| jschwarz | ZZelle, all examples are in the form of "raise MyException('message')", so I think it's heavily implied | 13:39 |
| *** wverdugo5001 has left #openstack-neutron | 13:39 | |
| ZZelle | jschwarz, nothing mentions you can not :) | 13:39 |
| pc_m | mestery: Also, how do we proceed with the VPN refactoring commits (there will be 3, one to add new method to neutron side, one to use on vpn repo side, and a third to remove the old method from neutron)? | 13:40 |
| pc_m | mestery: These were under progress as part of L3 agent refactoring, just before K-3. Two are out for review, one WIP. Would like to know how I should proceed. | 13:41 |
| mestery | pc_m: Do you think these are critical for Kilo? If not, we can let them wait a bit until Liberty opens. | 13:42 |
| pc_m | not critical. Cleaning up mostly, removing duplicate mechanisms, and eliminating dependency on L3 agent. | 13:42 |
| amuller | pc_m: mestery: Please keep backports in mind | 13:43 |
| *** nmagnezi__ has joined #openstack-neutron | 13:43 | |
| ajo | garyk, pong: https://review.openstack.org/#/c/165117/ | 13:43 |
| amuller | makes it harder to backport fixes if major refactors go in L and not K | 13:43 |
| ajo | :) | 13:43 |
| ajo | amuller, that's true | 13:43 |
| ajo | but I guess refactors become more risky as we approach the end of the cycle | 13:44 |
| pc_m | amuller: good point. It does change the way the event notification is handled. | 13:44 |
| openstackgerrit | Martin Kletzander proposed openstack/neutron: Fix common misspellings https://review.openstack.org/166832 | 13:44 |
| *** sudipto has joined #openstack-neutron | 13:46 | |
| amotoki | pc_m: at a glance, it seems vyatta-vpn-agent was not correctly monkey-patched. I think introducing the uniform way of neutron-*/cmd/eventlet is not directly related. | 13:46 |
| *** dane_leblanc has joined #openstack-neutron | 13:46 | |
| jschwarz | ZZelle, fine. then I raise the case of consistency | 13:46 |
| pc_m | mestery: I've got no strong desire either way, other than it's in progress and would like to complete it. | 13:46 |
| ZZelle | jschwarz, rololo, that's quite unfair | 13:46 |
| jschwarz | ZZelle, I can't find any code that raises an exception class (all 'raise' statements throw an instance of a subclass of Exception) | 13:46 |
| jschwarz | ZZelle, XD | 13:46 |
| *** nmagnezi_ has quit IRC | 13:46 | |
| jschwarz | ZZelle, Actually I can find one 'raise NotImplementedError' in ipam/driver.py | 13:47 |
| jschwarz | and a few in neutron/openstack/common | 13:47 |
| jschwarz | but the vast majority raises instances... | 13:47 |
| ZZelle | jschwarz, ah! | 13:47 |
| amotoki | pc_m: recent oslo.messaging change requires eventlet monkey-patch more compared to the older versions, so I think the problem occurs frequently. | 13:48 |
| pc_m | amotoki: Sridhar R mentioned that Vyatta VPN was broken after the change to cmd/eventlet. His commit is to switch that implementation over, to match what is done in other implementations. Looks like it was missed (not sure if VYatta VPN was added after the change) | 13:48 |
| ajo | what's wrong with gerrit? :] | 13:48 |
| ajo | do we know? (just curious) | 13:48 |
| jschwarz | ZZelle, :) | 13:48 |
| amotoki | pc_m: hmm... | 13:48 |
| *** aranjan has quit IRC | 13:49 | |
| pc_m | amotoki: He was testing it late last week and hence provided this commit to fix it. | 13:49 |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Remove downgrade from existing migrations https://review.openstack.org/165740 | 13:49 |
| *** bishtsanddev has quit IRC | 13:49 | |
| amotoki | pc_m: cmd/eventlet was introduced to ensure to apply monkey-patch in neutorn code, so it might change the RPC behavior. | 13:49 |
| *** salv-orlando has joined #openstack-neutron | 13:50 | |
| amotoki | pc_m: it mihgt change the frequency of the problem. | 13:50 |
| mestery | pc_m amuller: We can look into getting these in then, especially if carl thinks they'll help | 13:50 |
| *** shwetaap has joined #openstack-neutron | 13:50 | |
| *** jpena|lunch is now known as jpena | 13:52 | |
| amuller | jschwarz: ZZelle: http://stackoverflow.com/questions/16706956/is-there-a-difference-between-raising-an-exception-and-exception | 13:52 |
| pc_m | mestery: We should see what carl says. Not sure if he's looked at these yet. | 13:52 |
| mestery | pc_m: Good idea | 13:53 |
| sudipto | amotoki: One review for this patch: https://review.openstack.org/#/c/165539/4 whenever you have time! :) | 13:53 |
| pc_m | amotoki: Ah, so you think a separate issue, and the cmd/eventlet may be showing the issue more? | 13:53 |
| ZZelle | amuller, jschwarz, ha! | 13:53 |
| pc_m | amotoki: Do you think his change then is masking an issue? | 13:53 |
| amuller | ZZelle: jschwarz: Raising a naked ValueError lacks information | 13:54 |
| amuller | The exception message should contain a summary of why the exception was raised | 13:54 |
| amuller | what value was missing? | 13:54 |
| amotoki | pc_m: I don't think so. new oslo.messaging turns out some module/processes are not monkey-patched correctly and cmd/eventlet was introduced. | 13:54 |
| amuller | what went wrong? | 13:54 |
| amuller | otherwise the log only contains the trace and there's no clue of why the exception was raised | 13:54 |
| jschwarz | ZZelle, amuller, ha! | 13:55 |
| amuller | Help the admin help you ;-) | 13:55 |
| amotoki | pc_m: if the patch fixes the problem, the cause must be non-appropriate monkey-patch, so I think the way is correct. | 13:55 |
| jschwarz | ZZelle, there *is* no difference (sorry for being misleading there), but an instance is preferable | 13:55 |
| jschwarz | to me at least | 13:55 |
| *** topshare has quit IRC | 13:55 | |
| *** shwetaap has quit IRC | 13:55 | |
| ZZelle | amuller, if i add a message, it will be requested to i18n it ... which is dumm for test | 13:55 |
| jschwarz | ZZelle, I don't mind not having a message | 13:56 |
| *** banix has joined #openstack-neutron | 13:56 | |
| jschwarz | as long as it's an instance XD | 13:56 |
| amuller | ZZelle: I'm pretty sure the guidlines state do not translate messages in the test tree | 13:56 |
| *** blues-man has joined #openstack-neutron | 13:56 | |
| ZZelle | amuller, good new | 13:56 |
| ZZelle | amuller, good news | 13:56 |
| *** localloop127 has joined #openstack-neutron | 13:57 | |
| jschwarz | ZZelle, how come you split your 2 patches into... 4? | 13:57 |
| *** topshare has joined #openstack-neutron | 13:57 | |
| pc_m | amotoki: If you can check out the review, that would be great. | 13:57 |
| *** armax has joined #openstack-neutron | 13:58 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 13:59 |
| *** blues-man has quit IRC | 14:00 | |
| *** blues-man has joined #openstack-neutron | 14:00 | |
| ZZelle | jschwarz, sorry, could you rephrase? | 14:00 |
| *** nyechiel__ has joined #openstack-neutron | 14:00 | |
| *** nyechiel_ has quit IRC | 14:03 | |
| jschwarz | ZZelle, I saw your patches got multiplied | 14:03 |
| jschwarz | ZZelle, the ones about refactoring the BaseOVSTestCase got turned into 4 different patches | 14:03 |
| jschwarz | was wondering why | 14:03 |
| *** chandankumar_ has joined #openstack-neutron | 14:03 | |
| *** prad has joined #openstack-neutron | 14:04 | |
| ZZelle | jschwarz, iirc, a maru proposal as the change was becoming quite large with LinuxBridgeFixture | 14:04 |
| *** topshare has quit IRC | 14:05 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers https://review.openstack.org/165866 | 14:05 |
| *** jamie_h has joined #openstack-neutron | 14:05 | |
| *** topshare has joined #openstack-neutron | 14:05 | |
| *** coolsvap is now known as coolsvap|afk | 14:05 | |
| garyk | mestery: you around to chat about https://review.openstack.org/#/c/164208/? | 14:05 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 14:05 |
| *** topshare has quit IRC | 14:05 | |
| *** Marga_ has quit IRC | 14:05 | |
| mestery | garyk: Indeed! You're in my list to talk to, in a call, give me 10 minutes? | 14:05 |
| mestery | garyk: Will that work? | 14:05 |
| *** _amrith_ is now known as amrith | 14:06 | |
| garyk | mestery: sure np. ping me when you are done | 14:06 |
| *** topshare has joined #openstack-neutron | 14:06 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 14:07 |
| *** julim has joined #openstack-neutron | 14:07 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Clean neutron.tests.functional.agent.linux.base https://review.openstack.org/161913 | 14:07 |
| *** nmagnezi_ has joined #openstack-neutron | 14:08 | |
| *** shwetaap has joined #openstack-neutron | 14:08 | |
| *** topshare has quit IRC | 14:10 | |
| *** nmagnezi__ has quit IRC | 14:12 | |
| *** zz_jgrimm- is now known as jgrimm | 14:12 | |
| *** yamamoto_ has quit IRC | 14:12 | |
| jschwarz | ZZelle, ack | 14:14 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Simplify keepalived.virtual_routes https://review.openstack.org/160430 | 14:14 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Add extra subnet route to ha router https://review.openstack.org/159772 | 14:14 |
| *** Rajeev has joined #openstack-neutron | 14:17 | |
| *** absubram has joined #openstack-neutron | 14:17 | |
| *** yfried__ is now known as yfried|afk | 14:17 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Simplify keepalived.virtual_routes https://review.openstack.org/160430 | 14:18 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Add extra subnet route to ha router https://review.openstack.org/159772 | 14:18 |
| *** yfried|afk is now known as yfried__ | 14:18 | |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 14:18 |
| *** coolsvap has joined #openstack-neutron | 14:19 | |
| *** shwetaap has quit IRC | 14:20 | |
| *** tiswanso has quit IRC | 14:21 | |
| jschwarz | ZZelle, re: https://review.openstack.org/#/c/143412/22..26/neutron/tests/common/helpers.py (the increment_ip_cidr comment on L72) | 14:27 |
| jschwarz | I didn't see any change that checks this, thought I might have missed it? | 14:27 |
| ZZelle | jschwarz, i forgot it :( | 14:28 |
| ZZelle | jschwarz, i will update | 14:28 |
| jschwarz | ZZelle, that's ok | 14:28 |
| jschwarz | ZZelle, Wait a few minutes, I'm going through that review so I might have more questions | 14:28 |
| ZZelle | ok | 14:28 |
| *** annegentle has quit IRC | 14:29 | |
| *** nmagnezi_ has quit IRC | 14:33 | |
| *** carlp has joined #openstack-neutron | 14:33 | |
| *** dannywilson has joined #openstack-neutron | 14:33 | |
| *** achanda has joined #openstack-neutron | 14:34 | |
| *** annegentle has joined #openstack-neutron | 14:34 | |
| *** annegentle has quit IRC | 14:34 | |
| *** shaohe_feng has joined #openstack-neutron | 14:35 | |
| *** jobewan has joined #openstack-neutron | 14:36 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 14:36 |
| *** Marga_ has joined #openstack-neutron | 14:38 | |
| *** shwetaap has joined #openstack-neutron | 14:40 | |
| *** achanda has quit IRC | 14:41 | |
| *** shaohe_feng has quit IRC | 14:42 | |
| *** localloo1 has joined #openstack-neutron | 14:42 | |
| jschwarz | ZZelle, go ahead :) | 14:42 |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Allow update of ext gateway IP's w/out port delete https://review.openstack.org/161073 | 14:42 |
| *** shaohe_feng has joined #openstack-neutron | 14:42 | |
| otherwiseguy | mestery: https://review.openstack.org/#/c/166294/ is part of the ovsdb-native patch/blueprint, it ensures people can run a sanity check to make sure that ovsdb native has the necessary packages to run. | 14:44 |
| *** localloop127 has quit IRC | 14:44 | |
| mestery | otherwiseguy: +2/+A, agreed | 14:45 |
| mestery | otherwiseguy: I was thinking about that after I posted that -1, and I'm comfortable merigng that one | 14:45 |
| *** aranjan has joined #openstack-neutron | 14:45 | |
| otherwiseguy | mestery: cool. just wanted to make sure people weren't just getting random import errors and being confused. "What, I've got openvswitch installed!?" or something. :D | 14:46 |
| mestery | otherwiseguy: lol :) | 14:46 |
| *** Marga_ has quit IRC | 14:46 | |
| otherwiseguy | And hopefully distros are using the sanity checks. :p | 14:46 |
| garyk | otherwiseguy: they should use linux bridge instead :) | 14:46 |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 14:46 |
| *** Marga_ has joined #openstack-neutron | 14:46 | |
| otherwiseguy | garyk: Have you been talking to ijw again? :p | 14:46 |
| garyk | :) | 14:47 |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 14:47 |
| russellb | nah, should use *more* ovs | 14:47 |
| garyk | otherwiseguy: rumour has it that retro is in | 14:47 |
| russellb | otherwiseguy: once that blueprint settles, i'd love to talk about how we can make networking-ovn share some of the code and use ovsdb directly | 14:48 |
| otherwiseguy | garyk: I will admit that OVSDB/the python OVS IDL library is not the easiest protocol in the world to use correctly, and that there might be some things I'd do differently. :p | 14:49 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Refactoring to use callback mechanism (WIP) https://review.openstack.org/165226 | 14:49 |
| otherwiseguy | russellb: sounds like a plan to me. :) | 14:49 |
| russellb | otherwiseguy: in the meantime though, just using a command line util ... see https://review.openstack.org/#/c/166815/2 and https://review.openstack.org/#/c/166851/1 | 14:49 |
| *** r-daneel has joined #openstack-neutron | 14:49 | |
| openstackgerrit | Kyle Mestery proposed openstack/neutron: Move README.odl into opendaylight directory https://review.openstack.org/166863 | 14:50 |
| *** annegentle has joined #openstack-neutron | 14:51 | |
| otherwiseguy | russellb: yeah, the things I see there shouldn't be too hard to get working with the other stuff. Just need to add the ability to specify a different database, and the rest mostly just setting/getting properties? | 14:52 |
| *** pradip_vedams has quit IRC | 14:52 | |
| garyk | russellb: we should move this code to a library - similary to the ovs lib. | 14:53 |
| *** moshele has quit IRC | 14:53 | |
| garyk | russellb: i can create the library | 14:53 |
| russellb | otherwiseguy: the db arg is so you can talk to a remote ovsdb | 14:54 |
| russellb | garyk: cool ... though we should be careful it actually needs to be openstack specific, and just upstream code back to ovs' python ovsdb code as appropriate | 14:55 |
| russellb | i haven't really looked to see what parts fit that | 14:55 |
| russellb | just speaking in general :) | 14:55 |
| russellb | otherwiseguy: it's pretty basic ovsdb stuff i guess ... creating/deleting rows, changing some attributes on them sometimes | 14:55 |
| russellb | otherwiseguy: you can look at the ovn-nbctl code as an example | 14:56 |
| * russellb gets link | 14:56 | |
| otherwiseguy | garyk: should be possible to use the ovsdb api to handle the ovsdb-specific stuff. I'd like to see it extended if possible. | 14:56 |
| russellb | http://openvswitch.org/pipermail/dev/2015-March/052752.html | 14:56 |
| garyk | russellb: otherwiseguy: my understanidng is that the python bindings need a lot of work and it may not be ready to consume. i may be mistaken | 14:56 |
| otherwiseguy | email for patches, what is this the 1800s? | 14:56 |
| otherwiseguy | :p | 14:56 |
| russellb | otherwiseguy: ikr? | 14:56 |
| russellb | otherwiseguy: the nbrec_*() calls are the stuff that's actually adding to a transaction | 14:57 |
| *** thedodd has joined #openstack-neutron | 14:57 | |
| *** neeti has joined #openstack-neutron | 14:57 | |
| *** dannywilson has quit IRC | 14:57 | |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Auto-update gateway port after subnet-create https://review.openstack.org/161085 | 14:57 |
| *** dannywilson has joined #openstack-neutron | 14:58 | |
| otherwiseguy | garyk: the python bindings are...complicated, but useable. They are much slower than they should be because they include a line-by-line port of OVS's custom C JSON parser, which is about 60x slower than any other JSON parser (the python version, the C is fast). | 14:58 |
| otherwiseguy | They can't use the built-in python JSON parser because they need to be able to do stream parsing. | 14:58 |
| *** chandankumar_ has quit IRC | 14:58 | |
| *** nmagnezi_ has joined #openstack-neutron | 14:58 | |
| otherwiseguy | But either a) using py-yajl instead with some patches I have in a pull request, or b) just writing some bindings to the C-version should fix most of the speed issues. | 14:59 |
| garyk | otherwiseguy: ok, thanks for the clarificiation | 15:00 |
| otherwiseguy | The complexity issues, those would require some serious work. | 15:00 |
| otherwiseguy | There are some bugs I keep meaning to send patches for, but none have been showstoppers for anything I've tried. | 15:01 |
| *** tiswanso has joined #openstack-neutron | 15:01 | |
| otherwiseguy | garyk: As a C developer, I can definitely tell that the Python OVS stuff was written by a C developer. :p | 15:02 |
| otherwiseguy | And I get the feeling it was written very quickly by someone very smart. | 15:03 |
| *** carl_baldwin has joined #openstack-neutron | 15:03 | |
| amuller | HenryG: I reviewed the dual stack patch, essentially it was still not asserting the new functionality... | 15:03 |
| garyk | otherwiseguy: yeah, it was written by someone super smart | 15:04 |
| *** neeti has quit IRC | 15:04 | |
| ihrachyshka | zigo_, hey | 15:05 |
| zigo_ | ihrachyshka: Hi ! | 15:06 |
| ihrachyshka | zigo_, I had a question to you. regarding the patch for test suite that allows to run unit tests from site-packages. I saw your comment for lbaas part that it worked for you. But I expected that it will work with neutron part merged only. Am I wrong? | 15:07 |
| ihrachyshka | zigo_, neutron part is https://review.openstack.org/165237 | 15:07 |
| ihrachyshka | not merged yet | 15:08 |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron-vpnaas: Scenario test for vpnaas: ipsec-site-connection https://review.openstack.org/159746 | 15:08 |
| *** nyechiel__ has quit IRC | 15:08 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Move network_type constants to plugin constants.py https://review.openstack.org/166879 | 15:09 |
| zigo_ | ihrachyshka: To be honest, I first applied the patch to Neutron, then to lbaas, so I have no idea if the lbaas patch works without the Neutron one. | 15:11 |
| ihrachyshka | zigo_, I right, that's what I expected. thanks! | 15:11 |
| *** nyechiel__ has joined #openstack-neutron | 15:12 | |
| *** yamamoto_ has joined #openstack-neutron | 15:13 | |
| *** yfried has joined #openstack-neutron | 15:13 | |
| *** yfried has quit IRC | 15:14 | |
| *** yfried has joined #openstack-neutron | 15:14 | |
| *** yfried__ has quit IRC | 15:16 | |
| *** xgerman has joined #openstack-neutron | 15:17 | |
| *** kerpan has joined #openstack-neutron | 15:18 | |
| *** aranjan has quit IRC | 15:18 | |
| *** yamamoto_ has quit IRC | 15:19 | |
| *** lpetrut has quit IRC | 15:20 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed https://review.openstack.org/162260 | 15:20 |
| *** gsagie has quit IRC | 15:20 | |
| *** emagana has joined #openstack-neutron | 15:23 | |
| *** jorgem has joined #openstack-neutron | 15:23 | |
| *** SridharG has left #openstack-neutron | 15:25 | |
| *** sdague has quit IRC | 15:26 | |
| *** yfried has quit IRC | 15:26 | |
| *** claudiub has joined #openstack-neutron | 15:27 | |
| *** tidwellr has quit IRC | 15:27 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Deprecate use_namespaces option https://review.openstack.org/166888 | 15:27 |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 15:27 |
| *** aepifanov has quit IRC | 15:29 | |
| zigo_ | ihrachyshka: I still don't know what to do with neutron and neutron-lbaas build-depending on each other... :( | 15:30 |
| ihrachyshka | zigo_, nothing, that's basically the way it will go until we decouple them properly | 15:31 |
| zigo_ | Ah ok. | 15:31 |
| ihrachyshka | zigo_, or you don't run tests :) | 15:31 |
| zigo_ | Happy to know it will at least happen someday! :) | 15:31 |
| ihrachyshka | zigo_, there is a spec to discuss in Van | 15:31 |
| zigo_ | ihrachyshka: Yeah, black-listing some of the unit tests is one of the ways, but not the best one... | 15:31 |
| ihrachyshka | zigo_, https://review.openstack.org/#/c/154736/ | 15:31 |
| ihrachyshka | zigo_, that's PITA not for packagers only, so hopefully it won't too long :) | 15:32 |
| zigo_ | :) | 15:32 |
| ihrachyshka | won't take | 15:32 |
| *** ajmiller has joined #openstack-neutron | 15:33 | |
| *** akamyshnikova_ has quit IRC | 15:33 | |
| *** absubram has quit IRC | 15:35 | |
| *** yamahata has joined #openstack-neutron | 15:39 | |
| *** sdague has joined #openstack-neutron | 15:40 | |
| *** jorgem has quit IRC | 15:41 | |
| *** jaypipes has joined #openstack-neutron | 15:42 | |
| *** afazekas_ has quit IRC | 15:42 | |
| *** irenab has quit IRC | 15:43 | |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers https://review.openstack.org/149068 | 15:44 |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 15:44 |
| ZZelle | jschwarz, plop | 15:45 |
| jschwarz | ZZelle, plip | 15:45 |
| ZZelle | jschwarz, what would be the correct place to add test for neutron/tests/common/net_helpers.py? neutron/tests/common/test_net_helpers? | 15:46 |
| *** banix has quit IRC | 15:46 | |
| jschwarz | (peh, tests for tests... Maru suggested that?) | 15:46 |
| jschwarz | ZZelle, probably functional, or fullstack when it merges | 15:47 |
| jschwarz | ZZelle, neutron/tests/functional/helpers/ | 15:47 |
| jschwarz | (create that dir) | 15:47 |
| ZZelle | jschwarz, perhaps, but some features become complicated and we should (or not) test them | 15:48 |
| jschwarz | no testing is my kind of testing! ;) | 15:48 |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Move external port processing to router classes https://review.openstack.org/158495 | 15:49 |
| *** dannywilson has quit IRC | 15:49 | |
| armax | ihrachyshka: | 15:50 |
| armax | ping | 15:50 |
| ihrachyshka | armax, hey | 15:50 |
| armax | ihrachyshka: question on https://review.openstack.org/#/c/160596/ | 15:51 |
| ihrachyshka | armax, shoot! | 15:51 |
| ihrachyshka | oh, I was too scared to look into this in the morning, and forgot about it | 15:51 |
| armax | I noticed that the author did some ovs agent reshuffling related to eventlet monkey patching | 15:51 |
| *** xgerman has quit IRC | 15:52 | |
| armax | ihrachyshka: can’t think of a reason why you left it behind when you did yours, but I wonder if you fill me in | 15:52 |
| openstackgerrit | Cedric Brandily proposed openstack/python-neutronclient: Support fwaasrouterinsertion extension https://review.openstack.org/166776 | 15:52 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers https://review.openstack.org/165866 | 15:53 |
| ihrachyshka | armax, you mean https://review.openstack.org/#/c/160596/21/neutron/cmd/eventlet/__init__.py ? | 15:53 |
| openstackgerrit | Rajeev Grover proposed openstack/neutron: Preserve DVR FIP rule priority over Agent restarts https://review.openstack.org/150194 | 15:53 |
| *** rkukura has joined #openstack-neutron | 15:53 | |
| carl_baldwin | HenryG: ping regarding conflict in https://review.openstack.org/#/c/158495/ | 15:53 |
| armax | ihrachyshka: no, this https://review.openstack.org/#/c/160596/21/neutron/cmd/eventlet/plugins/ovs_neutron_agent.py | 15:53 |
| HenryG | carl_baldwin: looking ... | 15:54 |
| *** chandankumar has joined #openstack-neutron | 15:54 | |
| carl_baldwin | HenryG: Thanks. | 15:54 |
| *** itzikb has quit IRC | 15:54 | |
| *** emagana has quit IRC | 15:54 | |
| *** emagana has joined #openstack-neutron | 15:55 | |
| HenryG | carl_baldwin: makes perfect sense | 15:55 |
| jschwarz | armax, re: your comments on https://review.openstack.org/#/c/128259/ late last week | 15:55 |
| jschwarz | armax, I assume the resident expert is Maru? :) | 15:55 |
| ihrachyshka | armax, seems reasonable. I wanted to do the same for all plugin entry points, but markmcclain and others were against since it could interact with decomp progress that vendors are in the middle of | 15:55 |
| *** reed has joined #openstack-neutron | 15:55 | |
| armax | ihrachyshka: right, but even for ovs | 15:56 |
| armax | ? | 15:56 |
| *** YorikSar has quit IRC | 15:56 | |
| *** _cjones_ has joined #openstack-neutron | 15:56 | |
| ihrachyshka | armax, my patch included all of plugin entry points, including ovs and lb. I was instead asked to mention in devref that plugin maintainers are encouraged to apply the approach. | 15:57 |
| *** emagana has quit IRC | 15:57 | |
| *** emagana has joined #openstack-neutron | 15:57 | |
| ihrachyshka | armax, honestly, I don't see a reason to leave it as-is for vendor plugins too. if there is any conflict, it's so insignificant and easy to resolve... | 15:57 |
| armax | ihrachyshka: got it, so why are we adding this here: https://review.openstack.org/#/c/160596/21/neutron/cmd/eventlet/plugins/ovs_neutron_agent.py | 15:57 |
| armax | ihrachyshka: I am confused as to why we’re doing this in the context of patch 160596 | 15:58 |
| *** _cjones_ has quit IRC | 15:58 | |
| ihrachyshka | armax, so that when the module is imported via setup.cfg console_scripts entry point, code from neutron/eventlet/__init__.py is magically applied | 15:58 |
| *** _cjones_ has joined #openstack-neutron | 15:58 | |
| armax | ihrachyshka: that I understadn | 15:58 |
| *** emagana has quit IRC | 15:59 | |
| ihrachyshka | armax, not sure why in the context of the patch. seems that there is no big reason to include it there, though not a big deal not to do it either. | 15:59 |
| *** garyk has quit IRC | 15:59 | |
| armax | ihrachyshka: I don’t understand why we move the ovs agent to the eventlet module under cmd, for a patch that involves Hyper-V | 15:59 |
| ihrachyshka | not to -> to | 15:59 |
| ihrachyshka | well, maybe that's because they want to consolidate their change for eventlet/__init__.py so that it's not needed to later copy-paste it in other agents | 16:00 |
| *** chandankumar has quit IRC | 16:00 | |
| armax | ihrachyshka: so I am assuming you did not do it back then because you treated the ovs agent as any other plugin’s agent? | 16:00 |
| ihrachyshka | if/when they want to execute them on win32 | 16:00 |
| *** emagana has joined #openstack-neutron | 16:00 | |
| ihrachyshka | actually, seems like a good thing to do | 16:00 |
| ihrachyshka | armax, yes, I did (and so did other reviewers I guess) | 16:00 |
| amotoki | armax: ihrachyshka: At first the author modified monkey-patch in ovs-agent in neutron/plugin/openvswitch directly, but according to the author monkey-patch for windows is not specific to the agent. | 16:00 |
| *** yeungp has joined #openstack-neutron | 16:00 | |
| openstackgerrit | Merged openstack/neutron: Add sanity check for OVSDB native support https://review.openstack.org/166294 | 16:01 |
| ihrachyshka | amotoki, if there is a bug, it's probably agent agnostic, right | 16:01 |
| amotoki | then I suggested to update cmd/eventlet/__init__.py. i might have do beyond the pathc scope. | 16:01 |
| armax | amotoki: it still feels out of place | 16:01 |
| amotoki | but the goal of the patch become ambiguous and it seems better to split into sub pieces. | 16:01 |
| *** ijw has joined #openstack-neutron | 16:02 | |
| armax | amotoki, ihrachyshka but I think I have the full picture now | 16:02 |
| ihrachyshka | cool | 16:02 |
| amotoki | :-) | 16:03 |
| amotoki | I agree that patch now becomes bigger than what I thought before. | 16:04 |
| *** ijw has quit IRC | 16:04 | |
| *** ijw has joined #openstack-neutron | 16:04 | |
| armax | amotoki: what do you think is the best cource of action? | 16:04 |
| armax | amotoki: we could in theory pull that ovs stuff out of the patch and rebase the hyper-v one on top of it | 16:04 |
| *** yog__ has quit IRC | 16:05 | |
| amotoki | armax: agree. | 16:05 |
| *** Rajeev has quit IRC | 16:05 | |
| *** zhhuabj has joined #openstack-neutron | 16:05 | |
| *** apuimedo has quit IRC | 16:06 | |
| amotoki | armax: in addition vendor plugins can consume the current ovs_lib (i.e., under agent/linux/ovs_lib). | 16:06 |
| atuvenie | armax: could you please explain what do you mean by pulling the ovs stuff out and rebaseing the hyper-v? I don't quite understand | 16:06 |
| openstackgerrit | Michael Smith proposed openstack/neutron: delay fip namespace cleanup/destroy https://review.openstack.org/151758 | 16:06 |
| ihrachyshka | armax, how about completing monkey patch consolidation at the very start of L? | 16:06 |
| *** thedodd has quit IRC | 16:07 | |
| *** jorgem has joined #openstack-neutron | 16:07 | |
| amotoki | eventlet monkey-patch on windows can be split out to a separate patch/bug too. | 16:07 |
| armax | atuvenie: extract the changes that affect the ovs agent or are monkey patch specific to a differnt patch | 16:07 |
| armax | atuvenie: and then rebase https://review.openstack.org/#/c/160596/ on top of it | 16:07 |
| armax | ihrachyshka: what do you mean for consolidation? | 16:07 |
| *** Bsony has quit IRC | 16:08 | |
| ihrachyshka | armax, getting rid of monkey_patch() calls from any code outside of neutron/cmd/eventlet/... | 16:09 |
| *** mlavalle has joined #openstack-neutron | 16:09 | |
| ihrachyshka | armax, I envision that our libraries are importable without monkey patching | 16:09 |
| ZZelle | jschwarz, not sure to understand your -1 on https://review.openstack.org/143412? | 16:09 |
| *** chandankumar has joined #openstack-neutron | 16:10 | |
| armax | ihrachyshka: um, is it left in quite a few places? | 16:10 |
| ihrachyshka | armax, all plugin agents are filled in with it | 16:11 |
| ihrachyshka | armax, still 20 entries | 16:12 |
| ihrachyshka | ah, not really, less | 16:12 |
| armax | ihrachyshka: right, but that’s the plugin’s maintainer’s responsibility | 16:12 |
| ihrachyshka | 9 | 16:12 |
| armax | ihrachyshka: I always hated the blanket patches | 16:12 |
| ihrachyshka | armax, while it's true, it's like a plague, if any module monkey patches anything, then the whole test suite is cursed with it | 16:13 |
| ihrachyshka | armax, I can split it if it's better, though I don't follow how it would help. | 16:13 |
| armax | but I thought the test suite in monkey patched as early at it can, no? | 16:13 |
| armax | ihrachyshka: it’ll help in the sense that you’re gonna get more time to do something more useful? | 16:13 |
| ihrachyshka | armax, well, for now yes, but if we want to eventually see agents not patched, then we need to make sure that we are able to run tests without it. | 16:14 |
| ihrachyshka | that would probably mean adding an envvar that would control monkey patching for the test suite | 16:14 |
| armax | ihrachyshka: but the agent would need to run patched nonetheless, no? | 16:15 |
| ihrachyshka | armax, well, we discussed that we want to start moving from eventlet, no? | 16:15 |
| ihrachyshka | armax, actually no, agents should be able to run with threads | 16:15 |
| armax | moving from eventlet to where? | 16:15 |
| ihrachyshka | threads | 16:15 |
| armax | if we had this conversation, I wasn’t part of it! :) | 16:15 |
| ihrachyshka | you're probably right, it was probably markmcclain | 16:16 |
| armax | ihrachyshka: of if I was, I didn’t think that was a serious proposal | 16:16 |
| *** tflynn has joined #openstack-neutron | 16:16 | |
| armax | ihrachyshka: or that the time for discussing such a tectonic shift was appropriate | 16:16 |
| ihrachyshka | armax, it's not short term | 16:17 |
| jschwarz | ZZelle, one was on a comment in patchset 22 | 16:18 |
| obondarev | carl_baldwin: ping | 16:19 |
| *** yamahata has quit IRC | 16:19 | |
| jschwarz | ZZelle, (and also on that "assert increment_ip_cidr" thing) | 16:19 |
| *** yamahata has joined #openstack-neutron | 16:19 | |
| *** coolsvap has quit IRC | 16:19 | |
| carl_baldwin | obondarev: pong | 16:20 |
| obondarev | carl_baldwin: can you please check my inline comments on https://review.openstack.org/#/c/165539 ? | 16:20 |
| *** chandankumar has quit IRC | 16:20 | |
| carl_baldwin | obondarev: looking... | 16:20 |
| ZZelle | jschwarz, i responded to the 1st, and 2nd is solved by https://review.openstack.org/#/c/165866/4/neutron/tests/common/net_helpers.py | 16:20 |
| ZZelle | jschwarz, but it's the wrong change :( | 16:21 |
| jschwarz | ZZelle, haha! I missed that | 16:21 |
| amotoki | atuvenie: have you started to splitting your ovs-agent on hyper-v patch into pieces? | 16:22 |
| jschwarz | ZZelle, either way see https://review.openstack.org/#/c/143412/22/neutron/tests/common/helpers.py , which I don't see addressed yet | 16:22 |
| amotoki | atuvenie: I can help you. | 16:24 |
| atuvenie | amotoki: yes, I'm looking over it now | 16:24 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseLinuxTestCase methods in helpers https://review.openstack.org/159852 | 16:24 |
| *** localloop127 has joined #openstack-neutron | 16:25 | |
| *** Marga_ has quit IRC | 16:25 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers https://review.openstack.org/165866 | 16:25 |
| amotoki | atuvenie: I think your patch now covers all we need, so splitting it out helps reviewers to highlight individual changes. | 16:25 |
| obondarev | carl_baldwin: we can probably remove from the queue for now | 16:26 |
| atuvenie | amotoki: I would apreciate some help. From what I understand, I should move the ovs agent parts, meaning the part where I added ovs-neutron-agent to /cmd/eventlet/plugins and the monkey_patching | 16:26 |
| atuvenie | right? | 16:26 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 16:27 |
| *** xgerman_ has joined #openstack-neutron | 16:27 | |
| *** localloo1 has quit IRC | 16:27 | |
| ZZelle | jschwarz, sorry which comment? | 16:27 |
| jschwarz | ZZelle, the one before the absolute last one in that link | 16:27 |
| carl_baldwin | obondarev: You can make the call. | 16:27 |
| amotoki | atuvenie: we can split it into several pieces: (a) monkey-patch change, (b) agent/common/ovs_lib (with compat layer), (c) changes in other plugins | 16:28 |
| amotoki | atuvenie: (b) can be split into more pieces (ovs_lib, polling, utils). If it is easier to understand, it would be nice. | 16:28 |
| *** armax has quit IRC | 16:29 | |
| *** absubram has joined #openstack-neutron | 16:29 | |
| sudipto | carl_baldwin, obondarev: Please let me know what I should do now with the patch and I will do it. | 16:30 |
| atuvenie | amotoki: ok, I'll start doing the changes and put it up for review. | 16:30 |
| obondarev | sudipto: do you agree with my comment on raising only on agent startup? | 16:30 |
| *** armax has joined #openstack-neutron | 16:31 | |
| *** dboik_ has joined #openstack-neutron | 16:31 | |
| *** baoli has quit IRC | 16:31 | |
| ZZelle | jschwarz, responded | 16:32 |
| amotoki | atuvenie: thanks for understanding. there are several efforts on ovs-agent in parallel, and reviewers are scared with big patches, so I believe splitting into pieces makes everyone happy. | 16:32 |
| salv-orlando | pavel_bondar: hello... what timezone are you in this week? | 16:32 |
| *** baoli has joined #openstack-neutron | 16:32 | |
| atuvenie | amotoki: of course, it makes sense. | 16:33 |
| sudipto | obondarev: Is the start_flag supposed to be within: kwargs['agent_state']['agent_state'] ? | 16:34 |
| amuller | sudipto: obondarev: I don't think the start_flag is serialized? | 16:34 |
| amuller | it's used only in the agent right now | 16:34 |
| *** YorikSar has joined #openstack-neutron | 16:34 | |
| sudipto | amuller, yeah - i couldn't find it in that dict i pasted, and hence asked. | 16:34 |
| jschwarz | sudipto, congratz on having your patch approved :) | 16:35 |
| *** dboik has quit IRC | 16:35 | |
| *** tsg_ has joined #openstack-neutron | 16:35 | |
| *** marun has joined #openstack-neutron | 16:35 | |
| obondarev | amuller: sudipto: start flag is should be there on the first agent report, on startup | 16:35 |
| *** chandankumar has joined #openstack-neutron | 16:36 | |
| obondarev | amuller: sudipto: please check https://github.com/openstack/neutron/blob/master/neutron/agent/dhcp/agent.py#L540 | 16:36 |
| *** sbfox has joined #openstack-neutron | 16:36 | |
| sudipto | obondarev: Also, when you say, the neutron server is down - do you mean the AMQP server is actually down? | 16:36 |
| *** chandankumar has quit IRC | 16:36 | |
| *** chandankumar has joined #openstack-neutron | 16:37 | |
| obondarev | sudipto: no I meant neutron server | 16:38 |
| obondarev | sudipto: like in upgrade scenario | 16:38 |
| jschwarz | ZZelle, ask and you shall receive | 16:38 |
| jschwarz | marun, hello :) | 16:38 |
| obondarev | sudipto: so after start server will recieve outdated reports from agents | 16:39 |
| obondarev | sudipto: which are not errors | 16:39 |
| amuller | obondarev: sudipto: Looks like OVS, DHCP, metadata and L3 agents send the 'start_flag', even still, maybe other agents don't serialize it, I'd make the check for it on the server optional | 16:40 |
| sudipto | obondarev: Hmm, but this error would be thrown only when you have the time offset right? So why do you say these are not errors once the server comes up? The miss match would still persist right? | 16:40 |
| *** yamahata has quit IRC | 16:41 | |
| obondarev | amuller: not exactly | 16:41 |
| *** yamahata has joined #openstack-neutron | 16:41 | |
| obondarev | amuller: server will recieve reports which were queued while server was down, so there is no time offset between server and agents | 16:41 |
| *** ajo has quit IRC | 16:42 | |
| obondarev | amuller: what other agents do you mean? | 16:42 |
| sudipto | obondarev: Got your point now :) They would have a outdated time... | 16:42 |
| amuller | obondarev: I see the current check for start_flag on the server is conditional, so my point is moot anyway | 16:42 |
| amuller | but I was thinking about out of tree agents | 16:43 |
| obondarev | amuller: do we have server side check for start_flag? | 16:43 |
| roaet | HI neutron folks. Anyone know off the top of their head how to configure neutron so that 'oslo_messaging' doesn't error all the time. related: Deprecated: fake_rabbit option is deprecated, set rpc_backend to kombu+memory or use the fake driver instead. | 16:43 |
| amuller | obondarev: it's in _create_or_update_agent | 16:44 |
| roaet | I don't have it (messaging) and just want to make it go away. | 16:44 |
| amrith | hi ... i'm having trouble getting neutron setup right. environment is devstack, single node, ubuntu 14.04. I'm setup and running. In my setup I need to be able to ssh from the nova guest to the host and rsync some code. The private ip for host is 10.0.0.1. I can't ssh to it. ssh is up and running (I can ssh to public interface, localhost etc) just not to private ip. is there something specific about neutron that makes | 16:44 |
| amrith | this not work? ifconfig doens't show an interface for 10.0.0.1 so maybe ssh isn't binding it? | 16:44 |
| obondarev | amuller: yeah, I see | 16:44 |
| openstackgerrit | Merged openstack/neutron: Start metadata agent without trying to connect db https://review.openstack.org/159365 | 16:45 |
| *** reed has quit IRC | 16:45 | |
| *** nati_ueno has joined #openstack-neutron | 16:45 | |
| *** reed has joined #openstack-neutron | 16:46 | |
| *** yatin has joined #openstack-neutron | 16:46 | |
| *** Marga_ has joined #openstack-neutron | 16:47 | |
| obondarev | amuller: sudipto: I think we can rely each agent should set 'start_flag' on startup | 16:47 |
| roaet | nevermind.. rpc_backend = fake | 16:47 |
| roaet | thanks! | 16:47 |
| *** emagana has quit IRC | 16:49 | |
| *** jorgem has quit IRC | 16:50 | |
| *** irenab has joined #openstack-neutron | 16:50 | |
| *** rotbeard has quit IRC | 16:51 | |
| *** ihrachyshka has quit IRC | 16:51 | |
| sudipto | obondarev,amuller: So the net is, that we need to have a check like: if kwargs['agent_state']['agent_state'].get('start_flag') before we throw the error right? | 16:51 |
| jschwarz | ZZelle, I hope you'll agree with me though that it's confusing | 16:52 |
| obondarev | sudipto: that's my point | 16:52 |
| jschwarz | ZZelle, in a non-trivial way | 16:52 |
| sudipto | obondarev: Alright that makes perfect sense. Let me update the patch! | 16:52 |
| obondarev | sudipto: thanks! | 16:53 |
| ZZelle | jschwarz, i would say the opposite as i am used that first ip in network ip | 16:53 |
| *** yamahata has quit IRC | 16:54 | |
| *** yamahata has joined #openstack-neutron | 16:54 | |
| sudipto | jschwarz: Yeah almost there :), need another patch and would need another review from you :) | 16:55 |
| jschwarz | ZZelle, why are you the first? can't there be an existing port prior to your FakeMachine | 16:55 |
| jschwarz | sudipto, ack | 16:55 |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 16:55 |
| *** emagana has joined #openstack-neutron | 16:57 | |
| ZZelle | jschwarz, nope but first ip in a network is network ip X.Y.Z.0/24 => X.Y.Z.0 which cannot be used | 16:57 |
| *** jamie_h has quit IRC | 16:58 | |
| jschwarz | ZZelle, and if there's a pre-existing router there, 'X.Y.Z.0' + 1 == 'X.Y.Z.1' can't also be used | 16:58 |
| *** annegentle has quit IRC | 16:59 | |
| *** tflynn has quit IRC | 16:59 | |
| *** emagana has quit IRC | 16:59 | |
| *** matrohon has quit IRC | 17:00 | |
| *** emagana has joined #openstack-neutron | 17:01 | |
| *** sripriya has joined #openstack-neutron | 17:02 | |
| *** scheuran has quit IRC | 17:02 | |
| *** absubram has quit IRC | 17:02 | |
| *** aranjan has joined #openstack-neutron | 17:03 | |
| *** yamahata has quit IRC | 17:04 | |
| *** jlanoux has quit IRC | 17:04 | |
| *** zhhuabj has quit IRC | 17:06 | |
| ZZelle | jschwarz, i think there is no good solution, so you should choose one | 17:06 |
| jschwarz | ZZelle, dropping the '+1' and having the consumer to explicitly choose the first IP to be used seems very reasonable | 17:06 |
| jschwarz | ZZelle, don't you think? | 17:06 |
| *** dan has quit IRC | 17:07 | |
| *** banix has joined #openstack-neutron | 17:07 | |
| *** aepifanov has joined #openstack-neutron | 17:08 | |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 17:09 |
| *** annegentle has joined #openstack-neutron | 17:10 | |
| sudipto | jschwarz, obondarev, amuller, carl_baldwin: Mind taking a look at https://review.openstack.org/165539 once more pls? | 17:12 |
| jschwarz | ack | 17:12 |
| *** ygbo has quit IRC | 17:12 | |
| *** dan has joined #openstack-neutron | 17:12 | |
| obondarev | sudipto: sure | 17:12 |
| *** evgenyf has quit IRC | 17:12 | |
| sudipto | Thank you. | 17:13 |
| amuller | sudipto: There's some whitespace issue in https://review.openstack.org/#/c/165539/6..7/neutron/tests/unit/openvswitch/test_agent_scheduler.py | 17:13 |
| sudipto | amuller: darn. | 17:14 |
| sudipto | Let me recheck | 17:14 |
| sudipto | amuller: which line #? | 17:15 |
| *** ronis has quit IRC | 17:16 | |
| sudipto | amuller: Ok #193? | 17:16 |
| *** ijw has quit IRC | 17:17 | |
| openstackgerrit | enikanorov proposed openstack/neutron: Fix handling of before/after notifications in linuxbridge agent https://review.openstack.org/166931 | 17:17 |
| sudipto | amuller: figured it...sorting it out. | 17:17 |
| *** zhhuabj has joined #openstack-neutron | 17:18 | |
| *** sbalukoff has quit IRC | 17:18 | |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 17:18 |
| *** lykinsbd has quit IRC | 17:19 | |
| *** rotbeard has joined #openstack-neutron | 17:19 | |
| *** amotoki has quit IRC | 17:19 | |
| *** lykinsbd has joined #openstack-neutron | 17:19 | |
| sudipto | amuller: fixed. | 17:19 |
| sudipto | Think it's too late to be up :) | 17:20 |
| *** absubram has joined #openstack-neutron | 17:21 | |
| obondarev | sudipto: one more thing, please check inline | 17:21 |
| sudipto | obondarev: Just checked... | 17:21 |
| *** melwitt has joined #openstack-neutron | 17:22 | |
| *** xgerman_ has quit IRC | 17:22 | |
| *** Marga_ has quit IRC | 17:22 | |
| *** lykinsbd has quit IRC | 17:24 | |
| *** dannywilson has joined #openstack-neutron | 17:24 | |
| *** dannywilson has quit IRC | 17:24 | |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 17:24 |
| *** _cjones_ has quit IRC | 17:24 | |
| *** dannywilson has joined #openstack-neutron | 17:25 | |
| *** _cjones_ has joined #openstack-neutron | 17:25 | |
| sudipto | obondarev: Hopefully done this time :) | 17:25 |
| openstackgerrit | John Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 17:25 |
| obondarev | sudipto: cool, thanks! | 17:26 |
| obondarev | sudipto: looking | 17:26 |
| *** xgerman_ has joined #openstack-neutron | 17:27 | |
| *** yeungp has quit IRC | 17:27 | |
| *** harlowja has joined #openstack-neutron | 17:27 | |
| *** achanda has joined #openstack-neutron | 17:29 | |
| *** _cjones_ has quit IRC | 17:29 | |
| *** carl_baldwin has quit IRC | 17:30 | |
| sudipto | obondarev, amuller, jschwarz: Thx for the quick review. | 17:30 |
| jschwarz | :) | 17:30 |
| *** blues-man has quit IRC | 17:31 | |
| amuller | marun: looks like there's a new non deterministic functional test failure: http://logs.openstack.org/88/166888/1/check/check-neutron-dsvm-functional/f3d4100/testr_results.html.gz | 17:31 |
| *** _cjones_ has joined #openstack-neutron | 17:32 | |
| *** sbfox has quit IRC | 17:33 | |
| *** anshul has quit IRC | 17:35 | |
| *** aranjan has quit IRC | 17:36 | |
| *** sbfox has joined #openstack-neutron | 17:36 | |
| *** amotoki has joined #openstack-neutron | 17:36 | |
| *** amotoki has quit IRC | 17:36 | |
| *** annegentle has quit IRC | 17:37 | |
| *** carl_baldwin has joined #openstack-neutron | 17:38 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron-vpnaas: Scenario test for vpnaas: ipsec-site-connection https://review.openstack.org/159746 | 17:40 |
| *** chandankumar has quit IRC | 17:40 | |
| *** rotbeard has quit IRC | 17:41 | |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 17:43 |
| *** sudipto has quit IRC | 17:43 | |
| *** sthillma has joined #openstack-neutron | 17:43 | |
| *** sthillma has quit IRC | 17:43 | |
| *** kerpan has quit IRC | 17:43 | |
| *** carl_baldwin has quit IRC | 17:44 | |
| *** ivar-lazzaro has joined #openstack-neutron | 17:45 | |
| *** ivar-lazzaro has quit IRC | 17:45 | |
| *** rook_ has quit IRC | 17:45 | |
| *** jschwarz has quit IRC | 17:45 | |
| *** padkrish has joined #openstack-neutron | 17:46 | |
| *** aranjan has joined #openstack-neutron | 17:47 | |
| *** aranjan has quit IRC | 17:47 | |
| *** aranjan has joined #openstack-neutron | 17:48 | |
| *** abehl has joined #openstack-neutron | 17:48 | |
| *** ivar-lazzaro has joined #openstack-neutron | 17:49 | |
| *** yamahata has joined #openstack-neutron | 17:49 | |
| *** leenheer has joined #openstack-neutron | 17:51 | |
| *** dannywilson has quit IRC | 17:54 | |
| *** lykinsbd has joined #openstack-neutron | 17:54 | |
| *** armax has quit IRC | 17:54 | |
| *** jistr has quit IRC | 17:54 | |
| *** yatin has quit IRC | 17:56 | |
| *** achanda has quit IRC | 17:57 | |
| *** setuid has quit IRC | 17:58 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Use first() instead of one() in tunnel endpoint query https://review.openstack.org/166947 | 17:59 |
| *** setuid has joined #openstack-neutron | 18:00 | |
| *** sridhar_ram has joined #openstack-neutron | 18:01 | |
| *** tflynn has joined #openstack-neutron | 18:02 | |
| *** igordcard has quit IRC | 18:02 | |
| *** nati_ueno has quit IRC | 18:02 | |
| *** tflynn has quit IRC | 18:02 | |
| *** jpena has quit IRC | 18:02 | |
| *** nati_ueno has joined #openstack-neutron | 18:02 | |
| *** tflynn has joined #openstack-neutron | 18:03 | |
| *** armax has joined #openstack-neutron | 18:04 | |
| *** nyechiel__ has quit IRC | 18:05 | |
| *** dane_leblanc has quit IRC | 18:05 | |
| *** marun has quit IRC | 18:06 | |
| *** ChuckC has quit IRC | 18:08 | |
| openstackgerrit | Nurit Vilosny proposed openstack/neutron: Add notification flag (True / False) to extension_manager process_update_port https://review.openstack.org/162648 | 18:08 |
| *** dan has quit IRC | 18:08 | |
| *** ronis has joined #openstack-neutron | 18:09 | |
| *** bishtsanddev has joined #openstack-neutron | 18:09 | |
| *** emagana has quit IRC | 18:11 | |
| *** jorgem has joined #openstack-neutron | 18:12 | |
| *** kickinz1 is now known as kickinz1|afk | 18:12 | |
| *** dane_leblanc has joined #openstack-neutron | 18:12 | |
| *** carl_baldwin has joined #openstack-neutron | 18:13 | |
| *** emagana has joined #openstack-neutron | 18:13 | |
| *** Marga_ has joined #openstack-neutron | 18:14 | |
| openstackgerrit | veena proposed openstack/python-neutronclient: Replace underscore with hyphen in CLI options https://review.openstack.org/166949 | 18:14 |
| *** dank_ has joined #openstack-neutron | 18:14 | |
| *** igordcard has joined #openstack-neutron | 18:14 | |
| *** achanda has joined #openstack-neutron | 18:15 | |
| *** dannywilson has joined #openstack-neutron | 18:15 | |
| *** abehl has quit IRC | 18:16 | |
| *** enikanorov__ has joined #openstack-neutron | 18:16 | |
| *** marun has joined #openstack-neutron | 18:17 | |
| *** emagana has quit IRC | 18:18 | |
| *** gsagie has joined #openstack-neutron | 18:18 | |
| *** harlowja has quit IRC | 18:18 | |
| *** enikanorov has quit IRC | 18:19 | |
| *** achanda has quit IRC | 18:19 | |
| *** achanda has joined #openstack-neutron | 18:20 | |
| john-davidge | carl_baldwin: Hi o/ | 18:20 |
| carl_baldwin | john-davidge, hi | 18:20 |
| carl_baldwin | john-davidge, So, we need to be able to set a PD subnet pool and set it as the default. Right? | 18:21 |
| *** harlowja has joined #openstack-neutron | 18:21 | |
| john-davidge | carl_baldwin: Ideally, yes. Is the intention still to do that through a config option? | 18:21 |
| *** safchain_ has quit IRC | 18:21 | |
| *** lykinsbd has quit IRC | 18:22 | |
| john-davidge | carl_baldwin: My thinking is if it's set to "prefix_delegation" or a similar constant, then we can set our ipv6_pd_enabled attribute and continue the PD workflow without many chnages to the current implementation | 18:22 |
| carl_baldwin | john-davidge, I think a config option is best for now. What do you think? | 18:23 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 18:24 |
| john-davidge | carl_baldwin: Yes, that would definitely work for us | 18:24 |
| carl_baldwin | john-davidge, I think that works. | 18:24 |
| john-davidge | carl_baldwin: excellent | 18:24 |
| *** atuvenie has quit IRC | 18:25 | |
| carl_baldwin | john-davidge, I tend to think that the config option should be proposed in its own patch. Do you want to propose that patch against the ipam blueprint or do you want me to do it? | 18:25 |
| *** leenheer has quit IRC | 18:26 | |
| john-davidge | carl_baldwin: would it fit better under the subnet-allocation bp? | 18:27 |
| carl_baldwin | john-davidge, since it was proposed first there, I think so. It has an FFE. | 18:27 |
| *** achanda has quit IRC | 18:27 | |
| *** Marga_ has quit IRC | 18:27 | |
| *** achanda has joined #openstack-neutron | 18:28 | |
| *** emagana has joined #openstack-neutron | 18:28 | |
| *** Bhavya has quit IRC | 18:29 | |
| *** thedodd has joined #openstack-neutron | 18:29 | |
| *** pritesh has joined #openstack-neutron | 18:30 | |
| john-davidge | carl_baldwin: Great, I'll take a look at doing that before the end of today | 18:30 |
| *** sbalukoff has joined #openstack-neutron | 18:31 | |
| carl_baldwin | john-davidge, great. I think we should be able to turn it out pretty quickly. Let me check where Ryan's patches stand at the moment. | 18:31 |
| *** nuritv has quit IRC | 18:31 | |
| carl_baldwin | john-davidge, I think we should put up a simple patch that adds the config option. Then make yours and https://review.openstack.org/#/c/157597/ dependent on it. | 18:32 |
| carl_baldwin | john-davidge, So, if you could get that up today, we can move around it. | 18:32 |
| *** evgenyf has joined #openstack-neutron | 18:33 | |
| john-davidge | carl_baldwin: Sounds good to me. Will ping you when the patch is up, shouldn't take me long | 18:33 |
| *** nati_ueno has quit IRC | 18:33 | |
| carl_baldwin | john-davidge, great. | 18:34 |
| *** dannywilson has quit IRC | 18:35 | |
| *** nati_ueno has joined #openstack-neutron | 18:35 | |
| *** padkrish has quit IRC | 18:35 | |
| *** dannywilson has joined #openstack-neutron | 18:35 | |
| *** padkrish has joined #openstack-neutron | 18:36 | |
| amuller | armax: carl_baldwin: Quick question about https://review.openstack.org/#/c/126189/20/neutron/db/l3_agentschedulers_db.py | 18:37 |
| *** padkrish_ has joined #openstack-neutron | 18:37 | |
| armax | amuller: shoot | 18:37 |
| *** ParsectiX has joined #openstack-neutron | 18:37 | |
| *** dave-mccowan has quit IRC | 18:37 | |
| armax | amuller: question for a question? | 18:37 |
| *** leenheer has joined #openstack-neutron | 18:37 | |
| amuller | assuming the plugin supports the HA extension, should it return the ha_state column for both HA routers and non-HA routers? I'm leaning towards yes | 18:37 |
| amuller | for HA routers it would be in ('active', 'standby') and for non-HA routers it would return None for that column | 18:38 |
| amuller | like how it is in the patch now | 18:38 |
| armax | amuller: probably | 18:38 |
| carl_baldwin | amuller, looking | 18:39 |
| *** ParsectiX has quit IRC | 18:39 | |
| *** ParsectiX has joined #openstack-neutron | 18:40 | |
| amuller | armax: no matter where I put that code, I still need to check somewhere if the router is HA or not | 18:40 |
| *** ParsectiX has quit IRC | 18:40 | |
| *** padkrish has quit IRC | 18:40 | |
| *** dboik_ has quit IRC | 18:40 | |
| amuller | armax: As we don't have a router class representing a router in the server code | 18:40 |
| armax | amuller: where you put the code can make quite a bit of difference though | 18:40 |
| carl_baldwin | amuller is right. I've thought about that code a little. It would take lots more refactoring to get rid of all that if/then/else junk. | 18:41 |
| carl_baldwin | armax may also have a point. I don't have an opinion yet for where this code should live. | 18:41 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 18:41 |
| amuller | armax: carl_baldwin: I changed some things around in that revision | 18:41 |
| amuller | please check it out | 18:41 |
| armax | amuller, carl_baldwin it looked like that code could live in the ha scheduler mixin | 18:42 |
| armax | but I was waiting for amuller to confirm | 18:42 |
| amuller | armax: that's what I tried out in the latest revision | 18:42 |
| *** nyechiel__ has joined #openstack-neutron | 18:42 | |
| armax | amuller: cool, lloookkkiiinnng.. | 18:42 |
| amuller | armax: err, that revision fails some tests in the HA DB test case, need to change some initialization code there | 18:44 |
| *** Swami has joined #openstack-neutron | 18:44 | |
| armax | amuller: nooooooooo | 18:45 |
| *** smillward has joined #openstack-neutron | 18:45 | |
| amuller | heh | 18:45 |
| *** leenheer has quit IRC | 18:46 | |
| *** leenheer has joined #openstack-neutron | 18:47 | |
| *** nyechiel has joined #openstack-neutron | 18:47 | |
| *** nyechiel__ has quit IRC | 18:47 | |
| *** davrodpin has joined #openstack-neutron | 18:47 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 18:48 |
| amuller | armax: that will pass unit tests | 18:49 |
| *** ronis has quit IRC | 18:49 | |
| *** rpothier has joined #openstack-neutron | 18:50 | |
| *** Marga_ has joined #openstack-neutron | 18:50 | |
| armax | amuller: too late, I lost interest in this :P | 18:51 |
| amuller | :( | 18:51 |
| *** nlahouti has joined #openstack-neutron | 18:51 | |
| * armax kids, I’ll look at it in a few | 18:51 | |
| *** carl_baldwin has quit IRC | 18:51 | |
| *** dboik has joined #openstack-neutron | 18:52 | |
| openstackgerrit | Kyle Mestery proposed openstack/neutron: Move README.odl into opendaylight directory https://review.openstack.org/166863 | 18:52 |
| *** nlahouti has quit IRC | 18:54 | |
| *** nlahouti has joined #openstack-neutron | 18:55 | |
| *** leenheer has quit IRC | 18:55 | |
| *** fcorral_hp has joined #openstack-neutron | 18:58 | |
| *** nyechiel has quit IRC | 18:59 | |
| *** nyechiel has joined #openstack-neutron | 19:00 | |
| *** padkrish_ has quit IRC | 19:01 | |
| *** swat30 has joined #openstack-neutron | 19:01 | |
| *** padkrish has joined #openstack-neutron | 19:01 | |
| *** raginbajin has quit IRC | 19:04 | |
| *** SamYaple has quit IRC | 19:04 | |
| *** pmath has quit IRC | 19:04 | |
| *** frickler has quit IRC | 19:04 | |
| *** swat30 has quit IRC | 19:04 | |
| *** SpamapS has quit IRC | 19:04 | |
| *** SamYaple has joined #openstack-neutron | 19:04 | |
| *** SpamapS has joined #openstack-neutron | 19:04 | |
| *** SpamapS has quit IRC | 19:04 | |
| *** SpamapS has joined #openstack-neutron | 19:04 | |
| *** raginbajin has joined #openstack-neutron | 19:04 | |
| *** sgran has quit IRC | 19:04 | |
| *** otherwiseguy has quit IRC | 19:04 | |
| *** mikal has quit IRC | 19:04 | |
| *** heyongli has quit IRC | 19:04 | |
| *** cgoncalves has quit IRC | 19:04 | |
| *** frickler has joined #openstack-neutron | 19:04 | |
| *** pmath has joined #openstack-neutron | 19:04 | |
| *** mikal has joined #openstack-neutron | 19:04 | |
| *** otherwiseguy has joined #openstack-neutron | 19:04 | |
| *** swat30 has joined #openstack-neutron | 19:04 | |
| *** cgoncalves has joined #openstack-neutron | 19:04 | |
| *** sgran has joined #openstack-neutron | 19:04 | |
| *** baoli has quit IRC | 19:05 | |
| *** moshele has joined #openstack-neutron | 19:06 | |
| *** padkrish has quit IRC | 19:06 | |
| openstackgerrit | John Davidge proposed openstack/neutron: Add the default_ipv6_subnet_pool config option https://review.openstack.org/166973 | 19:07 |
| *** _cjones_ has quit IRC | 19:07 | |
| john-davidge | carl_baldwin: Patch up https://review.openstack.org/#/c/166973/ | 19:08 |
| *** Marga_ has quit IRC | 19:08 | |
| *** Marga_ has joined #openstack-neutron | 19:09 | |
| *** marrusl has quit IRC | 19:11 | |
| *** ZZelle_ has joined #openstack-neutron | 19:12 | |
| *** marrusl has joined #openstack-neutron | 19:13 | |
| *** duarte has joined #openstack-neutron | 19:15 | |
| *** armax has quit IRC | 19:15 | |
| *** yamamoto_ has joined #openstack-neutron | 19:16 | |
| *** gsagie has quit IRC | 19:17 | |
| *** evgenyf has quit IRC | 19:17 | |
| *** ronis has joined #openstack-neutron | 19:18 | |
| *** lilynn has joined #openstack-neutron | 19:20 | |
| *** evgenyf has joined #openstack-neutron | 19:21 | |
| *** yamamoto_ has quit IRC | 19:21 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 19:21 |
| *** Marga_ has quit IRC | 19:26 | |
| openstackgerrit | John Davidge proposed openstack/neutron: Add the default_ipv6_subnet_pool config option https://review.openstack.org/166973 | 19:28 |
| *** carl_baldwin has joined #openstack-neutron | 19:29 | |
| *** _cjones_ has joined #openstack-neutron | 19:30 | |
| *** dboik has quit IRC | 19:31 | |
| *** dboik has joined #openstack-neutron | 19:32 | |
| *** sripriya has quit IRC | 19:32 | |
| *** prad has quit IRC | 19:32 | |
| *** sripriya has joined #openstack-neutron | 19:32 | |
| *** nyechiel has quit IRC | 19:35 | |
| *** nyechiel has joined #openstack-neutron | 19:36 | |
| john-davidge | carl_baldwin: Hey Carl, pushed that patch while you were away https://review.openstack.org/#/c/166973/ | 19:36 |
| carl_baldwin | john-davidge, already looking at it. | 19:36 |
| carl_baldwin | thanks. | 19:36 |
| john-davidge | carl_baldwin: Great, ping me if I've done anything dumb | 19:37 |
| *** armax has joined #openstack-neutron | 19:38 | |
| *** alexpilotti has quit IRC | 19:39 | |
| openstackgerrit | Merged openstack/neutron: Remove unneeded DVRAgentRpcApiMixin from OVSDVRNeutronAgent https://review.openstack.org/164212 | 19:42 |
| *** ajo has joined #openstack-neutron | 19:43 | |
| *** marun has quit IRC | 19:43 | |
| *** padkrish has joined #openstack-neutron | 19:44 | |
| *** evgenyf has quit IRC | 19:44 | |
| *** yfried has joined #openstack-neutron | 19:45 | |
| *** heyongli has joined #openstack-neutron | 19:46 | |
| *** evgenyf has joined #openstack-neutron | 19:47 | |
| carl_baldwin | john-davidge, I've just made a couple of comments. Nothing dumb. It is on the right track. | 19:48 |
| john-davidge | carl_baldwin: Thanks, will take a look | 19:48 |
| carl_baldwin | john-davidge, One more in a second post. | 19:49 |
| carl_baldwin | john-davidge, thanks. | 19:49 |
| *** sripriya has quit IRC | 19:52 | |
| *** sripriya has joined #openstack-neutron | 19:52 | |
| *** marun has joined #openstack-neutron | 19:54 | |
| *** chandankumar has joined #openstack-neutron | 19:54 | |
| *** jorgem has quit IRC | 19:54 | |
| *** dave-mccowan has joined #openstack-neutron | 19:55 | |
| *** jorgem has joined #openstack-neutron | 19:58 | |
| *** yfauser has joined #openstack-neutron | 19:58 | |
| *** yfauser has left #openstack-neutron | 19:58 | |
| salv-orlando | pavel_bondar: around? | 20:01 |
| *** evgenyf has quit IRC | 20:01 | |
| john-davidge | carl_baldwin: To your comments about UUID vs human-readable name, the way I see this working for PD is that we won't actually be creating subnet-pools for our purposes. If the default is set such that PD should be used we defer to the PD mechanism rather than using a subnet pool | 20:03 |
| *** matrohon has joined #openstack-neutron | 20:03 | |
| john-davidge | carl_baldwin: The reason for this being that it isn't possible to configure a PD-server by API, so this has to be done manually by the operator | 20:03 |
| *** jgrimm is now known as zz_jgrimm | 20:05 | |
| armax | amuller: ping | 20:05 |
| carl_baldwin | john-davidge, my motivation is not to configure PD through the API like another subnet pool. The motivation is to recognize a PD domain as a kind of subnet pool so that it can be included in address scope work that I'm planning for Kilo. | 20:05 |
| salv-orlando | carl_baldwin: do you mean livberty? | 20:06 |
| carl_baldwin | john-davidge, It may even be viewable through the API as a read-only subnet pool. | 20:06 |
| carl_baldwin | salv-orlando, Yes, I do mean Liberty. I think I was even thinking Liberty when my hands typed Kilo. | 20:06 |
| salv-orlando | carl_baldwin: I love being pedant ;) | 20:07 |
| *** tsg_ has quit IRC | 20:07 | |
| *** Marga_ has joined #openstack-neutron | 20:07 | |
| *** tsg has joined #openstack-neutron | 20:08 | |
| carl_baldwin | salv-orlando: Your correction is welcome. | 20:08 |
| john-davidge | carl_baldwin: Ah I see. Multiple PD domains are out of scope for the current PD blueprint, and I'm not sure if they would be practical going forward | 20:08 |
| *** Swami has quit IRC | 20:08 | |
| john-davidge | carl_baldwin: The intention is for PD to be used in cases where it's desired that all subnets are allocated from a single global pool | 20:09 |
| carl_baldwin | john-davidge, are you restricting PD to a single external network? | 20:09 |
| john-davidge | carl_baldwin: No, perhaps I'm misunderstanding what you mean by PD domains | 20:10 |
| carl_baldwin | john-davidge, I'm not asking for support for multiple PD domains. Just wondering if we can avoid getting painted in to a corner. | 20:10 |
| *** annegentle has joined #openstack-neutron | 20:11 | |
| john-davidge | carl_baldwin: Sure, so are you suggesting that the admin would first create a subnet-pool for PD, before setting the UUID of that pool as the value for default_ipv6_subnet_pool? | 20:11 |
| *** fitoduarte has joined #openstack-neutron | 20:13 | |
| *** duarte has quit IRC | 20:13 | |
| carl_baldwin | john-davidge, Not exactly. I think all that is really needed is a UUID for it at this point. Maybe "prefix_delegation" fits the bill but it seems a bit funny to me lined up against all of the other real-looking UUIDs. | 20:13 |
| carl_baldwin | john-davidge, The way I see it, we're implementing a sort of limited way of having multiple drivers for subnet pools. One driver is what Ryan has developed. The other is PD. They work a bit differently but they have one thing in common: the subnet pool UUID. | 20:15 |
| carl_baldwin | "all the other real-looking UUIDs | 20:15 |
| carl_baldwin | ... which will come from the subnet pools API. | 20:15 |
| *** davrodpin has quit IRC | 20:17 | |
| *** fitoduarte has quit IRC | 20:17 | |
| *** duartea__ has joined #openstack-neutron | 20:17 | |
| john-davidge | carl_baldwin: Yeah I see where you're coming from. I guess the next question then would be how do we generate said UUID, and how do we ensure it's uniqueness? I'd assume the best way to do that would be to define it as a constant, but if we do that is there much benefit over using a readable name? | 20:17 |
| *** duartea__ has quit IRC | 20:17 | |
| *** flwang1 has joined #openstack-neutron | 20:17 | |
| carl_baldwin | john-davidge, I think we're on the same page now. Here's one: "de60fbf0-5133-4f1f-9a23-370225f1c356". So, the question remains, is this better than "prefix_delegation"? | 20:18 |
| *** davrodpin has joined #openstack-neutron | 20:18 | |
| carl_baldwin | john-davidge, So, if there will ever be more than one PD pool, what will they be named? | 20:19 |
| *** Ila has joined #openstack-neutron | 20:20 | |
| *** mlavalle has quit IRC | 20:22 | |
| *** mlavalle has joined #openstack-neutron | 20:22 | |
| *** ritesh has joined #openstack-neutron | 20:22 | |
| *** crc32 has joined #openstack-neutron | 20:22 | |
| *** crc32 has quit IRC | 20:23 | |
| john-davidge | carl_baldwin: Ok I see what you're getting at. Given that it will be manually entered into a config file my an admin I'd say that the readable version is preferable, but if we did eventually implement multiple PD pools then I agree that would be an issue. | 20:24 |
| *** davrodpi1 has joined #openstack-neutron | 20:25 | |
| carl_baldwin | john-davidge, I can go either way at this point. I don't have a strong opinion. If we support multiple in the future, we could just change the way it is done. They are just unique strings after all. | 20:25 |
| *** lilynn has quit IRC | 20:25 | |
| *** nyechiel has quit IRC | 20:26 | |
| *** lilynn has joined #openstack-neutron | 20:26 | |
| john-davidge | carl_baldwin: This is true. I don't see that happening anytime soon anyway, as it raises some interesting questions about whether it could be done without stretching beyond the standard PD mechanism | 20:26 |
| *** amrith is now known as _amrith_ | 20:27 | |
| *** banix has quit IRC | 20:27 | |
| *** kfox1111 has joined #openstack-neutron | 20:28 | |
| kfox1111 | can I get some love please: https://review.openstack.org/#/c/157167/ | 20:28 |
| *** davrodpi1 has quit IRC | 20:28 | |
| *** rangansw has joined #openstack-neutron | 20:28 | |
| kfox1111 | I feel like I'm here now: http://openstackreactions.enovance.com/2014/06/nitpicking-on-code-reviews/ | 20:28 |
| john-davidge | carl_baldwin: Perhaps adding a constant IPV6_PD_POOL_ID = "prefix_delegation" would be sensible to make checking for it easier | 20:35 |
| *** crc32 has joined #openstack-neutron | 20:36 | |
| absubram | hi all.. I had a question about the functional tests if someone could provide a little insight please? I’m trying this out for the first time.. | 20:36 |
| absubram | when I run a specific test.. in this case - tox -e dsvm-functional -v -- neutron.tests.functional.agent.test_l3_agent.L3AgentTestCase.test_legacy_router_lifecycle | 20:36 |
| *** nlahouti has quit IRC | 20:36 | |
| *** xgerman_ has quit IRC | 20:36 | |
| absubram | I see that it is run twice? as vsctl and as ‘native’? Anyone know why this might be the case? | 20:37 |
| *** Swami has joined #openstack-neutron | 20:38 | |
| *** eduardoj has joined #openstack-neutron | 20:38 | |
| *** bishtsanddev has quit IRC | 20:38 | |
| *** tsg has quit IRC | 20:38 | |
| *** davrodpin has quit IRC | 20:39 | |
| *** haleyb has joined #openstack-neutron | 20:39 | |
| *** fcorral_hp has quit IRC | 20:40 | |
| kevinbenton | carl_baldwin: ping! | 20:41 |
| carl_baldwin | kevinbenton, pong! | 20:42 |
| kevinbenton | change 165539 | 20:42 |
| kevinbenton | can you pull your +A off for a sec | 20:42 |
| openstackgerrit | John Davidge proposed openstack/neutron: Add the default_ipv6_subnet_pool config option https://review.openstack.org/166973 | 20:44 |
| kevinbenton | carl_baldwin: added my comment | 20:46 |
| *** angela-s has joined #openstack-neutron | 20:46 | |
| kevinbenton | carl_baldwin: let me know if i missed something that was already discussed | 20:46 |
| *** mwagner_lap has quit IRC | 20:47 | |
| *** jaypipes has quit IRC | 20:47 | |
| carl_baldwin | kevinbenton, looking. This is the second time today on this patch. If I remove my +A, I think I'll be too trigger shy to add it a third time. | 20:48 |
| *** nati_ueno has quit IRC | 20:48 | |
| *** nyechiel has joined #openstack-neutron | 20:48 | |
| *** Ila has quit IRC | 20:49 | |
| kevinbenton | carl_baldwin: well just check out my comment. i think this might be something that should not be fixed here anyway | 20:49 |
| kevinbenton | carl_baldwin: if i'm understanding it correctly | 20:49 |
| *** nati_ueno has joined #openstack-neutron | 20:49 | |
| kevinbenton | carl_baldwin: i hadn't seen this one until now | 20:49 |
| *** padkrish has quit IRC | 20:49 | |
| *** padkrish has joined #openstack-neutron | 20:50 | |
| *** tiswanso has quit IRC | 20:50 | |
| salv-orlando | kevinbenton: you need to push another patchset to remove the patch from the gate | 20:50 |
| * salv-orlando thniks | 20:50 | |
| kevinbenton | salv-orlando: if i click the rebase button, will that do it? | 20:50 |
| carl_baldwin | kevinbenton, salv-orlando It isn't in the gate. | 20:51 |
| kevinbenton | carl_baldwin: oh, jenkins failure | 20:51 |
| carl_baldwin | kevinbenton, right. | 20:51 |
| *** john-davidge has quit IRC | 20:51 | |
| kevinbenton | who was complaining that an unstable gate was bad ;) | 20:52 |
| openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 20:52 |
| *** xgerman_ has joined #openstack-neutron | 20:52 | |
| *** jaypipes has joined #openstack-neutron | 20:52 | |
| salv-orlando | kevinbenton, carl_baldwin: cooll just seen that | 20:53 |
| *** Zengfa has quit IRC | 20:53 | |
| carl_baldwin | kevinbenton, this change is only effective if agent's start_flag is true. | 20:53 |
| *** nyechiel has quit IRC | 20:53 | |
| carl_baldwin | kevinbenton, That said, I started out thinking the way you are. Maybe this isn't the right way to "fix" this bug. I almost wanted to mark the bug as invalid when I first read it. | 20:54 |
| *** padkrish has quit IRC | 20:54 | |
| kevinbenton | carl_baldwin: oh, i see. i take it that flag happens on the first iteration or something? | 20:55 |
| carl_baldwin | On agent start. | 20:55 |
| *** baoli has joined #openstack-neutron | 20:55 | |
| kevinbenton | carl_baldwin: right, but then the comparison to self.START_TIME on the server doesn't make any sense | 20:55 |
| *** chandankumar has quit IRC | 20:55 | |
| kevinbenton | carl_baldwin: if a server has been running for a week, it wouldn't detect a clock drift < a week | 20:56 |
| *** raildo is now known as raildo|away | 20:57 | |
| *** padkrish has joined #openstack-neutron | 20:57 | |
| kevinbenton | carl_baldwin: if we do want a clock watch, we should probably define some acceptable difference and then check for that if the 'start_flag' is present | 20:58 |
| *** hichihara has joined #openstack-neutron | 20:58 | |
| kevinbenton | carl_baldwin: regardless of whatever value START_TIME has | 20:58 |
| *** baoli has quit IRC | 20:59 | |
| *** annegentle has quit IRC | 20:59 | |
| *** baoli has joined #openstack-neutron | 21:00 | |
| carl_baldwin | kevinbenton, All great comments for the patch review. | 21:00 |
| kevinbenton | carl_baldwin: ack. i'll move it there | 21:01 |
| carl_baldwin | kevinbenton, cool | 21:01 |
| *** padkrish has quit IRC | 21:01 | |
| *** jamielennox|away is now known as jamielennox | 21:02 | |
| *** padkrish has joined #openstack-neutron | 21:02 | |
| *** nyechiel has joined #openstack-neutron | 21:04 | |
| *** eduardoj has quit IRC | 21:04 | |
| *** bobmel_ has joined #openstack-neutron | 21:04 | |
| *** nyechiel_ has joined #openstack-neutron | 21:05 | |
| *** jorgem has quit IRC | 21:05 | |
| *** bobmel has quit IRC | 21:07 | |
| *** nyechiel has quit IRC | 21:09 | |
| *** afazekas_ has joined #openstack-neutron | 21:09 | |
| *** claudiub has quit IRC | 21:09 | |
| *** nati_ueno has quit IRC | 21:10 | |
| *** nati_ueno has joined #openstack-neutron | 21:10 | |
| *** igordcard_ has joined #openstack-neutron | 21:13 | |
| *** evgenyf has joined #openstack-neutron | 21:13 | |
| *** annegentle has joined #openstack-neutron | 21:14 | |
| *** ronis has quit IRC | 21:14 | |
| *** carl_baldwin has quit IRC | 21:15 | |
| *** nati_ueno has quit IRC | 21:16 | |
| *** nati_ueno has joined #openstack-neutron | 21:16 | |
| *** carl_baldwin has joined #openstack-neutron | 21:18 | |
| *** yamamoto_ has joined #openstack-neutron | 21:18 | |
| *** Marga_ has quit IRC | 21:18 | |
| *** prad has joined #openstack-neutron | 21:20 | |
| *** nyechiel_ has quit IRC | 21:20 | |
| armax | amuller: ping | 21:22 |
| *** nyechiel_ has joined #openstack-neutron | 21:22 | |
| *** yamamoto_ has quit IRC | 21:23 | |
| *** kfox1111 has quit IRC | 21:23 | |
| *** localloop127 has quit IRC | 21:25 | |
| *** evgenyf has quit IRC | 21:25 | |
| *** yfauser has joined #openstack-neutron | 21:25 | |
| *** yfauser has left #openstack-neutron | 21:25 | |
| *** evgenyf has joined #openstack-neutron | 21:26 | |
| *** akanksha has joined #openstack-neutron | 21:26 | |
| *** nati_ueno has quit IRC | 21:26 | |
| *** dboik_ has joined #openstack-neutron | 21:27 | |
| *** nati_ueno has joined #openstack-neutron | 21:27 | |
| *** melwitt has quit IRC | 21:28 | |
| *** nyechiel__ has joined #openstack-neutron | 21:28 | |
| *** thedodd has quit IRC | 21:28 | |
| openstackgerrit | Merged openstack/neutron: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 21:28 |
| *** nati_ueno has quit IRC | 21:28 | |
| *** nati_ueno has joined #openstack-neutron | 21:29 | |
| *** nyechiel_ has quit IRC | 21:30 | |
| *** dboik has quit IRC | 21:30 | |
| *** nati_ueno has quit IRC | 21:31 | |
| *** nmagnezi_ has quit IRC | 21:31 | |
| *** nati_ueno has joined #openstack-neutron | 21:31 | |
| carl_baldwin | /msg NickServ identify 9GS7WQiojOv6 | 21:32 |
| carl_baldwin | oops | 21:32 |
| kevinbenton | carl_baldwin: might want to change that :) | 21:32 |
| carl_baldwin | kevinbenton, :) | 21:32 |
| kevinbenton | carl_baldwin: especially since eavesdrop is on this channel now IIRC | 21:32 |
| carl_baldwin | Good think about throw-away passwords. | 21:33 |
| carl_baldwin | s/think/thing/ | 21:33 |
| carl_baldwin | Amazing what a little space in front of a message like that will do. :) | 21:33 |
| *** afazekas_ has quit IRC | 21:34 | |
| kevinbenton | carl_baldwin: yeah, i always do it in the network channel after i learned my lesson doing that once | 21:34 |
| kevinbenton | carl_baldwin: this thing is quick | 21:34 |
| kevinbenton | carl_baldwin: http://eavesdrop.openstack.org/irclogs/%23openstack-neutron/%23openstack-neutron.2015-03-23.log | 21:34 |
| carl_baldwin | It does stay up to date. | 21:35 |
| *** rpothier has quit IRC | 21:35 | |
| carl_baldwin | kevinbenton, It is already changed. | 21:35 |
| *** sripriya has quit IRC | 21:35 | |
| *** yamahata has quit IRC | 21:35 | |
| *** sripriya has joined #openstack-neutron | 21:36 | |
| *** yamahata has joined #openstack-neutron | 21:36 | |
| carl_baldwin | kevinbenton, could have been much worse. | 21:36 |
| *** afazekas_ has joined #openstack-neutron | 21:37 | |
| *** yamahata has quit IRC | 21:37 | |
| *** yamahata has joined #openstack-neutron | 21:38 | |
| *** ritesh has quit IRC | 21:38 | |
| *** ritesh has joined #openstack-neutron | 21:38 | |
| *** tsg has joined #openstack-neutron | 21:40 | |
| *** rangansw_ has joined #openstack-neutron | 21:40 | |
| *** Swami has quit IRC | 21:40 | |
| *** melwitt has joined #openstack-neutron | 21:41 | |
| *** nyechiel__ has quit IRC | 21:42 | |
| *** afazekas_ has quit IRC | 21:43 | |
| *** pritesh has quit IRC | 21:43 | |
| *** ritesh has quit IRC | 21:43 | |
| *** rangansw has quit IRC | 21:44 | |
| *** lilynn has quit IRC | 21:44 | |
| amuller | armax: heya, was running an errand | 21:44 |
| armax | amuller: no! | 21:45 |
| armax | amuller: you gotta sit to your chair until you’re done! | 21:45 |
| amuller | armax: You and Kevin share a similar life philosophy | 21:45 |
| armax | amuller: we do, don’t we | 21:45 |
| armax | amuller: see your last HA patches | 21:45 |
| armax | there’s some love for you | 21:45 |
| amuller | armax: So looks like I screwed up something there | 21:45 |
| *** lilynn has joined #openstack-neutron | 21:46 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Handle non-index lookups in native OVSDB backend https://review.openstack.org/167017 | 21:46 |
| amuller | armax: some tests are passing, some aren't, curious :) | 21:46 |
| armax | amuller: yah | 21:46 |
| armax | amuller: that’s dodgy | 21:46 |
| *** rangansw has joined #openstack-neutron | 21:48 | |
| *** baoli has quit IRC | 21:48 | |
| *** oda-g has joined #openstack-neutron | 21:50 | |
| *** rangansw_ has quit IRC | 21:51 | |
| *** pc_m has quit IRC | 21:51 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers https://review.openstack.org/149068 | 21:53 |
| *** evgenyf has quit IRC | 21:53 | |
| *** matrohon has quit IRC | 21:54 | |
| amuller | armax: The tests that were failing are tests that initialize a plugin that uses the L3 DB scheduler base directly, and not the L3 HA DB scheduler mixin (Like the L3 service plugin), so that's solid | 21:54 |
| amuller | Good that we have those tests | 21:54 |
| armax | so my suggestion is completely bollocks? | 21:55 |
| amuller | armax: No what you pointed out is true | 21:55 |
| armax | amuller: ah, ok | 21:55 |
| amuller | Interestingly there's some plugins that use the L3 DB scheduler base class directly and their CI passed | 21:55 |
| *** _amrith_ is now known as amrith | 21:55 | |
| amuller | they should have exploded | 21:55 |
| amuller | oh well | 21:55 |
| * armax barfs | 21:56 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 21:57 |
| openstackgerrit | Merged openstack/neutron: Move README.odl into opendaylight directory https://review.openstack.org/166863 | 22:01 |
| *** dannywilson has quit IRC | 22:01 | |
| amuller | HenryG: +1'd the dual gateway patch | 22:01 |
| amuller | dane_leblanc++ | 22:01 |
| amuller | dane_leblanc: Next time I could only advise to make or change the functional test *before* you start development on such a patch. If you're anywhere as stupid as I am that would have saved you probably 2-3 hours of your life easy | 22:03 |
| HenryG | amuller: thanks for helping with that | 22:03 |
| dane_leblanc | amuller: Ha! Thanks for your help. | 22:03 |
| *** rangansw has quit IRC | 22:03 | |
| HenryG | amuller: did you see absubram's question above? | 22:04 |
| *** rangansw has joined #openstack-neutron | 22:04 | |
| amuller | I did not | 22:04 |
| HenryG | amuller: what is the difference between vsctl and native for functional tests? | 22:05 |
| amuller | HenryG: otherwiseguy: The functional tests (At least the ones using OVS) run twice | 22:05 |
| amuller | once with an ovs-vsctl based OVS driver | 22:05 |
| *** melwitt has quit IRC | 22:05 | |
| amuller | and one that uses a Python binding that talks directly via the OVSDB protocol to the local ovs daemon | 22:06 |
| amuller | Thanks to otherwiseguy's work | 22:06 |
| amuller | The native implementation should end up faster | 22:06 |
| *** rangansw has quit IRC | 22:06 | |
| *** rangansw has joined #openstack-neutron | 22:06 | |
| *** Marga_ has joined #openstack-neutron | 22:06 | |
| *** yamamoto_ has joined #openstack-neutron | 22:06 | |
| HenryG | amuller: ah yes, I knew of that work, didn't realize how the functional tests were updated. Thanks | 22:07 |
| *** lori is now known as lori|away | 22:08 | |
| *** melwitt has joined #openstack-neutron | 22:08 | |
| *** armax has quit IRC | 22:08 | |
| *** sripriya has quit IRC | 22:08 | |
| *** bjornar_ has quit IRC | 22:09 | |
| *** julim has quit IRC | 22:09 | |
| *** kbringard has quit IRC | 22:09 | |
| HenryG | amuller: Know anything about DBusException when running functional tests? | 22:11 |
| amuller | HenryG: I don't think so, you got a trace/log? | 22:11 |
| HenryG | amuller: http://paste.openstack.org/show/195608/ | 22:11 |
| *** padkrish has quit IRC | 22:12 | |
| *** padkrish has joined #openstack-neutron | 22:13 | |
| amuller | HenryG: that's freaky, I don't know why during a DVR functional test it would use keystoneclient and try to connect to something? | 22:14 |
| *** padkrish_ has joined #openstack-neutron | 22:14 | |
| amuller | oh ok it looks like it's a part of the metadata proxy somehow | 22:14 |
| amuller | HenryG: What patch is that running against? | 22:15 |
| HenryG | amuller: master, no patch | 22:15 |
| HenryG | amuller: master from 15 minutes ago | 22:15 |
| absubram | Yep.. I see the same behavior.. | 22:15 |
| amuller | I'll try running it locally | 22:15 |
| HenryG | amuller: could be something in my localrc that hasn't been a problem until something changed recently? | 22:16 |
| *** moshele has quit IRC | 22:16 | |
| *** baoli has joined #openstack-neutron | 22:17 | |
| *** padkrish has quit IRC | 22:17 | |
| *** thedodd has joined #openstack-neutron | 22:18 | |
| amuller | HenryG: well, many things can change, like new binaries, new rootwrap filters, no library requirements | 22:18 |
| amuller | so neutron.tests.functional.agent.linux.test_iptables_firewall.IptablesFirewallTestCase.test_port_sec_within_firewall just failed for me | 22:18 |
| *** thedodd has quit IRC | 22:18 | |
| amuller | and I saw it failing at the gate 2 times today | 22:18 |
| amuller | we're going to have to deal with that very soon | 22:19 |
| *** aranjan has quit IRC | 22:19 | |
| amuller | HenryG: But only that test failed, the DVR ones passed | 22:20 |
| amuller | with F20, selinux off | 22:20 |
| HenryG | amuller: what OS? | 22:20 |
| HenryG | amuller: ok, I am using ubuntu 14.04 | 22:20 |
| *** apuimedo has joined #openstack-neutron | 22:20 | |
| *** sbfox has quit IRC | 22:21 | |
| amuller | My Ubuntu VM is being mean to me lately | 22:21 |
| amuller | I can try again | 22:21 |
| amuller | HenryG: Did you get any failures on that run? Did you run all tests? | 22:23 |
| amuller | any other* | 22:23 |
| amuller | also how many cores on machine that's running those tests? | 22:23 |
| HenryG | amuller: ran all tests. 28 failed with this error. 4 cores | 22:23 |
| *** jobewan has quit IRC | 22:23 | |
| amuller | oh, all tests failed | 22:24 |
| amuller | wow :) | 22:24 |
| amuller | well, 28 | 22:24 |
| salv-orlando | I'd say that on HenryG machine the fake messaging driver does not know he's fake | 22:25 |
| salv-orlando | it might have developed an ego. I suggest you beat him hard with a stick | 22:25 |
| kevinbenton | the pinocchio messaging driver | 22:25 |
| salv-orlando | kevinbenton: was it wooden in the first place? | 22:26 |
| HenryG | amuller: What I don't like is that the functional test is being run from ~/Dev/neutron and is calling /usr/local/bin/neutron-keepalived-state-change which is linked to /opt/stack/neutron/... | 22:26 |
| amuller | HenryG: that's not supposed to happen | 22:26 |
| amuller | Maru made a change recently where all binaries are run from .tox/dsvm-functional/bin/... | 22:27 |
| kevinbenton | salv-orlando: possibly :) | 22:27 |
| *** sbfox has joined #openstack-neutron | 22:28 | |
| *** baoli has quit IRC | 22:30 | |
| *** dboik_ has quit IRC | 22:30 | |
| *** dboik has joined #openstack-neutron | 22:30 | |
| *** smillward has quit IRC | 22:30 | |
| amuller | HenryG: All functional tests passed on my Ubuntu VM, sorry =/ | 22:31 |
| amuller | Don't really know what to do from here | 22:31 |
| *** prad has quit IRC | 22:31 | |
| HenryG | amuller: Fly over here and fix my environment for me? :) | 22:32 |
| amuller | I'd love that honestly heh | 22:32 |
| *** dboik has quit IRC | 22:32 | |
| *** salv-orl_ has joined #openstack-neutron | 22:34 | |
| *** salv-orlando has quit IRC | 22:34 | |
| *** sridhar_ram has quit IRC | 22:35 | |
| *** salv-orl_ has quit IRC | 22:38 | |
| *** tflynn_ has joined #openstack-neutron | 22:40 | |
| *** tflynn_ has quit IRC | 22:42 | |
| *** salv-orlando has joined #openstack-neutron | 22:42 | |
| *** tflynn_ has joined #openstack-neutron | 22:42 | |
| *** tflynn has quit IRC | 22:42 | |
| *** armax has joined #openstack-neutron | 22:49 | |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Move external port processing to router classes https://review.openstack.org/158495 | 22:51 |
| carl_baldwin | HenryG, ping re: https://review.openstack.org/#/c/158495/ | 22:51 |
| *** ijw has joined #openstack-neutron | 22:52 | |
| HenryG | carl_baldwin: on it | 22:52 |
| carl_baldwin | HenryG, I added the change required to fix that UT and ran UTs after --amend. They pass. Set the comparison base in to PS23 and the review is easy. | 22:52 |
| * carl_baldwin a bit embarrassed | 22:53 | |
| *** crc32 has quit IRC | 22:53 | |
| *** annegentle has quit IRC | 22:53 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Fixes floating IP regression with multiple routers https://review.openstack.org/157167 | 22:53 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add no_delete flag to UT router context manager https://review.openstack.org/167030 | 22:53 |
| *** absubram has quit IRC | 22:55 | |
| carl_baldwin | kevinbenton, I was tempted to fix https://review.openstack.org/#/c/157167 the same way you did. | 22:56 |
| HenryG | carl_baldwin: the fix is what I expected, all done. Good work. | 22:56 |
| kevinbenton | carl_baldwin: yeah, that's long been on my list of things to do after we finished cleaning it up for the core resources | 22:57 |
| carl_baldwin | HenryG, many thanks for your continued attention. | 22:57 |
| kevinbenton | carl_baldwin: lots of annoying teardown logic allover that file that has nothing to do with the tests | 22:57 |
| HenryG | marun: around? | 22:57 |
| *** aepifanov has quit IRC | 22:57 | |
| HenryG | amuller: do you have a /usr/local/bin/neutron-ns-metadata-proxy (in your ubuntu vm)? | 22:58 |
| *** padkrish_ has quit IRC | 23:00 | |
| *** sripriya has joined #openstack-neutron | 23:00 | |
| *** padkrish has joined #openstack-neutron | 23:00 | |
| ZZelle_ | HenryG, i have it | 23:01 |
| *** markvoelker has quit IRC | 23:01 | |
| ZZelle_ | HenryG, it seems required by test_l3_agent functional test | 23:01 |
| *** padkrish_ has joined #openstack-neutron | 23:02 | |
| carl_baldwin | kevinbenton, had to rebase it, didn't you. ;) | 23:02 |
| kevinbenton | carl_baldwin: yeah, i emailed him to make sure he was okay with me doing that | 23:02 |
| carl_baldwin | kevinbenton, that's okay. I have my trusty hacked up script to compare it with the old one. | 23:02 |
| *** rangansw_ has joined #openstack-neutron | 23:02 | |
| *** shwetaap has quit IRC | 23:02 | |
| kevinbenton | carl_baldwin: oh, you mean against master | 23:02 |
| *** sridhar_ram has joined #openstack-neutron | 23:02 | |
| *** Marga_ has quit IRC | 23:02 | |
| carl_baldwin | kevinbenton, yes | 23:03 |
| kevinbenton | carl_baldwin: yeah, sorry about that | 23:03 |
| *** Marga_ has joined #openstack-neutron | 23:03 | |
| carl_baldwin | kevinbenton, I'm collecting votes here: https://review.openstack.org/#/c/140863/ You are welcome to weigh in. | 23:03 |
| *** thurloat has joined #openstack-neutron | 23:03 | |
| HenryG | ZZelle_: I am trying to figure out why my functional tests are running the system installed neutron scripts instead of .tox/dsvm-functiona/bin ones. | 23:03 |
| *** rangansw has quit IRC | 23:03 | |
| *** ivar-laz_ has joined #openstack-neutron | 23:04 | |
| *** padkrish has quit IRC | 23:05 | |
| *** ivar-lazzaro has quit IRC | 23:07 | |
| *** dannywilson has joined #openstack-neutron | 23:08 | |
| *** dannywilson has quit IRC | 23:08 | |
| *** nati_ueno has quit IRC | 23:09 | |
| ZZelle_ | HenryG, i have the same trouble, test_l3_agent test uses /usr/local/bin/neutron-ns... | 23:09 |
| *** dannywilson has joined #openstack-neutron | 23:09 | |
| *** nati_ueno has joined #openstack-neutron | 23:09 | |
| ZZelle_ | am i wrong or neutron code references ovs package but ovs is not declared in requirements.txt | 23:14 |
| ZZelle_ | ? | 23:14 |
| *** dane_leblanc has quit IRC | 23:15 | |
| *** Swami has joined #openstack-neutron | 23:17 | |
| *** banix has joined #openstack-neutron | 23:17 | |
| *** yuanying has joined #openstack-neutron | 23:22 | |
| HenryG | which ovs package? | 23:22 |
| HenryG | requirements.txt is only for python packages | 23:23 |
| ZZelle_ | HenryG, but native implementation references an ovs python package | 23:24 |
| *** ijw_ has joined #openstack-neutron | 23:24 | |
| ZZelle_ | HenryG, https://github.com/openstack/neutron/commit/0363ec23e744a5cae0f85318031e3acaa9e45d15#diff-ce641aa9c73700a9cda4d5beb32b117fR21 | 23:24 |
| *** sbfox has quit IRC | 23:24 | |
| *** ijw_ has quit IRC | 23:25 | |
| *** ijw_ has joined #openstack-neutron | 23:26 | |
| *** thomasem has quit IRC | 23:26 | |
| *** ijw has quit IRC | 23:27 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Refactor Collapse tests.sub_base module into https://review.openstack.org/167041 | 23:27 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Ml2 Mechanism Driver for OVSvApp Solution https://review.openstack.org/160247 | 23:29 |
| *** chlong has joined #openstack-neutron | 23:30 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Handle non-index lookups in native OVSDB backend https://review.openstack.org/167017 | 23:32 |
| *** ijw has joined #openstack-neutron | 23:33 | |
| HenryG | otherwiseguy: ping | 23:33 |
| *** leenheer has joined #openstack-neutron | 23:33 | |
| otherwiseguy | HenryG: pong | 23:33 |
| HenryG | otherwiseguy: did you see ZZelle_'s question above ^^ | 23:33 |
| *** thomasem has joined #openstack-neutron | 23:33 | |
| *** mwagner_lap has joined #openstack-neutron | 23:33 | |
| otherwiseguy | HenryG: ZZelle: I hadn't. The ovs package does not exist in PyPI. It lives in the openvswitch tree, distro packages are built from there. | 23:35 |
| HenryG | otherwiseguy: so devstack installs it? | 23:35 |
| otherwiseguy | I'd like to work on getting it added to PyPI, but there is a bit of work to get that done. It doesn't have any setuptools stuff in it, it has some generated files containing hard-coded paths from ./configure, etc. | 23:35 |
| ZZelle_ | otherwiseguy, HenryG, not on my devstack env at least | 23:35 |
| *** baoli has joined #openstack-neutron | 23:36 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Simplify base test cases https://review.openstack.org/167041 | 23:36 |
| otherwiseguy | HenryG: I know the configure_for_func_testing.sh script installs it for functional tests. YorikSar did the devstack patch. Not sure if he added the package or not. | 23:36 |
| *** ijw_ has quit IRC | 23:36 | |
| *** akanksha has quit IRC | 23:37 | |
| *** tflynn_ has quit IRC | 23:37 | |
| *** tflynn has joined #openstack-neutron | 23:38 | |
| otherwiseguy | ZZelle_: HenryG: Doesn't look like https://review.openstack.org/#/c/107386/ added the package requirement. | 23:38 |
| *** zzzeek has quit IRC | 23:39 | |
| *** chem has quit IRC | 23:40 | |
| kevinbenton | carl_baldwin: can you get this one too? https://review.openstack.org/#/c/167030/ | 23:40 |
| *** tflynn has quit IRC | 23:40 | |
| *** nati_ueno has quit IRC | 23:40 | |
| *** nati_ueno has joined #openstack-neutron | 23:40 | |
| *** tflynn has joined #openstack-neutron | 23:40 | |
| ZZelle_ | otherwiseguy, not sure to understand how installing python-openvswitch, allow ovs package from a tox virtualenv | 23:41 |
| *** chem has joined #openstack-neutron | 23:41 | |
| *** annegentle has joined #openstack-neutron | 23:41 | |
| otherwiseguy | ZZelle_: dsvm-functional allows external pacakges | 23:41 |
| otherwiseguy | sitepackages=True | 23:42 |
| ZZelle_ | otherwiseguy, ok | 23:43 |
| *** ajmiller has quit IRC | 23:44 | |
| *** flwang1 has quit IRC | 23:45 | |
| *** davrodpin has joined #openstack-neutron | 23:45 | |
| *** hitalia has joined #openstack-neutron | 23:46 | |
| *** markvoelker has joined #openstack-neutron | 23:47 | |
| *** davrodpin has left #openstack-neutron | 23:48 | |
| *** haleyb has quit IRC | 23:48 | |
| *** davrodpin has joined #openstack-neutron | 23:48 | |
| amuller | HenryG: checking | 23:48 |
| amuller | HenryG: I do | 23:49 |
| HenryG | amuller: thanks. still poking around trying to figure this out | 23:50 |
| carl_baldwin | kevinbenton, looking... | 23:50 |
| *** leenheer has quit IRC | 23:50 | |
| *** davrodpin has quit IRC | 23:51 | |
| kevinbenton | carl_baldwin: it's the parent to the one you approved | 23:51 |
| carl_baldwin | kevinbenton, got it. I didn't notice you'd added a dependency. | 23:51 |
| *** markvoelker has quit IRC | 23:51 | |
| kevinbenton | carl_baldwin: yeah, i wanted to keep them separate | 23:52 |
| carl_baldwin | Makes sense. | 23:52 |
| kevinbenton | carl_baldwin: i'm with you on disabling the rebase | 23:52 |
| carl_baldwin | :) | 23:53 |
| amuller | HenryG: I renamed that binary and re-ran the tests, everything still worked | 23:53 |
| amuller | it's using the one in the .tox dir | 23:53 |
| HenryG | amuller: it seems to go wrong for me only when executing commands in namespaces. | 23:56 |
| *** padkrish_ has quit IRC | 23:56 | |
| HenryG | somehow namespace is losing track of venv? | 23:56 |
| *** padkrish has joined #openstack-neutron | 23:56 | |
| *** carl_baldwin has quit IRC | 23:57 | |
| *** aranjan has joined #openstack-neutron | 23:57 | |
| *** ijw_ has joined #openstack-neutron | 23:58 | |
| ZZelle_ | HenryG, because of sudo which reset PATH env var? | 23:58 |
| *** dims_ has joined #openstack-neutron | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!