Friday, 2016-08-12

*** yb has joined #openstack-neutron00:02
*** itsuugo has quit IRC00:04
*** SumitNaiksatam has quit IRC00:15
*** hoangcx2 has quit IRC00:16
*** hoangcx has joined #openstack-neutron00:16
*** baoli has joined #openstack-neutron00:17
*** wolverin_ has joined #openstack-neutron00:19
*** iwamoto has joined #openstack-neutron00:20
*** Swami has quit IRC00:20
*** banix has joined #openstack-neutron00:21
*** wolverineav has quit IRC00:23
*** gouthamr has joined #openstack-neutron00:23
*** mmalik4 has quit IRC00:25
*** rajinir has quit IRC00:35
*** abhiraut1 has joined #openstack-neutron00:36
*** abhiraut has quit IRC00:36
*** mlavalle has quit IRC00:42
*** janzian has quit IRC00:42
*** abhiraut1 has quit IRC00:50
*** abhiraut has joined #openstack-neutron00:52
openstackgerritQunyingRan proposed openstack/neutron: Add error informations for users when value is invalid in database  https://review.openstack.org/35049100:53
*** davidlenwell has quit IRC00:53
openstackgerritzhangyanxian proposed openstack/neutron: Fix some spelling errors in net_helpers.py  https://review.openstack.org/35446800:54
openstackgerritzhangyanxian proposed openstack/neutron: Fix some spelling errors in net_helpers.py  https://review.openstack.org/35446800:55
*** banix has quit IRC00:58
*** banix has joined #openstack-neutron00:58
*** banix has quit IRC01:05
*** jckasper has quit IRC01:05
*** jckasper has joined #openstack-neutron01:05
*** davidlenwell has joined #openstack-neutron01:05
*** gouthamr_ has joined #openstack-neutron01:06
*** gouthamr has quit IRC01:10
*** abhiraut has quit IRC01:10
*** jckasper has quit IRC01:10
*** abhiraut has joined #openstack-neutron01:10
*** vishwanathj has quit IRC01:10
*** tonytan4ever has joined #openstack-neutron01:11
*** wolverin_ has quit IRC01:11
*** kevo has quit IRC01:12
*** wolverineav has joined #openstack-neutron01:13
*** baoli has quit IRC01:13
*** gouthamr_ is now known as gouthamr01:14
*** abhiraut has quit IRC01:15
*** tonytan4ever has quit IRC01:16
*** wolverineav has quit IRC01:17
*** rpodolyaka_ has joined #openstack-neutron01:18
*** rpodolyaka has quit IRC01:21
*** rpodolyaka_ is now known as rpodolyaka01:21
openstackgerritZhigang Li proposed openstack/python-neutronclient: Fix the problem of mox in test_cli20_speaker.py  https://review.openstack.org/34201501:24
*** Leo_ has quit IRC01:26
HenryGarmax: kevinbenton: I know the gate is ill, but please spend a few seconds on https://review.openstack.org/35253101:28
*** baoli has joined #openstack-neutron01:29
*** EinstCrazy has joined #openstack-neutron01:30
*** vishwanathj has joined #openstack-neutron01:32
*** banix has joined #openstack-neutron01:33
openstackgerritgongysh proposed openstack/neutron: Use neutron lib project's Invalid exception type  https://review.openstack.org/35397601:33
*** s3wong has quit IRC01:34
*** stanzgy has joined #openstack-neutron01:37
*** mdih has joined #openstack-neutron01:40
*** bvandewa_ has quit IRC01:46
*** gvrangan has quit IRC01:49
*** jamesden_ has joined #openstack-neutron01:59
openstackgerritMiguel Lavalle proposed openstack/neutron: Publish segment id in port responses  https://review.openstack.org/35066502:01
*** yamahata has quit IRC02:02
*** jckasper has joined #openstack-neutron02:02
*** damia_pi has joined #openstack-neutron02:03
*** jamesden_ has quit IRC02:04
HenryGkevinbenton: how do I intercept a Class.tenant_id access?02:04
*** damia_pi has quit IRC02:06
*** jckasper has quit IRC02:07
*** yamahata has joined #openstack-neutron02:07
*** hoangcx has quit IRC02:07
HenryGkevinbenton: with __get__?02:08
HenryGseems extreme02:08
*** iyamahat has quit IRC02:09
*** a_ta has joined #openstack-neutron02:13
*** jamesdenton has quit IRC02:13
*** jamesdenton has joined #openstack-neutron02:13
*** gongysh has joined #openstack-neutron02:17
*** ivar-lazzaro has quit IRC02:17
*** bvandewa has joined #openstack-neutron02:17
HenryGdoesn't work - looks like sqlalchemy overrides __get__02:19
*** itsuugo has joined #openstack-neutron02:31
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115902:35
*** shashank_hegde has quit IRC02:38
*** EinstCrazy has quit IRC02:39
*** hoangcx has joined #openstack-neutron02:40
*** EinstCrazy has joined #openstack-neutron02:43
*** bvandewa has quit IRC02:50
*** lbrune has quit IRC02:51
*** bvandewa has joined #openstack-neutron02:52
*** fnaval has quit IRC02:52
*** lbrune has joined #openstack-neutron02:52
openstackgerritKengo Hobo proposed openstack/neutron: Add string validation on security group's name  https://review.openstack.org/35219002:52
openstackgerritHenry Gessau proposed openstack/neutron: Emit DeprecationWarning on using tenant_id in models  https://review.openstack.org/35253102:54
*** zhiyan has quit IRC02:55
*** johnsom has quit IRC02:55
*** ijw has quit IRC02:59
*** johnsom has joined #openstack-neutron03:00
openstackgerritchandanc proposed openstack/neutron: The Iptables manager and firewall driver in Neutron must be enhanced for co-existence of SecurityGroup and FWaaS v2 APIs.  This patch re-factors the IPTables driver for enabling FWaaS and SG chain to be interleaved preserving ordering of rules.  https://review.openstack.org/34817703:00
*** amotoki has joined #openstack-neutron03:02
*** hoangcx has quit IRC03:03
*** zhiyan has joined #openstack-neutron03:03
*** bvandewa has quit IRC03:06
openstackgerritMerged openstack/python-neutronclient: Add 'shared' option to neutron address-scope-update  https://review.openstack.org/27236503:07
*** fnaval has joined #openstack-neutron03:09
*** bvandewa has joined #openstack-neutron03:10
*** amotoki has quit IRC03:11
*** yamahata has quit IRC03:14
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: Faster _get_devices_details_list_and_failed_devices  https://review.openstack.org/28453503:16
*** yamamoto has joined #openstack-neutron03:17
*** amotoki has joined #openstack-neutron03:22
*** mickeys has joined #openstack-neutron03:23
*** mickeys has quit IRC03:27
*** EinstCra_ has joined #openstack-neutron03:29
*** amotoki has quit IRC03:31
*** EinstCrazy has quit IRC03:32
*** bvandewa has quit IRC03:32
*** links has joined #openstack-neutron03:33
*** gouthamr has quit IRC03:35
*** amotoki has joined #openstack-neutron03:35
*** jckasper has joined #openstack-neutron03:36
*** a_ta has quit IRC03:36
*** a_ta has joined #openstack-neutron03:37
*** a_ta_ has joined #openstack-neutron03:41
*** a_ta has quit IRC03:41
*** baoli has quit IRC03:43
*** azbiswas has joined #openstack-neutron03:46
*** numans has joined #openstack-neutron03:46
*** ramishra_ has joined #openstack-neutron03:49
*** ramishra has quit IRC03:49
openstackgerritHong Hui Xiao proposed openstack/neutron: Enable create and delete segments in ML2  https://review.openstack.org/31735803:50
*** hoangcx has joined #openstack-neutron03:51
*** hoangcx has quit IRC03:57
*** amotoki has quit IRC04:01
*** numans has quit IRC04:03
*** vishwanathj has quit IRC04:06
*** Sukhdev has joined #openstack-neutron04:09
*** amotoki has joined #openstack-neutron04:09
*** amotoki has quit IRC04:10
*** amotoki has joined #openstack-neutron04:11
*** vijaykc4 has joined #openstack-neutron04:11
*** tonytan4ever has joined #openstack-neutron04:12
*** itzikb has joined #openstack-neutron04:13
*** fragatina has quit IRC04:16
*** tonytan4ever has quit IRC04:17
*** yamamoto_ has joined #openstack-neutron04:18
*** yamamoto_ has quit IRC04:18
openstackgerritHenry Gessau proposed openstack/neutron-dynamic-routing: Remove temporary local HasProject  https://review.openstack.org/35450404:18
*** wolverineav has joined #openstack-neutron04:18
*** yamamoto has quit IRC04:18
*** yamamoto has joined #openstack-neutron04:19
*** yamamoto has quit IRC04:23
openstackgerritHenry Gessau proposed openstack/neutron-fwaas: Remove temporary local HasProject  https://review.openstack.org/35450604:25
*** vishwanathj has joined #openstack-neutron04:26
*** Shahid_ has joined #openstack-neutron04:26
*** vishwanathj has quit IRC04:26
*** gongysh has quit IRC04:26
*** jckasper has quit IRC04:29
*** jckasper has joined #openstack-neutron04:29
openstackgerritPaddu Krishnan proposed openstack/neutron: [WIP] Defining the RPC resources for FWaaS V2  https://review.openstack.org/34709204:32
*** jamesdenton has quit IRC04:33
*** jckasper has quit IRC04:34
*** gongysh has joined #openstack-neutron04:36
*** banix has quit IRC04:36
*** a_ta_ has quit IRC04:38
openstackgerritHenry Gessau proposed openstack/neutron-vpnaas: Remove temporary local HasProject  https://review.openstack.org/35451304:39
openstackgerritOpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/35190504:40
openstackgerritInessa Vasilevskaya proposed openstack/neutron: ovsfw: fix single port rule bad masking  https://review.openstack.org/35378204:43
*** ratailor has joined #openstack-neutron04:45
*** vijaykc4 has quit IRC04:48
openstackgerritMerged openstack/neutron: Always start transactions in quota cleanup methods  https://review.openstack.org/33922604:48
openstackgerritMerged openstack/neutron: ovs-agent: Seperate VLAN mapping outside of the agent  https://review.openstack.org/34999004:50
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/35445704:51
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-dynamic-routing: Updated from global requirements  https://review.openstack.org/35452304:51
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/35452404:51
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lib: Updated from global requirements  https://review.openstack.org/35445804:51
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/35452604:52
*** fnaval_ has joined #openstack-neutron04:54
*** Sukhdev has quit IRC04:54
openstackgerritbin proposed openstack/neutron: Add routed network scenario test cases  https://review.openstack.org/34718804:55
*** kobis has joined #openstack-neutron04:56
*** fnaval has quit IRC04:57
*** fragatina has joined #openstack-neutron04:58
*** fragatina has quit IRC05:03
*** hoangcx has joined #openstack-neutron05:03
*** yamamoto has joined #openstack-neutron05:04
liuyulongkevinbenton, ping05:04
*** hoangcx has quit IRC05:06
*** vijaykc4 has joined #openstack-neutron05:06
*** anilvenkata has joined #openstack-neutron05:06
*** prateek_ has joined #openstack-neutron05:10
*** prateek_ has quit IRC05:11
*** prateek has joined #openstack-neutron05:11
*** kobis has quit IRC05:15
*** dkehn_ has quit IRC05:21
*** dkehn has quit IRC05:22
openstackgerritQunyingRan proposed openstack/neutron: Add error informations for users when value is invalid in database  https://review.openstack.org/35049105:25
*** vishwanathj has joined #openstack-neutron05:29
*** dkehn has joined #openstack-neutron05:29
*** vishwanathj has quit IRC05:29
*** manikanta_tadi has joined #openstack-neutron05:29
anilvenkatashihanzhang, ping05:30
*** amotoki_ has joined #openstack-neutron05:40
*** dkehn_ has joined #openstack-neutron05:40
*** pbandark has joined #openstack-neutron05:41
*** amotoki has quit IRC05:43
openstackgerritArmando Migliaccio proposed openstack/neutron: Allow bound ports to be trunked if the driver can support it  https://review.openstack.org/35137505:44
*** manikanta_ has joined #openstack-neutron05:45
openstackgerritReedip proposed openstack/python-neutronclient: Trivial correction in variable name  https://review.openstack.org/35453905:50
*** rcernin has quit IRC05:51
openstackgerritHenry Gessau proposed openstack/neutron: Emit DeprecationWarning on using tenant_id in models  https://review.openstack.org/35253105:51
*** vijaykc4 has quit IRC05:52
*** vijaykc4 has joined #openstack-neutron05:54
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/35445705:59
*** armax has quit IRC06:05
reedipkevinbenton : Hi. If you have time, can you review https://review.openstack.org/#/c/295677/ ??06:06
reedipthis patch has been up for a long time ^^06:06
reedipamotoki_ : Hi, can you please provide review comments to https://review.openstack.org/#/c/230321/  if you have some bandwidth ??06:07
shihanzhanganilvenkata, pong06:07
openstackgerritvenkata anil proposed openstack/neutron: Delete conntrack when remote ipset member removed  https://review.openstack.org/35244006:09
openstackgerritvenkata anil proposed openstack/neutron: Avoid apply firewall if remote sg members update  https://review.openstack.org/35177306:10
anilvenkatashihanzhang, in https://review.openstack.org/#/c/352440/3/neutron/agent/linux/ipset_manager.py06:11
anilvenkatashihanzhang, https://review.openstack.org/#/c/352440/3/neutron/agent/linux/ipset_manager.py@8606:11
anilvenkatashihanzhang, why do we need synchronized06:11
*** jprovazn has joined #openstack-neutron06:11
*** bvandewa has joined #openstack-neutron06:12
anilvenkatashihanzhang, https://review.openstack.org/#/c/352440/2/neutron/agent/linux/ipset_manager.py06:12
anilvenkatashihanzhang, we should cleanup ipset stuff in both https://review.openstack.org/#/c/352440/ and https://review.openstack.org/#/c/351773/06:13
*** shashank_hegde has joined #openstack-neutron06:14
amotoki_reedip: looking yours now. regarding lbaas quota one, do we have the same features in OSC? Your patch on neutronclient looks good though.06:16
amotoki_reedip: for duplicated option check one, I'm a bit afraid how it changes the existing behavior or if there is a potential regression on extra arguments.06:18
amotoki_reedip: are you confident there is no side effect.06:18
*** iyamahat has joined #openstack-neutron06:20
*** rcernin has joined #openstack-neutron06:21
*** bvandewa has quit IRC06:22
*** anilvenkata is now known as anilvenkata_afk06:24
openstackgerritMerged openstack/neutron: Add linux bridge trunk server side driver  https://review.openstack.org/34927406:29
openstackgerritKevin Benton proposed openstack/neutron: Make callback manager Object Oriented friendly  https://review.openstack.org/35454406:29
*** pgadiya has joined #openstack-neutron06:30
*** lbrune has quit IRC06:30
*** pgadiya has quit IRC06:31
*** vijaykc4 has quit IRC06:33
shihanzhangreedip, I have address the patch as your suggestion, please help review again https://review.openstack.org/#/c/342536/06:34
shihanzhanghi all, who can help to review this patch https://review.openstack.org/#/c/213047/06:36
openstackgerritKevin Benton proposed openstack/neutron: Add debug option to verify iptables rules  https://review.openstack.org/34875306:37
*** vijaykc4 has joined #openstack-neutron06:40
openstackgerritKevin Benton proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/35445706:40
*** nyechiel has joined #openstack-neutron06:41
*** diga_ has joined #openstack-neutron06:44
*** lbrune has joined #openstack-neutron06:46
*** nyechiel has quit IRC06:49
*** iranzo has joined #openstack-neutron06:50
*** kevo has joined #openstack-neutron06:51
*** liuyulong has quit IRC06:54
*** stanzgy has quit IRC06:56
openstackgerritKengo Hobo proposed openstack/neutron: Check content type by completely match instead of partial match  https://review.openstack.org/35455106:57
*** anilvenkata_afk is now known as anilvenkata06:58
*** tesseract- has joined #openstack-neutron06:59
*** lbrune has quit IRC07:00
*** lbrune has joined #openstack-neutron07:03
*** oanson has joined #openstack-neutron07:07
*** liuyulong has joined #openstack-neutron07:08
liuyulongkevinbenton, available now?07:12
*** kobis has joined #openstack-neutron07:15
*** vijaykc4 has quit IRC07:16
*** rkrum has quit IRC07:16
*** stanzgy has joined #openstack-neutron07:18
*** bvandewa has joined #openstack-neutron07:19
*** yamahata has joined #openstack-neutron07:19
*** lbrune has quit IRC07:21
*** hynekm has joined #openstack-neutron07:22
*** akamyshnikova has joined #openstack-neutron07:23
*** lbrune has joined #openstack-neutron07:23
*** obondarev has joined #openstack-neutron07:24
*** sridharg has joined #openstack-neutron07:24
*** anilvenkata is now known as anilvenakata_lun07:24
*** jpena|off is now known as jpena07:25
*** liuyulong is now known as liuyulong|AFK07:32
*** jlibosva has joined #openstack-neutron07:34
*** safchain has joined #openstack-neutron07:34
*** markvoelker has quit IRC07:34
*** ArchiFleKs has quit IRC07:38
*** jamielennox|away has quit IRC07:38
*** stanzgy has quit IRC07:38
*** sindhude has joined #openstack-neutron07:40
*** ArchiFleKs has joined #openstack-neutron07:41
*** sindhude has quit IRC07:41
*** sindhude has joined #openstack-neutron07:41
*** kobis has quit IRC07:42
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron: Add new configuration option in sanity check: vf_extended_management  https://review.openstack.org/35183307:43
*** pgadiya has joined #openstack-neutron07:44
*** pgadiya has quit IRC07:44
*** amoralej|off is now known as amoralej07:44
*** jamielennox|away has joined #openstack-neutron07:47
*** jamielennox|away is now known as jamielennox07:48
*** akamyshnikova has quit IRC07:48
*** akamyshnikova has joined #openstack-neutron07:49
*** sindhude has quit IRC07:49
*** kobis has joined #openstack-neutron07:49
*** sindhude has joined #openstack-neutron07:49
*** kobis has quit IRC07:54
openstackgerritQunyingRan proposed openstack/neutron: Add error informations for users when value is invalid in database  https://review.openstack.org/35049107:54
*** iyamahat has quit IRC07:55
*** yamahata has quit IRC07:55
openstackgerritQunyingRan proposed openstack/neutron: Remove used ip which not in subnet pool from network used ip statistics  https://review.openstack.org/32238707:55
*** eezhova has joined #openstack-neutron07:57
*** vijaykc4 has joined #openstack-neutron07:57
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron: SR-IOV: add agent QoS driver to support egress minimum bandwidth  https://review.openstack.org/34730207:58
*** tonytan4ever has joined #openstack-neutron07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-neutron08:01
*** tonytan4ever has quit IRC08:03
*** bfernando has joined #openstack-neutron08:05
*** kevo has quit IRC08:07
*** matrohon has joined #openstack-neutron08:07
*** shashank_hegde has quit IRC08:10
*** sindhude has quit IRC08:10
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: Faster _get_devices_details_list_and_failed_devices  https://review.openstack.org/28453508:10
*** sindhude has joined #openstack-neutron08:10
*** zhhuabj has quit IRC08:11
*** zhhuabj has joined #openstack-neutron08:11
*** azbiswas has quit IRC08:13
*** lucas-dinner is now known as lucasagomes08:15
*** vijaykc4 has quit IRC08:17
*** vijaykc4 has joined #openstack-neutron08:19
*** rmart04 has joined #openstack-neutron08:20
*** azbiswas has joined #openstack-neutron08:21
*** oanson has quit IRC08:21
*** jlibosva has quit IRC08:22
*** amitkqed has quit IRC08:23
*** azbiswas has quit IRC08:24
*** amitkqed has joined #openstack-neutron08:24
*** tflynn has quit IRC08:25
*** mickeys has joined #openstack-neutron08:27
*** sindhude has quit IRC08:28
openstackgerritKengo Hobo proposed openstack/neutron: Check content type by completely match instead of partial match  https://review.openstack.org/35455108:31
*** jlibosva has joined #openstack-neutron08:31
*** carl_baldwin_ has joined #openstack-neutron08:31
*** mickeys has quit IRC08:31
*** mickeys has joined #openstack-neutron08:32
*** mhickey has joined #openstack-neutron08:34
*** anilvenakata_lun has quit IRC08:34
*** eezhova has quit IRC08:34
*** markvoelker has joined #openstack-neutron08:34
*** carl_baldwin has quit IRC08:35
*** carl_baldwin_ is now known as carl_baldwin08:35
*** kozhukalov_ has joined #openstack-neutron08:35
*** iranzo has quit IRC08:35
*** oanson has joined #openstack-neutron08:36
*** zhhuabj_ has joined #openstack-neutron08:36
*** mickeys has quit IRC08:36
*** anilvenakata_lun has joined #openstack-neutron08:37
*** numans has joined #openstack-neutron08:37
*** anilvenakata_lun is now known as anilvenkata08:37
*** kozhukalov has quit IRC08:38
*** kozhukalov_ is now known as kozhukalov08:38
*** markvoelker has quit IRC08:39
*** djan has quit IRC08:40
*** rossella_s has joined #openstack-neutron08:40
*** zhhuabj has quit IRC08:40
*** stanzgy has joined #openstack-neutron08:46
*** ihrachys has joined #openstack-neutron08:47
*** eezhova has joined #openstack-neutron08:47
*** davidsha has joined #openstack-neutron08:48
*** oshvartz has joined #openstack-neutron08:49
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/35445708:52
openstackgerritMerged openstack/neutron: Refactoring config options for services opts  https://review.openstack.org/34704408:53
*** liyifeng has quit IRC08:59
*** saggi has joined #openstack-neutron09:01
*** numans has quit IRC09:01
openstackgerritOmer Anson proposed openstack/neutron: Log get_schema_name failure in ovsdb connection  https://review.openstack.org/33741809:03
openstackgerritOmer Anson proposed openstack/neutron: Generate OVSDB schema helper in a separate method  https://review.openstack.org/32370709:03
openstackgerritOmer Anson proposed openstack/neutron: Add support to filter OVSDB columns  https://review.openstack.org/32903109:03
*** sambetts|afk is now known as sambetts09:04
*** mbanszex has quit IRC09:05
*** yb has quit IRC09:07
reedipamotoki_ : can you elaborate the side effects which you are thinking about>09:11
reedipamotoki_ : I will put up a patch in OSC for the quota update09:11
*** numans has joined #openstack-neutron09:13
*** eezhova has quit IRC09:15
*** yamamoto has quit IRC09:17
*** manikanta_ has quit IRC09:18
*** eezhova has joined #openstack-neutron09:18
*** malos_ is now known as malos09:19
*** amotoki has joined #openstack-neutron09:20
*** manikanta_ has joined #openstack-neutron09:21
*** amotoki_ has quit IRC09:23
shihanzhangping reedip09:24
*** hynekm has quit IRC09:27
*** mosulica has joined #openstack-neutron09:28
openstackgerritIhar Hrachyshka proposed openstack/neutron: Constrain remaining tox targets  https://review.openstack.org/35461209:29
*** saggi has quit IRC09:29
*** vijaykc4 has quit IRC09:33
*** akamyshnikova has quit IRC09:35
*** markvoelker has joined #openstack-neutron09:36
*** akamyshnikova has joined #openstack-neutron09:37
*** markvoelker has quit IRC09:40
*** amitkqed has quit IRC09:40
*** amitkqed has joined #openstack-neutron09:40
*** amotoki_ has joined #openstack-neutron09:41
*** liuyulong|AFK has quit IRC09:42
*** liuyulong has joined #openstack-neutron09:43
openstackgerritIhar Hrachyshka proposed openstack/neutron-vpnaas: Constrain remaining tox targets  https://review.openstack.org/35461809:43
*** liuyulong is now known as liuyulong|AFK09:44
*** amotoki has quit IRC09:44
openstackgerritIhar Hrachyshka proposed openstack/neutron-fwaas: Constrain remaining tox targets  https://review.openstack.org/35461909:44
ihrachyskevinbenton: HenryG: to close unconstrained gaps in our gate: https://review.openstack.org/#/q/I02b28d3b354c3b175147c5be36eea4dc7e05f2a3,n,z09:45
openstackgerritMerged openstack/neutron: Make revision bump robust to concurrent removals  https://review.openstack.org/35327709:45
*** sdague has joined #openstack-neutron09:46
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/35445709:47
*** matrohon has quit IRC09:48
*** zhhuabj_ has quit IRC09:49
*** liuyulong|AFK has quit IRC09:50
openstackgerritKengo Hobo proposed openstack/neutron: Raise 501 instead of 500 when updating meter-lebel and rule  https://review.openstack.org/35462409:53
*** liuyulong has joined #openstack-neutron09:56
*** TRUMP-PENCE has joined #openstack-neutron09:58
*** EinstCra_ has quit IRC10:05
*** zhhuabj_ has joined #openstack-neutron10:05
*** liuyulong has quit IRC10:07
*** gongysh has quit IRC10:08
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for ports  https://review.openstack.org/35136810:12
*** jlibosva has quit IRC10:13
*** mvk has quit IRC10:14
*** mvk has joined #openstack-neutron10:14
*** amotoki_ has quit IRC10:15
*** vimal7370 has joined #openstack-neutron10:16
*** saggi has joined #openstack-neutron10:16
*** eileen has joined #openstack-neutron10:17
*** azbiswas has joined #openstack-neutron10:24
*** amotoki has joined #openstack-neutron10:27
*** vimal7370 has quit IRC10:29
*** azbiswas has quit IRC10:29
*** jckasper has joined #openstack-neutron10:31
*** vijaykc4 has joined #openstack-neutron10:32
*** iwamoto has quit IRC10:32
*** sbalukoff has quit IRC10:32
*** eileen has quit IRC10:33
*** sbalukoff has joined #openstack-neutron10:33
*** amitkqed has quit IRC10:35
*** amitkqed has joined #openstack-neutron10:35
*** jckasper has quit IRC10:36
*** boden has joined #openstack-neutron10:36
*** saggi has quit IRC10:37
*** trinaths has joined #openstack-neutron10:49
*** iyamahat has joined #openstack-neutron10:51
*** otherwiseguy has quit IRC10:51
*** otherwiseguy has joined #openstack-neutron10:52
*** stanzgy has quit IRC10:56
*** bvandewa has quit IRC10:58
*** rossella_s has quit IRC10:59
*** iyamahat has quit IRC11:00
openstackgerritEster Niclos proposed openstack/neutron: Fix neutron_lib.constants DeprecationWarning from db.models_v2  https://review.openstack.org/35465111:02
*** en_austin has joined #openstack-neutron11:02
*** mdih has quit IRC11:03
*** jlibosva has joined #openstack-neutron11:04
*** obondarev has quit IRC11:07
*** eezhova has quit IRC11:08
*** eezhova has joined #openstack-neutron11:09
*** manikanta_ has quit IRC11:11
*** manikanta_tadi has quit IRC11:11
*** trinaths has quit IRC11:15
*** kobis has joined #openstack-neutron11:15
*** sambetts is now known as sambetts|lunch11:16
*** dkehn has quit IRC11:20
*** dkehn_ has quit IRC11:21
*** dkehn has joined #openstack-neutron11:27
*** amotoki has quit IRC11:30
openstackgerritvenkata anil proposed openstack/neutron: fullstack: test dvr router functionality  https://review.openstack.org/31830711:34
*** dkehn_ has joined #openstack-neutron11:34
*** kragniz is now known as Guest6666611:36
ajoihrachys, electrocucaracha , ping https://review.openstack.org/#/c/351101/4/neutron/objects/common_types.py@14211:37
*** markvoelker has joined #openstack-neutron11:37
*** jistr has quit IRC11:39
ihrachysajo: replied. good catch, not for this patch, but yeah11:40
ihrachysajo: I will put it into my todo list11:40
*** dansmith has quit IRC11:40
ajoyeah, it's not specific to this patch, but we may want to fixate it, and may be some sort of mechanism to catch a divergence with whatever we declared on neutron lib... ?11:40
ajomay be an unit test would do11:40
ajobut that makes updating more complicated (lock-step additions on both sides)11:41
*** StevenK has quit IRC11:41
*** amotoki has joined #openstack-neutron11:41
ajo(updating the enum and updating neutron lib, I mean)11:41
openstackgerritvenkata anil proposed openstack/neutron: Delete conntrack when remote ipset member removed  https://review.openstack.org/35244011:41
*** dansmith has joined #openstack-neutron11:41
*** dansmith is now known as Guest1395711:42
*** markvoelker has quit IRC11:42
*** StevenK has joined #openstack-neutron11:42
openstackgerritAnn Taraday proposed openstack/neutron: Refactor setting OSprofiler for db calls  https://review.openstack.org/34032411:42
*** eezhova has quit IRC11:43
ihrachysajo: unit test should already fail if you change it11:44
ihrachysajo: we have test_objects.py just for that11:44
ajoah, it will detect that we need a version bump?11:45
ihrachysajo: well, I need to check that object hash changes, to be sure11:45
ajoas long as neutron unit tests are ran on neutron_lib changes... yey11:45
ihrachysajo: I hope it will. :)11:45
ajo+++ goood then11:45
*** davidsha has quit IRC11:46
*** sambetts|lunch is now known as sambetts11:46
ajothanks for checkin ihrachys sorry for the noise then11:46
ihrachysajo: no noise, and I haven't actually checked it yet :011:46
ihrachysajo: we still should fix it, because it's a potential gate breakage11:46
ihrachysor even a silent object change, which is even worse. I will follow up on it.11:47
ajowell, if neutron_lib runs neutron unit-tests, the test_object will always catch it11:47
ihrachysajo: it does not11:47
ajooh11:47
ihrachysajo: we run neutron tests with neutron-lib master in periodic track though11:47
ajomay be we should start doing such thing11:47
ihrachysajo: and have it on grafana11:47
ihrachysso we are checking it post merge11:47
ajoalso, neutron_lib to neutron version divergences,11:48
*** gouthamr has joined #openstack-neutron11:48
openstackgerritAnn Taraday proposed openstack/neutron: Get rid of get_engine() in db/api.py  https://review.openstack.org/33978311:48
ajoI'm not sure if we can get into the case of having newton using mitaka neutron_lib11:48
ajoI guess not11:48
ajobut...11:48
ajo:)11:48
ihrachysajo: we cannot11:48
ihrachyswe bump minimal version pretty often11:49
ajogood11:49
*** obondarev has joined #openstack-neutron11:50
*** eezhova has joined #openstack-neutron11:51
*** sindhude has joined #openstack-neutron11:52
*** kobis has quit IRC11:52
openstackgerritvenkata anil proposed openstack/neutron: Avoid apply firewall if remote sg members update  https://review.openstack.org/35177311:53
*** anilvenkata has quit IRC11:54
*** boden has quit IRC11:55
*** ratailor has quit IRC11:57
*** gongysh has joined #openstack-neutron11:58
*** boden has joined #openstack-neutron11:59
*** tonytan4ever has joined #openstack-neutron12:00
*** lucasagomes is now known as lucas-hungry12:00
*** sindhude has quit IRC12:00
*** kobis has joined #openstack-neutron12:02
ajoihrachys, https://review.openstack.org/#/c/351101/4/neutron/objects/securitygroup.py@10412:02
ajodon't we want to expose revision_number through the object?12:02
ajokevinbenton, ^12:02
ihrachysajo: replied, it's already there, you just miss some magic logic in base class12:04
*** tonytan4ever has quit IRC12:05
*** diga_ has quit IRC12:05
*** alaski is now known as lascii12:05
ajooohhh :D12:05
ihrachysamotoki: I see you reviewed some models moving patches. thanks for that! it would be great to see https://review.openstack.org/#/c/351793/ going first though because there are some conflicts to resolve with those you already +2'd, and the one I referred to is needed for the port object that we will try to land asap.12:06
amotokiihrachys: thanks for letting it know. will look it soon.12:07
*** prateek has quit IRC12:08
*** amotoki has quit IRC12:10
*** amotoki has joined #openstack-neutron12:16
*** gvrangan has joined #openstack-neutron12:19
openstackgerritSergey Nechaev proposed openstack/neutron: Adding support of releasing DHCPv6 leases  https://review.openstack.org/30174712:19
ihrachysajo: re sg patch, anything more to do to get your blessing?12:20
*** amoralej is now known as amoralej|lunch12:23
*** nplanel has quit IRC12:23
*** jistr has joined #openstack-neutron12:24
*** jpena is now known as jpena|lunch12:25
ajoihrachys, I just found one thing12:26
openstackgerritIhar Hrachyshka proposed openstack/neutron: models: move AllowedAddressPair model under neutron/db/models  https://review.openstack.org/35179312:27
ajoihrachys, but nothing fundamentally wrong with the patch itself, just a testing/protocols detail12:27
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for ports  https://review.openstack.org/35136812:27
ihrachysajo: that one is a good catch, I will fix it, sec.12:28
ajoihrachys,  ack12:28
*** fxpester has joined #openstack-neutron12:29
*** amotoki has quit IRC12:29
*** jistr has quit IRC12:31
*** amotoki has joined #openstack-neutron12:31
fxpesterhi all, I`m trying to configure multiple external networks, do I need configure bridge_mappings on compute nodes ?12:32
*** jistr has joined #openstack-neutron12:32
*** gongysh has quit IRC12:33
*** adreznec has quit IRC12:33
openstackgerritboden proposed openstack/neutron-lib: Generate API report tooling  https://review.openstack.org/33857112:33
fxpesterI use default Mirantis, it is working perfectly with one network and bridge_mappings just commented out on compute configs12:33
openstackgerritAnn Taraday proposed openstack/neutron: Use dispose_pool() from oslo.db  https://review.openstack.org/35469912:33
*** chlong|mtg has quit IRC12:33
*** markvoelker has joined #openstack-neutron12:35
*** adreznec has joined #openstack-neutron12:35
*** vmtyler has joined #openstack-neutron12:35
*** amotoki has quit IRC12:35
*** baoli has joined #openstack-neutron12:36
*** baoli_ has joined #openstack-neutron12:37
*** Shahid_ has quit IRC12:38
openstackgerritboden proposed openstack/neutron-lib: Generate API report tooling  https://review.openstack.org/33857112:38
*** sindhude has joined #openstack-neutron12:41
*** baoli has quit IRC12:41
*** jistr has quit IRC12:41
*** pbandark has quit IRC12:43
*** rkrum has joined #openstack-neutron12:45
*** amotoki has joined #openstack-neutron12:47
jlibosvaotherwiseguy:12:47
jlibosvasorry, that was supposed to be pm :)12:48
openstackgerritAnn Taraday proposed openstack/neutron: Get rid of get_engine() in db/api.py  https://review.openstack.org/33978312:48
*** obondarev has quit IRC12:48
*** rkrum has quit IRC12:50
*** links has quit IRC12:53
*** amuller has joined #openstack-neutron12:56
*** vijaykc4 has quit IRC12:56
openstackgerritAnn Taraday proposed openstack/neutron: Use dispose_pool() from oslo.db  https://review.openstack.org/35469912:57
*** obondarev has joined #openstack-neutron12:58
*** amoralej|lunch is now known as amoralej12:59
en_austinhi guys!13:00
en_austini have a simple question13:00
*** rossella_s has joined #openstack-neutron13:00
en_austincan Neutron be used to allocate an IP pool, not single IP address from some pool?13:00
*** jlibosva has quit IRC13:00
*** lucas-hungry is now known as lucasagomes13:00
*** iranzo has joined #openstack-neutron13:01
*** tonytan4ever has joined #openstack-neutron13:01
*** baoli_ has quit IRC13:02
en_austinex: I have 192.168.0.0/16 pool and want to allocate 192.168.1.1/28 (and get 192.168.1. ... 192.168.1.15 following each other with no spaces in that allocated pool)13:02
*** sindhude has quit IRC13:02
*** jlibosva has joined #openstack-neutron13:03
*** jistr has joined #openstack-neutron13:03
en_austinit is not supposed to be used by virtual machines, it is for purposes of service that will be installed on those VMs - so I need entire pool without splits (.1, .2, .3, .4, ... .15, not .1 3 6 7 8 12..., for example)13:04
*** prateek has joined #openstack-neutron13:05
*** tonytan4ever has quit IRC13:06
*** mhayden has joined #openstack-neutron13:09
*** jistr has quit IRC13:13
*** fragatina has joined #openstack-neutron13:13
*** edmondsw has joined #openstack-neutron13:14
openstackgerritEster Niclos proposed openstack/neutron: Fix neutron_lib.constants DeprecationWarning from db.models_v2  https://review.openstack.org/35465113:15
*** sindhude has joined #openstack-neutron13:15
*** fragatina has quit IRC13:18
openstackgerritboden proposed openstack/neutron-lib: No hacking checks via entry points  https://review.openstack.org/35072313:19
*** itisha has joined #openstack-neutron13:20
*** tonytan4ever has joined #openstack-neutron13:22
*** jckasper has joined #openstack-neutron13:23
openstackgerritMerged openstack/neutron-vpnaas: Constrain remaining tox targets  https://review.openstack.org/35461813:26
*** ababich1 has quit IRC13:26
*** yamamoto has joined #openstack-neutron13:28
*** jamesdenton has joined #openstack-neutron13:29
openstackgerritMerged openstack/neutron: Set secure fail mode for physical bridges  https://review.openstack.org/34888913:30
openstackgerritMerged openstack/neutron: Updated from global requirements  https://review.openstack.org/35445713:31
*** jpena|lunch is now known as jpena13:31
*** jckasper has quit IRC13:31
*** saggi has joined #openstack-neutron13:32
njohnston_Hey, so it looks like the linuxbridge failure rate is going down now, but is still far from normal... what is/was the problem there?  http://grafana.openstack.org/dashboard/db/neutron-failure-rate?from=1470835865801&to=147100866580113:32
*** jckasper has joined #openstack-neutron13:32
*** ajmiller has joined #openstack-neutron13:32
*** janzian has joined #openstack-neutron13:33
*** njohnston_ is now known as njohnston13:34
ihrachysajo: PS1? :-x13:34
ihrachysc'mon!13:34
ihrachysnjohnston: grafana is sliding window13:34
ihrachysnjohnston: calculated for last 24h13:35
ajoihrachys, nonsense, I know13:35
ihrachysnjohnston: so it won't drop immediately13:35
ajoihrachys, I added some comments on last PS, I'll make another pass13:35
*** ajmiller has quit IRC13:35
*** jckasper has quit IRC13:36
njohnstonihrachys:  Ah, I see.  I was confused because I see other graphs that have stairstep functions like this one - is that due to lost data then?  http://grafana.openstack.org/dashboard/db/neutron-failure-rate?from=1470836197649&to=147100899765013:36
*** s1061123 has joined #openstack-neutron13:37
ihrachysnjohnston: not sure; that's all I know13:37
*** s1061123_ has quit IRC13:37
njohnstonihrachys: Thanks!13:37
*** sindhude has quit IRC13:39
*** rodrigods has quit IRC13:39
*** rodrigods has joined #openstack-neutron13:40
openstackgerritMerged openstack/neutron: Relocate Security Group DB models  https://review.openstack.org/35366413:40
*** yamamoto has quit IRC13:43
*** ajmiller has joined #openstack-neutron13:44
*** sindhude has joined #openstack-neutron13:47
*** Guest13957 is now known as dansmith13:50
*** kriskend has joined #openstack-neutron13:50
*** dansmith is now known as Guest3789013:50
*** Guest37890 is now known as dansmith13:51
*** vimal7370 has joined #openstack-neutron13:51
*** wolverineav has quit IRC13:51
*** ajmiller has quit IRC13:52
vimal7370hi, does DVR work with LinuxBridge?13:52
*** zhhuabj_ has quit IRC13:52
*** thorst_ has joined #openstack-neutron13:53
*** wolverineav has joined #openstack-neutron13:54
*** charlesr has joined #openstack-neutron13:55
*** vishwanathj has joined #openstack-neutron13:56
*** thorst_ has quit IRC13:57
*** dansmith is now known as superdan13:57
*** a_ta has joined #openstack-neutron13:58
openstackgerritSergey Nechaev proposed openstack/neutron: Adding support of releasing DHCPv6 leases  https://review.openstack.org/30174713:59
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for networks  https://review.openstack.org/26965814:02
*** zhhuabj_ has joined #openstack-neutron14:03
*** pradk has quit IRC14:05
*** baoli has joined #openstack-neutron14:05
*** wolverineav has quit IRC14:07
*** pradk has joined #openstack-neutron14:07
openstackgerritgaryk proposed openstack/neutron: Use neutron-lib constants  https://review.openstack.org/35474314:09
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for security groups  https://review.openstack.org/35110114:09
openstackgerritMerged openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/35452614:10
openstackgerritMerged openstack/neutron-vpnaas: Remove temporary local HasProject  https://review.openstack.org/35451314:10
*** mihero has joined #openstack-neutron14:11
*** mihero_ has quit IRC14:11
*** thorst_ has joined #openstack-neutron14:14
openstackgerritSindhu Devale proposed openstack/neutron: Refactoring db config options  https://review.openstack.org/34487714:14
pcm_sc68cal:ping14:14
*** iranzo has quit IRC14:16
*** jlibosva has quit IRC14:16
*** vijaykc4 has joined #openstack-neutron14:16
*** jlibosva has joined #openstack-neutron14:17
*** sindhude has quit IRC14:20
*** jckasper has joined #openstack-neutron14:20
*** sindhude has joined #openstack-neutron14:21
*** wolverineav has joined #openstack-neutron14:21
*** sindhu has joined #openstack-neutron14:21
openstackgerritSergey Belous proposed openstack/neutron-specs: Extend Quota API to report usage statistics  https://review.openstack.org/34894714:21
ihrachysare ml2 vif_details free form (beyond that it's a dict)?14:22
*** vijaykc4 has quit IRC14:24
*** ajmiller has joined #openstack-neutron14:27
*** azbiswas has joined #openstack-neutron14:27
*** wolverineav has quit IRC14:29
*** yamamoto has joined #openstack-neutron14:30
*** azbiswas has quit IRC14:32
*** ajmiller has quit IRC14:32
*** mlavalle has joined #openstack-neutron14:33
*** baoli has quit IRC14:36
*** baoli has joined #openstack-neutron14:36
*** tanvirt has joined #openstack-neutron14:36
*** sindhude has quit IRC14:38
*** sindhu has quit IRC14:38
*** sindhu has joined #openstack-neutron14:38
*** jistr has joined #openstack-neutron14:39
openstackgerritMerged openstack/neutron-vpnaas: Fix DeprecationWarnings part I  https://review.openstack.org/35323814:39
*** kbringard has joined #openstack-neutron14:43
tanvirtHey everyone! I want to work on a bug that has a proposed fix that was abandoned due to inactivity. Its Bug #1345341. How do I go about doing that? I was told someone from core has to "unlock" it? Never worked on an abandoned change before.14:45
openstackbug 1345341 in neutron "radvd needs functional tests" [Medium,Triaged] https://launchpad.net/bugs/1345341 - Assigned to Tanvir Talukder (tanvirt16)14:45
prometheanfireyamamoto: hi, can you give (dis)approval of the ryu update in https://review.openstack.org/35334914:46
*** michauds has joined #openstack-neutron14:50
tanvirtThe abandoned change is here: https://review.openstack.org/#/c/109889/14:51
amullertanvirt: restored it14:54
*** jckasper has quit IRC14:56
*** jckasper has joined #openstack-neutron14:56
*** akamyshnikova has quit IRC14:57
tanvirtamuller: thanks!14:57
*** kriskend has quit IRC14:58
*** rajinir has joined #openstack-neutron14:58
*** thiagolib has joined #openstack-neutron14:59
*** ijw has joined #openstack-neutron15:00
*** EinstCrazy has joined #openstack-neutron15:00
*** jckasper has quit IRC15:01
*** iyamahat has joined #openstack-neutron15:03
*** michauds has quit IRC15:03
*** jlibosva has quit IRC15:05
*** vimal7370 has quit IRC15:05
*** armax has joined #openstack-neutron15:08
*** yb has joined #openstack-neutron15:11
*** garyk1 has quit IRC15:11
*** iyamahat has quit IRC15:12
*** rmart04 has quit IRC15:13
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115915:17
*** michauds has joined #openstack-neutron15:18
*** yamahata has joined #openstack-neutron15:20
*** sindhu has quit IRC15:21
*** sindhu has joined #openstack-neutron15:21
*** thorst_ has quit IRC15:23
*** pcaruana has quit IRC15:23
*** EinstCrazy has quit IRC15:23
*** jistr is now known as jistr|call15:23
*** thorst_ has joined #openstack-neutron15:24
openstackgerritAnkur proposed openstack/neutron: Neutron Feature Classification  https://review.openstack.org/31819215:25
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115915:26
*** yamamoto has quit IRC15:26
*** akamyshnikova has joined #openstack-neutron15:27
*** mickeys has joined #openstack-neutron15:27
*** vimal7370 has joined #openstack-neutron15:27
*** fnaval_ has quit IRC15:27
*** thorst_ has quit IRC15:28
*** yamamoto has joined #openstack-neutron15:28
*** beagles is now known as beagles_brb15:29
*** sridharg has quit IRC15:30
*** haplo37_ has joined #openstack-neutron15:30
*** thorst_ has joined #openstack-neutron15:31
*** obondarev has quit IRC15:33
*** en_austin has quit IRC15:34
*** vimal7370 has quit IRC15:34
*** thorst__ has joined #openstack-neutron15:35
openstackgerritSindhu Devale proposed openstack/neutron: Relocate router route DB model  https://review.openstack.org/35479715:35
*** thorst_ has quit IRC15:36
*** mosulica has quit IRC15:36
*** pbandark has joined #openstack-neutron15:39
*** charlesr has quit IRC15:39
openstackgerritAnn Taraday proposed openstack/neutron: WIP another approach for step 3  https://review.openstack.org/35480115:40
*** thorst__ has quit IRC15:40
*** numans has quit IRC15:40
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: SNAT redirect rules should be removed only on Gateway clear  https://review.openstack.org/35378815:41
*** jistr|call is now known as jistr15:42
njohnstonrossella_s: I was looking at bug https://bugs.launchpad.net/bugs/1612697, how high do you think the importance on that should be?15:43
openstackLaunchpad bug 1612697 in neutron "ovs agent not ready ports should be listed as current" [Undecided,New]15:43
*** sindhu has quit IRC15:44
rossella_snjohnston, medium...I don't think it's causing any serious problem15:44
njohnstonrossella_s: Thank you!15:45
*** michauds has quit IRC15:46
*** yb has quit IRC15:46
*** rcernin has quit IRC15:47
*** vimal7370 has joined #openstack-neutron15:48
*** sbalukoff has quit IRC15:49
*** slaweq__ has quit IRC15:49
*** sbalukoff has joined #openstack-neutron15:49
*** sindhu has joined #openstack-neutron15:49
*** azbiswas has joined #openstack-neutron15:50
*** bfernando has quit IRC15:53
*** thorst_ has joined #openstack-neutron15:53
openstackgerritMerged openstack/neutron-lib: Add DeviceNotFoundError to neutron_lib exceptions  https://review.openstack.org/35393015:53
openstackgerritCarl Baldwin proposed openstack/neutron: Avoid IPAM driver reusing a session that has been rolled back  https://review.openstack.org/34895615:53
*** itzikb has quit IRC15:54
*** hockeynut has joined #openstack-neutron15:54
*** gvrangan has quit IRC15:54
openstackgerritCarl Baldwin proposed openstack/neutron: Switch to pluggable IPAM implementation  https://review.openstack.org/18102315:54
*** thorst_ has quit IRC15:55
*** jckasper has joined #openstack-neutron15:55
*** thorst_ has joined #openstack-neutron15:56
*** beagles_brb is now known as beagles15:57
openstackgerritMerged openstack/python-neutronclient: Add support to expose default quotas for tenants  https://review.openstack.org/31313415:57
*** ranjithd has quit IRC15:58
*** krtaylor has quit IRC15:59
*** haplo37_ has quit IRC16:00
*** jckasper has quit IRC16:00
*** fnaval has joined #openstack-neutron16:00
*** michauds has joined #openstack-neutron16:02
*** kobis has quit IRC16:03
*** yamahata has quit IRC16:03
openstackgerritMerged openstack/neutron-lib: Updated from global requirements  https://review.openstack.org/35445816:05
*** TRUMP-PENCE has quit IRC16:07
*** dsneddon has quit IRC16:08
*** dsneddon has joined #openstack-neutron16:09
otherwiseguyoanson: I added a comment on https://review.openstack.org/#/c/329031 with how to replace all of the unit tests with functional tests. Let me know if you need any other info (or if you want me to update the patch with a basic functional test in place that you could build off of if you wanted).16:09
otherwiseguyThanks for your patience on all of this.16:10
*** TRUMP-PENCE has joined #openstack-neutron16:10
*** oanson has quit IRC16:11
*** pbandark has quit IRC16:12
*** wolverineav has joined #openstack-neutron16:12
*** prateek_ has joined #openstack-neutron16:13
*** mmalik4 has joined #openstack-neutron16:13
*** prateek has quit IRC16:16
*** iyamahat has joined #openstack-neutron16:17
*** sindhu has quit IRC16:21
*** sindhu has joined #openstack-neutron16:21
*** oanson has joined #openstack-neutron16:24
*** parora has joined #openstack-neutron16:24
*** yamahata has joined #openstack-neutron16:26
*** tidwellr has joined #openstack-neutron16:26
*** prateek_ has quit IRC16:28
*** fragatina has joined #openstack-neutron16:30
*** jckasper has joined #openstack-neutron16:30
*** fragatin_ has joined #openstack-neutron16:31
openstackgerritMohit Malik proposed openstack/neutron: Move AddressScope DB model under neutron/db/models  https://review.openstack.org/35364316:33
openstackgerritManjeet Singh Bhatia proposed openstack/neutron: Relocate Metering DB models  https://review.openstack.org/34875716:33
openstackgerritManjeet Singh Bhatia proposed openstack/neutron: Relocate L3 DB Models  https://review.openstack.org/34856216:33
openstackgerritManjeet Singh Bhatia proposed openstack/neutron: Relocate external network db models  https://review.openstack.org/35195516:34
openstackgerritMohit Malik proposed openstack/neutron: Integrate Address Scope OVO in to database code  https://review.openstack.org/35442816:34
*** fragatina has quit IRC16:35
*** tesseract- has quit IRC16:35
*** EinstCrazy has joined #openstack-neutron16:36
openstackgerritLubosz Kosnik (diltram) proposed openstack/neutron: Add support for Keepalived VRRP health check  https://review.openstack.org/27354616:36
openstackgerritHenry Gessau proposed openstack/neutron: Add tool to list moved globals  https://review.openstack.org/35483316:38
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for ports  https://review.openstack.org/35136816:42
openstackgerritManjeet Singh Bhatia proposed openstack/neutron: Relocate external network db models  https://review.openstack.org/35195516:42
ihrachysajo: ^ ports patch16:43
ihrachysajo: I believe it's good enough now16:43
*** gvrangan has joined #openstack-neutron16:44
* electrocucaracha wants to take a look also16:45
*** lbrune has quit IRC16:45
electrocucarachaihrachys: good middle point on this https://review.openstack.org/#/c/351101/4..5/neutron/objects/securitygroup.py@3816:46
*** azbiswas has quit IRC16:46
*** shashank_hegde has joined #openstack-neutron16:47
ihrachyselectrocucaracha: saw your email. we need to rethink subteam structure I think.16:48
ihrachyselectrocucaracha: I am often a bottleneck, I need to delegate subtracks to someone16:48
ihrachyselectrocucaracha: like model moves, they can be tracked by someone else16:48
electrocucarachaihrachys: it's fine, basically OVO got a lot attention16:49
openstackgerritSindhu Devale proposed openstack/neutron: Relocate Flat Allocation DB model  https://review.openstack.org/35439316:49
ihrachyselectrocucaracha: yeah, but I feel like I fail to manage all resources it suddenly attracted :)16:51
ihrachyselectrocucaracha: especially now that I try to deliver those objects for push-notifications16:51
ihrachyselectrocucaracha: after the midcycle I should be in better position to rethink the tactics16:51
electrocucarachaihrachys: well, I shouldn'16:52
*** jpena is now known as jpena|off16:52
electrocucarachaihrachys: I shouldn't consider as failure, suddenly we have extra hands on this16:52
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for networks  https://review.openstack.org/26965816:52
ihrachyselectrocucaracha: not a failure, a challenge16:52
electrocucarachaihrachys: around 6 more people16:52
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115916:52
electrocucarachaihrachys: +1 ^16:54
ihrachysajo: also, the patch moving allowed address pairs, that one should hopefully be quick: https://review.openstack.org/#/c/351793/716:54
ihrachyselectrocucaracha: you refer to fwaas?16:55
*** lucasagomes is now known as lucas-dinner16:55
* njohnston thinks he hears sc68cal laughing16:55
ihrachysoh well16:55
ihrachysnjohnston: I remember there were objects for fwaasv216:55
ihrachysnjohnston: what happened?16:55
electrocucarachaihrachys: sorry to consider it as challenge16:56
njohnstonihrachys: In progress - https://review.openstack.org/#/c/342476/ - will get more attention once we have the basic functionality bits merged16:56
*** yamamoto has quit IRC16:57
Sam-I-Amtidwellr: yo17:00
Sam-I-Amtidwellr: added you to a patch that improves the bgp-dr content17:01
ajoihrachys, ack17:02
*** lbrune has joined #openstack-neutron17:02
*** sindhu has quit IRC17:03
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115917:03
*** sambetts is now known as sambetts|afk17:04
*** mmalik4_ has joined #openstack-neutron17:08
*** mmalik4 has quit IRC17:08
*** shashank_hegde has quit IRC17:08
ihrachysnjohnston: I left a bunch of comments there. I know most of them may be unclear, so please ask and I can walk you thru base class for objects.17:09
*** tpsilva has joined #openstack-neutron17:09
openstackgerritKevin Benton proposed openstack/neutron: Add debug option to verify iptables rules  https://review.openstack.org/34875317:09
ihrachysajo++ thanks man17:09
njohnstonihrachys: Thanks!  padkrish has been doing the heavy lifting there, but he and I will get our heads together and come to you with questions.17:10
tidwellrSam-I-Am: ack17:10
ajoyou're welcome ihrachys , thank *you* for tackling this17:12
*** kevo has joined #openstack-neutron17:16
*** sputnik13 has joined #openstack-neutron17:16
tidwellrSam-I-Am: the BGP docs need some updates, we've spun BGP out to a new project called "neutron-dynamic-routing" which means the sections describing configuration need to be tweaked17:18
tidwellrSam-I-Am: I'll make some notes in the review17:18
*** sindhu has joined #openstack-neutron17:18
*** baoli has quit IRC17:19
*** wolverineav has quit IRC17:19
*** wolverineav has joined #openstack-neutron17:20
openstackgerritSindhu Devale proposed openstack/neutron: Relocate router route DB model  https://review.openstack.org/35479717:23
*** krtaylor has joined #openstack-neutron17:24
*** prateek_ has joined #openstack-neutron17:24
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for networks  https://review.openstack.org/26965817:25
*** boden has quit IRC17:26
*** parora has quit IRC17:27
ihrachysajo: btw secgroups object should also be ready, https://review.openstack.org/#/c/351101/17:27
ihrachysajo: the remaining bit is networks, but that one will take some more time to shape (probably eta = Monday)17:27
*** amoralej is now known as amoralej|off17:27
*** wolverineav has quit IRC17:28
openstackgerritIhar Hrachyshka proposed openstack/neutron: Introduce ovo objects for security groups  https://review.openstack.org/35110117:28
*** abhiraut has joined #openstack-neutron17:30
Sam-I-Amtidwellr: aight17:30
*** boden has joined #openstack-neutron17:31
*** bvandewa has joined #openstack-neutron17:32
openstackgerritMohit Malik proposed openstack/neutron: Move AddressScope DB model under neutron/db/models  https://review.openstack.org/35364317:32
*** Sam-I-Am sets mode: -o Sam-I-Am17:32
openstackgerritKevin Benton proposed openstack/neutron: Make callback manager Object Oriented friendly  https://review.openstack.org/35454417:35
*** janzian has quit IRC17:35
*** jacob_l has joined #openstack-neutron17:37
*** hockeynut has quit IRC17:38
*** bvandewa has quit IRC17:39
*** rossella_s has quit IRC17:39
*** bvandewa has joined #openstack-neutron17:39
*** michauds has quit IRC17:39
*** ijw has quit IRC17:40
*** lbrune1 has joined #openstack-neutron17:41
*** lbrune has quit IRC17:43
*** shashank_hegde has joined #openstack-neutron17:43
*** wolverineav has joined #openstack-neutron17:43
*** sambetts|afk has quit IRC17:43
*** ivar-lazzaro has joined #openstack-neutron17:44
*** wolverineav has quit IRC17:45
*** sambetts has joined #openstack-neutron17:45
*** vhosakot has joined #openstack-neutron17:45
*** wolverineav has joined #openstack-neutron17:45
*** ihrachys has quit IRC17:46
*** Swami has joined #openstack-neutron17:46
*** oanson has quit IRC17:46
*** EinstCrazy has quit IRC17:48
*** s3wong has joined #openstack-neutron17:49
*** tonytan4ever has quit IRC17:49
*** mhickey has quit IRC17:50
*** sdague has quit IRC17:51
*** michauds has joined #openstack-neutron17:51
*** krtaylor has quit IRC17:54
openstackgerritAradhana Singh proposed openstack/neutron: Refactoring config options for ml2 plugin drivers  https://review.openstack.org/34018717:57
*** oanson has joined #openstack-neutron17:59
*** sdague has joined #openstack-neutron18:00
*** TRUMP-PENCE has quit IRC18:02
*** fragatin_ has quit IRC18:02
openstackgerritMerged openstack/neutron: Refactoring config options of l3 agent keepalived  https://review.openstack.org/33855918:02
*** yamamoto has joined #openstack-neutron18:02
*** tflynn has joined #openstack-neutron18:03
*** TRUMP-PENCE has joined #openstack-neutron18:03
*** ivar-lazzaro has quit IRC18:04
*** yamamoto has quit IRC18:07
*** bvandewa has quit IRC18:09
*** janzian has joined #openstack-neutron18:11
*** azbiswas has joined #openstack-neutron18:11
openstackgerritHenry Gessau proposed openstack/neutron: [WORK IN PROGRESS] Fix deprecation warnings  https://review.openstack.org/35347218:11
*** ivar-lazzaro has joined #openstack-neutron18:12
*** azbiswas_ has joined #openstack-neutron18:13
*** parora has joined #openstack-neutron18:14
*** azbiswas has quit IRC18:16
openstackgerritHenry Gessau proposed openstack/neutron: [WORK IN PROGRESS] Fix deprecation warnings  https://review.openstack.org/35347218:16
*** baoli has joined #openstack-neutron18:17
sc68calnjohnston: I only laugh about things I care about18:17
* sc68cal drops mic18:17
*** prateek_ has quit IRC18:18
*** bvandewa has joined #openstack-neutron18:18
*** vimal7370 has quit IRC18:20
openstackgerritArmando Migliaccio proposed openstack/neutron: Broken extensions should not show up in the extension list  https://review.openstack.org/35195918:20
*** slaweq has joined #openstack-neutron18:21
*** vimal7370 has joined #openstack-neutron18:22
*** ivar-lazzaro has quit IRC18:23
bodenarmax: just left you comment on ^   if I’m missing something pls let me know and I’ll promptly change my review18:25
armaxboden: ack18:26
*** ivar-lazzaro has joined #openstack-neutron18:28
openstackgerritShashank Kumar Shankar proposed openstack/neutron: WIP - RouterExtraAttributes-OVO  https://review.openstack.org/35489318:29
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115918:29
openstackgerritShashank Kumar Shankar proposed openstack/neutron: WIP - RouterExtraAttributes-OVO  https://review.openstack.org/35489318:32
*** zhhuabj_ has quit IRC18:33
*** vimal_ has joined #openstack-neutron18:34
armaxboden: let me know if that clears your mind18:34
HenryGarmax: bug 1612615 :(18:37
openstackbug 1612615 in neutron "test_get_allocated_net_topology_as_tenant failure in api job" [High,New] https://launchpad.net/bugs/1612615 - Assigned to Henry Gessau (gessau)18:37
openstackgerritShashank Kumar Shankar proposed openstack/neutron: WIP - RouterExtraAttributes-OVO  https://review.openstack.org/35489318:37
*** vimal7370 has quit IRC18:38
*** krtaylor has joined #openstack-neutron18:39
*** amotoki has quit IRC18:42
*** jacob_l has quit IRC18:42
openstackgerritAradhana Singh proposed openstack/neutron: Refactoring config options for ml2 config opts  https://review.openstack.org/34022818:43
*** bvandewa has quit IRC18:44
*** zhhuabj_ has joined #openstack-neutron18:45
*** wasmum has quit IRC18:47
*** agireud has quit IRC18:47
*** gvrangan has quit IRC18:49
*** wasmum has joined #openstack-neutron18:49
*** tonytan4ever has joined #openstack-neutron18:50
* armax looks18:50
*** Shahid_ has joined #openstack-neutron18:51
*** ivasilevskaya has quit IRC18:53
armaxHenryG: have you dug deeper?18:55
*** agireud has joined #openstack-neutron18:55
*** tonytan4ever has quit IRC18:55
armaxHenryG: the root cause seems related to a db error of some sort, that’s not caught correctly throughout the stack18:55
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115918:56
armaxit might as well be bug that carl_baldwin is trying to fix here:18:56
HenryGarmax: you may be right, still looking18:56
armaxhttps://review.openstack.org/#/c/348956/18:56
armaxwhich we masked a bit here:https://review.openstack.org/#/c/337863/18:57
armaxbut we only reduced the likehood of that issue to surface18:58
*** tidwellr has left #openstack-neutron18:58
armaxthat said, this manifested itself on https://review.openstack.org/#/c/354614/18:59
armaxthat has nothing to do with the pluggable ipam18:59
armaxso there might be another issue that we need to smoke out18:59
openstackgerritShih-Hao Li proposed openstack/neutron: Fix test_router_add_interface_delete_port_after_failure  https://review.openstack.org/35495118:59
armaxHenryG: btw I meant https://review.openstack.org/#/c/354612/19:00
*** ivar-lazzaro has quit IRC19:00
*** tflynn_ has joined #openstack-neutron19:01
armaxHenryG: basically here we must have hit a db error on subnet create19:01
armaxand we ended up with a duplicate19:01
armaxthat’s kevinbenton’s retry nonsense’s fault19:01
armaxI blame him19:01
armax:)19:01
*** fragatina has joined #openstack-neutron19:01
*** mmalik4_ has quit IRC19:02
*** mmalik4 has joined #openstack-neutron19:02
*** jprovazn has quit IRC19:03
*** fifieldt has quit IRC19:03
armaxHenryG: the smoking gun19:04
armaxhttp://logs.openstack.org/12/354612/1/check/gate-neutron-dsvm-api/09c5da5/logs/screen-q-svc.txt.gz#_2016-08-12_10_20_10_88619:04
*** vhosakot has quit IRC19:04
*** tflynn has quit IRC19:04
armaxHenryG: another smoking gun:19:05
armaxhttp://logs.openstack.org/12/354612/1/check/gate-neutron-dsvm-api/09c5da5/logs/screen-q-svc.txt.gz#_2016-08-12_10_20_16_96719:05
kevinbentonarmax: what am i being blamed for?19:06
armaxand another19:06
armaxhttp://logs.openstack.org/12/354612/1/check/gate-neutron-dsvm-api/09c5da5/logs/screen-q-svc.txt.gz#_2016-08-12_10_20_24_57219:06
HenryGkevinbenton: everything, of course19:06
kevinbentonlooks like the auto allocated topology code is not idempotent!19:06
kevinbentonarmax's fault :)19:07
*** krtaylor has quit IRC19:07
armaxkevinbenton: it looks like something doesn’t surface all the way up to the auto allocated plugin19:07
armaxkevinbenton: and things get retried under the hood?19:07
armaxwhich lead to multiple resources being created19:07
kevinbentonarmax: no, it's making it all of the way back up to the api layer19:08
kevinbentonarmax: http://logs.openstack.org/12/354612/1/check/gate-neutron-dsvm-api/09c5da5/logs/screen-q-svc.txt.gz#_2016-08-12_10_20_10_88719:08
*** thiagolib has quit IRC19:08
*** krtaylor has joined #openstack-neutron19:08
armaxkevinbenton: ack, that was the first smoking gun I found19:09
*** sindhu has quit IRC19:09
*** sindhu has joined #openstack-neutron19:09
*** bfernando has joined #openstack-neutron19:09
kevinbentonarmax: ah https://github.com/openstack/neutron/blob/master/neutron/services/auto_allocate/db.py#L27219:09
*** ivar-lazzaro has joined #openstack-neutron19:09
kevinbentonarmax: you probably need to catch retriables here and undo the router creation19:10
*** tonytan4ever has joined #openstack-neutron19:10
kevinbentonarmax: otherwise a failure on interface add will throw up without cleanup being called19:10
kevinbentonarmax: either that or now is the time to move this retry logic down into the plugins19:11
armaxI blame https://review.openstack.org/#/c/332487/19:12
armaxI feared this would happen19:12
kevinbentonarmax: no19:12
armaxnow I have proof19:12
kevinbentonarmax: that isn't the problem19:12
armaxretrying on dbduplicate is evil19:13
kevinbentonarmax: if we didn't retry duplicates you would have just had a 500 error here19:13
armaxwhich it’s easier to debug19:13
armaxrather than chase this weird failure modes19:14
armaxthis retry logic is spreading uncontrollably19:14
kevinbentonthat's why i just suggested it's time to move it down to the plugin calls19:14
*** saggi has quit IRC19:16
*** Shahid_ has quit IRC19:16
*** vimal_ has quit IRC19:17
kevinbentonarmax: the closer we can get it to the DB, the less difficult it is to reason about19:17
*** fifieldt has joined #openstack-neutron19:18
armaxwhat do you mean by moving it down to plugin calls?19:19
*** kbringard has quit IRC19:19
kevinbentonhave each plugin method protected by the decorator19:19
kevinbentonso add_router_interface would be protected19:19
armaxkevinbenton: but then backends may receive multiple calls19:19
kevinbentonarmax: ?19:20
kevinbentonarmax: backends shouldn't receive anything before commit19:20
armaxbut that depends on what the method you’re wrapping does19:20
armaxwe can’t blindly decorate19:20
kevinbentonarmax: well that's what doing it at the API layer does19:21
kevinbentonarmax: in the base plugin classes, i can add an attribute called 'retriable_methods' that has the default stuff19:21
armaxI am getting numb19:21
kevinbentonarmax: then we decorate those19:22
armaxno19:22
armaxstop19:22
armaxyou must stop19:22
HenryGarmax: kevinbenton: I am going offline for bit, back in about an hour19:22
kevinbentonarmax: what do you want? telling me to stop is not constructive19:22
armaxI ban any more retry magic19:22
kevinbentonarmax: decorating plugin methods is better than at the API layer19:23
armaxif nothing else we should extract all the db logic and decorate that19:23
armaxand leave everything else as is19:23
*** ivar-lazzaro has quit IRC19:23
*** ivar-lazzaro has joined #openstack-neutron19:23
armaxand stop the retry applied all over the place19:23
kevinbentonarmax: oh, so all you want is a massive refactor19:24
kevinbentoneasy enough19:24
*** rajinir has quit IRC19:25
armaxkevinbenton: it’s the lesser of the two evils19:25
kevinbentonarmax: if you would rather rewrite all of neutrons DB code than fix the auto allocate plugin to handle a failure in router interface creation, fine19:26
armaxkevinbenton: no19:26
armaxI am not suggesting that19:26
*** banix has joined #openstack-neutron19:26
armaxkevinbenton: the fix for the auto-allocated plugin is still required as a stop gap19:26
armaxbut we can’t keep adding stop-gaps19:27
armaxor even worse19:27
armaxmove things the way you’re suggesting19:27
armaxas that’s gonna cause us even more side effectes19:27
armaxeffects19:27
kevinbentonno it's not19:27
armaxoh yeah?19:27
*** thorst_ has quit IRC19:27
kevinbentonyou realize what i'm suggesting is to move it closed to just DB methods, right?19:27
armaxsee where we got?19:27
armaxsince you’re started this quest?19:27
kevinbentoncloser*19:27
armaxkevinbenton: then perhaps we’re saying the same thing19:28
armaxI would need to see what you actually mean19:28
kevinbentonarmax: the retries actually got us to a point where we have a chance at running on galera19:28
*** fragatina has quit IRC19:28
*** bfernando has quit IRC19:28
armaxI am not arguing on the adoption of the retries19:28
armaxI am arguing how they were applied19:29
armaxwe’re all to blame, I am not blaming you19:29
*** fragatina has joined #openstack-neutron19:29
*** fragatina has quit IRC19:29
kevinbentonarmax: we have this same discussion every time the retry logic uncovers a bad assumption in a code-path19:30
kevinbentonarmax: it doesn't ever go anywhere19:30
armaxit’s not a bad assumption if the way retry errors surface the way they do19:30
armaxbecause they sneak up on you19:30
kevinbentonarmax: it's a bad assumption. this code litterally doesn't handle any failures to add interfaces unless they are a bad request exception19:31
armaxthe bad request comes from a retry!19:31
kevinbentonno it doesn't19:31
kevinbentonit only catches bad requests19:31
armaxcan you show me where’ the badrequest exception in the log?19:31
kevinbentonthere is no badrequest19:31
kevinbentonthat's the point19:32
armaxright19:32
kevinbentonthe auto allocate code fails to catch any errors19:32
kevinbentonthat aren't BadRequest19:32
armaxI mean the DBDuplicateEntry19:32
kevinbentonarmax: https://github.com/openstack/neutron/blob/fed5e760928fc78dfa6533139f6bae4e08f0a7af/neutron/services/auto_allocate/db.py#L272-L28019:32
armaxwhy is it coming from?19:32
kevinbentonarmax: that should be called on any exception19:32
*** vijaykc4 has joined #openstack-neutron19:33
kevinbentonarmax: otherwise you leak whatever exception it was up to the API layer without cleaning up first19:33
kevinbentonarmax: in this case it was a retry19:33
kevinbentonarmax: well a DBDuplicate19:33
kevinbentonarmax: next time it could be a connection error19:34
*** daneyon has quit IRC19:34
*** abhiraut has quit IRC19:34
armaxthat’s defensive and I can agree, but in reality a DBDuplicate shouldn’t have bubbled up19:34
kevinbentonarmax: what about a RetryRequest or a DBConnection error?19:34
armaxwe have exceptions going all the way through the call stack19:34
armaxwe have breaches of layers all over the place19:34
armaxin https://github.com/openstack/neutron/blob/fed5e760928fc78dfa6533139f6bae4e08f0a7af/neutron/services/auto_allocate/db.py#L25319:35
armaxI am asking to create a router19:35
armaxas a user of create_router I don’t care of DB error19:35
armaxs19:35
armaxall I care is if I am making the call correctly19:35
*** thorst_ has joined #openstack-neutron19:35
armaxhence the badrequest being caught19:35
armaxDB errors shouldn’t bubble up the way they do19:36
kevinbentonarmax: that assumes there is no possible backend failure19:36
armaxotherwise it’s a mess19:36
kevinbentonarmax: only expecting an exception on bad input19:36
armaxthe backend failure should be called by the layer underneath19:36
armaxand shouldn’t surface19:36
armaxif it does19:36
armaxit’s a bug19:36
armaxand should be fixed19:36
kevinbentonarmax: what happens if it continues to fail because the DB is dead?19:36
armaxnot by catching it iin the the layers above19:36
kevinbentonarmax: or there is some L3 plugin backend that isn't responding?19:36
armaxthen we should encapsulate that into BackEndIsTitsUpError19:37
kevinbentonarmax: that's why i suggested we decorate the plugin methods19:37
armaxand let the whole thing fail19:37
kevinbentonarmax: then you told me to stop19:37
armaxwell, it depends what you mean with decorating the plugin methods19:37
kevinbenton@db_api.retry_db_errors19:37
armaxI am arguing we should decorate just the parts that touch the db19:38
kevinbentondef create_router():19:38
armaxright now if the create_router() method or hypotetical one talks to a backend19:38
armaxwe need to change the implementation in such a way that it can handle the retry19:39
kevinbentonarmax: it shouldn't talk to a backend before committing19:39
armaxbecause REST calls to a backend cannot be undone19:39
armaxIt’s hypotetical!19:39
armaxI am saying we can’t add this blindly19:39
*** thorst_ has quit IRC19:40
*** mmalik4 has quit IRC19:40
armaxso to your point, perhaps a mix of adding decorator/refactoring is required19:40
kevinbentonit's better to do it blindly at the plugin layer19:40
*** mmalik4 has joined #openstack-neutron19:40
kevinbentonthan at the API layer19:40
armaxI’d argue tha it’s better to do right, at the heart of the db operation wherever it is19:41
kevinbentondo you want an improvement or do you want to wait until we refactor the entire way plugins interact with the DB?19:41
*** oanson has quit IRC19:41
*** abhiraut has joined #openstack-neutron19:41
armaxwe’d have to look at each plugin method we want to protect and see what the implications are if we apply a decorator19:42
armaxbut I still don’t like the idea that the decorator is sprinkled all over the place19:42
kevinbentonarmax: we'd also be throwing out of tree plugins under the bus if we don't automatically protect things19:42
armaxit might be wise to extract the db logic in the process in the places where it make sense19:43
armaxwhat do you mean by automatically?19:43
kevinbentonif we change so all plugins have to opt-in, that will be a regression from what we have now19:43
kevinbentonif you're okay with that. i'll pull it out of the API now and run through the code base slapping the decorator all over operations that talk to the DB19:45
*** elo has quit IRC19:46
armaxkevinbenton: let’s talk about this in more details at the mid-cycle19:48
armaxI need to look you in the eye19:48
armax:)19:48
kevinbentoni've cancelled my plane tickets19:48
kevinbentoni'm so upset over this!19:48
armaxkevinbenton: don’t be, I’ll bring candies for you19:49
*** matrohon has joined #openstack-neutron19:49
*** liusheng has quit IRC19:50
*** liusheng has joined #openstack-neutron19:50
armaxkevinbenton: we don’t happen to have a bug tracking this?19:50
kevinbentonarmax: the bug in the auto allocate plugin?19:50
armaxno19:50
armaxthe one that’s tracking the retry cleanup19:50
kevinbentonarmax: no, we don't have an "i don't like the way this works" bug yet19:51
kevinbentonarmax: you'll need to file it19:51
armaxkevinbenton: me?19:51
openstackgerritArmando Migliaccio proposed openstack/neutron: Make auto-allocate plugin handle sneaky DB errors  https://review.openstack.org/35505019:51
armaxwhy me? :'(19:52
armaxkevinbenton: ^19:52
armaxkevinbenton: wanna check if that does it?19:52
armaxfor now?19:52
kevinbentonarmax: because it's you complaining about it!19:52
armaxkevinbenton: and you don’t love me anymore?19:52
armaxHenryG: ^19:53
*** mvk has quit IRC19:54
armaxkevinbenton: I am not complaining, I am brainstorming btw19:56
*** daneyon has joined #openstack-neutron19:56
*** gouthamr has quit IRC19:59
*** john_a_joyce has joined #openstack-neutron20:01
*** john_a_joyce has quit IRC20:06
openstackgerritMerged openstack/neutron: Fix neutron_lib.constants DeprecationWarning from db.models_v2  https://review.openstack.org/35465120:08
openstackgerritSindhu Devale proposed openstack/neutron: Relocate L3HARouter DB model  https://review.openstack.org/35505920:09
*** sbalukoff has quit IRC20:10
*** sbalukoff has joined #openstack-neutron20:11
openstackgerritArmando Migliaccio proposed openstack/neutron: Make auto-allocate plugin handle sneaky DB errors  https://review.openstack.org/35505020:11
armaxkevinbenton: how about now?20:11
armax^20:11
*** banix has quit IRC20:11
armaxor I can do it differrently20:12
armaxlet me have another try20:12
*** tflynn_ has quit IRC20:14
*** tflynn has joined #openstack-neutron20:15
*** banix has joined #openstack-neutron20:15
*** sindhu has quit IRC20:17
*** gouthamr has joined #openstack-neutron20:17
*** daneyon has quit IRC20:19
*** tflynn has quit IRC20:19
*** john-a-joyce has joined #openstack-neutron20:19
*** sindhu has joined #openstack-neutron20:21
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Create router to fip namespace connection based on gateway state  https://review.openstack.org/28375720:23
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: (WIP)DVR: Enable/Disable snat redirect rules based on snat state  https://review.openstack.org/35506220:23
*** yamahata has quit IRC20:25
*** rajinir has joined #openstack-neutron20:27
*** ihrachys has joined #openstack-neutron20:33
*** vimal_ has joined #openstack-neutron20:35
openstackgerritArmando Migliaccio proposed openstack/neutron: Make auto-allocate plugin handle sneaky DB errors  https://review.openstack.org/35505020:37
armaxkevinbenton: ok, see now if it’s to your liking ^20:37
*** haplo37_ has joined #openstack-neutron20:39
*** gvrangan has joined #openstack-neutron20:40
openstackgerritMerged openstack/neutron: Use neutron-lib add_validator for registration  https://review.openstack.org/35025920:40
*** parora has quit IRC20:42
kevinbentonarmax: you wanna write a test for that?20:44
armaxkevinbenton: no20:44
armaxI don’t want you20:44
armaxto*20:44
armaxbut if you want me to20:44
armaxI can write one20:44
openstackgerritAdolfo Duarte proposed openstack/neutron: Add RPC layer for Trunk Plugin and driver plumbing  https://review.openstack.org/34766220:48
*** agireud has quit IRC20:48
*** freerunner has quit IRC20:49
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting for a scheduled upgrade, but should return to service momentarily: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101394.html20:50
*** freerunner has joined #openstack-neutron20:51
*** daneyon has joined #openstack-neutron20:55
*** agireud has joined #openstack-neutron20:55
openstackgerritKevin Benton proposed openstack/neutron: Increase rally port count and add quotas  https://review.openstack.org/35507820:56
*** gouthamr has quit IRC20:58
-openstackstatus- NOTICE: The Mediawiki service at wiki.openstack.org will be offline from 21:00 UTC until approximately 23:00 UTC for a planned upgrade http://lists.openstack.org/pipermail/openstack-dev/2016-August/101395.html20:58
*** ChanServ changes topic to "The Mediawiki service at wiki.openstack.org will be offline from 21:00 UTC until approximately 23:00 UTC for a planned upgrade http://lists.openstack.org/pipermail/openstack-dev/2016-August/101395.html"20:58
*** ivar-laz_ has joined #openstack-neutron20:58
kevinbentonarmax: is openstack-health limited to gate jobs?20:58
armaxkevinbenton: yes20:58
armaxto the best of my knowledge20:59
kevinbentonarmax: ack21:01
*** ivar-lazzaro has quit IRC21:01
kevinbentonarmax: it's also based off of the testr db right?21:01
openstackgerritArmando Migliaccio proposed openstack/neutron: Reduce DB hit when checking for trunk-able ports  https://review.openstack.org/35508021:01
armaxkevinbenton: it’s gonna be subunit complaint so to speak21:01
armaxwhy?21:01
*** mmalik4_ has joined #openstack-neutron21:01
*** ivar-laz_ has quit IRC21:02
kevinbentonarmax: i really like these average runtime graphs21:03
kevinbentonarmax: but we don't have anything other than rally that actually creates a lot of ports21:03
kevinbentonarmax: so i was hoping to see the rally port list test over time21:03
*** shivrao has joined #openstack-neutron21:04
kevinbentonarmax: but it doesn't output to subunit IIUC21:04
*** mmalik4 has quit IRC21:05
*** tflynn has joined #openstack-neutron21:07
amullerkevinbenton: we need rally2subunit21:11
amullerkevinbenton: created :)21:11
*** hockeynut has joined #openstack-neutron21:11
kevinbentonamuller: yeah, it seems that would be the answer21:11
kevinbentonamuller: i need to find boris-42 and harass him :)21:11
*** fragatina has joined #openstack-neutron21:11
amullerkevinbenton: sounds like a great RFE for the rally folks, if it's technically feasible21:11
*** ivasilevskaya has joined #openstack-neutron21:14
*** ivar-lazzaro has joined #openstack-neutron21:15
*** yamahata has joined #openstack-neutron21:16
*** fragatina has quit IRC21:19
*** edmondsw has quit IRC21:22
*** lascii is now known as alaski21:23
*** mmalik4_ has quit IRC21:27
*** mmalik4_ has joined #openstack-neutron21:27
*** amuller has quit IRC21:28
*** abhiraut1 has joined #openstack-neutron21:30
*** abhiraut has quit IRC21:30
*** tonytan4ever has quit IRC21:30
openstackgerritMatt Riedemann proposed openstack/os-vif: Make Route.interface nullable  https://review.openstack.org/35509321:35
*** kobis has joined #openstack-neutron21:35
openstackgerritShih-Hao Li proposed openstack/neutron: Fix potential problem in test_router_add_interface_port  https://review.openstack.org/35509421:37
*** sindhu has quit IRC21:37
*** ihrachys has quit IRC21:39
*** gvrangan has quit IRC21:39
*** vimal_ has quit IRC21:40
*** jamesdenton has quit IRC21:41
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: (WIP)DVR: Add static routes to allow traffic directed at fixed_ip  https://review.openstack.org/29746821:41
*** vijaykc4 has quit IRC21:45
*** jistr has quit IRC21:47
*** fragatina has joined #openstack-neutron21:48
*** fragatina has quit IRC21:50
*** haplo37_ has quit IRC21:50
*** fragatina has joined #openstack-neutron21:50
*** jistr has joined #openstack-neutron21:51
openstackgerritManjeet Singh Bhatia proposed openstack/neutron: Add OVO for dns Objects  https://review.openstack.org/33469521:54
*** thorst_ has joined #openstack-neutron21:54
*** shashank_hegde has quit IRC21:55
*** gvrangan has joined #openstack-neutron21:58
*** tanvirt has quit IRC21:59
*** shashank_hegde has joined #openstack-neutron22:00
*** pradk has quit IRC22:00
*** ijw has joined #openstack-neutron22:00
*** jistr has quit IRC22:00
*** kobis has quit IRC22:01
*** kobis has joined #openstack-neutron22:03
*** shivrao has quit IRC22:04
*** shashank_hegde has quit IRC22:04
*** boden has quit IRC22:07
*** sdague has quit IRC22:07
*** tpsilva has quit IRC22:08
*** jistr has joined #openstack-neutron22:09
*** shivrao has joined #openstack-neutron22:10
openstackgerritHenry Gessau proposed openstack/neutron: Emit DeprecationWarning on using tenant_id in models  https://review.openstack.org/35253122:10
*** michauds has quit IRC22:11
*** banix has quit IRC22:14
*** ivar-lazzaro has quit IRC22:16
*** jistr has quit IRC22:19
*** iranzo has joined #openstack-neutron22:19
*** iranzo has joined #openstack-neutron22:19
*** kobis has quit IRC22:22
*** banix has joined #openstack-neutron22:24
*** tonytan4ever has joined #openstack-neutron22:25
*** jistr has joined #openstack-neutron22:25
*** lbrune has joined #openstack-neutron22:26
*** abhiraut1 has quit IRC22:28
*** abhiraut has joined #openstack-neutron22:28
*** lbrune1 has quit IRC22:28
*** ijw has quit IRC22:28
openstackgerritArmando Migliaccio proposed openstack/neutron: Make auto-allocate plugin handle sneaky DB errors  https://review.openstack.org/35505022:29
*** ivar-lazzaro has joined #openstack-neutron22:33
*** abhiraut has quit IRC22:40
*** armax has quit IRC22:40
*** abhiraut has joined #openstack-neutron22:41
*** palexster has quit IRC22:42
*** abhiraut has quit IRC22:44
*** abhiraut has joined #openstack-neutron22:45
openstackgerritMerged openstack/neutron: Add flush command to iproute in ip_lib  https://review.openstack.org/34958722:47
*** mmalik4_ has quit IRC22:47
*** tflynn_ has joined #openstack-neutron22:47
*** regXboi has joined #openstack-neutron22:49
*** regXboi has quit IRC22:50
*** lbrune has quit IRC22:51
*** tflynn has quit IRC22:51
*** john-a-joyce has quit IRC22:52
*** tflynn_ has quit IRC22:52
*** john-a-joyce has joined #openstack-neutron22:52
*** markvoelker has quit IRC22:53
-openstackstatus- NOTICE: ok https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong23:00
*** ChanServ changes topic to "ok https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong"23:00
*** vishwanathj has quit IRC23:02
*** iranzo has quit IRC23:03
*** armax has joined #openstack-neutron23:04
*** abhiraut has quit IRC23:05
*** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack"23:06
-openstackstatus- NOTICE: https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong23:06
*** janzian has quit IRC23:06
*** sputnik13 has quit IRC23:10
*** jckasper has quit IRC23:13
*** jckasper has joined #openstack-neutron23:13
*** jckasper has quit IRC23:14
*** jckasper has joined #openstack-neutron23:15
*** thorst_ has quit IRC23:17
openstackgerritHenry Gessau proposed openstack/neutron: [WIP] debtcollector for globals  https://review.openstack.org/35260123:19
HenryGkevinbenton: would appreciate your thought on ^^23:20
HenryGthoughts even23:20
*** harmw has quit IRC23:20
*** banix has quit IRC23:23
*** banix has joined #openstack-neutron23:24
*** thorst_ has joined #openstack-neutron23:26
*** packet_sh_ has quit IRC23:29
*** thorst_ has quit IRC23:30
*** shivrao has quit IRC23:32
*** ijw has joined #openstack-neutron23:34
*** hockeynut has quit IRC23:35
*** mickeys has quit IRC23:40
*** ijw has quit IRC23:41
*** a_ta has quit IRC23:43
*** agireud has quit IRC23:44
openstackgerritHenry Gessau proposed openstack/neutron: Add deprecation warnings to models  https://review.openstack.org/33579223:46
*** ijw has joined #openstack-neutron23:46
*** agireud has joined #openstack-neutron23:53
*** markvoelker has joined #openstack-neutron23:54
*** Swami has quit IRC23:55
*** baoli has quit IRC23:56
*** markvoelker has quit IRC23:59
*** gouthamr has joined #openstack-neutron23:59
*** yamahata has quit IRC23:59

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