*** bandarji has quit IRC | 00:00 | |
*** tflynn_ has joined #openstack-neutron | 00:01 | |
*** tflynn_ has quit IRC | 00:01 | |
mlavalle | amuller: aren't you in Israel? What time is it there? | 00:01 |
---|---|---|
amuller | mlavalle: My wife and I relocated to Boston a couple of months ago | 00:01 |
amuller | But you have a good memory :) | 00:02 |
carl_baldwin | amuller, btw, I finally followed-up on your comments on https://review.openstack.org/#/c/154513/. Thanks for the feedback. | 00:02 |
amuller | carl_baldwin: Sure, I'll review tomorrow hopefully | 00:02 |
mlavalle | amuller: ahhhh! cool.... And it welcomed you with this miserable winter...... My daughter is a senior at MIT.... so she is sharing your weather adventures | 00:03 |
amuller | mlavalle: Cool :) The weather is getting a lot better now though | 00:03 |
*** yuanying has quit IRC | 00:03 | |
*** Marga_ has joined #openstack-neutron | 00:04 | |
*** tflynn has quit IRC | 00:04 | |
*** yuanying has joined #openstack-neutron | 00:05 | |
carl_baldwin | amuller, No rush. It is late for you. | 00:05 |
*** banix has quit IRC | 00:05 | |
carl_baldwin | ... even in Boston. | 00:05 |
mlavalle | amuller: 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 3 | 00:05 |
*** oomichi has quit IRC | 00:06 | |
amuller | mlavalle: hah yeah there were 2 or 3 pretty nasty storms (Especially from my warm blooded perspective) | 00:06 |
carl_baldwin | mlavalle, I think I agree with amuller's feedback. He's got a pretty good eye for how to approach testing. | 00:08 |
mlavalle | amuller, carl_baldwin: perfect.... I'll take it from here.... and thanks for taking the time to give me feedback :-) | 00:08 |
carl_baldwin | amuller: 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_baldwin | mlavalle, It is a pleasure. | 00:09 |
*** shivharis has quit IRC | 00:09 | |
*** shivharis has joined #openstack-neutron | 00:09 | |
*** yuanying_ has joined #openstack-neutron | 00:10 | |
*** fredlhsu has quit IRC | 00:10 | |
*** yuanying has quit IRC | 00:10 | |
*** igordcard has quit IRC | 00:12 | |
amuller | armax: 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 the | 00:12 |
amuller | correct parameters, not to test the actual server methods (As those are already tested in their respective places) | 00:12 |
amuller | So we assume that the non-fake oslo messaging driver works (As that's tested already) | 00:13 |
amuller | And we assume that the RPC methods themselves are tested | 00:13 |
amuller | but these tests actually invoke a method from the client and have the code executed on the server | 00:13 |
amuller | which is something that we currently only have in Tempest | 00:13 |
amuller | armax: This should catch regressions we had in DVR, where the solution was to introduce unit tests that practically mock what they're testing | 00:14 |
*** vivek-ebay has quit IRC | 00:16 | |
*** s3wong has quit IRC | 00:16 | |
*** vivek-ebay has joined #openstack-neutron | 00:16 | |
openstackgerrit | Rajeev Grover proposed openstack/neutron: If configured, set the MTU for fpr/rfp intefaces https://review.openstack.org/162312 | 00:17 |
*** s3wong has joined #openstack-neutron | 00:17 | |
*** mlavalle has quit IRC | 00:19 | |
openstackgerrit | Isaku Yamahata proposed openstack/neutron: internal callers of create_{network, subnet, port} ignore extension https://review.openstack.org/129773 | 00:19 |
*** xgerman has quit IRC | 00:23 | |
*** thomasem has quit IRC | 00:23 | |
*** Marga_ has quit IRC | 00:23 | |
*** Marga_ has joined #openstack-neutron | 00:24 | |
carl_baldwin | amuller, sounds like some good stuff. | 00:25 |
*** sbfox has quit IRC | 00:26 | |
*** thomasem has joined #openstack-neutron | 00:28 | |
*** achanda has quit IRC | 00:30 | |
*** yamamoto has quit IRC | 00:30 | |
openstackgerrit | Shiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo. https://review.openstack.org/152383 | 00:31 |
*** banix has joined #openstack-neutron | 00:31 | |
*** baoli has quit IRC | 00:34 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Introduce RPC tests https://review.openstack.org/162811 | 00:35 |
*** hichihara has joined #openstack-neutron | 00:36 | |
*** hichihara has quit IRC | 00:36 | |
openstackgerrit | Rajeev Grover proposed openstack/neutron: Functional Test for DVR fpr/rfp interface MTU set https://review.openstack.org/162812 | 00:36 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Introduce RPC tests https://review.openstack.org/162811 | 00:37 |
amuller | night | 00:39 |
*** amuller has quit IRC | 00:39 | |
*** smillward has quit IRC | 00:40 | |
*** baoli has joined #openstack-neutron | 00:40 | |
*** SumitNaiksatam has joined #openstack-neutron | 00:42 | |
*** ijw_ has joined #openstack-neutron | 00:42 | |
*** ChuckC has quit IRC | 00:42 | |
*** ijw has quit IRC | 00:45 | |
*** _cjones_ has quit IRC | 00:46 | |
*** pavel_bondar has quit IRC | 00:46 | |
*** pavel_bondar has joined #openstack-neutron | 00:47 | |
*** Rajeev has quit IRC | 00:48 | |
*** ChuckC has joined #openstack-neutron | 00:49 | |
*** r-daneel has quit IRC | 00:49 | |
openstackgerrit | Merged openstack/neutron: Initial copy of api tests from tempest https://review.openstack.org/162257 | 00:50 |
*** ChuckC has quit IRC | 00:53 | |
*** ChuckC has joined #openstack-neutron | 00:53 | |
*** s3wong has quit IRC | 00:54 | |
*** s3wong has joined #openstack-neutron | 00:55 | |
*** tiswanso has joined #openstack-neutron | 00:55 | |
*** yamamoto has joined #openstack-neutron | 00:55 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move external port processing to router classes https://review.openstack.org/158495 | 00:56 |
*** tiswanso has quit IRC | 00:56 | |
*** tiswanso has joined #openstack-neutron | 00:57 | |
openstackgerrit | Shiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo. https://review.openstack.org/152383 | 00:59 |
*** yuanying_ has quit IRC | 00:59 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move external port processing to router classes https://review.openstack.org/158495 | 01:00 |
*** yuanying has joined #openstack-neutron | 01:01 | |
*** tflynn has joined #openstack-neutron | 01:02 | |
*** banix has quit IRC | 01:02 | |
openstackgerrit | JoeJiang proposed openstack/python-neutronclient: conform to the PEP8 https://review.openstack.org/162820 | 01:05 |
*** britthouser has joined #openstack-neutron | 01:05 | |
*** tflynn has quit IRC | 01:06 | |
*** xuhanp has joined #openstack-neutron | 01:08 | |
*** nlahouti has quit IRC | 01:08 | |
*** xuhanp_ has joined #openstack-neutron | 01:08 | |
*** melwitt has quit IRC | 01:11 | |
*** baoli has quit IRC | 01:14 | |
*** SridharRamaswamy has quit IRC | 01:20 | |
*** leenheer has quit IRC | 01:25 | |
*** vivek-ebay has quit IRC | 01:25 | |
*** vivek-eb_ has joined #openstack-neutron | 01:25 | |
*** vmtrooper has joined #openstack-neutron | 01:30 | |
*** banix has joined #openstack-neutron | 01:30 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: linuxbridge UT: Do not create the same instance in each cases https://review.openstack.org/162825 | 01:30 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: linuxbridge UT: Mock get_interface_by_ip https://review.openstack.org/162826 | 01:30 |
*** s3wong has quit IRC | 01:33 | |
*** stanzgy has joined #openstack-neutron | 01:34 | |
*** stanzgy_ has joined #openstack-neutron | 01:34 | |
*** vmtrooper has quit IRC | 01:35 | |
*** popw has joined #openstack-neutron | 01:37 | |
*** Zemeio has joined #openstack-neutron | 01:38 | |
*** ijw_ has quit IRC | 01:42 | |
*** rwsu has quit IRC | 01:47 | |
*** absubram has quit IRC | 01:48 | |
*** yamahata has quit IRC | 01:49 | |
*** jyuso1 has joined #openstack-neutron | 01:50 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Don't start transaction during floating IP delete https://review.openstack.org/160036 | 01:53 |
*** _cjones_ has joined #openstack-neutron | 01:56 | |
*** shivharis has quit IRC | 01:56 | |
*** shivharis has joined #openstack-neutron | 01:56 | |
*** banix has quit IRC | 01:57 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: ofagent: Have a thin driver module https://review.openstack.org/162555 | 02:00 |
*** armax has quit IRC | 02:00 | |
*** shivharis has quit IRC | 02:01 | |
*** diegows has quit IRC | 02:02 | |
*** yapeng has joined #openstack-neutron | 02:06 | |
*** yog_ has joined #openstack-neutron | 02:07 | |
*** ivar-lazzaro has quit IRC | 02:11 | |
*** carl_baldwin has quit IRC | 02:12 | |
*** tellesnobrega_ has joined #openstack-neutron | 02:14 | |
*** tellesnobrega_ has quit IRC | 02:14 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit https://review.openstack.org/162840 | 02:16 |
*** salv-orlando has quit IRC | 02:20 | |
*** banix has joined #openstack-neutron | 02:21 | |
*** Networkn3rd has joined #openstack-neutron | 02:21 | |
*** lsg has joined #openstack-neutron | 02:24 | |
gongysh_ | amotoki: hi | 02:24 |
*** erkules_ has joined #openstack-neutron | 02:30 | |
*** baoli has joined #openstack-neutron | 02:32 | |
*** coolsvap has joined #openstack-neutron | 02:33 | |
*** erkules has quit IRC | 02:33 | |
*** lsg has quit IRC | 02:34 | |
*** kiran has joined #openstack-neutron | 02:35 | |
*** amotoki has quit IRC | 02:35 | |
*** lsg has joined #openstack-neutron | 02:35 | |
openstackgerrit | Wei Tie proposed openstack/neutron: Add row lock when update dhcp reserved port https://review.openstack.org/159110 | 02:41 |
*** shivharis has joined #openstack-neutron | 02:43 | |
*** vivek-eb_ has quit IRC | 02:45 | |
*** shivharis has quit IRC | 02:45 | |
*** coolsvap has quit IRC | 02:45 | |
openstackgerrit | Kyle Mestery proposed openstack/python-neutronclient: Copy functional tests from tempest cli https://review.openstack.org/162848 | 02:45 |
*** coolsvap has joined #openstack-neutron | 02:46 | |
*** coolsvap has quit IRC | 02:46 | |
*** coolsvap has joined #openstack-neutron | 02:46 | |
*** yuanying has quit IRC | 02:46 | |
openstackgerrit | Kyle Mestery proposed openstack/python-neutronclient: Copy functional tests from tempest cli https://review.openstack.org/162848 | 02:47 |
*** achanda has joined #openstack-neutron | 02:47 | |
*** gildub has quit IRC | 02:49 | |
*** gildub has joined #openstack-neutron | 02:50 | |
*** shwetaap has joined #openstack-neutron | 02:50 | |
*** markvoelker has joined #openstack-neutron | 02:51 | |
*** sbalukoff has quit IRC | 02:53 | |
*** banix has quit IRC | 02:55 | |
*** chandankumar has joined #openstack-neutron | 02:57 | |
*** _cjones_ has quit IRC | 03:00 | |
*** _cjones_ has joined #openstack-neutron | 03:00 | |
*** yuanying has joined #openstack-neutron | 03:01 | |
*** yuanying has quit IRC | 03:02 | |
*** bquan has joined #openstack-neutron | 03:02 | |
kiran | Hi. 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 IRC | 03:03 | |
*** gongysh_ has quit IRC | 03:05 | |
*** yuanying has joined #openstack-neutron | 03:06 | |
openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers https://review.openstack.org/152697 | 03:09 |
*** gildub has quit IRC | 03:10 | |
*** gongysh_ has joined #openstack-neutron | 03:17 | |
*** SumitNaiksatam has quit IRC | 03:18 | |
*** vmtrooper has joined #openstack-neutron | 03:19 | |
*** salv-orlando has joined #openstack-neutron | 03:21 | |
*** kiran has quit IRC | 03:23 | |
*** jyuso1 has quit IRC | 03:24 | |
*** vmtrooper has quit IRC | 03:24 | |
*** salv-orlando has quit IRC | 03:26 | |
*** zzzeek has quit IRC | 03:26 | |
*** gildub has joined #openstack-neutron | 03:31 | |
*** dims_ has quit IRC | 03:32 | |
*** claudiub has quit IRC | 03:37 | |
*** yuanying has quit IRC | 03:38 | |
*** xuhanp has quit IRC | 03:38 | |
*** xuhanp_ has quit IRC | 03:38 | |
*** achanda has quit IRC | 03:40 | |
*** yuanying has joined #openstack-neutron | 03:41 | |
*** tiswanso has quit IRC | 03:43 | |
*** shivharis has joined #openstack-neutron | 03:44 | |
*** shivharis has quit IRC | 03:45 | |
*** sbalukoff has joined #openstack-neutron | 03:49 | |
*** dboik_ has quit IRC | 03:51 | |
*** yapeng has quit IRC | 03:52 | |
*** markvoelker has quit IRC | 03:53 | |
*** markvoelker has joined #openstack-neutron | 03:53 | |
*** markvoelker has quit IRC | 03:57 | |
*** Longgeek has joined #openstack-neutron | 03:59 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: portsecurity_db: Fix a usage of is_attr_set https://review.openstack.org/162563 | 03:59 |
*** salv-orlando has joined #openstack-neutron | 04:01 | |
*** tflynn has joined #openstack-neutron | 04:02 | |
*** yatin has joined #openstack-neutron | 04:03 | |
*** sbfox has joined #openstack-neutron | 04:04 | |
*** sbfox has quit IRC | 04:04 | |
*** yog_ has quit IRC | 04:06 | |
*** achanda has joined #openstack-neutron | 04:06 | |
*** tflynn has quit IRC | 04:07 | |
*** tflynn has joined #openstack-neutron | 04:07 | |
*** jorgem has joined #openstack-neutron | 04:08 | |
*** gongysh has joined #openstack-neutron | 04:09 | |
*** absubram has joined #openstack-neutron | 04:10 | |
*** jorgem has quit IRC | 04:11 | |
*** absubram has quit IRC | 04:14 | |
*** salv-orlando has quit IRC | 04:14 | |
*** sthillma has joined #openstack-neutron | 04:15 | |
*** absubram has joined #openstack-neutron | 04:15 | |
*** sthillma has quit IRC | 04:15 | |
*** yog_ has joined #openstack-neutron | 04:18 | |
*** yog_ has quit IRC | 04:20 | |
*** saikrishna has joined #openstack-neutron | 04:23 | |
openstackgerrit | Brian Haley proposed openstack/neutron: Change linux/ip_lib code to better handle address families https://review.openstack.org/157555 | 04:23 |
*** markvoelker has joined #openstack-neutron | 04:24 | |
*** pradipta has joined #openstack-neutron | 04:26 | |
*** amotoki has joined #openstack-neutron | 04:30 | |
*** pradip_vedams has quit IRC | 04:31 | |
*** Bhavya has joined #openstack-neutron | 04:32 | |
*** dims has joined #openstack-neutron | 04:32 | |
*** gongysh has quit IRC | 04:33 | |
*** pradip_vedams has joined #openstack-neutron | 04:34 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 04:35 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention https://review.openstack.org/144391 | 04:35 |
*** dims has quit IRC | 04:38 | |
*** SridharG has joined #openstack-neutron | 04:39 | |
*** yfried|afk is now known as yfried | 04:40 | |
*** dims has joined #openstack-neutron | 04:40 | |
*** anish_ has joined #openstack-neutron | 04:41 | |
*** markvoelker has quit IRC | 04:41 | |
*** trinaths has joined #openstack-neutron | 04:45 | |
*** shivharis has joined #openstack-neutron | 04:45 | |
*** dims has quit IRC | 04:45 | |
*** shivharis has quit IRC | 04:46 | |
*** baoli has quit IRC | 04:48 | |
*** alexpilotti has joined #openstack-neutron | 04:48 | |
*** pradipta has quit IRC | 04:49 | |
*** baoli has joined #openstack-neutron | 04:50 | |
openstackgerrit | Angela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches. https://review.openstack.org/154709 | 04:53 |
*** tflynn has quit IRC | 04:55 | |
*** yfried is now known as yfried|afk | 04:56 | |
*** baoli has quit IRC | 04:56 | |
*** tflynn has joined #openstack-neutron | 04:58 | |
openstackgerrit | Merged openstack/neutron: Use an existing function in process monitor tests https://review.openstack.org/161698 | 04:58 |
*** yfried|afk is now known as yfried | 04:58 | |
*** leenheer has joined #openstack-neutron | 05:01 | |
*** baoli has joined #openstack-neutron | 05:02 | |
*** pradipta has joined #openstack-neutron | 05:06 | |
*** baoli has quit IRC | 05:07 | |
*** vmtrooper has joined #openstack-neutron | 05:08 | |
*** yfried is now known as yfried|afk | 05:10 | |
*** Sukhdev has joined #openstack-neutron | 05:11 | |
*** yfried|afk is now known as yfried | 05:12 | |
*** topshare has joined #openstack-neutron | 05:12 | |
*** vmtrooper has quit IRC | 05:12 | |
*** yshafrir has joined #openstack-neutron | 05:13 | |
*** salv-orlando has joined #openstack-neutron | 05:13 | |
*** nlahouti has joined #openstack-neutron | 05:15 | |
Sukhdev | salv-orlando: Ping | 05:15 |
openstackgerrit | Qin Zhao proposed openstack/python-neutronclient: Add InvalidIpForSubnetClient exception https://review.openstack.org/162062 | 05:16 |
*** nlahouti1 has joined #openstack-neutron | 05:16 | |
*** nlahouti has quit IRC | 05:19 | |
*** salv-orlando has quit IRC | 05:19 | |
*** zhhuabj has quit IRC | 05:22 | |
*** salv-orlando has joined #openstack-neutron | 05:22 | |
*** salv-orlando has quit IRC | 05:26 | |
*** yamamoto has quit IRC | 05:27 | |
*** yfried has quit IRC | 05:27 | |
*** vthapar has joined #openstack-neutron | 05:28 | |
*** _cjones_ has quit IRC | 05:29 | |
*** anilvenkata has joined #openstack-neutron | 05:32 | |
*** markvoelker has joined #openstack-neutron | 05:38 | |
*** chandankumar has quit IRC | 05:38 | |
*** shwetaap has quit IRC | 05:41 | |
*** shivharis has joined #openstack-neutron | 05:45 | |
*** zhhuabj has joined #openstack-neutron | 05:46 | |
*** shivharis has quit IRC | 05:47 | |
*** nlahouti1 has quit IRC | 05:48 | |
*** nlahouti has joined #openstack-neutron | 05:49 | |
*** ajo has joined #openstack-neutron | 05:50 | |
*** chandankumar has joined #openstack-neutron | 05:54 | |
*** achanda has quit IRC | 05:54 | |
*** sbalukoff has quit IRC | 05:55 | |
*** achanda has joined #openstack-neutron | 05:55 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 05:55 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention https://review.openstack.org/144391 | 05:55 |
*** Tusker has joined #openstack-neutron | 05:56 | |
Tusker | heya guys | 05:56 |
*** saikrishna has quit IRC | 05:57 | |
*** harlowja_ is now known as harlowja_away | 05:57 | |
Tusker | having a problem deploying a fresh juno install of neutron DB | 05:57 |
Tusker | with the inability to modify fk cisco_n1kv_vlan_allocations | 05:57 |
Tusker | i tried to add a "op.drop_constraint('cisco_n1kv_vlan_allocations_ibfk_1'," before the "op.create_foreign_key(" but still encounter the same error | 05:58 |
Tusker | the heal script is trying to do a modify of a FK, but mysql is denying it | 05:59 |
*** irenab has joined #openstack-neutron | 05:59 | |
Tusker | no-one run up a fresh juno db of neutron on mysql recently ? | 06:01 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 06:02 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 06:02 |
*** yamamoto has joined #openstack-neutron | 06:02 | |
*** bquan has joined #openstack-neutron | 06:04 | |
*** Longgeek has quit IRC | 06:04 | |
*** bquan has quit IRC | 06:08 | |
*** salv-orlando has joined #openstack-neutron | 06:08 | |
*** neeti has joined #openstack-neutron | 06:08 | |
*** moha_hunt has quit IRC | 06:08 | |
*** salv-orlando has quit IRC | 06:08 | |
*** achanda has quit IRC | 06:11 | |
*** achanda has joined #openstack-neutron | 06:11 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/162540 | 06:15 |
*** dims has joined #openstack-neutron | 06:21 | |
*** Longgeek has joined #openstack-neutron | 06:23 | |
*** SridharG has left #openstack-neutron | 06:24 | |
*** tflynn has quit IRC | 06:24 | |
*** dims has quit IRC | 06:26 | |
Tusker | OK, 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) CO | 06:26 |
*** yog__ has joined #openstack-neutron | 06:27 | |
*** achanda has quit IRC | 06:28 | |
*** salv-orlando has joined #openstack-neutron | 06:28 | |
*** sgordon has quit IRC | 06:30 | |
*** david-lyle has quit IRC | 06:30 | |
*** _cjones_ has joined #openstack-neutron | 06:31 | |
*** yfried has joined #openstack-neutron | 06:36 | |
*** salv-orlando has quit IRC | 06:38 | |
openstackgerrit | Akihiro Motoki proposed openstack/neutron: NEC plugin code split https://review.openstack.org/162103 | 06:38 |
*** david-lyle has joined #openstack-neutron | 06:39 | |
*** tflynn has joined #openstack-neutron | 06:39 | |
*** sgordon has joined #openstack-neutron | 06:40 | |
*** yfried_ has joined #openstack-neutron | 06:40 | |
*** yfried has quit IRC | 06:41 | |
*** yfried_ is now known as yfried|afk | 06:44 | |
*** shivharis has joined #openstack-neutron | 06:46 | |
*** coolsvap is now known as coolsvap|afk | 06:47 | |
*** shivharis has quit IRC | 06:48 | |
*** bobmel has joined #openstack-neutron | 06:49 | |
*** nlahouti has quit IRC | 06:50 | |
*** bobmel has quit IRC | 06:50 | |
*** bobmel has joined #openstack-neutron | 06:51 | |
*** salv-orlando has joined #openstack-neutron | 06:54 | |
*** anshul has joined #openstack-neutron | 06:54 | |
*** nmagnezi_ has joined #openstack-neutron | 06:55 | |
enikanorov_ | amotoki: hi, could you please take a look at https://review.openstack.org/#/c/162576/ | 06:55 |
amotoki | enikanorov: hi, sure | 06:56 |
*** vmtrooper has joined #openstack-neutron | 06:56 | |
*** lori|away is now known as lori | 06:57 | |
*** coolsvap|afk is now known as coolsvap | 06:58 | |
*** salv-orlando has quit IRC | 06:59 | |
*** gildub has quit IRC | 07:00 | |
*** vmtrooper has quit IRC | 07:01 | |
*** ruijing has joined #openstack-neutron | 07:02 | |
*** SridharG has joined #openstack-neutron | 07:02 | |
*** _cjones_ has quit IRC | 07:02 | |
*** _cjones_ has joined #openstack-neutron | 07:03 | |
*** _cjones_ has quit IRC | 07:03 | |
*** yfried|afk is now known as yfried_ | 07:04 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Remove the reference for non-existent cisco.l3.plugging_drivers https://review.openstack.org/162887 | 07:06 |
*** garyk has joined #openstack-neutron | 07:06 | |
openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers https://review.openstack.org/152697 | 07:10 |
*** Sukhdev has quit IRC | 07:10 | |
*** david-lyle is now known as david-lyle_afk | 07:11 | |
*** itzikb has joined #openstack-neutron | 07:12 | |
*** irenab has quit IRC | 07:12 | |
*** numan has joined #openstack-neutron | 07:13 | |
*** zhhuabj has quit IRC | 07:18 | |
*** zhhuabj has joined #openstack-neutron | 07:18 | |
*** tflynn has quit IRC | 07:19 | |
*** scheuran has joined #openstack-neutron | 07:21 | |
*** chlong has quit IRC | 07:21 | |
*** sbalukoff has joined #openstack-neutron | 07:25 | |
*** killer_prince is now known as lazy_prince | 07:27 | |
openstackgerrit | Angus Lees proposed openstack/neutron: Introduce privilege separation process https://review.openstack.org/155631 | 07:30 |
*** anish__ has joined #openstack-neutron | 07:31 | |
*** jistr has joined #openstack-neutron | 07:31 | |
*** xuhanp has joined #openstack-neutron | 07:32 | |
*** xuhanp_ has joined #openstack-neutron | 07:32 | |
*** anish_ has quit IRC | 07:32 | |
openstackgerrit | Akihiro Motoki proposed openstack/neutron: NEC plugin code split https://review.openstack.org/162103 | 07:32 |
*** openstackgerrit has quit IRC | 07:35 | |
*** openstackgerrit has joined #openstack-neutron | 07:36 | |
*** gridinv has joined #openstack-neutron | 07:36 | |
enikanorov_ | amotoki: ping | 07:37 |
openstackgerrit | Nurit Vilosny proposed openstack/neutron: Add notification flag (True / False) to extension_manager process_update_port https://review.openstack.org/162648 | 07:38 |
amotoki | enikanorov_: pong | 07:38 |
enikanorov_ | amotoki: how do you see set() is passed to agent/linux/iptables_firewall.py ? | 07:38 |
amotoki | enikanorov_: 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 list | 07: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 method | 07:40 |
amotoki | enikanorov_: I see, so there is no problem. | 07:40 |
*** gridinv has quit IRC | 07:41 | |
amotoki | enikanorov_: 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 sense | 07:43 |
*** kbringard has joined #openstack-neutron | 07:44 | |
*** kbringard has quit IRC | 07:44 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: test_l2population: Simplify migration tests https://review.openstack.org/158182 | 07:44 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: test_l2population: Simplify migration tests https://review.openstack.org/158182 | 07:44 |
*** ronis has joined #openstack-neutron | 07:45 | |
*** leenheer has quit IRC | 07:46 | |
*** shivharis has joined #openstack-neutron | 07:47 | |
*** irenab has joined #openstack-neutron | 07:48 | |
*** shivharis has quit IRC | 07:49 | |
*** coolsvap is now known as coolsvap|afk | 07:53 | |
openstackgerrit | dql proposed openstack/neutron: fix DHCP port changed when dhcp-agent restart https://review.openstack.org/162522 | 07:54 |
*** adelinatuvenie has joined #openstack-neutron | 08:00 | |
*** pask81 has joined #openstack-neutron | 08:00 | |
*** coolsvap|afk is now known as coolsvap | 08:00 | |
openstackgerrit | enikanorov proposed openstack/neutron: Improve performance of _get_security_group_member_ips https://review.openstack.org/162576 | 08:03 |
*** nuritv has joined #openstack-neutron | 08:05 | |
*** jyuso1 has joined #openstack-neutron | 08:05 | |
*** bobmel has quit IRC | 08:05 | |
*** erkules_ is now known as erkules | 08:09 | |
*** erkules has joined #openstack-neutron | 08:09 | |
*** safchain has joined #openstack-neutron | 08:13 | |
*** liusheng has joined #openstack-neutron | 08:14 | |
*** kawa2014 has joined #openstack-neutron | 08:14 | |
*** matrohon has joined #openstack-neutron | 08:15 | |
*** moshele has joined #openstack-neutron | 08:15 | |
*** yamahata has joined #openstack-neutron | 08:17 | |
*** luqas has joined #openstack-neutron | 08:19 | |
openstackgerrit | Merged openstack/neutron: In Arista ML2 delete tenant without any resources https://review.openstack.org/162778 | 08:20 |
*** gridinv has joined #openstack-neutron | 08:23 | |
*** ronis has quit IRC | 08:24 | |
*** claudiub has joined #openstack-neutron | 08:25 | |
*** adelinatuvenie has quit IRC | 08:27 | |
*** gridinv_ has joined #openstack-neutron | 08:28 | |
*** gridinv has quit IRC | 08:28 | |
*** lukasa has joined #openstack-neutron | 08:30 | |
openstackgerrit | Sergey Belous proposed openstack/neutron: Added check for emptyness where in_ is being used https://review.openstack.org/150034 | 08:32 |
*** gridinv_ has quit IRC | 08:33 | |
*** adelinatuvenie has joined #openstack-neutron | 08:34 | |
*** yfried_ is now known as yfried|afk | 08:36 | |
*** [1]adelinatuveni has joined #openstack-neutron | 08:37 | |
*** jlanoux has joined #openstack-neutron | 08:38 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 08:39 |
*** adelinatuvenie has quit IRC | 08:39 | |
*** [1]adelinatuveni is now known as adelinatuvenie | 08:39 | |
*** nyechiel has joined #openstack-neutron | 08:39 | |
*** salv-orlando has joined #openstack-neutron | 08:42 | |
*** vmtrooper has joined #openstack-neutron | 08:45 | |
*** salv-orlando has quit IRC | 08:46 | |
*** akamyshnikova has joined #openstack-neutron | 08:47 | |
*** shivharis has joined #openstack-neutron | 08:48 | |
*** shivharis has quit IRC | 08:49 | |
*** jpena has joined #openstack-neutron | 08:50 | |
*** irenab has quit IRC | 08:51 | |
*** vmtrooper has quit IRC | 08:51 | |
*** topshare_ has joined #openstack-neutron | 08:51 | |
*** topshare_ has quit IRC | 08:51 | |
*** irenab has joined #openstack-neutron | 08:52 | |
*** alexpilotti_ has joined #openstack-neutron | 08:52 | |
*** topshare_ has joined #openstack-neutron | 08:52 | |
*** topshare has quit IRC | 08:53 | |
*** alexpilotti has quit IRC | 08:54 | |
*** alexpilotti_ is now known as alexpilotti | 08:54 | |
*** yfried|afk is now known as yfried_ | 08:55 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron: Prevent calling waitall() inside a GreenPool's greenthread https://review.openstack.org/157320 | 08:58 |
*** veena has joined #openstack-neutron | 09:01 | |
*** luqas has quit IRC | 09:02 | |
*** devvesa has joined #openstack-neutron | 09:02 | |
*** baoli has joined #openstack-neutron | 09:04 | |
*** gildub has joined #openstack-neutron | 09:06 | |
*** pradipta has quit IRC | 09:06 | |
*** bobmel has joined #openstack-neutron | 09:09 | |
*** ygbo has joined #openstack-neutron | 09:10 | |
*** dims has joined #openstack-neutron | 09:11 | |
*** apuimedo has joined #openstack-neutron | 09:16 | |
*** kawa2014 has quit IRC | 09:16 | |
*** kawa2014 has joined #openstack-neutron | 09:17 | |
*** Murali has joined #openstack-neutron | 09:18 | |
*** dims has quit IRC | 09:18 | |
*** aix has joined #openstack-neutron | 09:19 | |
*** markvoelker has quit IRC | 09:22 | |
veena | Hi, 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 |
veena | How do I solve these errors? | 09:23 |
*** neeti has quit IRC | 09:28 | |
*** neeti has joined #openstack-neutron | 09:28 | |
*** david-lyle_afk has quit IRC | 09:30 | |
*** david-lyle_afk has joined #openstack-neutron | 09:30 | |
anteaya | I just checked the wikipage for the upcoming Neutron meeting: Monday (3/10/2015) at 1400 UTC on #openstack-meeting | 09:31 |
anteaya | so it is helpful when the calendar date and the day of the week sync | 09:32 |
anteaya | as they don't now and didn't last week | 09:32 |
*** kickinz1 is now known as kickinz1|afk | 09:32 | |
*** kickinz1|afk is now known as kickinz1 | 09:33 | |
*** aepifanov has joined #openstack-neutron | 09:33 | |
*** jistr has quit IRC | 09:34 | |
*** Flcn has joined #openstack-neutron | 09:34 | |
*** yamamoto has quit IRC | 09:35 | |
*** ihrachyshka has joined #openstack-neutron | 09:38 | |
*** yuanying has quit IRC | 09:39 | |
*** roeyc has joined #openstack-neutron | 09:40 | |
*** adelinatuvenie has quit IRC | 09:42 | |
*** Bhavya has quit IRC | 09:43 | |
*** jistr has joined #openstack-neutron | 09:46 | |
*** amotoki_ has joined #openstack-neutron | 09:48 | |
*** amotoki_ has quit IRC | 09:48 | |
*** lsg has quit IRC | 09:48 | |
*** shivharis has joined #openstack-neutron | 09:48 | |
*** shivharis has quit IRC | 09:50 | |
openstackgerrit | yalei wang proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test https://review.openstack.org/160051 | 09:50 |
openstackgerrit | yalei wang proposed openstack/neutron: Add portsecurity extension support https://review.openstack.org/126552 | 09:50 |
*** afazekas has joined #openstack-neutron | 09:52 | |
*** dims has joined #openstack-neutron | 09:56 | |
*** popw has quit IRC | 10:00 | |
*** xuhanp has quit IRC | 10:03 | |
*** xuhanp_ has quit IRC | 10:03 | |
*** _cjones_ has joined #openstack-neutron | 10:03 | |
*** _cjones_ has quit IRC | 10:04 | |
*** anish__ has quit IRC | 10:04 | |
*** _cjones_ has joined #openstack-neutron | 10:05 | |
*** trinaths has left #openstack-neutron | 10:07 | |
*** jp_at_hp has joined #openstack-neutron | 10:08 | |
*** _cjones_ has quit IRC | 10:09 | |
*** ronis has joined #openstack-neutron | 10:11 | |
*** marrusl has quit IRC | 10:14 | |
*** marrusl has joined #openstack-neutron | 10:17 | |
*** chlong has joined #openstack-neutron | 10:17 | |
*** neeti has quit IRC | 10:18 | |
*** irenab has quit IRC | 10:22 | |
*** topshare_ has quit IRC | 10:25 | |
*** irenab has joined #openstack-neutron | 10:25 | |
*** yfauser has joined #openstack-neutron | 10:26 | |
*** yfauser has left #openstack-neutron | 10:26 | |
*** topshare has joined #openstack-neutron | 10:26 | |
*** topshare has quit IRC | 10:28 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Migrate to oslo.log https://review.openstack.org/159638 | 10:30 |
*** jamie_h has joined #openstack-neutron | 10:31 | |
*** aix has quit IRC | 10:32 | |
*** coolsvap is now known as coolsvap|afk | 10:33 | |
*** yfried_ is now known as yfried|afk | 10:34 | |
*** vmtrooper has joined #openstack-neutron | 10:34 | |
*** adelinatuvenie has joined #openstack-neutron | 10:35 | |
*** pradip_vedams has quit IRC | 10:36 | |
*** neeti has joined #openstack-neutron | 10:36 | |
*** neeti has quit IRC | 10:36 | |
*** chandankumar has quit IRC | 10:37 | |
*** yfried|afk is now known as yfried_ | 10:38 | |
*** neeti has joined #openstack-neutron | 10:38 | |
*** vmtrooper has quit IRC | 10:39 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron: Prevent calling waitall() inside a GreenPool's greenthread https://review.openstack.org/157320 | 10:44 |
*** irenab has quit IRC | 10:47 | |
*** salv-orlando has joined #openstack-neutron | 10:48 | |
*** irenab has joined #openstack-neutron | 10:48 | |
*** pradipta has joined #openstack-neutron | 10:48 | |
*** shivharis has joined #openstack-neutron | 10:49 | |
*** chandankumar has joined #openstack-neutron | 10:50 | |
*** chandankumar has quit IRC | 10:50 | |
*** chandankumar has joined #openstack-neutron | 10:51 | |
*** chlong has quit IRC | 10:51 | |
*** shivharis has quit IRC | 10:51 | |
*** markvoelker has joined #openstack-neutron | 10:53 | |
*** salv-orlando has quit IRC | 10:53 | |
*** baoli has quit IRC | 10:54 | |
*** stanzgy_ has quit IRC | 10:55 | |
*** salv-orlando has joined #openstack-neutron | 10:55 | |
*** stanzgy has quit IRC | 10:55 | |
*** pask81 has quit IRC | 10:56 | |
*** aix has joined #openstack-neutron | 10:57 | |
*** pc_m has joined #openstack-neutron | 11:00 | |
*** irenab has quit IRC | 11:01 | |
*** kawa2014 has quit IRC | 11:04 | |
*** moha_hunt has joined #openstack-neutron | 11:05 | |
*** kawa2014 has joined #openstack-neutron | 11:05 | |
*** alexpilotti_ has joined #openstack-neutron | 11:08 | |
*** Krast has quit IRC | 11:09 | |
*** alexpilotti has quit IRC | 11:10 | |
*** alexpilotti_ is now known as alexpilotti | 11:10 | |
*** Guest69217 has quit IRC | 11:13 | |
*** Longgeek has quit IRC | 11:14 | |
*** diegows has joined #openstack-neutron | 11:20 | |
*** shaohe_feng_ has quit IRC | 11:20 | |
*** alexpilotti has quit IRC | 11:21 | |
*** kawa2014 has quit IRC | 11:21 | |
*** kawa2014 has joined #openstack-neutron | 11:21 | |
*** shaohe_robot has joined #openstack-neutron | 11:21 | |
openstackgerrit | enikanorov proposed openstack/neutron: Don't notify dead DHCP agent of removed networks https://review.openstack.org/162559 | 11:26 |
*** yatin has quit IRC | 11:28 | |
*** pradipta has quit IRC | 11:29 | |
*** jlanoux_ has joined #openstack-neutron | 11:29 | |
*** jlanoux has quit IRC | 11:32 | |
*** absubram has quit IRC | 11:34 | |
*** Longgeek has joined #openstack-neutron | 11:36 | |
openstackgerrit | Merged openstack/neutron: Added check for emptyness where in_ is being used https://review.openstack.org/150034 | 11:41 |
pc_m | mestery: ping? | 11:43 |
pc_m | salv-orlando: ping | 11:44 |
salv-orlando | hi pc_m | 11:44 |
*** yatin has joined #openstack-neutron | 11:44 | |
pc_m | salv-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/+A | 11:44 |
pc_m | (or I can +A) | 11:45 |
salv-orlando | pc_m: ah is neutron-vpnaas completely blocked now? | 11:45 |
pc_m | yup | 11:45 |
salv-orlando | pc_m: approved | 11:46 |
*** mwagner_lap has quit IRC | 11:46 | |
pc_m | salv-orlando: fails all jenkins jobs | 11:46 |
pc_m | salv-orlando: Thank you! | 11:46 |
salv-orlando | pc_m: np | 11:46 |
*** jlanoux has joined #openstack-neutron | 11:46 | |
*** sbelous has quit IRC | 11:47 | |
*** irenab has joined #openstack-neutron | 11:47 | |
*** jlanoux_ has quit IRC | 11:49 | |
*** shivharis has joined #openstack-neutron | 11:50 | |
*** sbelous has joined #openstack-neutron | 11:50 | |
*** yatin has quit IRC | 11:51 | |
*** shivharis has quit IRC | 11:52 | |
*** coolsvap|afk is now known as coolsvap | 11:54 | |
*** Murali has quit IRC | 11:55 | |
*** baoli has joined #openstack-neutron | 11:58 | |
*** gildub has quit IRC | 12:00 | |
*** karimb has joined #openstack-neutron | 12:01 | |
*** yfried_ is now known as yfried|afk | 12:02 | |
*** yatin has joined #openstack-neutron | 12:04 | |
*** yfried|afk is now known as yfried_ | 12:05 | |
openstackgerrit | Claudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 12:06 |
*** igordcard_afk has joined #openstack-neutron | 12:11 | |
*** igordcard_afk has quit IRC | 12:11 | |
*** igordcard has joined #openstack-neutron | 12:13 | |
*** gildub has joined #openstack-neutron | 12:13 | |
*** raildo_away is now known as raildo | 12:14 | |
*** igordcard has quit IRC | 12:15 | |
*** sambetts has quit IRC | 12:16 | |
*** Tusker has quit IRC | 12:17 | |
*** igordcard has joined #openstack-neutron | 12:17 | |
*** thomasem has quit IRC | 12:18 | |
*** thomasem has joined #openstack-neutron | 12:18 | |
*** gridinv_ has joined #openstack-neutron | 12:19 | |
*** bjornar has quit IRC | 12:20 | |
*** sambetts has joined #openstack-neutron | 12:21 | |
*** vmtrooper has joined #openstack-neutron | 12:23 | |
*** gridinv_ has quit IRC | 12:23 | |
*** dims has quit IRC | 12:25 | |
*** gildub has quit IRC | 12:25 | |
*** dims has joined #openstack-neutron | 12:25 | |
*** mwagner_lap has joined #openstack-neutron | 12:26 | |
*** neeti has quit IRC | 12:27 | |
*** dane_leblanc has joined #openstack-neutron | 12:27 | |
*** vmtrooper has quit IRC | 12:28 | |
*** chandankumar has quit IRC | 12:35 | |
*** amuller has joined #openstack-neutron | 12:37 | |
*** vthapar has quit IRC | 12:38 | |
*** markvoelker has quit IRC | 12:38 | |
*** markvoelker has joined #openstack-neutron | 12:39 | |
HenryG | amotoki: ping | 12:39 |
mestery | Folks, 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 DST | 12:41 | |
salv-orlando | mestery: that obviously applies only to folks in the US | 12:42 |
salv-orlando | it's the same time for me and all the other communists out there I guess | 12:42 |
*** markvoelker has quit IRC | 12:43 | |
ihrachyshka | :) | 12:43 |
*** mwagner_lap has quit IRC | 12:43 | |
pc_m | mestery: ping | 12:44 |
*** mwagner_lap has joined #openstack-neutron | 12:44 | |
ihrachyshka | mestery, I also have a tiny thing to discuss with you. ping when avail please. | 12:45 |
mestery | pc_m: pong | 12:47 |
mestery | ihrachyshka: Good morning/afternoon! | 12:47 |
pc_m | mestery: 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 IRC | 12:47 | |
mestery | pc_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-neutron | 12:48 | |
*** jistr has joined #openstack-neutron | 12:48 | |
pc_m | mestery: 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 |
ihrachyshka | mestery, 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 |
mestery | ihrachyshka: 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 IRC | 12:50 | |
*** shivharis has joined #openstack-neutron | 12:51 | |
*** annegentle has joined #openstack-neutron | 12:51 | |
*** ihrachyshka has quit IRC | 12:51 | |
*** ihrachyshka has joined #openstack-neutron | 12:51 | |
ihrachyshka | mestery, sounds fine for me | 12:52 |
mestery | ihrachyshka: Thank you! | 12:52 |
pc_m | mestery: Does only commit owner have the ability to rebase via Gerrit button? | 12:52 |
openstackgerrit | Kyle Mestery proposed openstack/python-neutronclient: Copy functional tests from tempest cli https://review.openstack.org/162848 | 12:52 |
*** shivharis has quit IRC | 12:53 | |
pc_m | https://review.openstack.org/#/c/162887 failed verify... looks like Jenkins is saying a rebase is needed. | 12:53 |
*** yfried_ is now known as yfried|afk | 12:54 | |
salv-orlando | ihrachyshka: 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 libraries | 12:54 |
mestery | pc_m: Nope, others can as well. | 12:54 |
*** cbouch has joined #openstack-neutron | 12:54 | |
pc_m | mestery: I don;t see the button... | 12:54 |
salv-orlando | this brings on us synchronization onus, but removes a source of instability due to change in neutron.openstack.common | 12:54 |
mestery | pc_m: You can just pull it down, rebase it, and push it back up. | 12:54 |
* mestery will BRB | 12:55 | |
ihrachyshka | salv-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/VendorSplitPackaging | 12:55 |
pc_m | mestery: Right. Just wondered about the button. Will do. | 12:55 |
salv-orlando | ihrachyshka: ah thanks a lot | 12:55 |
salv-orlando | I will pass that on to the guys that build our own internal packages | 12:55 |
ihrachyshka | it's WIP and I'll fill it once I find more issues | 12:55 |
ihrachyshka | thanks | 12:55 |
*** irenab has quit IRC | 12:57 | |
pc_m | mestery: Can you look at https://review.openstack.org/#/c/162887? | 12:57 |
pc_m | mestery: It has X for Jenkins and says verify failed, needing rebase. | 12:57 |
pc_m | mestery: I pulled it down, fetched origin, rebased, and tried to push up, and it says rejected (no new changes). | 12:58 |
salv-orlando | ihrachyshka: I found annoying that Apple's mail auto correct insists of changing your name into "char" | 12:58 |
* pc_m confused | 12:58 | |
ihrachyshka | salv-orlando, ok sir. I will consider changing it | 12:58 |
*** dboik has joined #openstack-neutron | 12:59 | |
amotoki | HenryG: pong | 13:04 |
HenryG | amotoki: Will you update https://wiki.openstack.org/wiki/Neutron/NEC_OpenFlow_Plugin for the nec decomposition? | 13:04 |
*** jistr has quit IRC | 13:04 | |
*** emagana has joined #openstack-neutron | 13:05 | |
HenryG | amotoki: Say yes so I can approve your decomp patch. :) | 13:05 |
*** jistr has joined #openstack-neutron | 13:05 | |
*** coolsvap is now known as coolsvap|afk | 13:06 | |
*** banix has joined #openstack-neutron | 13:06 | |
amotoki | HenryG: sure. It is in my mind, but it is just a bit lower priority than other stuff :-( | 13:06 |
HenryG | amotoki: no problem | 13:06 |
amotoki | HenryG: i need to move from now and will be back in the team meeting. I will update after that. Thanks. | 13:08 |
HenryG | amotoki: No rush, your word is good. Thanks. | 13:08 |
*** cference has joined #openstack-neutron | 13:10 | |
*** mdavidson has joined #openstack-neutron | 13:10 | |
openstackgerrit | enikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations https://review.openstack.org/149261 | 13:12 |
*** amotoki has quit IRC | 13:12 | |
*** markvoelker has joined #openstack-neutron | 13:13 | |
*** yfried|afk is now known as yfried_ | 13:14 | |
mestery | pc_m: Let me try | 13:15 |
pc_m | mestery: I did a 'recheck' command, not sure if it would go through. | 13:15 |
*** anilvenkata has left #openstack-neutron | 13:16 | |
*** cference has quit IRC | 13:16 | |
*** cference has joined #openstack-neutron | 13:17 | |
*** sripriya has joined #openstack-neutron | 13:20 | |
amuller | salv-orlando: ping about https://review.openstack.org/149261 | 13:20 |
amuller | if you happen to be up | 13:20 |
*** adelinatuvenie has quit IRC | 13:21 | |
*** yfried_ is now known as yfried|afk | 13:24 | |
*** changbl has quit IRC | 13:26 | |
*** bjornar has joined #openstack-neutron | 13:26 | |
*** rkukura has quit IRC | 13:27 | |
*** yamamoto has joined #openstack-neutron | 13:28 | |
*** yamamoto has quit IRC | 13:29 | |
*** yamamoto has joined #openstack-neutron | 13:29 | |
*** veena has quit IRC | 13:30 | |
*** topshare has joined #openstack-neutron | 13:31 | |
garyk | mestery: quick question - when i run tox on ovn it blows up. do you experience the same? | 13:32 |
mestery | garyk: Checking ... | 13:33 |
mestery | garyk: Exact command? | 13:33 |
*** ajo has quit IRC | 13:33 | |
garyk | mestery: tox | 13:33 |
*** britthouser has joined #openstack-neutron | 13:33 | |
*** yfried|afk is now known as yfried_ | 13:34 | |
mestery | garyk: 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-neutron | 13:34 | |
garyk | mestery: ok, thanks. but even when i do py27 it failed | 13:35 |
*** topshare has quit IRC | 13:35 | |
mestery | garyk: Really? py27 worked fine for me (and pep8) | 13:35 |
*** tiswanso has joined #openstack-neutron | 13:36 | |
*** saikrishna has joined #openstack-neutron | 13:36 | |
garyk | mestery: please see http://paste.openstack.org/show/191278/ - maybe you can find what i did | 13:36 |
*** txomon|fon has joined #openstack-neutron | 13:36 | |
*** txomon|fon has joined #openstack-neutron | 13:36 | |
mestery | garyk: That looks like the same as for me, let me blow away my .tox and have it recreate | 13:37 |
*** rkukura has joined #openstack-neutron | 13:37 | |
mestery | garyk: Worked fine for me: http://paste.openstack.org/show/191284/ | 13:38 |
garyk | mestery: no idea… | 13:40 |
*** zhipeng has joined #openstack-neutron | 13:41 | |
*** chandankumar has joined #openstack-neutron | 13:43 | |
*** salv-orlando has quit IRC | 13:46 | |
*** armax has joined #openstack-neutron | 13:46 | |
garyk | mestery: can we start bitchin about not getting reviews in the ovn repo? | 13:47 |
mestery | garyk: lol | 13:47 |
garyk | damn, i posetd a patch 30 secodns ago and have not got a review. | 13:47 |
mestery | garyk: rofl | 13:47 |
mestery | garyk: I just merged one, is that not fast enough? ;) | 13:47 |
openstackgerrit | Merged openstack/neutron: Improve performance of _get_security_group_member_ips https://review.openstack.org/162576 | 13:47 |
garyk | mestery: :) | 13:48 |
garyk | mestery: 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 issues | 13:48 |
*** salv-orlando has joined #openstack-neutron | 13:48 | |
mestery | garyk: I agree, I'll do the same for networking-odl, thanks! | 13:48 |
garyk | cool. | 13:48 |
mestery | garyk: networking-odl already had it, I forgot I added that a while back :) | 13:50 |
ihrachyshka | salv-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 IRC | 13:51 | |
*** shivharis has joined #openstack-neutron | 13:51 | |
*** shivharis has quit IRC | 13:53 | |
*** pavel_bondar has quit IRC | 13:54 | |
*** yamahata has quit IRC | 13:56 | |
*** yamahata has joined #openstack-neutron | 13:57 | |
*** r-daneel has joined #openstack-neutron | 13:58 | |
*** marun has joined #openstack-neutron | 13:58 | |
*** irenab has joined #openstack-neutron | 13:59 | |
*** carl_baldwin has joined #openstack-neutron | 13:59 | |
*** ajo has joined #openstack-neutron | 13:59 | |
*** adelinatuvenie has quit IRC | 14:00 | |
*** rushil has joined #openstack-neutron | 14:01 | |
*** ajo has quit IRC | 14:01 | |
*** watanabe_isao has joined #openstack-neutron | 14:01 | |
*** ajo has joined #openstack-neutron | 14:02 | |
*** jprovazn has joined #openstack-neutron | 14:02 | |
*** ajo has quit IRC | 14:02 | |
*** ajo has joined #openstack-neutron | 14:03 | |
*** mlavalle has joined #openstack-neutron | 14:03 | |
itzikb | armax: ping | 14:03 |
*** yshafrir has quit IRC | 14:03 | |
*** saikrishna has quit IRC | 14:04 | |
*** ajo has quit IRC | 14:04 | |
*** shwetaap has joined #openstack-neutron | 14:05 | |
*** ajo has joined #openstack-neutron | 14:06 | |
*** britthouser has quit IRC | 14:07 | |
*** annegentle has joined #openstack-neutron | 14:08 | |
*** zzzeek has joined #openstack-neutron | 14:08 | |
*** zhipeng_ has joined #openstack-neutron | 14:08 | |
openstackgerrit | Merged openstack/neutron: NEC plugin code split https://review.openstack.org/162103 | 14:10 |
*** absubram has joined #openstack-neutron | 14:10 | |
*** zhipeng has quit IRC | 14:10 | |
*** annegentle has quit IRC | 14:11 | |
*** wznoinsk has joined #openstack-neutron | 14:11 | |
*** vmtrooper has joined #openstack-neutron | 14:12 | |
*** adelinatuvenie has joined #openstack-neutron | 14:13 | |
*** yfried__ has joined #openstack-neutron | 14:14 | |
*** yfried_ has quit IRC | 14:15 | |
*** ajo has quit IRC | 14:15 | |
*** ajo has joined #openstack-neutron | 14:16 | |
*** garyk has quit IRC | 14:16 | |
*** shivharis has joined #openstack-neutron | 14:17 | |
*** vmtrooper has quit IRC | 14:17 | |
*** shivharis has quit IRC | 14:17 | |
*** garyk has joined #openstack-neutron | 14:20 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Migrate to oslo.log https://review.openstack.org/159638 | 14:20 |
*** topshare has joined #openstack-neutron | 14:21 | |
*** topshare has quit IRC | 14:21 | |
*** amotoki has joined #openstack-neutron | 14:21 | |
*** jckasper has joined #openstack-neutron | 14:22 | |
*** topshare has joined #openstack-neutron | 14:22 | |
*** topshare has quit IRC | 14:22 | |
*** prad has joined #openstack-neutron | 14:23 | |
*** ChuckC has quit IRC | 14:23 | |
*** topshare has joined #openstack-neutron | 14:23 | |
*** Guest75125 has joined #openstack-neutron | 14:30 | |
*** emagana has quit IRC | 14:31 | |
sbelous | garyk: 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-neutron | 14:34 | |
*** zz_jgrimm is now known as jgrimm | 14:35 | |
*** carl_baldwin_ has joined #openstack-neutron | 14:35 | |
*** carl_baldwin has quit IRC | 14:37 | |
*** carl_baldwin_ is now known as carl_baldwin | 14:37 | |
*** markvoelker has quit IRC | 14:37 | |
openstackgerrit | Moshe Levi proposed openstack/neutron: Move mlnx agent to be under ml2/drivers/mlnx https://review.openstack.org/162693 | 14:37 |
*** markvoelker has joined #openstack-neutron | 14:37 | |
*** absubram has quit IRC | 14:39 | |
*** chandankumar has quit IRC | 14:39 | |
*** garyk has quit IRC | 14:39 | |
*** budnyjj has joined #openstack-neutron | 14:40 | |
*** markvoelker has quit IRC | 14:42 | |
*** pask81 has joined #openstack-neutron | 14:42 | |
*** achanda has quit IRC | 14:43 | |
*** Guest75125 is now known as notsogentle | 14:43 | |
*** notsogentle is now known as agentle_ | 14:43 | |
*** rwsu has joined #openstack-neutron | 14:44 | |
*** sballe__ has joined #openstack-neutron | 14:44 | |
*** vishwana_ has joined #openstack-neutron | 14:44 | |
*** xgerman_ has joined #openstack-neutron | 14:46 | |
*** vishwan__ has joined #openstack-neutron | 14:46 | |
*** vishwanathj has quit IRC | 14:48 | |
*** markvoelker has joined #openstack-neutron | 14:49 | |
*** vishwana_ has quit IRC | 14:49 | |
*** SridharRamaswamy has joined #openstack-neutron | 14:50 | |
*** jaypipes has quit IRC | 14:53 | |
*** nyechiel_ has joined #openstack-neutron | 14:56 | |
*** vishwan__ has quit IRC | 14:56 | |
*** vishwanathj has joined #openstack-neutron | 14:57 | |
*** anshul has quit IRC | 14:57 | |
*** dims has quit IRC | 14:58 | |
*** carl_baldwin has quit IRC | 14:59 | |
*** nyechiel has quit IRC | 14:59 | |
marun | dougwig: 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-neutron | 15:00 | |
*** SridharRamaswamy has quit IRC | 15:01 | |
*** dimsum__ has joined #openstack-neutron | 15:01 | |
*** fredlhsu has joined #openstack-neutron | 15:01 | |
dougwig | marun: 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 |
banix | enikanorov: mestery: Will there be a milestone attached to this bug which has been fixed: https://bugs.launchpad.net/neutron/+bug/1373634 | 15:01 |
openstack | Launchpad 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-neutron | 15:02 | |
*** budnyjj has quit IRC | 15:02 | |
ihrachyshka | dougwig, it's not just import (at least not in all cases) | 15:02 |
dougwig | marun: pinning unit tests is on the lbaas meeting agenda, which starts in 58 minutes. | 15:02 |
ihrachyshka | dougwig, option registration, library setup changed, audit dropped... | 15:03 |
marun | dougwig: ok, where? I'm happy to join. | 15:03 |
dougwig | marun: #openstack-meeting-4 | 15:03 |
marun | dougwig: re: driver, not sure you can escape multiple branches over the long term. | 15:03 |
dougwig | marun: https://wiki.openstack.org/wiki/Network/LBaaS | 15:03 |
dougwig | marun: i can also ping you when we get to that item. | 15:03 |
marun | dougwig: in-process dependency conflict would seem inevitable :/ | 15:03 |
*** achanda has joined #openstack-neutron | 15:03 | |
marun | dougwig: thanks | 15:04 |
*** ChuckC has joined #openstack-neutron | 15:04 | |
dougwig | marun: yeah, my goal to be lazy is not something to base process around, agreed. | 15:04 |
ajo | salv-orlando: quesiton, in the OVS agent, do you know if vlan mappings are consistent across agent restarts? | 15:04 |
marun | dougwig: 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 |
ajo | quesiton? question... | 15:04 |
*** agentle_ has quit IRC | 15:05 | |
marun | ajo: local vlan mappings are not | 15:05 |
ajo | hmmm, bad | 15:05 |
marun | ajo: they are stored in memory | 15:05 |
ajo | salv-orlando ^ | 15:05 |
marun | ajo: why? | 15:05 |
ajo | because there is a patch to re-use that mapping for connection tracking... | 15:05 |
marun | ajo: hmmmm | 15:05 |
marun | ajo: maybe I'm wrong... | 15:05 |
*** achanda has quit IRC | 15:05 | |
ajo | and that means, in-memory connection tracking in kernel marks, will out-sync from the marks we set in iptables | 15:05 |
* marun goes to investigate | 15:06 | |
* ajo goes too | 15:06 | |
dougwig | marun, 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 disappearing | 15:07 |
dougwig | interfaces? | 15:07 |
marun | dougwig: yeah, maybe wrappers of some kind? | 15:07 |
*** Murali has joined #openstack-neutron | 15:07 | |
marun | dougwig: that could be deprecated upon graduation to oslo library? | 15:07 |
ihrachyshka | dougwig, oslo incubator libraries must not be considered stable | 15:08 |
marun | ihrachyshka: so we either have to provide a stable interface in neutron-lib or simply not expose | 15:08 |
dougwig | ihrachyshka: right, but by using them a dependencies in another repo, we are implicitly treating them as such. | 15:08 |
dougwig | marun: yes, that'd be one way. | 15:08 |
ihrachyshka | dougwig, 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 miserable | 15:09 |
marun | dougwig: if we don't expose, maybe it's enough for *aas to import oslo incubator themselves. | 15:09 |
ihrachyshka | marun, for neutron-lib, I would put it under neutron._openstack.common | 15:09 |
marun | ihrachyshka: ah, so maybe next cycle *aas start copying? | 15:09 |
ihrachyshka | to indicate private nature of it | 15:09 |
marun | ihrachyshka: I guess the implication is that every repo needs to copy incubator | 15:09 |
dougwig | i'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 |
ihrachyshka | marun, that may be the case, the number of modules from incubator is now manageable | 15:10 |
marun | ihrachyshka: e.g. neutron and neutron-lib and neutron-vpnaas | 15:10 |
marun | ihrachyshka: and then we have to hope that the implementations don't conflict | 15:10 |
marun | hope -> trust our tests ? ;) | 15:10 |
dougwig | it 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 |
ihrachyshka | marun, we started with ~20 and now are around ~10, with most of them used isolated in neutron repo | 15:10 |
*** vivek-ebay has joined #openstack-neutron | 15:10 | |
ihrachyshka | marun, yeah, implementation conflict is a scary thing | 15:10 |
ihrachyshka | marun, for libraries, we solve it with proposal bot | 15:11 |
ihrachyshka | that's why we should run from incubator modules as far as possible. those are evil. | 15:11 |
dougwig | marun, ihrachyshka - about to drive to work. offline for a bit. | 15:11 |
marun | ihrachyshka: agreed | 15:11 |
marun | dougwig: ttiab | 15:12 |
openstackgerrit | rcurran proposed openstack/neutron: ML2 cisco_nexus MD: upstream updates required https://review.openstack.org/163040 | 15:12 |
*** tsufiev has quit IRC | 15:12 | |
ajo | marun, it looks to me like we don't have any persistence mechanism for the vlan mappings | 15:12 |
ajo | salv-orlando ^ :/ | 15:13 |
marun | ajo: we definitely don't persist it | 15:13 |
ajo | ok, that's going to be an issue | 15:13 |
*** tsufiev has joined #openstack-neutron | 15:13 | |
ajo | looking again at the patch.. | 15:13 |
ajo | (https://review.openstack.org/#/c/118274/22) | 15:13 |
*** garyk has joined #openstack-neutron | 15:13 | |
marun | ajo: https://github.com/openstack/neutron/blob/master/neutron/plugins/openvswitch/agent/ovs_neutron_agent.py#L680 | 15:13 |
*** tsufiev has quit IRC | 15:14 | |
marun | ajo: an ovs or agent restart will guarantee a fresh local vlan | 15:14 |
ajo | ahaa | 15:14 |
ajo | thanks maru | 15:14 |
*** yfried has joined #openstack-neutron | 15:14 | |
marun | ajo: not great news, I know | 15:14 |
marun | ajo: is there a reason the non-local vlan can't be used? | 15:14 |
* marun goes to read patch | 15:14 | |
*** vivek-ebay has quit IRC | 15:15 | |
openstackgerrit | Merged openstack/neutron: Fix wrong log output in neutron/neutron/agent/linux/dhcp.py https://review.openstack.org/162585 | 15:15 |
garyk | sbelous: done | 15:15 |
ajo | marun, that's hidden somewhere, there's a tiny ID limit for connection tracking marks on the kernel | 15:15 |
*** radez_g0n3 is now known as radez | 15:15 | |
ajo | can't remember it now exactly, but around 16bits | 15:15 |
*** vivek-ebay has joined #openstack-neutron | 15:15 | |
marun | ajo: so there might be another option... | 15:15 |
marun | ajo: well | 15:16 |
sbelous | garyk: thanks! | 15:16 |
marun | ajo: any option requires persistence | 15:16 |
marun | ajo: it's only a question of where | 15:16 |
ajo | I guess, yes... | 15:16 |
anteaya | mestery: I don't see edgar in channel | 15:16 |
anteaya | sc68cal: are you at the ops meetup? | 15:16 |
marun | ajo: I was thinking that we could add it to the port metadata that ovs retains | 15:16 |
marun | ajo: there's nothing stopping us from adding another field | 15:16 |
marun | ajo: that wouldn't be portable to lb, though. | 15:17 |
ajo | marun, that sounds interesting | 15:17 |
anteaya | sc68cal: can you shed more light on what edgar was saying in the meeting about nova net and ipv6? it didnt' make any sense to me | 15:17 |
yamamoto | dvr folks around? | 15:17 |
ajo | marun, btw, I think we can read the tag back from the ovs bridge, the same way we set it? | 15:17 |
*** yfried__ has quit IRC | 15:17 | |
marun | ajo: we can, yes. | 15:17 |
ajo | it would be the same, I guess | 15:17 |
marun | ajo: I'm not sure of the implications of doing so, though. | 15:17 |
*** chandankumar has joined #openstack-neutron | 15:17 | |
ajo | marun, you mean the metadata in ovs/ovsdb, or somewhere else? | 15:17 |
*** yatin has quit IRC | 15:17 | |
*** yfried_ has joined #openstack-neutron | 15:17 | |
marun | ajo: I mean, it should be possible to read all ports to reconstruct the table | 15:18 |
marun | ajo: we can id the ports based on metadata and then check their vlan tag | 15:18 |
ajo | marun, yes, I think that makes sense | 15:18 |
sc68cal | anteaya: yep, I'm at the summit | 15:18 |
sc68cal | let me read scrollback | 15:18 |
ajo | marun, I guess if we keep the net-id tied to the port metadata, in such case it would be very easy | 15:18 |
*** yfried has quit IRC | 15:18 | |
*** phschwartz has quit IRC | 15:18 | |
ajo | get all net-id / tag pairs.... and build a dictionary | 15:18 |
ajo | well.. build, the vlan mapping dict.. | 15:19 |
marun | ajo: I don't think we keep the net-id, only the iface id | 15:19 |
ajo | hmm | 15:19 |
*** phschwartz has joined #openstack-neutron | 15:19 | |
marun | ajo: but the iface/port id can be used to get the net-id easily enough | 15:19 |
sc68cal | anteaya: 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/1251235 | 15:19 |
openstack | Launchpad bug 1251235 in OpenStack Compute (nova) "IPv6 DAD failure due to hairpinning" [High,Fix released] - Assigned to Sean M. Collins (scollins) | 15:19 |
ajo | marun, 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 IRC | 15:20 | |
marun | ajo: it would need to be looked up, yes. | 15:20 |
ajo | most probably we're doing it already... | 15:20 |
marun | ajo: I think so. | 15:20 |
sc68cal | They also had some custom patches they are carrying internally around radvd management | 15:21 |
ajo | ok, I will re-check the implications of the patch + no-map-persistence, and propose the solution we were talking about | 15:21 |
ajo | thanks marun | 15:21 |
amuller | marun: Check out https://review.openstack.org/#/c/162811/ | 15:22 |
marun | ajo: np :) | 15:22 |
marun | amuller: ok | 15:22 |
amuller | Prepare to have your testing mind blown | 15:22 |
*** boris-42 has quit IRC | 15:22 | |
amuller | in a very limited capacity of the term 'blown' | 15:22 |
anteaya | sc68cal: okay so _one- operator is not what I am doing to steer my efforts aaround | 15:22 |
openstackgerrit | Timothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config https://review.openstack.org/153733 | 15:22 |
anteaya | sc68cal: I'm not sure what messageing came out around this | 15:22 |
*** david-lyle_afk is now known as david-lyle | 15:23 | |
marun | amuller: impressive | 15:23 |
anteaya | sc68cal: but many people do many things that is great for them and has no impact on my life | 15:23 |
anteaya | sc68cal: this is one of those instances | 15:23 |
anteaya | sc68cal: and do tell edgar I am disappointed that a report at the meeting consisted of one data point | 15:23 |
anteaya | without having the context of being one data point | 15:23 |
anteaya | and gives the impression we should redirect efforts | 15:24 |
amuller | marun: not sure how this relates to your blueprint about having the test tree mirror the directory paths of the non-test tree | 15:24 |
amuller | I lumped together the RPC tests in a single directory instead of having them spread around the tree | 15:24 |
*** tsufiev_ has joined #openstack-neutron | 15:24 | |
sc68cal | anteaya: agree - I've only heard rumors of one operator in China doing IPv6 successfully. This now makes two reports..... | 15:24 |
amuller | That made more sense to me | 15:24 |
anteaya | sc68cal: please try to convey my sentiments as fostering operators to have inappropriate expectations of this work is highly frustrating for me | 15:24 |
garyk | claudiub: please see https://review.openstack.org/163047 | 15:24 |
anteaya | sc68cal: the whole point was to tell operators what we are doing | 15:25 |
anteaya | sc68cal: not pivot because someone is trying to do something | 15:25 |
marun | amuller: it's only the unit test tree | 15:25 |
anteaya | sc68cal: I'm disappointed | 15:25 |
anteaya | sc68cal: not with you but edgar isn't in channel | 15:25 |
amuller | marun: 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 tests | 15:25 |
anteaya | sc68cal: thanks for being available | 15:26 |
amuller | marun: But I don't think those tests are 'unit' tests by some definitions | 15:26 |
*** imcsk8 has quit IRC | 15:26 | |
*** imcsk8 has joined #openstack-neutron | 15:26 | |
marun | amuller: there are different kinds of non-unit tests, but unit tests should only be locally targeted imho | 15:26 |
sc68cal | anteaya: Edgar is running a session on network performance, hence his unavailability | 15:26 |
marun | i.e. at functions in individual modules | 15:26 |
sc68cal | anteaya: I'm two feet away from him, so I will have him ping you when the session is over | 15:27 |
amuller | right | 15:27 |
marun | amuller: functional is a grab-bag for everything else at this point | 15:27 |
anteaya | please point him at the log so he can read my words when his if finished, if you could | 15:27 |
sc68cal | anteaya: will do | 15:27 |
*** zhipeng_ has quit IRC | 15:27 | |
anteaya | sc68cal: thanks, any idea what time? I'm trying to squeeze in breakfast | 15:27 |
marun | amuller: we may want to segment them at some point, but it's not clear to me how | 15:27 |
marun | amuller: so, your proposed structure lgtm | 15:27 |
sc68cal | anteaya: it just finished, so I will have him reach out now | 15:27 |
amuller | marun: 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 mocks | 15:28 |
*** sripriya has quit IRC | 15:28 | |
openstackgerrit | dql proposed openstack/neutron: fix DHCP port changed when dhcp-agent restart https://review.openstack.org/162522 | 15:28 |
marun | amuller: Please don't hesitate to refactor as necessary. | 15:28 |
anteaya | sc68cal: okay thanks | 15:28 |
*** emagana has joined #openstack-neutron | 15:28 | |
marun | amuller: refactoring code for testability > making tests unnecessarily complicated | 15:28 |
*** sripriya has joined #openstack-neutron | 15:28 | |
amuller | marun: 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 that | 15:29 |
amuller | I don't know, I'll think about it | 15:29 |
marun | amuller: object? | 15:29 |
*** numan has quit IRC | 15:29 | |
amuller | perhaps | 15:29 |
claudiub | garyk: sure, looking | 15:29 |
marun | amuller: my advice is useless without actually digging into code, of course | 15:29 |
*** ChuckC has quit IRC | 15:30 | |
emagana | sorry folks.. I had to moderate a session at the ops meetup and could not attend the rest of the neutron meeting | 15:31 |
garyk | thanks | 15:31 |
*** yog__ has quit IRC | 15:31 | |
*** bobmel has quit IRC | 15:32 | |
*** Murali has quit IRC | 15:32 | |
*** carl_baldwin has joined #openstack-neutron | 15:32 | |
anteaya | emagana: hi | 15:32 |
*** irenab has quit IRC | 15:32 | |
emagana | anteaya: Hello! | 15:33 |
anteaya | emagana: so I'll put my cards on the table | 15:33 |
*** bobmel has joined #openstack-neutron | 15:33 | |
emagana | anteaya: ? | 15:33 |
anteaya | emagana: what you presented at meeting really upset me | 15:33 |
anteaya | so let's start with this | 15:33 |
anteaya | what are you telling people about the nova net to neutron migration | 15:33 |
*** carl_baldwin has quit IRC | 15:33 | |
emagana | anteaya: 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 out | 15:34 |
anteaya | let's do so now please | 15:34 |
anteaya | as I would really like operators to have reasonable expectations | 15:34 |
anteaya | so do you want to start? | 15:34 |
emagana | anteaya: I share the gerrit review and I asked around for feedback. | 15:34 |
anteaya | I can listen first | 15:35 |
anteaya | ah | 15:35 |
anteaya | no | 15:35 |
anteaya | I'm not interested in feedback | 15:35 |
anteaya | I'm interested that they know | 15:35 |
anteaya | this has been going on for 3 years | 15:35 |
*** adelinatuvenie has quit IRC | 15:35 | |
anteaya | none of them will like it | 15:35 |
*** sripriya has quit IRC | 15:35 | |
anteaya | and I accept that | 15:35 |
emagana | anteaya: ok | 15:35 |
anteaya | this is what we are doing, end of story | 15:35 |
*** moshele has quit IRC | 15:35 | |
*** changbl has joined #openstack-neutron | 15:35 | |
anteaya | _now_ should someone want to get invovled | 15:35 |
anteaya | that is wonderful | 15:36 |
anteaya | and they are welcome | 15:36 |
*** evgenyf has joined #openstack-neutron | 15:36 | |
anteaya | but a bunch more storis of hwo what we are doing doens't suit them is not new nor is it helpful | 15:36 |
anteaya | how | 15:36 |
emagana | anteaya: That was actually my point, I believe that having them involved in the process will make it better | 15:36 |
anteaya | so anyone who wants to participate is welcome to attend the weekly meeting | 15:36 |
anteaya | tuesdays at 0900 utc | 15:36 |
anteaya | and there is an agenda | 15:36 |
anteaya | and they can review patches and test | 15:37 |
anteaya | anyone who tests I will listen to | 15:37 |
emagana | anteaya: Let me add that to the etherpad | 15:37 |
anteaya | thank you | 15:37 |
*** nyechiel_ has quit IRC | 15:37 | |
anteaya | if they want to be involved they can show up to the meeting and test code | 15:37 |
anteaya | that is the feedback I will listen to | 15:37 |
emagana | anteaya: 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 provides | 15:37 |
anteaya | okay well that is useful to | 15:38 |
anteaya | but not in the context of the migration story for me | 15:38 |
sc68cal | I felt like the operators are moving toe goalposts | 15:38 |
anteaya | but helpful for the tc | 15:38 |
anteaya | sc68cal: that is what they do | 15:38 |
sc68cal | first they wanted multi_host , we got it via DVR | 15:38 |
anteaya | which is why this has taken 3 years | 15:38 |
sc68cal | they don't like OVS now, that's the new object of dislike, they now demand it for linux bridge | 15:38 |
anteaya | and also why I only will listen to feedback to folks who show up to meetings and test code | 15:39 |
anteaya | sc68cal: colour me surprised | 15:39 |
sc68cal | I was glad it was only a couple minutes, I felt like neutron was the whipping boy | 15:39 |
emagana | anteaya: Let's make an extra effort to get these people on the summit sessions | 15:39 |
anteaya | no | 15:39 |
*** nyechiel_ has joined #openstack-neutron | 15:39 | |
anteaya | there will be no summit session about the migration | 15:39 |
marun | sc68cal: haters gotta hate ;) | 15:39 |
anteaya | I want table time on friday | 15:39 |
*** boris-42 has joined #openstack-neutron | 15:39 | |
marun | gotta -> gonna (oops) | 15:39 |
sc68cal | marun: haters ball was in session | 15:40 |
anteaya | but I have zero interest in have a room full of 400 people saying you need to make _me_ happy | 15:40 |
anteaya | I encourage table time on friday | 15:40 |
sc68cal | anteaya: +1 | 15:40 |
anteaya | for those who want to _work_ | 15:40 |
anteaya | workers welcome | 15:40 |
emagana | anteaya: sounds like a good idea! | 15:40 |
marun | anteaya: I don't really understand how ops should be driving this anyway | 15:40 |
marun | anteaya: works takes resources | 15:41 |
marun | anteaya: asking for things without providing resources? not going to accomplish anything | 15:41 |
anteaya | emagana: thank you | 15:41 |
anteaya | emagana sc68cal so do invite anyone who cares to the migration meeting tuesdays at 0900 utc in -meeting | 15:41 |
claudiub | garyk: done | 15:41 |
emagana | anteaya: sorry if this part got you upset, by not means I want people to get mad | 15:41 |
anteaya | folks who show up get heard | 15:41 |
sc68cal | anteaya: I will communicate that to operators here | 15:42 |
anteaya | emagana: no I understand, it just don't see the point in trying to make them happy | 15:42 |
anteaya | we won't | 15:42 |
anteaya | I have accepted that | 15:42 |
anteaya | sc68cal: thanks | 15:42 |
anteaya | marun: all fair points | 15:42 |
anteaya | okay thanks for your time and the channel space | 15:43 |
sc68cal | I brought up in the first session of today that Operators need to be invovled in specs | 15:43 |
anteaya | I need to get something to eat now | 15:43 |
anteaya | how did that go over? | 15:43 |
sc68cal | Good | 15:43 |
*** absubram has joined #openstack-neutron | 15:43 | |
anteaya | good | 15:43 |
emagana | anteaya: 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 all | 15:43 |
anteaya | glad to hear it | 15:43 |
garyk | claudiub: thanks! | 15:44 |
anteaya | emagana: thanks | 15:44 |
sc68cal | suggestion by Stefano Maffulli was to get specs sent to the operator's list | 15:44 |
anteaya | emagana: glad we could sync up | 15:44 |
sc68cal | sort of like how we get notifications in IRC | 15:44 |
emagana | anteaya: anytime! | 15:44 |
sc68cal | maybe on first patch | 15:44 |
anteaya | sc68cal: ah I doubt that will happen | 15:44 |
anteaya | sc68cal: better find monty before you broadcast that idea | 15:44 |
*** aepifanov has quit IRC | 15:44 | |
anteaya | they can just subscribe to email notifications on the specs projects of their choice | 15:45 |
sc68cal | good point | 15:45 |
anteaya | https://review.openstack.org/#/settings/projects | 15:45 |
emagana | whoever is really interested in the specs, they can do it.. all is open | 15:45 |
anteaya | watch a project, set email notifications | 15:45 |
marun | sc68cal: I'm not really sure how drive-by review by ops is going to help | 15:45 |
anteaya | teach them how to do that themselves | 15:45 |
*** watanabe_isao has quit IRC | 15:45 | |
marun | sc68cal: we need people who build relationships with the community | 15:45 |
marun | so that we know we can trust them | 15:46 |
sc68cal | marun: This is true, my hope was that having them review specs would help that | 15:46 |
*** thedodd has joined #openstack-neutron | 15:46 | |
sc68cal | Or at least get those introductions | 15:46 |
amuller | Multi host for LB is very wrong from Neutron's point of view | 15:46 |
marun | sc68cal: it could, as a first step, true. | 15:46 |
amuller | I hope no one is getting ideas here =D | 15:46 |
marun | amuller: wrong because we don't have the manpower? | 15:46 |
marun | sc68cal: what we're really lacking is engineering management | 15:46 |
marun | sc68cal: and product management | 15:47 |
*** kawa2014 has quit IRC | 15:47 | |
*** changbl has quit IRC | 15:47 | |
marun | sc68cal: we're so developer-centric that we as a community frequently lose sight of the bigger issues | 15:47 |
garyk | claudiub: can we approve https://review.openstack.org/#/c/156980/? | 15:47 |
*** kawa2014 has joined #openstack-neutron | 15:47 | |
marun | sc68cal: getting ops involved might help to some degree, but I think the root cause of our dysfunction will remain unaddressed | 15:47 |
marun | sc68cal: that's not to say that I don't think ops is important... | 15:48 |
marun | sc68cal: I think the huge advantage aws/google/etc have is the tight feedback loop between dev and ops. | 15:48 |
amuller | Wrong 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 it | 15:48 |
marun | amuller: manpower it is :) | 15:48 |
amuller | It's not only that but this is a moot discussion anyway | 15:49 |
*** yamamoto has quit IRC | 15:49 | |
marun | amuller: agreed | 15:49 |
*** garyk has left #openstack-neutron | 15:51 | |
sc68cal | marun: Yes, those are very good points | 15:53 |
*** yamamoto has joined #openstack-neutron | 15:53 | |
sc68cal | marun: Do you want to run for BDFL? | 15:53 |
sc68cal | ;) | 15:53 |
marun | sc68cal: I'm definitely not the man for the job. | 15:54 |
marun | sc68cal: But I would certainly appreciate some top-down vision to go with our bottom-up organization. | 15:54 |
emagana | my 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 specs | 15:55 |
marun | emagana: I think that's a solution in search of a problem. | 15:55 |
marun | emagana: 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 |
openstackgerrit | Merged openstack/neutron: Update default tox envlist to match voting gate checks https://review.openstack.org/137516 | 15:55 |
emagana | but that will convert us in a product-based program and not into an opensource team.. | 15:55 |
marun | emagana: but we're a developer-centric culture | 15:55 |
*** bandarji has joined #openstack-neutron | 15:56 | |
marun | emagana: it can't just be imposed | 15:56 |
*** banix has quit IRC | 15:56 | |
emagana | marun: I do agree! | 15:56 |
marun | emagana: right | 15:56 |
*** lazy_prince is now known as killer_prince | 15:56 | |
emagana | maybe we could find some common ground! | 15:56 |
marun | I think we need cohesive vision and effective management. I'm not sure how to achieve that when everything is by consensus. | 15:57 |
emagana | I 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 |
marun | emagana: it's definitely good to be exposed to different perspectives | 15:57 |
marun | emagana: 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 |
claudiub | garyk: i don't have permission to see the page, for some reason | 15:58 |
haleyb | there are some developers here that are also operators :) | 15:58 |
emagana | marun: but what is the point of developing something that will not be used? | 15:58 |
marun | emagana: I hear that hp may be donating it's defunct tripleo resources to infra, so that may be changing | 15:58 |
marun | emagana: such that we'll be able to run ci on our openstack cloud | 15:58 |
*** SridharG has left #openstack-neutron | 15:58 | |
emagana | marun: that will be awesome! | 15:59 |
marun | emagana: that might allow developers more exposure to ops in a way that is meaningful to them. | 15:59 |
openstackgerrit | Doug Wiegley proposed openstack/neutron: Add proposal-bot-esque job for updating child repo neutron pinning https://review.openstack.org/163062 | 15:59 |
* marun crosses his fingers | 15:59 | |
*** crose has joined #openstack-neutron | 15:59 | |
marun | dougwig: you don't waste any time :) | 15:59 |
*** markvoelker has quit IRC | 15:59 | |
emagana | will be around at the ops meetup but if someone wants to hear more just ping me! | 15:59 |
marun | emagana: have fun! | 16:00 |
*** kbringard has joined #openstack-neutron | 16:00 | |
*** markvoelker has joined #openstack-neutron | 16:00 | |
dougwig | marun: i guess i don't need to paste that to you. :) | 16:00 |
amuller | This 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 where | 16:00 |
amuller | drivers 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_m | zhhuabj: I just saw it fail once, when I tried strongswan. Will do more runs today. | 16:01 |
marun | amuller: we're a community | 16:01 |
*** vmtrooper has joined #openstack-neutron | 16:01 | |
marun | amuller: if participants want focus on things, they need to drive consensus support | 16:01 |
marun | amuller: it's not magic | 16:01 |
zhhuabj | pc_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 |
marun | amuller: All that to say that feedback isn't enough. | 16:02 |
*** banix has joined #openstack-neutron | 16:02 | |
*** fredlhsu has quit IRC | 16:02 | |
zhhuabj | pc_m, I will test this case (ubuntu 14.04 + ss driver) as well today | 16:02 |
amuller | That'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 |
marun | amuller: a cop out answer? | 16:02 |
marun | amuller: we're a community. | 16:02 |
marun | amuller: we're not a top-down engineering organization | 16:02 |
*** nlahouti has joined #openstack-neutron | 16:03 | |
*** changbl has joined #openstack-neutron | 16:03 | |
matrohon | roaet, roaet_ : ping | 16:03 |
marun | amuller: the goal is to make neutron lighter-weight to minimize the kind of arbitrary decision-making you're complaining about | 16:03 |
*** yfried_ has quit IRC | 16:03 | |
roaet | matrohon: hi | 16:03 |
roaet | roaet_: ? | 16:03 |
roaet | i have a clone! | 16:04 |
marun | amuller: but so long as something needs to happens in-tree, it has to be balanced against perceived priorities. | 16:04 |
matrohon | roaet : roaet_ seems on the room too! | 16:04 |
*** emagana has quit IRC | 16:04 | |
matrohon | roaet : I'd like to use the capacity to extend the neutron client that you recently merged | 16:04 |
*** markvoelker has quit IRC | 16:05 | |
roaet | matrohon: cool | 16:05 |
marun | amuller: how would you propose we make those decisions without relying on the relationships among the interested parties? | 16:05 |
*** saikrishna has joined #openstack-neutron | 16:05 | |
matrohon | roaet : it looks suitable to my need, thanks to you | 16:05 |
amuller | I'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 revolve | 16:05 |
amuller | around 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 IRC | 16:06 | |
marun | so you'd ignore the fact that some people have authority in this project by virtue of trust and technical competance? | 16:06 |
roaet | matrohon: well I'm glad it helps. did you need information on getting an extension working? | 16:06 |
marun | or the fact that the ptl was elected and gets to guide this decision-making process as he sees fit? | 16:06 |
matrohon | roaet : you've also proposed an example : fox_sockets | 16:06 |
matrohon | roaet : yep | 16:06 |
*** gongysh has joined #openstack-neutron | 16:06 | |
marun | process is *not* a substitute for culture | 16:06 |
marun | competance -> competence | 16:07 |
matrohon | roaet : do you already know a stand alone project that use it by adding it to its entry points? | 16:07 |
*** gongysh has quit IRC | 16:07 | |
roaet | matrohon: a stand-alone entrypoint example? one doesn't exist yet but I'm working on one right now | 16:08 |
marun | amuller: everyone is free to complain, but it would be better to work towards something better. | 16:08 |
matrohon | roaet : for the quark client? | 16:08 |
roaet | matrohon: indeed | 16:08 |
*** yamamoto has quit IRC | 16:08 | |
*** emagana_ has joined #openstack-neutron | 16:08 | |
roaet | https://github.com/roaet/quark_python_neutronclient_ext | 16:08 |
amuller | marun: 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 |
amuller | more seriously (As opposed to sitting down at a mid cycle for a few hours) *and* be more transparent. | 16:09 |
*** itzikb has quit IRC | 16:09 | |
roaet | it has been backburnered but I can make the modifications to it quickly if that'd help you out matrohon | 16:09 |
amuller | Approving and declining specs is a massively important responsibility, it shapes the project for the years to come | 16:09 |
marun | amuller: have you seen how many specs get proposed? | 16:09 |
marun | if there's a problem, it's allowing an open submission process | 16:09 |
amuller | marun: I understand that this is a massive undertaking, and still the filtering process should take more than an afternoon... | 16:10 |
matrohon | roaet : cool! yep it would surely help! | 16:10 |
amuller | It should be treated with respect | 16:10 |
marun | amuller: seriously? | 16:10 |
marun | amuller: you think that drivers spend only an afternoon? | 16:10 |
*** agentle_ has joined #openstack-neutron | 16:10 | |
marun | It can take an afternoon per-spec! | 16:10 |
amuller | marun: You know exactly what I'm specifically referring to and what I'm not, no need to twist my words | 16:11 |
amuller | I've spent hours reviewing single specs of course I know what is involved | 16:11 |
marun | amuller: The point is that arbitrary decision-making is part of the spec process | 16:11 |
marun | amuller: we can't give due process to all of them | 16:11 |
*** pal has joined #openstack-neutron | 16:11 | |
roaet | matrohon: you wouldn't also happen to need auth extensions? ;) | 16:11 |
marun | amuller: the only way out of this mess is limiting the scope of what has to have our approval in the first place | 16:12 |
*** apuimedo has quit IRC | 16:12 | |
*** SridharRamaswam1 has quit IRC | 16:12 | |
marun | i.e. focus on enabling out-of-tree development | 16:13 |
zhhuabj | pc_m, seems need ensure_process in _delete_vpn_processes, what do you think ? | 16:13 |
amuller | I 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 IRC | 16:13 | |
matrohon | roaet : 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 |
roaet | matrohon: no it was taken out by request. It's only there for unit testing and as an example. | 16:14 |
pc_m | zhhuabj: checking | 16:14 |
roaet | It's not loadable because of the _ in front of it | 16:14 |
marun | amuller: I wish I could live in the world you're describing. | 16:15 |
amuller | marun: Look, this is a scale, not a binary process | 16:15 |
amuller | You can't go through every spec on the same level | 16:16 |
amuller | Because that'd take possibly longer than a single cycle | 16:16 |
pc_m | zhhuabj: It seems like it is creating the process and then turning around and deleting it. No? | 16:16 |
matrohon | roaet : 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 IRC | 16:16 | |
pc_m | zhhuabj: 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 |
roaet | matrohon: I have a random auth example that shows how entry points work | 16:17 |
*** emagana has joined #openstack-neutron | 16:17 | |
roaet | https://github.com/rackerlabs/rackspace-auth-neutronclientext/blob/master/setup.py | 16:17 |
marun | amuller: agreed | 16:17 |
amuller | marun: 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 |
roaet | matrohon: the EP are put in setup.py I believe. | 16:17 |
marun | amuller: could things be better? yes. | 16:18 |
marun | amuller: I'm not going to argue against improving things. | 16:18 |
amuller | marun: 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 actionable | 16:18 |
marun | amuller: But the idea that things are arbitrary and don't have to be is, to me, a cop-out. | 16:18 |
marun | They *do* have to seem arbitrary at some level. | 16:18 |
zhhuabj | pc_m, check again, make sense | 16:18 |
marun | Because 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 IRC | 16:19 | |
marun | And 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 IRC | 16:19 | |
pc_m | mestery: ping | 16:20 |
ihrachyshka | marun, 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 IRC | 16:21 | |
marun | ihrachyshka: I'm not sure it's always obvious where that line lies. | 16:21 |
*** emagana has quit IRC | 16:21 | |
*** kbringard has quit IRC | 16:21 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move create_dvr_fip_interfaces in to DVR https://review.openstack.org/159663 | 16:21 |
ihrachyshka | marun, just curious whether the arbitrary part of decision making is because we limit ourselves in time when we consider new specs (?) | 16:22 |
matrohon | roaet : 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 think | 16:22 |
marun | ihrachyshka: that is likely part of it, yeah. | 16:22 |
marun | ihrachyshka: the other part is that anyone can submit a spec | 16:22 |
ihrachyshka | marun, do you want some ATC kind of limit for that? | 16:22 |
marun | ihrachyshka: 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 |
matrohon | roaet : I did enter an entry in my setup.cfg, for the neutronclient.extension namespace, but the neutron client doesn't load my extension | 16:23 |
ihrachyshka | because 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 features | 16:24 |
openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 16:24 |
mestery | pc_m: pong | 16:25 |
*** carl_baldwin has joined #openstack-neutron | 16:25 | |
pc_m | mestery: I just posted to QA IRC, but that 162887 job is still seeing strangeness. | 16:25 |
pc_m | I 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_m | it is in gate and is showing failing docs, pep8, pylint. | 16:26 |
pc_m | mestery: This is a really simple change too... I'm clueless as to what is going on. | 16:27 |
carl_baldwin | salv-orlando: ping | 16:27 |
*** sbfox has joined #openstack-neutron | 16:27 | |
pc_m | mestery: Is it merging in other changes from the queue and that is causing issues? | 16:27 |
mestery | pc_m: Me too, I couldn't rebase either when I tried | 16:27 |
mestery | Very strange | 16:27 |
*** scheuran has quit IRC | 16:28 | |
marun | ihrachyshka: 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 IRC | 16:30 | |
*** dboik_ has joined #openstack-neutron | 16:30 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit https://review.openstack.org/162840 | 16:30 |
roaet | matrohon: I'll start working on my extension soon and see if I can help you out | 16:30 |
amuller | Mentoring 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 doc | 16:30 |
amuller | + DVR doc | 16:31 |
pc_m | mestery: Any idea on how to approach this issue, or who to talk to? | 16:31 |
amuller | and take it from there... | 16:31 |
*** tiswanso has joined #openstack-neutron | 16:31 | |
matrohon | roaet : If I rename the file _fox_socket to fox_soccket, the extension should work? | 16:31 |
mestery | pc_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 |
roaet | matrohon: sadly no.. I forgot, but I was asked to remove contrib loading support | 16:31 |
pc_m | mestery: Thanks. will do... | 16:31 |
*** sripriya has joined #openstack-neutron | 16:32 | |
*** dboik has quit IRC | 16:33 | |
matrohon | roaet : ok, the neutronclient needs an update so... at least a removal of the contrib dir | 16:34 |
*** baoli has quit IRC | 16:34 | |
roaet | matrohon: we were able to leave it there to keep the example extension | 16:34 |
matrohon | roaet : 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-neutron | 16:36 | |
amuller | carl_baldwin: I left 1 comment on the internal interfaces patch | 16:36 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit https://review.openstack.org/162840 | 16:36 |
roaet | matrohon: the situation by which that patch was merged was complicated.. I think the commit message was a victim of that, heh. | 16:37 |
mestery | marun: 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 IRC | 16:38 | |
*** _cjones_ has joined #openstack-neutron | 16:39 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 16:39 |
*** yamahata has quit IRC | 16:39 | |
*** jorgem has joined #openstack-neutron | 16:40 | |
*** jaypipes has joined #openstack-neutron | 16:40 | |
matrohon | roaet : thanks for the discussion, I'll keep on digging into that, and I'll wait for the quark extensions | 16:41 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit https://review.openstack.org/162840 | 16:42 |
*** harlowja_away is now known as harlowja_ | 16:44 | |
marun | matrohon: unfortunately, yes. the experimental queue is terrible | 16:45 |
marun | oops, mestery ^^ | 16:45 |
mestery | marun: #sadpanda | 16:45 |
marun | mestery: I'd recommend adding it non-voting to the check pipeline instead | 16:46 |
*** kickinz1 is now known as kickinz1|afk | 16:46 | |
marun | mestery: that's the only way we were able to have reasonable turnaround when working on the api and functional jobs | 16:46 |
*** dimsum__ has quit IRC | 16:46 | |
marun | mestery: the biggest flaw in our current ci scheme is not being able to validate config changes by running actual jobs | 16:47 |
mestery | yeah | 16:47 |
mestery | it sucks | 16:47 |
*** dimsum__ has joined #openstack-neutron | 16:47 | |
mestery | marun: So I guess I sit and wait :( | 16:48 |
*** ihrachyshka has quit IRC | 16:48 | |
*** amotoki has quit IRC | 16:51 | |
*** amotoki has joined #openstack-neutron | 16:51 | |
*** amotoki has quit IRC | 16:51 | |
*** marun has quit IRC | 16:52 | |
amuller | marun: 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 |
amuller | only 'calls' | 16:52 |
openstackgerrit | Shiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo. https://review.openstack.org/152383 | 16:52 |
*** mlavalle has quit IRC | 16:52 | |
*** vivek-ebay has quit IRC | 16:53 | |
*** nmagnezi_ has quit IRC | 16:55 | |
*** jorgem has quit IRC | 16:57 | |
*** marun has joined #openstack-neutron | 16:57 | |
*** agentle_ has quit IRC | 16:58 | |
openstackgerrit | enikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations https://review.openstack.org/149261 | 16:59 |
*** jistr has quit IRC | 17:01 | |
carl_baldwin | amuller, ack | 17:01 |
*** leenheer has joined #openstack-neutron | 17:02 | |
*** yamahata has joined #openstack-neutron | 17:03 | |
*** banix has joined #openstack-neutron | 17:04 | |
*** vivek-ebay has joined #openstack-neutron | 17:04 | |
*** sbadia has quit IRC | 17:04 | |
*** _cjones_ has quit IRC | 17:05 | |
*** sbadia has joined #openstack-neutron | 17:05 | |
*** ivar-lazzaro has joined #openstack-neutron | 17:05 | |
*** changbl has joined #openstack-neutron | 17:06 | |
*** vivek-ebay has quit IRC | 17:06 | |
*** Longgeek has quit IRC | 17:06 | |
*** jaypipes has quit IRC | 17:07 | |
*** vivek-ebay has joined #openstack-neutron | 17:07 | |
*** britthouser has joined #openstack-neutron | 17:08 | |
*** markvoelker has joined #openstack-neutron | 17:09 | |
*** _cjones_ has joined #openstack-neutron | 17:09 | |
*** yamamoto has joined #openstack-neutron | 17:09 | |
*** pask81 has quit IRC | 17:09 | |
*** ygbo has quit IRC | 17:09 | |
*** aepifanov has joined #openstack-neutron | 17:11 | |
*** yfried_ has joined #openstack-neutron | 17:12 | |
*** britthouser has quit IRC | 17:13 | |
*** britthouser has joined #openstack-neutron | 17:13 | |
*** yamamoto has quit IRC | 17:14 | |
*** Akanksha08 has joined #openstack-neutron | 17:14 | |
*** krtaylor has quit IRC | 17:14 | |
Akanksha08 | Hi rossella_s | 17:14 |
*** kbringard has joined #openstack-neutron | 17:15 | |
*** shwetaap has quit IRC | 17:17 | |
*** melwitt has joined #openstack-neutron | 17:18 | |
*** markvoelker has quit IRC | 17:19 | |
*** achanda has joined #openstack-neutron | 17:19 | |
*** achanda has quit IRC | 17:19 | |
*** markvoelker has joined #openstack-neutron | 17:19 | |
*** achanda has joined #openstack-neutron | 17:19 | |
*** britthouser has quit IRC | 17:20 | |
*** yfried_ has quit IRC | 17:21 | |
*** yfried_ has joined #openstack-neutron | 17:21 | |
*** britthouser has joined #openstack-neutron | 17:21 | |
*** emagana has joined #openstack-neutron | 17:21 | |
*** changbl has quit IRC | 17:21 | |
*** markvoelker_ has joined #openstack-neutron | 17:21 | |
openstackgerrit | Claudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 17:21 |
openstackgerrit | Chuck Carlino proposed openstack/neutron: Prevent updating mac address of bound port https://review.openstack.org/162500 | 17:22 |
*** markvoelker has quit IRC | 17:24 | |
*** sbfox has quit IRC | 17:24 | |
dougwig | marun, mestery - is the non-voting check queue a good alternative? i dread trying to iterate in the experimental queue. | 17:24 |
marun | dougwig: I think so | 17:24 |
*** changbl has joined #openstack-neutron | 17:24 | |
marun | dougwig: it does add gate overhead, but the alternative is so much worse | 17:24 |
mestery | The experimental queue sucks | 17:25 |
mestery | My client patch is still rotting in there | 17:25 |
mestery | Almost 5 hours now | 17:25 |
mestery | dougwig: ^^^ | 17:25 |
dougwig | marun: have you chatted with infra about it? | 17:25 |
mestery | #sadpanda | 17:25 |
marun | I'm not entirely sure why the experimental queue doesn't get more resources | 17:25 |
*** devvesa has quit IRC | 17:25 | |
marun | dougwig: I had words at one point | 17:25 |
marun | dougwig: the tripleo experimental queue came back super quick and I was wondering wtf | 17:25 |
dougwig | let me see about tilting at that windmill. | 17:26 |
marun | dougwig: 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 it | 17:26 |
marun | dougwig: but it's definitely worth a revisit | 17:26 |
*** sbfox has joined #openstack-neutron | 17:26 | |
*** krtaylor has joined #openstack-neutron | 17:27 | |
mestery | dougwig: Loop me in too, I want to tilt with you | 17:27 |
dougwig | #openstack-infra, right now | 17:27 |
*** s3wong has joined #openstack-neutron | 17:28 | |
*** pal has joined #openstack-neutron | 17:29 | |
*** nyechiel_ has quit IRC | 17:30 | |
*** coolsvap|afk is now known as coolsvap | 17:32 | |
*** changbl has quit IRC | 17:33 | |
*** anish__ has joined #openstack-neutron | 17:34 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 17:35 |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 17:35 |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 17:35 |
openstackgerrit | Pavel Bondar proposed openstack/neutron: IPAM reference driver https://review.openstack.org/150485 | 17:35 |
*** achanda has quit IRC | 17:35 | |
*** roeyc has quit IRC | 17:35 | |
*** viveknarasimhan has joined #openstack-neutron | 17:36 | |
*** _cjones_ has quit IRC | 17:36 | |
*** achanda has joined #openstack-neutron | 17:37 | |
*** _cjones_ has joined #openstack-neutron | 17:37 | |
*** jp_at_hp has quit IRC | 17:38 | |
*** evgenyf has quit IRC | 17:38 | |
dougwig | mestery, marun - that was too easy. i feel robbed of "the openstack experience". | 17:40 |
marun | heh | 17:40 |
*** SridharRamaswamy has joined #openstack-neutron | 17:40 | |
mestery | dougwig: I'll save some of the openstack experience for you in person in vancouver | 17:41 |
mestery | ;) | 17:41 |
*** changbl has joined #openstack-neutron | 17:41 | |
mestery | dougwig: Also, it hasn't merged yet, so don't declare victory yet | 17:41 |
*** matrohon has quit IRC | 17:41 | |
dougwig | oh, true, true. | 17:41 |
openstackgerrit | Merged openstack/neutron: Imported Translations from Transifex https://review.openstack.org/162540 | 17:42 |
*** shivharis has joined #openstack-neutron | 17:42 | |
*** yfried_ is now known as yfried|afk | 17:42 | |
*** SridharRamaswamy has quit IRC | 17:43 | |
*** mrsmith has joined #openstack-neutron | 17:45 | |
*** kawa2014 has quit IRC | 17:45 | |
*** yfried|afk is now known as yfried_ | 17:46 | |
*** vmtrooper has joined #openstack-neutron | 17:50 | |
*** pal has quit IRC | 17:50 | |
*** Swami has joined #openstack-neutron | 17:50 | |
*** ChuckC has joined #openstack-neutron | 17:51 | |
*** jobewan has joined #openstack-neutron | 17:54 | |
*** vmtrooper has quit IRC | 17:54 | |
*** ajmiller has joined #openstack-neutron | 17:55 | |
*** tiswanso has quit IRC | 17:55 | |
*** tiswanso has joined #openstack-neutron | 17:56 | |
*** cbouch has quit IRC | 17:56 | |
*** Rajeev has joined #openstack-neutron | 17:56 | |
*** yfried_ is now known as yfried|afk | 17:56 | |
*** ronis has joined #openstack-neutron | 17:57 | |
*** shwetaap has joined #openstack-neutron | 17:57 | |
*** melwitt_ has joined #openstack-neutron | 17:58 | |
*** britthouser has quit IRC | 17:58 | |
*** melwitt has quit IRC | 17:58 | |
*** ihrachyshka has joined #openstack-neutron | 18:00 | |
*** jorgem has joined #openstack-neutron | 18:00 | |
*** jschwarz has joined #openstack-neutron | 18:01 | |
*** saikrishna has quit IRC | 18:02 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Add rootwrap daemon mode support https://review.openstack.org/158805 | 18:03 |
*** anish__ has quit IRC | 18:04 | |
*** jaypipes has joined #openstack-neutron | 18:04 | |
*** tsg_ has quit IRC | 18:04 | |
*** baoli_ has quit IRC | 18:05 | |
*** changbl has quit IRC | 18:05 | |
mestery | dougwig: Over 5 hours in the experimental queue waiting for my python-neutronclient job to get a node to run on | 18:05 |
mestery | dougwig: *sigh* | 18:05 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 18:05 |
*** baoli has joined #openstack-neutron | 18:07 | |
*** tsufiev_ has quit IRC | 18:07 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move internal port processing to router classes https://review.openstack.org/154513 | 18:08 |
openstackgerrit | rcurran proposed openstack/neutron: ML2 cisco_nexus MD: upstream updates required https://review.openstack.org/163040 | 18:09 |
marun | Rajeev: Do you honestly think that what you were adding qualifies as a 'unit test'? | 18:11 |
*** Flcn has quit IRC | 18:11 | |
*** tflynn has joined #openstack-neutron | 18:12 | |
Rajeev | marun: yes, within the context of the unit tests that are there. | 18:12 |
marun | Rajeev: That wasn't my question. | 18:12 |
*** sbfox has quit IRC | 18:12 | |
marun | Rajeev: Do you consider what you were adding a unit test, period. | 18:12 |
Rajeev | marun: yes | 18:12 |
marun | Rajeev: Not considering the context of Neutron's supposed 'unit test' suite. | 18:12 |
marun | Rajeev: Then I respectfully submit that you don't know what a unit test actually is. | 18:13 |
*** jpena has quit IRC | 18:13 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 18:13 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 18:13 |
Rajeev | marun: I am not sure why your perspective is so different from rest of the reviewers | 18:14 |
marun | Rajeev: Because I actually care about testing. | 18:14 |
marun | Rajeev: I don't write features, but I am trying to encourage a more effective approach to testing. | 18:15 |
amuller | Rajeev: 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 |
amuller | So the idea is to up our game | 18:15 |
amuller | And be more effective in testing, which for most of the agent-side testing should be functional testing | 18:15 |
*** tsufiev_ has joined #openstack-neutron | 18:16 | |
*** tflynn has quit IRC | 18:16 | |
marun | Rajeev: I entirely agree, though, that there needs to be better documentation. | 18:16 |
*** tflynn has joined #openstack-neutron | 18:16 | |
marun | Rajeev: 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 | |
marun | Rajeev: It is my intention to rectify this, probably post k3 when I'll have more time. | 18:17 |
Rajeev | marun: 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 |
marun | Rajeev: I disagree | 18:17 |
marun | Rajeev: If I can make things better by individual review, I'm going to do it. | 18:18 |
*** SridharRamaswamy has joined #openstack-neutron | 18:18 | |
marun | Rajeev: 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-neutron | 18:18 | |
*** tflynn_ has quit IRC | 18:19 | |
Rajeev | marun: I think it is not that difficult to work with the community. | 18:19 |
marun | Rajeev: Who says I'm not working with the community? | 18:19 |
marun | Rajeev: I mean, my individual effort doesn't scale. | 18:19 |
*** sbfox has joined #openstack-neutron | 18:20 | |
marun | Rajeev: 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 |
marun | Rajeev: 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 IRC | 18:21 | |
Rajeev | marun: I am trying to be supportive here, and that is why got help to get the dependent patch submitted so quickly. | 18:22 |
openstackgerrit | Baodong (Robert) Li proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 18:22 |
marun | Rajeev: So what are we arguing about? | 18:23 |
marun | Rajeev: What about my comments don't you agree with? | 18:23 |
*** karimb has quit IRC | 18:24 | |
Rajeev | marun: 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 in | 18:24 |
Rajeev | as follow ups | 18:25 |
*** yfried_ is now known as yfried|afk | 18:26 | |
*** dimsum__ is now known as dims | 18:27 | |
marun | Rajeev: Respectfully, why are you telling me how to do my job? | 18:27 |
marun | Rajeev: The functional change needs to be included in the originating patch, period. | 18:27 |
marun | Rajeev: There's no need to take this personally. | 18:27 |
marun | Rajeev: a -2 is 'this should not merge' | 18:28 |
marun | Rajeev: 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 |
openstackgerrit | Dane LeBlanc proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers https://review.openstack.org/149068 | 18:28 |
Rajeev | marun: 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 |
marun | Rajeev: You can fight me on this, but that won't help you get your code merged. | 18:28 |
amuller | Rajeev: Is there a reason not to squash the two patches in to one? | 18:29 |
amuller | I think that Maru is operating on the assumption that there isn't | 18:29 |
Rajeev | marun: Yes, these have different authorship | 18:29 |
marun | Rajeev: That is not a good reason | 18:29 |
marun | Rajeev: shared authorship attribution is always an option | 18:29 |
marun | Rajeev: You're even using it already! | 18:30 |
Rajeev | marun: yes, exactly because of the second patch | 18:31 |
marun | Rajeev: There is nothing to stop you from adding the second change to the first and having that patch have shared authorship. | 18:31 |
Rajeev | marun: I am trying to get functioal tests in to your endeavor, I am kind of not following why you are blocking it so much | 18:32 |
marun | As to what the expectation for testing, if you're merging functionalityk, is it tested? | 18:32 |
*** Swami has quit IRC | 18:32 | |
marun | That's a question that both reviewer and author need to ask. | 18:32 |
marun | Rajeev: If you want to do as I ask, your code will merge. | 18:33 |
marun | Rajeev: I'm not sure what you're hoping to accomplish by arguing so strongly for doing things your way. | 18:33 |
marun | Rajeev: 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-neutron | 18:35 | |
Rajeev | marun: 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 |
marun | Rajeev: What are you hearing from other contributors that contradicts me? | 18:35 |
marun | me -> my comments | 18:35 |
marun | Rajeev: that unit tests are required? | 18:36 |
openstackgerrit | Merged openstack/neutron-vpnaas: Remove the reference for non-existent cisco.l3.plugging_drivers https://review.openstack.org/162887 | 18:36 |
Rajeev | marun: the notion about unit tests vs, functional tests and what is a reasonable requirement as understood by people contributing code | 18:37 |
amuller | Rajeev: You are right in the sense that there have no written guidelines | 18:37 |
amuller | So far it's been communicated verbally | 18:37 |
*** ajmiller has quit IRC | 18:37 | |
amuller | And Maru is working on such a document | 18:37 |
amuller | In the mean time, code that touches the agent is expected to be tested effectively, meaning that parts of it should be tested with functional tests | 18:38 |
*** _cjones_ has quit IRC | 18:38 | |
amuller | Just like the current DVR functional tests | 18:38 |
Rajeev | marun: amuller That is very encouraging to hear | 18:38 |
jschwarz | marun, this reminds me that I ought to add some TESTING.rst documentation to the fullstack patch... will probably do so on my next patchset | 18:39 |
jschwarz | (after yours goes through) | 18:39 |
*** shivharis has quit IRC | 18:41 | |
*** jschwarz has quit IRC | 18:43 | |
*** changbl has joined #openstack-neutron | 18:44 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Enable StrongSwan in gate hook https://review.openstack.org/161714 | 18:45 |
*** yfried|afk is now known as yfried_ | 18:46 | |
*** zzzeek has quit IRC | 18:48 | |
*** baoli has quit IRC | 18:48 | |
*** agentle_ has joined #openstack-neutron | 18:50 | |
*** zzzeek has joined #openstack-neutron | 18:53 | |
*** baoli has joined #openstack-neutron | 18:53 | |
*** baoli has quit IRC | 18:54 | |
*** _cjones_ has joined #openstack-neutron | 18:54 | |
*** jorgem has quit IRC | 18:55 | |
*** agentle_ has quit IRC | 18:55 | |
*** Rajeev has quit IRC | 18:55 | |
*** agentle_ has joined #openstack-neutron | 18:55 | |
*** claudiub has quit IRC | 18:56 | |
*** claudiub has joined #openstack-neutron | 18:57 | |
*** SridharRamaswamy has quit IRC | 18:58 | |
*** jorgem has joined #openstack-neutron | 18:59 | |
*** markvoelker_ has quit IRC | 19:01 | |
*** prometheanfire is now known as blueness-evil-tw | 19:01 | |
*** emagana has quit IRC | 19:01 | |
*** kbringard has quit IRC | 19:01 | |
*** melwitt_ has quit IRC | 19:02 | |
*** emagana has joined #openstack-neutron | 19:02 | |
*** melwitt has joined #openstack-neutron | 19:02 | |
*** ihrachyshka has quit IRC | 19:02 | |
*** blueness-evil-tw is now known as prometheanfire | 19:02 | |
*** gongysh_ has quit IRC | 19:02 | |
*** Marga_ has quit IRC | 19:03 | |
*** claudiub has quit IRC | 19:04 | |
*** dboik_ has quit IRC | 19:04 | |
*** baoli has joined #openstack-neutron | 19:04 | |
*** dboik has joined #openstack-neutron | 19:04 | |
*** SridharRamaswamy has joined #openstack-neutron | 19:06 | |
*** emagana has quit IRC | 19:06 | |
*** viveknarasimhan has quit IRC | 19:06 | |
*** ajmiller_ is now known as ajmiller | 19:07 | |
*** dane_leblanc has quit IRC | 19:08 | |
*** bobmel has quit IRC | 19:08 | |
*** jaypipes is now known as jay-afk | 19:09 | |
*** jay-afk is now known as jaypipes-afk | 19:10 | |
*** sbfox has quit IRC | 19:10 | |
*** yamamoto has joined #openstack-neutron | 19:11 | |
*** jorgem has quit IRC | 19:14 | |
*** yamamoto has quit IRC | 19:15 | |
openstackgerrit | Baodong (Robert) Li proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 19:16 |
*** jorgem has joined #openstack-neutron | 19:17 | |
*** sbfox has joined #openstack-neutron | 19:18 | |
*** coolsvap is now known as coolsvap|afk | 19:20 | |
*** gongysh_ has joined #openstack-neutron | 19:21 | |
*** carl_baldwin has quit IRC | 19:23 | |
*** mriedem has joined #openstack-neutron | 19:24 | |
mriedem | i'm looking at some wonky code in the neutronv2 api and had a question, | 19:24 |
mriedem | is it possible to have more than one port associated with an instance in the same network? | 19:24 |
mriedem | i assume so | 19:24 |
mriedem | and each port would have a separate fixed_ip | 19:25 |
*** sripriya has quit IRC | 19:26 | |
*** sripriya has joined #openstack-neutron | 19:27 | |
russellb | mriedem: good question ... IIRC, there has been a lot of debate about whether we would allow that or not. | 19:27 |
russellb | huge debate to required code ratio, heh | 19:27 |
mriedem | russellb: well, the multinic api was clearly written for nova-network | 19:27 |
*** s3wong has quit IRC | 19:28 | |
mriedem | looking at the neutronv2 api code here: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L752 | 19:28 |
mriedem | notice how it loops over ports but will exit before getting to the 2nd one | 19:28 |
mriedem | that 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 |
mriedem | same for remove_fixed_ip_from_instance | 19:29 |
russellb | mriedem: this is what i was thinking about https://review.openstack.org/#/c/97716/ | 19:30 |
*** s3wong has joined #openstack-neutron | 19:30 | |
russellb | mriedem: supposedly implemented (meaning yes, it should be supported) | 19:30 |
*** emagana has joined #openstack-neutron | 19:31 | |
*** britthouser has joined #openstack-neutron | 19:32 | |
*** Flcn has joined #openstack-neutron | 19:32 | |
*** kbringard has joined #openstack-neutron | 19:32 | |
*** kbringard has quit IRC | 19:32 | |
*** carl_baldwin has joined #openstack-neutron | 19:34 | |
mriedem | hmm | 19:35 |
mriedem | pretty sure tempest tests never landed for that as promised in the spec | 19:35 |
*** nyechiel_ has joined #openstack-neutron | 19:36 | |
*** carl_baldwin has quit IRC | 19:36 | |
*** vishy has joined #openstack-neutron | 19:38 | |
*** vmtrooper has joined #openstack-neutron | 19:39 | |
*** carl_baldwin has joined #openstack-neutron | 19:40 | |
*** vmtrooper has quit IRC | 19:44 | |
*** claudiub has joined #openstack-neutron | 19:45 | |
*** nlahouti has quit IRC | 19:47 | |
openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 19:48 |
*** safchain has quit IRC | 19:49 | |
*** dims has quit IRC | 19:51 | |
*** sripriya has quit IRC | 19:51 | |
*** sripriya has joined #openstack-neutron | 19:51 | |
*** carl_baldwin has quit IRC | 19:53 | |
mriedem | russellb: 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 network | 19:54 |
mriedem | any fixed_ips on any other subnets wouldn't get on the port, | 19:54 |
mriedem | nor any subnets/fixed_ips for any other ports on the network for that instance | 19:54 |
mriedem | so that code seems pretty busted | 19:54 |
russellb | not surprising | 19:54 |
mriedem | sadly no | 19:54 |
mriedem | and since there was no tempest testing added for the blueprint, we don't have any cases to build from | 19:55 |
mriedem | in tempest i mean | 19:55 |
mriedem | barf | 19:55 |
mriedem | maybe 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 tempest | 19:55 |
*** mlavalle has joined #openstack-neutron | 19:56 | |
*** jpena has joined #openstack-neutron | 19:56 | |
*** aepifanov has quit IRC | 19:57 | |
*** aix has quit IRC | 19:57 | |
amuller | marun: 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 |
amuller | I added a while True: eventlet.sleep(0) | 19:57 |
amuller | and then the other end receives the message, it's pretty funny | 19:57 |
amuller | marun: I guess I'll add a utility that waits until the exchange queues are empty before proceeding | 19:58 |
*** sripriya has quit IRC | 20:00 | |
*** sripriya has joined #openstack-neutron | 20:00 | |
*** dims has joined #openstack-neutron | 20:01 | |
*** SridharRamaswamy has quit IRC | 20:01 | |
*** cference has quit IRC | 20:02 | |
*** vivek-ebay has quit IRC | 20:03 | |
*** vivek-ebay has joined #openstack-neutron | 20:03 | |
*** tflynn has joined #openstack-neutron | 20:04 | |
haleyb | sc68cal: i have some comments on your devstack patch, sorry for the late review | 20:05 |
*** shivharis has joined #openstack-neutron | 20:05 | |
*** thedodd has quit IRC | 20:06 | |
*** SumitNaiksatam has joined #openstack-neutron | 20:07 | |
*** s3wong has quit IRC | 20:09 | |
claudiub | mtreinish: hello | 20:09 |
*** matrohon has joined #openstack-neutron | 20:11 | |
*** VijayB has joined #openstack-neutron | 20:12 | |
*** thedodd has joined #openstack-neutron | 20:12 | |
*** s3wong has joined #openstack-neutron | 20:13 | |
*** britthouser has quit IRC | 20:14 | |
*** carl_baldwin has joined #openstack-neutron | 20:14 | |
*** britthouser has joined #openstack-neutron | 20:15 | |
*** banix has quit IRC | 20:18 | |
carl_baldwin | amuller: I addressed your comment on https://review.openstack.org/#/c/154513/ | 20:20 |
*** banix has joined #openstack-neutron | 20:20 | |
amuller | carl_baldwin: +1 | 20:20 |
amuller | the next patch is more complicated it'll take me a bit | 20:20 |
carl_baldwin | amuller: No worries. Thank you. | 20:21 |
carl_baldwin | amuller: 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|afk | 20:22 | |
*** thedodd has quit IRC | 20:23 | |
*** sbfox has quit IRC | 20:24 | |
*** jpena has quit IRC | 20:25 | |
*** jorgem has quit IRC | 20:25 | |
*** gildub has joined #openstack-neutron | 20:28 | |
pc_m | dougwig: ping | 20:29 |
*** yfried|afk is now known as yfried_ | 20:29 | |
dougwig | pc_m: ack | 20:30 |
pc_m | dougwig: 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 IRC | 20:30 | |
pc_m | Wondering what all needs to be done, to reference that import in neutron-vpnaas. Do you know (or anyone lurking)? | 20:31 |
dougwig | oh, that's gross. hmm. | 20:31 |
dougwig | choices are (thinking out loud): 1) refactor out the import, 2) wrap the import in a try, 3) add that project to vpnaas's requirements list | 20:32 |
dougwig | maybe a shim + requirements file in place, similar to the neutron decomp | 20:32 |
pc_m | In 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 IRC | 20:34 | |
*** Rajeev has joined #openstack-neutron | 20:34 | |
openstackgerrit | Michael Smith proposed openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 20:34 |
pc_m | Not sure the import could be refactored out. Wrapping would have to then locally define the value anyway. | 20:34 |
*** sbfox has joined #openstack-neutron | 20:35 | |
*** radez is now known as radez_g0n3 | 20:35 | |
pc_m | dougwig: 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-neutron | 20:35 | |
*** vivek-eb_ has joined #openstack-neutron | 20:36 | |
*** jorgem has joined #openstack-neutron | 20:37 | |
pc_m | dougwig: I see Brocade code that has a networking_vyatta import, and I don't see that in requirements.txt. Hmm | 20:38 |
pc_m | I mean networking_brocade | 20:39 |
*** vivek-ebay has quit IRC | 20:39 | |
*** britthouser has quit IRC | 20:40 | |
*** ijw has joined #openstack-neutron | 20:41 | |
openstackgerrit | enikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations https://review.openstack.org/149261 | 20:41 |
*** emagana has quit IRC | 20:41 | |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Raise QuotaResourceUknown in the quota engine https://review.openstack.org/163171 | 20:41 |
pc_m | dougwig: I don't see any reference to networking_brocade anywhere. Wondering how that import is ensured to be on the system. | 20:41 |
dougwig | it should have a requirements file in one of neutron's subduers. | 20:42 |
dougwig | subdirs | 20:42 |
*** boris-42 has quit IRC | 20:42 | |
Rajeev | marun: 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_m | dougwig: don't see it... | 20:44 |
pc_m | HenryG: ping | 20:44 |
*** moshele has joined #openstack-neutron | 20:44 | |
*** baoli has quit IRC | 20:45 | |
*** dboik_ has joined #openstack-neutron | 20:45 | |
*** Rajeev has quit IRC | 20:46 | |
*** melwitt has quit IRC | 20:46 | |
*** melwitt has joined #openstack-neutron | 20:47 | |
*** dboik has quit IRC | 20:49 | |
*** cference has joined #openstack-neutron | 20:49 | |
*** ronis has quit IRC | 20:50 | |
*** mriedem has left #openstack-neutron | 20:51 | |
ChuckC | carl_baldwin: haleyb: I've responded to your comments on https://review.openstack.org/#/c/162500/ | 20:51 |
*** PrincessRiikka has quit IRC | 20:51 | |
carl_baldwin | ChuckC: ack | 20:52 |
*** itzikb has joined #openstack-neutron | 20: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 |
itzikb | armax: ping | 20:53 |
openstackgerrit | Mathieu Rohon proposed openstack/neutron: ML2 : make port status change only when the port is bound to the host that asking https://review.openstack.org/163178 | 20:54 |
*** rpothier has joined #openstack-neutron | 20:55 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 20:56 |
*** fredlhsu has joined #openstack-neutron | 20:56 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 20:56 |
openstackgerrit | Merged openstack/neutron: Don't notify dead DHCP agent of removed networks https://review.openstack.org/162559 | 20:57 |
*** banix has quit IRC | 20:57 | |
*** SridharRamaswamy has joined #openstack-neutron | 20:58 | |
*** shivharis has quit IRC | 20:58 | |
*** changbl has quit IRC | 20:58 | |
*** shivharis has joined #openstack-neutron | 20:58 | |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method https://review.openstack.org/154103 | 20:59 |
*** ajo has quit IRC | 20:59 | |
*** nlahouti has joined #openstack-neutron | 20:59 | |
*** raildo is now known as raildo_away | 21:04 | |
*** VijayB has quit IRC | 21:05 | |
*** cference has quit IRC | 21:07 | |
*** dboik_ has quit IRC | 21:07 | |
*** dboik has joined #openstack-neutron | 21:08 | |
*** vishwana_ has joined #openstack-neutron | 21:08 | |
*** ihrachyshka has joined #openstack-neutron | 21:09 | |
openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 21:09 |
*** vishwanathj has quit IRC | 21:10 | |
*** VijayB has joined #openstack-neutron | 21:10 | |
*** pritesh has joined #openstack-neutron | 21:10 | |
*** vishwana_ has quit IRC | 21:10 | |
*** moshele has quit IRC | 21:13 | |
*** vishwanathj has joined #openstack-neutron | 21:14 | |
*** emagana has joined #openstack-neutron | 21:14 | |
*** s3wong has quit IRC | 21:18 | |
*** s3wong has joined #openstack-neutron | 21:18 | |
kevinbenton | ihrachyshka: hey | 21:18 |
ihrachyshka | kevinbenton, hey! | 21:18 |
kevinbenton | ihrachyshka: is the conversion away from neutron.openstack.common.log a simple import replacement | 21:19 |
kevinbenton | ? | 21:19 |
kevinbenton | ihrachyshka: or is there a new way to invoke the logger as well? | 21:19 |
ihrachyshka | kevinbenton, should be, in most cases. | 21:19 |
kevinbenton | ihrachyshka: ok, thanks | 21:19 |
ihrachyshka | kevinbenton, also see https://review.openstack.org/#/c/147312/ | 21:20 |
*** thedodd has joined #openstack-neutron | 21:20 | |
*** carl_baldwin has quit IRC | 21:20 | |
ihrachyshka | kevinbenton, but it's more of safety checks, in simple cases it's just a matter of import switches. | 21:20 |
*** leenheer has quit IRC | 21:20 | |
dims | ihrachyshka: 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-log | 21:21 |
ihrachyshka | dims, right, that's the patch I referred to | 21:21 |
kevinbenton | ihrachyshka: why not have openstack.common.log pass through directly the oslo.log then? | 21:22 |
dims | :) | 21:22 |
kevinbenton | ihrachyshka: just to make it more explicit? | 21:22 |
dims | kevinbenton: that comes from oslo-incubator and we are trying to get rid of the code in oslo-incubator | 21:22 |
*** boris-42 has joined #openstack-neutron | 21:22 | |
*** carl_baldwin has joined #openstack-neutron | 21:22 | |
*** VijayB has quit IRC | 21:22 | |
dims | end goal is to get rid of */openstack/common/* | 21:22 |
kevinbenton | dims: oh, all of oslo incubator is going away? | 21:23 |
kevinbenton | oh okay | 21:23 |
ihrachyshka | kevinbenton, well, because it works the same in *most* but not all of them (for some things, imports are switched, arguments are changed...) | 21:23 |
ihrachyshka | kevinbenton, well, that's the Holy Grail of graduation - getting rid of all incubator modules | 21:23 |
dims | when we release a library say oslo.log, log.py in oslo-incubator is removed | 21:23 |
ihrachyshka | kevinbenton, I think we may see it in L or M for neutron | 21:23 |
openstackgerrit | Doug Wiegley proposed openstack/neutron: Add proposal-bot-esque job for updating child repo neutron pinning https://review.openstack.org/163062 | 21:24 |
kevinbenton | ihrachyshka: ok | 21:24 |
*** sripriya has quit IRC | 21:25 | |
ihrachyshka | kevinbenton, in kilo, we already killed ~18 modules, 13 more left though | 21:26 |
*** sripriya has joined #openstack-neutron | 21:26 | |
*** ijw has quit IRC | 21:26 | |
ihrachyshka | (well, it will be 11 once we consume oslo.log) | 21:27 |
*** vmtrooper has joined #openstack-neutron | 21:28 | |
*** SridharRamaswam1 has joined #openstack-neutron | 21:29 | |
*** SridharRamaswamy has quit IRC | 21:30 | |
*** pc_m has quit IRC | 21:31 | |
*** yfried_ is now known as yfried|afk | 21:31 | |
*** vmtrooper has quit IRC | 21:33 | |
*** yfauser has joined #openstack-neutron | 21:33 | |
*** yfauser has left #openstack-neutron | 21:34 | |
*** shwetaap has quit IRC | 21:34 | |
*** jorgem has quit IRC | 21:34 | |
*** Zemeio has quit IRC | 21:35 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron: Add proposal-bot-esque job for updating child repo neutron pinning https://review.openstack.org/163062 | 21:35 |
*** VijayB has joined #openstack-neutron | 21:36 | |
kevinbenton | ihrachyshka: do you have the patch to neutron up to transition to oslo.log? | 21:39 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Add RPC test for DVR https://review.openstack.org/163201 | 21:39 |
*** tsg has joined #openstack-neutron | 21:39 | |
ihrachyshka | kevinbenton, neutron *up*? | 21:39 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Add RPC test for metadata https://review.openstack.org/163204 | 21:40 |
ihrachyshka | you mean migration patch for neutron repo? | 21:40 |
kevinbenton | ihrachyshka: sorry, up on gerrit. i found it though | 21:40 |
ihrachyshka | sure, https://review.openstack.org/159638 | 21:40 |
*** chlong has joined #openstack-neutron | 21:40 | |
kevinbenton | ihrachyshka: so can we merge the requirements for oslo.log first so third party libs can start to assume it will be available? | 21:41 |
ihrachyshka | kevinbenton, actually, third parties should *not* rely on neutron requirements | 21:42 |
ihrachyshka | kevinbenton, I mentioned it at https://wiki.openstack.org/wiki/Neutron/VendorSplitPackaging | 21:42 |
ihrachyshka | specifically, https://wiki.openstack.org/wiki/Neutron/VendorSplitPackaging#Dependencies | 21:43 |
*** crose has quit IRC | 21:43 | |
kevinbenton | ihrachyshka: why? i don't see any justification in there | 21:44 |
ihrachyshka | kevinbenton, 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 way | 21: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_baldwin | amuller: ping | 21:45 |
amuller | carl_baldwin: yo | 21:45 |
carl_baldwin | amuller: I have a vague memory of talking about DaemonMonitor with you. Were you thinking of moving it in to router_info? | 21:45 |
kevinbenton | ihrachyshka: frankly it's a tradeoff though | 21:45 |
*** matrohon has quit IRC | 21:45 | |
kevinbenton | ihrachyshka: i don't expect neutron to drop logging any time soone | 21:46 |
kevinbenton | ihrachyshka: 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 neutron | 21:46 |
amuller | carl_baldwin: I remember talking about it, I don't remember why anymore though | 21:47 |
*** jckasper has quit IRC | 21:47 | |
carl_baldwin | amuller: 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/160608 | 21:47 |
*** tiswanso has quit IRC | 21:47 | |
ihrachyshka | kevinbenton, do we set automatic requirements updates for vendor libs? | 21:47 |
carl_baldwin | amuller: If you don’t anything beyond that patch then I’ll just proceed. | 21:47 |
kevinbenton | ihrachyshka: i don't know what you mean by that | 21:48 |
*** chandankumar has quit IRC | 21:48 | |
amuller | carl_baldwin: Oh I definitely did not work on the restructuring stuff beyond what you see in that patch. | 21:49 |
amuller | carl_baldwin: However, check out this patch by Mike: https://review.openstack.org/#/c/153162/8 | 21:49 |
ihrachyshka | kevinbenton, 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 requirements | 21:49 |
amuller | carl_baldwin: and also a patch of mine here: https://review.openstack.org/#/c/125384/31/neutron/agent/l3/ha.py | 21:49 |
amuller | we both just pass the process_monitor from the agent to the router where needed | 21:49 |
amuller | mostly because changing the RouterInfo.__init__ signature is a huge pain in the ass now | 21:50 |
kevinbenton | ihrachyshka: 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_baldwin | amuller: ack | 21:50 |
kevinbenton | ihrachyshka: if they are just a copy of what is in neutron? | 21:50 |
openstackgerrit | Michael Smith proposed openstack/neutron: Fix non-existent self.local_subnets in DvrRouter class https://review.openstack.org/157621 | 21:50 |
kevinbenton | ihrachyshka: i suppose the requirements bump would at least fail a CI check | 21:51 |
kevinbenton | ihrachyshka: but is that all we are gaining? | 21:51 |
*** mwagner_lap has quit IRC | 21:51 | |
*** yfried|afk is now known as yfried_ | 21:52 | |
carl_baldwin | amuller: I wonder if we should just do it now (change __init__) anyway. It is either now or much later. | 21:52 |
amuller | carl_baldwin: If you're up for it | 21:52 |
amuller | the fwaas repo is still trying to recover from the last time we changed that signature | 21:53 |
ihrachyshka | kevinbenton, well ok, maybe shortcut assuming neutron fetches the proper lib is justified in this particular case. | 21:53 |
kevinbenton | ihrachyshka: i can see it's a tradeoff | 21:54 |
kevinbenton | ihrachyshka: assuming the presence of libraries can be dangeroues | 21:54 |
kevinbenton | ihrachyshka: but when something is as tightly integrated as a driver, requirements are far from the only thing that will break things | 21:55 |
ihrachyshka | kevinbenton, 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 |
kevinbenton | ihrachyshka: change in one spot vs change in however many vendors | 21:55 |
kevinbenton | O(1) vs O(n) :) | 21:56 |
ihrachyshka | kevinbenton, ok, I'll split the patch now | 21:56 |
ihrachyshka | if it helps you slightly, it's worth respin | 21:56 |
amuller | marun: Pushed https://review.openstack.org/#/c/163201/1 which adds DVR RPC tests | 21:57 |
amuller | armax: ^ | 21:57 |
kevinbenton | ihrachyshka: thanks. let me know when it's up and I'll +2 | 21:57 |
amuller | Also adds a pretty ugly way to make sure that cast messages are delivered | 21:57 |
*** vishwana_ has joined #openstack-neutron | 21:57 | |
*** jaypipes-afk is now known as JAYPIPES | 21:59 | |
*** JAYPIPES is now known as jaypipes | 21:59 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Migrate to oslo.log https://review.openstack.org/159638 | 22:00 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Added oslo.log dependency https://review.openstack.org/163207 | 22:00 |
ihrachyshka | kevinbenton, ^^ | 22:00 |
*** vishwanathj has quit IRC | 22:00 | |
*** yfried_ is now known as yfried|afk | 22:02 | |
*** coolsvap|afk has quit IRC | 22:03 | |
*** tsg has quit IRC | 22:03 | |
kevinbenton | ihrachyshka: thanks! | 22:04 |
openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 22:05 |
*** chlong has quit IRC | 22:07 | |
*** agentle_ has quit IRC | 22:07 | |
*** yfauser1 has joined #openstack-neutron | 22:08 | |
openstackgerrit | Michael Smith proposed openstack/neutron: WIP delay fip namespace cleanup/destroy https://review.openstack.org/151758 | 22:08 |
*** rpothier has quit IRC | 22:10 | |
*** yfauser1 has left #openstack-neutron | 22:10 | |
*** marrusl has quit IRC | 22:10 | |
carl_baldwin | amuller: 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_baldwin | Is that too hacky? | 22:11 |
openstackgerrit | Merged openstack/neutron: policy: don't hack around oslo.config path search algorithm https://review.openstack.org/159862 | 22:12 |
amuller | carl_baldwin: yes =D | 22:12 |
openstackgerrit | Merged openstack/neutron: tests: initialize policy in BaseTestCase https://review.openstack.org/159863 | 22:12 |
carl_baldwin | amuller: You started it. :P | 22:12 |
amuller | :) | 22:12 |
amuller | I expect more from you than from me, it's human nature | 22:13 |
carl_baldwin | Still thinking. Sometimes, when you’re stuck between a rock and a hard place... | 22:14 |
*** tsufiev_ has quit IRC | 22:14 | |
*** sbfox has quit IRC | 22:16 | |
openstackgerrit | Abhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 22:19 |
*** absubram_ has joined #openstack-neutron | 22:19 | |
openstackgerrit | Michael Smith proposed openstack/neutron: Fix non-existent self.local_subnets in DvrRouter class https://review.openstack.org/157621 | 22:21 |
*** absubram has quit IRC | 22:21 | |
*** absubram_ is now known as absubram | 22:21 | |
*** txomon|fon has quit IRC | 22:22 | |
*** marrusl has joined #openstack-neutron | 22:23 | |
*** jorgem has joined #openstack-neutron | 22:23 | |
*** agentle_ has joined #openstack-neutron | 22:25 | |
*** nati_ueno has joined #openstack-neutron | 22:29 | |
*** claudiub has quit IRC | 22:30 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Introduce RPC tests https://review.openstack.org/162811 | 22:31 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Add RPC test for DVR https://review.openstack.org/163201 | 22:31 |
*** Rajeev has joined #openstack-neutron | 22:31 | |
*** thedodd has quit IRC | 22:34 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Add RPC test for metadata https://review.openstack.org/163204 | 22:34 |
*** txomon|fon has joined #openstack-neutron | 22:35 | |
*** jorgem has quit IRC | 22:35 | |
*** tsufiev_ has joined #openstack-neutron | 22:36 | |
*** rushil has quit IRC | 22:36 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed https://review.openstack.org/162260 | 22:36 |
*** shivharis has quit IRC | 22:39 | |
*** shivharis has joined #openstack-neutron | 22:39 | |
*** amuller has quit IRC | 22:40 | |
*** fredlhsu has quit IRC | 22:41 | |
openstackgerrit | Rajeev Grover proposed openstack/neutron: Prevent race conditions on DVR FIP resources https://review.openstack.org/153422 | 22:42 |
*** agentle_ has quit IRC | 22:47 | |
*** sbfox has joined #openstack-neutron | 22:48 | |
*** shivharis has quit IRC | 22:51 | |
*** shivharis has joined #openstack-neutron | 22:51 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 22:52 |
*** dboik_ has joined #openstack-neutron | 22:52 | |
*** VijayB has quit IRC | 22:53 | |
*** Flcn has quit IRC | 22:53 | |
*** mlavalle has quit IRC | 22:53 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 22:53 |
*** fredlhsu has joined #openstack-neutron | 22:55 | |
*** dboik has quit IRC | 22:55 | |
*** Marga_ has quit IRC | 22:57 | |
*** bquan has quit IRC | 22:57 | |
*** Marga_ has joined #openstack-neutron | 22:57 | |
*** dboik_ has quit IRC | 22:57 | |
*** claudiub has joined #openstack-neutron | 22:59 | |
*** jobewan has quit IRC | 23:00 | |
*** ihrachyshka has quit IRC | 23:02 | |
openstackgerrit | Nachi Ueno proposed openstack/neutron: OpenContrail plugin code split https://review.openstack.org/163218 | 23:02 |
*** nati_ueno has quit IRC | 23:04 | |
*** nati_ueno has joined #openstack-neutron | 23:05 | |
*** _cjones_ has quit IRC | 23:05 | |
*** Marga_ has quit IRC | 23:06 | |
*** ijw has joined #openstack-neutron | 23:06 | |
*** Marga_ has joined #openstack-neutron | 23:06 | |
*** melwitt has quit IRC | 23:07 | |
*** Sukhdev has joined #openstack-neutron | 23:10 | |
*** Marga_ has quit IRC | 23:11 | |
*** prad has quit IRC | 23:14 | |
*** VijayB has joined #openstack-neutron | 23:15 | |
*** _cjones_ has joined #openstack-neutron | 23:15 | |
*** tsufiev_ has quit IRC | 23:16 | |
itzikb | Hi, Anyone familiar with this issue https://bugs.launchpad.net/neutron/+bug/1353165 ? | 23:16 |
openstack | Launchpad 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-neutron | 23:16 | |
*** Marga_ has joined #openstack-neutron | 23:17 | |
*** tsufiev_ has joined #openstack-neutron | 23:21 | |
*** vmtrooper has quit IRC | 23:21 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: WIP Last-ditch effort to get the router stuff out of the agent https://review.openstack.org/163222 | 23:22 |
carl_baldwin | amuller: ^ | 23:22 |
*** absubram has quit IRC | 23:24 | |
*** fredlhsu has quit IRC | 23:24 | |
*** gildub has quit IRC | 23:26 | |
*** tsufiev_ has quit IRC | 23:27 | |
*** sbfox has quit IRC | 23:30 | |
*** david-lyle is now known as david-lyle_afk | 23:33 | |
*** jprovazn has quit IRC | 23:33 | |
*** mwagner_lap has joined #openstack-neutron | 23:34 | |
*** sripriya has quit IRC | 23:34 | |
*** jamie_h has quit IRC | 23:37 | |
*** moha_hunt has quit IRC | 23:43 | |
*** nlahouti has quit IRC | 23:44 | |
*** banix has joined #openstack-neutron | 23:45 | |
*** britthouser has joined #openstack-neutron | 23:45 | |
*** tsufiev_ has joined #openstack-neutron | 23:46 | |
*** carl_baldwin has quit IRC | 23:48 | |
*** pritesh has quit IRC | 23:49 | |
*** fredlhsu has joined #openstack-neutron | 23:49 | |
*** bandarji has quit IRC | 23:50 | |
*** Sukhdev has quit IRC | 23:51 | |
openstackgerrit | Yanping Qu proposed openstack/neutron-fwaas: Provide hooks for Cisco FWaaS functional tests https://review.openstack.org/161938 | 23:52 |
*** chlong has joined #openstack-neutron | 23:54 | |
*** yamamoto has joined #openstack-neutron | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!