Friday, 2017-03-24

*** baoli has joined #openstack-neutron00:01
*** masaki has joined #openstack-neutron00:02
*** suresh12 has joined #openstack-neutron00:04
*** radhikam has quit IRC00:06
*** donghao has joined #openstack-neutron00:10
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: full sync: correct sync order of resource  https://review.openstack.org/44845800:11
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: full_sync: sync router before port  https://review.openstack.org/44936700:11
*** suresh12 has quit IRC00:11
*** jianghuaw has quit IRC00:13
*** ventifus has quit IRC00:16
*** baoli has quit IRC00:17
openstackgerritJesse proposed openstack/neutron master: Optimize pid property in AsyncProcess class  https://review.openstack.org/43187600:17
*** baoli has joined #openstack-neutron00:17
*** ZZelle has quit IRC00:19
*** donghao has quit IRC00:19
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: full sync: correct sync order of resource  https://review.openstack.org/44845800:22
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: full_sync: sync router before port  https://review.openstack.org/44936700:22
*** seanatcisco has quit IRC00:23
*** dave-mccowan has quit IRC00:29
*** Jiong has joined #openstack-neutron00:30
*** limao has joined #openstack-neutron00:30
*** armax has joined #openstack-neutron00:30
*** lujinluo has joined #openstack-neutron00:33
*** suresh12 has joined #openstack-neutron00:35
*** suresh12 has quit IRC00:35
*** suresh12 has joined #openstack-neutron00:35
*** armax has quit IRC00:38
*** vishwanathj has joined #openstack-neutron00:39
*** hoangcx has joined #openstack-neutron00:43
*** baoli has quit IRC00:44
*** Shunli has joined #openstack-neutron00:44
*** vishwanathj has quit IRC00:44
*** baoli has joined #openstack-neutron00:45
*** dave-mccowan has joined #openstack-neutron00:48
*** baoli has quit IRC00:49
*** bobmel has joined #openstack-neutron00:54
*** dave-mcc_ has joined #openstack-neutron00:55
*** reedip has quit IRC00:56
*** dave-mccowan has quit IRC00:57
*** mickeys has quit IRC00:58
*** bobmel has quit IRC00:59
*** armax has joined #openstack-neutron01:02
*** dave-mccowan has joined #openstack-neutron01:10
*** dave-mcc_ has quit IRC01:11
*** baoli has joined #openstack-neutron01:11
*** baoli has quit IRC01:15
*** donghao has joined #openstack-neutron01:16
*** MasterOfBugs has joined #openstack-neutron01:17
*** donghao has quit IRC01:20
*** abhiraut has quit IRC01:21
*** armax has quit IRC01:22
*** zhurong has joined #openstack-neutron01:26
*** cuongnv has joined #openstack-neutron01:28
*** yedongcan has joined #openstack-neutron01:33
*** sridharg has joined #openstack-neutron01:34
*** jamielennox is now known as jamielennox|away01:39
*** mickeys has joined #openstack-neutron01:40
*** vishwanathj has joined #openstack-neutron01:40
*** SumitNaiksatam has left #openstack-neutron01:41
*** nplanel has joined #openstack-neutron01:42
*** vishwanathj has quit IRC01:45
*** jamielennox|away is now known as jamielennox01:48
*** gcb has joined #openstack-neutron01:48
*** gongysh has joined #openstack-neutron01:49
*** ihrachys has joined #openstack-neutron01:52
*** Jiong has quit IRC01:52
*** reedip has joined #openstack-neutron01:53
*** xianghui has joined #openstack-neutron01:57
*** s3wong has quit IRC01:59
*** vishwanathj has joined #openstack-neutron02:06
*** vishwanathj has quit IRC02:07
*** vishwanathj has joined #openstack-neutron02:09
*** vishwanathj has quit IRC02:10
*** vishwanathj has joined #openstack-neutron02:11
*** blogan has quit IRC02:11
*** nyechiel has joined #openstack-neutron02:11
*** Sukhdev has quit IRC02:17
*** suresh12 has quit IRC02:18
openstackgerritIhar Hrachyshka proposed openstack/neutron master: WIP convert class throttler into a function  https://review.openstack.org/44935502:18
*** ihrachys has quit IRC02:18
*** yuanying_ has joined #openstack-neutron02:19
*** donghao has joined #openstack-neutron02:19
*** yuanying_ has quit IRC02:24
*** donghao has quit IRC02:24
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: rpc from odl to networking-odl  https://review.openstack.org/44940202:26
*** masaki has quit IRC02:27
*** karthikpr has joined #openstack-neutron02:29
*** tbachman_ has joined #openstack-neutron02:29
*** fzdarsky_ has joined #openstack-neutron02:29
*** tbachman has quit IRC02:30
*** tbachman_ is now known as tbachman02:30
*** fzdarsky|afk has quit IRC02:32
*** radhikam has joined #openstack-neutron02:33
*** namnh has joined #openstack-neutron02:33
*** karthikpr has quit IRC02:33
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: full sync: correct sync order of resource  https://review.openstack.org/44845802:33
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: full_sync: sync router before port  https://review.openstack.org/44936702:33
openstackgerritMerged openstack/networking-sfc master: tempest: Switch to local copy of tempset/scenario/manager.py  https://review.openstack.org/43990802:34
*** gkadam has joined #openstack-neutron02:35
*** limao has quit IRC02:36
*** limao_ has joined #openstack-neutron02:36
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: rpc from odl to networking-odl  https://review.openstack.org/44940202:36
*** tbarron has quit IRC02:38
*** knikolla has quit IRC02:38
*** aleph1 has quit IRC02:38
*** masaki has joined #openstack-neutron02:40
*** suresh12 has joined #openstack-neutron02:40
*** iyamahat has quit IRC02:43
*** yamahata has quit IRC02:43
*** reedip has quit IRC02:43
*** suresh12 has quit IRC02:44
*** aleph1 has joined #openstack-neutron02:47
*** raginbajin has quit IRC02:47
*** tbarron has joined #openstack-neutron02:48
*** knikolla has joined #openstack-neutron02:50
*** annp has joined #openstack-neutron02:51
*** raginbajin has joined #openstack-neutron02:52
*** bobmel has joined #openstack-neutron02:55
*** tuanluong has joined #openstack-neutron02:56
*** yuanying has quit IRC02:58
*** nyechiel has quit IRC02:58
openstackgerritMerged openstack/networking-sfc master: Use data_utils from tempest.lib  https://review.openstack.org/44720502:58
*** gcb has quit IRC02:58
*** gcb has joined #openstack-neutron02:59
*** mickeys has quit IRC02:59
*** bobmel has quit IRC03:00
*** radhikam has quit IRC03:01
openstackgerritxiaodongwang991481 proposed openstack/networking-sfc master: fix security group error in multinode tempest tests  https://review.openstack.org/44779703:03
*** zhhuabj has quit IRC03:05
*** dave-mccowan has quit IRC03:08
*** reedip has joined #openstack-neutron03:09
*** ramishra has joined #openstack-neutron03:12
*** jerryz has quit IRC03:13
*** fnaval has quit IRC03:13
*** fnaval has joined #openstack-neutron03:14
*** mickeys has joined #openstack-neutron03:16
*** zhhuabj has joined #openstack-neutron03:19
*** gongysh has quit IRC03:20
*** IRCFrEAK has joined #openstack-neutron03:22
*** IRCFrEAK has quit IRC03:24
*** psahoo has joined #openstack-neutron03:29
*** lujinluo has quit IRC03:35
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: rpc from odl to networking-odl  https://review.openstack.org/44940203:41
*** reedip has quit IRC03:43
*** Matias has quit IRC03:44
*** Matias has joined #openstack-neutron03:45
*** yamahata has joined #openstack-neutron03:47
*** reedip has joined #openstack-neutron03:48
*** fandi has joined #openstack-neutron03:53
*** bobmel has joined #openstack-neutron03:56
*** jhershbe has joined #openstack-neutron03:59
*** vthapar has joined #openstack-neutron04:00
*** fnaval has quit IRC04:00
*** bobmel has quit IRC04:00
*** lujinluo has joined #openstack-neutron04:01
*** hoangcx_ has joined #openstack-neutron04:04
*** zhurong has quit IRC04:07
*** hoangcx has quit IRC04:07
*** karthikpr has joined #openstack-neutron04:09
*** limao_ has quit IRC04:09
*** limao has joined #openstack-neutron04:10
*** jhershbe has quit IRC04:13
*** limao has quit IRC04:14
*** mickeys has quit IRC04:17
*** suresh12 has joined #openstack-neutron04:19
*** zhhuabj has quit IRC04:20
*** links has joined #openstack-neutron04:20
*** yuanying has joined #openstack-neutron04:20
*** zhhuabj has joined #openstack-neutron04:21
*** donghao has joined #openstack-neutron04:23
*** hoangcx has joined #openstack-neutron04:26
*** hoangcx_ has quit IRC04:29
*** radhikam has joined #openstack-neutron04:29
*** IRCFrEAK has joined #openstack-neutron04:30
*** amotoki has joined #openstack-neutron04:31
*** marst has joined #openstack-neutron04:31
*** IRCFrEAK has quit IRC04:31
*** donghao has quit IRC04:31
*** iyamahat has joined #openstack-neutron04:31
*** ihrachys has joined #openstack-neutron04:32
*** amotoki has quit IRC04:35
*** gcheresh_ has joined #openstack-neutron04:40
*** ratailor has joined #openstack-neutron04:40
*** limao has joined #openstack-neutron04:43
*** amotoki has joined #openstack-neutron04:44
*** yuanying_ has joined #openstack-neutron04:46
*** namnh has quit IRC04:47
*** reedip has quit IRC04:50
*** yuanying_ has quit IRC04:51
openstackgerritfumihiko kakuma proposed openstack/neutron-dynamic-routing master: Enable neutron-dynamic-routing scheduler  https://review.openstack.org/42754104:51
openstackgerritRuijing proposed openstack/networking-odl master: Fix devstack failure inside vagrant  https://review.openstack.org/43782604:52
*** xianghui has quit IRC04:52
*** janki has joined #openstack-neutron04:53
*** ayogi has joined #openstack-neutron04:53
*** reedip has joined #openstack-neutron04:54
*** shausy has joined #openstack-neutron04:55
*** xianghui has joined #openstack-neutron04:55
*** udesale has joined #openstack-neutron04:57
openstackgerritfumihiko kakuma proposed openstack/neutron-dynamic-routing master: Enable neutron-dynamic-routing scheduler  https://review.openstack.org/42754104:58
*** amotoki has quit IRC04:58
*** mickeys has joined #openstack-neutron04:59
*** amotoki has joined #openstack-neutron05:01
*** fragatin_ has joined #openstack-neutron05:01
*** fragati__ has joined #openstack-neutron05:03
*** Sukhdev has joined #openstack-neutron05:04
*** gcheresh_ has quit IRC05:04
*** fragatina has quit IRC05:05
*** fragatin_ has quit IRC05:06
*** fandi has quit IRC05:07
*** fragati__ has quit IRC05:07
*** sridharg has quit IRC05:08
*** amotoki has quit IRC05:09
*** udesale has quit IRC05:11
*** udesale__ has joined #openstack-neutron05:11
*** armax has joined #openstack-neutron05:11
*** zhurong has joined #openstack-neutron05:16
openstackgerritxiaodongwang991481 proposed openstack/networking-sfc master: fix security group error in multinode tempest tests  https://review.openstack.org/44779705:18
*** pgadiya has joined #openstack-neutron05:18
*** udesale__ has quit IRC05:20
*** karthikpr has quit IRC05:20
*** udesale has joined #openstack-neutron05:20
*** udesale has quit IRC05:21
*** fragatina has joined #openstack-neutron05:21
*** udesale has joined #openstack-neutron05:22
*** iyamahat has quit IRC05:25
*** fragatina has quit IRC05:25
*** Sukhdev has quit IRC05:29
*** gongysh has joined #openstack-neutron05:29
*** donghao has joined #openstack-neutron05:30
openstackgerritIhar Hrachyshka proposed openstack/neutron master: Throttle SIGHUPs to keepalived  https://review.openstack.org/40709905:30
openstackgerritrajiv proposed openstack/networking-odl master: V2.0 Driver for LBaaS V2.0  https://review.openstack.org/44943205:31
*** wu_ng has joined #openstack-neutron05:31
*** fragatina has joined #openstack-neutron05:33
*** fragatina has quit IRC05:33
*** fragatina has joined #openstack-neutron05:34
*** donghao has quit IRC05:34
openstackgerritfumihiko kakuma proposed openstack/neutron-dynamic-routing master: Add tempest scenario tests  https://review.openstack.org/40777905:35
*** reedip has quit IRC05:38
*** anilvenkata has joined #openstack-neutron05:41
*** reedip has joined #openstack-neutron05:42
openstackgerritArmando Migliaccio proposed openstack/neutron master: Inherit segmentation details for trunk subports if requested  https://review.openstack.org/43675605:48
*** sridharg has joined #openstack-neutron05:49
*** masaki has quit IRC05:51
*** masaki has joined #openstack-neutron05:51
*** Jack_Iv has joined #openstack-neutron05:52
*** mdnadeem has joined #openstack-neutron05:53
*** karthikpr has joined #openstack-neutron05:55
*** bobmel has joined #openstack-neutron05:57
*** amotoki has joined #openstack-neutron05:58
*** karthikpr has quit IRC05:59
*** mickeys has quit IRC06:00
*** iranzo has joined #openstack-neutron06:00
openstackgerritlan proposed openstack/os-vif master: Remove log translations  https://review.openstack.org/44786506:00
*** bobmel has quit IRC06:02
*** reedip has quit IRC06:05
*** yee37914 has quit IRC06:06
*** yee379 has joined #openstack-neutron06:08
*** Jack_Iv has quit IRC06:08
*** zenirc369 has joined #openstack-neutron06:10
*** yee379 has quit IRC06:12
*** aojea has joined #openstack-neutron06:12
*** yee379 has joined #openstack-neutron06:12
*** suresh12 has quit IRC06:13
*** yee379 has quit IRC06:14
*** yee379 has joined #openstack-neutron06:15
*** aojea has quit IRC06:18
*** yedongcan has quit IRC06:19
*** mickeys has joined #openstack-neutron06:23
anilvenkatakevinbenton, can you please review https://review.openstack.org/#/c/445996/ ?06:23
*** yedongcan has joined #openstack-neutron06:26
*** gcb has quit IRC06:28
*** suresh12 has joined #openstack-neutron06:29
*** tmorin has joined #openstack-neutron06:29
*** armax has quit IRC06:29
*** MasterOfBugs has quit IRC06:32
*** suresh12 has quit IRC06:33
*** reedip has joined #openstack-neutron06:42
*** slaweq__ is now known as slaweq_06:43
*** yee379 has quit IRC06:44
*** yee379 has joined #openstack-neutron06:44
*** ihrachys has quit IRC06:44
*** gouthamr has joined #openstack-neutron06:45
*** gcb has joined #openstack-neutron06:45
*** mvk has quit IRC06:47
*** Jack_Iv has joined #openstack-neutron06:49
*** tmorin has quit IRC06:49
*** gongysh has quit IRC06:51
openstackgerritThomas Morin proposed openstack/neutron master: delete_flows shall only touch flows with the bridge cookie  https://review.openstack.org/43795206:53
*** jhershbe has joined #openstack-neutron06:57
*** andreas_s has joined #openstack-neutron06:58
openstackgerritzhangyanxian proposed openstack/neutron-dynamic-routing master: Optimize the link address for fetching git code  https://review.openstack.org/44733706:59
openstackgerritzhangyanxian proposed openstack/neutron-dynamic-routing master: Optimize the link address for fetching git code  https://review.openstack.org/44733707:00
*** tbh has joined #openstack-neutron07:00
*** gongysh has joined #openstack-neutron07:02
*** ritz_ has quit IRC07:04
*** jhershbe has quit IRC07:09
openstackgerritOpenStack Proposal Bot proposed openstack/networking-midonet master: Imported Translations from Zanata  https://review.openstack.org/44944607:12
*** gouthamr has quit IRC07:12
*** tommylikehu_ has joined #openstack-neutron07:15
*** iwamoto has joined #openstack-neutron07:17
openstackgerritMerged openstack/neutron master: Configure tempest.conf via test-config phase  https://review.openstack.org/44918207:18
*** tommylikehu_ has quit IRC07:20
*** pmannidi has quit IRC07:21
*** manheim has joined #openstack-neutron07:22
*** manheim has quit IRC07:22
*** manheim has joined #openstack-neutron07:23
*** ralonsoh has joined #openstack-neutron07:23
*** Amit82 has joined #openstack-neutron07:28
*** pmannidi has joined #openstack-neutron07:28
Amit82Hi All, can we add VMs directly to external provider N/w? I don't want to use floating IP for accessing internet.07:29
*** eezhova has joined #openstack-neutron07:29
Amit82Sry, I don't want to use floating IP for accessing VMs via SSH07:30
*** tesseract has joined #openstack-neutron07:31
*** Jack_Iv has quit IRC07:31
*** pmannidi has quit IRC07:32
*** donghao has joined #openstack-neutron07:34
*** Jack_Iv has joined #openstack-neutron07:36
*** ltomasbo|away is now known as ltomasbo07:38
*** jprovazn has joined #openstack-neutron07:40
*** Jack_Iv has quit IRC07:40
*** donghao has quit IRC07:41
*** radhikam has quit IRC07:41
*** oreillyd has joined #openstack-neutron07:42
*** oreillyd_ has joined #openstack-neutron07:42
*** bobmel has joined #openstack-neutron07:43
*** ekuris has quit IRC07:43
*** ekuris has joined #openstack-neutron07:45
*** bobmel has quit IRC07:47
*** ekuris has quit IRC07:50
*** matrohon has joined #openstack-neutron07:54
*** Jack_Iv has joined #openstack-neutron07:54
*** gcheresh_ has joined #openstack-neutron07:54
*** jaosorior has joined #openstack-neutron07:55
*** jhershbe has joined #openstack-neutron07:58
openstackgerritMerged openstack/ovsdbapp master: Fix all broken jobs  https://review.openstack.org/44848107:58
*** zzzeek has quit IRC08:00
*** gcheresh_ has quit IRC08:00
*** zzzeek has joined #openstack-neutron08:01
*** iwamoto has quit IRC08:02
*** jlinkes has joined #openstack-neutron08:05
*** yamahata has quit IRC08:07
*** ralonsoh has quit IRC08:07
*** gcheresh_ has joined #openstack-neutron08:08
*** hoangcx_ has joined #openstack-neutron08:09
*** hoangcx has quit IRC08:11
openstackgerritTerry Wilson proposed openstack/ovsdbapp master: Fix Python 3 compatibility in idlutils  https://review.openstack.org/44876308:11
*** tmorin has joined #openstack-neutron08:15
*** amarao has joined #openstack-neutron08:15
*** mriedem has quit IRC08:16
*** eezhova has quit IRC08:17
*** aojea has joined #openstack-neutron08:18
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Improve validation of supported QoS rules  https://review.openstack.org/42694608:18
*** aojea has quit IRC08:18
*** aojea has joined #openstack-neutron08:19
*** janki has quit IRC08:20
*** nplanel has quit IRC08:22
*** gongysh has quit IRC08:22
*** jlibosva has joined #openstack-neutron08:23
*** amoralej|off is now known as amoralej08:23
openstackgerritlan proposed openstack/os-vif master: Remove log translations  https://review.openstack.org/44786508:25
*** manheim has quit IRC08:33
*** matrohon has quit IRC08:37
openstackgerritThomas Morin proposed openstack/neutron master: delete_flows shall only touch flows with the bridge cookie  https://review.openstack.org/43795208:38
*** gongysh has joined #openstack-neutron08:38
*** matrohon has joined #openstack-neutron08:41
*** aarefiev_pto is now known as aarefiev08:42
*** gongysh has quit IRC08:43
*** bobmel has joined #openstack-neutron08:43
*** jpena|off is now known as jpena08:44
*** jhershbe has quit IRC08:45
*** bobmel has quit IRC08:48
*** sean-k-mooney has quit IRC08:48
*** reedip has quit IRC08:49
*** reedip has joined #openstack-neutron08:50
*** eezhova has joined #openstack-neutron08:51
openstackgerritKevin Benton proposed openstack/neutron master: Move conntrack zones to IPTablesFirewall  https://review.openstack.org/44135308:52
*** ralonsoh has joined #openstack-neutron08:52
*** hoangcx has joined #openstack-neutron08:53
*** lujinluo has quit IRC08:54
*** gcheresh_ has quit IRC08:55
*** hoangcx_ has quit IRC08:56
*** jpena is now known as jpena|off08:57
*** alex_xu has quit IRC08:58
*** Jack_Iv has quit IRC08:58
*** alex_xu has joined #openstack-neutron08:59
*** alex_xu has quit IRC09:01
*** tommylikehu_ has joined #openstack-neutron09:02
*** alex_xu has joined #openstack-neutron09:02
*** tommylikehu_ has quit IRC09:02
*** reedip has quit IRC09:03
*** manheim has joined #openstack-neutron09:03
*** jpena|off is now known as jpena09:04
openstackgerritPrince Nana Owusu Boateng proposed openstack/neutron-specs master: Extend Quota API to report usage statistics  https://review.openstack.org/34894709:04
*** reedip has joined #openstack-neutron09:04
openstackgerritKevin Benton proposed openstack/neutron master: Skip DHCP agent query in provisioning block setup  https://review.openstack.org/44034009:05
*** ralonsoh_ has joined #openstack-neutron09:05
*** jhershbe has joined #openstack-neutron09:06
*** alex_xu has quit IRC09:06
*** tommylikehu_ has joined #openstack-neutron09:06
openstackgerritKevin Benton proposed openstack/neutron master: Log instance interface addrs in subport test  https://review.openstack.org/43840309:07
*** alex_xu has joined #openstack-neutron09:07
openstackgerritKevin Benton proposed openstack/neutron master: Delete segments using same pattern as everywhere else  https://review.openstack.org/42944009:08
*** claudiub|2 has joined #openstack-neutron09:08
*** ralonsoh has quit IRC09:09
*** mickeys has quit IRC09:09
*** aojea_ has joined #openstack-neutron09:10
*** tommylikehu_ has quit IRC09:11
*** tommylikehu_ has joined #openstack-neutron09:11
*** tommylikehu_ has quit IRC09:11
*** jlinkes has quit IRC09:12
*** yuanying has quit IRC09:12
*** aojea has quit IRC09:12
dalvarezkevinbenton, ping, I was trying to backport this: https://review.openstack.org/#/c/449058/  but I'm afraid that we need newer version of oslo_db.sqlalchemy, do you think we can set it as "non-backportable" patch? :)09:13
kevinbentondalvarez: which part is it exploding on that's not present?09:14
kevinbentondalvarez: sa_utils.get_unique_keys?09:14
dalvarezkevinbenton, yes, that's it09:14
openstackgerritKevin Benton proposed openstack/neutron master: Move _create_snat_intf_ports_if_not_exists to DVR SP  https://review.openstack.org/42997209:14
*** udesale has quit IRC09:15
dalvarezkevinbenton, for a moment i thought you just proposed it ^ :D09:15
kevinbentondalvarez: yeah, that code is pretty gnarly IIRC09:15
kevinbentondalvarez: http://git.openstack.org/cgit/openstack/deb-python-oslo.db/tree/oslo_db/sqlalchemy/utils.py#n7209:16
dalvarezkevinbenton, so what do you suggest? i was backporting a chain of your patches and i remember that the sorting patch was needed09:16
kevinbentondalvarez: how many versions are we off?09:16
dalvarezin newton? let me check09:16
*** Jack_Iv has joined #openstack-neutron09:16
*** tommylikehu_ has joined #openstack-neutron09:18
*** tommylikehu_ has quit IRC09:18
*** tommylikehu_ has joined #openstack-neutron09:19
*** ygbo has joined #openstack-neutron09:19
*** gongysh has joined #openstack-neutron09:20
*** tommylikehu_ has quit IRC09:20
*** limao has quit IRC09:21
openstackgerritMerged openstack/neutron-lib master: Stop enforcing non-debug log translations  https://review.openstack.org/44867109:21
openstackgerritKevin Benton proposed openstack/neutron master: Move _cleanup_after_interface_removal to DVR SP  https://review.openstack.org/42997609:22
openstackgerritKevin Benton proposed openstack/neutron master: Move _create_dvr_floating_gw_port to DVR flavor  https://review.openstack.org/42997109:22
openstackgerritKevin Benton proposed openstack/neutron master: Move _create_snat_intf_ports_if_not_exists to DVR SP  https://review.openstack.org/42997209:22
openstackgerritKevin Benton proposed openstack/neutron master: Move _delete_dvr_internal_ports to DVR SP  https://review.openstack.org/42997309:22
openstackgerritKevin Benton proposed openstack/neutron master: Move _add_csnat_on_interface_create to DVR SP  https://review.openstack.org/42997409:22
openstackgerritKevin Benton proposed openstack/neutron master: Move _update_snat_v6_addrs_after_intf_update to DVR SP  https://review.openstack.org/42997509:22
*** tuanluong has quit IRC09:23
dalvarezkevinbenton, this is what we have in newton https://github.com/openstack/oslo.db/blob/stable/newton/oslo_db/sqlalchemy/utils.py#L70      (https://github.com/openstack/oslo.db/commit/4314f3224f689c660d231183404f3af78310016e)09:24
openstackgerritMerged openstack/networking-ovn master: Switch trunk/cbs/buildlogs to use https  https://review.openstack.org/44805609:24
openstackgerritKevin Benton proposed openstack/neutron master: [wip]: some indexes  https://review.openstack.org/43806309:25
kevinbentondalvarez: so is it as easy as using the private method?09:26
dalvarezkevinbenton, i'll check, looks like so :)09:26
*** ralonsoh_ is now known as ralonsoh09:27
*** lucas-afk is now known as lucasagomes09:29
kevinbentondalvarez: gtg, battery is dying so it means bedtime. :)09:29
dalvarezkevinbenton, lol sleep tight! :)09:29
dalvarezand thanks!09:30
*** xianghui has quit IRC09:30
*** tmorin has quit IRC09:33
*** tmorin has joined #openstack-neutron09:34
*** davidsha has joined #openstack-neutron09:34
*** janki has joined #openstack-neutron09:38
*** jhershbe has quit IRC09:40
*** reedip has quit IRC09:41
*** donghao has joined #openstack-neutron09:41
openstackgerritThomas Morin proposed openstack/neutron master: delete_flows shall only touch flows with the bridge cookie  https://review.openstack.org/43795209:42
*** yedongcan1 has joined #openstack-neutron09:43
*** nplanel has joined #openstack-neutron09:43
*** yedongcan has quit IRC09:46
*** fzdarsky_ is now known as fzdarsky09:46
*** Shunli has quit IRC09:46
openstackgerritMerged openstack/os-vif master: Remove log translations  https://review.openstack.org/44786509:47
*** donghao has quit IRC09:48
*** yamamoto has quit IRC09:50
*** gkadam has quit IRC09:51
*** prateek has joined #openstack-neutron09:51
*** yuanying has joined #openstack-neutron09:54
*** mvk has joined #openstack-neutron09:57
tmorinhi jlibosva: perhaps you had a look at my comments, I finally understood the reason for the qos fullstack test failures with https://review.openstack.org/#/c/437952/10:01
*** karthiks is now known as karthiks_afk10:02
jlibosvatmorin: hi, I haven't got to it yet10:02
openstackgerritThomas Morin proposed openstack/neutron master: delete_flows shall only touch flows with the bridge cookie  https://review.openstack.org/43795210:02
tmorinjlibosva: The qos extension, when calling uinstall_flows on an OVSCookieBridge, ends up hitting bug 1557620 (doing a flow operation with the wrong cookie), but this issue was silent because the delete_flow code was not specific to a cookie... until  this change... which the fix for this bug depends on...10:03
openstackbug 1557620 in neutron "OVSCookieBridge not applying extension cookie to all flow operations" [Medium,In progress] https://launchpad.net/bugs/1557620 - Assigned to Thomas Morin (tmmorin-orange)10:03
tmorinjlibosva: the fullstack issue is thus fixed by doing this: https://review.openstack.org/#/c/437952/26/neutron/plugins/ml2/drivers/openvswitch/agent/extension_drivers/qos_driver.py10:04
*** pcaruana has joined #openstack-neutron10:05
*** cuongnv has quit IRC10:07
*** ociuhandu has joined #openstack-neutron10:08
*** annp has quit IRC10:10
*** mickeys has joined #openstack-neutron10:10
*** mickeys has quit IRC10:14
*** karthik__ has joined #openstack-neutron10:16
*** shausy has quit IRC10:18
*** karthik__ has quit IRC10:21
*** manheim has quit IRC10:24
*** gcheresh_ has joined #openstack-neutron10:25
*** manheim has joined #openstack-neutron10:26
*** reedip has joined #openstack-neutron10:28
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Add "default" behaviour to QoS policies  https://review.openstack.org/42830410:29
*** boden has joined #openstack-neutron10:29
*** matrohon has quit IRC10:30
*** ZZelle_ is now known as ZZelle10:31
*** wu_ng has quit IRC10:31
*** karthikpr has joined #openstack-neutron10:33
*** Amit82 has quit IRC10:34
*** manheim has quit IRC10:35
*** manheim has joined #openstack-neutron10:36
openstackgerritThomas Morin proposed openstack/neutron master: Refactor OVSCookieBridge: always use bridge cookie  https://review.openstack.org/42575610:37
*** gongysh has quit IRC10:37
*** karthikpr has quit IRC10:38
*** nicolasbock has joined #openstack-neutron10:38
*** masaki has quit IRC10:39
openstackgerritiswarya vakati proposed openstack/neutron master: Log messages are no longer being translated. This removes all use of the _LE, _LI, and _LW translation markers to simplify logging and to avoid confusion with new contributions  https://review.openstack.org/44852910:39
*** hoangcx has quit IRC10:39
*** zhurong has quit IRC10:40
*** matrohon has joined #openstack-neutron10:41
*** nyechiel has joined #openstack-neutron10:41
*** tbachman has quit IRC10:44
*** bobmel has joined #openstack-neutron10:44
*** jlibosva has quit IRC10:47
*** bobmel has quit IRC10:49
*** yamamoto has joined #openstack-neutron10:49
*** karthikpr has joined #openstack-neutron10:50
*** yamamoto_ has joined #openstack-neutron10:50
*** jprovazn has quit IRC10:51
*** bobmel has joined #openstack-neutron10:53
*** bobmel has quit IRC10:53
*** bobmel has joined #openstack-neutron10:53
*** yamamoto has quit IRC10:54
*** karthikpr has quit IRC10:54
*** gcb has quit IRC10:56
*** jlinkes has joined #openstack-neutron11:02
*** armax has joined #openstack-neutron11:02
*** yamamoto_ has quit IRC11:05
*** yamamoto has joined #openstack-neutron11:05
*** nyechiel has quit IRC11:06
*** karthikpr has joined #openstack-neutron11:07
*** karthikpr has quit IRC11:11
*** mickeys has joined #openstack-neutron11:11
*** kzaitsev_ws has joined #openstack-neutron11:12
kzaitsev_wsI'm getting a wierd error from neutron and oslo.privsep on my devstack installation. http://paste.openstack.org/show/604027/ here is a relevant bit from logs11:13
kzaitsev_wsthe interesting part for me is 'privsep log: root is not in the sudoers file.  This incident will be reported.'11:14
kzaitsev_wsand also adding root to the sudoers file obviously solves the problem11:14
kzaitsev_wsbut that means that at some point it's trying to sun sudo as root and fails. that feels wierd at least11:14
kzaitsev_wswould appreciate any thoughts on what might be misconfigured here11:15
*** armax has quit IRC11:16
*** mickeys has quit IRC11:16
*** yamamoto has quit IRC11:17
*** udesale has joined #openstack-neutron11:19
*** jlibosva has joined #openstack-neutron11:19
*** tommylikehu_ has joined #openstack-neutron11:21
*** sdague has joined #openstack-neutron11:21
*** udesale has quit IRC11:21
*** vthapar has quit IRC11:22
*** nplanel has quit IRC11:24
*** tommylikehu_ has quit IRC11:27
*** Matias has quit IRC11:27
*** gongysh has joined #openstack-neutron11:28
*** seanatcisco has joined #openstack-neutron11:28
*** seanatci_ has joined #openstack-neutron11:32
*** seanatcisco has quit IRC11:33
fricklerany ipv6 folk around? I'm still digging into the metadata issue discussed on the ML. it seems that the v6 subnet gets a dhcp_server assigned even when mode=slaac, which breaks the creation of the network info in http://git.openstack.org/cgit/openstack/nova/tree/nova/virt/netutils.py#n26711:33
*** pgadiya has quit IRC11:38
*** Matias has joined #openstack-neutron11:39
*** alexpilotti has quit IRC11:40
*** alexpilotti has joined #openstack-neutron11:41
*** alexpilotti has quit IRC11:45
openstackgerritMerged openstack/networking-ovn master: Updated from global requirements  https://review.openstack.org/44696911:50
*** jpena is now known as jpena|lunch11:56
*** jamesdenton has quit IRC11:56
*** seanatci_ has quit IRC11:58
*** tbachman has joined #openstack-neutron11:59
*** Matias has quit IRC12:00
*** udesale has joined #openstack-neutron12:01
*** Matias has joined #openstack-neutron12:01
*** tmorin has quit IRC12:02
*** links has quit IRC12:02
*** links has joined #openstack-neutron12:05
*** jhershbe has joined #openstack-neutron12:06
*** yamamoto has joined #openstack-neutron12:06
*** sputnik13 has joined #openstack-neutron12:10
*** mickeys has joined #openstack-neutron12:12
*** alexpilotti has joined #openstack-neutron12:13
*** lucasagomes is now known as lucas-hungry12:14
*** nyechiel has joined #openstack-neutron12:14
*** yamamoto has quit IRC12:15
*** mickeys has quit IRC12:16
*** gongysh has quit IRC12:17
*** cshastri has joined #openstack-neutron12:17
*** karthikpr has joined #openstack-neutron12:17
*** ratailor has quit IRC12:17
*** karthikpr has quit IRC12:18
*** wolverineav has joined #openstack-neutron12:20
*** manheim has quit IRC12:20
*** manheim has joined #openstack-neutron12:21
*** nyechiel has quit IRC12:23
openstackgerritMerged openstack/neutron master: Fix some reST field lists in docstrings  https://review.openstack.org/44883612:27
*** jhershbe has quit IRC12:29
*** yedongcan1 has left #openstack-neutron12:30
*** janki has quit IRC12:30
*** ayogi has quit IRC12:31
*** rkukura has quit IRC12:32
*** rkukura_ has joined #openstack-neutron12:32
*** jhershbe has joined #openstack-neutron12:32
*** rkukura_ has quit IRC12:36
*** Jack_Iv has quit IRC12:37
*** jhershbe has quit IRC12:38
*** seanatcisco has joined #openstack-neutron12:38
*** tbh has quit IRC12:38
openstackgerritCedric Brandily proposed openstack/neutron-fwaas master: Remove pyroute2.netns.setns patch  https://review.openstack.org/44961012:39
*** yuanying_ has joined #openstack-neutron12:39
*** tmorin has joined #openstack-neutron12:39
*** tmorin has quit IRC12:41
*** tmorin has joined #openstack-neutron12:41
*** udesale has quit IRC12:42
*** yuanying_ has quit IRC12:42
*** haleyb has joined #openstack-neutron12:45
*** seanatcisco has quit IRC12:45
*** cshastri has quit IRC12:46
*** donghao has joined #openstack-neutron12:47
*** psahoo has quit IRC12:48
openstackgerritJakub Libosvar proposed openstack/neutron master: Throttle SIGHUPs to keepalived  https://review.openstack.org/40709912:51
*** donghao has quit IRC12:53
*** charliejllewelly has joined #openstack-neutron12:54
*** david-lyle has quit IRC12:55
*** Jack_Iv has joined #openstack-neutron12:55
charliejllewellyHey all, I am trying to run a nested OpenStack to help up with our CI. I have disabled port security to allow PXE to work however I have an issue where our nodes also create additional interfaces which have MAC addresses not known to neutron12:56
*** ramishra has quit IRC12:56
charliejllewellyThis means that no openflow rules are created to match the traffic from the new interfaces causing traffic to blackhole12:56
charliejllewellyis it possible to associate multiple MAC's with ports with port security disabled12:57
charliejllewellyor is there a better way?12:57
*** shewless has joined #openstack-neutron13:01
*** amuller has joined #openstack-neutron13:01
*** liangy has joined #openstack-neutron13:02
*** jpena|lunch is now known as jpena13:03
*** karthik__ has joined #openstack-neutron13:03
*** anilvenkata has quit IRC13:04
*** karthik__ has quit IRC13:04
*** mriedem has joined #openstack-neutron13:06
*** karthikpr has joined #openstack-neutron13:07
*** gouthamr has joined #openstack-neutron13:08
*** ihrachys has joined #openstack-neutron13:11
*** karthikpr has quit IRC13:11
*** ihrachys has quit IRC13:12
*** tommylikehu_ has joined #openstack-neutron13:12
*** suresh12 has joined #openstack-neutron13:14
*** tommylikehu_ has quit IRC13:17
*** suresh12 has quit IRC13:18
*** yamamoto has joined #openstack-neutron13:19
*** jhershbe has joined #openstack-neutron13:20
*** anilvenkata has joined #openstack-neutron13:20
*** manheim has quit IRC13:21
*** seanatcisco has joined #openstack-neutron13:23
*** Jack_Iv has quit IRC13:24
*** mickeys has joined #openstack-neutron13:25
*** lucas-hungry is now known as lucasagomes13:27
openstackgerritDuan Jiong proposed openstack/networking-ovn master: Fix some reST field lists in docstrings  https://review.openstack.org/44962113:27
*** baoli has joined #openstack-neutron13:28
*** wim_declercq has quit IRC13:30
*** mickeys has quit IRC13:30
*** gcb has joined #openstack-neutron13:31
*** jhershbe has quit IRC13:31
*** seanatcisco has quit IRC13:32
tmorinjlibosva, yamamoto, ralonsoh: after a last bugfix (detected by a fullstack qos test), and addressing two comments from iwamoto, https://review.openstack.org/#/c/437952/ is waiting for you !   if you have the time...13:32
*** seanatcisco has joined #openstack-neutron13:32
jlibosvatmorin: oh, you found the issue?13:33
tmorinjlibosva: yep13:33
jlibosvatmorin: ah, I missed the messages from morning :) sorry, reading now13:33
tmorinsubtle circular issue13:33
*** karthikpr has joined #openstack-neutron13:34
tmorinwe could in fact have blindly ignored the issue, it would have been resolved by the change depending on this change  :)13:34
*** lucasagomes is now known as lucas-brb13:35
*** prateek has quit IRC13:36
*** karthikpr has quit IRC13:38
*** cleong has joined #openstack-neutron13:39
*** tommylikehu_ has joined #openstack-neutron13:42
*** manheim has joined #openstack-neutron13:42
*** jamesdenton has joined #openstack-neutron13:43
*** openstackrecheck has quit IRC13:43
*** openstackrecheck has joined #openstack-neutron13:43
*** janzian has joined #openstack-neutron13:44
tmorinjlibosva: btw, thanks for the tip to run one fullstack scenario by redefining the variable locally in the class in the testbed, it helped !! (I had missed something perhaps obvious -- )13:44
*** andreas_s has quit IRC13:45
jlibosvatmorin: fullstack has proven to be very helpful lately, it lacks a bit of better user experience13:45
tmorinjlibosva: yep, the framework is very nice, although rough in the corners13:45
tmorinjlibosva: we expect it to be very helpful for networking-bagpipe13:46
jlibosvatmorin: the plan is to make it stable and voting. Then we can work on improvements I think :)13:46
*** tommylikehu_ has quit IRC13:46
jlibosvatmorin: that plan has been around for couple of cycles now ...13:46
*** wim_declercq has joined #openstack-neutron13:47
tmorinjlibosva: well, I guess some improvements remain to make it voting :)13:47
*** ihrachys has joined #openstack-neutron13:47
ihrachysjlibosva: hi13:48
jlibosvaihrachys: morning13:48
ihrachysjlibosva: checked your revert to class for throttler13:48
jlibosvaihrachys: good findings yesterday, thanks for taking over13:49
ihrachyswhat's the reason to restore the complexity? :)13:49
*** dansmith is now known as superdan13:49
jlibosvaI find it better to implement the throttler in a class and keeping the logic in one place than need of customizing decorated class (KeepalivedManager)13:50
ihrachysyou do customize it there too, by setting the property\13:50
jlibosvawith function approach we need to have special handling as we start behaving to the function as an actual object13:51
*** donghao has joined #openstack-neutron13:51
ihrachysthe reason it's needed there in such form is not throttler issue per se13:51
ihrachysbut because the config value is not avail on class definition13:51
jlibosvait is, it's a workaround because we can't simply pass the threshold value to the decorator as config has not been loaded by the time of decorating13:51
ihrachysright13:51
jlibosvayep, so we need to do a lot of handling in the class like replacing methods13:52
ihrachysa lot as in one decoration on __init__?13:52
jlibosvawith class approach you just decorate method and then set the timeout a very common way13:52
ihrachysinstead of decoration on class definition, then override of the threshold on __init__?13:52
jlibosvano, I mean wrapping the spawn method and replacing the actual spawn method13:52
*** links has quit IRC13:53
ihrachysall I am saying is with the class, the complexity of tools used is enormous - you have update_wrapper, functools, properties, setters, __get__ override, setattr on functools.partial object13:53
*** rkukura has joined #openstack-neutron13:53
*** karthikpr has joined #openstack-neutron13:53
ihrachysand apparently it all didn't work on first attempts for those reasons13:54
jlibosvabut all those are common things to class decorators13:54
jlibosvawell it didn't work because we didn't test it properly with unittest :)13:54
ihrachysI don't know about you but I will trade a huge class for a simple decorator every day13:54
*** donghao has quit IRC13:55
ihrachysit didn't work also because the code was not really understandable for the author and reviewers13:55
ihrachysjust sayin'13:55
ihrachysI won't block on that, but I really struggle to understand why we need a class here. not every advanced tool is worth usage.13:56
*** rossella_s has joined #openstack-neutron13:56
ihrachyson contrast, the complexity of function decorator is, as I see it, in the storage logic, because of py2 instancemethod special behaviour. apart from that, it seems rather simple.13:57
jlibosvayes, I see. But we still inject methods to the function object - which implies the function needs to have a state.13:58
ihrachysjlibosva: btw it's not directly related to this discussion, but one issue with class level decorators is that the threshold won't be per instance13:58
jlibosvaand common way to introduce objects with a state is a class :)13:58
*** karthikpr has quit IRC13:58
ihrachysjlibosva: it is not with im_func usage anyway13:58
*** alexpilotti has quit IRC13:58
ihrachysthat's why it's not directly related13:58
ihrachysmaybe we should find a better place to store the state (something storable directly on instancemethod)13:59
*** alexpilotti has joined #openstack-neutron13:59
jlibosvaI didn't get the 'per instance' thingy13:59
jlibosvaclass level decorators == decorators implemented by class14:00
ihrachysjlibosva: if you want to have two managers with different thresholds14:00
jlibosvaor decorating methods?14:00
jlibosvafor different methods?14:00
ihrachyssame14:00
*** jhershbe has joined #openstack-neutron14:00
ihrachyssame class method, different instancemethod14:00
*** iyamahat has joined #openstack-neutron14:00
jlibosvaoh I see what you mean14:01
jlibosvaif I would instantiate the KeepalivedManager twice14:01
*** lucas-brb is now known as lucasagomes14:01
ihrachysas I said, in function approach, I still store it on im_func for py2, so it is not different14:01
ihrachysbut I could try to find something instance specific14:01
ihrachyslike func.im_self14:01
*** anilvenkata has quit IRC14:02
*** alexpilotti has quit IRC14:03
jlibosvaihrachys: I still don't know which approach are you talking about. Decorator is implemented using "def' or "calss" ?14:03
ihrachysjlibosva: function approach == def14:04
jlibosvaok14:04
ihrachysagain, I think with class we could also find some instance level space; but that would also imply decoration in __init__ not in class def14:05
jlibosvaok, now I see what you mean. If I have two instances of class with 'method', then 'method.im_func' are always a single object14:05
ihrachyswhich is my point that this part of complexity is not inherent to the throttler implementation but to keepalived config option loading time14:06
*** mdnadeem has quit IRC14:06
*** murphy_zhao has quit IRC14:06
ihrachysjlibosva: yes, im_func is the same. but we may replace it with im_self to get it per instance14:06
jlibosvaso I think if we need to have a state, we should instantiate an object from a class :)14:06
ihrachysthat's one approach, yes14:07
ihrachysmy concern is not about class vs def14:07
jlibosvaihrachys: maybe we should find some good document that would describe decorator using class with a pattern and that would explain the __get__ descriptor14:07
ihrachysbut code complexity, manual inspection wise14:07
*** tommylikehu_ has joined #openstack-neutron14:07
*** dave-mccowan has joined #openstack-neutron14:08
ihrachysmaybe it's just me and everyone grasps a __get__ returning a functools.partial object with a attr set to proxy a function14:08
*** donghao has joined #openstack-neutron14:08
jlibosvathe only not-clear thing there is the __get__ but that is a template for creating decorators implemented by class, that are supposed to decorate methods14:08
ihrachysjlibosva: I would say if you need to point to a document on a feature that is not widely used in tree, then maybe you shouldn't :P14:09
ihrachysanyhoo14:09
*** scottda has joined #openstack-neutron14:09
ihrachysI will leave it up to you, you are the author14:09
ihrachysbut I can't vouch for that piece of code, it's too complex for me14:10
jlibosvaihrachys: I like the class better but I want an agreement :)14:10
jlibosvaihrachys: the functool.wraps calls update_wrapper and returns partial object too14:10
*** bobmel has quit IRC14:11
jlibosvaihrachys: would it help if I would write a comment for every call so it's clearer why it's there?14:11
ihrachysyeah, but it isolates the reader from complexity, compare @functools.wraps(f) to __get__ you have :)14:11
jlibosvaihrachys: in __get__14:11
jlibosvaihrachys: I get your point and I agree. That's why maybe if every call would have an explanation it would ease understanding to reader14:12
ihrachysit may indeed help me to understand the piece of code, not to reduce complexity.14:12
*** psahoo has joined #openstack-neutron14:12
*** psahoo is now known as psahoo|away14:12
ihrachysI can live with complexity, and comments are good idea anyway14:12
*** thomas_oneill has joined #openstack-neutron14:13
*** wolverineav has quit IRC14:14
ihrachysjlibosva: you may want to ask someone else for feedback, I am biased14:18
electrocucarachaihrachys: ping about the default neutron quotas14:18
ihrachyselectrocucaracha: sup14:18
electrocucarachaihrachys: I asked that to openstack operators group http://lists.openstack.org/pipermail/openstack-operators/2017-March/013019.html14:19
electrocucarachaihrachys: short answer, their major concern was about the number fip14:19
ihrachysack, I saw the thread14:19
electrocucarachaihrachys: awesome14:20
*** alexpilotti has joined #openstack-neutron14:20
*** seanatci_ has joined #openstack-neutron14:21
*** amoralej is now known as amoralej|lunch14:21
ihrachysjlibosva: oh btw storing on im_self won't be ideal14:22
ihrachysjlibosva: at least not directly14:22
ihrachysotherwise multiple methods from the same class may clash14:22
ihrachyspy3 solves it all by allowing setattr on instance methods14:23
ihrachysbut py2 sucks14:23
*** seanatcisco has quit IRC14:23
*** bnemec is now known as beekneemech14:23
*** rossella_s has quit IRC14:24
*** chlong has joined #openstack-neutron14:25
*** zenirc369 has quit IRC14:26
*** david-lyle has joined #openstack-neutron14:26
*** suresh12 has joined #openstack-neutron14:27
*** ralonsoh_ has joined #openstack-neutron14:31
*** jhershbe has quit IRC14:32
tmorinhi ihrachys / I think I've found the right way to avoid the ugly "sort to facilitate writing the unit test" thing I had in https://review.openstack.org/#/c/437952/ -- if you want to have a look14:32
*** tonytan_brb has joined #openstack-neutron14:33
*** matrohon has quit IRC14:33
*** ralonsoh has quit IRC14:34
*** amotoki has quit IRC14:35
*** tonytan4ever has quit IRC14:35
openstackgerritJakub Libosvar proposed openstack/neutron master: Throttle SIGHUPs to keepalived  https://review.openstack.org/40709914:36
jlibosvaihrachys: ^^ tell me if the comments make sense and clears the idea behind __get__14:36
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: spec: rpc from odl to networking-odl  https://review.openstack.org/44940214:36
*** tommylikehu_ has quit IRC14:37
*** alex_xu has quit IRC14:37
*** alex_xu has joined #openstack-neutron14:38
*** XinBai has joined #openstack-neutron14:38
*** udesale has joined #openstack-neutron14:38
openstackgerritMerged openstack/networking-ovn master: Sync provnet lsp missing in OVN DB for provider network  https://review.openstack.org/44691214:40
*** marst has quit IRC14:41
*** murphy_zhao has joined #openstack-neutron14:42
ihrachysjlibosva: that's horrible we need to write it in the first place. -1 for some nits.14:43
ihrachysjlibosva: I can +1 after you fix those.14:44
*** m-greene_ has quit IRC14:44
*** amotoki has joined #openstack-neutron14:45
*** m-greene has quit IRC14:46
*** amotoki has quit IRC14:47
*** fragatina has quit IRC14:47
*** m-greene has joined #openstack-neutron14:48
*** m-greene_ has joined #openstack-neutron14:48
*** marst has joined #openstack-neutron14:49
*** erhudy has quit IRC14:50
*** murphy_zhao has quit IRC14:50
*** matrohon has joined #openstack-neutron14:51
*** tbachman has quit IRC14:52
*** fragatina has joined #openstack-neutron14:53
*** karthikpr has joined #openstack-neutron14:55
*** alexpilo_ has joined #openstack-neutron14:57
*** suresh12 has quit IRC14:57
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Remove minimum_bandwidth_rule from rules supported by Linuxbridge agent  https://review.openstack.org/44966614:58
slaweq_ajo: ^^ can You check it?14:58
*** alexpilotti has quit IRC14:58
openstackgerritSindhu Devale proposed openstack/neutron master: OVO for L3HARouter  https://review.openstack.org/36144314:59
*** lucasxu has joined #openstack-neutron14:59
*** alexpilo_ has quit IRC15:02
*** prateek has joined #openstack-neutron15:02
jlibosvaihrachys: but do make those comments sense? is it clear?15:02
prateekping ihrachys, please let me know when you have some time to discuss regarding https://review.openstack.org/#/c/448973/15:03
ihrachysjlibosva: yes, it explains the intent15:04
ihrachysif that's what you ask15:04
*** murphy_zhao has joined #openstack-neutron15:05
*** alexpilotti has joined #openstack-neutron15:06
jlibosvamaybe apuimedo's opinion can be helpful, he's the one who showed my descriptors couple of years back ;)15:07
*** jianghuaw has joined #openstack-neutron15:07
ihrachysapuimedo: ah you!.. you know it's illegal to spread drugs don't you15:08
ihrachysjlibosva: I am all for descriptors15:08
*** karthikpr has quit IRC15:08
ihrachysjlibosva: unless you can avoid them ;)15:08
ihrachysprateek: hey15:08
jlibosvaihrachys: you can with a price by injecting state to function object15:08
ihrachysright. in the end it's an attribute of the callable.15:09
ihrachysI don't think we will reach to agreement on that one15:09
ihrachysit's a matter of taste15:10
*** blogan has joined #openstack-neutron15:10
ihrachysit's like abstract art - some say it's beautiful, some hate it15:11
apuimedo:-)15:11
apuimedojlibosva: what should I look at?15:11
ihrachysapuimedo: but really, https://review.openstack.org/#/c/407099/42..44/neutron/agent/common/utils.py15:11
ihrachysis it right direction? wrong direction? does it help readability?15:12
*** jahsis has joined #openstack-neutron15:12
jlibosvaapuimedo: so the goal is to postpone calls if they are called soon after each other. We need to decorate an instance method while the decorator itself has a state15:12
jlibosvaihrachys: maybe the state could be stored in a closure15:12
ihrachysjlibosva: no, that's my nonlocal comment there15:13
ihrachyspy2 doesn't know how to get to the state of enclosing block15:13
jlibosvaapuimedo: but class decorator requires special handling for passing self to decorated method which is not clear as it's done via descriptor15:13
ihrachyswith py3, the code would be very simple15:13
*** sridharg has quit IRC15:14
prateek ihrachys, i tried explaining in https://review.openstack.org/#/c/448973/ why i am doing this way15:15
apuimedojlibosva: why is defer_call not _defer_call?15:15
*** david-lyle has quit IRC15:15
jlibosvaapuimedo: that's a detail, those methods are not exposed anyway15:15
prateekalso can you please elaborate on the 404 part ?15:15
ihrachysprateek: sure15:15
prateekihrachys, ^15:15
*** tmorin has quit IRC15:16
jlibosvaapuimedo: also I'm not a fan of _ at the beginning :) the important question is whether it's better to implement decorator with function and inject state to the decorated function or use class15:16
jlibosvaapuimedo: class has the downside of descriptor need, which is advanced15:16
apuimedogotta think about it15:17
apuimedoin general I abhor classes15:17
ihrachys+15:17
ihrachys:)15:17
ihrachysprateek: you said "All create_() are class methods and as such the addCleanup wont be registered on them."15:18
ihrachysprateek: does it stop us from converting them to instancemethods?15:18
prateekihrachys, there are a lot of methods using them15:18
ihrachysprateek: are they in our tree or in tempest?15:19
prateekso removing the @classmethod decorator would require a lot of unnecessary changes15:19
prateekour tree15:19
jlibosvaihrachys: so I guess will go with function then15:19
prateekihrachys, let me find the link for you15:19
ihrachysjlibosva: no wait, it seems apuimedo may have more to say after reading15:19
ihrachysprateek: unnecessary as in 'more work' or 'more work that doesn't benefit anyone'?15:20
*** donghao has quit IRC15:20
ihrachysI find it would be a nice improvement for the framework if we would clean up resources after they are no longer used. I understand you may chase a short term fix though, just asking if you think it's just plain bad idea.15:21
prateekihrachys, IMO it would be work that wont benefit anyone else since this is the only class that is hittting this issue15:21
ihrachysbut there is nothing inherently wrong with the test class, it abides to tempest practices when creating resources, and is screwed because of low quotas/tempest not bumping quotas for tests/resources not cleaned up in time.15:22
ihrachysprateek: in general case, I can imagine the case where you execute the tests in parallel on 100 cores and hit the same issue.15:22
ihrachysthe key here it seems is parallelism level15:23
*** suresh12 has joined #openstack-neutron15:23
ihrachysanyhoo, I start feeling that would be a major piece of work and won't fit your goal15:23
prateekihrachys, https://github.com/openstack/neutron/blob/master/neutron/tests/tempest/api/base.py#L21615:24
*** gkadam has joined #openstack-neutron15:24
prateekihrachys, yup , would be too much for the issue15:24
prateekalso as you have pasted links that in master it has been fixed by bumping up quotas15:24
prateekit wont be worth the effort15:24
prateekihrachys, ^15:24
ihrachysprateek: as for the other qos policy related comment, just see the logs, -api job fails15:24
*** rossella_s has joined #openstack-neutron15:25
ihrachysprateek: worthiness here would be defined as test correctness, not as gate failure in parallelism constraints we set to ourselves. surely would be Low prio.15:25
*** mickeys has joined #openstack-neutron15:26
*** udesale has quit IRC15:27
prateekihrachys, your last statement was again a bit tough on my small mind nD15:27
prateek*xD15:27
*** jahsis has quit IRC15:27
ihrachysprateek: nevermind, I don't think it affects your patch15:27
prateekihrachys, ok, the test does not fail in my environ15:28
prateekdo you have any idea why its specifically failing here ?15:28
prateekand how can i fix this ?15:29
prateeki see you have written the reason for failure15:29
*** jaosorior has quit IRC15:30
*** mickeys has quit IRC15:30
*** slaweq_ has quit IRC15:32
prateekihrachys, do you know the reason such bug is being hit only on qos_policy and not other resources ?15:32
ihrachyslemme see the failure and code once again15:34
prateekihrachys, the reason my environ didnt detect the failure was it is skipping when i run it on my devstack :\15:34
prateekihrachys, thanks!15:34
*** mlavalle has joined #openstack-neutron15:35
*** rossella_s has quit IRC15:35
ihrachysprateek: ok then you have path forward :)15:36
prateekihrachys, yup, would give it a go then, would catch you on monday for further discussion, thanks Ihar !15:38
dasmihrachys: any idea why this one is not picked for a gate? https://review.openstack.org/#/c/444384/ zuul is not showing it.15:40
dasmis it a problem with depends-on for stable?15:40
ihrachysno idea15:42
ihrachys+W'd15:42
ihrachysit's in gate15:42
*** prateek has quit IRC15:42
*** ventifus has joined #openstack-neutron15:44
dasmihrachys: ack15:45
*** vhosakot has joined #openstack-neutron15:48
*** yamahata has joined #openstack-neutron15:48
*** bobmel has joined #openstack-neutron15:51
*** amarao has quit IRC15:52
*** fnaval has joined #openstack-neutron15:52
openstackgerritManjeet Singh Bhatia proposed openstack/neutron master: OVO for Quotas and Reservation  https://review.openstack.org/33862515:52
*** amoralej|lunch is now known as amoralej15:52
*** suresh12 has quit IRC15:52
*** sterdnotshaken has joined #openstack-neutron15:53
*** annegentle has joined #openstack-neutron15:55
*** amotoki has joined #openstack-neutron15:56
*** manheim has quit IRC15:56
*** bobmel has quit IRC15:56
*** bobmel_ has joined #openstack-neutron15:56
*** manheim has joined #openstack-neutron15:56
*** vishwanathj has quit IRC15:58
*** vishwanathj has joined #openstack-neutron15:58
*** mohankumar has joined #openstack-neutron15:59
*** manheim has quit IRC16:01
*** pcaruana has quit IRC16:03
*** udesale has joined #openstack-neutron16:04
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Add "default" behaviour to QoS policies  https://review.openstack.org/42830416:06
jlibosvaihrachys: so how about storing the state to dedicated object, something like: http://paste.openstack.org/show/604060/16:07
*** iyamahat has quit IRC16:09
*** udesale has quit IRC16:09
*** udesale has joined #openstack-neutron16:09
*** manheim has joined #openstack-neutron16:13
*** vhosakot_ has joined #openstack-neutron16:14
*** vhosakot has quit IRC16:14
*** kzaitsev_ws has quit IRC16:20
ihrachysjlibosva: hmmm, does wrapper see lock_with_timer ?16:21
openstackgerritJakub Libosvar proposed openstack/neutron master: Throttle SIGHUPs to keepalived  https://review.openstack.org/40709916:21
*** donghao has joined #openstack-neutron16:21
ihrachysI think in py2 it doesn't, hence nonlocal comment16:21
ihrachysif it does, then of course it's better16:21
jlibosvaihrachys: it should, it's stored in decorator's func_closure16:21
openstackgerritDavid Shaughnessy proposed openstack/neutron master: Clear QoS rules from ports without a qos policy.  https://review.openstack.org/44971016:21
*** iyamahat has joined #openstack-neutron16:22
jlibosvaihrachys: I sent yet another version to the gerrit. I also added you as co-author so you can't +2 :D16:22
ihrachysjlibosva: have you tried? I think I initially tried to store locks and last_time_called like that and it complained about missing symbol16:22
*** manheim has quit IRC16:22
ihrachysjlibosva: smart move!16:22
ihrachys:)16:22
jlibosvaihrachys: it passed all the tests16:22
ihrachysjlibosva: ok then maybe I overcomplicated it before because I thought it won't work16:22
*** manheim has joined #openstack-neutron16:22
jlibosvaihrachys: it requires a bit of magic in UT: https://review.openstack.org/#/c/407099/45/neutron/tests/unit/agent/common/test_utils.py@10816:23
ihrachysoh shish16:23
*** gouthamr has quit IRC16:23
ihrachysI gotta look at it close. maybe we don't need to do that?16:23
davidshaslaweq: ping16:24
ihrachysI mean, maybe no need to check that16:24
ihrachyssec, lemme check the whole thing16:24
ihrachysjlibosva: it passes fullstack/functional?16:24
jlibosvaihrachys: running functional test_keepalived and test_ha_router now16:25
jlibosvaihrachys: it passes fullstack16:25
ihrachysok16:25
ihrachysthen I probably hit some other issue and misinterpreted it16:25
ihrachyskudos to you poking the patch16:25
jlibosvaihrachys: but fullstack can trigger two failovers ... and it would pass16:25
jlibosvaihrachys: I need to run now, I'll connect later today16:26
ihrachysjlibosva: you could spin a thread that validates it constantly16:26
ihrachysjlibosva: ok I will maybe poke the idea with the thread if you agree it's a good thing16:26
*** manheim has quit IRC16:27
jlibosvaihrachys: won't some revision number bump if router state changes?16:27
charliejllewellyHey all is there a way to assign additional macs to a port?16:27
davidsharalonsoh_, ajo, slaweq : https://review.openstack.org/#/c/449710/16:27
*** donghao has quit IRC16:27
openstackgerritAkihiro Motoki proposed openstack/python-neutronclient master: Fix label of allowed-address-pair ip_address  https://review.openstack.org/44971316:28
ihrachysjlibosva: I don't see L3HARouterAgentPortBinding have revises_on_change16:28
ihrachysjlibosva: also bump would result in router notification, which may be not ideal (at least we would then need to ignore the change)\16:29
jlibosvaihrachys: ok, was just an idea16:29
jlibosvaihrachys: ttyl16:30
*** jlibosva has quit IRC16:30
ihrachyso/16:30
*** eezhova has quit IRC16:31
*** gouthamr has joined #openstack-neutron16:32
*** seanatci_ has quit IRC16:33
*** david-lyle has joined #openstack-neutron16:33
*** seanatcisco has joined #openstack-neutron16:34
*** tbachman has joined #openstack-neutron16:35
openstackgerritVictor Morales proposed openstack/neutron master: DistributedVirtualRouter mac address to OVO  https://review.openstack.org/30487316:35
openstackgerritOpenStack Proposal Bot proposed openstack/ovsdbapp master: Updated from global requirements  https://review.openstack.org/44820716:35
*** Jack_Iv has joined #openstack-neutron16:38
*** radhikam has joined #openstack-neutron16:38
openstackgerritPaul Carver proposed openstack/neutron-lib master: Add bgpvpn to list of known extensions  https://review.openstack.org/44972016:38
*** sterdnotshaken1 has joined #openstack-neutron16:39
*** sterdnotshaken has quit IRC16:39
slaweqdavidsha: done :)16:44
davidshaslaweq: Thanks!16:44
*** fragatina has quit IRC16:46
*** MasterOfBugs has joined #openstack-neutron16:46
*** jlinkes has quit IRC16:47
*** charliejllewelly has quit IRC16:48
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: sgrule: auto delete before sg deletion  https://review.openstack.org/44972716:48
openstackgerritMerged openstack/ovsdbapp master: Fix Python 3 compatibility in idlutils  https://review.openstack.org/44876316:49
*** aarefiev is now known as aarefiev_afk16:49
*** ygbo has quit IRC16:50
*** matrohon has quit IRC16:52
*** sterdnotshaken has joined #openstack-neutron16:54
*** sterdnotshaken2 has joined #openstack-neutron16:56
*** aojea_ has quit IRC16:56
*** sridharg has joined #openstack-neutron16:57
*** sterdnotshaken1 has quit IRC16:57
*** sterdnotshaken has quit IRC16:57
*** iyamahat has quit IRC16:57
*** mickeys has joined #openstack-neutron16:59
*** yamahata has quit IRC17:00
*** radhikam has quit IRC17:00
*** radhikam has joined #openstack-neutron17:01
*** gouthamr has quit IRC17:01
*** blake has joined #openstack-neutron17:02
openstackgerritAnkur proposed openstack/neutron master: Feature Classification Cleanup  https://review.openstack.org/44765617:04
*** gouthamr has joined #openstack-neutron17:04
*** udesale has quit IRC17:06
*** sterdnotshaken has joined #openstack-neutron17:11
*** sterdnotshaken1 has joined #openstack-neutron17:12
*** sterdnotshaken has quit IRC17:12
*** sterdnotshaken2 has quit IRC17:13
*** Jack_Iv has quit IRC17:14
*** catintheroof has joined #openstack-neutron17:14
*** catintheroof has quit IRC17:15
*** Jack_Iv has joined #openstack-neutron17:15
*** catintheroof has joined #openstack-neutron17:15
*** iyamahat has joined #openstack-neutron17:17
*** ralonsoh_ has quit IRC17:17
*** davidsha has quit IRC17:18
*** yamamoto has quit IRC17:19
*** sterdnotshaken has joined #openstack-neutron17:24
*** jerryz has joined #openstack-neutron17:25
*** sterdnotshaken1 has quit IRC17:26
*** eezhova has joined #openstack-neutron17:29
*** abhiraut has joined #openstack-neutron17:29
*** seanatcisco has quit IRC17:29
*** s3wong has joined #openstack-neutron17:31
*** murphy_zhao has quit IRC17:32
*** strgout has joined #openstack-neutron17:33
*** rossella_s has joined #openstack-neutron17:33
openstackgerritLouis Fourie proposed openstack/networking-sfc master: Replace insertion-mode with tap-enabled  https://review.openstack.org/44975417:34
*** lucasxu has quit IRC17:36
*** annegentle has quit IRC17:36
*** yamahata has joined #openstack-neutron17:37
*** marst has quit IRC17:37
*** psahoo_ has joined #openstack-neutron17:39
*** psahoo|away has quit IRC17:39
*** sterdnotshaken1 has joined #openstack-neutron17:43
*** sterdnotshaken has quit IRC17:43
*** rossella_s has quit IRC17:44
*** lucasagomes is now known as lucas-afk17:44
sam_skevinbenton: Hi, there?17:44
*** ltomasbo is now known as ltomasbo|away17:45
*** suresh12 has joined #openstack-neutron17:45
*** murphy_zhao has joined #openstack-neutron17:46
*** suresh12 has quit IRC17:46
*** suresh12 has joined #openstack-neutron17:46
*** manheim has joined #openstack-neutron17:49
*** manheim has quit IRC17:49
*** tesseract has quit IRC17:50
*** psahoo_ has quit IRC17:50
*** manheim has joined #openstack-neutron17:50
*** alexpilotti has quit IRC17:52
*** alexpilotti has joined #openstack-neutron17:52
*** radhikam has quit IRC17:52
*** jpena is now known as jpena|off17:55
*** yamamoto has joined #openstack-neutron17:56
*** ltomasbo|away is now known as ltomasbo17:56
*** radhikam has joined #openstack-neutron17:56
*** reedip has quit IRC17:56
*** baoli has quit IRC17:56
*** huanxie has quit IRC17:58
*** huanxie has joined #openstack-neutron18:00
*** suresh12 has quit IRC18:00
*** jianghuaw has quit IRC18:01
*** marst has joined #openstack-neutron18:01
*** annegentle has joined #openstack-neutron18:02
*** Jack_Iv_ has joined #openstack-neutron18:02
*** suresh12 has joined #openstack-neutron18:02
*** Rajeev has joined #openstack-neutron18:04
*** Jack_Iv has quit IRC18:04
*** sridharg has quit IRC18:06
*** ltomasbo is now known as ltomasbo|away18:07
*** baoli has joined #openstack-neutron18:07
*** bobmel_ has quit IRC18:09
*** thiagolib has joined #openstack-neutron18:09
*** alexpilotti has quit IRC18:10
*** manheim has quit IRC18:14
*** sterdnotshaken has joined #openstack-neutron18:15
*** manheim has joined #openstack-neutron18:15
*** gkadam has quit IRC18:15
*** sterdnotshaken1 has quit IRC18:17
*** yamamoto has quit IRC18:19
*** iranzo has quit IRC18:21
*** mvk has quit IRC18:23
*** manheim has quit IRC18:24
openstackgerritAkihiro Motoki proposed openstack/python-neutronclient master: Add optional_keys and required_keys to --subport argument  https://review.openstack.org/37384718:24
*** chlong has quit IRC18:24
*** donghao has joined #openstack-neutron18:26
*** eezhova has quit IRC18:27
*** MasterOfBugs has quit IRC18:28
*** tbachman has quit IRC18:28
*** ociuhandu has quit IRC18:29
*** abhiraut has quit IRC18:31
*** XinBai_ has joined #openstack-neutron18:32
*** mohankumar has quit IRC18:32
*** donghao has quit IRC18:32
*** abhiraut has joined #openstack-neutron18:32
*** XinBai has quit IRC18:33
openstackgerritBrian Haley proposed openstack/neutron master: Update metering agent to use stevedore alias for driver  https://review.openstack.org/41988118:34
*** XinBai_ has quit IRC18:35
*** XinBai has joined #openstack-neutron18:35
openstackgerritBrian Haley proposed openstack/neutron master: Update metering agent to use stevedore alias for driver  https://review.openstack.org/41988118:36
*** zenirc369 has joined #openstack-neutron18:37
*** oreillyd has quit IRC18:37
*** oreillyd_ has quit IRC18:37
*** tbachman has joined #openstack-neutron18:41
*** sterdnotshaken1 has joined #openstack-neutron18:45
*** MasterOfBugs has joined #openstack-neutron18:46
*** sterdnotshaken2 has joined #openstack-neutron18:47
*** sterdnotshaken has quit IRC18:48
*** sterdnotshaken1 has quit IRC18:49
*** lucasxu has joined #openstack-neutron18:50
openstackgerritIsaku Yamahata proposed openstack/neutron master: update result before calling network_create_precommit  https://review.openstack.org/44977918:51
*** Jack_Iv_ has quit IRC18:55
openstackgerritMerged openstack/ovsdbapp master: Updated from global requirements  https://review.openstack.org/44820718:56
*** Jack_Iv has joined #openstack-neutron18:56
*** Jack_Iv has quit IRC18:57
kevinbentonsam_s: hey18:57
*** Jack_Iv has joined #openstack-neutron18:57
*** crushil has joined #openstack-neutron18:59
*** mvk has joined #openstack-neutron19:00
*** XinBai has quit IRC19:01
*** Jack_Iv has quit IRC19:02
*** sterdnotshaken has joined #openstack-neutron19:03
*** sterdnotshaken has quit IRC19:05
*** sterdnotshaken2 has quit IRC19:05
*** sterdnotshaken has joined #openstack-neutron19:05
*** kobis1 has joined #openstack-neutron19:10
*** kobis has quit IRC19:11
*** crushil has quit IRC19:11
*** mickeys has quit IRC19:16
*** sterdnotshaken has quit IRC19:16
*** yamamoto has joined #openstack-neutron19:17
*** erhudy has joined #openstack-neutron19:20
*** bobmel has joined #openstack-neutron19:24
*** yamamoto has quit IRC19:25
*** Jack_Iv has joined #openstack-neutron19:27
*** jgruber has joined #openstack-neutron19:28
openstackgerritAkihiro Motoki proposed openstack/python-neutronclient master: Allow to specify tenant_id in subnetpool-create  https://review.openstack.org/44978619:29
*** crushil has joined #openstack-neutron19:30
*** donghao has joined #openstack-neutron19:31
*** blake has quit IRC19:33
*** sterdnotshaken has joined #openstack-neutron19:33
kevinbentonzzzeek: can you check my response. i rambled a bit and what to make sure it makes sense19:36
*** donghao has quit IRC19:36
*** cleong has quit IRC19:37
*** crushil has quit IRC19:38
sam_skevinbenton: sorry for the late response.19:41
sam_skevinbenton: could you take a look the fix https://review.openstack.org/#/c/448822?19:42
kevinbentonsam_s: sure, in a meeting ATM19:44
kevinbentonsam_s: will look later this afternoone19:44
kevinbentonsam_s: afternoon*19:44
sam_skevinbenton:  thank you:)19:45
*** gcb has quit IRC19:49
*** trevormc has joined #openstack-neutron19:49
*** strgout has quit IRC19:52
*** slaweq has quit IRC19:52
*** annegentle has quit IRC19:53
*** slaweq has joined #openstack-neutron19:53
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: journal: partially implement sg/sgrule dependency  https://review.openstack.org/44980019:54
*** slaweq has quit IRC19:58
openstackgerritKevin Benton proposed openstack/neutron master: Delete segments on BEFORE_DELETE instead of PRECOMMIT  https://review.openstack.org/42944119:59
openstackgerritTrevor McCasland proposed openstack/neutron master: WIP: Add QinQ network type driver  https://review.openstack.org/44604719:59
*** eezhova has joined #openstack-neutron20:00
*** mlavalle has quit IRC20:03
*** Jack_Iv has quit IRC20:05
*** slaweq has joined #openstack-neutron20:06
*** annegentle has joined #openstack-neutron20:15
openstackgerritAnindita Das proposed openstack/neutron master: [OVO] Integration of RouterL3AgentBinding  https://review.openstack.org/37707420:16
*** jlibosva has joined #openstack-neutron20:16
*** mickeys has joined #openstack-neutron20:17
*** aojea has joined #openstack-neutron20:18
*** annegent_ has joined #openstack-neutron20:19
*** raj_ has quit IRC20:21
*** thiagolib has quit IRC20:21
openstackgerritJakub Libosvar proposed openstack/neutron master: Throttle SIGHUPs to keepalived  https://review.openstack.org/40709920:22
*** annegentle has quit IRC20:22
*** mickeys has quit IRC20:22
jlibosvaihrachys: ^^ I fixed the py3 - do you think it's worth to implement the threaded guard of router state in the patch or maybe we could try to get that in as is and improve the test as a followup20:22
*** gouthamr has quit IRC20:23
ihrachysmeh, ok. was actually working on the test, should have locked the patch20:23
*** yamamoto has joined #openstack-neutron20:23
ihrachysI want the test in shape, I am reworking it at this moment20:23
ihrachyswill make sure your change is included20:24
ihrachysjlibosva: not sure if it's just me, but it puzzles me why the test that just creates like 50 ports and FIPs take so long to pass20:24
ihrachysand neutron-server is multi workers all 100%, so apparently it does work20:25
*** amoralej is now known as amoralej|off20:25
*** jobewan has joined #openstack-neutron20:25
jlibosvaihrachys: maybe the nodes are not performant, remember that ubuntu boot took sometimes around 10 minutes20:25
ihrachysbut it's not like we boot anything20:26
ihrachysit's a bunch of ports with fake binding20:26
ihrachysanyhoo20:26
*** armax has joined #openstack-neutron20:29
sam_skevinbenton:  thank you for the comment. but I have different opinion.20:30
sam_skevinbenton:  As my understanding, segment mean a L2 resource and network mean a L3 resource, network depends on segment, when creating a network(L3) resource, it is better make related L2 resource ready first.20:30
sam_sSo my understanding order would be:20:30
sam_s    create network request20:30
sam_s        create segment(s)20:30
sam_s        create network20:31
sam_s    delete network request20:31
sam_s        delete network20:31
sam_s        delete segment(s)20:31
sam_swhat your thoughts?20:31
kevinbentonyou can't have a segment without a network20:31
kevinbentonsegments live in networks20:31
kevinbentonso that order won't work20:31
kevinbentonunless we allow segments to exist with null network_id references20:32
*** yamamoto has quit IRC20:33
*** sterdnotshaken1 has joined #openstack-neutron20:34
*** donghao has joined #openstack-neutron20:35
*** sterdnotshaken2 has joined #openstack-neutron20:36
*** sterdnotshaken1 has quit IRC20:36
*** amarao has joined #openstack-neutron20:37
*** sterdnotshaken has quit IRC20:37
*** bobmel has quit IRC20:40
*** donghao has quit IRC20:40
*** amuller has quit IRC20:40
*** trevormc has quit IRC20:45
*** mlavalle has joined #openstack-neutron20:46
*** gcheresh_ has quit IRC20:46
*** mlavalle has quit IRC20:47
*** sterdnotshaken has joined #openstack-neutron20:50
*** reedip has joined #openstack-neutron20:51
*** matrohon has joined #openstack-neutron20:52
*** sterdnotshaken2 has quit IRC20:52
*** mlavalle has joined #openstack-neutron20:55
*** baoli has quit IRC20:55
*** jlibosva has quit IRC20:55
sam_skevinbenton: as my know, some hardware devices need to know its L2 resource when creating L3 resource, if segment was created after related network, it would be an issue in that case.20:56
openstackgerritSindhu Devale proposed openstack/neutron master: Refactoring agent linux&ovsdb config  https://review.openstack.org/34786720:57
*** baoli has joined #openstack-neutron20:58
sam_skevinbenton: why it is not allowed to create segment with null network_id as the order create network request -> create segment(s)  -> create network  -> associate network with segment?20:58
sam_skevinbenton: is it because network_id was tagged nullable=False in the class NetworkSegment?21:00
*** dave-mccowan has quit IRC21:01
*** jamesdenton has quit IRC21:01
*** links has joined #openstack-neutron21:05
*** aojea has quit IRC21:06
*** suresh12 has quit IRC21:09
*** suresh12 has joined #openstack-neutron21:10
*** ranjithd has quit IRC21:12
*** amarao has quit IRC21:15
openstackgerritIhar Hrachyshka proposed openstack/neutron master: WIP rework keepalived master state forfeit test  https://review.openstack.org/44982021:15
openstackgerritLouis Fourie proposed openstack/networking-sfc master: Replace insertion-mode with tap-enabled  https://review.openstack.org/44975421:16
*** pramodrj07 has joined #openstack-neutron21:16
*** baoli has quit IRC21:17
*** MasterOfBugs has quit IRC21:20
*** ndahiwade has quit IRC21:21
*** saisriki has quit IRC21:21
*** vgadiraj has quit IRC21:21
*** sindhude has quit IRC21:21
*** suresh12 has quit IRC21:21
*** ihrachys has quit IRC21:21
*** aojea has joined #openstack-neutron21:22
*** ihrachys has joined #openstack-neutron21:22
*** crushil has joined #openstack-neutron21:23
*** Rajeev has quit IRC21:28
*** suresh12 has joined #openstack-neutron21:28
*** ihrachys has quit IRC21:28
*** fragatina has joined #openstack-neutron21:29
*** sterdnotshaken1 has joined #openstack-neutron21:38
*** donghao has joined #openstack-neutron21:38
*** sterdnotshaken has quit IRC21:40
*** donghao has quit IRC21:43
openstackgerritIsaku Yamahata proposed openstack/networking-odl master: secgroup: convert icmpv6 variant name into icmpv6  https://review.openstack.org/44982521:44
*** boden has quit IRC21:45
*** tmorin has joined #openstack-neutron21:53
*** suresh12 has quit IRC21:54
*** tmorin has quit IRC21:54
*** tmorin has joined #openstack-neutron21:55
*** annegent_ has quit IRC21:56
*** sterdnotshaken has joined #openstack-neutron21:58
*** sterdnotshaken1 has quit IRC21:59
*** sterdnotshaken2 has joined #openstack-neutron21:59
openstackgerritBrian Haley proposed openstack/neutron master: Verify metering label exists before applying rule  https://review.openstack.org/44983022:00
*** sterdnotshaken has quit IRC22:02
*** sterdnotshaken has joined #openstack-neutron22:02
*** pramodrj07 has quit IRC22:03
*** pramodrj07 has joined #openstack-neutron22:03
*** knikolla has left #openstack-neutron22:03
*** suresh12 has joined #openstack-neutron22:03
*** tmorin has quit IRC22:04
*** sterdnotshaken1 has joined #openstack-neutron22:04
*** janzian has quit IRC22:05
*** sterdnotshaken2 has quit IRC22:05
openstackgerritSlawek Kaplonski proposed openstack/neutron master: [WIP] Add QoS bandwidth limit for instance ingress traffic  https://review.openstack.org/44983122:06
*** sterdnotshaken1 has quit IRC22:06
*** sterdnotshaken has quit IRC22:07
*** abhiraut has quit IRC22:17
*** matrohon has quit IRC22:19
*** abhiraut has joined #openstack-neutron22:20
*** baoli has joined #openstack-neutron22:21
*** mickeys has joined #openstack-neutron22:22
*** sterdnotshaken has joined #openstack-neutron22:23
kevinbentonsam_s: right. and we could change it, but it doesn't make much logical sense to have a segment existing outside of a network22:24
*** lucasxu has quit IRC22:27
*** eezhova has quit IRC22:28
reediphaleyb : ping?22:28
reediphalyeb : wanted to know your opinion about https://review.openstack.org/#/c/427670/ , whenever you are available...22:29
*** jobewan has quit IRC22:33
*** yamahata has quit IRC22:36
*** iyamahat has quit IRC22:36
*** liangy has quit IRC22:36
*** abhiraut has quit IRC22:36
*** abhiraut has joined #openstack-neutron22:38
*** fnaval has quit IRC22:39
*** abhiraut has quit IRC22:40
*** sterdnotshaken1 has joined #openstack-neutron22:40
*** sterdnotshaken has quit IRC22:43
*** abhiraut has joined #openstack-neutron22:43
*** apuimedo has quit IRC22:45
openstackgerritKevin Benton proposed openstack/neutron master: Load all relationships on object pending_to_persistent  https://review.openstack.org/43445422:45
kevinbentonzzzeek: ^^22:46
kevinbentonzzzeek: ignore commit title, just realized that needs to be fixed :)22:46
openstackgerritKevin Benton proposed openstack/neutron master: Load all eager relationships on 'after_flush' event  https://review.openstack.org/43445422:47
*** fnaval has joined #openstack-neutron22:49
*** bobmel has joined #openstack-neutron22:52
sam_skevinbenton: how would mechnism driver be notified about the segments creation if it is independent from network creation(precommit or postcommit)22:53
kevinbentonsam_s: subscribe to segment create and delete events22:56
kevinbentonsam_s: even if we adjusted the semantics like you wanted, you would be missing all of the other segments in a multisegment network (e.g. the routed networks use case)22:57
slaweqkevinbenton: armax haleyb: hello, can You take a look on https://review.openstack.org/#/c/449666/ ? thx in advance22:58
*** bobmel has quit IRC22:58
*** abhiraut has quit IRC22:59
*** aojea has quit IRC22:59
*** erhudy has quit IRC23:00
kevinbentonslaweq: done23:01
slaweqkevinbenton: thx a lot :)23:02
sam_skevinbenton: but there is no related segment function in current mechnism driver. even if in the multisegment network case.23:03
kevinbentonsam_s: yes, subscribe to the segment events23:03
*** abhiraut has joined #openstack-neutron23:05
sam_skevinbenton: do you mean in _handle_segment_change to notifiy mechnism driver or we need add subscription directly in mechnisum driver functions?23:08
kevinbentonsam_s: subscribe directly in mechanism driver23:08
*** ndahiwade has joined #openstack-neutron23:10
*** ihrachys has joined #openstack-neutron23:14
*** sterdnotshaken has joined #openstack-neutron23:15
*** saisriki has joined #openstack-neutron23:17
*** sterdnotshaken1 has quit IRC23:17
*** sindhude has joined #openstack-neutron23:17
*** ihrachys has quit IRC23:18
*** vhosakot_ has quit IRC23:19
*** baoli_ has joined #openstack-neutron23:20
*** donghao has joined #openstack-neutron23:21
*** ijw has joined #openstack-neutron23:23
*** markvoelker has quit IRC23:24
*** baoli has quit IRC23:24
ijwI have a question on binding levels - if a port is being bound three times in quick succession (it's a router port and I'm not quite clear why it's happening, but let's skip that a moment) wouldn't I expect calls 2 and 3 to report the original binding?23:24
ijwIn update_port_precommit I'm not seeing them23:24
*** links has quit IRC23:24
*** donghao has quit IRC23:25
openstackgerritAnkur proposed openstack/neutron master: Feature Classification Cleanup  https://review.openstack.org/44765623:35
*** jamesden_ has joined #openstack-neutron23:35
*** iyamahat has joined #openstack-neutron23:43
*** sdague has quit IRC23:43
*** lucasxu has joined #openstack-neutron23:45
*** aojea has joined #openstack-neutron23:47
*** wu_ng has joined #openstack-neutron23:49
*** baoli_ has quit IRC23:51
*** aojea has quit IRC23:51
*** jamesden_ has quit IRC23:53

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