Thursday, 2015-01-29

*** dsundarraj has joined #openstack-neutron00:02
ChuckCZZelle: hi, any test results to share?00:03
*** dsundarr_ has joined #openstack-neutron00:04
*** gildub has joined #openstack-neutron00:06
*** dsundarraj has quit IRC00:07
*** dsundarr_ has quit IRC00:08
*** xuhanp has joined #openstack-neutron00:09
*** yamamoto has joined #openstack-neutron00:09
*** tomoe_ has joined #openstack-neutron00:12
*** dims has quit IRC00:13
*** seizadi has quit IRC00:13
*** padkrish_ has quit IRC00:14
*** jorgem has quit IRC00:14
*** padkrish has joined #openstack-neutron00:14
*** padkrish has quit IRC00:19
*** vishwanathj has joined #openstack-neutron00:19
*** padkrish has joined #openstack-neutron00:20
*** openstackgerrit has quit IRC00:21
*** packet has quit IRC00:21
*** openstackgerrit has joined #openstack-neutron00:21
*** dims has joined #openstack-neutron00:22
*** leenheer has quit IRC00:23
*** leenheer has joined #openstack-neutron00:23
*** david-lyle is now known as david-lyle_afk00:28
*** mlavalle has quit IRC00:29
*** Sukhdev has quit IRC00:30
*** sbfox has quit IRC00:30
*** xuhanp has quit IRC00:36
*** tomoe_ has quit IRC00:41
*** Sukhdev has joined #openstack-neutron00:44
*** tomoe_ has joined #openstack-neutron00:44
Sukhdevarosen: Ping00:44
Sukhdevarosen: wanted to chat about your comment on https://review.openstack.org/#/c/148749/00:45
*** dsneddon has quit IRC00:45
*** bjornar_ has joined #openstack-neutron00:46
*** suresh12_ has quit IRC00:47
*** markvoelker has quit IRC00:48
*** banix has joined #openstack-neutron00:48
*** yamamoto has quit IRC00:48
openstackgerritSukhdev Kapur proposed openstack/neutron: Thining Arista ML2 driver from neutron tree  https://review.openstack.org/14874900:50
*** isq has quit IRC00:51
*** RichardRaseley has quit IRC00:53
*** achanda has quit IRC00:56
*** manishg has quit IRC00:57
*** dsneddon has joined #openstack-neutron00:58
*** chenleji has quit IRC00:58
*** chenleji has joined #openstack-neutron00:58
*** manishg has joined #openstack-neutron00:58
*** markvoelker has joined #openstack-neutron00:59
*** oomichi has joined #openstack-neutron01:00
*** baoli has joined #openstack-neutron01:00
*** dsneddon has quit IRC01:00
*** achanda has joined #openstack-neutron01:00
*** dsneddon has joined #openstack-neutron01:01
*** manishg has quit IRC01:03
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Basic subnetpool CRUD. Do not merge  https://review.openstack.org/14869801:03
*** tidwellr has quit IRC01:03
*** addnull has joined #openstack-neutron01:03
*** openstackgerrit has quit IRC01:05
*** openstackgerrit has joined #openstack-neutron01:05
*** leenheer has quit IRC01:06
*** salv-orlando has joined #openstack-neutron01:07
*** prad has quit IRC01:08
*** salv-orlando has quit IRC01:08
*** dims has quit IRC01:10
*** banix has quit IRC01:10
*** tiswanso has quit IRC01:11
*** tellesnobrega_ has quit IRC01:15
*** jaypipes has quit IRC01:15
*** a_le has quit IRC01:21
*** seizadi has joined #openstack-neutron01:25
*** tiswanso has joined #openstack-neutron01:26
*** armax has quit IRC01:34
*** yapeng has joined #openstack-neutron01:35
*** a_le has joined #openstack-neutron01:35
*** banix has joined #openstack-neutron01:36
*** Marga_ has joined #openstack-neutron01:37
*** yapeng has quit IRC01:39
*** eliqiao has joined #openstack-neutron01:40
*** Sukhdev has quit IRC01:40
*** banix has quit IRC01:40
*** stanzgy has joined #openstack-neutron01:40
*** popw has joined #openstack-neutron01:41
*** Marga_ has quit IRC01:43
*** Marga_ has joined #openstack-neutron01:44
*** roeyc has quit IRC01:45
*** markmcclain has joined #openstack-neutron01:46
*** marun has quit IRC01:46
spandhegus: hey Angus! I have a very first cut of the tool here: https://github.com/shraddha-pandhe/Tools/tree/master/NovaNetToNeutronMigration01:48
*** bjornar_ has quit IRC01:48
*** xuhanp has joined #openstack-neutron01:48
spandheIts very immature right now.. Ill keep updating it. Just want to know if the overall logic or direction is OK or not..01:48
spandheanteaya: ^01:51
spandheThis is for avoiding hypervisor reboot during migration01:51
*** seizadi has quit IRC01:52
*** aranjan has quit IRC01:53
*** markmcclain has quit IRC01:54
*** banix has joined #openstack-neutron01:55
openstackgerritJian LI proposed openstack/neutron: Fix extra-dhcp-opt on stateless dhcpv6 subnet  https://review.openstack.org/14745501:57
*** markvoelker has quit IRC01:57
*** markvoelker has joined #openstack-neutron01:58
*** markvoelker has quit IRC02:02
*** markmcclain has joined #openstack-neutron02:03
*** tsg has quit IRC02:04
*** seizadi has joined #openstack-neutron02:06
gusspandhe: thanks! I'll take a look02:06
spandhethanks gus!02:06
*** manishg has joined #openstack-neutron02:08
*** yapeng has joined #openstack-neutron02:08
*** achanda has quit IRC02:09
gusspandhe: looks good from my quick read.02:09
*** achanda has joined #openstack-neutron02:10
*** SridharRamaswamy has quit IRC02:10
*** manishg_ has joined #openstack-neutron02:10
gusI see you have lots of logic to limit it to only a particular set of hypervisors (good), but the "List of all hypervisors in the cluster" on migrate_vifs_to_neutron.py:40 makes me nervous ;)02:10
gusit's really just the list passed in via -f, but if someone _were_ to put the full list there, then they'd shutdown all the hypervisors in the cluster at once (oops).02:11
*** nlahouti has quit IRC02:11
*** manishg__ has joined #openstack-neutron02:12
*** liusheng has quit IRC02:12
*** nlahouti has joined #openstack-neutron02:12
*** nlahouti has quit IRC02:12
*** killer_prince has quit IRC02:13
*** manishg has quit IRC02:13
*** manishg has joined #openstack-neutron02:13
*** rwsu is now known as rwsu-afk02:13
*** manishg_ has quit IRC02:15
*** s3wong has quit IRC02:16
*** manishg__ has quit IRC02:16
*** manishg_ has joined #openstack-neutron02:16
*** manishg has quit IRC02:18
*** openstackgerrit has quit IRC02:20
*** erkules_ has joined #openstack-neutron02:21
*** openstackgerrit has joined #openstack-neutron02:21
*** manishg_ has quit IRC02:21
*** erkules has quit IRC02:23
*** techcet has quit IRC02:23
*** techcet_ has joined #openstack-neutron02:23
*** padkrish has quit IRC02:24
*** padkrish has joined #openstack-neutron02:25
*** trinaths has joined #openstack-neutron02:26
openstackgerritXu Han Peng proposed openstack/neutron: Set locale before check dnsmasq version  https://review.openstack.org/14952402:28
*** markmcclain has quit IRC02:29
*** padkrish has quit IRC02:30
*** killer_prince has joined #openstack-neutron02:31
*** killer_prince is now known as lazy_prince02:32
*** banix has quit IRC02:35
*** addnull has quit IRC02:38
spandhegus: not shutting down hypervisors.. just the VMs02:38
spandhegus: comment bug. Ill fix the comment02:38
spandhegus: I am sure there will be a lot of revisions to the script02:39
*** trinaths has quit IRC02:43
*** tellesnobrega_ has joined #openstack-neutron02:43
*** dims has joined #openstack-neutron02:45
*** mlavalle has joined #openstack-neutron02:46
*** baoli has quit IRC02:47
*** techcet_ has quit IRC02:49
spandhegus: I think I missed your message just now.. saw the notification, but no message.. can you please resend?02:50
*** sripriya has quit IRC02:50
*** rkukura has quit IRC02:50
*** rkukura has joined #openstack-neutron02:51
*** mohitsharma has joined #openstack-neutron02:54
*** a_le has quit IRC02:59
*** radez_g0n3 is now known as radez03:04
*** seizadi has quit IRC03:07
*** techcet has joined #openstack-neutron03:07
*** jckasper has joined #openstack-neutron03:14
*** jckasper has quit IRC03:14
*** jckasper has joined #openstack-neutron03:14
*** Murali has joined #openstack-neutron03:18
*** lazy_prince has quit IRC03:19
*** openstackgerrit has quit IRC03:20
*** openstackgerrit has joined #openstack-neutron03:20
*** avozza is now known as zz_avozza03:24
*** Murali has quit IRC03:24
*** tiswanso has quit IRC03:30
*** tflynn_ has joined #openstack-neutron03:32
*** achanda has quit IRC03:32
*** sthillma has joined #openstack-neutron03:34
*** sbalukoff has quit IRC03:35
*** padkrish has joined #openstack-neutron03:35
*** tflynn has quit IRC03:36
*** tflynn_ has quit IRC03:36
*** oda-g has joined #openstack-neutron03:37
*** leenheer has joined #openstack-neutron03:37
*** Marga_ has quit IRC03:37
*** leenheer has quit IRC03:37
*** sthillma has quit IRC03:38
*** Marga_ has joined #openstack-neutron03:38
*** baoli has joined #openstack-neutron03:39
*** padkrish has quit IRC03:40
*** claudiub has quit IRC03:42
*** eliqiao has quit IRC03:43
*** Guest99731 has quit IRC03:43
*** xuhanp has quit IRC03:43
*** spandhe has quit IRC03:44
*** ivar-laz_ has joined #openstack-neutron03:44
*** eliqiao has joined #openstack-neutron03:45
*** ivar-lazzaro has quit IRC03:47
*** ivar-laz_ has quit IRC03:49
*** dkehn has quit IRC03:52
*** rkukura has quit IRC03:52
*** dkehn has joined #openstack-neutron03:52
*** achanda has joined #openstack-neutron03:56
*** manishg has joined #openstack-neutron03:58
*** a_le has joined #openstack-neutron03:59
*** manishg has quit IRC03:59
*** manishg has joined #openstack-neutron03:59
*** Guest99731 has joined #openstack-neutron03:59
*** yeungp has quit IRC04:00
*** xuhanp has joined #openstack-neutron04:00
*** oomichi has quit IRC04:01
*** yeungp has joined #openstack-neutron04:01
*** ijw has quit IRC04:03
*** Marga_ has quit IRC04:03
*** a_le has quit IRC04:03
*** manishg has quit IRC04:03
*** radez is now known as radez_g0n304:04
*** markmcclain has joined #openstack-neutron04:04
*** yeungp has quit IRC04:06
*** a_le has joined #openstack-neutron04:06
*** yamahata has joined #openstack-neutron04:06
*** achanda has quit IRC04:06
*** eliqiao has quit IRC04:07
*** dims has quit IRC04:08
*** dims has joined #openstack-neutron04:09
*** xuhanp has quit IRC04:09
*** pal has joined #openstack-neutron04:10
*** Guest99731 has quit IRC04:10
*** yamahata has quit IRC04:11
*** boris-42 has quit IRC04:13
*** eliqiao has joined #openstack-neutron04:15
*** mlavalle has quit IRC04:15
*** leenheer has joined #openstack-neutron04:18
*** markvoelker has joined #openstack-neutron04:21
*** xuhanp has joined #openstack-neutron04:22
*** sbfox has joined #openstack-neutron04:22
*** Guest99731 has joined #openstack-neutron04:22
*** dims has quit IRC04:23
*** vthapar has joined #openstack-neutron04:24
*** yamahata has joined #openstack-neutron04:25
*** SumitNaiksatam has quit IRC04:25
*** SumitNaiksatam has joined #openstack-neutron04:26
*** hanzhang has joined #openstack-neutron04:26
*** flwang has quit IRC04:27
*** shz has quit IRC04:28
*** addnull has joined #openstack-neutron04:29
*** killer_prince has joined #openstack-neutron04:29
*** killer_prince is now known as lazy_prince04:29
*** yamahata has quit IRC04:29
*** leenheer has quit IRC04:30
*** anish_ has joined #openstack-neutron04:31
*** anilvenkata has joined #openstack-neutron04:32
*** nlahouti has joined #openstack-neutron04:38
*** ijw has joined #openstack-neutron04:40
*** nlahouti has quit IRC04:41
*** nlahouti has joined #openstack-neutron04:41
openstackgerritMerged openstack/neutron: Fix AttributeError when using DVRServerRpcApi  https://review.openstack.org/15017904:42
*** lazy_prince has quit IRC04:44
*** achanda has joined #openstack-neutron04:45
*** ijw_ has joined #openstack-neutron04:45
*** shz has joined #openstack-neutron04:45
*** tellesnobrega_ has quit IRC04:46
*** killer_prince has joined #openstack-neutron04:46
*** killer_prince is now known as lazy_prince04:46
*** achanda has quit IRC04:46
*** yamahata has joined #openstack-neutron04:47
openstackgerritvishwanath jayaraman proposed openstack/neutron-fwaas: Implementation of Brocade Vyatta Firewall driver  https://review.openstack.org/14760004:47
*** harlowja is now known as harlowja_away04:48
*** ijw has quit IRC04:48
*** hanzhang has quit IRC04:48
*** a_le has quit IRC04:50
*** yamahata has quit IRC04:52
*** markvoelker has quit IRC04:53
*** markvoelker_ has joined #openstack-neutron04:53
*** neeti has joined #openstack-neutron04:56
*** achanda has joined #openstack-neutron04:57
*** eliqiao has quit IRC04:57
*** eliqiao has joined #openstack-neutron04:57
*** popw has quit IRC04:58
*** popw has joined #openstack-neutron04:58
*** f13o has quit IRC05:00
*** timbyr_ has quit IRC05:00
*** mohitsharma has quit IRC05:01
*** markmcclain has quit IRC05:02
*** stanzgy has quit IRC05:04
*** xgerman_ has joined #openstack-neutron05:04
*** sbalukoff has joined #openstack-neutron05:08
*** popw has quit IRC05:11
*** Sukhdev has joined #openstack-neutron05:11
*** markmcclain has joined #openstack-neutron05:11
*** zzzeek has quit IRC05:12
*** sbfox has quit IRC05:13
*** timbyr_ has joined #openstack-neutron05:13
*** f13o has joined #openstack-neutron05:13
*** seizadi has joined #openstack-neutron05:14
*** mestery has quit IRC05:15
*** a_le has joined #openstack-neutron05:16
*** mestery has joined #openstack-neutron05:16
*** markvoelker_ has quit IRC05:16
*** carl_baldwin has joined #openstack-neutron05:16
*** ajo has joined #openstack-neutron05:16
*** markvoelker has joined #openstack-neutron05:16
*** stanzgy has joined #openstack-neutron05:17
HenryGkevinbenton: ping05:18
kevinbentonHenryG: pong05:18
HenryGkevinbenton: two unrelated questions ...05:18
HenryGFirst a mock question05:19
kevinbentonto prepare me for the real question? ;)05:19
*** julim_ has joined #openstack-neutron05:19
openstackgerritBilly Olsen proposed openstack/neutron: Refactor logging in loop to only log debug messages once  https://review.openstack.org/14978405:20
HenryGkevinbenton: https://github.com/openstack/neutron/blob/master/neutron/tests/unit/test_linux_external_process.py#L14105:20
*** markmcclain has quit IRC05:21
HenryGkevinbenton: What is that actually mocking? Shouldn't it be mock.patch('os.path.isdir') ?05:21
*** trinaths has joined #openstack-neutron05:21
*** markvoelker has quit IRC05:22
HenryGkevinbenton: I am turning to you since I know you are an expert on mock :)05:22
kevinbentonso i think in that case they are ultimately mocking the same thing05:22
*** julim has quit IRC05:22
*** markmcclain has joined #openstack-neutron05:22
HenryGkevinbenton: THen how can line 146 make sense?05:23
*** vishwanathj has quit IRC05:23
kevinbentonHenryG: well it does, doesn't it?05:23
*** yapeng has quit IRC05:24
*** yshafrir has joined #openstack-neutron05:24
*** dims has joined #openstack-neutron05:24
*** popw has joined #openstack-neutron05:24
kevinbentonif they are setting ensure_pids_dir=False05:24
*** baoli has quit IRC05:24
HenryGkevinbenton: ah, ok05:24
kevinbentonHenryG: i think the part that might not make sense is setting the return value05:24
HenryGkevinbenton: yup05:24
kevinbentonHenryG: since they are asserting that nothing is calling it05:24
*** leenheer has joined #openstack-neutron05:26
*** popw1 has joined #openstack-neutron05:26
kevinbentonHenryG: yeah, it passes if i remove that return value05:26
*** yamamoto_ has joined #openstack-neutron05:27
*** popw has quit IRC05:28
*** dims has quit IRC05:28
HenryGkevinbenton: thanks for helping me get past a mind block there05:28
*** markmcclain has quit IRC05:29
HenryGkevinbenton: Second question: Role Base Access Control05:29
HenryGkevinbenton: You filed a spec05:29
kevinbentonHenryG: right, it didn't receiving an approval :(05:30
*** anilvenkata has quit IRC05:30
kevinbentonreceive *05:30
HenryGkevinbenton: Are you going to try again for The-release-that-should-be-named-Lemming?05:30
kevinbentonHenryG: yes, definitely05:31
HenryGkevinbenton: Cool. Have you done any implementing/POC/etc?05:31
*** Murali has joined #openstack-neutron05:32
kevinbentonHenryG: no, not yet. just some discussions with Salvatore and Carl of the approach05:32
kevinbentonHenryG: it got shot down early enough that there wasn't code yet05:32
*** tflynn has joined #openstack-neutron05:33
HenryGkevinbenton: NP. Let me know when that picks up again.05:33
kevinbentonHenryG: sounds good05:33
*** addnull has quit IRC05:33
*** seizadi has quit IRC05:33
kevinbentonHenryG: back to the mock thing. A time where patching a path would be different than the library directly is when you are patching a module's import05:35
kevinbentone.g. mock.patch.object(ep.utils, 'os') will change the ep.utils reference to os05:35
HenryGkevinbenton: Makes sense, thanks. And mock.patch('os') would mock it for everyone, not just ep.utils?05:36
*** tflynn has quit IRC05:37
HenryGThough I imagine that could break everything o_O05:38
kevinbentonHenryG: right, but mock won't even let you do that05:38
HenryGkevinbenton: If I want to shoot myself in the foot with mock I have to do it in more subtle ways?05:39
kevinbentonHenryG: it will just say it's an invalid target05:39
HenryG:)05:39
kevinbentonHenryG: yeah05:39
HenryGkevinbenton: Thanks for your time. I'm off to bed.05:40
kevinbentonHenryG: gnight!05:40
*** cburgess has quit IRC05:41
*** sthillma has joined #openstack-neutron05:41
*** cburgess has joined #openstack-neutron05:41
*** sthillma has quit IRC05:42
*** Longgeek has joined #openstack-neutron05:43
*** anilvenkata has joined #openstack-neutron05:46
*** markvoelker has joined #openstack-neutron05:47
*** anilvenkata has left #openstack-neutron05:48
*** anilvenkata has joined #openstack-neutron05:49
*** shz has quit IRC05:52
*** markvoelker has quit IRC05:52
*** popw1 has quit IRC05:52
*** popw has joined #openstack-neutron05:53
*** ijw_ has quit IRC05:54
*** ijw has joined #openstack-neutron05:55
openstackgerritMerged openstack/neutron: Drop bin/neutron-rootwrap  https://review.openstack.org/14897505:55
*** oomichi_ has joined #openstack-neutron05:57
*** oomichi_ has quit IRC05:57
*** leenheer has quit IRC05:58
*** salv-orlando has joined #openstack-neutron05:59
*** sbfox has joined #openstack-neutron05:59
*** josecastroleon_ has joined #openstack-neutron06:00
*** tflynn has joined #openstack-neutron06:00
*** tflynn has quit IRC06:01
*** carl_baldwin has quit IRC06:02
*** tflynn has joined #openstack-neutron06:02
*** Longgeek has quit IRC06:03
*** padkrish has joined #openstack-neutron06:03
*** josecastroleon has quit IRC06:03
*** vks has joined #openstack-neutron06:04
*** padkrish has quit IRC06:04
*** josecastroleon__ has joined #openstack-neutron06:05
*** padkrish has joined #openstack-neutron06:05
*** pradipta_away is now known as pradipta06:08
*** josecastroleon_ has quit IRC06:09
*** timbyr_ has quit IRC06:10
*** SridharG has joined #openstack-neutron06:10
*** leenheer has joined #openstack-neutron06:10
*** seizadi has joined #openstack-neutron06:14
*** spandhe has joined #openstack-neutron06:14
*** etacsufbo has joined #openstack-neutron06:15
*** markmcclain has joined #openstack-neutron06:17
*** Guest99731 has quit IRC06:17
*** Guest99731 has joined #openstack-neutron06:18
*** Longgeek has joined #openstack-neutron06:18
*** anshul has joined #openstack-neutron06:19
*** timbyr_ has joined #openstack-neutron06:19
*** xgerman_ has quit IRC06:21
*** Longgeek has quit IRC06:22
*** yatin has joined #openstack-neutron06:25
*** leenheer has quit IRC06:25
*** reed has quit IRC06:26
*** yapeng has joined #openstack-neutron06:27
*** seizadi has quit IRC06:28
*** yapeng has quit IRC06:28
*** lazy_prince has quit IRC06:28
*** Sukhdev has quit IRC06:30
*** vthapar has quit IRC06:31
*** mohitsharma has joined #openstack-neutron06:32
*** spandhe has quit IRC06:33
*** spandhe has joined #openstack-neutron06:37
*** tflynn has quit IRC06:39
*** tflynn has joined #openstack-neutron06:39
*** numan has joined #openstack-neutron06:40
*** nlahouti has quit IRC06:42
*** killer_prince has joined #openstack-neutron06:43
*** killer_prince is now known as lazy_prince06:43
openstackgerritJian LI proposed openstack/neutron: Include icmpv6 in sg_supported_protocols  https://review.openstack.org/15112806:44
*** nlahouti has joined #openstack-neutron06:46
*** aepifanov has joined #openstack-neutron06:47
*** zhiyuan has joined #openstack-neutron06:48
*** markvoelker has joined #openstack-neutron06:48
*** afazekas has joined #openstack-neutron06:51
*** JoshNang has quit IRC06:52
*** markvoelker has quit IRC06:52
*** achanda has quit IRC06:53
*** gildub has quit IRC06:53
*** evgenyf has joined #openstack-neutron06:55
*** nlahouti has quit IRC06:55
*** achanda has joined #openstack-neutron06:58
*** vthapar has joined #openstack-neutron06:59
*** JoshNang has joined #openstack-neutron07:01
*** irenab has joined #openstack-neutron07:05
openstackgerritMerged openstack/neutron: Make agent config available to the router classes  https://review.openstack.org/14906707:05
openstackgerritMerged openstack/neutron: test_agent_scheduler: Fix a misleading variable name  https://review.openstack.org/15068907:06
*** coolsvap|afk is now known as coolsvap07:10
*** Miouge has joined #openstack-neutron07:11
*** salv-orlando has quit IRC07:11
*** Miouge has quit IRC07:12
enikanorov_rossella_s: hi07:13
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ml2: Simplify _process_provider_create  https://review.openstack.org/15113007:14
*** markmcclain has quit IRC07:15
openstackgerritJian LI proposed openstack/neutron: Fix extra-dhcp-opt on stateless dhcpv6 subnet  https://review.openstack.org/14745507:16
*** lpetrut has joined #openstack-neutron07:16
*** achanda has quit IRC07:19
*** pal has quit IRC07:19
*** pal has joined #openstack-neutron07:20
*** markvoelker has joined #openstack-neutron07:21
*** jaypipes has joined #openstack-neutron07:22
*** rotbeard has joined #openstack-neutron07:23
*** ronis has joined #openstack-neutron07:24
*** apuimedo has joined #openstack-neutron07:26
*** markvoelker has quit IRC07:27
*** irenab has quit IRC07:29
*** sc68cal has quit IRC07:32
*** anish_ has quit IRC07:32
*** jaypipes has quit IRC07:33
*** scheuran has joined #openstack-neutron07:33
*** oomichi has joined #openstack-neutron07:33
*** sc68cal has joined #openstack-neutron07:36
*** padkrish has quit IRC07:37
*** doude has joined #openstack-neutron07:38
*** padkrish has joined #openstack-neutron07:38
*** belmoreira has joined #openstack-neutron07:40
*** aleksandr_null has quit IRC07:40
*** vthapar_ has joined #openstack-neutron07:41
*** aleksandr_null has joined #openstack-neutron07:42
*** yamahata has joined #openstack-neutron07:44
*** vthapar has quit IRC07:44
*** Miouge has joined #openstack-neutron07:48
*** yamahata has quit IRC07:49
*** shz has joined #openstack-neutron07:52
*** matrohon has joined #openstack-neutron07:53
*** Miouge has quit IRC07:54
*** pask81 has joined #openstack-neutron07:56
*** Guest99731 has quit IRC07:57
openstackgerritMerged openstack/neutron: Updated from global requirements  https://review.openstack.org/14997007:57
*** timbyr_ has quit IRC07:58
*** f13o has quit IRC07:58
*** devvesa has joined #openstack-neutron07:59
*** nyechiel__ has joined #openstack-neutron07:59
*** ronis_ has joined #openstack-neutron08:01
*** yamahata has joined #openstack-neutron08:02
*** ronis has quit IRC08:02
*** Miouge has joined #openstack-neutron08:05
*** addnull has joined #openstack-neutron08:06
*** pask81 has quit IRC08:06
*** yamahata has quit IRC08:07
*** spandhe has quit IRC08:09
*** etacsufbo has quit IRC08:10
*** itzikb has joined #openstack-neutron08:10
*** nmagnezi has joined #openstack-neutron08:10
*** jlibosva has joined #openstack-neutron08:12
*** shz has quit IRC08:12
*** salv-orlando has joined #openstack-neutron08:12
*** shz has joined #openstack-neutron08:12
*** safchain has joined #openstack-neutron08:13
*** timbyr_ has joined #openstack-neutron08:15
*** f13o has joined #openstack-neutron08:16
openstackgerritTrinath Somanchi proposed openstack/neutron-fwaas: Freescale FWaaS Plugin  https://review.openstack.org/14888408:16
*** salv-orlando has quit IRC08:16
*** yapeng has joined #openstack-neutron08:16
*** ronis_ has quit IRC08:17
*** yamamoto_ has quit IRC08:17
openstackgerritMerged openstack/neutron: Make the interface driver available to the router classes  https://review.openstack.org/15087208:18
openstackgerritMerged openstack/neutron: TestL2PopulationRpcTestCase: Stop loading linuxbridge mech driver  https://review.openstack.org/14995508:19
*** pask81 has joined #openstack-neutron08:19
*** yamahata has joined #openstack-neutron08:21
*** yapeng has quit IRC08:22
*** vthapar_ has quit IRC08:22
*** kawa2014 has joined #openstack-neutron08:23
*** markvoelker has joined #openstack-neutron08:25
*** padkrish has quit IRC08:25
*** yamahata has quit IRC08:25
*** Miouge has quit IRC08:26
*** pask81 has quit IRC08:26
*** mestery has quit IRC08:29
*** yamahata has joined #openstack-neutron08:29
*** bjornar_ has joined #openstack-neutron08:30
openstackgerritwatanabe isao proposed openstack/python-neutronclient: Skip None id when getting security_group_ids  https://review.openstack.org/15072808:30
*** markvoelker has quit IRC08:31
*** mestery has joined #openstack-neutron08:31
*** aveiga has quit IRC08:32
*** aveiga has joined #openstack-neutron08:32
*** ronis_ has joined #openstack-neutron08:32
*** tflynn has quit IRC08:33
*** tflynn has joined #openstack-neutron08:34
*** pask81 has joined #openstack-neutron08:34
*** yamahata has quit IRC08:34
*** etacsufbo has joined #openstack-neutron08:35
*** chlong has quit IRC08:36
*** zz_avozza is now known as avozza08:36
*** Miouge has joined #openstack-neutron08:38
*** etacsufbo has quit IRC08:41
*** lpetrut has quit IRC08:44
*** pask811 has joined #openstack-neutron08:45
*** yamahata has joined #openstack-neutron08:47
*** pask81 has quit IRC08:47
*** vthapar has joined #openstack-neutron08:50
*** yamahata has quit IRC08:51
*** jp_at_hp has joined #openstack-neutron08:51
*** jpena has joined #openstack-neutron08:52
*** irenab has joined #openstack-neutron08:53
*** eliqiao has left #openstack-neutron08:55
*** ygbo has joined #openstack-neutron08:58
ajokevinbenton, sometimes I feel like if we need a rewritten dnsmasq for ourselves ;D09:00
ajonot only that I found dnsmasq code not specially good (yet works),09:00
ajobut that we need to tweak it quite a bit everywhere ;)09:00
*** karimb has joined #openstack-neutron09:00
ajoit would be gorgeous to to be able to send FORCERENEWS for example09:01
rossella_senikanorov_: hi09:01
ajohi rossella_s  :)09:02
*** bjornar_ has quit IRC09:02
rossella_sajo: hi09:02
kevinbentonajo: yeah, definitely09:02
ajokevinbenton, may be we should start contributing dnsmasq, for that sort of things, but given it's tiny nature and embedded target, they would start refusing new features like that09:03
*** chenleji has quit IRC09:03
kevinbentonajo: most likely09:03
ajoI need to talk the author for a different reason, but I will rise that topic09:04
*** nyechiel has joined #openstack-neutron09:04
ajo(suppressing certain kind of logs, and being able to send FORCERENEW) if they'll accept such patches I can write them09:04
lukasaFWIW, I've found it fairly easy to work with Simon09:05
*** avozza is now known as zz_avozza09:05
ajoat least the first one should be quite easy, the 2nd one can be a bit more complicated depending on how they handle config reloads, and how does the readonly leases work09:05
*** Miouge has quit IRC09:05
*** nyechiel__ has quit IRC09:05
lukasaMy team's had a few patches go into dnsmasq and it hasn't really been a problem. Though I suspect the scope of the work you're considering is larger than what we did...09:05
*** irenab has quit IRC09:06
kevinbentonajo: well FORCERENEW is tricky too because if that packet is lost, the client will be stuck with the old address09:06
ajokevinbenton, that's true, but in combination with a lower lease, we can speedup a lot the ip change09:06
ajolike instantaneous09:06
ajosorry09:06
ajolower renewal-time09:06
*** ijw has quit IRC09:06
kevinbentonajo: definitely09:06
ajolukasa, thanks for the input, that's really good to know,09:07
kevinbentonajo: i think ideally with FORCERENEW you would have some state tracking that would retry several times if a follow-up renewal wasn't seen form the client09:07
kevinbentonajo: but that's quite a bit of complexity09:07
ajokevinbenton, yes, may be too much for an embedded dnsmasq,09:07
ajoor even for the time I could put together for this09:08
ajobut, a first implementation with blind FORCERENEW would be a good start09:08
kevinbentonajo: ack. don't spend time on the dhcp-renewal time though. i have a patch mostly ready to go for that09:08
ajokevinbenton ack09:08
kevinbentonajo: because i wanted to test it to make sure what i was saying wasn't bogus :)09:08
*** tflynn has quit IRC09:09
*** tflynn has joined #openstack-neutron09:09
*** chenleji has joined #openstack-neutron09:10
*** addnull has quit IRC09:10
openstackgerritwatanabe isao proposed openstack/neutron: Bug-Fix for unexpected DHCP agent redundant  https://review.openstack.org/13865909:12
*** salv-orlando has joined #openstack-neutron09:12
*** irenab has joined #openstack-neutron09:13
*** chenleji has quit IRC09:13
*** chenleji has joined #openstack-neutron09:14
*** jpich has joined #openstack-neutron09:14
*** flwang has joined #openstack-neutron09:14
*** tflynn has quit IRC09:15
ajokevinbenton, I also wonder how un-performant (or not so un-performant) would be to provide a python dhcp server from the dhcp-agent itself09:15
*** watanabe_isao has quit IRC09:15
kevinbentonajo: i had thought about that in another context09:15
kevinbentonajo: when there are 1000 networks being serviced by one node, the overhead of dnsmasq processes starts to add up09:16
*** jistr has joined #openstack-neutron09:16
kevinbentonajo: if that could all be serviced by the dhcp agent process, that would be nice09:17
ajokevinbenton, yes... the spawn time, etc...09:17
kevinbentonajo: right09:17
ajokevinbenton, we would yet have to plug all the interfaces, and listen into them, etc..09:17
*** irenab has quit IRC09:17
ajobut that would save a lot of dnsmasq respawning, as we would be in control of the network info all the time09:17
kevinbentonajo: precisely09:18
*** salv-orlando has quit IRC09:18
ajokevinbenton, I also heard somebody talking about OF-DHCP, but not sure if that's totally possible with openflow09:18
ajoand also, wouldn't work for all networks,09:18
ajothe good of the current dhcp-agent is that it works for all...09:18
kevinbentonajo: right09:19
kevinbentonajo: using linux bridge, etc09:19
ajoyup09:19
*** lpetrut has joined #openstack-neutron09:19
kevinbentonajo: i wonder if we still need namespaces at that point09:19
ajokevinbenton, I suppose we do, because we need to set an IP on the interface09:20
ajokevinbenton, unless all traffic was handled in raw, but that's mad :D09:20
kevinbentonajo: yeah, raw packet processing might be painful09:20
*** achanda has joined #openstack-neutron09:21
kevinbentonajo: not having the IP on the tap would lose the nice debugging feature of pinging from the dhcp interface09:21
kevinbentonajo: so it's probably worth keeping09:22
ajokevinbenton, true, even if UDP is doable in raw... debugging would be a PITA09:22
*** Longgeek has joined #openstack-neutron09:23
*** addnull has joined #openstack-neutron09:23
kevinbentonajo: so then we need something likea unix domain socket to communicate between the agent and the namespace09:23
kevinbentonajo: and then some mini agent running in the namespace...09:23
ajokarimb, hmmm, argh, we're back to the old model (at least from the startup perspective)09:24
ajokarimb-> kevinbenton  sorry09:24
ajo:)09:24
kevinbentonajo: right09:24
karimbkarimb, :)09:24
karimbajo, :)09:24
openstackgerritMerged openstack/neutron: tests: don't spread fixtures.TempDir throughout test cases  https://review.openstack.org/14805409:24
ajokevinbenton, couldn't we fork and switch to the namespace?09:24
*** erkules_ is now known as erkules09:24
ajowe would have a lot of processes for dhcp-agent, but yet, that'd be faster09:25
openstackgerritMerged openstack/neutron: Allow 'max_l3_agents_per_router' to be set to '0'  https://review.openstack.org/14994709:25
*** sbfox has quit IRC09:25
*** zz_avozza is now known as avozza09:25
ajokevinbenton, I have to experiment around this eventually to see if it's possible/reasonable09:25
kevinbentonajo: even if we had mini-agents running in the namespace, they wouldn't have to restart all of the time if their job was just to relay info to the main agent09:25
ajoalso forking, would help with really distributing the load across cores09:26
kevinbentonajo: so it wouldn't be quite as bad as now09:26
ajokevinbenton, totally09:26
kevinbentonajo: yeah, forking might be an option09:26
kevinbentonajo: ok. well i'm off to bed. catch you later09:26
ajokevinbenton, have a good sleep ;)09:26
kevinbentonajo: you too, whenever bed time rolls around :)09:27
*** ihrachyshka has joined #openstack-neutron09:27
*** Longgeek has quit IRC09:27
ajoI just started the day :D09:27
ajo(GMT+1 here)09:27
*** markvoelker has joined #openstack-neutron09:28
kevinbentonah, well then in 14 hours or so09:29
ajo:)09:29
*** chenleji has quit IRC09:32
*** chenleji has joined #openstack-neutron09:33
*** scheuran has quit IRC09:34
*** markvoelker has quit IRC09:34
openstackgerritXu Han Peng proposed openstack/neutron: Fix dist_fip_count problem during router migration  https://review.openstack.org/15115309:34
openstackgerritYoni Shafrir proposed openstack/neutron: Fixed L3 agent manual scheduling for HA routers  https://review.openstack.org/14329709:37
*** nyechiel has quit IRC09:37
*** Miouge has joined #openstack-neutron09:45
*** lpetrut has quit IRC09:47
*** lpetrut has joined #openstack-neutron09:47
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Refactor _convert_sgr_to_iptables_rules in iptables_firewall  https://review.openstack.org/15083309:48
ajohaleyb , rossella_s ^ addressed your comment, and refactored an extra (common) part out to a function09:48
rossella_sajo: thanks!09:49
*** scheuran has joined #openstack-neutron09:49
*** moshele has joined #openstack-neutron09:49
rossella_sajo: you are so fast09:50
rossella_s:p09:50
*** tomoe_ has quit IRC09:50
ajorossella_s, sadly sometimes only :/, when I'm not too swamped with D/S things09:50
rossella_sajo: I know that feeling09:51
ajo:]09:51
*** yfried__ has quit IRC09:52
*** a_le has quit IRC09:52
*** hanzhang has joined #openstack-neutron09:57
*** shz has quit IRC09:59
*** Marga_ has joined #openstack-neutron10:00
*** salv-orlando has joined #openstack-neutron10:00
*** igordcard has joined #openstack-neutron10:02
*** feleouet has quit IRC10:04
*** matrohon has quit IRC10:04
*** yapeng has joined #openstack-neutron10:06
*** mohitsharma has quit IRC10:07
*** mohitsharma has joined #openstack-neutron10:08
*** pask811 has quit IRC10:10
*** yapeng has quit IRC10:10
*** Miouge has quit IRC10:12
*** changbl has quit IRC10:12
trinathsInitial py27 tests fail for neutron-fwaas with the error http://paste.openstack.org/show/163598/ . Is there any wrong with the repository?10:12
*** chlong has joined #openstack-neutron10:13
*** karimb has quit IRC10:15
*** aepifanov has quit IRC10:15
*** aepifanov has joined #openstack-neutron10:15
*** tellesnobrega_ has joined #openstack-neutron10:17
*** a_le has joined #openstack-neutron10:19
*** oreillyd has joined #openstack-neutron10:20
*** akamyshnikova has quit IRC10:20
*** akamyshnikova has joined #openstack-neutron10:22
*** mohitsharma has quit IRC10:22
*** addnull has quit IRC10:24
openstackgerritCedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional tests  https://review.openstack.org/14341210:24
openstackgerritCedric Brandily proposed openstack/neutron: Refactor Pinger class  https://review.openstack.org/15116710:24
*** pask81 has joined #openstack-neutron10:25
*** anilvenkata has quit IRC10:25
*** chlong has quit IRC10:27
*** mohitsharma has joined #openstack-neutron10:27
*** hanzhang has quit IRC10:28
*** markvoelker has joined #openstack-neutron10:30
*** karimb has joined #openstack-neutron10:30
*** a_le has quit IRC10:30
*** karimb has quit IRC10:30
*** geekinutah has quit IRC10:31
*** karimb has joined #openstack-neutron10:31
*** pask811 has joined #openstack-neutron10:31
*** mohitsharma has quit IRC10:32
*** pask81 has quit IRC10:34
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Default security group table  https://review.openstack.org/14210110:34
*** markvoelker has quit IRC10:34
*** nyechiel has joined #openstack-neutron10:35
*** scheuran has quit IRC10:35
*** geekinutah has joined #openstack-neutron10:36
*** harlowja_away has quit IRC10:36
*** pradipta is now known as pradipta_away10:39
*** tellesnobrega_ has quit IRC10:43
*** chlong has joined #openstack-neutron10:44
*** scheuran has joined #openstack-neutron10:49
*** Marga_ has quit IRC10:49
*** stanzgy has quit IRC10:51
ajosalv-orlando, ping, what's the future of extensions?, are we going to pull all them in to the core models?,10:53
ajowill we allow extensions in the future, or that kind of stuff should go directly , not as a extension?10:54
*** pask811 has quit IRC10:54
ajo(I was looking at the -not for now- qos spec from Sean Collins)10:54
*** pask81 has joined #openstack-neutron10:55
*** tellesnobrega_ has joined #openstack-neutron11:00
*** tellesnobrega_ has quit IRC11:06
*** tellesnobrega_ has joined #openstack-neutron11:07
openstackgerritXu Han Peng proposed openstack/neutron: Fix dist_fip_count problem during router migration  https://review.openstack.org/15115311:07
openstackgerritXu Han Peng proposed openstack/neutron: Remove qg from router namespace for DVR migration  https://review.openstack.org/15118111:07
*** xuhanp has quit IRC11:07
*** Miouge has joined #openstack-neutron11:08
*** oomichi has quit IRC11:08
*** jlanoux has joined #openstack-neutron11:08
*** tellesnobrega_ has quit IRC11:11
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Default security group table  https://review.openstack.org/14210111:12
trinathsInitial py27 tests fail for neutron-fwaas with the error http://paste.openstack.org/show/163598/ . Is there any wrong with the repository?  An error on ReouterInfo, from neutron/agents/l3/reouter_info.py11:13
openstackgerritNuman Siddique proposed openstack/neutron-vpnaas: Fix the ipsec conn issue when peer addr is fqdn  https://review.openstack.org/14500511:15
*** Marga_ has joined #openstack-neutron11:19
*** Marga_ has quit IRC11:20
*** Marga_ has joined #openstack-neutron11:21
*** achanda has quit IRC11:22
*** anish_ has joined #openstack-neutron11:23
*** Marga_ has quit IRC11:25
openstackgerritMoshe Levi proposed openstack/neutron: Thin MLNX ML2 mechanism driver and agent  https://review.openstack.org/14861411:26
*** pask81 has quit IRC11:26
*** oomichi_ has joined #openstack-neutron11:27
*** pask81 has joined #openstack-neutron11:28
*** markvoelker has joined #openstack-neutron11:31
*** doude has quit IRC11:31
*** pask81 has quit IRC11:32
*** markvoelker has quit IRC11:35
*** igordcard has quit IRC11:37
*** igordcard has joined #openstack-neutron11:39
*** chlong has quit IRC11:39
*** aix has joined #openstack-neutron11:40
*** igordcard has quit IRC11:41
*** pc_m has joined #openstack-neutron11:43
*** Miouge has quit IRC11:43
*** karimb has quit IRC11:46
*** baoli has joined #openstack-neutron11:48
*** pask81 has joined #openstack-neutron11:49
*** amotoki has quit IRC11:51
enikanorov_rossella_s: wanted to ask, how that decorator should be used (from oslo.db)11:51
*** yapeng has joined #openstack-neutron11:55
*** karimb has joined #openstack-neutron11:55
*** olc has quit IRC11:56
trinathspc_m: Hi11:57
*** yapeng has quit IRC11:59
*** oomichi_ has quit IRC11:59
pc_mtrinaths: hi12:01
*** pal has quit IRC12:01
*** olc has joined #openstack-neutron12:02
*** chlong has joined #openstack-neutron12:05
trinathspc_m: thanks for the review. I have answered your questions/comments on the code with new patchset.12:07
pc_mtrinaths: ok. I'll be getting to reviews in a little while and can look at it.12:07
*** radez_g0n3 is now known as radez12:08
trinathspc_m: okay.  There is some issue with fwaas repo. py27 tests fail at RouterInfo lib.12:08
rossella_senikanorov_ : the same way the safe_for_db_retry is used, just for deadlock error12:09
rossella_senikanorov_: does that answer your question?12:09
openstackgerritYoni Shafrir proposed openstack/neutron: Fixed L3 agent manual scheduling for HA routers  https://review.openstack.org/14329712:09
enikanorov_rossella_s: i mean, i can't just wrap any method with  safe_for_db_retry, can I?12:09
trinathspc_m: here is the log I captured.  http://paste.openstack.org/show/163598/ .  Did not find the real reason behind.12:10
enikanorov_rossella_s: is there an example anywhare?12:10
*** trinaths has quit IRC12:10
enikanorov_rossella_s: when i was adding UT for the new feature of the wrapper, I had to call it directly, decorator itself is not enough12:11
rossella_senikanorov_ : I think you can add it to any method12:12
rossella_senikanorov_ : in the UT I added for my patch, I decorated a test method, that was enough12:13
rossella_senikanorov_ : look at DBDeadlockTestCase ...12:13
*** lazy_prince has quit IRC12:14
*** raildo has joined #openstack-neutron12:14
enikanorov_i don't quite understand, honestly. Code says that decorated dbapi calls will be repeated12:14
enikanorov_i'm not sure if that's what we want12:14
enikanorov_i just don't understand how the decorator is invoked12:15
enikanorov_when you decorate any method with any of these decorators - it will not add retries, unless decorated method has some dbapi call inside12:16
*** oomichi_ has joined #openstack-neutron12:16
*** Miouge has joined #openstack-neutron12:16
enikanorov_but i don't understand what part is then retried upon any of supported conditions12:16
rossella_senikanorov_ : what part...what do you mean? the whole method decorated will be retried, is that your question?12:17
*** killer_prince has joined #openstack-neutron12:18
*** killer_prince is now known as lazy_prince12:18
enikanorov_rossella_s: that would be my expectation, but that's not so I guess12:18
rossella_senikanorov_ : why do you say so? do you have any evidence? btw I am pretty sure there's an example for that decorator somewhere in the code but I can't find it right now12:18
enikanorov_def retry_on_deadlock(f):12:19
enikanorov_    f.enable_retry_on_deadlock = True12:19
enikanorov_    return f12:19
*** anilvenkata has joined #openstack-neutron12:19
enikanorov_it doesn't do anything, as you see12:19
enikanorov_so this decorator itself will not make f to repeat12:19
enikanorov_i guess f has to have calls to dbapi inside to make this magic work12:21
*** Marga_ has joined #openstack-neutron12:22
rossella_syep...it works only for dbapi, you are right12:22
rossella_sthe magic is in __getattr__ of DBApi12:22
*** baoli has quit IRC12:22
rossella_sso it won't work for any method, you are right12:22
rossella_ssorry I have to go right now, I will be back after lunch, will ping you then12:22
*** Marga_ has quit IRC12:26
*** vks has quit IRC12:29
*** markvoelker has joined #openstack-neutron12:31
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Implements the ProcessMonitor in the l3_agent  https://review.openstack.org/11493112:32
*** neeti has quit IRC12:33
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Implements the ProcessMonitor in the l3_agent  https://review.openstack.org/11493112:35
*** markvoelker has quit IRC12:36
openstackgerritDane LeBlanc proposed openstack/neutron: Support Multiple IPv6 Prefixes and Addresses for IPv6 Network  https://review.openstack.org/11333912:37
*** tomoe_ has joined #openstack-neutron12:37
*** amuller has joined #openstack-neutron12:41
*** julim has joined #openstack-neutron12:43
*** baoli has joined #openstack-neutron12:45
*** belmoreira has quit IRC12:46
*** julim_ has quit IRC12:46
*** avozza is now known as zz_avozza12:46
*** baoli_ has joined #openstack-neutron12:46
*** mwagner_lap has quit IRC12:47
*** baoli_ has quit IRC12:48
*** baoli has quit IRC12:50
*** baoli has joined #openstack-neutron12:50
*** zz_avozza is now known as avozza12:53
openstackgerritRussell Bryant proposed openstack/neutron: Scope dvr rpc api using a messaging namespace  https://review.openstack.org/14973512:56
*** adam_g is now known as adam_g_out12:57
*** Networkn3rd has quit IRC13:00
*** doude has joined #openstack-neutron13:04
*** yatin has quit IRC13:06
ZZelleChuckC, hi13:06
*** thorst1 has joined #openstack-neutron13:07
*** karimb has quit IRC13:07
*** Miouge has quit IRC13:09
*** Miouge has joined #openstack-neutron13:12
*** ronis_ has quit IRC13:13
*** oomichi_ has quit IRC13:14
*** EmilienM|afk is now known as EmilienM13:14
*** yshafrir has quit IRC13:15
*** radez is now known as radez_g0n313:19
*** Miouge has quit IRC13:20
*** Marga_ has joined #openstack-neutron13:22
*** pal has joined #openstack-neutron13:23
*** nmagnezi has quit IRC13:23
*** Miouge has joined #openstack-neutron13:23
*** Marga_ has quit IRC13:27
*** nmagnezi has joined #openstack-neutron13:30
*** irenab has joined #openstack-neutron13:31
*** boris-42 has joined #openstack-neutron13:31
*** markvoelker has joined #openstack-neutron13:33
openstackgerritNuman Siddique proposed openstack/neutron-vpnaas: Fix the neutron-vpnaas unit test failures  https://review.openstack.org/15123313:34
*** markvoelker has quit IRC13:35
*** markvoelker_ has joined #openstack-neutron13:35
rossella_senikanorov_ ping13:36
enikanorov_rossella_s: pong13:36
*** Miouge has quit IRC13:36
rossella_senikanorov_ : I am back, so yes if you were planning to use that decorator outside of the DBAPI it won't work13:36
rossella_senikanorov_ : it that what you are hitting?13:36
enikanorov_rossella_s: so basically my question was, is it enough to wrap any method having session.begin() with those decorators?13:37
rossella_senikanorov_ :  I don't think so...13:37
rossella_senikanorov_ : but I should check...I wrote that patch long ago13:38
enikanorov_rossella_s: then it's not clear how it can help13:38
*** amotoki has joined #openstack-neutron13:41
openstackgerritJian LI proposed openstack/neutron: Fix extra-dhcp-opt on stateless dhcpv6 subnet  https://review.openstack.org/14745513:42
rossella_senikanorov_ : sorry my memory is a bit blurred ...when I created that patch if i good remember we were not using oslo.db and we had the DBAPI defined in the neutron tree under openstack common. My idea to use that decorator was to put it on top of some method there13:42
enikanorov_rossella_s: yes, for sure, i'd expect that one decorator is enough.13:43
enikanorov_ok, i'll probably have to dig into the code and make some research13:43
rossella_senikanorov_ : sorry but I don't remember the details right now, I will also look into it and try to remember13:44
*** yapeng has joined #openstack-neutron13:44
rossella_senikanorov_ : it's probably worth asking to the oslo people...probably to the authors of the safe_for_db_retry13:44
*** chenleji has quit IRC13:45
*** chenleji has joined #openstack-neutron13:45
*** flaviof has quit IRC13:47
*** trinaths has joined #openstack-neutron13:48
*** yapeng has quit IRC13:48
*** txomon|fon has quit IRC13:53
*** moshele has quit IRC13:53
*** changbl has joined #openstack-neutron13:54
*** Miouge has joined #openstack-neutron13:55
*** moshele has joined #openstack-neutron13:55
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Implements the ProcessMonitor in the l3_agent  https://review.openstack.org/11493113:55
*** anshul has quit IRC13:56
*** moshele has quit IRC13:57
*** moshele has joined #openstack-neutron13:57
*** jistr has quit IRC13:58
openstackgerritLionel Zerbib proposed openstack/neutron-specs: RST for blueprint security-group-brute-force-prevention  https://review.openstack.org/15124713:58
*** trinaths has quit IRC13:59
*** jistr has joined #openstack-neutron14:00
*** trinaths has joined #openstack-neutron14:00
*** rushil has joined #openstack-neutron14:03
*** avozza is now known as zz_avozza14:04
*** txomon|fon has joined #openstack-neutron14:05
trinathsSumitNaiksatam: ping14:07
*** feleouet has joined #openstack-neutron14:07
*** rkukura has joined #openstack-neutron14:08
*** Marga_ has joined #openstack-neutron14:09
*** dims has joined #openstack-neutron14:10
*** mwagner_lap has joined #openstack-neutron14:12
*** claudiub has joined #openstack-neutron14:14
*** dims has quit IRC14:14
*** banix has joined #openstack-neutron14:16
*** dane_leblanc has quit IRC14:18
*** ronis_ has joined #openstack-neutron14:18
*** aleksandr_null has quit IRC14:19
*** aleksandr_null has joined #openstack-neutron14:21
*** zzzeek has joined #openstack-neutron14:22
*** banix has quit IRC14:25
*** Murali has quit IRC14:27
*** banix has joined #openstack-neutron14:27
*** flaviof has joined #openstack-neutron14:27
*** cference has joined #openstack-neutron14:28
*** mriedem has joined #openstack-neutron14:28
mriedemobondarev: have you seen this? https://review.openstack.org/#/c/151184/14:28
rossella_senikanorov_, maybe this can help #link http://lists.openstack.org/pipermail/openstack-dev/2013-July/012296.html14:30
*** aveiga has quit IRC14:31
*** vthapar has quit IRC14:31
*** aveiga has joined #openstack-neutron14:32
*** heyongli has quit IRC14:33
*** karimb has joined #openstack-neutron14:33
*** jasondot_ has joined #openstack-neutron14:34
*** tiswanso has joined #openstack-neutron14:34
*** heyongli has joined #openstack-neutron14:36
*** lpetrut1 has joined #openstack-neutron14:37
*** jlibosva has quit IRC14:37
*** lpetrut has quit IRC14:37
*** jlibosva has joined #openstack-neutron14:38
*** zzzeek has quit IRC14:39
*** cburgess has quit IRC14:39
*** irenab has quit IRC14:40
*** irenab has joined #openstack-neutron14:40
*** tomoe_ has quit IRC14:41
*** cburgess has joined #openstack-neutron14:41
*** dims has joined #openstack-neutron14:41
openstackgerritSergey Belous proposed openstack/neutron: Added check for emptyness where in_ is being used  https://review.openstack.org/15003414:44
*** aveiga has quit IRC14:45
*** jasondot_ has quit IRC14:45
*** pal has quit IRC14:46
*** aveiga has joined #openstack-neutron14:46
*** r-daneel has joined #openstack-neutron14:51
*** mlavalle has joined #openstack-neutron14:55
*** dane_leblanc has joined #openstack-neutron14:56
*** RichardRaseley has joined #openstack-neutron14:57
*** irenab has quit IRC15:01
*** yeungp has joined #openstack-neutron15:02
*** markvoelker_ has quit IRC15:03
*** TrevorV_ has joined #openstack-neutron15:03
*** heyongli has quit IRC15:04
*** gaud has joined #openstack-neutron15:04
*** irenab has joined #openstack-neutron15:04
*** seizadi has joined #openstack-neutron15:05
*** zz_avozza is now known as avozza15:05
*** heyongli has joined #openstack-neutron15:07
*** oreillyd has quit IRC15:08
*** oreillyd has joined #openstack-neutron15:08
*** jistr has quit IRC15:10
*** RichardRaseley has quit IRC15:11
*** jistr has joined #openstack-neutron15:12
*** Marga_ has quit IRC15:12
trinathspc_m: hi, I agree with your comments. Do I need to raise a bug on this.15:12
*** Marga_ has joined #openstack-neutron15:12
*** mriedem has left #openstack-neutron15:13
*** irenab has quit IRC15:13
*** irenab has joined #openstack-neutron15:14
*** jobewan has joined #openstack-neutron15:16
*** numan has quit IRC15:16
*** sergsh is now known as sergsh_away15:17
*** kbringard has joined #openstack-neutron15:18
*** irenab has quit IRC15:19
*** ZZelle has quit IRC15:19
*** jprovazn has joined #openstack-neutron15:20
*** ZZelle has joined #openstack-neutron15:20
*** nmagnezi has quit IRC15:20
*** obondarev_ has joined #openstack-neutron15:21
*** irenab has joined #openstack-neutron15:22
*** avozza is now known as zz_avozza15:24
*** obondarev has quit IRC15:24
*** radez_g0n3 is now known as radez15:26
*** zz_avozza is now known as avozza15:26
*** signed8bit has joined #openstack-neutron15:31
*** achanda has joined #openstack-neutron15:31
*** anish_ has quit IRC15:32
*** yapeng has joined #openstack-neutron15:32
*** TrevorV_ has quit IRC15:35
*** irenab has quit IRC15:35
*** packet has joined #openstack-neutron15:35
*** irenab has joined #openstack-neutron15:36
*** dane_leblanc has quit IRC15:36
*** yapeng has quit IRC15:37
*** tiswanso has quit IRC15:39
*** dane_leblanc has joined #openstack-neutron15:39
*** seizadi has quit IRC15:40
*** achanda has quit IRC15:41
*** jlibosva has quit IRC15:41
rossella_senikanorov_ : so I had some time to look into that issue again, now I remember. The magic is in __get_attr__ as I was saying before15:43
openstackgerritSridhar Gaddam proposed openstack/neutron: Add sanity_check for keepalived  https://review.openstack.org/15128415:43
*** irenab has quit IRC15:43
rossella_senikanorov_, As first step __get_attr__ will load a backend. If you see in the UT the backend is set to be test_api.DBAPI or in Nova the backend is db/sqlalchemy/api.py . After loading the backend, the methods who have a decorator set will be retried in the appropriate manner. This works only for the methods defined in the backend. Hope it's clear now.15:43
*** irenab has joined #openstack-neutron15:44
openstackgerritMichael Smith proposed openstack/neutron: This patch will be used to compare dvr vs full neturon runs  https://review.openstack.org/14189715:45
*** markvoelker has joined #openstack-neutron15:45
*** SridharG has left #openstack-neutron15:45
*** dims has quit IRC15:46
*** banix has quit IRC15:46
*** jlibosva has joined #openstack-neutron15:47
*** dims has joined #openstack-neutron15:48
*** devvesa has quit IRC15:48
*** Networkn3rd has joined #openstack-neutron15:49
*** banix has joined #openstack-neutron15:50
*** irenab has quit IRC15:55
*** marun has joined #openstack-neutron15:55
*** irenab has joined #openstack-neutron15:56
*** salv-orlando has quit IRC15:57
*** salv-orlando has joined #openstack-neutron15:57
*** yamahata has joined #openstack-neutron15:57
*** obondarev_ has quit IRC15:58
*** jlibosva has quit IRC15:58
*** obondarev has joined #openstack-neutron15:59
*** dims has quit IRC15:59
*** prad has joined #openstack-neutron15:59
*** jlibosva has joined #openstack-neutron16:00
*** bharath has joined #openstack-neutron16:03
*** thedodd has joined #openstack-neutron16:04
*** ronis_ has quit IRC16:05
*** irenab has quit IRC16:05
*** carl_baldwin has joined #openstack-neutron16:05
carl_baldwinpc_m: ping16:06
*** padkrish has joined #openstack-neutron16:06
pc_mcarl_baldwin: pong16:06
pc_mcarl_baldwin: Hi!16:06
carl_baldwinThanks for taking the reins this morning.16:07
*** manishg has joined #openstack-neutron16:07
carl_baldwinpc_m: No excuse, I just dropped the ball this morning.16:07
*** a_le has joined #openstack-neutron16:07
*** apuimedo has quit IRC16:07
pc_mcarl_baldwin: np. It was a feeble attempt, but we got through16:07
carl_baldwinI’m reading through the logs.16:08
pc_mcarl_baldwin: Please check the logs. There is an issue with DVR tests failing all over. Something with server startup.16:08
*** krtaylor has quit IRC16:08
*** irenab has joined #openstack-neutron16:09
pc_mcarl_baldwin: Also, I added you to a VPN review, just as an FYI. https://review.openstack.org/15123316:09
carl_baldwinpc_m: Thanks.16:09
pc_mcarl_baldwin: Looks like RouterInfo signature change, broke VPN tests.16:10
*** padkrish has quit IRC16:10
pc_mcarl_baldwin: Would be nice if we can mitigate those types of issues.16:10
*** manishg_ has joined #openstack-neutron16:10
*** zz_jgrimm is now known as jgrimm16:10
*** apuimedo has joined #openstack-neutron16:10
*** devvesa has joined #openstack-neutron16:10
*** avozza is now known as zz_avozza16:10
*** padkrish has joined #openstack-neutron16:11
carl_baldwinpc_m: It is going to take a little bit before a stable interface can be achieved.16:11
carl_baldwinpc_m: I understand though.16:11
pc_mcarl_baldwin: Not sure if there is a good solution, without a lot of effort.16:11
pc_mcarl_baldwin: Just wondering how we can catch that, before commits.16:11
pc_mbefore merge that is.16:11
*** manishg has quit IRC16:12
carl_baldwinIt would be best if there was a testing which could +1/16:12
*** rpothier has joined #openstack-neutron16:12
carl_baldwin-1 on patches that break such things.16:12
carl_baldwinpc_m: ^16:12
*** nyechiel has quit IRC16:12
pc_mcarl_baldwin: Yeah, but I think it went through and was merged. and then, when someone did a VPN commit, tests failed, due to the Neutron change.16:13
pc_mcarl_baldwin: It's gonna be hard to tell if a change effects one of the services repos...16:13
pc_mcarl_baldwin: Unless all Neutron commits run services' tests?16:14
carl_baldwinpc_m: Right, that is how it is now.  My suggestion is that VPN have a test which runs before merge in the Neutron repo.  Either a separate CI or a test in the Jenkins suite.16:14
carl_baldwinpc_m: It is something worth discussing with the Neutron team.16:14
*** obondarev has quit IRC16:15
pc_mcarl_baldwin: So when a neutron commit is done, it runs the VPN tests?16:15
pc_mUTs16:15
*** apuimedo has quit IRC16:15
*** obondarev has joined #openstack-neutron16:15
carl_baldwinpc_m: If I understand you correctly, no.  VPN tests would run on each patch set in neutron is what I’m thinking.16:16
carl_baldwinpc_m: But I’m not sure what you mean by “when a neutron commit is done"16:16
openstackgerritTrinath Somanchi proposed openstack/neutron-fwaas: Move 'def _make_firewall_dict_with_rules' from fwaas_plugin.py to firewall_db.py  https://review.openstack.org/15129816:16
*** pal has joined #openstack-neutron16:16
pc_mcarl_baldwin: So, I think a neutron commit, changed RouterInfo() signature. All the neutron tests were updated, so everything passed.16:17
*** evgenyf has quit IRC16:17
carl_baldwinpc_m: I know how it happened.  I’m suggesting a possible change to mitigate it.16:17
pc_mcarl_baldwin: However, the VPN repo uses RouterInfo() in tests (for good or bad). When those tests run, on any VPN commit, they fail.16:17
*** jkraj has joined #openstack-neutron16:18
ZZelleajo, hi16:18
pc_mcarl_baldwin: Seems like we need Neutron commits to run client (services') tests?16:18
carl_baldwinpc_m:  Yes, I’m suggesting something like that.16:18
carl_baldwinpc_m: I’m not sure what the best way to handle doing that is, though.16:19
pc_mI can't see any other way that is easy...16:19
*** seizadi has joined #openstack-neutron16:19
*** ihrachyshka has quit IRC16:20
carl_baldwinpc_m: Of course, we still get breakage.  However, we’ll get a little more advanced notice and be prepared to merge a corresponding fix in VPN.16:20
pc_mcarl_baldwin: true.16:21
*** krtaylor has joined #openstack-neutron16:21
carl_baldwinpc_m: We can’t have a voting or gating job on neutron because of the whole chicken and egg problem.  However, we can get the commits as close as possible to shorten failure time.16:22
openstackgerritBilly Olsen proposed openstack/neutron: Refactor logging in loop to only log debug messages once  https://review.openstack.org/14978416:22
carl_baldwinpc_m: I was going to write a spec on how to get patches gated together across repositories but I have not finished that spec.16:22
pc_mcarl_baldwin: I think the FW folks had similar concerns with commits that needed to be made in two repos for work.16:23
*** Sukhdev has joined #openstack-neutron16:23
carl_baldwinpc_m: It is a legitimate concern that we need to look into.16:24
pc_mcarl_baldwin: I think LBaaS did it by committing with a commit message indicating that the UTs are broken, until the other repo commit goes in.16:24
pc_mcarl_baldwin: Granted, I think that was for a new change and not refactoring/bugfixing16:24
pc_mcarl_baldwin: Good idea to be bringing it up with community.16:25
pc_mcarl_baldwin: awareness is half the battle.16:25
carl_baldwinpc_m: Yes.16:25
*** zzzeek has joined #openstack-neutron16:25
*** jschwarz has joined #openstack-neutron16:26
*** armax has joined #openstack-neutron16:27
*** armax has quit IRC16:28
*** lazy_prince is now known as killer_prince16:30
*** regXboi has joined #openstack-neutron16:31
*** dims has joined #openstack-neutron16:32
*** itzikb has quit IRC16:32
*** tflynn has joined #openstack-neutron16:32
bharathHi, anyone reproduced this bug https://bugs.launchpad.net/neutron/+bug/1317363 ?16:33
*** dims has quit IRC16:34
*** tflynn has quit IRC16:35
*** seizadi has quit IRC16:35
marunHenryG: I think we have a winner - https://review.openstack.org/#/c/142101/16:35
*** tflynn has joined #openstack-neutron16:35
*** devvesa has quit IRC16:36
openstackgerritTrinath Somanchi proposed openstack/neutron-fwaas: Move '_make_firewall_dict_with_rules' from fwaas_plugin.py to firewall_db.py  https://review.openstack.org/15129816:36
*** vhosakot has joined #openstack-neutron16:36
pc_mcarl_baldwin: Can you review this, when you can: https://review.openstack.org/15099016:36
pc_mcarl_baldwin: It is the vpn repo side of the functional gate setup work16:37
carl_baldwinpc_m: Yes.16:37
pc_mcarl_baldwin: Thanks!16:37
*** dims has joined #openstack-neutron16:38
*** armax has joined #openstack-neutron16:38
vhosakotHi, I observed that the python neutron client is not thread safe, and if I call the "create_port" API twice simultaneously in two threads, I see that only one of them pass.. Sometimes, both pass... Is this the expected behavior ? Can anyone please tell me if they have used the "create_port" API simultaneously in multiple threads ? If yes, how to do it ?16:39
carl_baldwinpc_m: re: https://review.openstack.org/#/c/151233 I gave -1 with one suggestion.  If you don’t want to turn around the patch for it, I can change my mind.16:39
*** Murali has joined #openstack-neutron16:39
carl_baldwinpc_m: I’d even be happy to do the turn-around.  But, then we’ll have to wait for tests.16:40
pc_mcarl_baldwin: looking16:40
*** obondarev has quit IRC16:41
pc_mcarl_baldwin: Actually, that is a great idea! Probably the best way to handle modifying the signature too.16:42
HenryGmarun: will get to it soon16:42
*** obondarev has joined #openstack-neutron16:42
*** kbringard has quit IRC16:42
carl_baldwinpc_m: There is actually one wrinkle.  The tests are across two classes which makes sharing the kwargs dict a little less advantageous.16:43
*** doude has quit IRC16:43
pc_mcarl_baldwin: I guess the big question is whether we should try to fix it ASAP, or get the tests working and do a simple commit for the change to make it less fragile. Thoughts?16:45
carl_baldwinEither way.16:45
carl_baldwinIf you want to get the tests fixed quickly, we can just approve this one now.16:45
pc_mGiven, everything is broken... as separate commit may be better.16:45
pc_mcarl_baldwin: Yeah, I'm thinking we can just follow up with another fix for that.16:46
carl_baldwinpc_m: +2/+A16:47
pc_mthanks!16:47
pc_mI can file a bug and get that change in.16:47
openstackgerritCedric Brandily proposed openstack/neutron: Do not run neutron-ns-metadata-proxy as root on dhcp agent  https://review.openstack.org/14743716:48
*** jaypipes has joined #openstack-neutron16:48
pc_mcarl_baldwin: Appreciate the help on this.  Will ttyl. I have to get some lunch...16:48
*** packet has quit IRC16:49
*** tflynn has quit IRC16:49
carl_baldwinpc_m: later.16:49
*** pc_m has quit IRC16:49
*** tflynn has joined #openstack-neutron16:49
*** david-lyle_afk is now known as david-lyle16:49
*** nlahouti has joined #openstack-neutron16:50
*** vhosakot has quit IRC16:52
*** SridharRamaswamy has joined #openstack-neutron16:53
*** RichardRaseley has joined #openstack-neutron16:55
*** padkrish has quit IRC16:55
*** rwsu-afk is now known as rwsu16:55
*** packet has joined #openstack-neutron16:57
*** dane_leblanc has quit IRC16:57
*** jlanoux has quit IRC16:58
*** kbringard has joined #openstack-neutron16:59
openstackgerritCedric Brandily proposed openstack/neutron: Refactor Pinger class  https://review.openstack.org/15116717:00
*** pask81 has quit IRC17:00
*** padkrish has joined #openstack-neutron17:01
*** rotbeard has quit IRC17:01
*** dane_leblanc has joined #openstack-neutron17:01
*** scheuran has quit IRC17:01
*** mlavalle has quit IRC17:01
*** dsundarraj has joined #openstack-neutron17:02
*** krtaylor has quit IRC17:03
*** markvoelker has quit IRC17:03
*** krtaylor has joined #openstack-neutron17:05
*** tflynn has quit IRC17:07
*** afazekas has quit IRC17:07
*** jkraj has quit IRC17:08
*** coolsvap is now known as coolsvap|afk17:10
*** safchain has quit IRC17:10
*** _cjones_ has joined #openstack-neutron17:10
openstackgerritSukhdev Kapur proposed openstack/neutron: Fixing a log message in Arista L3 Service Plugin  https://review.openstack.org/14797617:11
*** manishg_ has quit IRC17:11
openstackgerritTrinath Somanchi proposed openstack/neutron-fwaas: Freescale FWaaS Plugin  https://review.openstack.org/14888417:13
*** ihrachyshka has joined #openstack-neutron17:16
*** jpich has quit IRC17:17
*** ygbo has quit IRC17:18
*** amotoki has quit IRC17:18
*** heyongli has quit IRC17:18
*** yfauser has joined #openstack-neutron17:19
*** yfauser has left #openstack-neutron17:19
*** manishg has joined #openstack-neutron17:21
*** yapeng has joined #openstack-neutron17:21
*** heyongli has joined #openstack-neutron17:22
*** tflynn has joined #openstack-neutron17:24
*** padkrish has quit IRC17:26
*** yapeng has quit IRC17:27
*** EmilienM is now known as EmilienM|afk17:27
*** suresh12 has joined #openstack-neutron17:28
*** karimb has quit IRC17:28
*** gaud has quit IRC17:29
*** markvoelker has joined #openstack-neutron17:30
*** irenab has quit IRC17:31
*** jistr has quit IRC17:31
*** ronis has joined #openstack-neutron17:32
*** SumitNaiksatam has quit IRC17:32
*** bharath has quit IRC17:32
*** Murali has quit IRC17:33
*** trinaths has left #openstack-neutron17:34
*** afazekas has joined #openstack-neutron17:34
*** moshele has quit IRC17:34
*** matrohon has joined #openstack-neutron17:34
matrohonamuller : hi17:35
amullermatrohon: in meeting17:35
openstackgerritCarl Baldwin proposed openstack/neutron-vpnaas: Handle common boilerplate arguments to RouterInfo  https://review.openstack.org/15132617:35
amullermatrohon: I'll be online for the next few hours17:36
*** padkrish has joined #openstack-neutron17:36
matrohonsafchain, amuller : I'll leave in few minutes :) I just wanted you to give your opinion on  : https://review.openstack.org/#/c/141114/17:37
amullermatrohon: Ahh, I'm pretty much with Mike on this one17:38
amullerBut I need to review it17:38
*** bradjones has quit IRC17:38
*** bradjones has joined #openstack-neutron17:39
matrohonamuller : me either, I'm happy that mike is trying to fix the issue as early as possible. I just think the code needs to be cleaner for maintainability17:40
*** manishg has quit IRC17:40
amullerOK, hopefully we can clean it up then17:40
*** aranjan has joined #openstack-neutron17:41
*** manishg has joined #openstack-neutron17:41
*** bjornar_ has joined #openstack-neutron17:41
carl_baldwinihrachyshka: ping17:41
*** tomoe_ has joined #openstack-neutron17:41
*** bharath has joined #openstack-neutron17:41
ihrachyshkacarl_baldwin, pong17:42
bharathAnyone can reproduce this bug https://bugs.launchpad.net/neutron/+bug/1317363 ?17:42
bharathI am unable to update the gateway ip17:42
carl_baldwinihrachyshka: I’m following up on the L3 meeting this morning.  You mentioned ipam interface maturity.  Would you like to elaborate?17:43
carl_baldwinihrachyshka: I will also take a quick look at your test_l3_agent refactoring series.17:44
ihrachyshkacarl_baldwin, the patch for abstraction has multiple comments, I wonder whether those are worked upon.17:44
ihrachyshkacarl_baldwin, thanks17:44
*** tflynn has quit IRC17:44
openstackgerritRossella Sblendido proposed openstack/neutron: Add devices to update in RPC call security_groups_provider_updated  https://review.openstack.org/14823517:44
*** aranjan has quit IRC17:45
*** manishg has quit IRC17:45
*** tomoe_ has quit IRC17:45
*** matrohon has quit IRC17:46
*** carl_baldwin has quit IRC17:49
openstackgerritMerged openstack/neutron-vpnaas: Fix the neutron-vpnaas unit test failures  https://review.openstack.org/15123317:51
*** krtaylor has quit IRC17:51
openstackgerritSandhya Dasu proposed openstack/neutron: Remove duplicate logging of attribute validation errors  https://review.openstack.org/14985317:51
*** tflynn has joined #openstack-neutron17:53
*** sbfox has joined #openstack-neutron17:53
*** SumitNaiksatam has joined #openstack-neutron17:53
*** jp_at_hp has quit IRC17:54
*** salv-orlando has quit IRC17:54
*** carl_baldwin has joined #openstack-neutron17:55
carl_baldwinihrachyshka: Sorry, my internet connection went down for a minute but I just caught up on eavesdrop.17:56
*** padkrish has quit IRC17:56
carl_baldwinihrachyshka: There has been a rash of new reviews on the ipam interface review in the last couple of days.  I have not had a chance to look at them all yet.17:57
ihrachyshkacarl_baldwin, that's not bad, it's just that the number of comments is around 30, so it's better to have those handled before we're dug deep into them17:58
*** thorst1 has left #openstack-neutron17:58
carl_baldwinihrachyshka: I will get to them.  The review has been up for months.  I think taking a couple of days to respond to a rash of new comments is reasonable.17:59
*** Youcef has joined #openstack-neutron18:00
ihrachyshkathat's fair, I am guilty myself to get to it now only :(18:00
*** padkrish has joined #openstack-neutron18:00
*** reed has joined #openstack-neutron18:00
*** oreillyd has quit IRC18:00
*** radez is now known as radez_g0n318:00
carl_baldwinihrachyshka: All good.  :)18:01
*** anilvenkata has left #openstack-neutron18:01
carl_baldwinihrachyshka: re: split_ra_unit_tests.  It will be difficult to work that in at the moment because there is so much churn on that file right now.18:01
carl_baldwinihrachyshka: I hope to settle down in a week or two and then we can get to that patch series.18:02
*** suresh12 has quit IRC18:02
enikanorov_rossella_s: thanks for the explanation. what is backend in neutron?18:02
*** mpath-rax has quit IRC18:02
*** leenheer has joined #openstack-neutron18:03
*** afazekas_ has joined #openstack-neutron18:03
*** krtaylor has joined #openstack-neutron18:04
*** dsundarraj has quit IRC18:05
*** pal has quit IRC18:05
*** f13o has quit IRC18:06
*** timbyr_ has quit IRC18:06
ihrachyshkacarl_baldwin, ack, I'm not pushing for that one, just had an idea of enhancing coverage for the module and started from tiny preparation steps which ended up with 5 invasive patches :(18:06
*** moha_hunt has joined #openstack-neutron18:06
*** ivar-lazzaro has joined #openstack-neutron18:07
carl_baldwinihrachyshka: Let’s keep them on hold and look at how they fit it after the refactoring.  The refactoring has been splitting UTs out of that massive file.18:07
*** pc_m has joined #openstack-neutron18:07
*** kawa2014 has quit IRC18:07
*** f13o has joined #openstack-neutron18:08
*** timbyr_ has joined #openstack-neutron18:08
carl_baldwinihrachyshka: At a glance, I think you’re on a good track with those patches.  But, it is in direct conflict with other stuff and will be difficult to merge now.18:09
ihrachyshkayeah, I realize that and hence want to sync up with potential reviewers on better time to push for it since I don't want to rebase after rebase :)18:09
ihrachyshkatell me when you think it's a good time, and I'll try to be prepared18:09
*** regXboi has left #openstack-neutron18:11
*** doude has joined #openstack-neutron18:14
*** spandhe has joined #openstack-neutron18:15
*** jschwarz has quit IRC18:15
rossella_senikanorov_ : I think neutron doesn't implement any backend module18:15
*** jlibosva has quit IRC18:15
enikanorov_rossella_s: ok, so it's useless :)18:15
rossella_senikanorov_ yes for neutron yes18:16
rossella_senikanorov_ : we can add a module if needed18:16
enikanorov_yep, i think eventually we'll do it18:16
rossella_senikanorov_ :) gotta go now! bye18:16
enikanorov_rossella_s: good luck!18:17
*** padkrish has quit IRC18:18
*** tflynn has quit IRC18:18
*** amotoki has joined #openstack-neutron18:18
*** padkrish has joined #openstack-neutron18:19
*** s3wong has joined #openstack-neutron18:20
*** jpena has quit IRC18:21
*** coolsvap|afk is now known as coolsvap18:22
*** moshele has joined #openstack-neutron18:23
*** amotoki has quit IRC18:24
*** salv-orlando has joined #openstack-neutron18:25
*** jorgem has joined #openstack-neutron18:26
openstackgerritSridhar Gaddam proposed openstack/neutron: Add sanity_check for keepalived  https://review.openstack.org/15128418:27
*** markmcclain has joined #openstack-neutron18:27
*** RichardRaseley has quit IRC18:27
*** padkrish has quit IRC18:27
*** Sukhdev has quit IRC18:28
*** thedodd has quit IRC18:28
*** achanda has joined #openstack-neutron18:29
*** tflynn has joined #openstack-neutron18:33
ChuckCZZelle: thanks for your help with https://review.openstack.org/#/c/11212918:35
*** thomasem has quit IRC18:35
*** EmilienM|afk is now known as EmilienM18:36
*** bharath has quit IRC18:36
*** doude has quit IRC18:37
*** tflynn has quit IRC18:38
*** RichardR1seley has joined #openstack-neutron18:38
*** harlowja has joined #openstack-neutron18:38
openstackgerritCarl Baldwin proposed openstack/neutron: Move ha router functionality from the agent to ha_router  https://review.openstack.org/15063018:38
*** Sukhdev has joined #openstack-neutron18:40
Sukhdevmestery: Hey Kyle, I had to rebase https://review.openstack.org/#/c/147976/ - can you please put your vote back on this, please?18:42
mesterySukhdev: Looking18:42
*** harlowja_ has joined #openstack-neutron18:42
*** aranjan has joined #openstack-neutron18:42
Sukhdevmestery: Thanks - appreciate it18:43
*** aveiga has quit IRC18:43
*** krtaylor has quit IRC18:43
*** amotoki has joined #openstack-neutron18:45
*** dims has quit IRC18:46
*** harlowja has quit IRC18:46
openstackgerritCarl Baldwin proposed openstack/neutron-vpnaas: Handle common boilerplate arguments to RouterInfo  https://review.openstack.org/15132618:46
pc_mcarl_baldwin: ping18:47
*** aranjan has quit IRC18:47
carl_baldwinpc_m: pong18:47
pc_mPlease see my comments in PS1 of 15132618:47
openstackgerritMoshe Levi proposed openstack/neutron: Thin MLNX ML2 mechanism driver and agent  https://review.openstack.org/14861418:47
*** dims has joined #openstack-neutron18:47
*** fandi has joined #openstack-neutron18:48
carl_baldwinpc_m: I will update.18:49
*** padkrish has joined #openstack-neutron18:49
*** manishg has joined #openstack-neutron18:49
*** jorgem has quit IRC18:50
pc_mcarl_baldwin: THanks for doing this. I was going to do it later, but glad you jumped in (it was your idea anyway).18:50
carl_baldwinpc_m: Sometimes it is just easiest to knock something like this out quickly and be done with it.18:50
*** openstackgerrit has quit IRC18:50
pc_mcarl_baldwin: Yeah, especially when it is fresh in mind (and easy change :)18:51
*** openstackgerrit has joined #openstack-neutron18:51
carl_baldwinpc_m: I’ve updated the patch.18:51
carl_baldwinamuller: ping18:52
*** moshele has quit IRC18:52
pc_mcarl_baldwin: Does the upstream neutron code have kwargs for those two arguments? I had a pull of view and it has them as positional args currently.18:52
*** mwagner_lap has quit IRC18:53
carl_baldwinpc_m: Any arguments can be passed as positional or keyword args anytime.  There is no distinction.18:53
carl_baldwinThe restriction is that positional arguments must be passed first, and then any keyword arguments after tht.18:54
carl_baldwins/tht/that/18:54
*** krtaylor has joined #openstack-neutron18:54
pc_mcarl_baldwin: duh, as long as no default args before them...18:54
*** RichardR1seley has quit IRC18:54
carl_baldwinpc_m: Right.  And all arguments without a default value must be passed.18:54
*** manishg has quit IRC18:55
*** itzikb has joined #openstack-neutron18:55
*** tidwellr has joined #openstack-neutron18:56
*** flwang has quit IRC18:56
*** coolsvap is now known as coolsvap|afk18:56
pc_mdougwig: mestery: carl_baldwin: Please review https://review.openstack.org/#/c/150990/1 so we can move the VPN functional tests to check queue.18:58
mesterypc_m: Looking18:59
openstackgerritNuman Siddique proposed openstack/neutron-vpnaas: Fix the ipsec conn issue when peer addr is fqdn  https://review.openstack.org/14500518:59
*** ZZelle_ has joined #openstack-neutron18:59
openstackgerritMichael Smith proposed openstack/neutron: Debug log msgs for experimental dvr job  https://review.openstack.org/15099818:59
mesterypc_m: Is there precedent for copying devstack-vm-gate.sh like you did in that review?18:59
*** aranjan has joined #openstack-neutron19:00
*** dsneddon is now known as dsneddon_appt19:00
*** thomasem has joined #openstack-neutron19:00
pc_mmestery: Don't know, marun suggested using the Neutron hook files, so I took all the files and copied and tweaked paths and updated copyright notices.19:01
clarkbI would highly recommend aginst not doing that19:01
*** salv-orlando has quit IRC19:01
clarkband instead just run that script from devstack-gate19:01
mesteryclarkb: That was my thinking as well. pc_m, can we just run them instead?19:01
mesteryclarkb: ++19:01
*** thomasem has quit IRC19:01
*** thomasem has joined #openstack-neutron19:02
*** lpetrut1 has quit IRC19:03
*** manishg has joined #openstack-neutron19:04
pc_mmestery: clarkb: not sure I fully understand what you're suggesting (I'm barely figuring out what these all do). Can you elaborate?19:04
*** aepifanov has quit IRC19:04
*** amotoki has quit IRC19:05
pc_mRight now, gate_hook.sh runs gevstack-vm-gate.sh, from within the repo area.19:05
*** kbringard has quit IRC19:06
*** salv-orlando has joined #openstack-neutron19:07
*** ijw has joined #openstack-neutron19:08
clarkbbut as a gate hook you know you are running as part of devstack gate so no reason to copy paste the devstack gate script19:08
*** RichardRaseley has joined #openstack-neutron19:09
*** mlavalle has joined #openstack-neutron19:09
*** achanda has quit IRC19:10
*** yapeng_ has joined #openstack-neutron19:10
*** ivar-lazzaro has quit IRC19:10
pc_mclarkb: OK, so this script exists elsewhere and will be run as part of the devstack gate?19:11
clarkbpc_m: http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree it exists there and will not be run as part of devstack gate if you override the gate_hook but when you override the gate hook you can run it19:12
*** yapeng_ has quit IRC19:15
enikanorov_question to all19:15
pc_mclarkb: Any reason why Neutron has a copy of the script?19:15
enikanorov_if i'm getting19:15
enikanorov_Non-zero exit code (2) from test listing.19:15
enikanorov_error: testr failed (3)19:15
enikanorov_The test run didn't actually run any tests19:15
clarkbpc_m: I have no idea but if it does it should probably be deleted too19:15
enikanorov_what's the best way to troubleshoot that?19:16
clarkbenikanorov_: it should tell you which module failed to import19:16
clarkbthen you can attempt to import that module and see why it broke19:16
enikanorov_clarkb: nope, it doesn't tell anything19:16
enikanorov_i guess it's some import failure, but have no clue19:17
clarkbenikanorov_: can you paste the entire output from tox -epyXX?19:17
clarkbuse paste.openstack.org or similar19:17
enikanorov_clarkb: that was pretty much it ^^^19:17
*** achanda has joined #openstack-neutron19:18
enikanorov_clarkb: tried to reset tox environment, making it to install all dependencies, but getting the same19:18
enikanorov_clarkb: i had to install nose and run UTs with nosetests (which works)19:19
clarkbenikanorov_: try `.tox/pyXX/bin/testr list-tests`19:19
clarkbenikanorov_: nosetests working is probably a side effect of it not properly doing test discovery (eg it probably ignored the tests which failed to import)19:19
enikanorov_clarkb: ok, that cmd actually gives some clue, thanks!19:20
clarkbthis is probably a bug in the testr wrapper that neutron uses if the outputs are different19:20
pc_mclarkb: In https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/requirements.yaml#L22 I see it run devstack-vm-gate.sh19:21
*** jorgem has joined #openstack-neutron19:21
clarkbpc_m: yes devstack-vm-gate.sh should come from the repo I linked19:21
pc_mclarkb: I take it I run it from my gate_hook function.19:21
clarkbpc_m: but it shouldn't run its own copy and if it does that copy should go away19:21
pc_mclarkb: Right. The one in requirements.yaml also runs some integration.sh. Do I need to do that as well?19:22
* pc_m no idea what that does...19:22
clarkbno you run what you need to ru nyour tests19:23
*** mlavalle has quit IRC19:24
*** mlavalle has joined #openstack-neutron19:25
*** dims has quit IRC19:26
*** lpetrut has joined #openstack-neutron19:27
*** reed has quit IRC19:27
*** smillward has joined #openstack-neutron19:27
pc_mclarkb: Thanks. I'll push another change...19:28
amullercarl_baldwin: I have to run an errand or two19:30
amullercarl_baldwin: I'll be back later19:30
*** tflynn has joined #openstack-neutron19:30
carl_baldwinamuller: Okay.  I fixed the functional tests on that ha refactor patch.  I’m hoping it can go in soon.19:30
carl_baldwinamuller: ttyl19:31
amullercarl_baldwin: OK I'll be sure to look when I get back19:31
*** yfauser has joined #openstack-neutron19:31
*** yfauser has left #openstack-neutron19:31
carl_baldwinamuller: Thanks.19:31
*** jaypipes has quit IRC19:31
*** wpward has joined #openstack-neutron19:32
*** aveiga has joined #openstack-neutron19:33
*** popw has quit IRC19:33
*** ronis has quit IRC19:34
*** popw has joined #openstack-neutron19:34
wpwardQuick question for anyone that wants to respond... when doing an attach vnic via rest api, we see that passing an IP of 0.0.0.0 causes openstack to assign the first available IP in the subnet.  Is this expected and by design?19:35
openstackgerritCedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional tests  https://review.openstack.org/14341219:35
*** aix has quit IRC19:35
*** manishg has quit IRC19:35
*** markmcclain has quit IRC19:35
*** markmcclain has joined #openstack-neutron19:36
*** manishg has joined #openstack-neutron19:36
*** dims has joined #openstack-neutron19:36
*** manishg_ has joined #openstack-neutron19:37
*** manishg has quit IRC19:37
*** dutsmoc has quit IRC19:38
*** comstud has joined #openstack-neutron19:38
ZZelle_ajo, hi19:38
*** smillward has quit IRC19:40
*** manishg_ has quit IRC19:41
*** jprovazn has quit IRC19:43
*** enikanorov has joined #openstack-neutron19:47
*** smillward has joined #openstack-neutron19:47
*** ajo has quit IRC19:48
*** ajo_ has joined #openstack-neutron19:48
*** arif-ali_ has joined #openstack-neutron19:49
*** smillward has quit IRC19:50
*** smillward has joined #openstack-neutron19:50
*** ChuckC_ has joined #openstack-neutron19:50
*** ivar-lazzaro has joined #openstack-neutron19:50
*** dsundarraj has joined #openstack-neutron19:51
*** SpamapS_ has joined #openstack-neutron19:51
*** s3wong_ has joined #openstack-neutron19:51
*** dsundarraj has quit IRC19:51
*** dsundarraj has joined #openstack-neutron19:52
*** manishg has joined #openstack-neutron19:54
*** baoli has quit IRC19:54
*** rha_ has joined #openstack-neutron19:54
*** mikal_ has joined #openstack-neutron19:55
*** dims_ has joined #openstack-neutron19:56
*** kickinz1` has joined #openstack-neutron19:56
*** rodrigod` has joined #openstack-neutron19:56
*** arif-ali has quit IRC19:56
*** ChuckC has quit IRC19:56
*** enikanorov_ has quit IRC19:56
*** SpamapS has quit IRC19:56
*** s3wong has quit IRC19:56
*** rodrigods has quit IRC19:56
*** a2hill has quit IRC19:56
*** clarkb has quit IRC19:56
*** kickinz1_mob|off has quit IRC19:56
*** rha has quit IRC19:56
*** mikal has quit IRC19:56
*** txomon|fon has quit IRC19:56
*** arif-ali_ is now known as arif-ali19:56
*** ptoohill has joined #openstack-neutron19:56
*** flwang has joined #openstack-neutron19:56
*** wpward1 has joined #openstack-neutron19:57
*** txomon|fon has joined #openstack-neutron19:57
*** padkrish has quit IRC19:58
*** sbfox has quit IRC19:58
*** manishg has quit IRC19:58
*** padkrish has joined #openstack-neutron19:58
*** wpward has quit IRC19:58
*** rkukura has quit IRC19:58
*** dims has quit IRC19:58
*** manishg has joined #openstack-neutron19:59
*** manishg has quit IRC19:59
*** manishg has joined #openstack-neutron19:59
*** thedodd has joined #openstack-neutron20:00
*** dims_ has quit IRC20:00
*** sbfox has joined #openstack-neutron20:01
*** padkrish has quit IRC20:02
*** zz_avozza is now known as avozza20:02
anteayagus: hello, the nova meeting takes place in 57 minutes, would be awesome to have your attendance if that is possible20:03
*** manishg has quit IRC20:04
*** manishg has joined #openstack-neutron20:04
*** openstackgerrit has quit IRC20:04
*** openstackgerrit has joined #openstack-neutron20:05
*** amotoki has joined #openstack-neutron20:05
*** clarkb has joined #openstack-neutron20:06
*** rkukura has joined #openstack-neutron20:07
*** manishg_ has joined #openstack-neutron20:07
*** _cjones_ has quit IRC20:07
*** manishg has quit IRC20:08
*** amotoki has quit IRC20:10
*** carl_baldwin has quit IRC20:13
ajo_hi ZZelle20:13
ajo_sorry, I'm a bit out of office, and I won't be much around, but please tell me :D20:14
ZZelle_ajo_, is it fine if i rebase https://review.openstack.org/114931? as my change requires a lastest master version20:14
*** tidwellr has quit IRC20:15
ajo_ZZelle, do you mean to rebase on top of 114931 ?20:16
ajo_or whtat do you mean?20:16
ajo_ahhh20:16
ajo_to rebase 114931 on top of last master?20:16
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: Provide hooks for VPNaaS repo functional gate  https://review.openstack.org/15099020:16
ajo_ok, I will20:16
ajo_ZZelle_ give me a sec20:16
*** aranjan has quit IRC20:16
*** s3wong_ has quit IRC20:17
*** aranjan has joined #openstack-neutron20:17
ZZelle_ajo_, just run rebase command in gerrit for 11493120:18
ajo_ajo_, sure, I'm addressing a tiny comment,20:19
ajo_so if we are lucky 114931 will be final, and you don't need to rebase anymore on top of it20:19
*** dsundarraj has quit IRC20:19
*** leenheer has quit IRC20:19
ZZelle_ok20:20
*** ChuckC has joined #openstack-neutron20:20
*** Youcef has quit IRC20:20
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Implements the ProcessMonitor in the l3_agent  https://review.openstack.org/11493120:20
*** itzikb has quit IRC20:20
*** mwagner_lap has joined #openstack-neutron20:21
*** leenheer has joined #openstack-neutron20:22
*** aranjan has quit IRC20:22
*** SpamapS_ is now known as SpamapS20:23
*** ChuckC_ has quit IRC20:23
*** s3wong has joined #openstack-neutron20:24
*** fifieldt has quit IRC20:24
*** SpamapS has quit IRC20:24
*** SpamapS has joined #openstack-neutron20:24
*** itzikb has joined #openstack-neutron20:24
*** fifieldt has joined #openstack-neutron20:25
*** smillward has quit IRC20:25
*** smillward has joined #openstack-neutron20:25
*** itzikb has quit IRC20:26
*** salv-orlando has quit IRC20:27
*** ChuckC has quit IRC20:27
*** wpward1 has left #openstack-neutron20:27
*** yfauser1 has joined #openstack-neutron20:28
*** padkrish has joined #openstack-neutron20:29
*** annegent_ has joined #openstack-neutron20:29
openstackgerritDoug Wiegley proposed openstack/neutron: Mechanims to move extensions and config into service repos  https://review.openstack.org/15137520:29
*** SridharRamaswamy has quit IRC20:30
*** SridharRamaswamy has joined #openstack-neutron20:31
*** leenheer has quit IRC20:33
*** RichardRaseley has quit IRC20:33
*** leenheer has joined #openstack-neutron20:33
*** padkrish has quit IRC20:36
anteayaobondarev: where is your proxy patch? I can't find anything under your name that is open in the neutron repo20:36
*** manishg_ has quit IRC20:37
*** matrohon has joined #openstack-neutron20:37
anteayaobondarev: found it, guess status:open doesn't find wip patches20:37
anteayahttps://review.openstack.org/#/c/150490/20:37
*** manishg has joined #openstack-neutron20:39
*** yfauser1 has left #openstack-neutron20:39
*** Sukhdev has quit IRC20:40
pc_marmax: ping20:40
armaxpc_m: pong, going to luck in a sec20:40
*** Miouge has quit IRC20:40
armax*lunch20:40
pc_marmax: I'm setting up functional gate for vpnaas repo, and was copying from neutron.20:40
*** vishwanathj has joined #openstack-neutron20:40
*** markmcclain has quit IRC20:40
pc_marmax: There is a devstack-vm-gate.sh and functions.sh in neutron. Any reason why these were customizations of the devstack-gate repo files?20:41
pc_marmax: Trying to understand why the overriding... not doing that in vpn repo.20:41
armaxcheck the commit message that introduced those files20:42
pc_marmax: If too long of an answer needed, just let me know later. Lunch is way more important :)20:42
armaxit’s a stop-gap solution20:42
pc_marmax: OK20:42
vishwanathjanybody know when the gate-neutron-fwaas-python27 check will be fixed?20:43
pc_marmax: Do you think this will also be needed for vpnaas repo?20:43
*** banix has quit IRC20:43
armaxit depends how close you are20:43
armaxto completion20:43
armaxI have the respective changes in project-config and devstack-gate that should be merging soon in which case we can drop those files20:44
armaxbut I have no control as to when those merge20:44
clarkbarmax: stop gap for what?20:44
pc_mWell, I've got the functional tests as experimental and hooks are out for review now.  But, there are no API tests currently.20:44
clarkbarmax: that job is literally running inside of devstack-gate...20:44
*** Sukhdev has joined #openstack-neutron20:44
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Implements the ProcessMonitor in the l3_agent  https://review.openstack.org/11493120:45
*** tidwellr has joined #openstack-neutron20:45
ajo_ZZelle_ ^ last patchset, also rebased20:45
ajo_HenryG ^20:45
pc_mclarkb: See commit 0edd89fc20:45
ajo_going to get the kids to sleep20:45
ajo_and ready for my trip tomorrow! ;)20:45
*** banix has joined #openstack-neutron20:46
ZZelle_ajo_, thx20:46
clarkbarmax: pc_m just disable tempest in your job20:46
*** dboik has joined #openstack-neutron20:46
*** amuller has quit IRC20:47
armaxclarkb: please see https://review.openstack.org/#/c/148052/ and https://review.openstack.org/#/c/148641/20:47
armaxonce these two merge we can stop carrying over the files in the contrib directory in the neutron repo20:48
clarkbso I am prett sure you can already do that20:48
clarkbbut I need to go look at devstack20:48
pc_mclarkb: cool. Let me know how I can do that in the VPN repo case.20:49
openstackgerritsajuptpm proposed openstack/neutron: Removed redundant statement from l3agentscheduler  https://review.openstack.org/15138020:49
clarkbset DEVSTACK_GATE_TEMPEST to 020:50
armaxclarkb: by doing that devstack-gate does not pull/configure tempest20:50
clarkbarmax: and we need tempest but don't want to run it because?20:51
clarkbI am all confused20:51
armaxtempest it’s a dependency for the neutron api job, marun should be able to explain more20:51
*** aepifanov has joined #openstack-neutron20:52
clarkbhrm I thought tempest was specifically saying don't do that and use tempest-lib instead20:53
*** britthouser has quit IRC20:54
*** ChuckC_ has joined #openstack-neutron20:54
*** britthouser has joined #openstack-neutron20:54
*** achanda has quit IRC20:55
openstackgerritenikanorov proposed openstack/neutron: Add option to remove networks from dead DHCP agents  https://review.openstack.org/13115020:55
*** armax has quit IRC20:56
pc_mclarkb: As I understand from marun, there is an effort to move tempest tests to in-tree (Neutron).20:58
pc_mmarun: ^^20:58
*** annegent_ has quit IRC20:58
clarkbI have left a comment on the change with suggestions to make this much less confusing20:58
*** yapeng_ has joined #openstack-neutron20:59
*** SridharRamaswamy has quit IRC21:00
pc_mclarkb: Wondering how to handle the VPNaaS repo functional tests.21:00
openstackgerritCedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional tests  https://review.openstack.org/14341221:00
pc_mThe API tests for VPN repo are over in Tempest right now, and I guess we'll move them to the VPN repo, once the tests have been moved from Tempest to Neutron in-tree.21:02
*** carlp has quit IRC21:02
*** claudiub has quit IRC21:02
pc_mclarkb: Suggestions for handling the vpnaas repo functional gate?21:02
openstackgerritCedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional tests  https://review.openstack.org/14341221:02
pc_mclarkb: Should we comment out/disable the API part right now or something else?21:03
clarkbpc_m: so I will admit to not understanding why tempest is a dependency21:03
clarkbI thought that tempest was explicitly not supposed to be a dependency for any of this stuff21:03
*** igordcard has joined #openstack-neutron21:03
*** padkrish has joined #openstack-neutron21:03
*** killer_prince has quit IRC21:03
pc_mmarun: ping21:03
clarkbbut if you are going to use tempest I would get https://review.openstack.org/#/c/148052/ in and configure your job to use that flag21:03
clarkbrather than copying devstack gate into your repo21:03
*** RichardR1seley has joined #openstack-neutron21:04
*** s3wong has quit IRC21:04
*** yapeng_ has quit IRC21:04
*** killer_prince has joined #openstack-neutron21:04
*** killer_prince is now known as lazy_prince21:04
*** SridharRamaswamy has joined #openstack-neutron21:06
*** padkrish has quit IRC21:08
*** annegent_ has joined #openstack-neutron21:08
*** RichardR1seley has quit IRC21:08
openstackgerritCedric Brandily proposed openstack/neutron: Do not run neutron-ns-metadata-proxy as root on dhcp agent  https://review.openstack.org/14743721:08
*** lpetrut has quit IRC21:08
*** dims has joined #openstack-neutron21:09
*** s3wong has joined #openstack-neutron21:10
pc_mclarkb: thanks. I'll talk with Maru and try to sort this out.21:11
*** sripriya has joined #openstack-neutron21:13
*** dims has quit IRC21:14
*** aranjan has joined #openstack-neutron21:17
*** _cjones_ has joined #openstack-neutron21:19
*** achanda has joined #openstack-neutron21:19
*** manishg has quit IRC21:20
*** amotoki has joined #openstack-neutron21:20
*** manishg has joined #openstack-neutron21:20
openstackgerritenikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations  https://review.openstack.org/14926121:21
enikanorovzzzeek: hi21:22
zzzeekenikanorov: hey21:22
*** aranjan has quit IRC21:22
enikanorovzzzeek: so, regarding https://review.openstack.org/#/c/151017/21:22
enikanorovit appears that neutron doesn't yet supports this functionality21:22
enikanorovit has to implement DBAPI backend, does it?21:23
zzzeekym21:23
zzzeekum21:23
*** mwagner_lap has quit IRC21:24
*** amotoki has quit IRC21:25
zzzeekenikanorov: im not sure if thats the case21:27
*** bjornar_ has quit IRC21:27
*** armax has joined #openstack-neutron21:27
zzzeekenikanorov: DBAPI has self-based methods and in the decorator, that “self” would be args[0] - I dont see anything accessed from args[0]21:28
enikanorovzzzeek: i mean, it clearly not enough to wrap amy method i want in 'retry_on_request' wrapper21:29
enikanorovthe method has to make calls to DBAPI from oslo_db i guess21:29
*** salv-orlando has joined #openstack-neutron21:30
anteayagus: and the nova meeting never happened21:30
zzzeekenikanorov: not sure if im missing something, looking at api.py -> wrap_db_retry(), I dont see that anything is happening here specific to api.DBAPI.  the DBDeadlock and DBConnectionError are oslo.db errors which neutron is definitely getting because neurtron uses oslo.db’s create_engine21:31
*** rodrigod` is now known as rodrigods21:31
*** rodrigods has quit IRC21:31
*** rodrigods has joined #openstack-neutron21:31
zzzeekenikanorov: projects currently get that from EngineFacade, every project is using that21:31
enikanorovzzzeek: my question basically is how we get to 'wrap_db_retry'21:31
*** radez_g0n3 is now known as radez21:32
zzzeekenikanorov: I would think you could use it directly.  there’s no need to pull it through the DBAPI.__getattr__ scheme there21:33
enikanorovzzzeek: well, looking at the code i think wrap_db_retry is not for direct use due to its big list of parameters21:33
zzzeekenikanorov: which means the part above where you added the “f.enable_retry” thing, yeah you’d not use that21:34
*** tiswanso has joined #openstack-neutron21:34
zzzeekenikanorov: i dont see an underscore preceding it21:34
zzzeekenikanorov: and I hate the DBAPI class21:34
zzzeekenikanorov: I’d favor adding more to this so that it is more friendly as a standalone decorator21:35
zzzeekenikanorov: e.g. default of False for the various flags21:35
enikanorovyep, it's not a decorator right now21:35
zzzeekenikanorov: it is a decorator21:35
enikanorovbut it's usage is quite convluted21:35
*** carlp has joined #openstack-neutron21:35
zzzeekit has a __call__ method that accepts a function21:35
enikanorovyep, but with such list of iit parameters...21:36
zzzeekenikanorov: it just needs defaults for all the params in __init__21:36
enikanorov*init21:36
zzzeekenikanorov: add them in, I’ll +2 it21:36
zzzeekenikanorov: i hate the DBAPI part below21:36
enikanorovok, i got your point21:36
zzzeekenikanorov: that’s the convoluted part :)21:36
zzzeekenikanorov: what i can see about the DBAPI class is that it allows these defaults to be set up in one place21:37
*** jorgem has quit IRC21:37
zzzeekenikanorov: but at the same time, you could just have a module-level instance of wrap_db_retry() with those defaults that you want21:37
*** aranjan has joined #openstack-neutron21:37
enikanorovzzzeek: the only usage of wrap_db_retry made me think that it's for internal use21:38
zzzeekenikanorov: whoever wrote it probably had that in mind :).   but whoever that person was, overused classes like mad :).   a handy decorator that doesnt need framework behind it should be available by itself21:38
*** dims has joined #openstack-neutron21:39
enikanorovagree21:40
*** dims has quit IRC21:40
*** flwang has quit IRC21:41
*** flwang has joined #openstack-neutron21:41
openstackgerritMerged openstack/neutron: Fixing a log message in Arista L3 Service Plugin  https://review.openstack.org/14797621:41
*** baoli has joined #openstack-neutron21:42
openstackgerritMerged openstack/neutron: Default security group table  https://review.openstack.org/14210121:43
*** carl_baldwin has joined #openstack-neutron21:46
*** afazekas_ has quit IRC21:46
*** moha_hunt has quit IRC21:49
openstackgerritMerged openstack/neutron: Do not check twice IP allocations for auto-address subnets  https://review.openstack.org/14976721:49
*** Marga_ has quit IRC21:50
*** Marga_ has joined #openstack-neutron21:50
*** leenheer has quit IRC21:51
*** ajo_ has quit IRC21:52
*** baoli has quit IRC21:55
*** Marga_ has quit IRC21:55
*** armax has quit IRC21:55
*** baoli has joined #openstack-neutron21:56
*** cference has quit IRC21:56
*** RichardRaseley has joined #openstack-neutron21:56
*** techcet has quit IRC21:59
*** salv-orlando has quit IRC22:00
*** radez is now known as radez_g0n322:00
*** aranjan has quit IRC22:01
*** aranjan_ has joined #openstack-neutron22:01
openstackgerritManish Godara proposed openstack/neutron: Add filter for provider network attributes in ML2  https://review.openstack.org/12491722:02
*** jamielennox|away is now known as jamielennox22:03
*** achanda has quit IRC22:03
*** ajo has joined #openstack-neutron22:04
*** ajo has quit IRC22:04
jogohttps://bugs.launchpad.net/neutron/+bug/135705522:07
jogotop gate bug22:07
jogoonly happens on neutron jobs22:07
jogomestery carl_baldwin enikanorov: ^22:07
mesteryjogo: Looking22:08
enikanorovjogo: thanks for letting us know22:08
*** chlong has quit IRC22:08
mesteryenikanorov: Reading Salv's comments from last October ...22:09
*** flaviof has quit IRC22:10
openstackgerritCarl Baldwin proposed openstack/neutron: WIP Move Floating IP processing to Router classes  https://review.openstack.org/14286322:11
*** claudiub has joined #openstack-neutron22:12
jogolooks like this is an old bug that came back22:13
mesteryjogo: Yes, salv's analysis was it was a tempest issue and not a neutron issue last October22:13
*** achanda has joined #openstack-neutron22:14
jogomestery: do you think that is still valid?22:14
*** achanda has quit IRC22:14
mesteryjogo: I don't have enough data yet, just reading back through the ntoes in the bug.22:14
jogomestery: thanks22:15
jogomtreinish: ^ may be a tempest bug22:15
*** pradk has joined #openstack-neutron22:15
*** aranjan_ has quit IRC22:16
*** aranjan has joined #openstack-neutron22:16
*** tiswanso has quit IRC22:17
*** achanda has joined #openstack-neutron22:17
openstackgerritCarl Baldwin proposed openstack/neutron: WIP Move Floating IP processing to Router classes  https://review.openstack.org/14286322:20
openstackgerritCarl Baldwin proposed openstack/neutron: Encapsulate DVR Fip namespace  https://review.openstack.org/14556522:20
openstackgerritCarl Baldwin proposed openstack/neutron: Move DVR floating ip methods to dvr_router  https://review.openstack.org/15062822:20
*** thomasem has quit IRC22:22
*** r-daneel has quit IRC22:22
*** lori is now known as lori|away22:23
*** s3wong has quit IRC22:23
*** padkrish has joined #openstack-neutron22:23
*** armax has joined #openstack-neutron22:25
*** techcet has joined #openstack-neutron22:25
mtreinishjogo: do I need to do anything? Or is someone else diving into it?22:28
*** ihrachyshka has quit IRC22:28
*** jorgem has joined #openstack-neutron22:29
jogomtreinish: mestery is diving in a bit on the neutron side, but last time this was triaged as a tempest bug22:31
jogoand I am not diving in22:31
mesteryjogo: I see salv removed neutron as an affected project last time as well22:32
mesteryjogo: I'll sync with salv on this one as well and see if anything jogs his memory22:32
*** leenheer has joined #openstack-neutron22:32
jogomestery: yeah I want to rule neutron out again on this. and hopefully mtreinish can look at the tempest side22:33
enikanorovjogo: i see most failures come from rally job, i guess it's some bug there. (I don't mean that there is no issue in neutron)22:33
mtreinishjogo: well, right now it'll be tough being in mid-flight. Probably tomorrow I can look at it22:34
*** matrohon has quit IRC22:34
jogoenikanorov: good poiint about rally22:35
jogobut there is even more in dsvm-neutron-dvr22:35
*** aranjan has quit IRC22:35
enikanorovdvr job is broken at this moment afaik, so it might be expected that some test fails to cleanup ports22:35
mesteryarmax: ^^^22:36
*** tflynn_ has joined #openstack-neutron22:36
*** rpothier has quit IRC22:36
armaxit got healed today afaik22:37
*** banix has quit IRC22:37
*** pc_m has quit IRC22:38
armaxit was broken until this morning22:39
*** tflynn has quit IRC22:39
*** annegent_ has quit IRC22:42
*** cference has joined #openstack-neutron22:44
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Basic subnetpool CRUD. Do not merge  https://review.openstack.org/14869822:44
*** Sukhdev has quit IRC22:44
*** aranjan has joined #openstack-neutron22:45
*** padkrish has quit IRC22:46
*** markvoelker has quit IRC22:46
*** Networkn3rd has quit IRC22:46
*** jorgem1 has joined #openstack-neutron22:46
jogoahhh22:46
*** padkrish has joined #openstack-neutron22:46
jogobut we were seeing these failures in the gate22:47
*** bjornar_ has joined #openstack-neutron22:47
*** markvoelker_ has joined #openstack-neutron22:47
*** avozza is now known as zz_avozza22:47
*** jorgem has quit IRC22:48
*** yapeng_ has joined #openstack-neutron22:48
*** crc32 has joined #openstack-neutron22:51
*** padkrish has quit IRC22:51
*** openstackgerrit has quit IRC22:51
*** sbfox has quit IRC22:51
*** tellesnobrega_ has joined #openstack-neutron22:51
*** openstackgerrit has joined #openstack-neutron22:52
*** yapeng_ has quit IRC22:53
openstackgerritenikanorov proposed openstack/neutron: Log tenant ports if subnet could not be deleted  https://review.openstack.org/15141522:54
*** aranjan has quit IRC22:54
*** jckasper has quit IRC22:54
*** smillward has quit IRC22:57
*** sballe__ has joined #openstack-neutron22:57
*** Sukhdev has joined #openstack-neutron22:59
*** mlavalle has quit IRC23:00
*** rushil has quit IRC23:00
jogoohh rally gate23:01
*** yamahata has quit IRC23:01
*** jckasper has joined #openstack-neutron23:02
*** tflynn_ has quit IRC23:03
openstackgerritChuck Carlino proposed openstack/neutron: Allow port mac_address to be modified  https://review.openstack.org/11212923:03
*** tflynn has joined #openstack-neutron23:03
*** zz_avozza is now known as avozza23:03
openstackgerritDoug Wiegley proposed openstack/neutron: Mechanims to move extensions and config into service repos  https://review.openstack.org/15137523:06
*** padkrish has joined #openstack-neutron23:08
*** jorgem1 is now known as jorgem23:08
*** cference has quit IRC23:09
*** amotoki has joined #openstack-neutron23:09
*** salv-orlando has joined #openstack-neutron23:12
*** jckasper has quit IRC23:13
*** jckasper has joined #openstack-neutron23:13
*** amotoki has quit IRC23:14
*** gildub has joined #openstack-neutron23:15
*** salv-orlando has quit IRC23:17
openstackgerritCedric Brandily proposed openstack/neutron: Do not run neutron-ns-metadata-proxy as root on dhcp agent  https://review.openstack.org/14743723:20
*** dims has joined #openstack-neutron23:22
openstackgerritItsuro Oda proposed openstack/neutron: Metaplugin decomposition  https://review.openstack.org/15146423:23
*** crc32 has quit IRC23:24
*** dims has quit IRC23:25
*** dims has joined #openstack-neutron23:25
*** bjornar_ has quit IRC23:25
*** crc32 has joined #openstack-neutron23:27
*** salv-orlando has joined #openstack-neutron23:28
*** aepifanov has quit IRC23:28
*** tiswanso has joined #openstack-neutron23:28
*** pradk has quit IRC23:28
*** imcsk8 has quit IRC23:32
*** imcsk8 has joined #openstack-neutron23:32
*** tiswanso has quit IRC23:32
*** aranjan has joined #openstack-neutron23:34
*** dboik has quit IRC23:35
*** thomasem has joined #openstack-neutron23:36
*** watanabe_isao has joined #openstack-neutron23:36
openstackgerritItsuro Oda proposed openstack/neutron: Metaplugin decomposition  https://review.openstack.org/15146423:36
*** prad has quit IRC23:37
*** yfauser has joined #openstack-neutron23:38
*** yfauser has left #openstack-neutron23:38
*** aranjan has quit IRC23:39
*** baoli has quit IRC23:41
*** dboik has joined #openstack-neutron23:41
*** thedodd has quit IRC23:41
*** dims has quit IRC23:41
*** rushil has joined #openstack-neutron23:42
*** dims has joined #openstack-neutron23:42
*** ZZelle_ has quit IRC23:46
watanabe_isaoGood morning to ALL.:)23:48
*** seizadi has joined #openstack-neutron23:50
*** baoli has joined #openstack-neutron23:51
*** seizadi1 has joined #openstack-neutron23:53
*** chlong has joined #openstack-neutron23:53
*** seizadi has quit IRC23:54
*** dane_leblanc has quit IRC23:55
*** salv-orlando has quit IRC23:56
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Basic subnetpool CRUD. Do not merge  https://review.openstack.org/14869823:56
*** baoli has quit IRC23:57
*** salv-orlando has joined #openstack-neutron23:57
*** manishg has quit IRC23:58
*** manishg has joined #openstack-neutron23:58
*** jorgem has quit IRC23:59

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