Tuesday, 2015-03-10

*** bandarji has quit IRC00:00
*** tflynn_ has joined #openstack-neutron00:01
*** tflynn_ has quit IRC00:01
mlavalleamuller: aren't you in Israel? What time is it there?00:01
amullermlavalle: My wife and I relocated to Boston a couple of months ago00:01
amullerBut you have a good memory :)00:02
carl_baldwinamuller, btw, I finally followed-up on your comments on https://review.openstack.org/#/c/154513/.  Thanks for the feedback.00:02
amullercarl_baldwin: Sure, I'll review tomorrow hopefully00:02
mlavalleamuller: ahhhh! cool.... And it welcomed you with this miserable winter...... My daughter is a senior at MIT.... so she is sharing your weather adventures00:03
amullermlavalle: Cool :) The weather is getting a lot better now though00:03
*** yuanying has quit IRC00:03
*** Marga_ has joined #openstack-neutron00:04
*** tflynn has quit IRC00:04
*** yuanying has joined #openstack-neutron00:05
carl_baldwinamuller, No rush.  It is late for you.00:05
*** banix has quit IRC00:05
carl_baldwin... even in Boston.00:05
mlavalleamuller: it is almost imposible for the MIT kids to get snow days. But this winter was sobad that this semester I think she's got 300:05
*** oomichi has quit IRC00:06
amullermlavalle: hah yeah there were 2 or 3 pretty nasty storms (Especially from my warm blooded perspective)00:06
carl_baldwinmlavalle, I think I agree with amuller's feedback.   He's got a pretty good eye for how to approach testing.00:08
mlavalleamuller, carl_baldwin: perfect.... I'll take it from here.... and thanks for taking the time to give me feedback :-)00:08
carl_baldwinamuller:  Those storms were pretty nasty even from my perspective.  I wasn't there but Brian and Jack from my team are out there.00:08
carl_baldwinmlavalle, It is a pleasure.00:09
*** shivharis has quit IRC00:09
*** shivharis has joined #openstack-neutron00:09
*** yuanying_ has joined #openstack-neutron00:10
*** fredlhsu has quit IRC00:10
*** yuanying has quit IRC00:10
*** igordcard has quit IRC00:12
amullerarmax: carl_baldwin: I'm close to pushing a patch which I'm a bit excited about, as excited as I could possibly about testing: I'm adding tests for RPC interaction between agents and the controller. It uses the Oslo messaging fake driver (Like all of our tests), but it doesn't monkey patch away a critical bit. The fake driver passes messages around in-process. The idea is only to test that a client calls a method on the server with the00:12
amullercorrect parameters, not to test the actual server methods (As those are already tested in their respective places)00:12
amullerSo we assume that the non-fake oslo messaging driver works (As that's tested already)00:13
amullerAnd we assume that the RPC methods themselves are tested00:13
amullerbut these tests actually invoke a method from the client and have the code executed on the server00:13
amullerwhich is something that we currently only have in Tempest00:13
amullerarmax: This should catch regressions we had in DVR, where the solution was to introduce unit tests that practically mock what they're testing00:14
*** vivek-ebay has quit IRC00:16
*** s3wong has quit IRC00:16
*** vivek-ebay has joined #openstack-neutron00:16
openstackgerritRajeev Grover proposed openstack/neutron: If configured, set the MTU for fpr/rfp intefaces  https://review.openstack.org/16231200:17
*** s3wong has joined #openstack-neutron00:17
*** mlavalle has quit IRC00:19
openstackgerritIsaku Yamahata proposed openstack/neutron: internal callers of create_{network, subnet, port} ignore extension  https://review.openstack.org/12977300:19
*** xgerman has quit IRC00:23
*** thomasem has quit IRC00:23
*** Marga_ has quit IRC00:23
*** Marga_ has joined #openstack-neutron00:24
carl_baldwinamuller, sounds like some good stuff.00:25
*** sbfox has quit IRC00:26
*** thomasem has joined #openstack-neutron00:28
*** achanda has quit IRC00:30
*** yamamoto has quit IRC00:30
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238300:31
*** banix has joined #openstack-neutron00:31
*** baoli has quit IRC00:34
openstackgerritAssaf Muller proposed openstack/neutron: Introduce RPC tests  https://review.openstack.org/16281100:35
*** hichihara has joined #openstack-neutron00:36
*** hichihara has quit IRC00:36
openstackgerritRajeev Grover proposed openstack/neutron: Functional Test for DVR fpr/rfp interface MTU set  https://review.openstack.org/16281200:36
openstackgerritAssaf Muller proposed openstack/neutron: Introduce RPC tests  https://review.openstack.org/16281100:37
amullernight00:39
*** amuller has quit IRC00:39
*** smillward has quit IRC00:40
*** baoli has joined #openstack-neutron00:40
*** SumitNaiksatam has joined #openstack-neutron00:42
*** ijw_ has joined #openstack-neutron00:42
*** ChuckC has quit IRC00:42
*** ijw has quit IRC00:45
*** _cjones_ has quit IRC00:46
*** pavel_bondar has quit IRC00:46
*** pavel_bondar has joined #openstack-neutron00:47
*** Rajeev has quit IRC00:48
*** ChuckC has joined #openstack-neutron00:49
*** r-daneel has quit IRC00:49
openstackgerritMerged openstack/neutron: Initial copy of api tests from tempest  https://review.openstack.org/16225700:50
*** ChuckC has quit IRC00:53
*** ChuckC has joined #openstack-neutron00:53
*** s3wong has quit IRC00:54
*** s3wong has joined #openstack-neutron00:55
*** tiswanso has joined #openstack-neutron00:55
*** yamamoto has joined #openstack-neutron00:55
openstackgerritCarl Baldwin proposed openstack/neutron: Move external port processing to router classes  https://review.openstack.org/15849500:56
*** tiswanso has quit IRC00:56
*** tiswanso has joined #openstack-neutron00:57
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238300:59
*** yuanying_ has quit IRC00:59
openstackgerritCarl Baldwin proposed openstack/neutron: Move external port processing to router classes  https://review.openstack.org/15849501:00
*** yuanying has joined #openstack-neutron01:01
*** tflynn has joined #openstack-neutron01:02
*** banix has quit IRC01:02
openstackgerritJoeJiang proposed openstack/python-neutronclient: conform to the PEP8  https://review.openstack.org/16282001:05
*** britthouser has joined #openstack-neutron01:05
*** tflynn has quit IRC01:06
*** xuhanp has joined #openstack-neutron01:08
*** nlahouti has quit IRC01:08
*** xuhanp_ has joined #openstack-neutron01:08
*** melwitt has quit IRC01:11
*** baoli has quit IRC01:14
*** SridharRamaswamy has quit IRC01:20
*** leenheer has quit IRC01:25
*** vivek-ebay has quit IRC01:25
*** vivek-eb_ has joined #openstack-neutron01:25
*** vmtrooper has joined #openstack-neutron01:30
*** banix has joined #openstack-neutron01:30
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: linuxbridge UT: Do not create the same instance in each cases  https://review.openstack.org/16282501:30
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: linuxbridge UT: Mock get_interface_by_ip  https://review.openstack.org/16282601:30
*** s3wong has quit IRC01:33
*** stanzgy has joined #openstack-neutron01:34
*** stanzgy_ has joined #openstack-neutron01:34
*** vmtrooper has quit IRC01:35
*** popw has joined #openstack-neutron01:37
*** Zemeio has joined #openstack-neutron01:38
*** ijw_ has quit IRC01:42
*** rwsu has quit IRC01:47
*** absubram has quit IRC01:48
*** yamahata has quit IRC01:49
*** jyuso1 has joined #openstack-neutron01:50
openstackgerritKevin Benton proposed openstack/neutron: Don't start transaction during floating IP delete  https://review.openstack.org/16003601:53
*** _cjones_ has joined #openstack-neutron01:56
*** shivharis has quit IRC01:56
*** shivharis has joined #openstack-neutron01:56
*** banix has quit IRC01:57
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ofagent: Have a thin driver module  https://review.openstack.org/16255502:00
*** armax has quit IRC02:00
*** shivharis has quit IRC02:01
*** diegows has quit IRC02:02
*** yapeng has joined #openstack-neutron02:06
*** yog_ has joined #openstack-neutron02:07
*** ivar-lazzaro has quit IRC02:11
*** carl_baldwin has quit IRC02:12
*** tellesnobrega_ has joined #openstack-neutron02:14
*** tellesnobrega_ has quit IRC02:14
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284002:16
*** salv-orlando has quit IRC02:20
*** banix has joined #openstack-neutron02:21
*** Networkn3rd has joined #openstack-neutron02:21
*** lsg has joined #openstack-neutron02:24
gongysh_amotoki: hi02:24
*** erkules_ has joined #openstack-neutron02:30
*** baoli has joined #openstack-neutron02:32
*** coolsvap has joined #openstack-neutron02:33
*** erkules has quit IRC02:33
*** lsg has quit IRC02:34
*** kiran has joined #openstack-neutron02:35
*** amotoki has quit IRC02:35
*** lsg has joined #openstack-neutron02:35
openstackgerritWei Tie proposed openstack/neutron: Add row lock when update dhcp reserved port  https://review.openstack.org/15911002:41
*** shivharis has joined #openstack-neutron02:43
*** vivek-eb_ has quit IRC02:45
*** shivharis has quit IRC02:45
*** coolsvap has quit IRC02:45
openstackgerritKyle Mestery proposed openstack/python-neutronclient: Copy functional tests from tempest cli  https://review.openstack.org/16284802:45
*** coolsvap has joined #openstack-neutron02:46
*** coolsvap has quit IRC02:46
*** coolsvap has joined #openstack-neutron02:46
*** yuanying has quit IRC02:46
openstackgerritKyle Mestery proposed openstack/python-neutronclient: Copy functional tests from tempest cli  https://review.openstack.org/16284802:47
*** achanda has joined #openstack-neutron02:47
*** gildub has quit IRC02:49
*** gildub has joined #openstack-neutron02:50
*** shwetaap has joined #openstack-neutron02:50
*** markvoelker has joined #openstack-neutron02:51
*** sbalukoff has quit IRC02:53
*** banix has quit IRC02:55
*** chandankumar has joined #openstack-neutron02:57
*** _cjones_ has quit IRC03:00
*** _cjones_ has joined #openstack-neutron03:00
*** yuanying has joined #openstack-neutron03:01
*** yuanying has quit IRC03:02
*** bquan has joined #openstack-neutron03:02
kiranHi. Can you please point me to the code where net-create command is implemented for Neutron? Specifically, I'm looking for how bridge_mappings configuration is used. Thanks,03:02
*** bquan has quit IRC03:03
*** gongysh_ has quit IRC03:05
*** yuanying has joined #openstack-neutron03:06
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers  https://review.openstack.org/15269703:09
*** gildub has quit IRC03:10
*** gongysh_ has joined #openstack-neutron03:17
*** SumitNaiksatam has quit IRC03:18
*** vmtrooper has joined #openstack-neutron03:19
*** salv-orlando has joined #openstack-neutron03:21
*** kiran has quit IRC03:23
*** jyuso1 has quit IRC03:24
*** vmtrooper has quit IRC03:24
*** salv-orlando has quit IRC03:26
*** zzzeek has quit IRC03:26
*** gildub has joined #openstack-neutron03:31
*** dims_ has quit IRC03:32
*** claudiub has quit IRC03:37
*** yuanying has quit IRC03:38
*** xuhanp has quit IRC03:38
*** xuhanp_ has quit IRC03:38
*** achanda has quit IRC03:40
*** yuanying has joined #openstack-neutron03:41
*** tiswanso has quit IRC03:43
*** shivharis has joined #openstack-neutron03:44
*** shivharis has quit IRC03:45
*** sbalukoff has joined #openstack-neutron03:49
*** dboik_ has quit IRC03:51
*** yapeng has quit IRC03:52
*** markvoelker has quit IRC03:53
*** markvoelker has joined #openstack-neutron03:53
*** markvoelker has quit IRC03:57
*** Longgeek has joined #openstack-neutron03:59
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: portsecurity_db: Fix a usage of is_attr_set  https://review.openstack.org/16256303:59
*** salv-orlando has joined #openstack-neutron04:01
*** tflynn has joined #openstack-neutron04:02
*** yatin has joined #openstack-neutron04:03
*** sbfox has joined #openstack-neutron04:04
*** sbfox has quit IRC04:04
*** yog_ has quit IRC04:06
*** achanda has joined #openstack-neutron04:06
*** tflynn has quit IRC04:07
*** tflynn has joined #openstack-neutron04:07
*** jorgem has joined #openstack-neutron04:08
*** gongysh has joined #openstack-neutron04:09
*** absubram has joined #openstack-neutron04:10
*** jorgem has quit IRC04:11
*** absubram has quit IRC04:14
*** salv-orlando has quit IRC04:14
*** sthillma has joined #openstack-neutron04:15
*** absubram has joined #openstack-neutron04:15
*** sthillma has quit IRC04:15
*** yog_ has joined #openstack-neutron04:18
*** yog_ has quit IRC04:20
*** saikrishna has joined #openstack-neutron04:23
openstackgerritBrian Haley proposed openstack/neutron: Change linux/ip_lib code to better handle address families  https://review.openstack.org/15755504:23
*** markvoelker has joined #openstack-neutron04:24
*** pradipta has joined #openstack-neutron04:26
*** amotoki has joined #openstack-neutron04:30
*** pradip_vedams has quit IRC04:31
*** Bhavya has joined #openstack-neutron04:32
*** dims has joined #openstack-neutron04:32
*** gongysh has quit IRC04:33
*** pradip_vedams has joined #openstack-neutron04:34
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856004:35
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439104:35
*** dims has quit IRC04:38
*** SridharG has joined #openstack-neutron04:39
*** yfried|afk is now known as yfried04:40
*** dims has joined #openstack-neutron04:40
*** anish_ has joined #openstack-neutron04:41
*** markvoelker has quit IRC04:41
*** trinaths has joined #openstack-neutron04:45
*** shivharis has joined #openstack-neutron04:45
*** dims has quit IRC04:45
*** shivharis has quit IRC04:46
*** baoli has quit IRC04:48
*** alexpilotti has joined #openstack-neutron04:48
*** pradipta has quit IRC04:49
*** baoli has joined #openstack-neutron04:50
openstackgerritAngela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches.  https://review.openstack.org/15470904:53
*** tflynn has quit IRC04:55
*** yfried is now known as yfried|afk04:56
*** baoli has quit IRC04:56
*** tflynn has joined #openstack-neutron04:58
openstackgerritMerged openstack/neutron: Use an existing function in process monitor tests  https://review.openstack.org/16169804:58
*** yfried|afk is now known as yfried04:58
*** leenheer has joined #openstack-neutron05:01
*** baoli has joined #openstack-neutron05:02
*** pradipta has joined #openstack-neutron05:06
*** baoli has quit IRC05:07
*** vmtrooper has joined #openstack-neutron05:08
*** yfried is now known as yfried|afk05:10
*** Sukhdev has joined #openstack-neutron05:11
*** yfried|afk is now known as yfried05:12
*** topshare has joined #openstack-neutron05:12
*** vmtrooper has quit IRC05:12
*** yshafrir has joined #openstack-neutron05:13
*** salv-orlando has joined #openstack-neutron05:13
*** nlahouti has joined #openstack-neutron05:15
Sukhdevsalv-orlando: Ping05:15
openstackgerritQin Zhao proposed openstack/python-neutronclient: Add InvalidIpForSubnetClient exception  https://review.openstack.org/16206205:16
*** nlahouti1 has joined #openstack-neutron05:16
*** nlahouti has quit IRC05:19
*** salv-orlando has quit IRC05:19
*** zhhuabj has quit IRC05:22
*** salv-orlando has joined #openstack-neutron05:22
*** salv-orlando has quit IRC05:26
*** yamamoto has quit IRC05:27
*** yfried has quit IRC05:27
*** vthapar has joined #openstack-neutron05:28
*** _cjones_ has quit IRC05:29
*** anilvenkata has joined #openstack-neutron05:32
*** markvoelker has joined #openstack-neutron05:38
*** chandankumar has quit IRC05:38
*** shwetaap has quit IRC05:41
*** shivharis has joined #openstack-neutron05:45
*** zhhuabj has joined #openstack-neutron05:46
*** shivharis has quit IRC05:47
*** nlahouti1 has quit IRC05:48
*** nlahouti has joined #openstack-neutron05:49
*** ajo has joined #openstack-neutron05:50
*** chandankumar has joined #openstack-neutron05:54
*** achanda has quit IRC05:54
*** sbalukoff has quit IRC05:55
*** achanda has joined #openstack-neutron05:55
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856005:55
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439105:55
*** Tusker has joined #openstack-neutron05:56
Tuskerheya guys05:56
*** saikrishna has quit IRC05:57
*** harlowja_ is now known as harlowja_away05:57
Tuskerhaving a problem deploying a fresh juno install of neutron DB05:57
Tuskerwith the inability to modify fk cisco_n1kv_vlan_allocations05:57
Tuskeri tried to add a "op.drop_constraint('cisco_n1kv_vlan_allocations_ibfk_1'," before the "op.create_foreign_key(" but still encounter the same error05:58
Tuskerthe heal script is trying to do a modify of a FK, but mysql is denying it05:59
*** irenab has joined #openstack-neutron05:59
Tuskerno-one run up a fresh juno db of neutron on mysql recently ?06:01
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver  https://review.openstack.org/15394606:02
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver  https://review.openstack.org/16024506:02
*** yamamoto has joined #openstack-neutron06:02
*** bquan has joined #openstack-neutron06:04
*** Longgeek has quit IRC06:04
*** bquan has quit IRC06:08
*** salv-orlando has joined #openstack-neutron06:08
*** neeti has joined #openstack-neutron06:08
*** moha_hunt has quit IRC06:08
*** salv-orlando has quit IRC06:08
*** achanda has quit IRC06:11
*** achanda has joined #openstack-neutron06:11
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/16254006:15
*** dims has joined #openstack-neutron06:21
*** Longgeek has joined #openstack-neutron06:23
*** SridharG has left #openstack-neutron06:24
*** tflynn has quit IRC06:24
*** dims has quit IRC06:26
TuskerOK, had to work around using native SQL: alter table cisco_n1kv_vlan_allocations drop foreign key cisco_n1kv_vlan_allocations_ibfk_1; ALTER TABLE cisco_n1kv_vlan_allocations MODIFY network_profile_id VARCHAR(36) COLLATE utf8_unicode_ci NOT NULL; alter table cisco_n1kv_vxlan_allocations drop foreign key cisco_n1kv_vxlan_allocations_ibfk_1; ALTER TABLE cisco_n1kv_vxlan_allocations MODIFY network_profile_id VARCHAR(36) CO06:26
*** yog__ has joined #openstack-neutron06:27
*** achanda has quit IRC06:28
*** salv-orlando has joined #openstack-neutron06:28
*** sgordon has quit IRC06:30
*** david-lyle has quit IRC06:30
*** _cjones_ has joined #openstack-neutron06:31
*** yfried has joined #openstack-neutron06:36
*** salv-orlando has quit IRC06:38
openstackgerritAkihiro Motoki proposed openstack/neutron: NEC plugin code split  https://review.openstack.org/16210306:38
*** david-lyle has joined #openstack-neutron06:39
*** tflynn has joined #openstack-neutron06:39
*** sgordon has joined #openstack-neutron06:40
*** yfried_ has joined #openstack-neutron06:40
*** yfried has quit IRC06:41
*** yfried_ is now known as yfried|afk06:44
*** shivharis has joined #openstack-neutron06:46
*** coolsvap is now known as coolsvap|afk06:47
*** shivharis has quit IRC06:48
*** bobmel has joined #openstack-neutron06:49
*** nlahouti has quit IRC06:50
*** bobmel has quit IRC06:50
*** bobmel has joined #openstack-neutron06:51
*** salv-orlando has joined #openstack-neutron06:54
*** anshul has joined #openstack-neutron06:54
*** nmagnezi_ has joined #openstack-neutron06:55
enikanorov_amotoki: hi, could you please take a look at https://review.openstack.org/#/c/162576/06:55
amotokienikanorov: hi, sure06:56
*** vmtrooper has joined #openstack-neutron06:56
*** lori|away is now known as lori06:57
*** coolsvap|afk is now known as coolsvap06:58
*** salv-orlando has quit IRC06:59
*** gildub has quit IRC07:00
*** vmtrooper has quit IRC07:01
*** ruijing has joined #openstack-neutron07:02
*** SridharG has joined #openstack-neutron07:02
*** _cjones_ has quit IRC07:02
*** _cjones_ has joined #openstack-neutron07:03
*** _cjones_ has quit IRC07:03
*** yfried|afk is now known as yfried_07:04
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Remove the reference for non-existent cisco.l3.plugging_drivers  https://review.openstack.org/16288707:06
*** garyk has joined #openstack-neutron07:06
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers  https://review.openstack.org/15269707:10
*** Sukhdev has quit IRC07:10
*** david-lyle is now known as david-lyle_afk07:11
*** itzikb has joined #openstack-neutron07:12
*** irenab has quit IRC07:12
*** numan has joined #openstack-neutron07:13
*** zhhuabj has quit IRC07:18
*** zhhuabj has joined #openstack-neutron07:18
*** tflynn has quit IRC07:19
*** scheuran has joined #openstack-neutron07:21
*** chlong has quit IRC07:21
*** sbalukoff has joined #openstack-neutron07:25
*** killer_prince is now known as lazy_prince07:27
openstackgerritAngus Lees proposed openstack/neutron: Introduce privilege separation process  https://review.openstack.org/15563107:30
*** anish__ has joined #openstack-neutron07:31
*** jistr has joined #openstack-neutron07:31
*** xuhanp has joined #openstack-neutron07:32
*** xuhanp_ has joined #openstack-neutron07:32
*** anish_ has quit IRC07:32
openstackgerritAkihiro Motoki proposed openstack/neutron: NEC plugin code split  https://review.openstack.org/16210307:32
*** openstackgerrit has quit IRC07:35
*** openstackgerrit has joined #openstack-neutron07:36
*** gridinv has joined #openstack-neutron07:36
enikanorov_amotoki: ping07:37
openstackgerritNurit Vilosny proposed openstack/neutron: Add notification flag (True / False) to extension_manager process_update_port  https://review.openstack.org/16264807:38
amotokienikanorov_: pong07:38
enikanorov_amotoki: how do you see set() is passed to agent/linux/iptables_firewall.py ?07:38
amotokienikanorov_: I haven't checked the real env. I might be wrong if RPC serialization replace it to JSON list.07:39
enikanorov_amotoki: yes, that's what i know, it should be simply json list07:40
enikanorov_amotoki: one idea i had: may be just use set in the method, but leave list in sg dict?07:40
enikanorov_e.g. convert set into list before exit from the method07:40
amotokienikanorov_: I see, so there is no problem.07:40
*** gridinv has quit IRC07:41
amotokienikanorov_: converting it to a list for all entries needs additional nested loops... I think it is better to add a note that it is converted into JSON list on RPC. it must helps us understand the code.07:43
enikanorov_yep, makes sense07:43
*** kbringard has joined #openstack-neutron07:44
*** kbringard has quit IRC07:44
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_l2population: Simplify migration tests  https://review.openstack.org/15818207:44
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_l2population: Simplify migration tests  https://review.openstack.org/15818207:44
*** ronis has joined #openstack-neutron07:45
*** leenheer has quit IRC07:46
*** shivharis has joined #openstack-neutron07:47
*** irenab has joined #openstack-neutron07:48
*** shivharis has quit IRC07:49
*** coolsvap is now known as coolsvap|afk07:53
openstackgerritdql proposed openstack/neutron: fix DHCP port changed when dhcp-agent restart  https://review.openstack.org/16252207:54
*** adelinatuvenie has joined #openstack-neutron08:00
*** pask81 has joined #openstack-neutron08:00
*** coolsvap|afk is now known as coolsvap08:00
openstackgerritenikanorov proposed openstack/neutron: Improve performance of _get_security_group_member_ips  https://review.openstack.org/16257608:03
*** nuritv has joined #openstack-neutron08:05
*** jyuso1 has joined #openstack-neutron08:05
*** bobmel has quit IRC08:05
*** erkules_ is now known as erkules08:09
*** erkules has joined #openstack-neutron08:09
*** safchain has joined #openstack-neutron08:13
*** liusheng has joined #openstack-neutron08:14
*** kawa2014 has joined #openstack-neutron08:14
*** matrohon has joined #openstack-neutron08:15
*** moshele has joined #openstack-neutron08:15
*** yamahata has joined #openstack-neutron08:17
*** luqas has joined #openstack-neutron08:19
openstackgerritMerged openstack/neutron: In Arista ML2 delete tenant without any resources  https://review.openstack.org/16277808:20
*** gridinv has joined #openstack-neutron08:23
*** ronis has quit IRC08:24
*** claudiub has joined #openstack-neutron08:25
*** adelinatuvenie has quit IRC08:27
*** gridinv_ has joined #openstack-neutron08:28
*** gridinv has quit IRC08:28
*** lukasa has joined #openstack-neutron08:30
openstackgerritSergey Belous proposed openstack/neutron: Added check for emptyness where in_ is being used  https://review.openstack.org/15003408:32
*** gridinv_ has quit IRC08:33
*** adelinatuvenie has joined #openstack-neutron08:34
*** yfried_ is now known as yfried|afk08:36
*** [1]adelinatuveni has joined #openstack-neutron08:37
*** jlanoux has joined #openstack-neutron08:38
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver  https://review.openstack.org/15394608:39
*** adelinatuvenie has quit IRC08:39
*** [1]adelinatuveni is now known as adelinatuvenie08:39
*** nyechiel has joined #openstack-neutron08:39
*** salv-orlando has joined #openstack-neutron08:42
*** vmtrooper has joined #openstack-neutron08:45
*** salv-orlando has quit IRC08:46
*** akamyshnikova has joined #openstack-neutron08:47
*** shivharis has joined #openstack-neutron08:48
*** shivharis has quit IRC08:49
*** jpena has joined #openstack-neutron08:50
*** irenab has quit IRC08:51
*** vmtrooper has quit IRC08:51
*** topshare_ has joined #openstack-neutron08:51
*** topshare_ has quit IRC08:51
*** irenab has joined #openstack-neutron08:52
*** alexpilotti_ has joined #openstack-neutron08:52
*** topshare_ has joined #openstack-neutron08:52
*** topshare has quit IRC08:53
*** alexpilotti has quit IRC08:54
*** alexpilotti_ is now known as alexpilotti08:54
*** yfried|afk is now known as yfried_08:55
openstackgerritElena Ezhova proposed openstack/neutron: Prevent calling waitall() inside a GreenPool's greenthread  https://review.openstack.org/15732008:58
*** veena has joined #openstack-neutron09:01
*** luqas has quit IRC09:02
*** devvesa has joined #openstack-neutron09:02
*** baoli has joined #openstack-neutron09:04
*** gildub has joined #openstack-neutron09:06
*** pradipta has quit IRC09:06
*** bobmel has joined #openstack-neutron09:09
*** ygbo has joined #openstack-neutron09:10
*** dims has joined #openstack-neutron09:11
*** apuimedo has joined #openstack-neutron09:16
*** kawa2014 has quit IRC09:16
*** kawa2014 has joined #openstack-neutron09:17
*** Murali has joined #openstack-neutron09:18
*** dims has quit IRC09:18
*** aix has joined #openstack-neutron09:19
*** markvoelker has quit IRC09:22
veenaHi, I cloned stable/icehouse code of neutron and ran ./run_tests.sh without changing any files. It has many errors, pasting the error in bin.  http://paste.openstack.org/show/191221/  http://paste.openstack.org/show/191222/  http://paste.openstack.org/show/191223/09:23
veenaHow do I solve these errors?09:23
*** neeti has quit IRC09:28
*** neeti has joined #openstack-neutron09:28
*** david-lyle_afk has quit IRC09:30
*** david-lyle_afk has joined #openstack-neutron09:30
anteayaI just checked the wikipage for the upcoming Neutron meeting: Monday (3/10/2015) at 1400 UTC on #openstack-meeting09:31
anteayaso it is helpful when the calendar date and the day of the week sync09:32
anteayaas they don't now and didn't last week09:32
*** kickinz1 is now known as kickinz1|afk09:32
*** kickinz1|afk is now known as kickinz109:33
*** aepifanov has joined #openstack-neutron09:33
*** jistr has quit IRC09:34
*** Flcn has joined #openstack-neutron09:34
*** yamamoto has quit IRC09:35
*** ihrachyshka has joined #openstack-neutron09:38
*** yuanying has quit IRC09:39
*** roeyc has joined #openstack-neutron09:40
*** adelinatuvenie has quit IRC09:42
*** Bhavya has quit IRC09:43
*** jistr has joined #openstack-neutron09:46
*** amotoki_ has joined #openstack-neutron09:48
*** amotoki_ has quit IRC09:48
*** lsg has quit IRC09:48
*** shivharis has joined #openstack-neutron09:48
*** shivharis has quit IRC09:50
openstackgerrityalei wang proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test  https://review.openstack.org/16005109:50
openstackgerrityalei wang proposed openstack/neutron: Add portsecurity extension support  https://review.openstack.org/12655209:50
*** afazekas has joined #openstack-neutron09:52
*** dims has joined #openstack-neutron09:56
*** popw has quit IRC10:00
*** xuhanp has quit IRC10:03
*** xuhanp_ has quit IRC10:03
*** _cjones_ has joined #openstack-neutron10:03
*** _cjones_ has quit IRC10:04
*** anish__ has quit IRC10:04
*** _cjones_ has joined #openstack-neutron10:05
*** trinaths has left #openstack-neutron10:07
*** jp_at_hp has joined #openstack-neutron10:08
*** _cjones_ has quit IRC10:09
*** ronis has joined #openstack-neutron10:11
*** marrusl has quit IRC10:14
*** marrusl has joined #openstack-neutron10:17
*** chlong has joined #openstack-neutron10:17
*** neeti has quit IRC10:18
*** irenab has quit IRC10:22
*** topshare_ has quit IRC10:25
*** irenab has joined #openstack-neutron10:25
*** yfauser has joined #openstack-neutron10:26
*** yfauser has left #openstack-neutron10:26
*** topshare has joined #openstack-neutron10:26
*** topshare has quit IRC10:28
openstackgerritIhar Hrachyshka proposed openstack/neutron: Migrate to oslo.log  https://review.openstack.org/15963810:30
*** jamie_h has joined #openstack-neutron10:31
*** aix has quit IRC10:32
*** coolsvap is now known as coolsvap|afk10:33
*** yfried_ is now known as yfried|afk10:34
*** vmtrooper has joined #openstack-neutron10:34
*** adelinatuvenie has joined #openstack-neutron10:35
*** pradip_vedams has quit IRC10:36
*** neeti has joined #openstack-neutron10:36
*** neeti has quit IRC10:36
*** chandankumar has quit IRC10:37
*** yfried|afk is now known as yfried_10:38
*** neeti has joined #openstack-neutron10:38
*** vmtrooper has quit IRC10:39
openstackgerritElena Ezhova proposed openstack/neutron: Prevent calling waitall() inside a GreenPool's greenthread  https://review.openstack.org/15732010:44
*** irenab has quit IRC10:47
*** salv-orlando has joined #openstack-neutron10:48
*** irenab has joined #openstack-neutron10:48
*** pradipta has joined #openstack-neutron10:48
*** shivharis has joined #openstack-neutron10:49
*** chandankumar has joined #openstack-neutron10:50
*** chandankumar has quit IRC10:50
*** chandankumar has joined #openstack-neutron10:51
*** chlong has quit IRC10:51
*** shivharis has quit IRC10:51
*** markvoelker has joined #openstack-neutron10:53
*** salv-orlando has quit IRC10:53
*** baoli has quit IRC10:54
*** stanzgy_ has quit IRC10:55
*** salv-orlando has joined #openstack-neutron10:55
*** stanzgy has quit IRC10:55
*** pask81 has quit IRC10:56
*** aix has joined #openstack-neutron10:57
*** pc_m has joined #openstack-neutron11:00
*** irenab has quit IRC11:01
*** kawa2014 has quit IRC11:04
*** moha_hunt has joined #openstack-neutron11:05
*** kawa2014 has joined #openstack-neutron11:05
*** alexpilotti_ has joined #openstack-neutron11:08
*** Krast has quit IRC11:09
*** alexpilotti has quit IRC11:10
*** alexpilotti_ is now known as alexpilotti11:10
*** Guest69217 has quit IRC11:13
*** Longgeek has quit IRC11:14
*** diegows has joined #openstack-neutron11:20
*** shaohe_feng_ has quit IRC11:20
*** alexpilotti has quit IRC11:21
*** kawa2014 has quit IRC11:21
*** kawa2014 has joined #openstack-neutron11:21
*** shaohe_robot has joined #openstack-neutron11:21
openstackgerritenikanorov proposed openstack/neutron: Don't notify dead DHCP agent of removed networks  https://review.openstack.org/16255911:26
*** yatin has quit IRC11:28
*** pradipta has quit IRC11:29
*** jlanoux_ has joined #openstack-neutron11:29
*** jlanoux has quit IRC11:32
*** absubram has quit IRC11:34
*** Longgeek has joined #openstack-neutron11:36
openstackgerritMerged openstack/neutron: Added check for emptyness where in_ is being used  https://review.openstack.org/15003411:41
pc_mmestery: ping?11:43
pc_msalv-orlando: ping11:44
salv-orlandohi pc_m11:44
*** yatin has joined #openstack-neutron11:44
pc_msalv-orlando: Can you review and approve https://review.openstack.org/#/c/162887/? Looks like UT breakage from a Neutron commit. Need a driver team member for another +2/+A11:44
pc_m(or I can +A)11:45
salv-orlandopc_m: ah is neutron-vpnaas completely blocked now?11:45
pc_myup11:45
salv-orlandopc_m: approved11:46
*** mwagner_lap has quit IRC11:46
pc_msalv-orlando: fails all jenkins jobs11:46
pc_msalv-orlando: Thank you!11:46
salv-orlandopc_m: np11:46
*** jlanoux has joined #openstack-neutron11:46
*** sbelous has quit IRC11:47
*** irenab has joined #openstack-neutron11:47
*** jlanoux_ has quit IRC11:49
*** shivharis has joined #openstack-neutron11:50
*** sbelous has joined #openstack-neutron11:50
*** yatin has quit IRC11:51
*** shivharis has quit IRC11:52
*** coolsvap|afk is now known as coolsvap11:54
*** Murali has quit IRC11:55
*** baoli has joined #openstack-neutron11:58
*** gildub has quit IRC12:00
*** karimb has joined #openstack-neutron12:01
*** yfried_ is now known as yfried|afk12:02
*** yatin has joined #openstack-neutron12:04
*** yfried|afk is now known as yfried_12:05
openstackgerritClaudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition  https://review.openstack.org/15325912:06
*** igordcard_afk has joined #openstack-neutron12:11
*** igordcard_afk has quit IRC12:11
*** igordcard has joined #openstack-neutron12:13
*** gildub has joined #openstack-neutron12:13
*** raildo_away is now known as raildo12:14
*** igordcard has quit IRC12:15
*** sambetts has quit IRC12:16
*** Tusker has quit IRC12:17
*** igordcard has joined #openstack-neutron12:17
*** thomasem has quit IRC12:18
*** thomasem has joined #openstack-neutron12:18
*** gridinv_ has joined #openstack-neutron12:19
*** bjornar has quit IRC12:20
*** sambetts has joined #openstack-neutron12:21
*** vmtrooper has joined #openstack-neutron12:23
*** gridinv_ has quit IRC12:23
*** dims has quit IRC12:25
*** gildub has quit IRC12:25
*** dims has joined #openstack-neutron12:25
*** mwagner_lap has joined #openstack-neutron12:26
*** neeti has quit IRC12:27
*** dane_leblanc has joined #openstack-neutron12:27
*** vmtrooper has quit IRC12:28
*** chandankumar has quit IRC12:35
*** amuller has joined #openstack-neutron12:37
*** vthapar has quit IRC12:38
*** markvoelker has quit IRC12:38
*** markvoelker has joined #openstack-neutron12:39
HenryGamotoki: ping12:39
mesteryFolks, just a reminder, with it being DST in the US, the Neutron meeting is 1 hour later this morning.12:41
* mestery shakes his angry fist at DST12:41
salv-orlandomestery: that obviously applies only to folks in the US12:42
salv-orlandoit's the same time for me and all the other communists out there I guess12:42
*** markvoelker has quit IRC12:43
ihrachyshka:)12:43
*** mwagner_lap has quit IRC12:43
pc_mmestery: ping12:44
*** mwagner_lap has joined #openstack-neutron12:44
ihrachyshkamestery, I also have a tiny thing to discuss with you. ping when avail please.12:45
mesterypc_m: pong12:47
mesteryihrachyshka: Good morning/afternoon!12:47
pc_mmestery: For VPN team discussion on meeting time/date, there were some votes for later time, one for same time, and everyone voted to have on-demand.  If we shift the time and have on demand, should I still reserve a day/time/channel slot, even though on demand?12:47
*** jistr has quit IRC12:47
mesterypc_m: If you do on-demand, yes, reserve a time so it's there when you need it, and then just make sure to email the list when you have the meeting and update the wiki page for it so people know.12:48
*** lionelz has joined #openstack-neutron12:48
*** jistr has joined #openstack-neutron12:48
pc_mmestery: OK. will do. Planning on seeing what time exactly to hold the meeting as 2 votes for 1600, 1 for 1630, 1 for 1500.12:48
ihrachyshkamestery, ok, assuming gm means 'I'm avail' :) I have remaining oslo.log patch for neutron in queue. 1) it would be great to review/merge it quickly since it's enormous in terms of potential conflicts 2) it breaks most CI jobs because those use code from neutron.openstack.common.log which is now dropped. Are we ok with it (they are not expected to use those modules)?12:48
mesteryihrachyshka: Lets bring it up in the meeting in 1 hour and discuss there. I am ok with this, but lets make sure people are aware before we drop it. Sound ok?12:49
*** britthouser has quit IRC12:50
*** shivharis has joined #openstack-neutron12:51
*** annegentle has joined #openstack-neutron12:51
*** ihrachyshka has quit IRC12:51
*** ihrachyshka has joined #openstack-neutron12:51
ihrachyshkamestery, sounds fine for me12:52
mesteryihrachyshka: Thank you!12:52
pc_mmestery: Does only commit owner have the ability to rebase via Gerrit button?12:52
openstackgerritKyle Mestery proposed openstack/python-neutronclient: Copy functional tests from tempest cli  https://review.openstack.org/16284812:52
*** shivharis has quit IRC12:53
pc_mhttps://review.openstack.org/#/c/162887 failed verify... looks like Jenkins is saying a rebase is needed.12:53
*** yfried_ is now known as yfried|afk12:54
salv-orlandoihrachyshka: For the vmware repo we're already aware of that and are doing a patch to fix it. As a general rule, we'll stop using neutron.openstack.common - and have our own openstack/common pkg for non graduated libraries12:54
mesterypc_m: Nope, others can as well.12:54
*** cbouch has joined #openstack-neutron12:54
pc_mmestery: I don;t see the button...12:54
salv-orlandothis brings on us synchronization onus, but removes a source of instability due to change in neutron.openstack.common12:54
mesterypc_m: You can just pull it down, rebase it, and push it back up.12:54
* mestery will BRB12:55
ihrachyshkasalv-orlando, right, thanks. I've tried to capture stuff like that that may interact with packaging (not only oslo graduations) at: https://wiki.openstack.org/wiki/Neutron/VendorSplitPackaging12:55
pc_mmestery: Right. Just wondered about the button. Will do.12:55
salv-orlandoihrachyshka: ah thanks a lot12:55
salv-orlandoI will pass that on to the guys that build our own internal packages12:55
ihrachyshkait's WIP and I'll fill it once I find more issues12:55
ihrachyshkathanks12:55
*** irenab has quit IRC12:57
pc_mmestery: Can you look at https://review.openstack.org/#/c/162887?12:57
pc_mmestery: It has X for Jenkins and says verify failed, needing rebase.12:57
pc_mmestery: I pulled it down, fetched origin, rebased, and tried to push up, and it says rejected (no new changes).12:58
salv-orlandoihrachyshka: I found annoying that Apple's mail auto correct insists of changing your name into "char"12:58
* pc_m confused12:58
ihrachyshkasalv-orlando, ok sir. I will consider changing it12:58
*** dboik has joined #openstack-neutron12:59
amotokiHenryG: pong13:04
HenryGamotoki: Will you update https://wiki.openstack.org/wiki/Neutron/NEC_OpenFlow_Plugin for the nec decomposition?13:04
*** jistr has quit IRC13:04
*** emagana has joined #openstack-neutron13:05
HenryGamotoki: Say yes so I can approve your decomp patch. :)13:05
*** jistr has joined #openstack-neutron13:05
*** coolsvap is now known as coolsvap|afk13:06
*** banix has joined #openstack-neutron13:06
amotokiHenryG: sure. It is in my mind, but it is just a bit lower priority than other stuff :-(13:06
HenryGamotoki: no problem13:06
amotokiHenryG: i need to move from now and will be back in the team meeting. I will update after that. Thanks.13:08
HenryGamotoki: No rush, your word is good. Thanks.13:08
*** cference has joined #openstack-neutron13:10
*** mdavidson has joined #openstack-neutron13:10
openstackgerritenikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations  https://review.openstack.org/14926113:12
*** amotoki has quit IRC13:12
*** markvoelker has joined #openstack-neutron13:13
*** yfried|afk is now known as yfried_13:14
mesterypc_m: Let me try13:15
pc_mmestery: I did a 'recheck' command, not sure if it would go through.13:15
*** anilvenkata has left #openstack-neutron13:16
*** cference has quit IRC13:16
*** cference has joined #openstack-neutron13:17
*** sripriya has joined #openstack-neutron13:20
amullersalv-orlando: ping about https://review.openstack.org/14926113:20
amullerif you happen to be up13:20
*** adelinatuvenie has quit IRC13:21
*** yfried_ is now known as yfried|afk13:24
*** changbl has quit IRC13:26
*** bjornar has joined #openstack-neutron13:26
*** rkukura has quit IRC13:27
*** yamamoto has joined #openstack-neutron13:28
*** yamamoto has quit IRC13:29
*** yamamoto has joined #openstack-neutron13:29
*** veena has quit IRC13:30
*** topshare has joined #openstack-neutron13:31
garykmestery: quick question - when i run tox on ovn it blows up. do you experience the same?13:32
mesterygaryk: Checking ...13:33
mesterygaryk: Exact command?13:33
*** ajo has quit IRC13:33
garykmestery: tox13:33
*** britthouser has joined #openstack-neutron13:33
*** yfried|afk is now known as yfried_13:34
mesterygaryk: Looks like the py26, py33 and py34 stuff dies, I'm going to submit a patch to remove those for now.13:34
*** adelinatuvenie has joined #openstack-neutron13:34
garykmestery: ok, thanks. but even when i do py27 it failed13:35
*** topshare has quit IRC13:35
mesterygaryk: Really? py27 worked fine for me (and pep8)13:35
*** tiswanso has joined #openstack-neutron13:36
*** saikrishna has joined #openstack-neutron13:36
garykmestery: please see http://paste.openstack.org/show/191278/ - maybe you can find what i did13:36
*** txomon|fon has joined #openstack-neutron13:36
*** txomon|fon has joined #openstack-neutron13:36
mesterygaryk: That looks like the same as for me, let me blow away my .tox and have it recreate13:37
*** rkukura has joined #openstack-neutron13:37
mesterygaryk: Worked fine for me: http://paste.openstack.org/show/191284/13:38
garykmestery: no idea…13:40
*** zhipeng has joined #openstack-neutron13:41
*** chandankumar has joined #openstack-neutron13:43
*** salv-orlando has quit IRC13:46
*** armax has joined #openstack-neutron13:46
garykmestery: can we start bitchin about not getting reviews in the ovn repo?13:47
mesterygaryk: lol13:47
garykdamn, i posetd a patch 30 secodns ago and have not got a review.13:47
mesterygaryk: rofl13:47
mesterygaryk: I just merged one, is that not fast enough? ;)13:47
openstackgerritMerged openstack/neutron: Improve performance of _get_security_group_member_ips  https://review.openstack.org/16257613:47
garykmestery: :)13:48
garykmestery: btw the hacking one is worth while adding to other repos that are external. we did it in the vmware_nsx and it found a ton of issues13:48
*** salv-orlando has joined #openstack-neutron13:48
mesterygaryk: I agree, I'll do the same for networking-odl, thanks!13:48
garykcool.13:48
mesterygaryk: networking-odl already had it, I forgot I added that a while back :)13:50
ihrachyshkasalv-orlando, may I ask you to check comments at https://review.openstack.org/#/c/159862/ ? I hope they address your concern, and we can merge.13:50
*** annegentle has quit IRC13:51
*** shivharis has joined #openstack-neutron13:51
*** shivharis has quit IRC13:53
*** pavel_bondar has quit IRC13:54
*** yamahata has quit IRC13:56
*** yamahata has joined #openstack-neutron13:57
*** r-daneel has joined #openstack-neutron13:58
*** marun has joined #openstack-neutron13:58
*** irenab has joined #openstack-neutron13:59
*** carl_baldwin has joined #openstack-neutron13:59
*** ajo has joined #openstack-neutron13:59
*** adelinatuvenie has quit IRC14:00
*** rushil has joined #openstack-neutron14:01
*** ajo has quit IRC14:01
*** watanabe_isao has joined #openstack-neutron14:01
*** ajo has joined #openstack-neutron14:02
*** jprovazn has joined #openstack-neutron14:02
*** ajo has quit IRC14:02
*** ajo has joined #openstack-neutron14:03
*** mlavalle has joined #openstack-neutron14:03
itzikbarmax: ping14:03
*** yshafrir has quit IRC14:03
*** saikrishna has quit IRC14:04
*** ajo has quit IRC14:04
*** shwetaap has joined #openstack-neutron14:05
*** ajo has joined #openstack-neutron14:06
*** britthouser has quit IRC14:07
*** annegentle has joined #openstack-neutron14:08
*** zzzeek has joined #openstack-neutron14:08
*** zhipeng_ has joined #openstack-neutron14:08
openstackgerritMerged openstack/neutron: NEC plugin code split  https://review.openstack.org/16210314:10
*** absubram has joined #openstack-neutron14:10
*** zhipeng has quit IRC14:10
*** annegentle has quit IRC14:11
*** wznoinsk has joined #openstack-neutron14:11
*** vmtrooper has joined #openstack-neutron14:12
*** adelinatuvenie has joined #openstack-neutron14:13
*** yfried__ has joined #openstack-neutron14:14
*** yfried_ has quit IRC14:15
*** ajo has quit IRC14:15
*** ajo has joined #openstack-neutron14:16
*** garyk has quit IRC14:16
*** shivharis has joined #openstack-neutron14:17
*** vmtrooper has quit IRC14:17
*** shivharis has quit IRC14:17
*** garyk has joined #openstack-neutron14:20
openstackgerritIhar Hrachyshka proposed openstack/neutron: Migrate to oslo.log  https://review.openstack.org/15963814:20
*** topshare has joined #openstack-neutron14:21
*** topshare has quit IRC14:21
*** amotoki has joined #openstack-neutron14:21
*** jckasper has joined #openstack-neutron14:22
*** topshare has joined #openstack-neutron14:22
*** topshare has quit IRC14:22
*** prad has joined #openstack-neutron14:23
*** ChuckC has quit IRC14:23
*** topshare has joined #openstack-neutron14:23
*** Guest75125 has joined #openstack-neutron14:30
*** emagana has quit IRC14:31
sbelousgaryk: Hi. I replied to your comments on this patch: https://review.openstack.org/#/c/154116/ can you review it, please?14:32
*** achanda has joined #openstack-neutron14:34
*** zz_jgrimm is now known as jgrimm14:35
*** carl_baldwin_ has joined #openstack-neutron14:35
*** carl_baldwin has quit IRC14:37
*** carl_baldwin_ is now known as carl_baldwin14:37
*** markvoelker has quit IRC14:37
openstackgerritMoshe Levi proposed openstack/neutron: Move mlnx agent to be under ml2/drivers/mlnx  https://review.openstack.org/16269314:37
*** markvoelker has joined #openstack-neutron14:37
*** absubram has quit IRC14:39
*** chandankumar has quit IRC14:39
*** garyk has quit IRC14:39
*** budnyjj has joined #openstack-neutron14:40
*** markvoelker has quit IRC14:42
*** pask81 has joined #openstack-neutron14:42
*** achanda has quit IRC14:43
*** Guest75125 is now known as notsogentle14:43
*** notsogentle is now known as agentle_14:43
*** rwsu has joined #openstack-neutron14:44
*** sballe__ has joined #openstack-neutron14:44
*** vishwana_ has joined #openstack-neutron14:44
*** xgerman_ has joined #openstack-neutron14:46
*** vishwan__ has joined #openstack-neutron14:46
*** vishwanathj has quit IRC14:48
*** markvoelker has joined #openstack-neutron14:49
*** vishwana_ has quit IRC14:49
*** SridharRamaswamy has joined #openstack-neutron14:50
*** jaypipes has quit IRC14:53
*** nyechiel_ has joined #openstack-neutron14:56
*** vishwan__ has quit IRC14:56
*** vishwanathj has joined #openstack-neutron14:57
*** anshul has quit IRC14:57
*** dims has quit IRC14:58
*** carl_baldwin has quit IRC14:59
*** nyechiel has quit IRC14:59
marundougwig: I'm not sure what you mean by same out of tree support...  wouldn't out-of-tree code need to be branched in the same way that neutron is?14:59
*** SridharRamaswam1 has joined #openstack-neutron15:00
*** SridharRamaswamy has quit IRC15:01
*** dimsum__ has joined #openstack-neutron15:01
*** fredlhsu has joined #openstack-neutron15:01
dougwigmarun: you can mirror the release branches, yes, but it's not needed now.  i have a driver package that works out of the box with icehouse, juno, and (soon) kilo, which is very handy.  maintaining separate branches just adds cost, and it's easier to put in a compatibility shim to select the right log import instead.15:01
banixenikanorov: mestery: Will there be a milestone attached to this bug which has been fixed: https://bugs.launchpad.net/neutron/+bug/137363415:01
openstackLaunchpad bug 1373634 in neutron "ML2 plugin bulk operations require multiple notifications being sent for sec group member updates" [Medium,Fix committed] - Assigned to Mohammad Banikazemi (mb-s)15:02
*** bquan has joined #openstack-neutron15:02
*** budnyjj has quit IRC15:02
ihrachyshkadougwig, it's not just import (at least not in all cases)15:02
dougwigmarun: pinning unit tests is on the lbaas meeting agenda, which starts in 58 minutes.15:02
ihrachyshkadougwig, option registration, library setup changed, audit dropped...15:03
marundougwig: ok, where? I'm happy to join.15:03
dougwigmarun: #openstack-meeting-415:03
marundougwig: re: driver, not sure you can escape multiple branches over the long term.15:03
dougwigmarun: https://wiki.openstack.org/wiki/Network/LBaaS15:03
dougwigmarun: i can also ping you when we get to that item.15:03
marundougwig: in-process dependency conflict would seem inevitable :/15:03
*** achanda has joined #openstack-neutron15:03
marundougwig: thanks15:04
*** ChuckC has joined #openstack-neutron15:04
dougwigmarun: yeah, my goal to be lazy is not something to base process around, agreed.15:04
ajosalv-orlando: quesiton, in the OVS agent, do you know if vlan mappings are consistent across agent restarts?15:04
marundougwig: Not saying that we shouldn't consider it, just that we'll need to understand better how to make it possible without too much cost.15:04
ajoquesiton? question...15:04
*** agentle_ has quit IRC15:05
marunajo: local vlan mappings are not15:05
ajohmmm, bad15:05
marunajo: they are stored in memory15:05
ajosalv-orlando ^15:05
marunajo: why?15:05
ajobecause there is a patch to re-use that mapping for connection tracking...15:05
marunajo: hmmmm15:05
marunajo: maybe I'm wrong...15:05
*** achanda has quit IRC15:05
ajoand that means, in-memory connection tracking in kernel marks, will out-sync from the marks we set in iptables15:05
* marun goes to investigate15:06
* ajo goes too15:06
dougwigmarun, ihrachyshka - it is making me wonder about putting oslo into a neutron-lib.  if entry points disappear quickly due to graduation, that's not really the kind of stable interface a lib should maintain.  so either we wrap those interfaces from a non-mechanically copied location, or i'm not sure we make a habit of publishing a library with disappearing15:07
dougwiginterfaces?15:07
marundougwig: yeah, maybe wrappers of some kind?15:07
*** Murali has joined #openstack-neutron15:07
marundougwig: that could be deprecated upon graduation to oslo library?15:07
ihrachyshkadougwig, oslo incubator libraries must not be considered stable15:08
marunihrachyshka: so we either have to provide a stable interface in neutron-lib or simply not expose15:08
dougwigihrachyshka: right, but by using them a dependencies in another repo, we are implicitly treating them as such.15:08
dougwigmarun: yes, that'd be one way.15:08
ihrachyshkadougwig, ideally, each project maintains its own copies of incubator modules. but for kilo, we avoided it for *aas repos since there were lots of switches this cycle, and it would make our life even more miserable15:09
marundougwig: if we don't expose, maybe it's enough for *aas to import oslo incubator themselves.15:09
ihrachyshkamarun, for neutron-lib, I would put it under neutron._openstack.common15:09
marunihrachyshka: ah, so maybe next cycle *aas start copying?15:09
ihrachyshkato indicate private nature of it15:09
marunihrachyshka: I guess the implication is that every repo needs to copy incubator15:09
dougwigi'd support the repos having their own oslo's.  it's more work, but it's mechanical work, and it severs dependencies, which are pernicious.15:10
ihrachyshkamarun, that may be the case, the number of modules from incubator is now manageable15:10
marunihrachyshka: e.g. neutron and neutron-lib and neutron-vpnaas15:10
marunihrachyshka: and then we have to hope that the implementations don't conflict15:10
marunhope -> trust our tests ? ;)15:10
dougwigit might be good to grep 'neutron.openstack.common' in all the new vendor decomp repos so we can see the scope of the bleeding.15:10
ihrachyshkamarun, we started with ~20 and now are around ~10, with most of them used isolated in neutron repo15:10
*** vivek-ebay has joined #openstack-neutron15:10
ihrachyshkamarun, yeah, implementation conflict is a scary thing15:10
ihrachyshkamarun, for libraries, we solve it with proposal bot15:11
ihrachyshkathat's why we should run from incubator modules as far as possible. those are evil.15:11
dougwigmarun, ihrachyshka - about to drive to work.  offline for a bit.15:11
marunihrachyshka: agreed15:11
marundougwig: ttiab15:12
openstackgerritrcurran proposed openstack/neutron: ML2 cisco_nexus MD: upstream updates required  https://review.openstack.org/16304015:12
*** tsufiev has quit IRC15:12
ajomarun, it looks to me like we don't have any persistence mechanism for the vlan mappings15:12
ajosalv-orlando ^ :/15:13
marunajo: we definitely don't persist it15:13
ajook, that's going to be an issue15:13
*** tsufiev has joined #openstack-neutron15:13
ajolooking again at the patch..15:13
ajo(https://review.openstack.org/#/c/118274/22)15:13
*** garyk has joined #openstack-neutron15:13
marunajo: https://github.com/openstack/neutron/blob/master/neutron/plugins/openvswitch/agent/ovs_neutron_agent.py#L68015:13
*** tsufiev has quit IRC15:14
marunajo: an ovs or agent restart will guarantee a fresh local vlan15:14
ajoahaa15:14
ajothanks maru15:14
*** yfried has joined #openstack-neutron15:14
marunajo: not great news, I know15:14
marunajo: is there a reason the non-local vlan can't be used?15:14
* marun goes to read patch15:14
*** vivek-ebay has quit IRC15:15
openstackgerritMerged openstack/neutron: Fix wrong log output in neutron/neutron/agent/linux/dhcp.py  https://review.openstack.org/16258515:15
garyksbelous: done15:15
ajomarun, that's hidden somewhere, there's a tiny ID limit for connection tracking marks on the kernel15:15
*** radez_g0n3 is now known as radez15:15
ajocan't remember it now exactly, but around 16bits15:15
*** vivek-ebay has joined #openstack-neutron15:15
marunajo: so there might be another option...15:15
marunajo: well15:16
sbelousgaryk: thanks!15:16
marunajo: any option requires persistence15:16
marunajo: it's only a question of where15:16
ajoI guess, yes...15:16
anteayamestery: I don't see edgar in channel15:16
anteayasc68cal: are you at the ops meetup?15:16
marunajo: I was thinking that we could add it to the port metadata that ovs retains15:16
marunajo: there's nothing stopping us from adding another field15:16
marunajo: that wouldn't be portable to lb, though.15:17
ajomarun, that sounds interesting15:17
anteayasc68cal: can you shed more light on what edgar was saying in the meeting about nova net and ipv6? it didnt' make any sense to me15:17
yamamotodvr folks around?15:17
ajomarun, btw, I think we can read the tag back from the ovs bridge, the same way we set it?15:17
*** yfried__ has quit IRC15:17
marunajo: we can, yes.15:17
ajoit would be the same, I guess15:17
marunajo: I'm not sure of the implications of doing so, though.15:17
*** chandankumar has joined #openstack-neutron15:17
ajomarun, you mean the metadata in ovs/ovsdb, or somewhere else?15:17
*** yatin has quit IRC15:17
*** yfried_ has joined #openstack-neutron15:17
marunajo: I mean, it should be possible to read all ports to reconstruct the table15:18
marunajo: we can id the ports based on metadata and then check their vlan tag15:18
ajomarun, yes, I think that makes sense15:18
sc68calanteaya: yep, I'm at the summit15:18
sc68callet me read scrollback15:18
ajomarun, I guess if we keep the net-id tied to the port metadata, in such case it would be very easy15:18
*** yfried has quit IRC15:18
*** phschwartz has quit IRC15:18
ajoget all net-id / tag pairs.... and build a dictionary15:18
ajowell.. build, the vlan mapping dict..15:19
marunajo: I don't think we keep the net-id, only the iface id15:19
ajohmm15:19
*** phschwartz has joined #openstack-neutron15:19
marunajo: but the iface/port id can be used to get the net-id easily enough15:19
sc68calanteaya: Basically we have one operator who is successfully using Nova-Network with IPv6. However, I believe they are using a different VIF driver than the default, which is bitten by https://bugs.launchpad.net/nova/+bug/125123515:19
openstackLaunchpad bug 1251235 in OpenStack Compute (nova) "IPv6 DAD failure due to hairpinning" [High,Fix released] - Assigned to Sean M. Collins (scollins)15:19
ajomarun, yes, but I guess it needs syncing it from neutron-server via RPC , which could be a good thing btw...15:20
*** topshare has quit IRC15:20
marunajo: it would need to be looked up, yes.15:20
ajomost probably we're doing it already...15:20
marunajo: I think so.15:20
sc68calThey also had some custom patches they are carrying internally around radvd management15:21
ajook, I will re-check the implications of the patch + no-map-persistence, and propose the solution we were talking about15:21
ajothanks marun15:21
amullermarun: Check out https://review.openstack.org/#/c/162811/15:22
marunajo: np :)15:22
marunamuller: ok15:22
amullerPrepare to have your testing mind blown15:22
*** boris-42 has quit IRC15:22
amullerin a very limited capacity of the term 'blown'15:22
anteayasc68cal: okay so _one- operator is not what I am doing to steer my efforts aaround15:22
openstackgerritTimothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config  https://review.openstack.org/15373315:22
anteayasc68cal: I'm not sure what messageing came out around this15:22
*** david-lyle_afk is now known as david-lyle15:23
marunamuller: impressive15:23
anteayasc68cal: but many people do many things that is great for them and has no impact on my life15:23
anteayasc68cal: this is one of those instances15:23
anteayasc68cal: and do tell edgar I am disappointed that a report at the meeting consisted of one data point15:23
anteayawithout having the context of being one data point15:23
anteayaand gives the impression we should redirect efforts15:24
amullermarun: not sure how this relates to your blueprint about having the test tree mirror the directory paths of the non-test tree15:24
amullerI lumped together the RPC tests in a single directory instead of having them spread around the tree15:24
*** tsufiev_ has joined #openstack-neutron15:24
sc68calanteaya: agree - I've only heard rumors of one operator in China doing IPv6 successfully. This now makes two reports.....15:24
amullerThat made more sense to me15:24
anteayasc68cal: please try to convey my sentiments as fostering operators to have inappropriate expectations of this work is highly frustrating for me15:24
garykclaudiub: please see https://review.openstack.org/16304715:24
anteayasc68cal: the whole point was to tell operators what we are doing15:25
anteayasc68cal: not pivot because someone is trying to do something15:25
marunamuller: it's only the unit test tree15:25
anteayasc68cal: I'm disappointed15:25
anteayasc68cal: not with you but edgar isn't in channel15:25
amullermarun: ok, I had it in the unit test tree in the beginning, the only reason I moved it to the functional tree is because of semantics... The tests dont need any special setup, they can be run on your laptop like unit tests15:25
anteayasc68cal: thanks for being available15:26
amullermarun: But I don't think those tests are 'unit' tests by some definitions15:26
*** imcsk8 has quit IRC15:26
*** imcsk8 has joined #openstack-neutron15:26
marunamuller: there are different kinds of non-unit tests, but unit tests should only be locally targeted imho15:26
sc68calanteaya: Edgar is running a session on network performance, hence his unavailability15:26
maruni.e. at functions in individual modules15:26
sc68calanteaya: I'm two feet away from him, so I will have him ping you when the session is over15:27
amullerright15:27
marunamuller: functional is a grab-bag for everything else at this point15:27
anteayaplease point him at the log so he can read my words when his if finished, if you could15:27
sc68calanteaya: will do15:27
*** zhipeng_ has quit IRC15:27
anteayasc68cal: thanks, any idea what time? I'm trying to squeeze in breakfast15:27
marunamuller: we may want to segment them at some point, but it's not clear to me how15:27
marunamuller: so, your proposed structure lgtm15:27
sc68calanteaya: it just finished, so I will have him reach out now15:27
amullermarun: Working on more RPC tests now, I hit a snag with the OVS agent and way it sets up its RPC listeners... As it is you have to instantiate an OVS agent, and that would require a lot of ugly mocks15:28
*** sripriya has quit IRC15:28
openstackgerritdql proposed openstack/neutron: fix DHCP port changed when dhcp-agent restart  https://review.openstack.org/16252215:28
marunamuller: Please don't hesitate to refactor as necessary.15:28
anteayasc68cal: okay thanks15:28
*** emagana has joined #openstack-neutron15:28
marunamuller: refactoring code for testability > making tests unnecessarily complicated15:28
*** sripriya has joined #openstack-neutron15:28
amullermarun: I agree, just not entirely sure how. My first thought was to extract the setup_rpc method, make it a class method or something, but it would return like 15 variables if I did that15:29
amullerI don't know, I'll think about it15:29
marunamuller: object?15:29
*** numan has quit IRC15:29
amullerperhaps15:29
claudiubgaryk: sure, looking15:29
marunamuller: my advice is useless without actually digging into code, of course15:29
*** ChuckC has quit IRC15:30
emaganasorry folks.. I had to moderate a session at the ops meetup and could not attend the rest of the neutron meeting15:31
garykthanks15:31
*** yog__ has quit IRC15:31
*** bobmel has quit IRC15:32
*** Murali has quit IRC15:32
*** carl_baldwin has joined #openstack-neutron15:32
anteayaemagana: hi15:32
*** irenab has quit IRC15:32
emaganaanteaya: Hello!15:33
anteayaemagana: so I'll put my cards on the table15:33
*** bobmel has joined #openstack-neutron15:33
emaganaanteaya: ?15:33
anteayaemagana: what you presented at meeting really upset me15:33
anteayaso let's start with this15:33
anteayawhat are you telling people about the nova net to neutron migration15:33
*** carl_baldwin has quit IRC15:33
emaganaanteaya: sorry to hear that, I am just the voice of what I heard. I am pretty sure there is some misunderstanding and we can work it out15:34
anteayalet's do so now please15:34
anteayaas I would really like operators to have reasonable expectations15:34
anteayaso do you want to start?15:34
emaganaanteaya: I share the gerrit review and I asked around for feedback.15:34
anteayaI can listen first15:35
anteayaah15:35
anteayano15:35
anteayaI'm not interested in feedback15:35
anteayaI'm interested that they know15:35
anteayathis has been going on for 3 years15:35
*** adelinatuvenie has quit IRC15:35
anteayanone of them will like it15:35
*** sripriya has quit IRC15:35
anteayaand I accept that15:35
emaganaanteaya: ok15:35
anteayathis is what we are doing, end of story15:35
*** moshele has quit IRC15:35
*** changbl has joined #openstack-neutron15:35
anteaya_now_ should someone want to get invovled15:35
anteayathat is wonderful15:36
anteayaand they are welcome15:36
*** evgenyf has joined #openstack-neutron15:36
anteayabut a bunch more storis of hwo what we are doing doens't suit them is not new nor is it helpful15:36
anteayahow15:36
emaganaanteaya: That was actually my point, I believe that having them involved in the process will make it better15:36
anteayaso anyone who wants to participate is welcome to attend the weekly meeting15:36
anteayatuesdays at 0900 utc15:36
anteayaand there is an agenda15:36
anteayaand they can review patches and test15:37
anteayaanyone who tests I will listen to15:37
emaganaanteaya: Let me add that to the etherpad15:37
anteayathank you15:37
*** nyechiel_ has quit IRC15:37
anteayaif they want to be involved they can show up to the meeting and test code15:37
anteayathat is the feedback I will listen to15:37
emaganaanteaya: we did not spend more that two minutes on the topic.. most of the feedback was related to neutron missing some features that nova-network provides15:37
anteayaokay well that is useful to15:38
anteayabut not in the context of the migration story for me15:38
sc68calI felt like the operators are moving toe goalposts15:38
anteayabut helpful for the tc15:38
anteayasc68cal: that is what they do15:38
sc68calfirst they wanted multi_host , we got it via DVR15:38
anteayawhich is why this has taken 3 years15:38
sc68calthey don't like OVS now, that's the new object of dislike, they now demand it for linux bridge15:38
anteayaand also why I only will listen to feedback to folks who show up to meetings and test code15:39
anteayasc68cal: colour me surprised15:39
sc68calI was glad it was only a couple minutes, I felt like neutron was the whipping boy15:39
emaganaanteaya: Let's make an extra effort to get these people on the summit sessions15:39
anteayano15:39
*** nyechiel_ has joined #openstack-neutron15:39
anteayathere will be no summit session about the migration15:39
marunsc68cal: haters gotta hate ;)15:39
anteayaI want table time on friday15:39
*** boris-42 has joined #openstack-neutron15:39
marungotta -> gonna (oops)15:39
sc68calmarun: haters ball was in session15:40
anteayabut I have zero interest in have a room full of 400 people saying you need to make _me_ happy15:40
anteayaI encourage table time on friday15:40
sc68calanteaya: +115:40
anteayafor those who want to _work_15:40
anteayaworkers welcome15:40
emaganaanteaya: sounds like a good idea!15:40
marunanteaya: I don't really understand how ops should be driving this anyway15:40
marunanteaya: works takes resources15:41
marunanteaya: asking for things without providing resources?  not going to accomplish anything15:41
anteayaemagana: thank you15:41
anteayaemagana sc68cal so do invite anyone who cares to the migration meeting tuesdays at 0900 utc in -meeting15:41
claudiubgaryk: done15:41
emaganaanteaya: sorry if this part got you upset, by not means I want people to get mad15:41
anteayafolks who show up get heard15:41
sc68calanteaya: I will communicate that to operators here15:42
anteayaemagana: no I understand, it just don't see the point in trying to make them happy15:42
anteayawe won't15:42
anteayaI have accepted that15:42
anteayasc68cal: thanks15:42
anteayamarun: all fair points15:42
anteayaokay thanks for your time and the channel space15:43
sc68calI brought up in the first session of today that Operators need to be invovled in specs15:43
anteayaI need to get something to eat now15:43
anteayahow did that go over?15:43
sc68calGood15:43
*** absubram has joined #openstack-neutron15:43
anteayagood15:43
emaganaanteaya: I got it.. anyway, this is just a small and very small number of all openstack operators looking for a migration plan.. so, it should not affect the work at all15:43
anteayaglad to hear it15:43
garykclaudiub: thanks!15:44
anteayaemagana: thanks15:44
sc68calsuggestion by Stefano Maffulli  was to get specs sent to the operator's list15:44
anteayaemagana: glad we could sync up15:44
sc68calsort of like how we get notifications in IRC15:44
emaganaanteaya: anytime!15:44
sc68calmaybe on first patch15:44
anteayasc68cal: ah I doubt that will happen15:44
anteayasc68cal: better find monty before you broadcast that idea15:44
*** aepifanov has quit IRC15:44
anteayathey can just subscribe to email notifications on the specs projects of their choice15:45
sc68calgood point15:45
anteayahttps://review.openstack.org/#/settings/projects15:45
emaganawhoever is really interested in the specs, they can do it.. all is open15:45
anteayawatch a project, set email notifications15:45
marunsc68cal: I'm not really sure how drive-by review by ops is going to help15:45
anteayateach them how to do that themselves15:45
*** watanabe_isao has quit IRC15:45
marunsc68cal: we need people who build relationships with the community15:45
marunso that we know we can trust them15:46
sc68calmarun: This is true, my hope was that having them review specs would help that15:46
*** thedodd has joined #openstack-neutron15:46
sc68calOr at least get those introductions15:46
amullerMulti host for LB is very wrong from Neutron's point of view15:46
marunsc68cal: it could, as a first step, true.15:46
amullerI hope no one is getting ideas here =D15:46
marunamuller: wrong because we don't have the manpower?15:46
marunsc68cal: what we're really lacking is engineering management15:46
marunsc68cal: and product management15:47
*** kawa2014 has quit IRC15:47
*** changbl has quit IRC15:47
marunsc68cal: we're so developer-centric that we as a community frequently lose sight of the bigger issues15:47
garykclaudiub: can we approve https://review.openstack.org/#/c/156980/?15:47
*** kawa2014 has joined #openstack-neutron15:47
marunsc68cal: getting ops involved might help to some degree, but I think the root cause of our dysfunction will remain unaddressed15:47
marunsc68cal: that's not to say that I don't think ops is important...15:48
marunsc68cal: I think the huge advantage aws/google/etc have is the tight feedback loop between dev and ops.15:48
amullerWrong because we already have a remarkably invasive and hard to maintain distributed routing solution and introducing a second one for a vast miniority because of arbitrary reasons is a huge misuse of developer time. Frankly, we can barely seem to be able to support DVR as it is, I don't think we have the manpower to support another spin off of it15:48
marunamuller: manpower it is :)15:48
amullerIt's not only that but this is a moot discussion anyway15:49
*** yamamoto has quit IRC15:49
marunamuller: agreed15:49
*** garyk has left #openstack-neutron15:51
sc68calmarun: Yes, those are very good points15:53
*** yamamoto has joined #openstack-neutron15:53
sc68calmarun: Do you want to run for BDFL?15:53
sc68cal;)15:53
marunsc68cal: I'm definitely not the man for the job.15:54
marunsc68cal: But I would certainly appreciate some top-down vision to go with our bottom-up organization.15:54
emaganamy opinion on this topic is that this is why all OpenStack projects need a Product Manager Team, who will collect the received feedback vs the development team efforts and drive the specs15:55
marunemagana: I think that's a solution in search of a problem.15:55
marunemagana: there is, in fact, a product managers mailing list and a group of them presume to be able to save openstack from itself.15:55
openstackgerritMerged openstack/neutron: Update default tox envlist to match voting gate checks  https://review.openstack.org/13751615:55
emaganabut that will convert us in a product-based program and not into an opensource team..15:55
marunemagana: but we're a developer-centric culture15:55
*** bandarji has joined #openstack-neutron15:56
marunemagana: it can't just be imposed15:56
*** banix has quit IRC15:56
emaganamarun: I do agree!15:56
marunemagana: right15:56
*** lazy_prince is now known as killer_prince15:56
emaganamaybe we could find some common ground!15:56
marunI think we need cohesive vision and effective management. I'm not sure how to achieve that when everything is by consensus.15:57
emaganaI do nto know.. I like to hear the operators community because I like to see the work of the development team in production systems..15:57
marunemagana: it's definitely good to be exposed to different perspectives15:57
marunemagana: I don't think enough of us know what running a cloud is really like such that it could drive our development priorities.15:57
claudiubgaryk: i don't have permission to see the page, for some reason15:58
haleybthere are some developers here that are also operators :)15:58
emaganamarun: but what is the point of developing something that will not be used?15:58
marunemagana: I hear that hp may be donating it's defunct tripleo resources to infra, so that may be changing15:58
marunemagana: such that we'll be able to run ci on our openstack cloud15:58
*** SridharG has left #openstack-neutron15:58
emaganamarun: that will be awesome!15:59
marunemagana: that might allow developers more exposure to ops in a way that is meaningful to them.15:59
openstackgerritDoug Wiegley proposed openstack/neutron: Add proposal-bot-esque job for updating child repo neutron pinning  https://review.openstack.org/16306215:59
* marun crosses his fingers15:59
*** crose has joined #openstack-neutron15:59
marundougwig: you don't waste any time :)15:59
*** markvoelker has quit IRC15:59
emaganawill be around at the ops meetup but if someone wants to hear more just ping me!15:59
marunemagana: have fun!16:00
*** kbringard has joined #openstack-neutron16:00
*** markvoelker has joined #openstack-neutron16:00
dougwigmarun: i guess i don't need to paste that to you.  :)16:00
amullerThis ties in to the drivers team in my opinion. There's obviously an issue with a group of developers deciding which features should be given -2, when there's no feedback from users, customers (Or product managers). This cycle we *kind of* avoided the problem by having a cycle where we focused on technical debt, which developers love (Yay refactoring), so there wasn't a lot of complaint. But what happens in the next 2-3 cycles where16:00
amullerdrivers have to decide on more technical features, I don't know. Things like NFV related features, distributing services, who's to decide what is more important, without feedback and a more transparent process?16:00
pc_mzhhuabj: I just saw it fail once, when I tried strongswan. Will do more runs today.16:01
marunamuller: we're a community16:01
*** vmtrooper has joined #openstack-neutron16:01
marunamuller: if participants want focus on things, they need to drive consensus support16:01
marunamuller: it's not magic16:01
zhhuabjpc_m, I just test it in ubuntu 14.10 with strongswan driver and ubuntu 14.04 with openswan driver, I haven't test in ubuntu 14.04 with strongswan driver,16:02
marunamuller: All that to say that feedback isn't enough.16:02
*** banix has joined #openstack-neutron16:02
*** fredlhsu has quit IRC16:02
zhhuabjpc_m, I will test this case (ubuntu 14.04 + ss driver) as well today16:02
amullerThat's a cop out answer... Some specs were approved, others rejected, where there was no more or less 'community support'. At this point, the stuff that wasn't related to technical debt, was very much arbitrary.16:02
marunamuller: a cop out answer?16:02
marunamuller: we're a community.16:02
marunamuller: we're not a top-down engineering organization16:02
*** nlahouti has joined #openstack-neutron16:03
*** changbl has joined #openstack-neutron16:03
matrohonroaet, roaet_ : ping16:03
marunamuller: the goal is to make neutron lighter-weight to minimize the kind of arbitrary decision-making you're complaining about16:03
*** yfried_ has quit IRC16:03
roaetmatrohon: hi16:03
roaetroaet_: ?16:03
roaeti have a clone!16:04
marunamuller: but so long as something needs to happens in-tree, it has to be balanced against perceived priorities.16:04
matrohonroaet : roaet_ seems on the room too!16:04
*** emagana has quit IRC16:04
matrohonroaet : I'd like to use the capacity to extend the neutron client that you recently merged16:04
*** markvoelker has quit IRC16:05
roaetmatrohon: cool16:05
marunamuller: how would you propose we make those decisions without relying on the relationships among the interested parties?16:05
*** saikrishna has joined #openstack-neutron16:05
matrohonroaet : it looks suitable to my need, thanks to you16:05
amullerI'm very much aware that we are a community. I'm saying that some specs that were approved had no more community support than some specs that were given -2 for arbitrary reasons. The notion of 'community support' can be arbitrary in itself sometimes and at the very least, it's very hard to measure if we're not quantifying it. The drivers team made no effort to explain what are the perceived priorities for areas that did not revolve16:05
amulleraround technical debt and why some specs were cut while others were not. Just throwing around 'community support' as a magic explanation can be a cop out.16:05
*** vmtrooper has quit IRC16:06
marunso you'd ignore the fact that some people have authority in this project by virtue of trust and technical competance?16:06
roaetmatrohon: well I'm glad it helps. did you need information on getting an extension working?16:06
marunor the fact that the ptl was elected and gets to guide this decision-making process as he sees fit?16:06
matrohonroaet : you've also proposed an example : fox_sockets16:06
matrohonroaet : yep16:06
*** gongysh has joined #openstack-neutron16:06
marunprocess is *not* a substitute for culture16:06
maruncompetance -> competence16:07
matrohonroaet : do you already know a stand alone project that use it by adding it to its entry points?16:07
*** gongysh has quit IRC16:07
roaetmatrohon: a stand-alone entrypoint example? one doesn't exist yet but I'm working on one right now16:08
marunamuller: everyone is free to complain, but it would be better to work towards something better.16:08
matrohonroaet : for the quark client?16:08
roaetmatrohon: indeed16:08
*** yamamoto has quit IRC16:08
*** emagana_ has joined #openstack-neutron16:08
roaethttps://github.com/roaet/quark_python_neutronclient_ext16:08
amullermarun: I'm in favor of stronger leadership on basis of trust and technical competence and I think that the people currently in the drivers team have proven themselves. I agree with you that his project lacks vision, that's very appearent. Perhaps getting rid of a lot of stuff that we know nothing about will mitigate that issue. What I am saying, is that the process being used to decide on +2 and -2 for specs needs to be taken more *much*16:09
amullermore seriously (As opposed to sitting down at a mid cycle for a few hours) *and* be more transparent.16:09
*** itzikb has quit IRC16:09
roaetit has been backburnered but I can make the modifications to it quickly if that'd help you out matrohon16:09
amullerApproving and declining specs is a massively important responsibility, it shapes the project for the years to come16:09
marunamuller: have you seen how many specs get proposed?16:09
marunif there's a problem, it's allowing an open submission process16:09
amullermarun: I understand that this is a massive undertaking, and still the filtering process should take more than an afternoon...16:10
matrohonroaet : cool! yep it would surely help!16:10
amullerIt should be treated with respect16:10
marunamuller: seriously?16:10
marunamuller: you think that drivers spend only an afternoon?16:10
*** agentle_ has joined #openstack-neutron16:10
marunIt can take an afternoon per-spec!16:10
amullermarun: You know exactly what I'm specifically referring to and what I'm not, no need to twist my words16:11
amullerI've spent hours reviewing single specs of course I know what is involved16:11
marunamuller: The point is that arbitrary decision-making is part of the spec process16:11
marunamuller: we can't give due process to all of them16:11
*** pal has joined #openstack-neutron16:11
roaetmatrohon: you wouldn't also happen to need auth extensions? ;)16:11
marunamuller: the only way out of this mess is limiting the scope of what has to have our approval in the first place16:12
*** apuimedo has quit IRC16:12
*** SridharRamaswam1 has quit IRC16:12
maruni.e. focus on enabling out-of-tree development16:13
zhhuabjpc_m, seems need ensure_process in _delete_vpn_processes, what do you think ?16:13
amullerI can't argue with those two statements and I disagree on a fundemental level. Even after the advanced services and vendors split you're going to have a massive amount of specs to look at. 'arbitrary decision making' should NOT be a part of the spec process.16:13
*** changbl has quit IRC16:13
matrohonroaet : no auth extension, only extensions to my service_plugin API! IFAIU the code of the neutronclient has the fox_sockets example embedded : shouldn't I be able to run 'neutron fox-sockets-list'?16:13
roaetmatrohon: no it was taken out by request. It's only there for unit testing and as an example.16:14
pc_mzhhuabj: checking16:14
roaetIt's not loadable because of the _ in front of it16:14
marunamuller: I wish I could live in the world you're describing.16:15
amullermarun: Look, this is a scale, not a binary process16:15
amullerYou can't go through every spec on the same level16:16
amullerBecause that'd take possibly longer than a single cycle16:16
pc_mzhhuabj: It seems like it is creating the process and then turning around and deleting it. No?16:16
matrohonroaet : ok! I'll have a look your UT, I hope it will help me to better understand to stevedore magic!16:16
*** emagana_ has quit IRC16:16
pc_mzhhuabj: I don't see why ensure_process() call is needed. If it already exists, it'll delete it. If not, then there will be nothing to do. Or am I missing something?16:17
roaetmatrohon: I have a random auth example that shows how entry points work16:17
*** emagana has joined #openstack-neutron16:17
roaethttps://github.com/rackerlabs/rackspace-auth-neutronclientext/blob/master/setup.py16:17
marunamuller: agreed16:17
amullermarun: But a) We could improve, and b) The process itself is opaque and seems arbitrary from the outside. Even if you don't spend more time on examining a  spec on technical grounds, the way to communicate a denial can be made more humane.16:17
roaetmatrohon: the EP are put in setup.py I believe.16:17
marunamuller: could things be better?  yes.16:18
marunamuller: I'm not going to argue against improving things.16:18
amullermarun: I'm trying to point out two issues, the first one I cannot do much about, but the 2nd point is communication, which is very actionable16:18
marunamuller: But the idea that things are arbitrary and don't have to be is, to me, a cop-out.16:18
marunThey *do* have to seem arbitrary at some level.16:18
zhhuabjpc_m, check again, make sense16:18
marunBecause we get so many useless submissions of specs or code from people that don't understand wtf is going on...16:19
*** ronis has quit IRC16:19
marunAnd any process, no matter how well reasoned, is going to seem arbitrary to those folks because they aren't engaged enough to undertand how to contribute.16:19
*** pal has quit IRC16:19
pc_mmestery: ping16:20
ihrachyshkamarun, I think the point is to split technical grounds for denial from organizational ('we don't have cycles for this')16:20
*** banix has quit IRC16:21
marunihrachyshka: I'm not sure it's always obvious where that line lies.16:21
*** emagana has quit IRC16:21
*** kbringard has quit IRC16:21
openstackgerritCarl Baldwin proposed openstack/neutron: Move create_dvr_fip_interfaces in to DVR  https://review.openstack.org/15966316:21
ihrachyshkamarun, just curious whether the arbitrary part of decision making is because we limit ourselves in time when we consider new specs (?)16:22
matrohonroaet : EP has to be registered in the setup.cfg of one project, which will be translated by pbr in an entry in project.egg-info/entry_points.txt, I think16:22
marunihrachyshka: that is likely part of it, yeah.16:22
marunihrachyshka: the other part is that anyone can submit a spec16:22
ihrachyshkamarun, do you want some ATC kind of limit for that?16:22
marunihrachyshka: Something like that, yeah. A trust relationship between the proposer and the community is suggested.16:23
ihrachyshka(in an ideal world, I would actually like if people would need to earn some points doing bug fixing before getting a chance for spec submit)16:23
matrohonroaet : I did enter an entry in my setup.cfg, for the neutronclient.extension namespace, but the neutron client doesn't load my extension16:23
ihrachyshkabecause in the end, if you haven't done any low hanging fruit kind of work for the community, it's not clear whether you'll be effective when doing submissions for new features16:24
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543616:24
mesterypc_m: pong16:25
*** carl_baldwin has joined #openstack-neutron16:25
pc_mmestery: I just posted to QA IRC, but that 162887 job is still seeing strangeness.16:25
pc_mI couldn't rebase, with it saying no difference. Then I did a recheck, and it failed. Someone else did a recheck and it passed.16:26
pc_mit is in gate and is showing failing docs, pep8, pylint.16:26
pc_mmestery: This is a really simple change too... I'm clueless as to what is going on.16:27
carl_baldwinsalv-orlando: ping16:27
*** sbfox has joined #openstack-neutron16:27
pc_mmestery: Is it merging in other changes from the queue and that is causing issues?16:27
mesterypc_m: Me too, I couldn't rebase either when I tried16:27
mesteryVery strange16:27
*** scheuran has quit IRC16:28
marunihrachyshka: agreed.  we do need to get better at mentoring new contributors and having sane things for them to work on.16:28
*** tiswanso has quit IRC16:30
*** dboik_ has joined #openstack-neutron16:30
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284016:30
roaetmatrohon: I'll start working on my extension soon and see if I can help you out16:30
amullerMentoring in a community such as ours is problematic. I think that additional devref is more scalable (Cough: https://review.openstack.org/#/c/150918/)16:30
amuller+ Testing doc16:30
amuller+ DVR doc16:31
pc_mmestery: Any idea on how to approach this issue, or who to talk to?16:31
amullerand take it from there...16:31
*** tiswanso has joined #openstack-neutron16:31
matrohonroaet : If I rename the file _fox_socket to fox_soccket, the extension should work?16:31
mesterypc_m: Best bet is to see if clarkb or fungi has some time to help you I think. anteaya may be able to help find someone as well.16:31
roaetmatrohon: sadly no.. I forgot, but I was asked to remove contrib loading support16:31
pc_mmestery: Thanks. will do...16:31
*** sripriya has joined #openstack-neutron16:32
*** dboik has quit IRC16:33
matrohonroaet : ok, the neutronclient needs an update so... at least a removal of the contrib dir16:34
*** baoli has quit IRC16:34
roaetmatrohon: we were able to leave it there to keep the example extension16:34
matrohonroaet : of fine... it's pretty confusing given your commit message, which mentions the use of the contrib dir. thanks for the info, it'll help :)16:35
*** baoli_ has joined #openstack-neutron16:36
amullercarl_baldwin: I left 1 comment on the internal interfaces patch16:36
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284016:36
roaetmatrohon: the situation by which that patch was merged was complicated.. I think the commit message was a victim of that, heh.16:37
mesterymarun: I'm trying to run the (currently experimental) functional job for the python-neutronclient, and it's been sitting waiting for a node for 3.5 hours. Is this normal for experimental jobs?16:37
*** jlanoux has quit IRC16:38
*** _cjones_ has joined #openstack-neutron16:39
openstackgerritPavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM  https://review.openstack.org/15323616:39
*** yamahata has quit IRC16:39
*** jorgem has joined #openstack-neutron16:40
*** jaypipes has joined #openstack-neutron16:40
matrohonroaet : thanks for the discussion, I'll keep on digging into that, and I'll wait for the quark extensions16:41
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284016:42
*** harlowja_away is now known as harlowja_16:44
marunmatrohon: unfortunately, yes.  the experimental queue is terrible16:45
marunoops, mestery ^^16:45
mesterymarun: #sadpanda16:45
marunmestery: I'd recommend adding it non-voting to the check pipeline instead16:46
*** kickinz1 is now known as kickinz1|afk16:46
marunmestery: that's the only way we were able to have reasonable turnaround when working on the api and functional jobs16:46
*** dimsum__ has quit IRC16:46
marunmestery: the biggest flaw in our current ci scheme is not being able to validate config changes by running actual jobs16:47
mesteryyeah16:47
mesteryit sucks16:47
*** dimsum__ has joined #openstack-neutron16:47
mesterymarun: So I guess I sit and wait :(16:48
*** ihrachyshka has quit IRC16:48
*** amotoki has quit IRC16:51
*** amotoki has joined #openstack-neutron16:51
*** amotoki has quit IRC16:51
*** marun has quit IRC16:52
amullermarun: There might be an issue with 'cast' and the Oslo messaging fake driver, I'll look in to it after lunch. Right now I'm unable to get a test where the receipient actually receives a 'cast'.16:52
amulleronly 'calls'16:52
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238316:52
*** mlavalle has quit IRC16:52
*** vivek-ebay has quit IRC16:53
*** nmagnezi_ has quit IRC16:55
*** jorgem has quit IRC16:57
*** marun has joined #openstack-neutron16:57
*** agentle_ has quit IRC16:58
openstackgerritenikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations  https://review.openstack.org/14926116:59
*** jistr has quit IRC17:01
carl_baldwinamuller, ack17:01
*** leenheer has joined #openstack-neutron17:02
*** yamahata has joined #openstack-neutron17:03
*** banix has joined #openstack-neutron17:04
*** vivek-ebay has joined #openstack-neutron17:04
*** sbadia has quit IRC17:04
*** _cjones_ has quit IRC17:05
*** sbadia has joined #openstack-neutron17:05
*** ivar-lazzaro has joined #openstack-neutron17:05
*** changbl has joined #openstack-neutron17:06
*** vivek-ebay has quit IRC17:06
*** Longgeek has quit IRC17:06
*** jaypipes has quit IRC17:07
*** vivek-ebay has joined #openstack-neutron17:07
*** britthouser has joined #openstack-neutron17:08
*** markvoelker has joined #openstack-neutron17:09
*** _cjones_ has joined #openstack-neutron17:09
*** yamamoto has joined #openstack-neutron17:09
*** pask81 has quit IRC17:09
*** ygbo has quit IRC17:09
*** aepifanov has joined #openstack-neutron17:11
*** yfried_ has joined #openstack-neutron17:12
*** britthouser has quit IRC17:13
*** britthouser has joined #openstack-neutron17:13
*** yamamoto has quit IRC17:14
*** Akanksha08 has joined #openstack-neutron17:14
*** krtaylor has quit IRC17:14
Akanksha08Hi rossella_s17:14
*** kbringard has joined #openstack-neutron17:15
*** shwetaap has quit IRC17:17
*** melwitt has joined #openstack-neutron17:18
*** markvoelker has quit IRC17:19
*** achanda has joined #openstack-neutron17:19
*** achanda has quit IRC17:19
*** markvoelker has joined #openstack-neutron17:19
*** achanda has joined #openstack-neutron17:19
*** britthouser has quit IRC17:20
*** yfried_ has quit IRC17:21
*** yfried_ has joined #openstack-neutron17:21
*** britthouser has joined #openstack-neutron17:21
*** emagana has joined #openstack-neutron17:21
*** changbl has quit IRC17:21
*** markvoelker_ has joined #openstack-neutron17:21
openstackgerritClaudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition  https://review.openstack.org/15325917:21
openstackgerritChuck Carlino proposed openstack/neutron: Prevent updating mac address of bound port  https://review.openstack.org/16250017:22
*** markvoelker has quit IRC17:24
*** sbfox has quit IRC17:24
dougwigmarun, mestery - is the non-voting check queue a good alternative?  i dread trying to iterate in the experimental queue.17:24
marundougwig: I think so17:24
*** changbl has joined #openstack-neutron17:24
marundougwig: it does add gate overhead, but the alternative is so much worse17:24
mesteryThe experimental queue sucks17:25
mesteryMy client patch is still rotting in there17:25
mesteryAlmost 5 hours now17:25
mesterydougwig: ^^^17:25
dougwigmarun: have you chatted with infra about it?17:25
mestery#sadpanda17:25
marunI'm not entirely sure why the experimental queue doesn't get more resources17:25
*** devvesa has quit IRC17:25
marundougwig: I had words at one point17:25
marundougwig: the tripleo experimental queue came back super quick and I was wondering wtf17:25
dougwiglet me see about tilting at that windmill.17:26
marundougwig: it didn't seem that anyone was concerned about the experimental queue being slow, presumably because none of the people I was talking to were affected by it17:26
marundougwig: but it's definitely worth a revisit17:26
*** sbfox has joined #openstack-neutron17:26
*** krtaylor has joined #openstack-neutron17:27
mesterydougwig: Loop me in too, I want to tilt with you17:27
dougwig#openstack-infra, right now17:27
*** s3wong has joined #openstack-neutron17:28
*** pal has joined #openstack-neutron17:29
*** nyechiel_ has quit IRC17:30
*** coolsvap|afk is now known as coolsvap17:32
*** changbl has quit IRC17:33
*** anish__ has joined #openstack-neutron17:34
openstackgerritPavel Bondar proposed openstack/neutron: Implement IPAM Driver loader  https://review.openstack.org/14747917:35
openstackgerritPavel Bondar proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433917:35
openstackgerritPavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM  https://review.openstack.org/15323617:35
openstackgerritPavel Bondar proposed openstack/neutron: IPAM reference driver  https://review.openstack.org/15048517:35
*** achanda has quit IRC17:35
*** roeyc has quit IRC17:35
*** viveknarasimhan has joined #openstack-neutron17:36
*** _cjones_ has quit IRC17:36
*** achanda has joined #openstack-neutron17:37
*** _cjones_ has joined #openstack-neutron17:37
*** jp_at_hp has quit IRC17:38
*** evgenyf has quit IRC17:38
dougwigmestery, marun - that was too easy.  i feel robbed of "the openstack experience".17:40
marunheh17:40
*** SridharRamaswamy has joined #openstack-neutron17:40
mesterydougwig: I'll save some of the openstack experience for you in person in vancouver17:41
mestery;)17:41
*** changbl has joined #openstack-neutron17:41
mesterydougwig: Also, it hasn't merged yet, so don't declare victory yet17:41
*** matrohon has quit IRC17:41
dougwigoh, true, true.17:41
openstackgerritMerged openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/16254017:42
*** shivharis has joined #openstack-neutron17:42
*** yfried_ is now known as yfried|afk17:42
*** SridharRamaswamy has quit IRC17:43
*** mrsmith has joined #openstack-neutron17:45
*** kawa2014 has quit IRC17:45
*** yfried|afk is now known as yfried_17:46
*** vmtrooper has joined #openstack-neutron17:50
*** pal has quit IRC17:50
*** Swami has joined #openstack-neutron17:50
*** ChuckC has joined #openstack-neutron17:51
*** jobewan has joined #openstack-neutron17:54
*** vmtrooper has quit IRC17:54
*** ajmiller has joined #openstack-neutron17:55
*** tiswanso has quit IRC17:55
*** tiswanso has joined #openstack-neutron17:56
*** cbouch has quit IRC17:56
*** Rajeev has joined #openstack-neutron17:56
*** yfried_ is now known as yfried|afk17:56
*** ronis has joined #openstack-neutron17:57
*** shwetaap has joined #openstack-neutron17:57
*** melwitt_ has joined #openstack-neutron17:58
*** britthouser has quit IRC17:58
*** melwitt has quit IRC17:58
*** ihrachyshka has joined #openstack-neutron18:00
*** jorgem has joined #openstack-neutron18:00
*** jschwarz has joined #openstack-neutron18:01
*** saikrishna has quit IRC18:02
openstackgerritTerry Wilson proposed openstack/neutron: Add rootwrap daemon mode support  https://review.openstack.org/15880518:03
*** anish__ has quit IRC18:04
*** jaypipes has joined #openstack-neutron18:04
*** tsg_ has quit IRC18:04
*** baoli_ has quit IRC18:05
*** changbl has quit IRC18:05
mesterydougwig: Over 5 hours in the experimental queue waiting for my python-neutronclient job to get a node to run on18:05
mesterydougwig: *sigh*18:05
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759718:05
*** baoli has joined #openstack-neutron18:07
*** tsufiev_ has quit IRC18:07
openstackgerritCarl Baldwin proposed openstack/neutron: Move internal port processing to router classes  https://review.openstack.org/15451318:08
openstackgerritrcurran proposed openstack/neutron: ML2 cisco_nexus MD: upstream updates required  https://review.openstack.org/16304018:09
marunRajeev: Do you honestly think that what you were adding qualifies as a 'unit test'?18:11
*** Flcn has quit IRC18:11
*** tflynn has joined #openstack-neutron18:12
Rajeevmarun: yes, within the context of the unit tests that are there.18:12
marunRajeev: That wasn't my question.18:12
*** sbfox has quit IRC18:12
marunRajeev: Do you consider what you were adding a unit test, period.18:12
Rajeevmarun: yes18:12
marunRajeev: Not considering the context of Neutron's supposed 'unit test' suite.18:12
marunRajeev: Then I respectfully submit that you don't know what a unit test actually is.18:13
*** jpena has quit IRC18:13
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869818:13
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759718:13
Rajeevmarun: I am not sure why your perspective is so different from rest of the reviewers18:14
marunRajeev: Because I actually care about testing.18:14
marunRajeev: I don't write features, but I am trying to encourage a more effective approach to testing.18:15
amullerRajeev: Truthfully for years adding unit tests as a checkmark so you could get your patch merged, but most of the agent-site unit testing is not effective, it's just a checkmark...18:15
amullerSo the idea is to up our game18:15
amullerAnd be more effective in testing, which for most of the agent-side testing should be functional testing18:15
*** tsufiev_ has joined #openstack-neutron18:16
*** tflynn has quit IRC18:16
marunRajeev: I entirely agree, though, that there needs to be better documentation.18:16
*** tflynn has joined #openstack-neutron18:16
marunRajeev: You're right to complain that my commentary seems arbitrary without clear and published guidelines.18:16
*** yfried|afk is now known as yfried_18:16
marunRajeev: It is my intention to rectify this, probably post k3 when I'll have more time.18:17
Rajeevmarun: don't get me wrong, I do appreciate your intent. But I think it needs to be worked out into the system a bit more.18:17
marunRajeev: I disagree18:17
marunRajeev: If I can make things better by individual review, I'm going to do it.18:18
*** SridharRamaswamy has joined #openstack-neutron18:18
marunRajeev: It's not an effective way of educating the broader community, but it's all I have right now.18:18
*** tflynn_ has joined #openstack-neutron18:18
*** tflynn_ has quit IRC18:19
Rajeevmarun: I think it is not that difficult to work with the community.18:19
marunRajeev: Who says I'm not working with the community?18:19
marunRajeev: I mean, my individual effort doesn't scale.18:19
*** sbfox has joined #openstack-neutron18:20
marunRajeev: But as far as what needs to be done, I haven't heard anyone dissenting about the approach I'm advocating (assuming they actually understand it)18:20
marunRajeev: That we have functional testing as an option at all is the result of people like me working hard to make it happen.18:20
*** tflynn has quit IRC18:21
Rajeevmarun: I am trying to be supportive here, and that is why got help to get the dependent patch submitted so quickly.18:22
openstackgerritBaodong (Robert) Li proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation  https://review.openstack.org/15869718:22
marunRajeev: So what are we arguing about?18:23
marunRajeev: What about my comments don't you agree with?18:23
*** karimb has quit IRC18:24
Rajeevmarun: two things, one please get clear word out in wnat circumstances you expect functional tests. Secondly please don't block current patches, especially if you see people trying to get functional tests in18:24
Rajeevas follow ups18:25
*** yfried_ is now known as yfried|afk18:26
*** dimsum__ is now known as dims18:27
marunRajeev: Respectfully, why are you telling me how to do my job?18:27
marunRajeev: The functional change needs to be included in the originating patch, period.18:27
marunRajeev: There's no need to take this personally.18:27
marunRajeev: a -2 is 'this should not merge'18:28
marunRajeev: I've given you a clear indication as to how you can get the code in that patch merged - include it in the parent.18:28
openstackgerritDane LeBlanc proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers  https://review.openstack.org/14906818:28
Rajeevmarun: I am not telling you how to do your job, no that is not my intent. I was responding to your comment "So what are we arguing about?"18:28
marunRajeev: You can fight me on this, but that won't help you get your code merged.18:28
amullerRajeev: Is there a reason not to squash the two patches in to one?18:29
amullerI think that Maru is operating on the assumption that there isn't18:29
Rajeevmarun: Yes, these have different authorship18:29
marunRajeev: That is not a good reason18:29
marunRajeev: shared authorship attribution is always an option18:29
marunRajeev: You're even using it already!18:30
Rajeevmarun: yes, exactly because of the second patch18:31
marunRajeev: There is nothing to stop you from adding the second change to the first and having that patch have shared authorship.18:31
Rajeevmarun: I am trying to get functioal tests in to your endeavor, I am kind of not following why you are blocking it so much18:32
marunAs to what the expectation for testing, if you're merging functionalityk, is it tested?18:32
*** Swami has quit IRC18:32
marunThat's a question that both reviewer and author need to ask.18:32
marunRajeev: If you want to do as I ask, your code will merge.18:33
marunRajeev: I'm not sure what you're hoping to accomplish by arguing so strongly for doing things your way.18:33
marunRajeev: I have my reasons for asking for changes, and I haven't heard any reason from you why those reasons are not valid.18:34
*** ajmiller_ has joined #openstack-neutron18:35
Rajeevmarun: I am not intending to argue but voicing what I am hearing from other contributors too so that we can keep things moving smoothly.18:35
marunRajeev: What are you hearing from other contributors that contradicts me?18:35
marunme -> my comments18:35
marunRajeev: that unit tests are required?18:36
openstackgerritMerged openstack/neutron-vpnaas: Remove the reference for non-existent cisco.l3.plugging_drivers  https://review.openstack.org/16288718:36
Rajeevmarun: the notion about unit tests vs, functional tests and what is a reasonable requirement as understood by people contributing code18:37
amullerRajeev: You are right in the sense that there have no written guidelines18:37
amullerSo far it's been communicated verbally18:37
*** ajmiller has quit IRC18:37
amullerAnd Maru is working on such a document18:37
amullerIn the mean time, code that touches the agent is expected to be tested effectively, meaning that parts of it should be tested with functional tests18:38
*** _cjones_ has quit IRC18:38
amullerJust like the current DVR functional tests18:38
Rajeevmarun: amuller That is very encouraging to hear18:38
jschwarzmarun, this reminds me that I ought to add some TESTING.rst documentation to the fullstack patch... will probably do so on my next patchset18:39
jschwarz(after yours goes through)18:39
*** shivharis has quit IRC18:41
*** jschwarz has quit IRC18:43
*** changbl has joined #openstack-neutron18:44
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Enable StrongSwan in gate hook  https://review.openstack.org/16171418:45
*** yfried|afk is now known as yfried_18:46
*** zzzeek has quit IRC18:48
*** baoli has quit IRC18:48
*** agentle_ has joined #openstack-neutron18:50
*** zzzeek has joined #openstack-neutron18:53
*** baoli has joined #openstack-neutron18:53
*** baoli has quit IRC18:54
*** _cjones_ has joined #openstack-neutron18:54
*** jorgem has quit IRC18:55
*** agentle_ has quit IRC18:55
*** Rajeev has quit IRC18:55
*** agentle_ has joined #openstack-neutron18:55
*** claudiub has quit IRC18:56
*** claudiub has joined #openstack-neutron18:57
*** SridharRamaswamy has quit IRC18:58
*** jorgem has joined #openstack-neutron18:59
*** markvoelker_ has quit IRC19:01
*** prometheanfire is now known as blueness-evil-tw19:01
*** emagana has quit IRC19:01
*** kbringard has quit IRC19:01
*** melwitt_ has quit IRC19:02
*** emagana has joined #openstack-neutron19:02
*** melwitt has joined #openstack-neutron19:02
*** ihrachyshka has quit IRC19:02
*** blueness-evil-tw is now known as prometheanfire19:02
*** gongysh_ has quit IRC19:02
*** Marga_ has quit IRC19:03
*** claudiub has quit IRC19:04
*** dboik_ has quit IRC19:04
*** baoli has joined #openstack-neutron19:04
*** dboik has joined #openstack-neutron19:04
*** SridharRamaswamy has joined #openstack-neutron19:06
*** emagana has quit IRC19:06
*** viveknarasimhan has quit IRC19:06
*** ajmiller_ is now known as ajmiller19:07
*** dane_leblanc has quit IRC19:08
*** bobmel has quit IRC19:08
*** jaypipes is now known as jay-afk19:09
*** jay-afk is now known as jaypipes-afk19:10
*** sbfox has quit IRC19:10
*** yamamoto has joined #openstack-neutron19:11
*** jorgem has quit IRC19:14
*** yamamoto has quit IRC19:15
openstackgerritBaodong (Robert) Li proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation  https://review.openstack.org/15869719:16
*** jorgem has joined #openstack-neutron19:17
*** sbfox has joined #openstack-neutron19:18
*** coolsvap is now known as coolsvap|afk19:20
*** gongysh_ has joined #openstack-neutron19:21
*** carl_baldwin has quit IRC19:23
*** mriedem has joined #openstack-neutron19:24
mriedemi'm looking at some wonky code in the neutronv2 api and had a question,19:24
mriedemis it possible to have more than one port associated with an instance in the same network?19:24
mriedemi assume so19:24
mriedemand each port would have a separate fixed_ip19:25
*** sripriya has quit IRC19:26
*** sripriya has joined #openstack-neutron19:27
russellbmriedem: good question ... IIRC, there has been a lot of debate about whether we would allow that or not.19:27
russellbhuge debate to required code ratio, heh19:27
mriedemrussellb: well, the multinic api was clearly written for nova-network19:27
*** s3wong has quit IRC19:28
mriedemlooking at the neutronv2 api code here: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L75219:28
mriedemnotice how it loops over ports but will exit before getting to the 2nd one19:28
mriedemthat code is pretty terrible on many levels actually, it gets the client multiple times, it's swallowing an exception, and it's only updating the first port (which is probably by design)19:29
mriedemsame for remove_fixed_ip_from_instance19:29
russellbmriedem: this is what i was thinking about https://review.openstack.org/#/c/97716/19:30
*** s3wong has joined #openstack-neutron19:30
russellbmriedem: supposedly implemented (meaning yes, it should be supported)19:30
*** emagana has joined #openstack-neutron19:31
*** britthouser has joined #openstack-neutron19:32
*** Flcn has joined #openstack-neutron19:32
*** kbringard has joined #openstack-neutron19:32
*** kbringard has quit IRC19:32
*** carl_baldwin has joined #openstack-neutron19:34
mriedemhmm19:35
mriedempretty sure tempest tests never landed for that as promised in the spec19:35
*** nyechiel_ has joined #openstack-neutron19:36
*** carl_baldwin has quit IRC19:36
*** vishy has joined #openstack-neutron19:38
*** vmtrooper has joined #openstack-neutron19:39
*** carl_baldwin has joined #openstack-neutron19:40
*** vmtrooper has quit IRC19:44
*** claudiub has joined #openstack-neutron19:45
*** nlahouti has quit IRC19:47
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961819:48
*** safchain has quit IRC19:49
*** dims has quit IRC19:51
*** sripriya has quit IRC19:51
*** sripriya has joined #openstack-neutron19:51
*** carl_baldwin has quit IRC19:53
mriedemrussellb: ok, so given that change, if i allowed multiple ports on the same network and then called add_fixed_ip_to_instance, only the first port would get the fixed_ips from the first subnet for that network19:54
mriedemany fixed_ips on any other subnets wouldn't get on the port,19:54
mriedemnor any subnets/fixed_ips for any other ports on the network for that instance19:54
mriedemso that code seems pretty busted19:54
russellbnot surprising19:54
mriedemsadly no19:54
mriedemand since there was no tempest testing added for the blueprint, we don't have any cases to build from19:55
mriedemin tempest i mean19:55
mriedembarf19:55
mriedemmaybe i'll write a test then...i seem to be doing this a lot recently to test these edge cases between nova and neutron - the code in nova is so ugly it's always hard to tell what's actually tested in tempest19:55
*** mlavalle has joined #openstack-neutron19:56
*** jpena has joined #openstack-neutron19:56
*** aepifanov has quit IRC19:57
*** aix has quit IRC19:57
amullermarun: Slight snag with the RPC tests. Cast, as opposed to call, is happening asynchronously even with the fake driver. The test thread is putting a message in a queue, and then you have another eventlet green thread polling from that queue. Under normal conditions the test finishes before the polling happens so the message is not actually received by the other end...19:57
amullerI added a while True: eventlet.sleep(0)19:57
amullerand then the other end receives the message, it's pretty funny19:57
amullermarun: I guess I'll add a utility that waits until the exchange queues are empty before proceeding19:58
*** sripriya has quit IRC20:00
*** sripriya has joined #openstack-neutron20:00
*** dims has joined #openstack-neutron20:01
*** SridharRamaswamy has quit IRC20:01
*** cference has quit IRC20:02
*** vivek-ebay has quit IRC20:03
*** vivek-ebay has joined #openstack-neutron20:03
*** tflynn has joined #openstack-neutron20:04
haleybsc68cal: i have some comments on your devstack patch, sorry for the late review20:05
*** shivharis has joined #openstack-neutron20:05
*** thedodd has quit IRC20:06
*** SumitNaiksatam has joined #openstack-neutron20:07
*** s3wong has quit IRC20:09
claudiubmtreinish: hello20:09
*** matrohon has joined #openstack-neutron20:11
*** VijayB has joined #openstack-neutron20:12
*** thedodd has joined #openstack-neutron20:12
*** s3wong has joined #openstack-neutron20:13
*** britthouser has quit IRC20:14
*** carl_baldwin has joined #openstack-neutron20:14
*** britthouser has joined #openstack-neutron20:15
*** banix has quit IRC20:18
carl_baldwinamuller: I addressed your comment on https://review.openstack.org/#/c/154513/20:20
*** banix has joined #openstack-neutron20:20
amullercarl_baldwin: +120:20
amullerthe next patch is more complicated it'll take me a bit20:20
carl_baldwinamuller: No worries.  Thank you.20:21
carl_baldwinamuller: I need to buckle down and get one more patch up for the remaining stuff in agent.py.20:21
*** yfried_ is now known as yfried|afk20:22
*** thedodd has quit IRC20:23
*** sbfox has quit IRC20:24
*** jpena has quit IRC20:25
*** jorgem has quit IRC20:25
*** gildub has joined #openstack-neutron20:28
pc_mdougwig: ping20:29
*** yfried|afk is now known as yfried_20:29
dougwigpc_m: ack20:30
pc_mdougwig: Do you know how imports for out of repo code are supposed to be handled? Specifically, in the neutron-vpnaas repo, there is a Cisco device driver that refers to a neutron import that has now been moved out to stackforge.20:30
*** marios has quit IRC20:30
pc_mWondering what all needs to be done, to reference that import in neutron-vpnaas. Do you know (or anyone lurking)?20:31
dougwigoh, that's gross.  hmm.20:31
dougwigchoices are (thinking out loud): 1) refactor out the import, 2) wrap the import in a try, 3) add that project to vpnaas's requirements list20:32
dougwigmaybe a shim + requirements file in place, similar to the neutron decomp20:32
pc_mIn this case, it is only referring to a constant in that vendor repo (so the workaround was to locally duplicate the definition).20:33
*** nyechiel_ has quit IRC20:34
*** Rajeev has joined #openstack-neutron20:34
openstackgerritMichael Smith proposed openstack/neutron: manual add/remove router for dvr_snat agent  https://review.openstack.org/12202420:34
pc_mNot sure the import could be refactored out. Wrapping would have to then locally define the value anyway.20:34
*** sbfox has joined #openstack-neutron20:35
*** radez is now known as radez_g0n320:35
pc_mdougwig: Seems like the options are to just have the duplicate definition, or add it to requirements. Though I didn't know if we want to have a vendor repo in requirements.20:35
*** Marga_ has joined #openstack-neutron20:35
*** vivek-eb_ has joined #openstack-neutron20:36
*** jorgem has joined #openstack-neutron20:37
pc_mdougwig: I see Brocade code that has a networking_vyatta import, and I don't see that in requirements.txt. Hmm20:38
pc_mI mean networking_brocade20:39
*** vivek-ebay has quit IRC20:39
*** britthouser has quit IRC20:40
*** ijw has joined #openstack-neutron20:41
openstackgerritenikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations  https://review.openstack.org/14926120:41
*** emagana has quit IRC20:41
openstackgerritSalvatore Orlando proposed openstack/neutron: Raise QuotaResourceUknown in the quota engine  https://review.openstack.org/16317120:41
pc_mdougwig: I don't see any reference to networking_brocade anywhere. Wondering how that import is ensured to be on the system.20:41
dougwigit should have a requirements file in one of neutron's subduers.20:42
dougwigsubdirs20:42
*** boris-42 has quit IRC20:42
Rajeevmarun: amuller I had to leave abruptly earlier because I lost track of time. I have to be offline for some more time. But thanks for sharing your thoughts, it helps.20:43
pc_mdougwig: don't see it...20:44
pc_mHenryG: ping20:44
*** moshele has joined #openstack-neutron20:44
*** baoli has quit IRC20:45
*** dboik_ has joined #openstack-neutron20:45
*** Rajeev has quit IRC20:46
*** melwitt has quit IRC20:46
*** melwitt has joined #openstack-neutron20:47
*** dboik has quit IRC20:49
*** cference has joined #openstack-neutron20:49
*** ronis has quit IRC20:50
*** mriedem has left #openstack-neutron20:51
ChuckCcarl_baldwin: haleyb: I've responded to your comments on https://review.openstack.org/#/c/162500/20:51
*** PrincessRiikka has quit IRC20:51
carl_baldwinChuckC: ack20:52
*** itzikb has joined #openstack-neutron20:52
enikanorov_folks, i'm looking and some logs http://logs.openstack.org/61/149261/14/check/check-tempest-dsvm-neutron-full/caf5e9e/logs/screen-q-svc.txt.gz and see log lines format being different from line to line, is that some known issue?20:52
enikanorov_(lines are different - some have pids, some don't)20:52
itzikbarmax: ping20:53
openstackgerritMathieu Rohon proposed openstack/neutron: ML2 : make port status change only when the port is bound to the host that asking  https://review.openstack.org/16317820:54
*** rpothier has joined #openstack-neutron20:55
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759720:56
*** fredlhsu has joined #openstack-neutron20:56
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759720:56
openstackgerritMerged openstack/neutron: Don't notify dead DHCP agent of removed networks  https://review.openstack.org/16255920:57
*** banix has quit IRC20:57
*** SridharRamaswamy has joined #openstack-neutron20:58
*** shivharis has quit IRC20:58
*** changbl has quit IRC20:58
*** shivharis has joined #openstack-neutron20:58
openstackgerritZengfa Gao proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method  https://review.openstack.org/15410320:59
*** ajo has quit IRC20:59
*** nlahouti has joined #openstack-neutron20:59
*** raildo is now known as raildo_away21:04
*** VijayB has quit IRC21:05
*** cference has quit IRC21:07
*** dboik_ has quit IRC21:07
*** dboik has joined #openstack-neutron21:08
*** vishwana_ has joined #openstack-neutron21:08
*** ihrachyshka has joined #openstack-neutron21:09
openstackgerritAngela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX  https://review.openstack.org/16148921:09
*** vishwanathj has quit IRC21:10
*** VijayB has joined #openstack-neutron21:10
*** pritesh has joined #openstack-neutron21:10
*** vishwana_ has quit IRC21:10
*** moshele has quit IRC21:13
*** vishwanathj has joined #openstack-neutron21:14
*** emagana has joined #openstack-neutron21:14
*** s3wong has quit IRC21:18
*** s3wong has joined #openstack-neutron21:18
kevinbentonihrachyshka: hey21:18
ihrachyshkakevinbenton, hey!21:18
kevinbentonihrachyshka: is the conversion away from neutron.openstack.common.log a simple import replacement21:19
kevinbenton?21:19
kevinbentonihrachyshka: or is there a new way to invoke the logger as well?21:19
ihrachyshkakevinbenton, should be, in most cases.21:19
kevinbentonihrachyshka: ok, thanks21:19
ihrachyshkakevinbenton, also see https://review.openstack.org/#/c/147312/21:20
*** thedodd has joined #openstack-neutron21:20
*** carl_baldwin has quit IRC21:20
ihrachyshkakevinbenton, but it's more of safety checks, in simple cases it's just a matter of import switches.21:20
*** leenheer has quit IRC21:20
dimsihrachyshka: kevinbenton: migrating notes - http://docs-draft.openstack.org/12/147312/5/check/gate-oslo.log-docs/5b2a401//doc/build/html/usage.html#migrating-to-oslo-log21:21
ihrachyshkadims, right, that's the patch I referred to21:21
kevinbentonihrachyshka: why not have openstack.common.log pass through directly the oslo.log then?21:22
dims:)21:22
kevinbentonihrachyshka: just to make it more explicit?21:22
dimskevinbenton: that comes from oslo-incubator and we are trying to get rid of the code in oslo-incubator21:22
*** boris-42 has joined #openstack-neutron21:22
*** carl_baldwin has joined #openstack-neutron21:22
*** VijayB has quit IRC21:22
dimsend goal is to get rid of */openstack/common/*21:22
kevinbentondims: oh, all of oslo incubator is going away?21:23
kevinbentonoh okay21:23
ihrachyshkakevinbenton, well, because it works the same in *most* but not all of them (for some things, imports are switched, arguments are changed...)21:23
ihrachyshkakevinbenton, well, that's the Holy Grail of graduation - getting rid of all incubator modules21:23
dimswhen we release a library say oslo.log, log.py in oslo-incubator is removed21:23
ihrachyshkakevinbenton, I think we may see it in L or M for neutron21:23
openstackgerritDoug Wiegley proposed openstack/neutron: Add proposal-bot-esque job for updating child repo neutron pinning  https://review.openstack.org/16306221:24
kevinbentonihrachyshka: ok21:24
*** sripriya has quit IRC21:25
ihrachyshkakevinbenton, in kilo, we already killed ~18 modules, 13 more left though21:26
*** sripriya has joined #openstack-neutron21:26
*** ijw has quit IRC21:26
ihrachyshka(well, it will be 11 once we consume oslo.log)21:27
*** vmtrooper has joined #openstack-neutron21:28
*** SridharRamaswam1 has joined #openstack-neutron21:29
*** SridharRamaswamy has quit IRC21:30
*** pc_m has quit IRC21:31
*** yfried_ is now known as yfried|afk21:31
*** vmtrooper has quit IRC21:33
*** yfauser has joined #openstack-neutron21:33
*** yfauser has left #openstack-neutron21:34
*** shwetaap has quit IRC21:34
*** jorgem has quit IRC21:34
*** Zemeio has quit IRC21:35
openstackgerritDoug Wiegley proposed openstack/neutron: Add proposal-bot-esque job for updating child repo neutron pinning  https://review.openstack.org/16306221:35
*** VijayB has joined #openstack-neutron21:36
kevinbentonihrachyshka: do you have the patch to neutron up to transition to oslo.log?21:39
openstackgerritAssaf Muller proposed openstack/neutron: Add RPC test for DVR  https://review.openstack.org/16320121:39
*** tsg has joined #openstack-neutron21:39
ihrachyshkakevinbenton, neutron *up*?21:39
openstackgerritAssaf Muller proposed openstack/neutron: Add RPC test for metadata  https://review.openstack.org/16320421:40
ihrachyshkayou mean migration patch for neutron repo?21:40
kevinbentonihrachyshka: sorry, up on gerrit. i found it though21:40
ihrachyshkasure, https://review.openstack.org/15963821:40
*** chlong has joined #openstack-neutron21:40
kevinbentonihrachyshka: so can we merge the requirements for oslo.log first so third party libs can start to assume it will be available?21:41
ihrachyshkakevinbenton, actually, third parties should *not* rely on neutron requirements21:42
ihrachyshkakevinbenton, I mentioned it at https://wiki.openstack.org/wiki/Neutron/VendorSplitPackaging21:42
ihrachyshkaspecifically, https://wiki.openstack.org/wiki/Neutron/VendorSplitPackaging#Dependencies21:43
*** crose has quit IRC21:43
kevinbentonihrachyshka: why? i don't see any justification in there21:44
ihrachyshkakevinbenton, also, since 3party code currently relies on neutron.common.config import to register options and set up logging, it won't work in reverse order in any way21:44
ihrachyshka"The rationale behind it is that neutron may drop some of those dependencies later, and vendor libraries will become broken."21:44
carl_baldwinamuller: ping21:45
amullercarl_baldwin: yo21:45
carl_baldwinamuller: I have a vague memory of talking about DaemonMonitor with you.  Were you thinking of moving it in to router_info?21:45
kevinbentonihrachyshka: frankly it's a tradeoff though21:45
*** matrohon has quit IRC21:45
kevinbentonihrachyshka: i don't expect neutron to drop logging any time soone21:46
kevinbentonihrachyshka: by assuming neutron will import olso.log, then a third party lib doesn't have to worry about maintaining a requirements file that is compatible with neutron21:46
amullercarl_baldwin: I remember talking about it, I don't remember why anymore though21:47
*** jckasper has quit IRC21:47
carl_baldwinamuller: I’m trying to pick that area of the code apart today.  Was wondering how far you got, if anywhere, beyond https://review.openstack.org/#/c/16060821:47
*** tiswanso has quit IRC21:47
ihrachyshkakevinbenton, do we set automatic requirements updates for vendor libs?21:47
carl_baldwinamuller: If you don’t anything beyond that patch then I’ll just proceed.21:47
kevinbentonihrachyshka: i don't know what you mean by that21:48
*** chandankumar has quit IRC21:48
amullercarl_baldwin: Oh I definitely did not work on the restructuring stuff beyond what you see in that patch.21:49
amullercarl_baldwin: However, check out this patch by Mike: https://review.openstack.org/#/c/153162/821:49
ihrachyshkakevinbenton, openstack/requirements updates are automatically synchronized to all projects mentioned in projects.txt file in the requirements repo, via openstack proposal bot. Do we plan to add vendor libs in projects.txt to get those updates? that would solve the issue of keeping compatible requirements21:49
amullercarl_baldwin: and also a patch of mine here: https://review.openstack.org/#/c/125384/31/neutron/agent/l3/ha.py21:49
amullerwe both just pass the process_monitor from the agent to the router where needed21:49
amullermostly because changing the RouterInfo.__init__ signature is a huge pain in the ass now21:50
kevinbentonihrachyshka: oh, i'm not sure. but if we did do that, doesn't that just come back to the same question of why bother to have the vendor lib requirements at all?21:50
carl_baldwinamuller: ack21:50
kevinbentonihrachyshka: if they are just a copy of what is in neutron?21:50
openstackgerritMichael Smith proposed openstack/neutron: Fix non-existent self.local_subnets in DvrRouter class  https://review.openstack.org/15762121:50
kevinbentonihrachyshka: i suppose the requirements bump would at least fail a CI check21:51
kevinbentonihrachyshka: but is that all we are gaining?21:51
*** mwagner_lap has quit IRC21:51
*** yfried|afk is now known as yfried_21:52
carl_baldwinamuller: I wonder if we should just do it now (change __init__) anyway.  It is either now or much later.21:52
amullercarl_baldwin: If you're up for it21:52
amullerthe fwaas repo is still trying to recover from the last time we changed that signature21:53
ihrachyshkakevinbenton, well ok, maybe shortcut assuming neutron fetches the proper lib is justified in this particular case.21:53
kevinbentonihrachyshka: i can see it's a tradeoff21:54
kevinbentonihrachyshka: assuming the presence of libraries can be dangeroues21:54
kevinbentonihrachyshka: but when something is as tightly integrated as a driver, requirements are far from the only thing that will break things21:55
ihrachyshkakevinbenton, so back to your suggestion to merge in sole requirements update. I don't follow how it would differ from vendors adding a temporary dependency for the sake of migration and killing it once they are sure neutron fetches it. Of course we could split the dependency addition, but I don't see how it helps significantly for vendor switch to oslo.log.21:55
kevinbentonihrachyshka: change in one spot vs change in however many vendors21:55
kevinbentonO(1) vs O(n) :)21:56
ihrachyshkakevinbenton, ok, I'll split the patch now21:56
ihrachyshkaif it helps you slightly, it's worth respin21:56
amullermarun: Pushed https://review.openstack.org/#/c/163201/1 which adds DVR RPC tests21:57
amullerarmax: ^21:57
kevinbentonihrachyshka: thanks. let me know when it's up and I'll +221:57
amullerAlso adds a pretty ugly way to make sure that cast messages are delivered21:57
*** vishwana_ has joined #openstack-neutron21:57
*** jaypipes-afk is now known as JAYPIPES21:59
*** JAYPIPES is now known as jaypipes21:59
openstackgerritIhar Hrachyshka proposed openstack/neutron: Migrate to oslo.log  https://review.openstack.org/15963822:00
openstackgerritIhar Hrachyshka proposed openstack/neutron: Added oslo.log dependency  https://review.openstack.org/16320722:00
ihrachyshkakevinbenton, ^^22:00
*** vishwanathj has quit IRC22:00
*** yfried_ is now known as yfried|afk22:02
*** coolsvap|afk has quit IRC22:03
*** tsg has quit IRC22:03
kevinbentonihrachyshka: thanks!22:04
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961822:05
*** chlong has quit IRC22:07
*** agentle_ has quit IRC22:07
*** yfauser1 has joined #openstack-neutron22:08
openstackgerritMichael Smith proposed openstack/neutron: WIP delay fip namespace cleanup/destroy  https://review.openstack.org/15175822:08
*** rpothier has quit IRC22:10
*** yfauser1 has left #openstack-neutron22:10
*** marrusl has quit IRC22:10
carl_baldwinamuller: I could avoid it for now by creating an initialize like you did for ha and pass the process_monitor to it.22:11
carl_baldwinIs that too hacky?22:11
openstackgerritMerged openstack/neutron: policy: don't hack around oslo.config path search algorithm  https://review.openstack.org/15986222:12
amullercarl_baldwin: yes =D22:12
openstackgerritMerged openstack/neutron: tests: initialize policy in BaseTestCase  https://review.openstack.org/15986322:12
carl_baldwinamuller: You started it.  :P22:12
amuller:)22:12
amullerI expect more from you than from me, it's human nature22:13
carl_baldwinStill thinking.  Sometimes, when you’re stuck between a rock and a hard place...22:14
*** tsufiev_ has quit IRC22:14
*** sbfox has quit IRC22:16
openstackgerritAbhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver  https://review.openstack.org/15724322:19
*** absubram_ has joined #openstack-neutron22:19
openstackgerritMichael Smith proposed openstack/neutron: Fix non-existent self.local_subnets in DvrRouter class  https://review.openstack.org/15762122:21
*** absubram has quit IRC22:21
*** absubram_ is now known as absubram22:21
*** txomon|fon has quit IRC22:22
*** marrusl has joined #openstack-neutron22:23
*** jorgem has joined #openstack-neutron22:23
*** agentle_ has joined #openstack-neutron22:25
*** nati_ueno has joined #openstack-neutron22:29
*** claudiub has quit IRC22:30
openstackgerritAssaf Muller proposed openstack/neutron: Introduce RPC tests  https://review.openstack.org/16281122:31
openstackgerritAssaf Muller proposed openstack/neutron: Add RPC test for DVR  https://review.openstack.org/16320122:31
*** Rajeev has joined #openstack-neutron22:31
*** thedodd has quit IRC22:34
openstackgerritAssaf Muller proposed openstack/neutron: Add RPC test for metadata  https://review.openstack.org/16320422:34
*** txomon|fon has joined #openstack-neutron22:35
*** jorgem has quit IRC22:35
*** tsufiev_ has joined #openstack-neutron22:36
*** rushil has quit IRC22:36
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed  https://review.openstack.org/16226022:36
*** shivharis has quit IRC22:39
*** shivharis has joined #openstack-neutron22:39
*** amuller has quit IRC22:40
*** fredlhsu has quit IRC22:41
openstackgerritRajeev Grover proposed openstack/neutron: Prevent race conditions on DVR FIP resources  https://review.openstack.org/15342222:42
*** agentle_ has quit IRC22:47
*** sbfox has joined #openstack-neutron22:48
*** shivharis has quit IRC22:51
*** shivharis has joined #openstack-neutron22:51
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759722:52
*** dboik_ has joined #openstack-neutron22:52
*** VijayB has quit IRC22:53
*** Flcn has quit IRC22:53
*** mlavalle has quit IRC22:53
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759722:53
*** fredlhsu has joined #openstack-neutron22:55
*** dboik has quit IRC22:55
*** Marga_ has quit IRC22:57
*** bquan has quit IRC22:57
*** Marga_ has joined #openstack-neutron22:57
*** dboik_ has quit IRC22:57
*** claudiub has joined #openstack-neutron22:59
*** jobewan has quit IRC23:00
*** ihrachyshka has quit IRC23:02
openstackgerritNachi Ueno proposed openstack/neutron: OpenContrail plugin code split  https://review.openstack.org/16321823:02
*** nati_ueno has quit IRC23:04
*** nati_ueno has joined #openstack-neutron23:05
*** _cjones_ has quit IRC23:05
*** Marga_ has quit IRC23:06
*** ijw has joined #openstack-neutron23:06
*** Marga_ has joined #openstack-neutron23:06
*** melwitt has quit IRC23:07
*** Sukhdev has joined #openstack-neutron23:10
*** Marga_ has quit IRC23:11
*** prad has quit IRC23:14
*** VijayB has joined #openstack-neutron23:15
*** _cjones_ has joined #openstack-neutron23:15
*** tsufiev_ has quit IRC23:16
itzikbHi, Anyone familiar with this issue https://bugs.launchpad.net/neutron/+bug/1353165 ?23:16
openstackLaunchpad bug 1353165 in neutron "Namespaces not removed when the last VM using a DVR is deleted" [Medium,Fix released] - Assigned to Stephen Ma (stephen-ma)23:16
*** vmtrooper has joined #openstack-neutron23:16
*** Marga_ has joined #openstack-neutron23:17
*** tsufiev_ has joined #openstack-neutron23:21
*** vmtrooper has quit IRC23:21
openstackgerritCarl Baldwin proposed openstack/neutron: WIP Last-ditch effort to get the router stuff out of the agent  https://review.openstack.org/16322223:22
carl_baldwinamuller:  ^23:22
*** absubram has quit IRC23:24
*** fredlhsu has quit IRC23:24
*** gildub has quit IRC23:26
*** tsufiev_ has quit IRC23:27
*** sbfox has quit IRC23:30
*** david-lyle is now known as david-lyle_afk23:33
*** jprovazn has quit IRC23:33
*** mwagner_lap has joined #openstack-neutron23:34
*** sripriya has quit IRC23:34
*** jamie_h has quit IRC23:37
*** moha_hunt has quit IRC23:43
*** nlahouti has quit IRC23:44
*** banix has joined #openstack-neutron23:45
*** britthouser has joined #openstack-neutron23:45
*** tsufiev_ has joined #openstack-neutron23:46
*** carl_baldwin has quit IRC23:48
*** pritesh has quit IRC23:49
*** fredlhsu has joined #openstack-neutron23:49
*** bandarji has quit IRC23:50
*** Sukhdev has quit IRC23:51
openstackgerritYanping Qu proposed openstack/neutron-fwaas: Provide hooks for Cisco FWaaS functional tests  https://review.openstack.org/16193823:52
*** chlong has joined #openstack-neutron23:54
*** yamamoto has joined #openstack-neutron23:55

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!