Wednesday, 2015-03-11

*** gildub has joined #openstack-neutron00:00
*** r-daneel has quit IRC00:01
*** zzzeek has left #openstack-neutron00:02
*** SridharRamaswam1 has quit IRC00:05
*** kbringard has joined #openstack-neutron00:07
*** kbringard has quit IRC00:07
*** tsufiev_ has quit IRC00:08
*** marun has quit IRC00:09
*** hichihara has joined #openstack-neutron00:10
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_l2population: Reduce code duplication in migration tests  https://review.openstack.org/15818400:11
*** tsufiev_ has joined #openstack-neutron00:14
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_l2population: Reduce code duplication in migration tests  https://review.openstack.org/15818400:14
openstackgerritrcurran proposed openstack/neutron: ML2 cisco_nexus MD: sync config and models with vendor repo  https://review.openstack.org/16304000:14
shivharisyamamoto: hi00:15
yamamotoshivharis: hi00:15
*** nlahouti has joined #openstack-neutron00:15
shivharisthanks for the review of https://review.openstack.org/#/c/152383/00:15
*** tsufiev_ has quit IRC00:17
shivharisthe code in the shim of plugin usually is initiated by the ml2 driver using the config of L3 in its config file00:18
shivharisthe code should not be executed so i ended up printing out an error message. does that satisfy your concern?00:18
yamamotoshivharis: no problem00:18
yamamotoshivharis: why not remove?00:18
*** achanda has quit IRC00:18
*** achanda has joined #openstack-neutron00:18
shivharisyamamoto: because there may be a old configuration sitting around00:18
shivharisyamamoto: i could potentially remove the code,  but for the above concern00:19
yamamotoshivharis: you want to keep old config work?00:20
*** dims_ has joined #openstack-neutron00:20
*** bandarji has joined #openstack-neutron00:20
shivharisyamamoto: no, starting kilo i really want folks to move forward to the stackforge way00:21
*** itzikb has quit IRC00:22
*** xgerman_ has quit IRC00:22
yamamotoshivharis: then i guess removal is ok as it would cause old config fail loudly enough00:22
*** dims has quit IRC00:22
*** dims_ has quit IRC00:22
shivharisyamamoto: i am only providing some handholding, to avoid loud failure00:23
yamamotoshivharis: but i have no strong opinion on that as far as you make it clear that the class is not expected to be used00:23
*** dims has joined #openstack-neutron00:23
*** achanda has quit IRC00:23
shivharisyamamotoso you are ok with this scheme i have00:24
shivharisyamamoto: you are ok with this scheme i have (typo before)00:24
yamamotoshivharis: yes00:25
shivharisyamamoto: ok, thanks.00:25
*** ivar-lazzaro has quit IRC00:25
*** tsufiev_ has joined #openstack-neutron00:25
*** nlahouti has quit IRC00:25
*** mrsmith has quit IRC00:25
yamamotoshivharis: i made a comment on the new ps00:27
*** tflynn_ has joined #openstack-neutron00:30
*** claudiub has quit IRC00:30
*** tflynn has quit IRC00:33
*** tflynn_ has quit IRC00:33
*** david-lyle_afk has quit IRC00:34
shivharisyamamoto: thanks, got diverted.00:34
*** Marga_ has quit IRC00:40
shivharishichihara: hi00:43
*** shwetaap has joined #openstack-neutron00:44
*** yamamoto has quit IRC00:47
*** dboik has joined #openstack-neutron00:47
hichiharashivharis: hello. I got it with the above log.00:48
shivharishichihara: i also responded in the patch-set, more verbose00:49
openstackgerritSalvatore Orlando proposed openstack/neutron: Add README and requirements.txt for VMware plugins  https://review.openstack.org/16324400:50
*** tiswanso has joined #openstack-neutron00:50
hichiharashivharis: I also read it, no problem.00:51
*** topshare has joined #openstack-neutron00:51
*** yuanying has joined #openstack-neutron00:51
shivharishichihara: thanks.00:51
*** dboik_ has joined #openstack-neutron00:52
*** tiswanso has quit IRC00:52
*** tiswanso has joined #openstack-neutron00:53
*** dboik has quit IRC00:55
*** fredlhsu has quit IRC00:55
*** zzzeek has joined #openstack-neutron00:58
*** zzzeek has quit IRC00:59
*** nati_ueno has quit IRC01:00
*** nati_ueno has joined #openstack-neutron01:00
*** itzikb has joined #openstack-neutron01:00
*** zzzeek has joined #openstack-neutron01:00
*** yuanying has quit IRC01:02
*** sthillma has joined #openstack-neutron01:02
*** vmtrooper has joined #openstack-neutron01:05
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238301:06
*** ivar-lazzaro has joined #openstack-neutron01:09
*** vivek-eb_ has quit IRC01:10
*** ChuckC has quit IRC01:10
*** vmtrooper has quit IRC01:11
*** _cjones_ has quit IRC01:11
*** rook has joined #openstack-neutron01:12
*** liusheng has quit IRC01:12
*** rook is now known as Guest9091601:12
*** liusheng has joined #openstack-neutron01:12
openstackgerritJoeJiang proposed openstack/python-neutronclient: Fixes Bug1411115  https://review.openstack.org/16258201:14
*** yamamoto has joined #openstack-neutron01:15
*** topshare has quit IRC01:15
*** sripriya has joined #openstack-neutron01:18
*** nati_ueno has quit IRC01:19
openstackgerritRob Pothier proposed openstack/neutron: Add MTU selection to ML2  https://review.openstack.org/15631801:19
*** chenleji has joined #openstack-neutron01:20
*** crc32 has joined #openstack-neutron01:20
*** crc32 has quit IRC01:20
*** david-lyle_afk has joined #openstack-neutron01:21
*** crc32 has joined #openstack-neutron01:22
*** ivar-lazzaro has quit IRC01:25
*** sbfox has joined #openstack-neutron01:26
*** topshare has joined #openstack-neutron01:27
*** dims has quit IRC01:28
*** topshare has quit IRC01:28
*** itzikb has quit IRC01:28
*** dims has joined #openstack-neutron01:29
*** salv-orlando has quit IRC01:30
*** devlaps has joined #openstack-neutron01:31
*** topshare has joined #openstack-neutron01:33
*** dims has quit IRC01:34
*** banix has quit IRC01:36
*** vivek-ebay has joined #openstack-neutron01:36
*** pal has joined #openstack-neutron01:36
*** oda-g has quit IRC01:37
*** gongysh_ has quit IRC01:38
*** shivharis has quit IRC01:38
*** oda-g has joined #openstack-neutron01:38
*** SumitNaiksatam has quit IRC01:39
*** dims has joined #openstack-neutron01:40
yamamotoarmax: ping01:41
*** stanzgy has joined #openstack-neutron01:41
*** stanzgy_ has joined #openstack-neutron01:41
*** ChuckC has joined #openstack-neutron01:42
*** popw has joined #openstack-neutron01:44
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: contribute.rst: Use consistent tags  https://review.openstack.org/16325601:45
*** ChuckC has quit IRC01:46
*** sthillma has quit IRC01:46
*** ChuckC has joined #openstack-neutron01:47
*** gongysh has joined #openstack-neutron01:49
*** devlaps has quit IRC01:51
*** Guest90916 has quit IRC01:51
*** tflynn has joined #openstack-neutron01:51
*** rwsu is now known as rwsu-afk01:52
*** dims_ has joined #openstack-neutron01:55
*** diegows has quit IRC01:55
*** changbl has joined #openstack-neutron01:56
*** dims has quit IRC01:58
*** VijayB has quit IRC02:00
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_l2population: Reduce code duplication in migration tests  https://review.openstack.org/15818402:01
*** vivek-eb_ has joined #openstack-neutron02:05
*** vivek-ebay has quit IRC02:08
openstackgerritMerged openstack/neutron-specs: MTU selection spec update bridge_mappings new format  https://review.openstack.org/15914602:08
*** harlowja_ is now known as harlowja_away02:09
*** tflynn has quit IRC02:10
*** tflynn has joined #openstack-neutron02:10
*** s3wong has quit IRC02:12
*** pal has quit IRC02:12
*** pritesh has joined #openstack-neutron02:12
*** yamahata has quit IRC02:13
*** tflynn has quit IRC02:18
*** sbfox has quit IRC02:21
*** vishwana_ has quit IRC02:24
*** sripriya has quit IRC02:27
*** coolsvap has joined #openstack-neutron02:29
*** erkules_ has joined #openstack-neutron02:29
*** zzzeek has quit IRC02:30
*** salv-orlando has joined #openstack-neutron02:31
*** erkules has quit IRC02:32
*** shivharis has joined #openstack-neutron02:32
shivharishichihara: hi02:32
*** bandarji has quit IRC02:34
openstackgerritAssaf Muller proposed openstack/neutron: Introduce RPC tests  https://review.openstack.org/16281102:34
openstackgerritAssaf Muller proposed openstack/neutron: Introduce RPC tests  https://review.openstack.org/16281102:35
*** david-lyle_afk has quit IRC02:36
*** gongysh has quit IRC02:36
openstackgerritAssaf Muller proposed openstack/neutron: Add RPC test for DVR  https://review.openstack.org/16320102:37
*** salv-orlando has quit IRC02:38
openstackgerritAssaf Muller proposed openstack/neutron: Add RPC test for metadata  https://review.openstack.org/16320402:39
*** yog_ has joined #openstack-neutron02:39
hichiharashivharis: hi02:40
shivharishichihara: from what i understand, the config registration/options are supposed to be left behind02:40
*** ruijing has quit IRC02:42
hichiharashivharis: I don't know what dose it need for... But you can disregard my comment.02:43
*** killer_prince is now known as lazy_prince02:43
*** gongysh has joined #openstack-neutron02:44
shivharishichirara: ok02:44
shivharishichihara: thanks.02:44
*** shwetaap has quit IRC02:48
openstackgerritAssaf Muller proposed openstack/neutron: ML2: Change port status only when it's bound to the host  https://review.openstack.org/16317802:49
*** jckasper has joined #openstack-neutron02:51
*** vmtrooper has joined #openstack-neutron02:54
*** carl_baldwin has joined #openstack-neutron02:56
*** salv-orlando has joined #openstack-neutron02:57
*** achanda has joined #openstack-neutron02:58
*** vmtrooper has quit IRC02:59
*** banix has joined #openstack-neutron03:01
*** dims_ has quit IRC03:02
*** gongysh has quit IRC03:04
*** shwetaap has joined #openstack-neutron03:06
*** tflynn has joined #openstack-neutron03:07
*** armax has quit IRC03:09
*** salv-orlando has quit IRC03:10
*** achanda has quit IRC03:10
*** zhhuabj has quit IRC03:11
*** chandankumar has joined #openstack-neutron03:12
*** anish__ has joined #openstack-neutron03:13
*** jckasper has quit IRC03:15
*** coolsvap is now known as coolsvap|afk03:15
*** tflynn has quit IRC03:16
*** ijw has quit IRC03:17
*** armax has joined #openstack-neutron03:17
*** david-lyle_afk has joined #openstack-neutron03:17
*** sballe__ has quit IRC03:18
*** tflynn has joined #openstack-neutron03:19
*** armax has quit IRC03:20
openstackgerritMerged openstack/neutron: ofagent: Have a thin driver module  https://review.openstack.org/16255503:21
*** shivharis has quit IRC03:21
*** coolsvap|afk is now known as coolsvap03:22
*** tflynn has quit IRC03:23
*** tflynn has joined #openstack-neutron03:23
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ofagent: Update after networking-ofagent release  https://review.openstack.org/16327303:26
openstackgerritMerged openstack/neutron-fwaas: Add the missing parameter agent_mode to the FWaaSDriverBase class methods  https://review.openstack.org/15393003:27
openstackgerritMerged openstack/neutron-vpnaas: VPNaaS: Enable StrongSwan in gate hook  https://review.openstack.org/16171403:27
openstackgerritMerged openstack/neutron-vpnaas: Updating alembic HEAD file according to the current code.  https://review.openstack.org/16259503:27
*** zhhuabj has joined #openstack-neutron03:27
*** sbfox has joined #openstack-neutron03:29
*** armax has joined #openstack-neutron03:35
*** lazy_prince is now known as killer_prince03:36
openstackgerritpritesh proposed openstack/python-neutronclient: Adding VLAN Transparency support to neutronclient  https://review.openstack.org/16204003:37
*** armax has quit IRC03:40
*** jgrimm is now known as zz_jgrimm03:40
*** Akanksha08 has quit IRC03:40
*** yuanying has joined #openstack-neutron03:41
*** zz_jgrimm is now known as jgrimm03:47
*** tiswanso has quit IRC03:49
*** chenleji has quit IRC03:53
*** sbfox has quit IRC04:03
*** dims has joined #openstack-neutron04:03
*** Marga_ has joined #openstack-neutron04:04
*** Marga_ has quit IRC04:04
*** Marga_ has joined #openstack-neutron04:05
*** sthillma has joined #openstack-neutron04:06
*** sthillma has quit IRC04:06
*** achanda has joined #openstack-neutron04:07
*** fredlhsu has joined #openstack-neutron04:08
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238304:09
*** dims has quit IRC04:10
*** fredlhsu has quit IRC04:10
*** vivek-eb_ has quit IRC04:15
*** yatin has joined #openstack-neutron04:16
openstackgerritHong Hui Xiao proposed openstack/neutron: Always run dnsmasq as root  https://review.openstack.org/16156704:19
*** yalie has quit IRC04:31
*** absubram has joined #openstack-neutron04:35
*** vthapar has joined #openstack-neutron04:36
*** amotoki has joined #openstack-neutron04:36
*** gongysh has joined #openstack-neutron04:37
*** crc32 has quit IRC04:39
*** dboik_ has quit IRC04:42
openstackgerritAdolfo Duarte proposed openstack/neutron: Functional Test for DVR fpr/rfp interface MTU set  https://review.openstack.org/16281204:43
*** dboik has joined #openstack-neutron04:43
*** vmtrooper has joined #openstack-neutron04:43
*** sbfox has joined #openstack-neutron04:45
*** banix has quit IRC04:48
*** vmtrooper has quit IRC04:48
openstackgerritMerged openstack/neutron: FIP debug messages  https://review.openstack.org/15851304:52
*** nlahouti has joined #openstack-neutron04:53
*** pritesh has quit IRC04:53
*** trinaths has joined #openstack-neutron04:56
*** salv-orlando has joined #openstack-neutron04:57
*** anilvenkata has joined #openstack-neutron04:58
*** yushiro has joined #openstack-neutron05:00
*** carl_baldwin has quit IRC05:01
openstackgerritAkihiro Motoki proposed openstack/python-neutronclient: exec permission to port_test_hook.sh  https://review.openstack.org/16328905:01
*** nlahouti has quit IRC05:04
*** nlahouti has joined #openstack-neutron05:05
*** salv-orlando has quit IRC05:10
*** yushiro has quit IRC05:10
*** salv-orlando has joined #openstack-neutron05:12
*** yfried|afk is now known as yfried_05:14
*** salv-orlando has quit IRC05:16
*** tflynn has quit IRC05:16
*** chandankumar has quit IRC05:17
*** dboik has quit IRC05:20
*** dboik has joined #openstack-neutron05:20
*** shwetaap has quit IRC05:21
*** killer_prince is now known as lazy_prince05:28
*** yfried_ is now known as yfried|afk05:28
*** tflynn has joined #openstack-neutron05:29
*** salv-orlando has joined #openstack-neutron05:29
*** achanda has quit IRC05:31
*** yfried|afk is now known as yfried_05:31
*** salv-orlando has quit IRC05:34
*** achanda has joined #openstack-neutron05:34
*** salv-orlando has joined #openstack-neutron05:35
*** salv-orlando has quit IRC05:35
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ovs_lib: Add a debug-level log to record actual flow strings  https://review.openstack.org/16329305:37
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ovs_lib: Add a debug-level log to record actual flow strings  https://review.openstack.org/16329305:39
*** lazy_prince has quit IRC05:39
*** salv-orlando has joined #openstack-neutron05:41
*** killer_prince has joined #openstack-neutron05:42
*** numan has joined #openstack-neutron05:42
*** killer_prince is now known as lazy_prince05:42
*** salv-orlando has quit IRC05:45
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856005:46
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284005:46
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439105:46
*** achanda has quit IRC05:48
*** vishwanathj has joined #openstack-neutron05:50
*** achanda has joined #openstack-neutron05:52
*** yshafrir has joined #openstack-neutron05:52
gongyshamotoki: hi05:53
amotokigongysh: hi05:54
amotokigongysh: i need to check your code.05:54
gongyshsure05:57
*** achanda has quit IRC05:58
*** irenab has joined #openstack-neutron05:59
*** achanda has joined #openstack-neutron06:01
*** dboik has quit IRC06:01
*** yfried_ is now known as yfried|afk06:01
*** jyuso1 has quit IRC06:01
*** ajo has joined #openstack-neutron06:03
*** salv-orlando has joined #openstack-neutron06:04
*** viveknarasimhan has joined #openstack-neutron06:04
*** yfried|afk is now known as yfried_06:06
*** Murali has joined #openstack-neutron06:08
*** salv-orlando has quit IRC06:08
*** pradipta has joined #openstack-neutron06:09
*** lionelz has quit IRC06:12
*** Sukhdev has joined #openstack-neutron06:12
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/16330206:14
*** achanda has quit IRC06:14
*** dmellado has quit IRC06:15
*** salv-orlando has joined #openstack-neutron06:15
*** nlahouti has quit IRC06:18
*** dmellado has joined #openstack-neutron06:19
*** salv-orlando has quit IRC06:20
*** salv-orlando has joined #openstack-neutron06:21
*** Murali_ has joined #openstack-neutron06:22
*** Murali has quit IRC06:23
*** Murali_ is now known as Murali06:23
*** yfried_ is now known as yfried|afk06:23
openstackgerritMerged openstack/python-neutronclient: Reinstate Max URI length checking to V2_0 Client  https://review.openstack.org/16092306:24
*** yfried|afk is now known as yfried_06:26
*** salv-orlando has quit IRC06:26
*** salv-orlando has joined #openstack-neutron06:27
*** salv-orlando has quit IRC06:32
*** vmtrooper has joined #openstack-neutron06:32
*** Sukhdev has quit IRC06:33
*** anshul has joined #openstack-neutron06:37
*** vmtrooper has quit IRC06:37
*** salv-orlando has joined #openstack-neutron06:38
*** Rajeev has quit IRC06:40
*** chandankumar has joined #openstack-neutron06:41
*** salv-orlando has quit IRC06:43
*** gildub has quit IRC06:45
*** SridharG has joined #openstack-neutron06:45
*** anish__ has quit IRC06:45
*** sbfox has quit IRC06:51
*** nmagnezi_ has joined #openstack-neutron06:52
*** kevinbenton has quit IRC06:59
*** itzikb has joined #openstack-neutron07:01
*** moshele has joined #openstack-neutron07:02
*** bobmel has joined #openstack-neutron07:05
*** bobmel has quit IRC07:06
*** bobmel has joined #openstack-neutron07:07
*** ajo has quit IRC07:19
*** adelinatuvenie has joined #openstack-neutron07:23
*** ajo has joined #openstack-neutron07:23
*** scheuran has joined #openstack-neutron07:25
*** yfried_ is now known as yfried|afk07:26
*** yfried|afk is now known as yfried_07:26
*** lazy_prince is now known as killer_prince07:26
*** lukasa has quit IRC07:30
*** lukasa has joined #openstack-neutron07:32
*** anish_ has joined #openstack-neutron07:35
*** apuimedo has joined #openstack-neutron07:36
*** Murali has quit IRC07:40
*** neeti has joined #openstack-neutron07:40
*** Murali has joined #openstack-neutron07:41
*** kickinz1|afk is now known as kickinz107:42
*** tflynn has quit IRC07:44
*** Murali has quit IRC07:44
*** tflynn has joined #openstack-neutron07:45
*** Murali has joined #openstack-neutron07:47
*** tflynn has quit IRC07:49
*** kawa2014 has joined #openstack-neutron07:49
*** Murali has quit IRC07:52
*** Murali has joined #openstack-neutron07:54
*** coolsvap is now known as coolsvap|brb07:54
*** evgenyf has joined #openstack-neutron07:56
*** erkules_ is now known as erkules07:56
*** erkules has quit IRC07:56
*** erkules has joined #openstack-neutron07:56
*** Murali has quit IRC07:58
*** Murali has joined #openstack-neutron07:59
*** chlong has quit IRC08:01
*** prometheanfire has quit IRC08:04
*** salv-orlando has joined #openstack-neutron08:05
*** Murali has quit IRC08:08
*** Murali has joined #openstack-neutron08:09
*** pask81 has joined #openstack-neutron08:10
openstackgerritMerged openstack/neutron: Prevent calling waitall() inside a GreenPool's greenthread  https://review.openstack.org/15732008:10
*** Murali has quit IRC08:10
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: utils.execute: Add a debug-level log to record stdin  https://review.openstack.org/16329308:11
*** Murali has joined #openstack-neutron08:13
*** vishwanathj has quit IRC08:14
*** topshare has quit IRC08:15
*** matrohon has joined #openstack-neutron08:15
*** salv-orlando has quit IRC08:15
*** pradipta has quit IRC08:16
*** Murali has quit IRC08:17
*** Murali has joined #openstack-neutron08:18
*** topshare has joined #openstack-neutron08:18
*** topshare has quit IRC08:21
*** vmtrooper has joined #openstack-neutron08:21
*** topshare has joined #openstack-neutron08:23
*** salv-orlando has joined #openstack-neutron08:25
*** salv-orlando has quit IRC08:25
*** vmtrooper has quit IRC08:26
*** coolsvap|brb is now known as coolsvap08:26
*** moha_hunt has joined #openstack-neutron08:33
*** Flcn has joined #openstack-neutron08:33
*** killer_prince is now known as lazy_prince08:33
*** kawa2014 has quit IRC08:35
*** kawa2014 has joined #openstack-neutron08:36
*** devvesa has joined #openstack-neutron08:36
openstackgerritMerged openstack/neutron: Add proccess monitor to keepalived  https://review.openstack.org/15316208:37
*** ajo has quit IRC08:46
*** safchain has joined #openstack-neutron08:46
*** ygbo has joined #openstack-neutron08:47
*** ajo has joined #openstack-neutron08:47
*** jpena has joined #openstack-neutron08:49
openstackgerritYoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script  https://review.openstack.org/15656308:51
*** nyechiel_ has joined #openstack-neutron08:51
*** topshare has quit IRC08:53
*** topshare has joined #openstack-neutron08:55
*** veena has joined #openstack-neutron08:55
*** marios has joined #openstack-neutron08:57
*** topshare has quit IRC08:59
*** yfried_ is now known as yfried|pto09:01
*** topshare has joined #openstack-neutron09:01
*** topshare has quit IRC09:03
*** kickinz1 is now known as kickinz1|afk09:04
*** kickinz1|afk is now known as kickinz109:04
*** topshare has joined #openstack-neutron09:05
*** jlanoux has joined #openstack-neutron09:08
*** ronis has joined #openstack-neutron09:10
*** irenab has quit IRC09:15
openstackgerritJoeJiang proposed openstack/python-neutronclient: Set specify IP address when use 'router-gateway-set' subcommand.  https://review.openstack.org/16333209:16
*** irenab has joined #openstack-neutron09:16
*** afazekas_ has joined #openstack-neutron09:16
*** gongysh has quit IRC09:17
*** irenab has quit IRC09:17
*** Murali has quit IRC09:18
*** Murali has joined #openstack-neutron09:19
*** topshare has quit IRC09:20
*** Murali has quit IRC09:20
*** aepifanov has joined #openstack-neutron09:21
*** irenab has joined #openstack-neutron09:21
*** Murali has joined #openstack-neutron09:23
*** yfried|pto is now known as yfried|afk09:23
*** irenab has quit IRC09:24
*** pradipta has joined #openstack-neutron09:24
*** irenab has joined #openstack-neutron09:24
*** yfried|afk is now known as yfried|pto09:25
*** jistr has joined #openstack-neutron09:25
*** hichihara has quit IRC09:25
*** kevinbenton has joined #openstack-neutron09:27
*** ihrachyshka has joined #openstack-neutron09:28
*** chandankumar has quit IRC09:32
*** anilvenkata has left #openstack-neutron09:34
*** ihrachyshka has quit IRC09:37
*** ihrachyshka has joined #openstack-neutron09:38
*** numan has quit IRC09:39
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: utils.execute: Add a debug-level log to record stdin  https://review.openstack.org/16329309:42
*** salv-orlando has joined #openstack-neutron09:44
*** chandankumar has joined #openstack-neutron09:45
*** claudiub has joined #openstack-neutron09:45
*** salv-orlando has quit IRC09:46
openstackgerritdql proposed openstack/neutron: fix DHCP port changed when dhcp-agent restart  https://review.openstack.org/16252209:48
yamamotoi have a lot of trivial ut patches waiting for review.09:48
*** jistr has quit IRC09:50
*** jistr has joined #openstack-neutron09:52
veenaWhere are the funtionalities in neutron/openstack/common/rpc in Icehouse release implemented in Juno release? I see that in Juno code base of neutron neutron/openstack/common doesn't have rpc directory itself09:53
*** yamamoto has quit IRC09:54
*** numan has joined #openstack-neutron09:55
*** anilvenkata has joined #openstack-neutron09:56
*** Deepak has joined #openstack-neutron10:03
*** oj has joined #openstack-neutron10:03
ojwhy are router interfaces down.. ???10:04
*** anilvenkata has left #openstack-neutron10:05
ojI'm using router connecting an external and an internal network. both interfaces are down.. why??10:05
*** aepifanov has quit IRC10:06
*** jp_at_hp has joined #openstack-neutron10:07
*** anish_ has quit IRC10:07
ihrachyshkaveena, in Juno, we switched to oslo.messaging10:08
*** moha_hunt has quit IRC10:10
*** vmtrooper has joined #openstack-neutron10:10
*** salv-orlando has joined #openstack-neutron10:10
*** aepifanov has joined #openstack-neutron10:13
*** salv-orlando has quit IRC10:14
*** salv-orl_ has joined #openstack-neutron10:15
*** vmtrooper has quit IRC10:15
*** salv-orl_ has quit IRC10:15
*** Flcn_ has joined #openstack-neutron10:16
*** Flcn has quit IRC10:17
*** dims has joined #openstack-neutron10:18
veenaihrachyshka: Thanks for the info. But I'm unable to understand the mechanism behind oslo.messaging. Please help me with some references10:18
*** yamamoto has joined #openstack-neutron10:19
ihrachyshkaveena, there are quite good docs for the library: http://docs.openstack.org/developer/oslo.messaging/10:19
ihrachyshkaveena, and some relevant code in neutron is at neutron/common/rpc.py10:20
veenaihrachyshka: Okay, will have a look. For what is RpcCallback class is used in rpc.py?10:20
veenaihrachyshka, basically am back porting sriov related code from Juno to Icehouse and facing errors due to rpc10:21
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: utils.execute: Add a debug-level log to record stdin  https://review.openstack.org/16329310:21
ihrachyshkaveena, well, callbacks are entry points in server that are called by agents thru message queue10:23
*** chlong has joined #openstack-neutron10:23
openstackgerritMathieu Rohon proposed openstack/neutron: ML2: Change port status only when it's bound to the host  https://review.openstack.org/16317810:24
*** kerpan has joined #openstack-neutron10:25
*** pc_m has joined #openstack-neutron10:30
amotokiihrachyshka: ping10:32
veenaihrachyshka: Sorry about the basic questions, working with rpc for the first time. Where is the entry point defined in the icehouse?10:32
ihrachyshkaveena, which entry point? there are lots of those, depending on services enabled in your setup.10:32
ihrachyshkaamotoki, pong10:32
amotokiihrachyshka: question about your vendor demposition wiki https://wiki.openstack.org/wiki/Neutron/VendorSplitPackaging10:33
amotokiihrachyshka: re: "pinning neutron revision in requirements.txt.", what do you mean?10:33
amotokiihrachyshka: do we need to pin neutron in vendor requirements in their reoisitory? not in neutron/plugijns/xxxx/requirements.txt10:34
veenaihrachyshka: I'll understand the code difference and get back to you. Probably I'll have some understanding at that time10:34
ihrachyshkaamotoki, currently, we make vendor libraries require neutron master from git (via -e git://... in requirements.txt). the idea would be to instead make them depend on hardcoded revision (and move the revision forward periodically)10:34
ihrachyshkaamotoki, it was discussed yesterday on the meeting10:35
ihrachyshkaamotoki, also in the channel after the meeting (marun and dougwig were there)10:35
ihrachyshkaamotoki, neutron/plugijns/xxxx/requirements.txt just points to vendor library needed for the plugin to operate10:36
ihrachyshkaamotoki, this is about requirements.txt inside the vendor library repo10:36
amotokiihrachyshka: yes. the sentence is in olso module section.10:36
ihrachyshkaamotoki, well, that is just to show that some vendors may decide to still rely on incubator files from neutron, and the only way they can do it without being broken each random day is by pinning10:37
openstackgerritMathieu Rohon proposed openstack/neutron: ML2: Change port status only when it's bound to the host  https://review.openstack.org/16317810:37
amotokiihrachyshka: I see. pinning neutron revision helps us avoid conflicts in oslo module versions or other modules.10:38
*** jistr has quit IRC10:38
kevinbentonihrachyshka: how is the end of the cycle handled? what happens when we find out a vendor has a library that isn't actually compatible with the final release?10:38
*** jistr has joined #openstack-neutron10:39
amotokiihrachyshka: I am not opposing neutron pinning in vendor repo. It is a good thing. I just want to say the description in the wiki page is a bit confusing.10:39
kevinbentonihrachyshka: actually pinning completely eliminates the point of 3rd party CI, right?10:39
*** chandankumar has quit IRC10:40
amotokiihrachyshka: kevinbenton: I think neutron team need to declare which revision should be pinned. In the end of the cycle (RC phase) it seems better to use the master HEAD.10:40
amotokiihrachyshka: kevinbenton: especially after dependency freeze date I think.10:40
ihrachyshkakevinbenton, CI should not pin, unit tests should. but yeah, that's tough to find proper balance10:41
ihrachyshkakevinbenton, the idea would be to pin to stable branch once it's officially released, no?10:42
ihrachyshkakevinbenton, meaning, vendors probably will still need to maintain branches10:42
*** yuanying has quit IRC10:43
ihrachyshka(at least until they rely on internal modules from neutron and its dependencies)10:43
*** kevinbenton_ has joined #openstack-neutron10:44
ihrachyshkaamotoki, ok, I'll try to change wording10:44
kevinbenton_ihrachyshka, amotoki: sorry. i got disconnected.10:44
*** trinaths has left #openstack-neutron10:45
*** kevinbenton has quit IRC10:45
amotokiihrachyshka: thanks. anyway I appreciate your writing and I would like to help you too.10:45
kevinbenton_ihrachyshka, amotoki: was there a response to my point about third party CI being pointless if libraries are pinned?10:45
ihrachyshkakevinbenton_, http://eavesdrop.openstack.org/irclogs/%23openstack-neutron/%23openstack-neutron.2015-03-11.log10:47
kevinbenton_ihrachyshka: thanks.10:48
ihrachyshkaamotoki, does it sound better? "* pinning neutron revision in requirements.txt to avoid breakages at random time. Note that in this case vendors should also make sure they update the pin quite frequently to keep up with neutron development. Note that revision pinning has its own benefits beyond oslo modules' consumption."10:48
kevinbenton_ihrachyshka: i assume the vendor lib would have stable branches pinned to stable branches of course10:48
*** salv-orlando has joined #openstack-neutron10:48
ihrachyshkakevinbenton_, btw you can +A oslo.log dep addition10:49
amotokiihrachyshka: http://eavesdrop.openstack.org/irclogs/%23openstack-neutron/%23openstack-neutron.2015-03-10.log is the correct one? around 15UTC.10:50
openstackgerritIhar Hrachyshka proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method  https://review.openstack.org/15410310:53
ihrachyshkaamotoki, yes10:54
*** trinaths has joined #openstack-neutron10:54
*** stanzgy_ has quit IRC10:55
*** stanzgy has quit IRC10:55
*** Marga_ has quit IRC10:55
*** Marga_ has joined #openstack-neutron10:57
*** marrusl has quit IRC11:01
*** claudiub has quit IRC11:01
*** marrusl has joined #openstack-neutron11:03
*** anshul has quit IRC11:06
*** yamamoto has quit IRC11:08
*** anshul has joined #openstack-neutron11:09
*** claudiub has joined #openstack-neutron11:12
*** Marga_ has quit IRC11:20
*** claudiub has quit IRC11:21
*** kevinbenton_ has quit IRC11:25
*** evgenyf has quit IRC11:26
*** anilvenkata has joined #openstack-neutron11:27
*** yamamoto has joined #openstack-neutron11:29
*** anshul has quit IRC11:30
openstackgerritenikanorov proposed openstack/neutron: Set default of api_workers to number of CPUs  https://review.openstack.org/14049311:30
*** evgenyf has joined #openstack-neutron11:39
*** vivek-ebay has joined #openstack-neutron11:40
*** anshul has joined #openstack-neutron11:42
*** vivek-ebay has quit IRC11:42
*** thomasem has quit IRC11:43
*** lazy_prince is now known as killer_prince11:46
*** yamamoto has quit IRC11:46
*** lizk has quit IRC11:50
*** lizk has joined #openstack-neutron11:50
*** thomasem has joined #openstack-neutron11:55
*** thomasem has quit IRC11:56
*** thomasem has joined #openstack-neutron11:57
openstackgerritSalvatore Orlando proposed openstack/neutron: Raise QuotaResourceUnknown in the quota engine  https://review.openstack.org/16317111:58
*** vmtrooper has joined #openstack-neutron11:59
*** anshul has quit IRC11:59
*** kickinz1 is now known as kickinz1|afk12:02
*** de has joined #openstack-neutron12:02
*** Deepak has quit IRC12:03
*** vmtrooper has quit IRC12:04
*** vthapar has quit IRC12:04
*** anshul has joined #openstack-neutron12:05
openstackgerritMamta Prabhu proposed openstack/neutron: IBM SDN-VE Plugin decomposition  https://review.openstack.org/13047112:06
*** killer_prince is now known as lazy_prince12:08
openstackgerritMoshe Levi proposed openstack/neutron: Move mlnx agent to be under ml2/drivers/mlnx  https://review.openstack.org/16269312:10
openstackgerritMerged openstack/neutron: contribute.rst: Use consistent tags  https://review.openstack.org/16325612:10
*** viveknarasimhan has quit IRC12:11
*** pradipta has quit IRC12:11
*** kickinz1|afk is now known as kickinz112:13
*** markvoelker has joined #openstack-neutron12:14
*** britthouser has quit IRC12:18
*** rook has joined #openstack-neutron12:24
*** rook is now known as Guest287612:24
*** raildo_away is now known as raildo12:26
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Fix netns-cleanup broken by ProcessMonitor refactor  https://review.openstack.org/16340112:26
*** claudiub has joined #openstack-neutron12:28
ihrachyshkaajo, ^^ still no func tests for cleanup tool?12:28
ajoihrachyshka, it's WIP :)12:29
mesteryHenryG salv-orlando: Could use a quick merge on this one, it will unblock the client functional tests: https://review.openstack.org/#/c/163289/12:29
ajoihrachyshka somebody needed the bare minimum to keep working12:29
ajoihrachyshka, but I didn't want to block on it for too much as I'm finishing something else12:29
ajoihrachyshka: reload the review, you will see my WIP12:29
ajoihrachyshka, answering to your question: still no func tests, now :(12:30
ajonow->no12:30
HenryGmestery: done12:31
ajoihrachyshka: I guess it should be easier when we have the full stack tests, because we can start all the agents to create stuff, and then get it cleaned up...12:31
mesteryHenryG: Thanks! Now to wait and then run a 15 hour experimental check on my patch :( #realtears12:31
ajoihrachyshka: until then I would stay with unit testing only, unless you can think of a simple way of not replicating the work of the full stack12:31
ajoihrachyshka: I wanted to think about it btw,12:32
ojI'm using router connecting an external and an internal network. both interfaces are down.. why??12:33
* ajo goes to grab something for lunch12:33
*** dims has quit IRC12:33
*** jlanoux has quit IRC12:34
*** dims has joined #openstack-neutron12:34
*** claudiub has quit IRC12:38
openstackgerritrcurran proposed openstack/neutron: ML2 cisco_nexus MD: sync config and models with vendor repo  https://review.openstack.org/16304012:40
openstackgerritItzik Brown proposed openstack/neutron: Adding a cleanup for 'fip-' namespaces in netns_cleanup  https://review.openstack.org/16340312:41
*** moha_hunt has joined #openstack-neutron12:41
*** dboik has joined #openstack-neutron12:41
*** amuller has joined #openstack-neutron12:43
*** chandankumar has joined #openstack-neutron12:43
openstackgerritMamta Prabhu proposed openstack/neutron: IBM SDN-VE Plugin decomposition  https://review.openstack.org/13047112:44
*** evgenyf has quit IRC12:44
ihrachyshkaajo, ideas on what's the culprit: http://logs.openstack.org/03/154103/14/check/gate-neutron-python27/10c3e06/testr_results.html.gz ?12:47
*** jpena is now known as jpena|lunch12:47
*** baoli has joined #openstack-neutron12:47
*** evgenyf has joined #openstack-neutron12:49
*** dboik has quit IRC12:49
*** yamamoto has joined #openstack-neutron12:51
*** trinaths has quit IRC12:51
*** trinaths has joined #openstack-neutron12:51
*** baoli has quit IRC12:52
*** baoli has joined #openstack-neutron12:53
*** trinaths has quit IRC12:56
*** Bhavya has joined #openstack-neutron12:59
*** banix has joined #openstack-neutron13:04
*** jckasper has joined #openstack-neutron13:05
*** banix has quit IRC13:05
openstackgerritrcurran proposed openstack/neutron: ML2 cisco_nexus MD: sync config and models with vendor repo  https://review.openstack.org/16304013:05
*** amotoki has quit IRC13:08
ajoihrachyshka, 1 sec, reading13:11
openstackgerritYoni Shafrir proposed openstack/neutron: Fixed L3 agent manual scheduling for HA routers  https://review.openstack.org/14329713:11
*** banix has joined #openstack-neutron13:12
*** arcimboldo has joined #openstack-neutron13:12
arcimboldoHi all, I'm testing neutron with gre network over a 10G network, and I'm only getting 1.5 Gb/s, how can I improve the performance?13:13
*** aix has joined #openstack-neutron13:13
ajoihrachyshka, seems like https://review.openstack.org/#/c/154103/ the patch is changing which methods to call? may be he needs to change the mock?13:14
ajoI didn't look on it in depth13:14
*** amotoki has joined #openstack-neutron13:15
*** jlanoux has joined #openstack-neutron13:15
ihrachyshkahm, ok, we'll see in the patch itself. I thought there may be some known problem due to recent refactor13:15
*** tiswanso has joined #openstack-neutron13:16
*** dboik has joined #openstack-neutron13:17
openstackgerritDoug Fish proposed openstack/python-neutronclient: Remove unused AlreadyAttachedClient  https://review.openstack.org/16341913:18
*** tiswanso has quit IRC13:18
ojI'm using router connecting an external and an internal network. both interfaces are down.. why??13:19
*** Longgeek has joined #openstack-neutron13:19
*** tiswanso has joined #openstack-neutron13:19
ajoihrachyshka, it seems patch related13:20
*** banix has quit IRC13:22
openstackgerritMerged openstack/python-neutronclient: exec permission to port_test_hook.sh  https://review.openstack.org/16328913:23
*** yatin has quit IRC13:25
*** Bhavya has quit IRC13:26
openstackgerritAdelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V  https://review.openstack.org/16059613:26
openstackgerritMerged openstack/neutron: Added oslo.log dependency  https://review.openstack.org/16320713:28
*** banix has joined #openstack-neutron13:29
*** veena has quit IRC13:30
*** chandankumar has quit IRC13:30
*** shivharis has joined #openstack-neutron13:30
shivharisyamamoto: hi13:31
yamamotoshivharis: hi13:32
shivharisyamamoto: thanks for the review of https://review.openstack.org/#/c/152383/13:32
*** banix has quit IRC13:32
*** dboik has quit IRC13:32
*** dboik has joined #openstack-neutron13:33
shivharisyamamoto: does it really matter if i like to conttinue beyond the error. i might as well not define the class and let it crash13:33
*** dboik has quit IRC13:33
openstackgerritKyle Mestery proposed openstack/python-neutronclient: Copy functional tests from tempest cli  https://review.openstack.org/16284813:33
*** dboik has joined #openstack-neutron13:34
shivharisyamamoto: what is your suggestion?13:34
*** oj has quit IRC13:38
*** Bhavya has joined #openstack-neutron13:38
*** Networkn3rd has quit IRC13:41
openstackgerritMerged openstack/neutron: utils.execute: Add a debug-level log to record stdin  https://review.openstack.org/16329313:41
shivharisyamamoto: i prefer reporting error and continuing13:42
yamamotoshivharis: if you continue, won't it crash later?13:42
*** adelinatuvenie has quit IRC13:42
*** tiswanso has quit IRC13:42
shivharisyamamoto: what is your suggestion13:42
*** lazy_prince is now known as killer_prince13:42
*** Longgeek has quit IRC13:43
*** yuanying has joined #openstack-neutron13:43
arcimboldoIs anyone able to get more than 2Gb/s using gre networks over a 10GE card?13:44
shivharisyamamoto: i am seeking your suggestion13:44
yamamotoshivharis: my suggestion is to remove the class because it's simple :-)13:44
*** vmtrooper has joined #openstack-neutron13:48
*** yuanying has quit IRC13:48
*** annegentle has joined #openstack-neutron13:49
shivharisyamamoto: ok i will take your suggestion, this is not what i would have prefered13:49
*** devvesa has quit IRC13:50
*** vmtrooper has quit IRC13:53
*** jpena|lunch is now known as jpena13:54
*** pal has joined #openstack-neutron13:54
*** Longgeek has joined #openstack-neutron13:55
*** r-daneel has joined #openstack-neutron13:55
*** gongysh has joined #openstack-neutron13:56
*** changbl has quit IRC13:58
*** pal has quit IRC13:59
*** pal has joined #openstack-neutron13:59
*** topshare has joined #openstack-neutron14:00
*** jgrimm is now known as zz_jgrimm14:00
*** topshare has quit IRC14:00
shivharisyamamoto: on second thought, I am not leaning toward removing the class, that seems too drastic - exception may be better - thoughts?14:00
*** tellesnobrega_ has joined #openstack-neutron14:00
*** popw has quit IRC14:02
*** jschwarz has joined #openstack-neutron14:02
*** zzzeek has joined #openstack-neutron14:03
*** evgenyf has quit IRC14:04
*** tsg_ has joined #openstack-neutron14:05
*** tgohad has joined #openstack-neutron14:05
*** yamamoto has quit IRC14:05
*** yamamoto has joined #openstack-neutron14:05
openstackgerritPavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM  https://review.openstack.org/15323614:06
*** shwetaap has joined #openstack-neutron14:06
*** topshare has joined #openstack-neutron14:06
*** topshare has quit IRC14:06
*** topshare has joined #openstack-neutron14:07
*** changbl has joined #openstack-neutron14:09
*** shivharis has quit IRC14:09
openstackgerritMoshe Levi proposed openstack/neutron: Add Mellanox decomposition progress to chart  https://review.openstack.org/16165114:10
*** moha_hunt has quit IRC14:11
*** yshafrir has quit IRC14:16
*** moshele has quit IRC14:22
*** salv-orlando has quit IRC14:23
*** topshare has quit IRC14:23
*** absubram has quit IRC14:24
*** fredlhsu has joined #openstack-neutron14:25
*** topshare has joined #openstack-neutron14:25
*** salv-orlando has joined #openstack-neutron14:31
*** banix has joined #openstack-neutron14:33
openstackgerritBradley Jones proposed openstack/neutron: Advertise mtu over dhcp  https://review.openstack.org/15587414:35
*** radez_g0n3 is now known as radez14:36
*** numan has quit IRC14:36
openstackgerritPradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes  https://review.openstack.org/15492114:37
*** roeyc has joined #openstack-neutron14:37
openstackgerritZongKai LI proposed openstack/neutron: Removes router binding with router-interface-delete  https://review.openstack.org/16345414:39
*** SridharG has left #openstack-neutron14:40
*** dboik_ has joined #openstack-neutron14:43
*** vishwanathj has joined #openstack-neutron14:43
*** ChuckC_ has joined #openstack-neutron14:43
*** vivek-ebay has joined #openstack-neutron14:43
*** ChuckC has quit IRC14:44
*** gongysh has quit IRC14:45
*** vishwana_ has joined #openstack-neutron14:45
*** claudiub has joined #openstack-neutron14:45
*** dboik has quit IRC14:46
*** vishwanathj has quit IRC14:47
*** vivek-ebay has quit IRC14:47
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed  https://review.openstack.org/16226014:48
*** ChuckC_ has quit IRC14:49
mtreinishmestery: I was looking at your neutronclient functional test job, I was curious why did you make it experimental?14:50
mesterymtreinish: I was following jogo's suggestion in his email to make it experimental at first, but clearly I should have made it enbled in the check queue as non-voting instead.14:50
* mestery mtreinish: We REALLY need this to merge: https://review.openstack.org/#/c/163101/14:50
mtreinishanteaya: ^^^14:51
*** carl_baldwin has joined #openstack-neutron14:52
*** yfried|pto is now known as yfried|afk14:52
*** SridharRamaswamy has joined #openstack-neutron14:54
*** pal has quit IRC14:54
*** zz_jgrimm is now known as jgrimm14:55
*** cbouch has joined #openstack-neutron14:55
*** gongysh has joined #openstack-neutron14:56
mesterymtreinish: Once that merges, the experimental queue won'14:56
mesterywon't be so awful.14:56
mesterymtreinish: I still expect to get those tests out of tempest by Friday, fingers crossed ;)14:56
mtreinishmestery: yeah that would be awesome14:56
*** topshare has quit IRC14:58
*** pal has joined #openstack-neutron14:58
*** yfried|afk is now known as yfried|pto14:59
*** gongysh has quit IRC15:00
openstackgerritstephen-ma proposed openstack/neutron: Router is not unscheduled when the last VM is deleted  https://review.openstack.org/15929615:00
*** yamamoto has quit IRC15:00
*** chandankumar has joined #openstack-neutron15:00
*** bquan has joined #openstack-neutron15:04
*** banix has quit IRC15:07
*** tellesnobrega_ has quit IRC15:07
*** armax has joined #openstack-neutron15:07
*** irenab has quit IRC15:08
jschwarzarmax, morning15:09
jschwarzarmax, re: https://review.openstack.org/#/c/128259/ (fullstack framework), was wondering if you add a chance to think about that patch further15:09
*** Rajeev_ has joined #openstack-neutron15:10
*** zhipeng has joined #openstack-neutron15:10
*** yog_ has quit IRC15:10
*** banix has joined #openstack-neutron15:12
*** rodrigods has quit IRC15:13
ajohmm15:13
*** rodrigods has joined #openstack-neutron15:13
ajomestery, ihrachyshka , armax , pylint is crying at stuff I didn't modify:15:13
ajohttps://review.openstack.org/#/c/163401/15:13
ajohttp://logs.openstack.org/01/163401/1/check/gate-neutron-pep8/d059591/console.html15:13
ihrachyshkaajo, right, I told you it's crazy!15:14
ajohttp://logs.openstack.org/01/163401/1/check/gate-neutron-pep8/d059591/console.html#_2015-03-11_13_18_17_58715:14
ajoihrachyshka we were not using pylint before, right?15:14
ihrachyshkaajo, define before. it's new in kilo afaik15:14
amullerajo: if you run tox -e pep8 locally does it pass?15:15
ajoihrachyshka how long ago have we been using it? it's the first time I see it run locally for tox -epep815:15
amullerajo: pretty recent, few weeks15:15
ajoamuller, let me check I believe it passed locally15:15
*** ChuckC_ has joined #openstack-neutron15:16
*** banix has quit IRC15:16
ajocircus music while it runs...15:17
amullerajo: btw it's funny that the unit test is failing on that patch15:17
amullerso it's failing on the fix but not on what broke it15:17
ajopylint: tito titoriroritotito... tito titorirorirorito...15:17
*** tflynn has joined #openstack-neutron15:17
ajoit passed locally15:17
*** banix has joined #openstack-neutron15:17
*** carl_baldwin has quit IRC15:18
*** xgerman has joined #openstack-neutron15:18
*** tflynn has quit IRC15:18
ajo[centos@devstack neutron]$ .tox/pep8/bin/pylint --version15:18
ajoNo config file found, using default configuration15:18
ajopylint 1.4.1,15:18
ajoastroid 1.3.4, common 0.63.215:18
*** tflynn has joined #openstack-neutron15:18
amullerajo: The only explanation is that the gate is using a different version of pylint15:19
amullerperhaps only that VM15:19
amullerbecause other patches seem to be passing15:19
ihrachyshkaamuller, I saw my patches failing locally and not in gate. using f2115:19
*** pal has quit IRC15:19
amullerI've had different results before, depending on the VM that happened to check my patch15:19
openstackgerritMathieu Rohon proposed openstack/python-neutronclient: Add commands from extensions to available commands  https://review.openstack.org/16347815:20
ihrachyshkadougwig, is there any case when vpn agent is running in parallel to l3 agent on the same node?15:20
matrohonroaet_ : hi15:21
*** shwetaap has quit IRC15:21
*** tiswanso has joined #openstack-neutron15:21
ajoamuller, ihrachyshka  I'm recreating venv, to see if I get a new pylint15:21
*** shwetaap has joined #openstack-neutron15:22
amullerihrachyshka: pylint doesn't seem to be a requirement of neutron15:22
amullermaybe devstack installs pylint?15:22
*** Murali has quit IRC15:22
amuller(We should pin to a specific version so that running it locally and on the gate gives the same result, always)15:23
ajoha!, pep8 installdeps: -r/opt/stack/neutron/requirements.txt, -r/opt/stack/neutron/test-requirements.txt, pylint15:23
ajoI guess this answers the question15:23
ajoand yes, now I get the warning15:23
ajomay be we should just fix the warning?15:23
ajothere's only one15:23
*** anilvenkata has left #openstack-neutron15:23
ajo************* Module neutron.agent.l3.event_observers15:23
ajoW: 27,15: Using type() instead of isinstance() for a typecheck. (unidiomatic-typecheck)15:23
ihrachyshkawhy isn't it in test-requirements.txt?15:23
amullerihrachyshka: good question15:24
openstackgerritMerged openstack/neutron-vpnaas: Decouple L3 and VPN during DVR router migration  https://review.openstack.org/16238115:24
ajoihrachyshka, that's a good question15:24
ihrachyshkait's in global reqs: <ajo> astroid 1.3.4, common 0.63.215:24
ihrachyshkasorry...15:24
ihrachyshkahttps://github.com/openstack/requirements/blob/master/global-requirements.txt#L23315:24
ajothis is pilint 1.4.215:24
ajopylint, the one failing15:24
openstackgerritrcurran proposed openstack/neutron: ML2 cisco_nexus MD: sync config and models with vendor repo  https://review.openstack.org/16304015:24
ajoby the way, ... it's an stupid "complain" non-idiomatic type check15:25
ajono danger, but to the code reader15:25
ajoand yet, ... we... the type(xxx) is .....15:25
ajosounds reasonable to me15:25
ajopylint--15:26
ihrachyshkaajo, however reasonable the check is, it should not break people randomly15:26
ajoyes, probably we should pin pylint15:26
*** jprovazn has joined #openstack-neutron15:27
ajoI will propose a patch, 1 sec15:27
*** budnyjj has joined #openstack-neutron15:27
amullerajo: btw the netns cleanup really needs a functional test15:27
amullerajo: We talked about this before15:27
*** pmath has joined #openstack-neutron15:27
*** tflynn has quit IRC15:27
ajoamuller, yes, but I'm still waiting for the dhcp ones,15:27
ajoamuller, so I can reuse that logic to spawn, stuff, and then clean it up,15:27
ajoamuller, duplicating efforts when you're swamped is quite... swamping ?15:28
ajomarios,15:28
*** david-lyle_afk is now known as david-lyle15:28
ajodid the dhcp functional tests merge already? :)15:28
ajomay be I missed it15:28
*** gridinv_ has joined #openstack-neutron15:29
ajoahh, :/15:29
ajoseems quite stopped15:29
amullerI'm looking at the netns script... Why would you need to initialize the DHCP agent or clean it up?15:29
ajomarios , let me know if you need anyone to re-take your patch15:29
ihrachyshkaajo, there is no one on this earth who will push the effort15:29
ajohttps://review.openstack.org/#/c/136834/15:29
*** fredlhsu has quit IRC15:29
ajoamuller, because I need dnsmasq's created inside the namespace15:30
ihrachyshka(other than you)15:30
ajoto make sure it will go and clean up the stuff properly (service, ports, netns)15:30
amullerajo: you can start with just namespaces15:30
*** Marga_ has joined #openstack-neutron15:30
ajoamuller, yes, but that's the easy case, that's never been broken, but you're right15:30
ajowe can put at least the building blocks15:31
amullerbut such a test would catch the regression recently introduced...15:31
amullereven without initializing a DHCP agent15:31
ajoand then add more stuff as we have the dhcp bits15:31
mariosajo: if you have the time please go ahead as you've seen the firedrill i'm involved in15:31
ajoamuller, I'm not sure it would call kill_dhcp, as it isn't active?... ahh.. it would15:31
ajomarios, yes... I've seen it :/ lots of fun ;)15:32
mariosajo: hey man, thanks again, and sorry again. i am exploring therabbit issue some more and eck i think has progress with jan15:32
ajomarios, I will call the team for help to see if somebody can take it from there.15:32
mariosajo: i owe you multiple cerveza15:33
ajoamuller, I believe you're right, I will put some basic functional testing in the "revert" patch15:33
amullerihrachyshka: ajo: Looks like more patches failing now15:33
amullerI'll push a patch to fix that pylint issue15:33
ajothat will help us catch some of the issues15:33
ajothen we can add more stuff when we can spawn dhcp services functionally15:33
amullerAnd I think I'll push a patch to global requirements as well15:33
amullerto pin pylint15:33
ajoamuller, I was going to do it, but go ahead, may be you can fix both? the warning, and pin pylint?15:34
amullersure15:34
*** gridinv_ has quit IRC15:34
ajo++15:34
ihrachyshkaamuller, and probably also move pylint dep from tox.ini into test-req?15:35
amullerihrachyshka: of course15:35
ihrachyshkaack15:36
*** Marga_ has quit IRC15:36
*** shivharis has joined #openstack-neutron15:36
*** vmtrooper has joined #openstack-neutron15:36
*** Sukhdev has joined #openstack-neutron15:37
*** zhipeng has quit IRC15:39
*** carl_baldwin has joined #openstack-neutron15:40
*** Networkn3rd has joined #openstack-neutron15:41
*** Bhavya has quit IRC15:41
*** SridharRamaswamy has quit IRC15:41
*** vmtrooper has quit IRC15:42
*** afazekas_ has quit IRC15:42
carl_baldwinsalv-orlando, I wonder if it is time to merge the IPAM interface.  Does it need any updates that you know of?15:43
*** Longgeek has quit IRC15:45
openstackgerritAssaf Muller proposed openstack/neutron: Fix pylint issue with type VS isinstance in event_observers  https://review.openstack.org/16349315:45
amullercarl_baldwin: mestery: Can you approve that patch please to unblock the gate15:46
*** mriedem has joined #openstack-neutron15:47
*** dboik_ has quit IRC15:47
amullerit passes pep8/pylint and py2715:47
mesteryamuller: Ack, I can ninja merge it15:47
*** dboik has joined #openstack-neutron15:47
amullermestery: thanks15:47
mesterycarl_baldwin: Thanks to you too15:48
carl_baldwinamuller:  I can approve but I have to ask, why are we still at the mercy of pylint updates?15:48
amullerWe don't pin pylint15:48
*** britthouser has joined #openstack-neutron15:48
amullerso if it checks new stuff all of a sudden, we're broken15:48
carl_baldwinamuller:  I thought this came up before and we pinned it.15:48
carl_baldwinWe must pin it.15:48
amullercarl_baldwin: working on that15:48
amullerit isn't pinned atm15:48
*** vishwana_ has quit IRC15:49
*** vishwanathj has joined #openstack-neutron15:49
*** nati_ueno has joined #openstack-neutron15:50
carl_baldwinamuller, :)15:50
carl_baldwinmestery, sorry to steal your Ninja merge opportunity.15:50
amullerThe faster ninja always wins15:51
*** Marga_ has joined #openstack-neutron15:51
mesterycarl_baldwin: lol15:51
*** itzikb has quit IRC15:53
*** vivek-ebay has joined #openstack-neutron15:53
amullerihrachyshka: carl_baldwin: https://github.com/openstack/requirements#general-review-criteria15:53
amullercheck out the first bullet15:54
*** coolsvap is now known as coolsvap|afk15:54
mriedemis ijw ever in here?15:54
mriedemianw: ^?15:54
ihrachyshkaamuller, check hacking, it's capped15:54
mriedemwrong person15:54
mriedemnvm15:54
shivharisarmax: hi15:54
ihrachyshkaamuller, and exactly because we don't want a new release to break the world15:55
*** saggi has quit IRC15:55
*** rwsu-afk is now known as rwsu15:55
*** armax has quit IRC15:55
*** armax has joined #openstack-neutron15:55
shivharisarmax: hi15:56
amullerihrachyshka: sorry what do you mean by 'check hacking, it's capped'15:57
*** ronis has quit IRC15:57
carl_baldwinamuller, I'm not sure that review criteria applies.  Shipped code does not depend on pylint.  I recall that infra encouraged pinning pylint last time this came up.  I could dig up that conversation if you want.15:58
openstackgerritBradley Jones proposed openstack/neutron: Advertise mtu over dhcp  https://review.openstack.org/15587415:58
ihrachyshkaamuller, hacking library (used for pep8), it's capped in global requirements15:58
amullerihrachyshka: gotcha15:58
ihrachyshkaamuller, https://github.com/openstack/requirements/blob/master/global-requirements.txt#L20915:58
carl_baldwinamuller, we don't need gate breakage due to pylint updates a week before feature freeze.15:59
*** yamahata has joined #openstack-neutron16:00
armaxshivharis: hi16:00
shivharisRegarding https://review.openstack.org/#/c/152383/, do you think i need to update due to a -116:01
amullerihrachyshka: carl_baldwin: Thinking of capping it to 1.4.1, which is the version that we broke on. It's already fixed in Neutron but I wonder if other projects are voting on pylint, and maybe we should cap to an earlier version instead16:01
*** Swami has joined #openstack-neutron16:01
ihrachyshkaamuller, whatever suits *us* I guess16:01
shivharisarmax: let me know at your convenience16:01
ihrachyshkaamuller, if they care, they will lower the cap16:01
shivharisarmax: no hurry16:01
armaxshivharis: ok, I’ll look into it16:02
armaxshivharis: thanks16:02
carl_baldwinamuller:  I thought we were the only ones voting on pylint.16:02
shivharisarmax: thanks16:02
carl_baldwinamuller, I think nova uses it as an advisory job.16:02
*** _cjones_ has joined #openstack-neutron16:03
carl_baldwinThings change so fast that my knowledge could be out of date.16:03
*** nlahouti has joined #openstack-neutron16:04
armaxamuller: how long have we been broken?16:04
amulleran hour I think16:04
*** thedodd has joined #openstack-neutron16:04
amullerhttps://review.openstack.org/#/c/163506/16:04
armaxamuller: do you want to disable the pylint check for now?16:04
amullerarmax: no need16:05
amullerarmax: patch that fixes the issue was already approved16:05
amullerI linked the patch to global requirements above if you guys want to pitch in16:05
armaxamuller: which one?16:05
armaxamuller: I mean which one is approved?16:05
amullerhttps://review.openstack.org/#/c/163493/16:05
*** jlanoux has quit IRC16:06
armaxamuller: got it, tnx16:06
carl_baldwinsalv-orlando, moving to openstack-neutron because I didn't intend to start in meeting-3.16:06
openstackgerritMichael Smith proposed openstack/neutron: L3 Agent support of manual dvr snat add/remove  https://review.openstack.org/12381516:07
carl_baldwinsalv-orlando, I think we should merge it soon (https://review.openstack.org/#/c/134339/) so that we are free to merge the two dependent on it when they're ready.16:08
*** bobmel has quit IRC16:08
armaxamuller: there was a discussion about pinning global requirements to specific versions16:08
*** bobmel has joined #openstack-neutron16:08
armaxI remember that marun raised the point in the infra channel wrt tempestlib16:08
*** rushil has joined #openstack-neutron16:08
*** marun has joined #openstack-neutron16:09
*** vivek-ebay has quit IRC16:10
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961816:10
*** enikanorov__ has joined #openstack-neutron16:10
*** enikanorov has quit IRC16:10
*** sripriya has joined #openstack-neutron16:11
openstackgerritAssaf Muller proposed openstack/neutron: Move pylint dep from tox to test-requirements  https://review.openstack.org/16351016:11
openstackgerritAssaf Muller proposed openstack/neutron: Move pylint dep from tox to test-requirements  https://review.openstack.org/16351016:12
*** rossella_s has quit IRC16:12
*** amuller_ has joined #openstack-neutron16:13
*** britthouser has quit IRC16:13
*** markmcclain has quit IRC16:13
*** exed_ has quit IRC16:13
*** briancurtin has quit IRC16:13
*** wendar_ has joined #openstack-neutron16:14
*** sbalukoff has quit IRC16:14
*** comstud has quit IRC16:14
*** lbragstad has quit IRC16:14
*** zacksh has quit IRC16:14
*** jmeridth has quit IRC16:14
*** jroll has quit IRC16:14
*** edwarnicke has quit IRC16:14
*** amuller has quit IRC16:14
*** Nakato has quit IRC16:14
*** flwang1 has quit IRC16:14
*** wendar has quit IRC16:14
*** gus has quit IRC16:14
*** russellb has quit IRC16:14
*** zigo_ has quit IRC16:14
*** rstarr has quit IRC16:14
ajoamuller, armax , carl_baldwin , pining make sense for pylint, if they will be changing behavior from version to version this can happen again.16:14
*** zigo_ has joined #openstack-neutron16:14
ajoamuller_ ^16:15
armaxajo: agreed, we ping pep8 too16:15
armaxpin16:15
ajoah, ack16:15
*** russellb has joined #openstack-neutron16:15
armaxso I don’t see why pylint should be different16:15
*** exed_ has joined #openstack-neutron16:15
*** markmcclain has joined #openstack-neutron16:15
carl_baldwin+116:15
ajomakes sense16:15
*** zacksh has joined #openstack-neutron16:15
*** jroll has joined #openstack-neutron16:15
*** lbragstad has joined #openstack-neutron16:15
*** jmeridth has joined #openstack-neutron16:15
ajohttps://review.openstack.org/#/c/163506/ this is the pinning review16:15
*** comstud has joined #openstack-neutron16:15
*** rstarr has joined #openstack-neutron16:15
*** Nakato has joined #openstack-neutron16:16
*** gus has joined #openstack-neutron16:16
*** britthouser has joined #openstack-neutron16:16
*** scheuran has quit IRC16:16
*** jroll has quit IRC16:16
*** jroll has joined #openstack-neutron16:16
*** sbalukoff has joined #openstack-neutron16:16
*** rossella_s has joined #openstack-neutron16:16
*** flwang1 has joined #openstack-neutron16:17
*** annegentle has quit IRC16:18
*** briancurtin has joined #openstack-neutron16:19
*** nati_ueno has quit IRC16:20
*** anshul has quit IRC16:20
*** neeti has quit IRC16:20
*** edwarnicke has joined #openstack-neutron16:22
*** s3wong has joined #openstack-neutron16:24
*** s3wong has quit IRC16:26
*** tiswanso has quit IRC16:26
*** mriedem has left #openstack-neutron16:26
*** SumitNaiksatam has joined #openstack-neutron16:27
*** tiswanso has joined #openstack-neutron16:27
*** dboik_ has joined #openstack-neutron16:27
*** emagana has quit IRC16:28
*** emagana has joined #openstack-neutron16:28
*** sbfox has joined #openstack-neutron16:28
*** jschwarz has quit IRC16:29
*** ijw has joined #openstack-neutron16:29
openstackgerritenikanorov proposed openstack/neutron: Set default of api_workers to number of CPUs  https://review.openstack.org/14049316:29
*** dboik has quit IRC16:31
*** nmagnezi_ has quit IRC16:31
*** emagana has quit IRC16:33
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433916:34
openstackgerritPradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes  https://review.openstack.org/15492116:34
carl_baldwinsalv-orlando:  I just updated translations:  https://review.openstack.org/#/c/134339/16:35
*** jschwarz has joined #openstack-neutron16:35
carl_baldwinsalv-orlando:  I'm moving on to review https://review.openstack.org/#/c/150485/ now.16:36
*** leenheer has joined #openstack-neutron16:37
*** vivek-ebay has joined #openstack-neutron16:38
*** britthouser has quit IRC16:39
*** fredlhsu has joined #openstack-neutron16:39
openstackgerritTerry Wilson proposed openstack/neutron: Add rootwrap daemon mode support  https://review.openstack.org/15880516:40
*** carl_baldwin has quit IRC16:42
*** Longgeek has joined #openstack-neutron16:45
*** yfried|pto is now known as yfried|afk16:47
*** carl_baldwin has joined #openstack-neutron16:48
*** adelinatuvenie has joined #openstack-neutron16:50
*** melwitt has joined #openstack-neutron16:53
*** openstack has joined #openstack-neutron16:55
jschwarzrkukura, ping16:56
*** Flcn_ has joined #openstack-neutron16:57
rkukurajschwarz: On the ML2 meeting at the moment16:57
matrohonrossela_s : hi16:58
matrohonrossella_s : hi16:58
*** vivek-ebay has quit IRC17:00
matrohonroaet : hi17:01
roaetmatrohon: hi17:01
*** s3wong has joined #openstack-neutron17:01
*** vivek-ebay has joined #openstack-neutron17:01
*** leenheer has left #openstack-neutron17:01
*** vivek-ebay has quit IRC17:01
matrohonroaet : the client extension framework for me with this patch : https://review.openstack.org/#/c/163478/17:02
shivharismestery: hi17:02
*** Longgeek has quit IRC17:02
*** mwagner_lap has quit IRC17:03
roaetmatrohon: I'm sorry I didn't catch that.17:03
jschwarzrkukura, HenryG suggested I'd ask you to have a look at https://review.openstack.org/#/c/129852/17:03
matrohonroaet : do you think it's a valid patch?17:03
jschwarzit deals with some MTU stuff on the sanity script, nothing too major hopefully :)17:03
*** salv-orlando has quit IRC17:03
roaetmatrohon: external modules? from where? ep?17:04
matrohonroaet : ep, yes17:04
roaetmatrohon: I guess they weren't working? heh17:04
*** radez is now known as radez_g0n317:05
roaetmatrohon: I need to look into that17:05
matrohonroaet : actually, it works :) I described how to make it work on the bug report : https://bugs.launchpad.net/python-neutronclient/+bug/143082517:06
openstackLaunchpad bug 1430825 in python-neutronclient "extending python-neutronclient commands with an external module fails" [Undecided,In progress] - Assigned to Mathieu Rohon (mathieu-rohon)17:06
*** roeyc has quit IRC17:06
*** yfried|afk is now known as yfried|pto17:06
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869817:07
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759717:08
*** SridharRamaswamy has joined #openstack-neutron17:08
*** pask81 has quit IRC17:08
matrohonroaet : I was messed up by the fact that a module has to be set up in the setup.cfg, as the ep, not a class17:08
*** aepifanov has quit IRC17:09
*** SumitNaiksatam has quit IRC17:09
*** ivar-lazzaro has joined #openstack-neutron17:10
*** mlavalle has joined #openstack-neutron17:10
*** ivar-lazzaro has quit IRC17:11
*** ivar-lazzaro has joined #openstack-neutron17:11
*** ivar-lazzaro has quit IRC17:12
*** ygbo has quit IRC17:12
*** ivar-lazzaro has joined #openstack-neutron17:13
shivharismestery: hi17:14
*** harlowja_away is now known as harlowja_17:14
mesteryshivharis: Hey, in a few meetings, but whats up?17:14
shivharismestery: No hurry, need opinion: do we need this bug fixed for k3 : https://bugs.launchpad.net/neutron/+bug/137873217:14
openstackLaunchpad bug 1378732 in neutron "migrate_to_ml2 script doesn't work for Juno release" [High,New] - Assigned to Mark McClain (markmcclain)17:14
mesteryshivharis: I think so, it's assigned to markmcclain now, lets see if he's wokring on it ATM as well.17:15
shivharismestery: thanks17:15
*** nati_ueno has joined #openstack-neutron17:16
*** nati_ueno has quit IRC17:16
*** nati_ueno has joined #openstack-neutron17:17
*** matrohon has quit IRC17:18
*** ihrachyshka has quit IRC17:18
*** tflynn has joined #openstack-neutron17:20
*** tflynn has quit IRC17:20
*** chlong has quit IRC17:21
*** tflynn has joined #openstack-neutron17:22
pc_marmax: ping17:22
anteayaany one know the owner of this patch?17:24
anteayahttps://review.openstack.org/#/c/162681/317:24
*** vmtrooper has joined #openstack-neutron17:25
markmcclainshivharis: feel free to take it /cc mestery17:27
mesterythanks markmcclain17:28
*** wendar_ is now known as wendar17:28
anteayaokay so heads up I'm about to approve 16268117:29
anteayaso whoever that is none of their code will merge in their new repo until they have 4 passing python tests in their first patch17:29
*** _cjones_ has quit IRC17:30
anteayaso if they come stumbling in here asking why, it is because they have an empty repo with python-tests in their zuul/layout.yaml17:30
*** vmtrooper has quit IRC17:31
*** tsg_ has quit IRC17:31
HenryGanteaya: thanks, I think that info is for banix & co.17:32
*** shivharis has quit IRC17:32
anteayaHenryG: thanks17:32
*** SumitNaiksatam has joined #openstack-neutron17:32
banixanteaya: HenryG thanks17:33
*** Swami has quit IRC17:34
*** ronis has joined #openstack-neutron17:34
*** Flcn_ has quit IRC17:35
*** jdandrea has quit IRC17:35
openstackgerritChuck Carlino proposed openstack/neutron: Prevent updating mac address of bound port  https://review.openstack.org/16250017:36
*** chandankumar has quit IRC17:36
*** Flcn_ has joined #openstack-neutron17:37
anteayabanix: that is your repo?17:37
banixanteaya: yes17:37
anteayagood17:37
*** kawa2014 has quit IRC17:37
*** Sukhdev has quit IRC17:37
*** _cjones_ has joined #openstack-neutron17:38
*** gridinv_ has quit IRC17:39
*** Flcn_ has quit IRC17:41
*** annegentle has joined #openstack-neutron17:41
*** Flcn_ has joined #openstack-neutron17:41
*** fredlhsu has quit IRC17:41
*** yatin has joined #openstack-neutron17:41
*** igordcard_ has joined #openstack-neutron17:42
*** devlaps has joined #openstack-neutron17:42
*** pritesh has joined #openstack-neutron17:43
*** jschwarz has quit IRC17:43
*** chandankumar has joined #openstack-neutron17:44
*** sripriya has quit IRC17:45
*** chandankumar has quit IRC17:47
*** kerpan has quit IRC17:48
*** yfried|pto is now known as yfried|afk17:48
*** mwagner_lap has joined #openstack-neutron17:48
*** matrohon has joined #openstack-neutron17:49
*** sripriya has joined #openstack-neutron17:49
yatinneed some reviewers https://review.openstack.org/#/c/145290/17:51
*** Bhavya has joined #openstack-neutron17:52
*** jp_at_hp has quit IRC17:52
*** achanda has joined #openstack-neutron17:54
*** jpena has quit IRC17:55
openstackgerritTerry Wilson proposed openstack/neutron: Add rootwrap daemon mode support  https://review.openstack.org/15880517:56
*** claudiub has quit IRC17:57
*** radez_g0n3 is now known as radez18:00
*** nati_ueno has quit IRC18:01
*** Longgeek has joined #openstack-neutron18:02
*** leenheer has joined #openstack-neutron18:02
*** nati_ueno has joined #openstack-neutron18:02
*** reed has joined #openstack-neutron18:03
*** reed has quit IRC18:03
*** reed has joined #openstack-neutron18:03
*** yatin has quit IRC18:04
*** salv-orlando has joined #openstack-neutron18:04
*** Networkn3rd has quit IRC18:05
*** emagana has joined #openstack-neutron18:05
*** Flcn_ has quit IRC18:06
*** Flcn_ has joined #openstack-neutron18:06
*** VijayB has joined #openstack-neutron18:06
*** arcimboldo has quit IRC18:09
*** Longgeek has quit IRC18:09
*** Guest2876 is now known as rook18:10
*** carl_baldwin has quit IRC18:10
*** rook is now known as Guest3880018:10
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856018:10
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284018:10
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439118:10
*** chandankumar has joined #openstack-neutron18:11
*** Guest38800 is now known as rook-tmp18:11
*** pritesh has quit IRC18:13
*** Flcn_ has quit IRC18:13
*** kbyrne has quit IRC18:13
*** budnyjj has quit IRC18:13
*** kbyrne has joined #openstack-neutron18:14
*** salv-orlando has quit IRC18:14
*** Flcn_ has joined #openstack-neutron18:18
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: Adds FWaaS service plugin for Cisco CSR1kv  https://review.openstack.org/15228218:19
*** absubram has joined #openstack-neutron18:20
*** pritesh has joined #openstack-neutron18:20
*** aix has quit IRC18:21
*** Bhavya has quit IRC18:23
*** carl_baldwin has joined #openstack-neutron18:32
*** shivharis has joined #openstack-neutron18:33
*** jaypipes is now known as jaypipes-afk18:34
*** sthillma has joined #openstack-neutron18:37
*** tsg_ has joined #openstack-neutron18:37
*** nati_ueno has quit IRC18:38
*** nati_ueno has joined #openstack-neutron18:39
openstackgerritIsaku Yamahata proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test  https://review.openstack.org/16005118:39
*** Bhavya has joined #openstack-neutron18:42
openstackgerritAssaf Muller proposed openstack/neutron: Add RPC test for DVR  https://review.openstack.org/16320118:44
*** salv-orlando has joined #openstack-neutron18:44
*** banix has quit IRC18:45
*** jistr has quit IRC18:46
*** natarajk has joined #openstack-neutron18:47
*** Flcn_ has quit IRC18:48
natarajkarmax:Hi18:48
*** nyechiel_ has quit IRC18:48
armaxnatarajk: hello18:48
armaxanteaya: ping18:49
*** nyechiel_ has joined #openstack-neutron18:49
natarajkarmax: Can you please review https://review.openstack.org/#/c/159056/ (Brocade Vyatta L3 plugin)18:49
armaxnatarajk: it’s on my backlog18:49
natarajkarmax: Great, thanks18:49
armaxnatarajk: sorry if it’s taking longer than you’d expect18:49
natarajkarmax:np, i understand18:49
armaxnatarajk: thanks18:49
*** kbringard has joined #openstack-neutron18:50
anteayaarmax: how can I help?18:51
armaxanteaya: hi! quick question on the neutron-ci group18:51
openstackgerritIsaku Yamahata proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test  https://review.openstack.org/16005118:51
*** safchain has quit IRC18:51
* anteaya listens18:51
armaxanteaya: I assume you have permissions to add an account to the group?18:52
anteayaI do not18:52
armaxanteaya: or is there a formal process?18:52
anteayaI abdicated that right some time ago18:52
anteayaask mestery18:52
anteayait is as formal as he wants to make it18:52
armaxanteaya: ok18:52
armaxanteaya: thanks for the pointer18:52
anteayahaving something logged and archieved is certainly suggested18:52
* armax walks to mestery virtual office18:52
anteaya:)18:53
anteayaso that others can cargo cult a useful behaviour18:53
armaxmestery: knock knock18:53
openstackgerritAngela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX  https://review.openstack.org/16148918:53
*** nyechiel_ has quit IRC18:53
armaxanteaya: makes sense, I’ll sync up with him18:53
anteayasure18:53
anteayalet me know if you hit an obstacle18:53
anteayaI don't expect one though18:53
armaxanteaya: yes, thanks will do18:54
*** banix has joined #openstack-neutron18:54
*** s3wong has quit IRC18:54
*** chandankumar has quit IRC18:57
openstackgerritAngela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches.  https://review.openstack.org/15470918:58
*** Marga_ has quit IRC18:58
*** marun has quit IRC19:00
*** s3wong has joined #openstack-neutron19:03
*** leenheer has quit IRC19:03
*** bobmel_ has joined #openstack-neutron19:06
*** leenheer has joined #openstack-neutron19:07
*** bobmel has quit IRC19:08
*** Networkn3rd has joined #openstack-neutron19:13
*** aepifanov has joined #openstack-neutron19:13
*** vmtrooper has joined #openstack-neutron19:14
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Adds DVR functional test for multi-external networks  https://review.openstack.org/16357319:16
*** aepifanov has quit IRC19:18
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Adds DVR functional test for multi-external networks  https://review.openstack.org/16357319:19
*** aepifanov has joined #openstack-neutron19:19
*** pritesh has quit IRC19:19
*** vmtrooper has quit IRC19:19
*** rushil has quit IRC19:20
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856019:21
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284019:21
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439119:21
*** diegows has joined #openstack-neutron19:23
*** nati_ueno has quit IRC19:27
*** achanda has quit IRC19:27
*** emagana has quit IRC19:29
*** sbfox has quit IRC19:29
*** sbfox has joined #openstack-neutron19:30
*** amuller_ is now known as amuller19:30
*** de has quit IRC19:31
yamahataijw: ping?19:32
*** radez is now known as radez_g0n319:32
ijwyo19:32
*** kickinz1 is now known as kickinz1|afk19:32
yamahataijw: regarding to port security extension support19:33
ijwyup?19:33
yamahatahttps://review.openstack.org/#/c/126552/19:33
yamahatayou requested clean up db state at downgrade.19:33
yamahataI think the though itself is valid19:34
yamahatabut it's not easily cleaned up the db because NSX plugin already uses/populates it.19:34
*** yfried|afk is now known as yfried|pto19:35
yamahatathe request itself is valid.19:35
*** ronis has quit IRC19:35
yamahataSo in Kilo cycle it is better to do nothing at downgrade and address issues in Liberty cycle.19:36
ijwIt's not the end of the world, which is why I didn't -1 the patch19:36
yamahataijw: anyway I added my comments, so please give your opinion there.19:36
HenryGamuller: is the pep8 job broken until pylint is pinned?19:37
amullerHenryG: yes19:37
amullerwell19:37
amullerno19:37
yamahataijw: got your position. thanks19:37
amullerHenryG: until this is merged https://review.openstack.org/#/c/163493/19:37
*** achanda has joined #openstack-neutron19:38
amullerHenryG: Looks like that'll take another 2 hours19:38
*** rushil has joined #openstack-neutron19:38
HenryGamuller: thanks19:38
*** fredlhsu has joined #openstack-neutron19:39
*** banix has quit IRC19:40
openstackgerritPaul Michali proposed openstack/neutron: Remove dead code and PEP8 fix  https://review.openstack.org/16358619:41
*** chandankumar has joined #openstack-neutron19:41
*** s3wong has quit IRC19:42
amullerrussellb: https://bugs.launchpad.net/neutron/+bug/143098419:43
openstackLaunchpad bug 1430984 in neutron "Recent RPC namespacing breaks rolling upgrades" [Undecided,New] - Assigned to Assaf Muller (amuller)19:43
openstackgerritSumit Naiksatam proposed openstack/neutron-fwaas: Provide hooks for FWaaS repo functional gate  https://review.openstack.org/16147919:43
*** nlahouti has quit IRC19:43
amullerrussellb: I'd love your feedback before I move forward with a fix. Specifically about adding an optional flag in oslo messaging VS fixing it in Neutron (somehow, not clear how, apart from not using namespaces)19:43
*** fredlhsu has quit IRC19:44
*** sballe__ has joined #openstack-neutron19:45
*** fredlhsu has joined #openstack-neutron19:45
*** jorgem has joined #openstack-neutron19:46
*** baoli has quit IRC19:46
*** annegentle has quit IRC19:46
*** annegentle has joined #openstack-neutron19:47
*** cbouch has quit IRC19:47
*** s3wong has joined #openstack-neutron19:47
*** irenab has joined #openstack-neutron19:48
*** leenheer has quit IRC19:50
*** Swami has joined #openstack-neutron19:51
*** claudiub has joined #openstack-neutron19:51
*** adelinatuvenie has quit IRC19:51
*** leenheer has joined #openstack-neutron19:52
*** annegentle has quit IRC19:52
*** Longgeek has joined #openstack-neutron19:54
*** irenab has quit IRC19:55
pc_marmax: Hi19:55
armaxpc_m: hi19:55
*** s3wong has quit IRC19:56
*** s3wong has joined #openstack-neutron19:56
pc_marmax: Had some questions about imports for repos outside of openstack (related to the vendor decomp). Have time?19:56
armaxpc_m: I was going to lunch19:57
armaxpc_m: shall I ping you later?19:57
*** kevinbenton has joined #openstack-neutron19:57
pc_marmax: Yes please, I'll be online for another 90 mins.19:57
armaxok19:57
pc_mthanks19:57
armaxpc_m: I’ll be quick19:57
*** ajo has quit IRC20:00
*** Longgeek has quit IRC20:00
*** mrsmith has joined #openstack-neutron20:04
*** SridharG has joined #openstack-neutron20:06
*** SridharGaddam has quit IRC20:06
*** dboik_ has quit IRC20:06
*** dboik has joined #openstack-neutron20:07
*** annegentle has joined #openstack-neutron20:08
*** bquan_ has joined #openstack-neutron20:12
*** radez_g0n3 is now known as radez20:13
*** Flcn_ has joined #openstack-neutron20:16
*** bquan has quit IRC20:16
openstackgerritEd Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets  https://review.openstack.org/16359420:18
*** ihrachyshka has joined #openstack-neutron20:19
*** Marga_ has joined #openstack-neutron20:20
*** Flcn_ has quit IRC20:20
*** sthillma has quit IRC20:22
*** Flcn_ has joined #openstack-neutron20:24
*** chandankumar has quit IRC20:24
*** yfried|pto is now known as yfried|afk20:26
*** s3wong has quit IRC20:29
*** shwetaap has quit IRC20:29
*** ChuckC_ has quit IRC20:29
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961820:30
*** mwagner_lap has quit IRC20:30
*** trad511 has joined #openstack-neutron20:30
russellbamuller: k, will respond20:31
*** s3wong has joined #openstack-neutron20:32
armaxpc_m: ping20:32
pc_marmax: Hi20:32
pc_mSo, in the neutron-vpnaas repo, there are vendor drivers for VPN.20:33
armaxok20:33
*** carlp has joined #openstack-neutron20:33
pc_mIf they reference a vendor import out of tree (stackforge), how does that package get included?20:33
pc_mFor example, there is a driver that includes networking_brocade.20:34
pc_mI'm wondering how, when stacking, that package is made available.20:34
pc_mI don't see it in the VPN or neutron requirements.txt files. Trying to understand.20:35
armaxpc_m: have you seen how the extra hooks work?20:35
*** jistr has joined #openstack-neutron20:35
pc_marmax: no20:35
pc_mhaven't heard of them.20:35
armaxpc_m: https://github.com/openstack/neutron/blob/master/doc/source/devref/contribute.rst#devstack-integration-strategies20:36
*** elmiko has joined #openstack-neutron20:36
openstackgerritZengfa Gao proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method  https://review.openstack.org/15410320:36
pc_marmax: looking...20:36
elmikohey folks, i'm having some issues with traffic routing and devstack. i'm curious if there is a way to inject a rule into the neutron-openvswi-local chain through the config file or something?20:36
armaxpc_m: that’s the extra hook specific info: http://docs.openstack.org/developer/devstack/plugins.html#extras-d-hooks20:37
*** marun has joined #openstack-neutron20:37
*** natarajk has left #openstack-neutron20:40
haleybelmiko: not really, and if you do it manually the agent will remove it next time it updates things20:40
elmikohaleyb: yea, that's what i'm seeing20:42
*** VijayB has quit IRC20:42
*** ajo has joined #openstack-neutron20:42
*** salv-orlando has quit IRC20:43
elmikohaleyb: i'm having this issue where traffic routed from outside the box is not making it to the instances in my virtual network inside the box20:43
pc_marmax: So one could install the package as part of the extra hook?20:43
elmikohaleyb: is there another way to attack this?20:43
*** carlp has quit IRC20:45
*** yfried|afk is now known as yfried|pto20:45
*** nlahouti has joined #openstack-neutron20:45
haleybelmiko: why isn't the traffic going through the neutron router?20:46
elmikohaleyb: i'm not exactly sure, my physnet is 10.0.1.0 and the virt net is 10.1.0.0. i can see traffic making it to the box (10.0.1.0) but it's getting rejected with icmp host unreachable20:47
elmikohaleyb: if i add a rule to accept traffic to the subnet (10.1.0.0) then everything flows ok20:47
elmikohaleyb: i do see rules in the FORWARD chain that look like they are trying to match traffic based on the physnet tap device, but i don't think it's catching the traffic20:48
russellbamuller: responded on the bug20:48
haleybelmiko: this isn't a missing SG rule, right?20:48
elmikohaleyb: no, my sg accepts everything from everywhere20:49
elmikohaleyb: same for egress20:49
elmikohaleyb: this is with devstack too (in case that matters)20:49
*** irenab has joined #openstack-neutron20:50
*** jprovazn has quit IRC20:51
pc_marmax:I'm still a bit confused. I see there is this enable_plugin mechanism, that seems to be able to setup for an externally hosted plugin.20:51
haleybelmiko: it probably does.  typically devstack will install a network route for 10.0.0.0/24, but if add a new subnet it won't add one for it.  perhaps it's that?  I'm assuming if you ping from the router namespace it works?20:51
*** VijayB has joined #openstack-neutron20:52
pc_marmax: However, I don't see how some of the existing imports, like networking_brocade and networking_cisco, see in VPN repo are installed (don't see the enable_plugin for these).20:52
elmikohaleyb: hmm. i can ping from the router namespace.20:52
elmikohaleyb: i have added the subnet into my local.conf. maybe i'm just missing some config value here.20:52
amullerrussellb: Thank you20:53
elmikohaleyb: i've got FLOATING_RANGE=10.1.0.0/24 in my conf, that should do the trick?20:53
pc_marmax: Can you elaborate?20:53
haleybelmiko: so the private range is the same as the floating range?  i'm getting confused20:54
elmikohaleyb: no, priv range is 10.2.0.020:54
elmikohaleyb: i've got my main physnet (10.0.1.0) which my devstack machine lives on, then a public float net (10.1.0.0) and a private net (10.2.0.0)20:55
haleybelmiko: ok, up above you said your virt range was 10.1.0.020:55
*** achanda has quit IRC20:55
elmikohaleyb: maybe bad verabage on my part. sorry for the confusion20:56
haleybelmiko: and you're trying to ping, say, 10.2.0.100 ?20:56
elmikohaleyb: right20:56
elmikohaleyb: er, actually it would be ping 10.1.0.10020:56
elmikohaleyb: from say, 10.0.1.10020:56
*** nati_ueno has joined #openstack-neutron20:56
haleybelmiko: so you've assigned a floating ip to the port?20:57
*** thomasem has quit IRC20:57
elmikohaleyb: yes20:57
elmikohaleyb: the router is showing 10.1.0.1 (on the public float) and 10.2.0.1 (on the priv net)20:57
*** raildo has quit IRC20:58
haleybelmiko: and 'ip r g 10.1.0.100' shows next hop as 10.1.0.1 ?20:58
elmikohaleyb: let me double check20:58
*** carlp has joined #openstack-neutron20:58
haleybor this is another problem with single-interface devstack that someone is fixing20:58
elmikohmm could be the latter20:59
elmikoso on my machine (10.0.1.110), `ip r g 10.1.0.100` shows 10.0.1.151 as the next hop. 10.0.1.151 is the machine hosting devstack21:00
*** Swami has quit IRC21:00
armaxpc_m: sorry I stepped away for a bit, you still there?21:00
pc_myes21:00
armaxpc_m: they won’t be intsalled automatically via requirements.txt if that’s what you’re asking21:01
armaxpc_m: just the same way it’s always been21:01
*** jpena has joined #openstack-neutron21:01
elmikohaleyb: on my local machine (10.0.1.110) i setup that route manually21:01
*** VijayB has quit IRC21:01
pc_marmax: I'm looking at neutron-vpnaas, and trying to understand how, say networking_brocade is installed, when one stacks or runs a unit test, so that the import works.21:03
pc_marmax: I don't see where the package is installed.21:03
*** vmtrooper has joined #openstack-neutron21:03
*** thomasem has joined #openstack-neutron21:03
haleybelmiko: maybe look at https://review.openstack.org/#/c/158424/14/lib/neutron ?  Although I typically don't have a problem with devstack, but i'm usually running with default ranges21:04
elmikohaleyb: thanks for the help! i'll keep digging =)21:04
armaxunit tests is different from, say running devstack21:04
*** thomasem_ has joined #openstack-neutron21:05
*** r-daneel has quit IRC21:05
elmikohaleyb: usually i don't have issue with devstack either, but i'm trying something weird. i want to host a local dev service on my laptop and communicate with the devstack on a different vm in my network.21:05
armaxpc_m: do you have a specific concern in mind, like a patch for instance?21:05
armaxpc_m: it’ll help me understand the scope or context of your request21:05
elmikohaleyb: i think there might be other issues though because i'm seeing that some of the instances on the private network are not able to communicate with each other via their assigned floating ips21:06
pc_marmax: Well, trying to understand how it works. No specific patch, but I have a case where I need to understand this...21:06
haleybelmiko: so a VM outside of devstack?  Yeah, it would need a route added21:06
*** irenab has quit IRC21:07
armaxpc_m: I am a bit confused it was said that adv svcs wouldn’t decompose drivers21:07
elmikohaleyb: no no, all the VMs are inside devstack. but the devstack itself is in a VM.21:07
elmikohaleyb: my service is external to the devstack21:07
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869821:07
elmikohaleyb: and on a different network21:07
pc_marmax: Understood. At this time they aren't but some vendors are starting to pull out service code.21:08
*** yfauser has joined #openstack-neutron21:08
pc_marmax: Brocade has done it, and Cisco is considering.21:08
*** vmtrooper has quit IRC21:08
*** thomasem has quit IRC21:08
armaxpc_m: ok21:08
*** yfauser has left #openstack-neutron21:08
pc_mSo, if you look at brocade code, they have an import with networking_brocade.21:08
haleybelmiko: if you need to get from inside a devstack VM to an external IP you might need that patch, depends on your config21:09
pc_mIn the cisco driver in VPN it uses cisco plugin code that has just been decomposed and pulled out.21:09
elmikohaleyb: ack, i'll do some reading =)21:09
*** tellesnobrega_ has joined #openstack-neutron21:09
Rajeev_elmiko: one quick thing but otherwise back to haleyb: from the router namespace, are you able to ping outside as well as the VM ?21:09
*** Sayaji has joined #openstack-neutron21:09
pc_marmax: The code in VPN broke and a temp fix was to remove the import (to neutron as it is gone) and duplicate the define used.21:10
pc_marmax: https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/service_drivers/cisco_ipsec.py#L3821:10
haleybelmiko: it was discussed on the -dev ML last week if it's the same issue21:10
*** jistr has quit IRC21:10
pc_marmax: So, if we put the new networking_cisco import in, is that all that is needed? How does that package get installed, so that UTs pass?21:11
*** VijayB has joined #openstack-neutron21:11
*** tellesnobrega_ has quit IRC21:11
pc_marmax: That's the reasoning for the whole discussion. Am I making sense?21:11
armaxpc_m: this is true for other drivers in the neutron tree21:11
armaxnot quite21:11
elmikoRajeev_: i am able to ping from either side of the router, although i am noticing that some instances are having difficulty21:11
pc_marmax: :(21:11
pc_marmax: Let me try again...21:11
pc_marmax: In the link posted. It was importing from neutron/plugins/cisco/...21:12
pc_marmax: That code was moved out to stackforge and hence broke VPN code in neutron-vpnaas21:12
*** jckasper has quit IRC21:12
*** mlavalle has quit IRC21:12
*** andreykurilin has quit IRC21:12
pc_marmax: The workaround was to hard code the needed define.21:12
pc_marmax: I'm guessing the right fix is to import the new networking_cisco.blah that has the needed info.21:13
*** nati_ueno has quit IRC21:13
*** nati_ueno has joined #openstack-neutron21:13
pc_marmax: I'm wondering what else is needed if I do that. Will the package be there, when someone stacks with this device driver loaded?21:13
pc_marmax: Will unit tests on this module continue to run OK?21:14
*** jckasper has joined #openstack-neutron21:14
armaxpc_m: /me looking21:14
pc_marmax: Here's the workaround:  https://github.com/openstack/neutron-vpnaas/commit/f45fb8af13d72f5fd8fe2420a58c7aaf85e49525#diff-c1a27ad5586724ae053ff5dd867b801b21:15
armaxpc_m: I think the right fix is to minimize the dependency21:15
armaxlet’s resume this discussion later if you can21:16
*** matrohon has quit IRC21:16
armaxI need to step away now21:16
*** andreykurilin has joined #openstack-neutron21:16
pc_marmax: I can't today.  Can resume tomorrow, or via email.21:16
*** tiswanso has quit IRC21:16
armaxpc_m: tomorrow it is21:16
pc_marmax: Would like to understand what you mean by min the dependency.21:16
pc_marmax: OK. Tomorrow.21:16
kevinbentonarmax: hello!21:16
armaxpc_m: thanks21:17
pc_marmax: Thanks for listening to my concern!21:17
pc_mttyl21:18
*** pc_m has quit IRC21:20
*** bquan_ has quit IRC21:20
amullermarun: https://review.openstack.org/#/c/125384/21:21
*** kbringard has quit IRC21:21
*** yfried|pto is now known as yfried|afk21:22
SridharRamaswamypc_m: ping21:22
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433921:22
*** leenheer has quit IRC21:23
*** pritesh_ has joined #openstack-neutron21:24
*** yfried|afk is now known as yfried|pto21:25
SridharRamaswamypc_m: armax: just saw the above discussion on UT passing for decomposed service drivers. We got that working by mocking the networking_brocade imports. See #link https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/tests/unit/services/vpn/device_drivers/test_vyatta_ipsec.py21:26
marunamuller: http://governance.openstack.org/reference/new-projects-requirements.html21:27
*** achanda has joined #openstack-neutron21:28
*** salv-orlando has joined #openstack-neutron21:28
*** boris-42 has quit IRC21:32
*** jpena has quit IRC21:32
*** Networkn3rd has quit IRC21:33
*** sripriya has quit IRC21:34
*** sripriya has joined #openstack-neutron21:34
*** ijw has quit IRC21:34
kevinbentonsalv-orlando: are you around?21:36
*** shivharis has quit IRC21:37
*** shivharis has joined #openstack-neutron21:38
*** yfried|pto is now known as yfried|afk21:38
*** rushil_ has joined #openstack-neutron21:38
kevinbentonmarun: you've been involved with Neutron forever, maybe you can answer my question :)21:39
marunkevinbenton: I can certainly try.21:39
kevinbentonmarun: why is the creation of the DHCP port deferred until another port is created on the network?21:39
* sc68cal grabs a chair so he can learn21:39
*** rushil has quit IRC21:39
*** aepifanov has quit IRC21:39
*** radez is now known as radez_g0n321:40
marunnot entirely sure of the context21:40
amullerkevinbenton: it's an optimization21:40
kevinbentonamuller: not a good one :)21:40
amullerkevinbenton: like how legacy routers aren't scheduled until they have an interface21:41
amullerkevinbenton: btw are you sure it's the DHCP port and not the DHCP scheduling?21:41
kevinbentonamuller: same thing for this purpose21:41
kevinbentonamuller: those aren't equivalent21:41
amullerabsolutely21:41
kevinbentonamuller: a router isn't scheduled when it's not attached to a network, that's fine21:42
*** baoli has joined #openstack-neutron21:42
openstackgerritAngela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX  https://review.openstack.org/16148921:42
kevinbentonamuller: but dhcp isn't brought online for a network even though the subnet has dhcp enabled21:42
openstackgerritAngela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches.  https://review.openstack.org/15470921:42
*** apuimedo has quit IRC21:43
kevinbentonamuller: this creates an annoying race condition for systems under heavy load where a network/subnet that was created a long time ago doesn't bring DHCP online fast enough for a VM once its added21:43
openstackgerritMerged openstack/neutron: Fix pylint issue with type VS isinstance in event_observers  https://review.openstack.org/16349321:43
kevinbentonamuller: no matter how long you wait after creating the subnet, you still have to hope dhcp is provisioned faster than your first vm sends a DHCP request21:44
*** baoli has quit IRC21:44
marunkevinbenton: we should be waiting for dhcp provisioning, same as we wait for port wiring21:44
amuller++21:45
marunkevinbenton: no amount of optimization will save us once load gets high enough21:45
*** baoli has joined #openstack-neutron21:45
openstackgerritMichael Smith proposed openstack/neutron: delay fip namespace cleanup/destroy  https://review.openstack.org/15175821:45
kevinbentonmarun: we should do that on the subnet creation then, instead of on the first port21:45
marunkevinbenton: if you're interested in addressing this, feel free to look into something like dhcping configurably executed by the l2 agent21:46
marunkevinbenton: so that the existing notification pathway can be reused21:46
kevinbentonmarun: i'm not familiar with how that's used, what does it do in the L2 agent?21:47
openstackgerritSumit Naiksatam proposed openstack/python-neutronclient: Updating FWaaS CLI to allow router references  https://review.openstack.org/15811821:47
kevinbentonmarun: i don't get any hits grepping for dhcping in the neutron repo21:48
*** jckasper has quit IRC21:48
kevinbentonmarun: there isn't a clean place to put this barrier21:50
kevinbentonmarun: if we do it at the port level21:50
*** amotoki has quit IRC21:50
kevinbentonmarun: because the L2 agent doesn't know if the attached thing is going to use DHCP21:50
marunkevinbenton: that can be remedied21:50
*** banix has joined #openstack-neutron21:51
kevinbentonmarun: what did you have in mind?21:51
kevinbentonthe way I see it, the L2 agent is only responsible for wiring a port into a network, once that's done, it should report the port status as up21:53
*** fredlhsu has quit IRC21:53
*** yfried|afk is now known as yfried|pto21:58
*** arif-ali has quit IRC21:58
*** sripriya has quit IRC22:00
*** marun has quit IRC22:01
*** nati_ueno has quit IRC22:02
*** Flcn__ has joined #openstack-neutron22:02
SayajiTrying to understand allowed address pairs extension, would anyone be able to help me out ?22:02
*** sripriya has joined #openstack-neutron22:03
*** _cjones_ has quit IRC22:03
*** _cjones_ has joined #openstack-neutron22:03
amullerkevinbenton: Sorry running an errand but this is an area that interests me as well22:04
amullerkevinbenton: I'll try to catch you later22:04
kevinbentonamuller: sounds good22:04
kevinbentonamuller: i'll try to get a hold of Maru to see what he was implying22:05
*** Flcn_ has quit IRC22:05
kevinbentoncarl_baldwin: do you have a sec to chat about IPAM?22:05
amullerand the holy spirit...22:05
*** SridharRamaswamy has quit IRC22:05
carl_baldwinkevinbenton: I have a couple of minutes.22:05
kevinbentoncarl_baldwin: i'm looking at the uniqueness of subnets within a tenant thread22:05
*** annegentle has quit IRC22:05
kevinbentoncarl_baldwin: there is nothing in Neutron scoped to a tenant other than quotas, i would really like to avoid subnets being special in this case22:06
kevinbentoncarl_baldwin: i.e. right now i can create a thousand subnet/network/router architectures under 1 tenant or 1000 different tenants and they are the same22:07
kevinbentoncarl_baldwin: but IIUC IPAM is going to break that22:07
kevinbentoncarl_baldwin: is that correct?22:07
*** baoli has quit IRC22:08
*** yfried|pto is now known as yfried|afk22:08
carl_baldwinkevinbenton: No, it won’t.  The thread’s purpose was to see if we could do that.  However, the feedback has shown that we shouldn’t.22:08
*** csoukup has joined #openstack-neutron22:09
carl_baldwinkevinbenton: So, the current plan is to support backward compatibility as long as the new subnet pool feature is not used.22:09
carl_baldwinkevinbenton: … in which case, addresses will be scope to the subnet pool.22:10
*** arif-ali has joined #openstack-neutron22:10
carl_baldwinkevinbenton: Which makes sense to me.22:10
kevinbentoncarl_baldwin: do you have a link to the data model? i'm not seeing the connection to a tenant22:10
kevinbentoncarl_baldwin: i'm not familiar with the IPAM details22:10
*** amuller has quit IRC22:11
*** dboik has quit IRC22:11
*** bandarji has joined #openstack-neutron22:11
*** dims has quit IRC22:11
carl_baldwinkevinbenton: Sounds like we may need to connect tomorrow on this.  Unfortunately, I’ve got to duck out early today.22:13
*** baoli has joined #openstack-neutron22:13
carl_baldwinkevinbenton: Short answer is that there is no data model for these backward compatibility things we call “implicit pools”.  they just exist in code.22:14
kevinbentoncarl_baldwin: hmm. can i put this on the mailing list so we can continue asynchronously?22:14
carl_baldwinkevinbenton: Sounds good.22:14
openstackgerritEd Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets  https://review.openstack.org/16359422:14
kevinbentoncarl_baldwin: thanks for your time22:15
*** ijw has joined #openstack-neutron22:15
*** ijw has quit IRC22:15
*** arif-ali has quit IRC22:15
carl_baldwinkevinbenton: glad to help22:16
*** carl_baldwin has quit IRC22:16
openstackgerritYanping Qu proposed openstack/neutron-fwaas: Adds FWaaS driver for Cisco CSR1kv  https://review.openstack.org/16101922:17
*** ijw has joined #openstack-neutron22:17
openstackgerritpritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes  https://review.openstack.org/15842022:20
*** arif-ali has joined #openstack-neutron22:20
*** annegentle has joined #openstack-neutron22:25
*** pritesh_ has quit IRC22:26
*** mlavalle has joined #openstack-neutron22:27
*** fredlhsu has joined #openstack-neutron22:29
*** jaypipes-afk is now known as jaypipes22:31
*** dboik has joined #openstack-neutron22:34
*** annegentle has quit IRC22:34
*** dboik_ has joined #openstack-neutron22:34
salv-orlandokevinbenton: yes I am22:35
*** jgrimm is now known as zz_jgrimm22:36
*** dboik has quit IRC22:38
kevinbentonsalv-orlando: had a question earlier about why scheduling a subnet to a DHCP agent was deferred22:39
kevinbentonsalv-orlando: until the first VM port was created22:39
kevinbentonsalv-orlando: was there some scaling issue that led to that?22:39
salv-orlandokevinbenton: nope.22:40
*** tellesnobrega_ has joined #openstack-neutron22:40
salv-orlandosimply opinionated developers from what I recall22:40
kevinbentonsalv-orlando: what was the reason?22:40
*** tellesnobrega_ has quit IRC22:41
salv-orlandothat creating a dnsmasq instance for a network with no vm port yet was a waste22:41
salv-orlandokevinbenton: I did not really subscribe to that point. But on the other hand I have no strong argument against it22:41
*** banix has quit IRC22:41
kevinbentonsalv-orlando: a waste of what?22:42
salv-orlandokevinbenton: a namespace on the dhcp agent22:42
salv-orlandoand I guess resource for running dnsmasq (!)22:42
kevinbentonsalv-orlando: but if the dhcp agent can't handle that many namespaces, shouldn't the creation be blocked rather than exploding once a VM is started on the network?22:43
kevinbentoncreation of the subnet*22:43
salv-orlandoI said, the reason was actually over-zealous development. You know those situations where devs spend a lot of time thinking about corner cases and thn forget to properly do things such as transaction management22:43
salv-orlandokevinbenton: that is a rather valid point. However, the explosion you refer to, even if likely22:44
*** thedodd has quit IRC22:44
salv-orlandois not reported in the user in any way22:44
salv-orlandobut if you want to push for a change in that logic, I'd be supportive. It's much easier to handle22:45
kevinbentonsalv-orlando: right, so i was thinking something along the following lines22:45
kevinbentonsalv-orlando: during subnet create, if dhcp, set status to DOWN22:45
*** Longgeek has joined #openstack-neutron22:45
kevinbentonsalv-orlando: schedule dhcp agent22:45
_cjones_Hey guys. I just started playing around with LBaaS and had a quick development question.22:46
kevinbentonsalv-orlando: once dhcp port is active, set subnet status to ACTIVE22:46
*** dims_ has joined #openstack-neutron22:46
kevinbentonsalv-orlando: then we can worry later about nova doing a wait if the subnet status is DOWN22:46
_cjones_I notice when a VIP is created, it trickles down to create a port with a name of 'vip-{uuid}'. Does this UUID correspond with object in particular?22:46
salv-orlandokevinbenton: for the subnet status  the DHCP port is one aspect. We need a way to verify the sanity of the dnsmasq instance22:47
salv-orlandokevinbenton: for instance - is it running? does it have the appropriate configuration?22:47
kevinbentonsalv-orlando: oh right, DHCP port status just indicates that the port was wired...22:47
salv-orlandoI would at least - as a first step - mark the subnet as ACTIVE if DHCP port is active && dnsmasq was successfully started22:48
*** jckasper has joined #openstack-neutron22:48
salv-orlandofor the latter we might need some RPC changes, probably22:48
salv-orlandobecause the notification to the DHCP agent is, as of today, a fire & forget event22:48
kevinbentonsalv-orlando: right, i don't remember seeing any code for a notification from the agent back to the server22:49
*** nati_ueno has joined #openstack-neutron22:50
*** Flcn__ has quit IRC22:50
*** yfauser has joined #openstack-neutron22:50
*** yfauser has left #openstack-neutron22:50
*** amotoki has joined #openstack-neutron22:50
openstackgerritItsuro Oda proposed openstack/neutron: Make floatingip reachable from the same network  https://review.openstack.org/16194722:51
kevinbentonsalv-orlando: that's going to require quite a few changes the morw i think about it22:51
kevinbentonsalv-orlando: what exactly would the dhcp agent notify the server of?22:52
*** gildub has joined #openstack-neutron22:52
*** nati_uen_ has joined #openstack-neutron22:52
*** vmtrooper has joined #openstack-neutron22:52
*** banix has joined #openstack-neutron22:52
kevinbentonsalv-orlando: what if that notification is dropped?22:52
kevinbentonsalv-orlando: we could jam it into the agent state22:53
*** banix has quit IRC22:53
*** jckasper has quit IRC22:53
kevinbentonsalv-orlando: 'served_networks' as a list of network_ids with dnsmasq isntances22:53
salv-orlandoif you drop the notification then how would you react to port create events?22:54
openstackgerritMerged openstack/neutron: Change linux/ip_lib code to better handle address families  https://review.openstack.org/15755522:54
*** Longgeek has quit IRC22:55
*** SridharRamaswamy has joined #openstack-neutron22:55
kevinbentonsalv-orlando: i was just thinking one lost update22:55
*** nati_ueno has quit IRC22:56
*** amotoki has quit IRC22:56
kevinbentonsalv-orlando: the rest of the agent stuff has some periodic syncing stuff to handle unstable message buses22:56
salv-orlandokevinbenton: sorry when you say "dropped" I understood you want to drop the notification from the code22:57
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR qrouters not created on compute Node when vm added  https://review.openstack.org/14914322:57
kevinbentonsalv-orlando: oh, no. i just meant the notification gets lost22:57
salv-orlandokevinbenton: you're right, if notifications are dropped "eventually" the status will converge due to periodic resync22:57
*** vmtrooper has quit IRC22:57
salv-orlandokevinbenton: where eventually means actually eventually and not "eventually consistent"22:57
kevinbentonsalv-orlando: right22:58
kevinbentonsalv-orlando: let me think about this a bit more22:58
kevinbentonsalv-orlando: changing topics, carl_baldwin had to step away before he could explain the IPAM model to me22:59
*** baoli has quit IRC22:59
kevinbentonsalv-orlando: why does it require uniqueness of CIDRs within a tenant?22:59
*** SridharRamaswam1 has joined #openstack-neutron23:01
salv-orlandokevinbenton: Basically the concept is that there is a lack of a concept of l3 address pool. Which is a prefix, or something like that, from which all the addresses in a l3 domain are taken,.23:02
salv-orlandoAddresses in this domain should not overlap, as this defies the uniqueness required in a single l3 domain, which in neutron is realized by interconneting networks with routers23:03
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Don't call reschedule router with single L3 agent  https://review.openstack.org/14356723:03
*** leenheer has joined #openstack-neutron23:03
*** SridharRamaswamy has quit IRC23:03
salv-orlandowhat is coming out from this thread is that this domain scope should be the tenant itself. As you correctly point out a tenant does not mean anything in neutron23:03
salv-orlandoI reckon that this is a misunderstanding that has originated from a very specific issue I am facing with this new IPAM interface.23:04
salv-orlandoin making it work with the current IPAM model - which is being reimplemented in an IPAM driver.23:04
*** apuimedo has joined #openstack-neutron23:04
kevinbentonsalv-orlando: ah, then why not have an 'l3 domain' object instead of overloading the tenant?23:05
salv-orlandokevinbenton: Basically this IPAM interface assumes the existence of such domains - introducing the concept of subnet pools23:05
salv-orlandobut since the current API has not such concept, I am in bit of a pickle when implementing an IPAM driver which behaves exactly as today's driver23:06
salv-orlandoand this idea of restricting overlapping IPs came out - but it seems it cannot be brought forward23:06
*** fredlhsu has quit IRC23:07
salv-orlandokevinbenton: so I am going back to the drawing board and find a solution, but I think I will have to support subnetpools in this reference ipam driver. Which means that people which move to it will have to use necessarily subnet pools as well, which means that deprecation of the old baked IPAM will never happen, which means that things are turning ugly :(23:08
*** claired has joined #openstack-neutron23:09
*** jorgem has quit IRC23:09
kevinbentonsalv-orlando: can we just have the create subnet call result in a new pool simultaneously for the old version?23:10
sc68calkevinbenton, salv-orlando: I'll also note that dnsmasq has a cute trick where if Bad Things happen in the process of configuring - it will totally just sit there and stop offering leases - I don't think dnsmasq has good error reporting features23:10
salv-orlandokevinbenton: yeah I thought about that as a "degenerate solution".23:10
salv-orlandoI am actually trying something along those lines, by having the IPAM driver implement subnet pools, but hiding them23:11
sc68calmay influence RPC23:11
kevinbentonsc68cal: i noticed that when i was testing some other dhcp options :(23:11
kevinbentonsalv-orlando: ok23:11
kevinbentonsalv-orlando: i'm starting to understand now23:12
sc68calkevinbenton: like MTU? ;)23:12
salv-orlandosc68cal: rgith. I had hald an idea of  having a local process which generates fake DHCPREQUEST messages just to "heartbeat" dnsqmas23:12
kevinbentonsc68cal: not quite yet, was just testing some lease length options23:12
salv-orlandobut as with pretty much everything in my life, I never got around to do that23:12
kevinbentonsalv-orlando: i like that idea23:13
* sc68cal has to adjust his bouncer, one whole hour gap in my playback23:13
* sc68cal feeds his znc more memory23:13
kevinbentonsc68cal: i though that was a bit delayed :)23:13
openstackgerritIhar Hrachyshka proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method  https://review.openstack.org/15410323:14
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed  https://review.openstack.org/16226023:14
kevinbentonsalv-orlando: maybe we could have the tap interface itself ask for addresses from its dnsmasq process :)23:14
kevinbentonsalv-orlando: that would probably be more complex than a dhcpping process23:15
*** hichihara has joined #openstack-neutron23:16
openstackgerritvishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers'  https://review.openstack.org/16188123:17
openstackgerritIhar Hrachyshka proposed openstack/neutron: Migrate to oslo.log  https://review.openstack.org/15963823:19
*** jckasper has joined #openstack-neutron23:20
*** jckasper has quit IRC23:25
*** annegentle has joined #openstack-neutron23:32
*** sbfox has quit IRC23:33
*** ihrachyshka has quit IRC23:34
*** jckasper has joined #openstack-neutron23:38
*** chlong has joined #openstack-neutron23:41
*** annegentle has quit IRC23:43
*** jckasper has quit IRC23:44
*** absubram has quit IRC23:44
*** vivek-ebay has joined #openstack-neutron23:45
*** mrsmith has quit IRC23:48
openstackgerritSripriya Seetharam proposed openstack/neutron: Brocade Vyatta vrouter shim plugin for vendor decomposition  https://review.openstack.org/15905623:51
*** ajmiller has quit IRC23:51
*** wwriverrat has joined #openstack-neutron23:53
*** vivek-ebay has quit IRC23:54
*** SridharRamaswam1 has quit IRC23:56
*** jckasper has joined #openstack-neutron23:57
*** tflynn_ has joined #openstack-neutron23:57
*** amuller has joined #openstack-neutron23:59
openstackgerritMerged openstack/neutron: Raise QuotaResourceUnknown in the quota engine  https://review.openstack.org/16317123:59

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