Monday, 2014-06-16

*** VijayB_ has quit IRC00:03
*** nplanel has quit IRC00:03
*** yamahata has quit IRC00:08
*** xianghui has joined #openstack-neutron00:19
*** igordcard has quit IRC00:21
*** banix has quit IRC00:24
*** ianw has quit IRC00:26
*** ianw has joined #openstack-neutron00:26
*** dvorkinista has joined #openstack-neutron00:29
*** dfarrell07 has quit IRC00:31
*** haleyb has quit IRC00:34
*** matsuhashi has joined #openstack-neutron00:35
*** matsuhashi has quit IRC00:35
*** matsuhashi has joined #openstack-neutron00:35
*** igordcard has joined #openstack-neutron00:47
*** banix has joined #openstack-neutron01:00
*** igordcard has quit IRC01:19
*** yanfengxi has joined #openstack-neutron01:21
openstackgerritItsuro Oda proposed a change to openstack/neutron: Do not schedule reserved DHCP port to agents  https://review.openstack.org/9827801:22
*** xianghui has quit IRC01:27
openstackgerrityangxurong proposed a change to openstack/neutron: Invalid tunnel network shouldn't be created  https://review.openstack.org/8998201:41
*** sweston has joined #openstack-neutron01:42
openstackgerrityangxurong proposed a change to openstack/neutron: Invalid tunnel network shouldn't be created  https://review.openstack.org/8998201:45
*** gildub has joined #openstack-neutron01:46
*** alexpilotti has joined #openstack-neutron01:48
*** yamamoto_ has joined #openstack-neutron01:50
*** tomoe_ has quit IRC01:56
*** sweston has quit IRC01:58
*** VijayB_ has joined #openstack-neutron02:00
*** zhiyan_ is now known as zhiyan02:00
*** VijayB_ has quit IRC02:04
*** ramishra has joined #openstack-neutron02:12
*** essessv has joined #openstack-neutron02:18
*** zhiyan is now known as zhiyan_02:18
*** gildub has quit IRC02:26
*** gildub has joined #openstack-neutron02:26
*** zhiyan_ is now known as zhiyan02:27
*** zhipeng has quit IRC02:28
*** zhipeng has joined #openstack-neutron02:29
*** fifieldt__ is now known as fifieldt02:30
*** VijayB_ has joined #openstack-neutron02:36
*** yamahata has joined #openstack-neutron02:36
*** alexpilotti has quit IRC02:46
*** ranger81 has joined #openstack-neutron02:48
*** dims has quit IRC02:57
*** alexpilotti has joined #openstack-neutron02:58
*** VijayB_ has quit IRC03:05
*** VijayB has joined #openstack-neutron03:05
*** nlahouti_ has joined #openstack-neutron03:12
*** seizadi has joined #openstack-neutron03:15
*** nlahouti has quit IRC03:15
*** alexpilotti has quit IRC03:16
*** jeffreycoho has quit IRC03:20
*** essessv has quit IRC03:21
*** crc32 has joined #openstack-neutron03:27
openstackgerritA change was merged to openstack/neutron: Fix invalid IPv6 address used in FakeV6 variables  https://review.openstack.org/9939803:28
openstackgerritFeng Ju proposed a change to openstack/python-neutronclient: Hash credentials in neutron client log  https://review.openstack.org/9386603:32
*** banix has quit IRC03:44
*** essessv has joined #openstack-neutron03:45
*** amotoki has joined #openstack-neutron03:57
*** xianghui has joined #openstack-neutron04:02
*** morganfainberg has quit IRC04:04
*** morganfainberg has joined #openstack-neutron04:04
*** morganfainberg has quit IRC04:05
*** morganfainberg has joined #openstack-neutron04:06
*** yfried_ has quit IRC04:07
*** tomoe_ has joined #openstack-neutron04:09
*** ranger81 has quit IRC04:15
*** dims has joined #openstack-neutron04:26
*** crc32 has quit IRC04:29
*** vivek-ebay has joined #openstack-neutron04:29
*** xianghui has quit IRC04:30
*** crc32 has joined #openstack-neutron04:30
*** dims has quit IRC04:30
*** xianghui has joined #openstack-neutron04:35
*** kashyapk has joined #openstack-neutron04:39
*** Longgeek has joined #openstack-neutron04:39
*** Longgeek has quit IRC04:40
*** Longgeek has joined #openstack-neutron04:40
*** SridharG has joined #openstack-neutron04:42
*** coolsvap|afk is now known as coolsvap04:43
*** crc32 has quit IRC04:43
*** dims has joined #openstack-neutron04:46
*** dims has quit IRC04:51
*** tomoe_ has quit IRC04:51
*** kashyapk has quit IRC04:52
*** kashyapk has joined #openstack-neutron04:52
*** otherwiseguy has joined #openstack-neutron04:57
openstackgerritA change was merged to openstack/neutron: Control update, delete for cisco-network-profile  https://review.openstack.org/9608004:57
*** kashyapk_ has joined #openstack-neutron04:57
openstackgerritA change was merged to openstack/neutron: Big Switch: Catch exceptions in watchdog thread  https://review.openstack.org/10007504:58
*** kashyapk has quit IRC04:58
openstackgerritA change was merged to openstack/neutron: Increase default metadata_workers, backlog to 4096  https://review.openstack.org/9537204:58
*** coolsvap is now known as coolsvap|afk04:58
*** vivek-ebay has quit IRC04:59
*** shausy has joined #openstack-neutron05:01
*** yfried_ has joined #openstack-neutron05:02
*** ranger81 has joined #openstack-neutron05:03
*** otherwiseguy has quit IRC05:08
*** coolsvap|afk is now known as coolsvap05:09
*** otherwiseguy has joined #openstack-neutron05:12
*** terryw has joined #openstack-neutron05:17
*** VijayB has quit IRC05:19
*** otherwiseguy has quit IRC05:20
*** kashyapk_ has quit IRC05:21
*** tomoe_ has joined #openstack-neutron05:22
*** kashyapk has joined #openstack-neutron05:22
*** sweston has joined #openstack-neutron05:28
*** kashyapk has quit IRC05:30
*** kashyapk has joined #openstack-neutron05:30
*** zhhuabj has quit IRC05:41
*** dims has joined #openstack-neutron05:47
*** dims has quit IRC05:51
*** seizadi has quit IRC05:52
*** seizadi has joined #openstack-neutron05:52
*** zhhuabj has joined #openstack-neutron05:54
*** zhiyan is now known as zhiyan_05:55
*** zhiyan_ is now known as zhiyan05:56
*** seizadi has quit IRC05:57
*** matsuhashi has quit IRC06:00
*** matsuhashi has joined #openstack-neutron06:01
*** ranger81 has quit IRC06:07
*** rotbeard has joined #openstack-neutron06:07
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/9927506:09
*** nlahouti has joined #openstack-neutron06:11
openstackgerritFeng Ju proposed a change to openstack/python-neutronclient: Hash credentials in neutron client log  https://review.openstack.org/9386606:14
*** nlahouti_ has quit IRC06:14
*** essessv has quit IRC06:21
*** amotoki has quit IRC06:26
*** matsuhas_ has joined #openstack-neutron06:38
*** rotbeard has quit IRC06:38
*** kickinz1 has quit IRC06:38
*** kashyapk has quit IRC06:38
*** coolsvap is now known as coolsvap|afk06:38
*** coolsvap|afk is now known as coolsvap06:38
*** alexpilotti has joined #openstack-neutron06:38
*** alexpilotti has quit IRC06:38
*** kickinz1` has joined #openstack-neutron06:38
*** matsuhashi has quit IRC06:38
*** kashyapk_ has joined #openstack-neutron06:39
*** dims has joined #openstack-neutron06:48
*** [1]evgenyf has joined #openstack-neutron06:49
*** dims has quit IRC06:52
*** arosen has joined #openstack-neutron06:53
openstackgerritIsaku Yamahata proposed a change to openstack/neutron-specs: physical-network-topology extension spec  https://review.openstack.org/9127506:55
*** arosen1 has joined #openstack-neutron06:55
*** ihrachyshka has joined #openstack-neutron06:57
*** arosen has quit IRC07:00
*** armax has joined #openstack-neutron07:01
*** matsuhas_ has quit IRC07:01
*** armax has left #openstack-neutron07:01
*** devvesa has joined #openstack-neutron07:02
*** kashyapk_ has quit IRC07:03
*** yfried_ is now known as yfried07:03
*** matsuhashi has joined #openstack-neutron07:04
*** chandan_kumar has quit IRC07:05
*** coolsvap is now known as coolsvap|afk07:06
*** amotoki has joined #openstack-neutron07:06
ihrachyshkahey. has anyone noticed issues with open daylight negative voting on valid patches?07:09
*** rcarrillocruz has quit IRC07:13
*** rcarrillocruz has joined #openstack-neutron07:13
openstackgerritJyotsna proposed a change to openstack/python-neutronclient: neutron port-list -f csv outputs poorly formatted JSON strings  https://review.openstack.org/9992907:15
*** coolsvap|afk is now known as coolsvap07:17
*** nlahouti has quit IRC07:17
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Consistently use jsonutils instead of specific implementation  https://review.openstack.org/9976007:19
*** ijw has quit IRC07:19
*** gildub has quit IRC07:19
*** luqas has joined #openstack-neutron07:28
*** kashyapk has joined #openstack-neutron07:35
*** jaypipes has joined #openstack-neutron07:36
*** jaypipes has quit IRC07:36
*** leseb has joined #openstack-neutron07:37
*** jlibosva has joined #openstack-neutron07:38
*** kashyapk has quit IRC07:38
*** kashyapk has joined #openstack-neutron07:39
*** scheuran has joined #openstack-neutron07:39
*** lukego has joined #openstack-neutron07:41
*** karimb has joined #openstack-neutron07:41
*** rotbeard has joined #openstack-neutron07:42
*** amarao has joined #openstack-neutron07:42
*** SridharG has quit IRC07:44
*** kashyapk has quit IRC07:45
*** kashyapk has joined #openstack-neutron07:45
*** coolsvap is now known as coolsvap|afk07:46
*** kashyapk has quit IRC07:47
*** kashyapk has joined #openstack-neutron07:47
*** leseb has quit IRC07:47
*** afazekas_ has joined #openstack-neutron07:48
*** SridharG has joined #openstack-neutron07:48
*** nlahouti has joined #openstack-neutron07:48
*** dims has joined #openstack-neutron07:49
*** salv-orlando has joined #openstack-neutron07:49
*** Infitialis has joined #openstack-neutron07:52
*** a_le has quit IRC07:53
*** rcarrill` has joined #openstack-neutron07:53
*** dims has quit IRC07:53
*** a_le has joined #openstack-neutron07:53
openstackgerritKoteswara Rao Kelam proposed a change to openstack/neutron: Add -s option for neutron metering rules  https://review.openstack.org/9552607:54
*** armax has joined #openstack-neutron07:54
*** coolsvap|afk is now known as coolsvap07:54
*** rcarrillocruz has quit IRC07:55
*** jistr has joined #openstack-neutron07:57
*** nlahouti has quit IRC07:57
*** leseb has joined #openstack-neutron07:59
*** ijw has joined #openstack-neutron08:00
*** scheuran has quit IRC08:03
*** jpich has joined #openstack-neutron08:04
*** safchain has joined #openstack-neutron08:05
*** xianghui has quit IRC08:05
*** xianghui has joined #openstack-neutron08:05
*** ygbo has joined #openstack-neutron08:05
*** ijw has quit IRC08:06
*** jlibosva has quit IRC08:09
*** kashyapk_ has joined #openstack-neutron08:11
*** kashyapk has quit IRC08:11
*** jlibosva has joined #openstack-neutron08:11
*** jgallard has joined #openstack-neutron08:13
*** xianghui has quit IRC08:14
*** scheuran has joined #openstack-neutron08:15
*** ijw has joined #openstack-neutron08:15
*** alex_klimov has joined #openstack-neutron08:18
*** ijw has quit IRC08:21
*** karmatronic has joined #openstack-neutron08:23
*** jroovers has joined #openstack-neutron08:24
*** karimb has quit IRC08:25
*** xianghui has joined #openstack-neutron08:28
*** karmatronic has quit IRC08:29
openstackgerritstanzgy proposed a change to openstack/neutron: Ensure .egg-info metadata directory existed  https://review.openstack.org/10018608:31
ihrachyshkasalv-orlando: hey. I saw your comment in one of oslo.messaging reviews. I'd like to make sure we're on same side. I didn't consider your comments as bad or of no value or inappropriate, so there is no single reason for me to be 'angry'. I hope I didn't tell anything that would show my implied (and non-existent) angry. :) thanks for comments.08:32
*** jlibosva1 has joined #openstack-neutron08:32
*** kong has joined #openstack-neutron08:32
salv-orlandoihrachyshka: I was merely joking08:34
ihrachyshkabetter safe than sorry :)08:35
salv-orlandoI never get upset for work related stuff. Nothing that can happen at work is worth getting you upset08:35
ihrachyshkaI'll elaborate in comments on my take on the issue. :)08:35
salv-orlandounless you’re fired of course ;)08:35
*** jlibosva has quit IRC08:35
ihrachyshkasalv-orlando: fyi I've updated review with my comments08:41
salv-orlandoihrachyshka: ok, cool.08:41
*** isviridov|away is now known as isviridov08:42
*** zhhuabj has quit IRC08:45
*** zhhuabj has joined #openstack-neutron08:46
*** coolsvap is now known as coolsvap|afk08:47
*** dims has joined #openstack-neutron08:49
*** dims has quit IRC08:54
*** coolsvap|afk is now known as coolsvap08:54
*** zhiyan is now known as zhiyan_08:56
*** jp_at_hp has joined #openstack-neutron08:58
*** karimb has joined #openstack-neutron09:02
*** konglingxian has joined #openstack-neutron09:05
*** kong has quit IRC09:08
*** karimb has quit IRC09:14
*** ihrachyshka has quit IRC09:15
*** ijw has joined #openstack-neutron09:15
openstackgerritTom Fifield proposed a change to openstack/neutron: Fix opt helpstring for dhcp_lease_duration  https://review.openstack.org/10019609:17
*** ijw has quit IRC09:20
openstackgerritElena Ezhova proposed a change to openstack/neutron: Check port value when creating firewall rule with icmp protocol  https://review.openstack.org/9937209:20
*** kashyapk_ has quit IRC09:25
*** karimb has joined #openstack-neutron09:25
*** kashyapk has joined #openstack-neutron09:26
*** raies has joined #openstack-neutron09:26
raieshi someone09:26
*** eezhova has joined #openstack-neutron09:28
*** garyk has joined #openstack-neutron09:30
*** kashyapk has quit IRC09:30
*** kashyapk has joined #openstack-neutron09:30
*** kong has joined #openstack-neutron09:34
*** kashyap has quit IRC09:34
*** konglingxian has quit IRC09:37
*** scheuran has quit IRC09:38
openstackgerritCedric Brandily proposed a change to openstack/neutron: Use patch ports to interconnect integration/physical bridges  https://review.openstack.org/7668509:39
*** ihrachyshka has joined #openstack-neutron09:40
*** garyk has quit IRC09:46
*** rcarrillocruz has joined #openstack-neutron09:46
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Updated from global requirements  https://review.openstack.org/9976209:48
*** rcarrill` has quit IRC09:48
*** dims has joined #openstack-neutron09:50
*** dguitarbite has joined #openstack-neutron09:51
*** scheuran has joined #openstack-neutron09:51
openstackgerritOpenStack Proposal Bot proposed a change to openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/9626709:52
*** jroovers has quit IRC09:54
*** dims has quit IRC09:54
*** jroovers has joined #openstack-neutron09:55
*** coolsvap is now known as coolsvap|afk09:59
*** pcm_ has joined #openstack-neutron10:01
*** pcm_ has quit IRC10:01
*** pcm_ has joined #openstack-neutron10:02
*** leseb has quit IRC10:04
*** ondergetekende has quit IRC10:05
*** leseb has joined #openstack-neutron10:05
*** openstackgerrit has quit IRC10:06
*** openstackgerrit has joined #openstack-neutron10:07
*** leseb has quit IRC10:09
*** karimb has quit IRC10:10
*** coolsvap|afk is now known as coolsvap10:13
*** kashyapk has quit IRC10:14
*** ijw has joined #openstack-neutron10:15
*** ijw has quit IRC10:21
*** matsuhashi has quit IRC10:24
*** jgallard has quit IRC10:24
*** matsuhashi has joined #openstack-neutron10:24
*** ondergetekende has joined #openstack-neutron10:25
*** kashyapk has joined #openstack-neutron10:26
*** tomoe_ has quit IRC10:27
*** raies has quit IRC10:29
openstackgerritElena Ezhova proposed a change to openstack/neutron: Check port value when creating firewall rule with icmp protocol  https://review.openstack.org/9937210:32
*** matsuhashi has quit IRC10:33
*** matsuhashi has joined #openstack-neutron10:34
*** openstackgerrit has quit IRC10:35
*** openstackgerrit has joined #openstack-neutron10:36
*** leseb has joined #openstack-neutron10:38
*** matsuhashi has quit IRC10:41
*** Longgeek has quit IRC10:41
openstackgerritYuuichi Fujioka proposed a change to openstack/neutron-specs: change to allow setting ip address of floating ip  https://review.openstack.org/10021410:41
*** matsuhashi has joined #openstack-neutron10:41
*** Longgeek has joined #openstack-neutron10:42
*** leseb has quit IRC10:44
*** jroovers|afk has joined #openstack-neutron10:44
openstackgerritenikanorov proposed a change to openstack/neutron-specs: neutron-flavor-framework  https://review.openstack.org/9007010:45
*** jroovers has quit IRC10:46
*** Longgeek has quit IRC10:46
*** maishsk has joined #openstack-neutron10:47
openstackgerritYuuichi Fujioka proposed a change to openstack/neutron: change to allow setting ip address of floating ip  https://review.openstack.org/7028610:47
* maishsk says hello to you all10:47
*** moha_hunt has joined #openstack-neutron10:48
maishskGood afternoon all10:48
*** yamamoto_ has quit IRC10:48
*** igordcard has joined #openstack-neutron10:48
maishskI have a question about LBaaS - anyone around to answer?10:48
maishskWhat is the purpose of monitors in LB options?10:49
*** Zemeio has quit IRC10:50
*** shausy has quit IRC10:50
*** dims_ has joined #openstack-neutron10:51
*** shausy has joined #openstack-neutron10:51
*** harmw has quit IRC10:52
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Don't use root_helper when it's not needed  https://review.openstack.org/10021610:54
*** dims_ has quit IRC10:55
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Don't use root_helper when it's not needed  https://review.openstack.org/10021610:56
maishskAre they pnly to tell the LB if the member is responding - and if not remove it from the pool10:56
maishskonly*10:56
maishskor also to spawn new members as well - if a member dies?10:56
*** zzelle has joined #openstack-neutron10:57
*** Longgeek has joined #openstack-neutron10:57
*** xianghui has quit IRC10:59
*** kashyapk is now known as kashyap_11:08
*** kashyap_ is now known as kashyapk11:08
*** yamahata has quit IRC11:10
*** karimb has joined #openstack-neutron11:14
*** ijw has joined #openstack-neutron11:15
*** dims_ has joined #openstack-neutron11:18
*** ijw has quit IRC11:21
*** eezhova has quit IRC11:22
*** luqas has quit IRC11:22
*** leseb has joined #openstack-neutron11:23
*** tnurlygayanov has joined #openstack-neutron11:23
*** kashyapk is now known as kopparam11:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Configure agents using neutron.common.config.init (formerly .parse)  https://review.openstack.org/9750511:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Removed local modification in incubator code  https://review.openstack.org/9968611:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Don't instantiate RPC clients on import  https://review.openstack.org/9750611:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Introduced rpc_compat.create_connection()  https://review.openstack.org/9729011:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Port to oslo.messaging  https://review.openstack.org/9729111:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Copy-paste RPC Service class for backwards compatibility  https://review.openstack.org/9728911:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Make sure we call BaseTestCase.setUp() first  https://review.openstack.org/9762311:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Updated neutron.conf to reflect new RPC options  https://review.openstack.org/10001411:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Don't explicitly call .stop() on mock.patch objects  https://review.openstack.org/9880711:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Added notifier entry points for backward compatibility  https://review.openstack.org/10001311:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Moved rpc_compat.py code back into rpc.py  https://review.openstack.org/9881311:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Removed 'rpc' and 'notifier' incubator modules  https://review.openstack.org/9881211:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Introduce RpcCallback class  https://review.openstack.org/9679211:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Renamed consume_in_thread -> consume_in_threads  https://review.openstack.org/9880911:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Renamed start_rpc_listener -> start_rpc_listeners  https://review.openstack.org/9880811:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Removed create_rpc_dispatcher methods  https://review.openstack.org/9881111:25
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Renamed argument for create_consumer[s]  https://review.openstack.org/9881011:25
*** kopparam is now known as kashyapk11:26
*** kashyapk is now known as kopparam11:27
*** matsuhashi has quit IRC11:29
*** matsuhashi has joined #openstack-neutron11:29
enikanorovihrachyshka: hey11:32
ihrachyshkaenikanorov: hey11:32
enikanorovihrachyshka: might it make sense to reduce the number of patches by compacting them into several?11:33
ihrachyshkaenikanorov: what's your concern? spam in irc? :)11:33
*** matsuhashi has quit IRC11:33
ihrachyshkaenikanorov: I was explicitly asked to split one patch into separate logical parts11:33
openstackgerritJakub Libosvar proposed a change to openstack/neutron: FWaaS: Don't contact l3 agent if tenant has no routers  https://review.openstack.org/9995611:34
ihrachyshkaand that's what I did11:34
enikanorovright, but may be 20 patches is a little too much11:34
ihrachyshkaenikanorov: I think the best way to reduce spam is to merge them. I hope this will be done this week, and I'll stop spamming you that much11:35
ihrachyshkaenikanorov: well, I actually think that having nicely separated patches is a good thing, even if some of them are tiny. easier to track failures, bisect etc.11:36
enikanorovi'm more concerned about review queue. looks like updating one patch triggers every depending patches to be rebased11:37
*** harmw has joined #openstack-neutron11:37
ihrachyshkaenikanorov: yes, though if a patch is not touched, all votes remain11:38
ihrachyshkaenikanorov: only if you're concerned about push on gate...11:38
enikanorovme - not so much :)11:38
*** coolsvap is now known as coolsvap|afk11:41
*** avishayb has joined #openstack-neutron11:42
*** garyk has joined #openstack-neutron11:42
ihrachyshkaenikanorov: btw most of patches are > 50 lines, some of them are hundreds of lines, I think this is a reasonable threshold11:43
maishsk enikanorov: Hi - got a minute for some more LBaaS questions?11:44
enikanorovmaishsk: yes11:44
maishskwhat is the purpose of monitors?11:44
maishskTell me if I understand correctly...11:47
maishskThe monitor will check a member either through TCP/HTTP/S/PING to see it if is alive11:48
maishskif the checks fail according to the delay/timeout/retries - then the node will be marked as unavailable?11:49
maishskam I correct in my understanding?11:49
*** rcarrill` has joined #openstack-neutron11:50
*** jroovers|afk is now known as jroovers11:51
*** rcarrillocruz has quit IRC11:52
maishskenikanorov: am I correct in my understanding?11:53
enikanorovmaishsk: correct11:54
enikanorovit's marked as unavailable on both backend and in the database11:54
enikanorovand is excluded from palancing11:55
enikanorov*balancing11:55
maishskenikanorov: and does this tie somehow into autoscaling groups? i.e. if a node is marked as unavailable - can a new be spawned as a result?11:55
*** aburaschi has joined #openstack-neutron11:56
*** karimb has quit IRC11:57
*** jkary1 has joined #openstack-neutron12:01
enikanorovmaishsk: neutron lbaas doesn't do that for you12:03
enikanorovit's orchestration responsibility12:03
*** kong has quit IRC12:04
maishskSo how would you handle that today - from an orchestration perspective?12:04
*** dguitarbite has quit IRC12:05
openstackgerritZang MingJie proposed a change to openstack/neutron: db migration failed when plugin name alias used in config file  https://review.openstack.org/8122112:07
*** aburaschi has quit IRC12:08
*** dguitarbite has joined #openstack-neutron12:10
*** jkary1 has quit IRC12:11
*** yamamoto has joined #openstack-neutron12:14
*** ijw has joined #openstack-neutron12:15
openstackgerritCedric Brandily proposed a change to openstack/neutron: Add vlan type driver unittests  https://review.openstack.org/10023312:15
*** garyk has quit IRC12:16
*** morganfainberg has quit IRC12:18
*** igordcard has quit IRC12:19
*** morganfainberg has joined #openstack-neutron12:19
*** hanzhang has quit IRC12:19
*** ijw has quit IRC12:20
*** erecio has joined #openstack-neutron12:21
*** jgallard has joined #openstack-neutron12:23
*** b3nt_pin is now known as beagles_brb12:23
*** yamahata has joined #openstack-neutron12:23
zzelleyamamoto: hi12:34
yamamotohi12:34
*** feleouet has joined #openstack-neutron12:35
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS: Allow vendor extend/override valiadation  https://review.openstack.org/9694612:36
zzelleyamamoto: if i understood well, you are using OFagent ?12:37
yamamotoyes12:37
zzelleyamamoto: do it make sense to implement https://review.openstack.org/76685 for OFagent ?12:39
*** luqas has joined #openstack-neutron12:39
zzelleyamamoto: after reading https://review.openstack.org/#/c/98702/2/specs/juno/ofagent-merge-bridges.rst, i am not sure if i should or not12:39
*** shausy has quit IRC12:41
yamamotoin general we want to stay away from patch ports.  we haven’t decided how to handle physical bridges, though12:41
yamamotoie. please leave ofagent for the change in question, at least for now.  thanks for asking.12:43
zzelleyamamoto: fine, less work :))12:43
zzellewhy OFagent prefer veth over patch ports ?12:44
*** zhiyan_ is now known as zhiyan12:44
*** kopparam has quit IRC12:44
*** shausy has joined #openstack-neutron12:45
yamamotobecause patch ports is OVS feature and less likely supported by other soft switches12:47
*** sweston has quit IRC12:52
openstackgerritZang MingJie proposed a change to openstack/neutron-specs: Option to disable dns on some subnets  https://review.openstack.org/10024212:52
*** julim has joined #openstack-neutron12:55
*** jgallard has quit IRC12:56
*** jgallard has joined #openstack-neutron12:57
*** rotbeard has quit IRC12:57
*** aburaschi has joined #openstack-neutron12:59
*** jlibosva1 is now known as jlibosva12:59
*** aburaschi has left #openstack-neutron13:00
*** aburaschi has joined #openstack-neutron13:01
*** banix has joined #openstack-neutron13:01
*** jgallard has quit IRC13:04
*** jgallard has joined #openstack-neutron13:04
*** kbringard has joined #openstack-neutron13:09
*** dims_ has quit IRC13:10
*** dims_ has joined #openstack-neutron13:10
*** chuckC has joined #openstack-neutron13:10
*** dims_ has quit IRC13:11
*** dims has joined #openstack-neutron13:12
*** ijw has joined #openstack-neutron13:15
*** jmsoares has joined #openstack-neutron13:16
*** coolsvap|afk is now known as coolsvap13:17
*** krtaylor has joined #openstack-neutron13:18
*** ijw has quit IRC13:21
cgoncalvesbanix, yamahata, jmsoares: ping! I'd like to check if you have some time now and interested in further discussing the traffic steering bp13:22
banixcgoncalves: hi13:23
jmsoareshi13:23
*** tomoe_ has joined #openstack-neutron13:23
cgoncalvesbanix: hi13:24
cgoncalvesbanix: as per reviewers comments, seems that the traffic steering (TS) bp is getting quite some disagreements on how we've proposed it13:24
*** trinaths has joined #openstack-neutron13:25
cgoncalvesmostly because we first started it as a service function chaining (SFC) proposal and then moved to TS as part of the advanced services umbrella bp13:25
cgoncalvesthere are two main points being discussed right now13:26
cgoncalves1. 'conflict' with the GBP classifier resource13:26
*** crc32 has joined #openstack-neutron13:26
cgoncalves2. 'conflict' with the service chain bp and the proposed list of list of ports13:26
cgoncalvesas per 1. I think both works need the classifier resource and they should have its own classifier resources13:27
banixcgoncalves: makes sense13:27
cgoncalvesthat or come up with a generic classifier resource enough to meet both parties13:28
yamahatacgoncalves: hi13:28
cgoncalvesin fact, both share quite the same attributes and could potentially be merged as one13:28
yamahatacgoncalves: do you mean yamamoto?13:28
cgoncalvesyamahata: hi!13:28
yamahatacgoncalves: I'm quite fine with reviewing the BP13:29
cgoncalvesyamahata: are you Isaky Yamahata? https://wiki.openstack.org/wiki/Meetings/NFV :-)13:29
cgoncalvesIsaku, sorry13:29
yamahatacgoncalves: I'm Isaku Yamahata13:29
*** jecarey has joined #openstack-neutron13:30
cgoncalvesyamahata: I just called you because you mentioned on that NFV table you've some kind of interested in traffic steering13:30
cgoncalvesyamahata: if that's not the case or you're out for this discussion, no problem. just meant to call the gang together :)13:31
*** otherwiseguy has joined #openstack-neutron13:32
cgoncalvescarrying on...13:32
cgoncalvesbanix: before going to #2, what's your opinion on #1?13:32
*** maishsk has quit IRC13:32
banixcgoncalves: i think the traffic steering should remain independent from GBP which means it needs its own classifier; so i agree with you in this regard13:33
openstackgerritChuck Carlino proposed a change to openstack/neutron: Add test cases for plugins/ml2/plugin.py  https://review.openstack.org/9912913:33
*** garyk has joined #openstack-neutron13:34
*** jgrimm_ has joined #openstack-neutron13:34
*** terryw has quit IRC13:35
jmsoaresbanix: agree that we should keep independent from GBP, but couldn't the "classifier" be common?13:35
cgoncalvesbanix: they would still be independent if we consider a classifier resource suitable for both works. although I can understand that it's now too late for that as the GBP bp has been approved and implemantion has already began13:35
jmsoaresbanix: I mean, what is the obstacle on this? agreeing on specifying the "Classifier" (attributes, etc)13:36
banixjmsoares: you mean we have a classifier resource independent of GBP and TS13:36
cgoncalvesbanix: exactly13:37
jmsoaresmaybe in a first stage that is not possible, but we could propose it as part of a roadmap13:37
banixjmsoares: cgoncalves I am not sure how we go about it if that is the goal; so let’s think about this for a minute13:37
*** shausy has quit IRC13:39
*** beagles_brb is now known as beagles13:39
*** garyk has left #openstack-neutron13:39
banixI have no objections to using the classifier as defined in GBP but then we have to keep in mind that the TS spec will depend on it which is fine at this level of dependence13:39
banixis that what you are suggesting?13:39
*** dfarrell07 has joined #openstack-neutron13:40
*** kopparam has joined #openstack-neutron13:40
*** banix has left #openstack-neutron13:42
*** WackoRobie has joined #openstack-neutron13:42
*** garyk has joined #openstack-neutron13:42
cgoncalvesbanix: yes, but dettaching the classifier resource as part of the GBP explicitly. so instead of 'neutron policy-classifier-*' we could have 'neutron classifier-*'13:42
*** banix has joined #openstack-neutron13:42
cgoncalvesbaffle: I believe in a near future we will start to see more work being proposed which needs a classifier resource. if we could agree on a common classifier, and then also allowing extensions, we would be cutting lots of duplicated code and simplifying workflows to the user13:44
*** jp_at_hp has quit IRC13:44
cgoncalves14:42:24 < cgoncalves> banix: yes, but dettaching the classifier resource as part of the GBP explicitly. so instead of 'neutron policy-classifier-*' we could have 'neutron classifier-*'13:44
*** gabriel-bezerra has quit IRC13:45
banixcgoncalves: i think this sounds reasonable13:45
*** dfarrell07 has quit IRC13:45
*** dfarrell07 has joined #openstack-neutron13:46
*** jp_at_hp has joined #openstack-neutron13:47
cgoncalvesbanix: ok, but maybe postponing such classifier resource sharing to K-cycle? if you agree, we can continue for now with nothing changing on both ends (TS and GBP)13:47
banixcgoncalves: ok13:48
cgoncalvesbanix: thanks13:48
jmsoarescgoncalves: looks like a plan. It could be good to approach GBP to harmonize this approach and make that clear in the BP specification13:48
cgoncalvesbanix, jmsoares: moving on to point 2 (list of list of ports)13:48
cgoncalvesjmsoares: agreed13:49
jmsoaresit might clarify some doubts regarding this topic13:49
*** kopparam has quit IRC13:49
jmsoarescgoncalves: action points on this? Bring this to discussion in GBP?13:50
jmsoaresand also in advanced services meeting13:50
*** haleyb has joined #openstack-neutron13:50
cgoncalvesbanix, jmsoares: lots of discussing about this as been going on. list of list of ports was proposed to introduce the idea of chaining but caused lots of confusion with the service chaining bp and GBP13:50
*** kopparam has joined #openstack-neutron13:50
cgoncalvesjmsoares: I would like to bring that up on both teams, yes!13:51
*** rolledback has joined #openstack-neutron13:51
*** rolledback has quit IRC13:52
cgoncalvesbanix, jmsoares: for now and until we have a better understanding on how the service chaining is going to be implemented, we could start by proposing something a little bit easier also to implement on the TS bp which is a list of ports rather than list of list of ports13:52
*** jroovers has quit IRC13:52
*** nplanel has joined #openstack-neutron13:52
*** rolledback has joined #openstack-neutron13:53
*** tomoe_ has quit IRC13:53
cgoncalvesin a list of ports, the first port would represent the source of traffic and the remaining ports the ports to steer traffic to matching classifiers13:53
*** radez_g0n3 is now known as radez13:53
banixcgoncalves: I agree; I think that is fine; we just need to update the spec to reflect that. I think I expressed that in my review. At least I tried to do so.13:54
jmsoaresbanix: yes, you expressed that.13:55
openstackgerritArvind Somya proposed a change to openstack/neutron-specs: ML2 Type drivers refactor to allow extensiblity  https://review.openstack.org/9181113:55
cgoncalvesbanix: yes, you did and we got the message successfully ;-)13:55
banixjmsoares: cgoncalves great :)13:55
cgoncalvesbanix: we were just trying to also figure if the rest of the reviewers would be fine with such change too13:55
banixcgoncalves: sure i understand13:56
cgoncalvesbanix: my fear is that they would see it as an extension to the tap-as-a-service bp by adding traffic classification on top13:56
*** trinaths has quit IRC13:56
*** igordcard has joined #openstack-neutron13:57
banixcgoncalves: could you post the link to the above bp if handy13:57
cgoncalvesbanix: https://review.openstack.org/#/c/96149/13:57
banixneed to be in a meeting in 3 minutes but will read it13:58
cgoncalvesbanix: np, thanks!13:58
*** otherwiseguy has quit IRC13:59
*** jistr has quit IRC13:59
*** rolledback has quit IRC14:00
banixcgoncalves: there is a need for classifiers in that spec too14:01
cgoncalvesbanix: is there? I can't find any reference to traffic classification on that bp14:01
*** jistr has joined #openstack-neutron14:01
*** jobewan has joined #openstack-neutron14:03
banixcgoncalves: thought the Tap Flow is something like that but probably not?14:03
*** markmcclain has joined #openstack-neutron14:04
*** dvorkinista has quit IRC14:04
*** alex_klimov has quit IRC14:05
cgoncalvesbanix: it could be but as I see it now I don't think so14:05
*** markmcclain1 has joined #openstack-neutron14:06
*** rolledback has joined #openstack-neutron14:06
*** rolledback has quit IRC14:07
*** markmcclain has quit IRC14:08
*** TrevorV has joined #openstack-neutron14:08
*** jroovers has joined #openstack-neutron14:09
*** TrevorV has quit IRC14:10
*** TrevorV has joined #openstack-neutron14:10
*** rolledback has joined #openstack-neutron14:11
*** jroovers has quit IRC14:13
*** otherwiseguy has joined #openstack-neutron14:14
*** nlahouti has joined #openstack-neutron14:15
*** ijw has joined #openstack-neutron14:15
*** afazekas_ has quit IRC14:16
*** nlahouti has quit IRC14:16
*** nlahouti has joined #openstack-neutron14:17
openstackgerritA change was merged to openstack/neutron: Return no active network if the agent has not been learnt yet  https://review.openstack.org/9329014:18
*** crc32 has quit IRC14:18
jmsoarescgoncalves, banix: Tap-as-a-Service as it is described now does not have into account classification. Its purpose seems to be to steer all traffic. There is in fact an interesting feature that is proposed there, to be able to redirect traffic between two ports in two different networks (as long as they are owned by the same tenant I think) - this is something that we have not considered so far.14:18
jmsoarescgoncalves, banix: if we would be able to support such a feature, could Tap-as-a-Service be seen as a service built on top of the traffic steering feature?14:20
*** otherwiseguy has quit IRC14:20
*** terryw has joined #openstack-neutron14:20
openstackgerritVivekanandan Narasimhan proposed a change to openstack/neutron: L2 Agent/ML2 Plugin changes for L3 DVR  https://review.openstack.org/8773014:20
*** ijw has quit IRC14:21
cgoncalvesjmsoares: if we could steer traffic to any ports owned by the same user, then highly possible yes14:26
*** terryw has quit IRC14:27
*** zhhuabj has quit IRC14:28
*** zhhuabj has joined #openstack-neutron14:28
jmsoarescgoncalves: then...questions is, should we actually perform such thing? :) maybe this is something to bring up in the advanced meeting14:28
*** rwsu has joined #openstack-neutron14:29
*** coolsvap is now known as coolsvap|afk14:30
openstackgerritZang MingJie proposed a change to openstack/neutron-specs: Option to disable dns on some subnets  https://review.openstack.org/10024214:32
*** ramishra has quit IRC14:33
cgoncalvesjmsoares: well, we could also bring that up wednesday. hope we'll have time to discuss all these topics14:35
*** dvorkinista has joined #openstack-neutron14:35
*** jroovers has joined #openstack-neutron14:35
*** chandan_kumar has joined #openstack-neutron14:36
*** nplanel has quit IRC14:36
*** yamamoto has quit IRC14:37
*** jroovers|afk has joined #openstack-neutron14:38
*** amotoki_ has joined #openstack-neutron14:39
*** aburaschi1 has joined #openstack-neutron14:39
*** dvorkinista has quit IRC14:40
*** aburaschi has quit IRC14:40
*** kashyap has joined #openstack-neutron14:40
*** jroovers has quit IRC14:40
*** arosen1 has quit IRC14:48
*** prad_ has joined #openstack-neutron14:50
*** nlahouti has quit IRC14:51
*** obondarev has joined #openstack-neutron14:51
*** devlaps has joined #openstack-neutron14:52
*** phil_h has joined #openstack-neutron14:54
*** rkukura has quit IRC14:56
*** garyk has quit IRC14:56
*** trad511 has joined #openstack-neutron14:57
*** sbfox has joined #openstack-neutron14:57
*** jistr has quit IRC14:58
*** moha_hunt has quit IRC14:59
*** jistr has joined #openstack-neutron14:59
*** dvorkinista has joined #openstack-neutron14:59
*** leseb has quit IRC14:59
*** leseb_ has joined #openstack-neutron14:59
*** ramishra has joined #openstack-neutron15:00
*** networkstatic has joined #openstack-neutron15:02
*** ihrachyshka has quit IRC15:03
*** changbl has joined #openstack-neutron15:06
*** a_le has quit IRC15:06
*** yfried has quit IRC15:06
*** rkukura has joined #openstack-neutron15:08
*** Guest8031 is now known as mgagne15:08
*** mgagne has joined #openstack-neutron15:08
*** sbalukoff has quit IRC15:09
sc68calmestery: Here is the work that xuhanp and shixiong were going to work on for J2 - https://review.openstack.org/#/c/99595/15:09
sc68calmarkmcclain1: ^15:09
*** enikanorov_ has joined #openstack-neutron15:09
markmcclain1sc68cal: thanks for the link15:09
sc68calyeah, so if you guys can comment about not wanting dnsmasq to do RA's that would help15:09
*** julim has quit IRC15:12
markmcclain1I've added comments15:13
markmcclain1sc68cal: let me know if I need to clarify more15:13
sc68calmarkmcclain1: looks good to me15:14
*** ijw has joined #openstack-neutron15:15
*** scheuran has quit IRC15:16
*** Infitialis has quit IRC15:17
*** emagana has joined #openstack-neutron15:17
zigomarkmcclain1: Nobody answer my question here: http://lists.openstack.org/pipermail/openstack-dev/2014-June/037701.html Do you know what I should do in my package?15:18
*** alagalah has joined #openstack-neutron15:18
*** emagana has quit IRC15:18
*** amaretskiy has quit IRC15:18
*** emagana has joined #openstack-neutron15:19
markmcclain1zigo: I'm playing catchup from holiday weekend, but I've flagged you email15:20
zigomarkmcclain1: Cheers! :)15:20
*** ijw has quit IRC15:20
*** jaypipes has joined #openstack-neutron15:20
*** mlavalle has joined #openstack-neutron15:25
*** amaretskiy has joined #openstack-neutron15:25
*** kickinz1` has quit IRC15:26
zzellemarkmcclain1: hi15:26
*** kickinz1 has joined #openstack-neutron15:27
*** enikanorov__ has joined #openstack-neutron15:29
*** alexpilotti has joined #openstack-neutron15:30
markmcclain1zzelle: hi15:31
*** enikanorov_ has quit IRC15:32
zzellemarkmcclain1: i see your review about SELECT FOR UPDATE trouble15:34
zzellemarkmcclain1: is it acceptable to replace it by:15:34
mesteryflaviof: Hi15:36
zzellemarkmcclain1: 1) selecting a candidate (one SQL request) 2) updating candidate information if unchanged (one SQL request) 3) retry if 2) update 0 row ?15:36
*** sbfox has quit IRC15:36
flaviofmestery: hey Kyle15:36
openstackgerritRacha Ben Ali proposed a change to openstack/neutron-specs: GW API - Piece 0.1 : service API for L2 bridging tenants/provider networks  https://review.openstack.org/9361315:36
*** sbfox has joined #openstack-neutron15:36
markmcclain1zzelle: that should work15:36
mesteryflaviof: I think your patch looks fine, though when do you run devstack without nova?15:36
markmcclain1will also want to include some kind of bounding of retries15:36
flaviofmestery: I'm trying to break up control into control and network nodes, to run l3 agent by itself15:37
mesteryflaviof: Ah, right! OK, good.15:37
mesteryflaviof: +1 from me, looks good!15:37
zzellemarkmcclain1: x retries with x defined as a configuration option with default=3 ?15:37
markmcclain1zzelle: yeah that should work15:37
flaviofmestery: just not sure if there were any further expectation on ml2 side that requires nova.15:38
*** bandarji has joined #openstack-neutron15:38
mesteryflaviof: That should be the only one I believe.15:38
flaviofmestery: I'm thinking not15:38
flaviofmestery: cool beans!15:38
mesteryflaviof: ;)15:38
*** afazekas_ has joined #openstack-neutron15:38
zzellemarkmcclain1: should i split my change into 2 ? 1) for SELECT FOR UPDATE ==> retries, 2) BP update to support partial specs15:39
*** coolsvap|afk is now known as coolsvap15:39
*** kopparam has quit IRC15:41
markmcclain1zzelle: I dont think so.. I think that you can include the retries for the new code covered by your spec15:41
markmcclain1the other code impacted for select for update will be impacted by of the other work going on15:42
zzellemarkmcclain1: fine15:43
zzellemarkmcclain1: thanks15:43
*** amarao has quit IRC15:43
openstackgerritIsaku Yamahata proposed a change to openstack/neutron-specs: API Extension for l2-gateway  https://review.openstack.org/10027815:43
*** emagana has quit IRC15:44
*** dvorkinista has quit IRC15:48
*** ramishra has quit IRC15:49
*** emagana has joined #openstack-neutron15:51
zzellemarkmcclain1: it seems strange to define a global option nb_retries in a neutron-specs implementation ?15:51
*** emagana has quit IRC15:52
*** zhiyan is now known as zhiyan_15:52
*** emagana has joined #openstack-neutron15:52
*** igordcard has quit IRC15:52
*** bearhands is now known as comstud15:53
rkukurazzelle: Have you published a spec or sent an email or anything describing your strategy for eliminating SELECT FOR UPDATE?15:55
zzellerkukura: not for the moment, i was about to ask mike bayer on the right strategy15:55
markmcclain1zzelle: I think initially just define it in a constant for the file15:56
zzellerkukura: and share my stategy on the ML15:56
rkukuraAre you writing any general helper function to handle retries?15:56
*** sweston has joined #openstack-neutron15:57
*** jlibosva has quit IRC15:57
zzellerkukura: no, my current strategy is specific to network allocation in type drivers15:57
rkukurazzelle: If this is just local to specific files, I agree with markmcclain1’s advice15:58
zzellerkukura: i need it for vlan/gre/vxlan in partial specs15:58
*** jmsoares has left #openstack-neutron15:58
zzellerkukura: fine15:58
*** sbfox has quit IRC15:58
*** sbfox has joined #openstack-neutron15:58
zzellerkukura: is there an intention to backport nova db retries in neutron ?15:58
*** trad5111 has joined #openstack-neutron15:59
rkukurazzelle, markmcclain1: My bind-outside-transactions patch needs to be updated, and adds some loops, and I’d like to at least understand how these or additional retry loops might eliminate the need to do DB locking (and the extra layer of locking on top of that).15:59
pcm_Anyone: Are there issues with Jenkins right now? I have a patchset that is failing numerous tests, but they all are failing in setup with fatal errors 'Couldn't find remote ref'16:00
rkukurazzelle: I haven’t looked at how nova deals with this.16:00
*** Longgeek has quit IRC16:00
zzellepcm_: i had the same trouble ... recheck no bug or new pachset push solved it16:00
markmcclain1pcm_: link?16:01
pcm_http://logs.openstack.org/46/96946/4/check/check-tempest-dsvm-neutron-full/1643b8f/16:01
zzellerkukura: ok16:01
*** trad511 has quit IRC16:02
*** otherwiseguy has joined #openstack-neutron16:03
*** terryw has joined #openstack-neutron16:03
*** leseb_ has quit IRC16:04
*** dave_tucker is now known as dave_tucker_zzz16:04
*** emagana has quit IRC16:04
*** sbfox1 has joined #openstack-neutron16:04
pcm_zzelle: Yeah, wondering if I should recheck or rebase.16:05
*** [1]evgenyf has quit IRC16:05
*** afazekas_ has quit IRC16:06
*** marun has joined #openstack-neutron16:06
sc68calmarkmcclain1: do you think we can clear the workflow -1 from the python-neutronclient change for the ipv6 attributes? https://review.openstack.org/#/c/75871/16:07
*** otherwiseguy has quit IRC16:07
*** sbfox has quit IRC16:08
markmcclain1sc68cal: would be ok to wait until I write up a proposal for new names for the enums?16:09
sc68calI'd prefer we get that merged since i have devstack patches that depend on it16:09
sc68caland the devstack patches got merged already16:09
markmcclain1client libs are sticky and once we add the flags we have to provide compatibility for extended time16:09
*** catohornet1 has joined #openstack-neutron16:10
*** _cjones_ has joined #openstack-neutron16:12
*** spandhe has joined #openstack-neutron16:12
*** trad5111 has quit IRC16:12
*** trad511 has joined #openstack-neutron16:13
sc68calmarkmcclain1: ok - I don't want to be pushy but we're working on scenario tests and other stuff for tempest, I know those don't use the client, but the args they pass are those attributes16:14
sc68caland we'd have to update those when we change the enums or the attribute labels/names16:15
markmcclain1right the tempest doesn't use the same client16:15
*** ijw has joined #openstack-neutron16:15
markmcclain1and I'll make a note that we'll need to clean up tempest scenario tests too16:15
sc68calI believe the tempest scenarios are still in flight - https://review.openstack.org/#/c/100134/16:16
sc68calalthough it did just get a +2 - soooo that might land soon16:16
sc68caloh wait that's just the spec16:17
*** a_le has joined #openstack-neutron16:19
openstackgerritHenry Gessau proposed a change to openstack/neutron-specs: DB migration refactoring  https://review.openstack.org/9573816:19
*** ijw has quit IRC16:20
*** ramishra has joined #openstack-neutron16:23
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS: Allow vendor extend/override valiadation  https://review.openstack.org/9694616:23
*** ygbo has quit IRC16:26
*** emagana has joined #openstack-neutron16:26
*** emagana has quit IRC16:27
*** emagana has joined #openstack-neutron16:27
*** jgallard has quit IRC16:31
*** Ju_ has quit IRC16:31
*** markmcclain1 has quit IRC16:33
*** TrevorV has quit IRC16:36
*** trinaths has joined #openstack-neutron16:36
*** sweston has quit IRC16:37
*** trad511 has quit IRC16:37
*** jpich has quit IRC16:37
*** ramishra has quit IRC16:40
*** zzelle_ has joined #openstack-neutron16:40
*** trad511 has joined #openstack-neutron16:41
*** networkstatic has quit IRC16:41
sc68calmestery: does the patch for dnsmasq for the provider networking slaac need to wait for mark to do the relabel/rename?16:42
sc68calhttps://review.openstack.org/6457816:42
*** kopparam has joined #openstack-neutron16:43
*** beagles is now known as beagles_food16:43
mesterysc68cal: To me, I think that patch can merge independently.16:43
sc68calperfect :)16:44
*** catohornet1 has quit IRC16:44
*** TrevorV has joined #openstack-neutron16:44
*** jistr has quit IRC16:46
*** TrevorV has quit IRC16:46
*** TrevorV has joined #openstack-neutron16:47
*** jroovers|afk has quit IRC16:49
*** mlavalle has quit IRC16:50
*** kopparam has quit IRC16:50
sc68calI hate coming across as pushy, but I want to try and tie up loose ends in ipv6 so I can move to the qos api16:50
sc68calI feel bad basically being like, hey I know I need to update the spec, but I haven't yet. lame excuse16:50
*** sbfox1 has quit IRC16:52
*** arosen has joined #openstack-neutron16:53
*** sweston has joined #openstack-neutron16:53
*** sweston has quit IRC16:55
*** sweston has joined #openstack-neutron16:57
*** TrevorV has quit IRC16:58
*** TrevorV has joined #openstack-neutron16:59
*** TrevorV has quit IRC16:59
*** luqas has quit IRC17:04
*** ranger81 has joined #openstack-neutron17:05
*** sbfox has joined #openstack-neutron17:05
*** catohornet1 has joined #openstack-neutron17:05
*** dvorkinista has joined #openstack-neutron17:06
*** ijw has joined #openstack-neutron17:06
*** amotoki_ has quit IRC17:06
*** yfried has joined #openstack-neutron17:07
*** networkstatic has joined #openstack-neutron17:07
*** TrevorV has joined #openstack-neutron17:07
*** sbfox has quit IRC17:08
*** sbfox has joined #openstack-neutron17:08
openstackgerritA change was merged to openstack/neutron-specs: Spec for keystone V3 Support changes  https://review.openstack.org/9657517:09
*** trinaths has quit IRC17:10
openstackgerritVishal Agarwal proposed a change to openstack/neutron: Fix pool statistics for LBaaS Haproxy driver  https://review.openstack.org/10029917:10
*** TrevorV has quit IRC17:11
*** sbfox1 has joined #openstack-neutron17:11
*** sbfox has quit IRC17:11
*** arosen has quit IRC17:11
*** arosen has joined #openstack-neutron17:12
*** amaretskiy has quit IRC17:13
*** sbfox1 has quit IRC17:13
*** sbfox has joined #openstack-neutron17:13
*** sbfox1 has joined #openstack-neutron17:14
*** suresh12 has joined #openstack-neutron17:14
yfriedanyone knows if mlavell is around, or on vacation?17:14
*** trinaths has joined #openstack-neutron17:17
*** a_le has quit IRC17:17
*** rolledback has quit IRC17:17
*** TrevorV has joined #openstack-neutron17:17
*** sbfox has quit IRC17:18
*** garyk has joined #openstack-neutron17:22
*** markmcclain has joined #openstack-neutron17:22
*** dave_tucker_zzz is now known as dave_tucker17:24
*** julim has joined #openstack-neutron17:31
*** _cjones__ has joined #openstack-neutron17:31
*** leseb has joined #openstack-neutron17:33
*** garyk has quit IRC17:33
*** nati_ueno has joined #openstack-neutron17:33
*** sbfox1 has quit IRC17:35
*** _cjones_ has quit IRC17:35
*** networkstatic has quit IRC17:35
*** sbfox has joined #openstack-neutron17:35
*** julim has quit IRC17:35
*** alex_klimov has joined #openstack-neutron17:36
*** a_le has joined #openstack-neutron17:37
*** julim has joined #openstack-neutron17:39
*** manishg has joined #openstack-neutron17:40
*** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack"17:40
*** nlahouti has joined #openstack-neutron17:41
*** xmltok_ has joined #openstack-neutron17:41
*** leseb has quit IRC17:46
*** alex_klimov has quit IRC17:46
*** leseb has joined #openstack-neutron17:47
*** luqas has joined #openstack-neutron17:51
*** leseb has quit IRC17:51
*** dims has quit IRC17:53
*** leseb has joined #openstack-neutron17:53
*** emagana has quit IRC17:54
*** dims has joined #openstack-neutron17:54
mesteryyfried: mlavalle is around to my knowledge, just not on IRC at the moment.17:55
*** garyk has joined #openstack-neutron17:55
*** rolledback has joined #openstack-neutron17:56
*** ijw_ has joined #openstack-neutron17:58
*** garyk has quit IRC18:00
*** jp_at_hp has quit IRC18:00
*** SridharG has quit IRC18:01
yfriedmestery: tnx18:01
openstackgerritAndy Shi proposed a change to openstack/neutron-specs: Dell ml2 Mechanism Driver  https://review.openstack.org/9249518:01
*** ijw has quit IRC18:01
*** mlavalle has joined #openstack-neutron18:03
*** zzelle has quit IRC18:05
*** harlowja has joined #openstack-neutron18:05
*** jprovazn has joined #openstack-neutron18:06
*** dguitarbite has quit IRC18:07
*** thedodd has joined #openstack-neutron18:11
*** dguitarbite has joined #openstack-neutron18:13
*** jaypipes has quit IRC18:14
*** ramishra has joined #openstack-neutron18:15
*** networkstatic has joined #openstack-neutron18:16
*** harlowja has quit IRC18:19
*** harlowja has joined #openstack-neutron18:19
openstackgerritSumit Naiksatam proposed a change to openstack/neutron: Group Policy Plugin-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/9639318:19
*** ramishra has quit IRC18:20
*** beagles_food is now known as beagles18:21
*** a_le has quit IRC18:22
*** alex_klimov has joined #openstack-neutron18:26
*** emagana has joined #openstack-neutron18:27
*** Jay__ has joined #openstack-neutron18:28
*** nati_ueno has quit IRC18:29
*** banix has quit IRC18:30
*** nati_ueno has joined #openstack-neutron18:30
openstackgerritChuck Carlino proposed a change to openstack/neutron: Add test cases for plugins/ml2/plugin.py  https://review.openstack.org/9912918:32
*** a_le has joined #openstack-neutron18:37
*** kickinz1 has quit IRC18:37
*** kickinz1 has joined #openstack-neutron18:37
*** shivharis has joined #openstack-neutron18:37
*** arosen has quit IRC18:40
*** kopparam has joined #openstack-neutron18:40
sc68calmlavalle: ping18:42
mlavallesc68cal: pong18:45
sc68calI wanted to bring this to your attention instead of mucking with your section of the neutron meeting wiki page. https://review.openstack.org/10014318:46
*** nati_ueno has quit IRC18:46
sc68calSridharG and I are working on more advanced scenario tests18:46
*** kopparam has quit IRC18:47
*** a_le has quit IRC18:47
mlavallesc68cal: really cool, thanks….. as  far as the other scenarios, what are they about?18:47
sc68calmlavalle: The scenario tests reflect networking configurations that Comcast deploys in production, when it comes to v4 and v6 connectivity18:48
sc68calcurrently the scenario tests for networking do NAT and floating ips for connectivity18:48
sc68calwe're working on putting stuff into devstack to reflect the way we deploy neutron, so we can then also develop scenario tests that reflect the way we deploy.18:49
mlavallesc68cal: are those scenarios applicable the the community as a whole?18:49
sc68calBasically we do non-openstack hardware devices as the gateway18:49
*** a_le has joined #openstack-neutron18:49
sc68calmlavalle: yes, since it uses the provider networking API extension18:49
sc68calwhich I don't believe has any functional tests18:49
mlavallesc68cal: fantastic, I'll help reviewing this code and making sure we get core reviews from the tempest side18:50
mlavallesc68cal: is that helpful?18:51
sc68calyeah that sounds good, I'll keep you in the loop18:51
mlavalle:-)18:51
*** crc32 has joined #openstack-neutron18:51
*** nati_ueno has joined #openstack-neutron18:51
*** nerdonmountain is now known as geekinutah18:54
*** lykinsbd has joined #openstack-neutron18:54
*** Sukhdev has joined #openstack-neutron18:56
*** networkstatic has quit IRC18:56
*** leseb has quit IRC18:56
*** emagana has quit IRC18:57
*** emagana has joined #openstack-neutron18:57
*** doddstack has joined #openstack-neutron18:58
*** erecio has quit IRC18:59
*** erecio has joined #openstack-neutron18:59
*** thedodd has quit IRC19:00
openstackgerritSandhya Dasu proposed a change to openstack/neutron-specs: Blueprint for ML2 mechanism driver for Cisco UCS Manager  https://review.openstack.org/9523619:02
*** rm_work is now known as rm_work|away19:04
*** suresh12 has quit IRC19:05
*** suresh12 has joined #openstack-neutron19:05
*** suresh12 has quit IRC19:06
*** suresh12 has joined #openstack-neutron19:06
openstackgerritSumit Naiksatam proposed a change to openstack/neutron: Group Policy API-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/9590019:06
sbfoxHey neutroners, can someone have some time to talk private about haproxy LBaaS?19:07
*** dims has quit IRC19:07
*** trinaths has quit IRC19:09
*** jckasper has joined #openstack-neutron19:10
openstackgerritCedric Brandily proposed a change to openstack/neutron: Allow to delegate some vlan provider attributes choice to neutron  https://review.openstack.org/7190419:11
*** sweston has quit IRC19:11
*** mlavalle has quit IRC19:15
*** aburaschi1 has quit IRC19:16
*** dims has joined #openstack-neutron19:18
enikanorovsbfox: hi, why talk private?19:19
*** Jay__ has quit IRC19:22
*** Jay__ has joined #openstack-neutron19:22
*** luqas has quit IRC19:23
*** nati_ueno has quit IRC19:23
*** nati_ueno has joined #openstack-neutron19:24
*** armax has quit IRC19:25
*** armax has joined #openstack-neutron19:25
*** erecio has quit IRC19:25
*** erecio has joined #openstack-neutron19:26
openstackgerritDarragh O'Reilly proposed a change to openstack/neutron: linuxbridge-agent: process port updates in the main loop  https://review.openstack.org/6287519:27
*** mestery has quit IRC19:27
*** larsks has quit IRC19:27
*** larsks has joined #openstack-neutron19:27
sbfoxhey enikanorov, mostly because its not a development issue and I didnt wasnt to clutter the room with non-dev chatter19:31
larskssbfox: That's what #openstack is for :)19:32
sbfoxOn my way :)19:32
*** mlavalle has joined #openstack-neutron19:35
emagananati_ueno: ping!19:36
*** terryw has quit IRC19:38
*** sweston has joined #openstack-neutron19:39
*** catohornet1 has quit IRC19:41
*** nati_ueno has quit IRC19:44
*** kopparam has joined #openstack-neutron19:45
*** networkstatic has joined #openstack-neutron19:47
*** kopparam has quit IRC19:50
*** alagalah has quit IRC19:51
*** moha_hunt has joined #openstack-neutron19:51
*** ranger81_ has joined #openstack-neutron19:51
*** alagalah has joined #openstack-neutron19:54
*** ranger81 has quit IRC19:54
*** nlahouti has quit IRC19:55
openstackgerritA change was merged to openstack/neutron: Fix opt helpstring for dhcp_lease_duration  https://review.openstack.org/10019620:00
*** sweston has quit IRC20:00
*** rcarrillocruz has joined #openstack-neutron20:01
*** devlaps has quit IRC20:03
*** Sukhdev has quit IRC20:03
*** rcarrill` has quit IRC20:04
*** crc32 has quit IRC20:07
*** rkukura_ has joined #openstack-neutron20:09
*** rkukura has quit IRC20:11
*** rkukura_ is now known as rkukura20:11
*** arosen has joined #openstack-neutron20:12
*** nati_ueno has joined #openstack-neutron20:14
*** dims has quit IRC20:14
*** sweston has joined #openstack-neutron20:15
*** crc32 has joined #openstack-neutron20:15
*** dims has joined #openstack-neutron20:15
*** ramishra has joined #openstack-neutron20:17
*** Zemeio has joined #openstack-neutron20:17
*** rolledback has quit IRC20:19
*** nati_ueno_i has joined #openstack-neutron20:19
*** leenheer has joined #openstack-neutron20:19
openstackgerritA change was merged to openstack/neutron-specs: ofagent: port monitoring w/o ovsdb accesses  https://review.openstack.org/9174220:19
*** nati_ueno has quit IRC20:19
*** nati_ueno_i is now known as nati_ueno20:20
*** ramishra has quit IRC20:21
*** mlavalle has quit IRC20:22
*** catohornet1 has joined #openstack-neutron20:23
*** nati_ueno has quit IRC20:25
pcm_anyone: I'm still getting Jenkin's failures and having trouble identifying what is wrong from the logs. Can anyone advise?20:27
enikanorovpcm_: i'll try20:27
pcm_enikanorov: thanks! http://logs.openstack.org/46/96946/5/check/check-tempest-dsvm-neutron/ce1f29d/20:28
openstackgerritCedric Brandily proposed a change to openstack/neutron: Remove SELECT ... FOR UPDATE usages in ML2 type driver release_segment  https://review.openstack.org/10034520:28
*** manishg has quit IRC20:29
*** leseb has joined #openstack-neutron20:31
*** sweston has quit IRC20:32
pcm_enikanorov: On one of the Jenkins runs, I see these messages in setup log "fatal: Couldn't find remote ref refs/zuul/master/Z52d622fdd9914217a6e40ca6f41b751b"20:33
pcm_enikanorov: other logs show neutron didn't start, but I can see what failed.20:33
enikanorovpcm_: yep, neutron didn't start, i'm trying to find out why20:34
*** radez is now known as radez_g0n320:34
*** ranger81_ has quit IRC20:34
*** ranger81 has joined #openstack-neutron20:34
*** nati_ueno has joined #openstack-neutron20:34
pcm_enikanorov: I couldn't figure out why. Seems like it is loading the VPN plugin's service driver and that's all I see.20:34
*** arosen has quit IRC20:34
*** markmcclain has quit IRC20:35
pcm_enikanorov: In the dsvm-full test, it hase the above fatal remote ref messages in setup log.20:35
*** leseb has quit IRC20:35
*** sweston has joined #openstack-neutron20:35
*** radez_g0n3 is now known as radez20:36
enikanorovpcm_: i think generic driver loading mechanism doesn't support passing additional parameter that you've added20:36
*** rcarrill` has joined #openstack-neutron20:36
*** nati_ueno_i has joined #openstack-neutron20:37
*** julim has quit IRC20:38
*** rcarrillocruz has quit IRC20:38
*** nati_ueno has quit IRC20:39
pcm_enikanorov: Where do you see that added parameter? I'm looking through change set and I don't think I changed the loading mechanism.20:39
pcm_enikanorov: Unless I'm missing something.20:40
*** karimb has joined #openstack-neutron20:40
enikanorovpcm_: https://review.openstack.org/#/c/96946/5/neutron/services/vpn/service_drivers/__init__.py20:40
*** nati_ueno_i has quit IRC20:40
enikanorovline 3320:40
marunkevinbenton: ping20:41
marunkevinbenton: re: your comments here https://review.openstack.org/#/c/96050/7/neutron/tests/unit/db/grouppolicy/test_db_grouppolicy.py20:41
kevinbentonmarun: yeah?20:41
kevinbentonmarun: is that something we want to go away from?20:41
pcm_enikanorov: Oh. I was thinking "importing" of driver. So you mean init.20:41
marunkevinbenton: I thought so, but I'm wondering where your argument regarding ease of assertion fits into my thinking.20:42
marunkevinbenton: I wouldn't think that we'd have to do so much lifecycle testing that the added complexity of a context manager could be justified.20:42
marunkevinbenton: but maybe I'm missing something?20:42
*** nlahouti has joined #openstack-neutron20:42
*** ranger81 has quit IRC20:43
*** ranger81 has joined #openstack-neutron20:43
kevinbentonmarun: well i just preferred writing those when doing unit tests creating ports, networks, routers, etc20:44
*** mestery has joined #openstack-neutron20:44
kevinbentonmarun: but i suppose i can just explicitly call delete20:44
marunkevinbenton: I think deletion should be required only in specific circumstances, rather than pervasively.20:45
*** tomoe_ has joined #openstack-neutron20:45
marunkevinbenton: have you found that you're requiring deletion to be performed more commonly?20:45
*** alexpilotti has quit IRC20:45
pcm_enikanorov: not sure I understand what is wrong. I added another arg, of which each sub-class that implement the ABC provide that arg.20:46
pcm_enikanorov: by creating an instance of a validator object.20:46
marunkevinbenton: (this presupposes that I fix the tests to clean up after themselves rather than requiring explicit deletion, e.g. via a per-test transaction that gets rolled back)20:46
kevinbentonmarun: it was nice when verifying rollback logic during various cases20:47
*** nlahouti has quit IRC20:47
marunkevinbenton: do you have an example?20:47
enikanorovpcm_: you're right20:48
marunkevinbenton: My aversion to the way contextmanager is used right now is because it adds to the cost of already expensive functional testing.20:48
*** kbringard has quit IRC20:48
*** nati_ueno has joined #openstack-neutron20:49
marunkevinbenton: if a deletion were explicit, I think the cost would be obvious.20:49
openstackgerritSoheil Eizadi proposed a change to openstack/neutron-specs: Pluggable IPAM Subsystem in Neutron  https://review.openstack.org/9796720:51
*** leseb has joined #openstack-neutron20:52
*** radez is now known as radez_g0n320:52
kevinbentonmarun: no, actually it doesn’t look like a made any useful assertions outside of the context about the object it created20:52
enikanorovpcm_: it seems to me that neutron hangs during loading20:53
*** nlahouti has joined #openstack-neutron20:53
kevinbentonmarun: so then the only useful advantage was that it guarunteed objects were deletable20:53
kevinbentonmarun: which probably isn’t worth the cost then20:53
*** ihrachyshka has joined #openstack-neutron20:53
pcm_enikanorov: Yeah, the dsvm full seems to hang with VPN loading, but I can't figure out why.20:53
marunkevinbenton: it can always be explicitly done if required, but always deleting is unnecessary since the db state is reset for each test20:53
marunkevinbenton: I'm going to -1 and ask for removal of contextmanager from the patch.20:54
*** yamamoto has joined #openstack-neutron20:54
kevinbentonmarun: well those are different because removing directly from the db doesn’t exercise the delete code20:54
kevinbentonmarun: so people will just have to remember to write tests that actually exercise the delete path20:54
marunkevinbenton: so there should be tests that validate deletion20:55
*** lukego has quit IRC20:55
marunkevinbenton: right20:55
enikanorovpcm_: not clear to me either. I'll look tomorrow once more20:55
marunkevinbenton: good point, though.  I'll make sure my comment reflects the need for explicit testing of deletion.20:55
pcm_enikanorov: Thanks. I'll try to bring up at Neutron Open discussion.20:55
*** erecio has quit IRC20:56
*** nati_ueno has quit IRC20:56
kevinbentonmarun: the deletion tests will have to cover all of the scenarios with dependent objects that these tend to end up doing as well20:56
enikanorovpcm_: i'd not bring it to open discussion as it seems to be a problem of particular patch ;)20:56
kevinbentonmarun: e.g. deleting a network with a router attached, etc20:57
*** jprovazn has quit IRC20:57
openstackgerritCedric Brandily proposed a change to openstack/neutron: Remove SELECT ... FOR UPDATE usages in ML2 type driver release_segment  https://review.openstack.org/10034520:57
marunkevinbenton: *sigh*20:57
*** nati_ueno_i has joined #openstack-neutron20:57
*** banix has joined #openstack-neutron20:57
pcm_enikanorov: Just a request for help.20:57
kevinbentonmarun: it’s fine. it’s probably better to be explicit about these tests anyway20:57
pcm_enikanorov: Also, that patch is a WIP and I'd like to get feedback on whether the approach is OK, before doing more.20:57
kevinbentonmarun: not just have them be a hidden side effect of another test20:58
*** amotoki_ has joined #openstack-neutron20:58
marunkevinbenton: I think explicit will be better, but it's painful for me to contemplate how much of the existing testing is implicit.20:58
*** markmcclain has joined #openstack-neutron20:58
kevinbentonmarun: we’ll just have to have a context removal hackathon20:59
*** markmcclain1 has joined #openstack-neutron20:59
marunkevinbenton: hah20:59
*** markmcclain1 has quit IRC20:59
*** markmcclain1 has joined #openstack-neutron21:00
kevinbentonmarun: it’s too bad. i like the context style…21:00
marunkevinbenton: it's neat, but not really necessary for what we're doing.21:02
openstackgerritPraneet Bachheti proposed a change to openstack/neutron: Opencontrail plug-in implementation for core resources  https://review.openstack.org/9663021:02
marunkevinbenton: as per the retargetable poc, there are different requirements for testing interface vs testing via rest, and it's not helpful to bake in the assumption that cleanup is required.21:03
*** markmcclain has quit IRC21:03
marunkevinbenton: it is required for REST, but not for local testing where db state can be reset21:03
*** rcarrill` has quit IRC21:03
*** sbfox has quit IRC21:04
*** leseb has quit IRC21:05
*** Sukhdev has joined #openstack-neutron21:05
kevinbentonmarun: even though the context managers are calling the rest api :-)21:05
*** leseb has joined #openstack-neutron21:05
*** osnew_ has joined #openstack-neutron21:06
marunkevinbenton: ambigious term, sorry21:06
marunkevinbenton: deployed rest vs local rest?21:06
marunkevinbenton: confusing :/21:06
kevinbentonmarun: yeah21:08
*** sbfox has joined #openstack-neutron21:09
kevinbentonmarun: i’m going to propose a patch to move them from context managers into decorators21:09
ihrachyshkasalv-orlando: what's your timezone? I would like to sync up with you on locks issues tomorrow (need general advice)21:09
marunkevinbenton: ?21:09
kevinbentonmarun: would that help? :-)21:09
marunkevinbenton: I'm not sure what 'them' you're referring to21:09
*** rolledback has joined #openstack-neutron21:09
*** tomoe_ has quit IRC21:09
kevinbentonmarun: the test objects21:10
kevinbentonmarun: so you just decorate each test with all of the objects that you want created for the test…21:10
*** tomoe_ has joined #openstack-neutron21:10
*** leseb has quit IRC21:10
marunkevinbenton: oh :p21:10
marunkevinbenton: heh21:10
*** isviridov is now known as isviridov|away21:11
salv-orlandoihrachyska: I’m CET - but I might be offline tomorrow between 10 AM and 12 PM21:11
kevinbentonmarun: i wonder if a good start would be setting no_delete to True on most of the uses that don’t appear to be doing anything that requires delete21:11
salv-orlandoCET for this week - BST usually21:11
marunkevinbenton: sadly that won't work21:11
marunkevinbenton: there is no guarantee of the uniqueness of resources21:12
ihrachyshkasalv-orlando: great, thanks. same as mine :)21:12
openstackgerritArvind Somya proposed a change to openstack/neutron-specs: ML2 Type drivers refactor to allow extensiblity  https://review.openstack.org/9181121:12
kevinbentonmarun: oh, i thought the db was currently rebuilt for each test21:12
marunkevinbenton: and some tests depend on automatic deletion - when it's removed then they start failing due to duplicattion errors21:12
marunkevinbenton: it is, but some tests use the creation methods more than once21:13
marunkevinbenton: a good example of bad decisions snowballing21:13
kevinbentonmarun: oh yes, that’s what i mean. the majority of them will be fine though21:13
*** mlavalle has joined #openstack-neutron21:13
kevinbentonmarun: based on what i remember seeing21:13
marunkevinbenton: true, just that the cases that aren't are pretty annoying21:14
marunkevinbenton: I started work on that but have been sidetracked21:14
marunkevinbenton: I'd be very happy if you you want to take it over and remove contextmanager,21:14
marunkevinbenton: it's the first step in switching to using transactions instead of db resets on every test, and should decrease test execution time all by itself21:15
openstackgerritCedric Brandily proposed a change to openstack/neutron: Remove SELECT ... FOR UPDATE usages in ML2 type driver release_segment  https://review.openstack.org/10034521:16
kevinbentonmarun: i have a busy couple of weeks though so i’m not sure i will make it to that.21:16
marunkevinbenton: fair enough.  how about we reach out to the other if either of us ends up finding time so that we don't duplicate?21:17
kevinbentonmarun: sounds good. i might get some time middle of this week so i’ll ping you if i do21:17
*** ramishra has joined #openstack-neutron21:18
zzelle_amotoki_, hi21:18
marunkevinbenton: I'll be on pto from Wed till next Mon, so go ahead if you have the time.21:19
kevinbentonmarun: ok21:19
*** TrevorV has quit IRC21:20
*** ramishra has quit IRC21:22
*** WackoRob_ has joined #openstack-neutron21:22
*** leseb has joined #openstack-neutron21:22
amotoki_zzelle_: hi21:23
zzelle_amotoki_, i updated https://review.openstack.org/76685 on patch port interconnection in order to correct conflicts with last merges in master branch21:24
*** WackoRob_ has quit IRC21:26
*** WackoRobie has quit IRC21:26
*** leseb has quit IRC21:27
*** nati_ueno has joined #openstack-neutron21:28
zzelle_amotoki_, if you have time to look ... it is a bit a mess to update associated modified tests :s21:28
*** otherwiseguy has joined #openstack-neutron21:29
amotoki_zzelle_: will look it later today.21:29
zzelle_amotoki_, thanks fro your time21:30
zzelle_s/fro/for/21:30
*** nati_ueno_i has quit IRC21:30
openstackgerritA change was merged to openstack/neutron: Big Switch: Add missing data to topology sync  https://review.openstack.org/9885521:32
openstackgerritA change was merged to openstack/neutron: Introduce RpcCallback class  https://review.openstack.org/9679221:32
*** nati_ueno_i has joined #openstack-neutron21:33
*** nati_uen_ has joined #openstack-neutron21:34
*** aaronorosen has joined #openstack-neutron21:34
*** banix has quit IRC21:35
*** nati_ueno has quit IRC21:35
*** leseb has joined #openstack-neutron21:36
*** banix has joined #openstack-neutron21:37
*** shivharis has quit IRC21:37
*** nati_ueno_i has quit IRC21:38
*** dvorkinista has quit IRC21:38
*** alagalah has quit IRC21:38
*** leseb has quit IRC21:40
*** catohornet1 has quit IRC21:41
*** alagalah has joined #openstack-neutron21:41
*** aaronorosen has quit IRC21:41
*** suresh12 has quit IRC21:42
*** suresh12 has joined #openstack-neutron21:42
*** emagana has quit IRC21:42
*** rolledback has quit IRC21:44
*** ranger81 has quit IRC21:44
*** aaronorosen has joined #openstack-neutron21:45
*** Jay__ has quit IRC21:46
*** suresh12 has quit IRC21:47
*** ranger81 has joined #openstack-neutron21:49
*** kopparam has joined #openstack-neutron21:49
*** kopparam has quit IRC21:54
*** Zemeio has quit IRC21:56
*** mlavalle has quit IRC21:56
*** banix has quit IRC21:58
*** catohornet1 has joined #openstack-neutron21:58
*** julim has joined #openstack-neutron22:00
*** armax has left #openstack-neutron22:00
*** nati_uen_ has quit IRC22:00
*** nati_ueno has joined #openstack-neutron22:00
*** nati_ueno has quit IRC22:01
*** nati_ueno has joined #openstack-neutron22:01
*** pcm_ has quit IRC22:02
*** markmcclain1 has quit IRC22:02
*** jobewan has quit IRC22:04
*** VijayB_ has joined #openstack-neutron22:05
Sukhdevsalv-orlando: Ping22:06
*** vivek-ebay has joined #openstack-neutron22:06
*** jckasper has quit IRC22:06
*** leenheer has quit IRC22:07
*** leenheer has joined #openstack-neutron22:07
*** mestery has quit IRC22:07
*** xmltok_ has quit IRC22:09
*** ijw_ has quit IRC22:10
salv-orlandoHi Sukhdev22:10
*** dvorkinista has joined #openstack-neutron22:10
*** rolledback has joined #openstack-neutron22:10
Sukhdevsalv-orlando: saw your comments on my BP - thanks22:10
salv-orlandoit’s just pedantry22:11
Sukhdevsalv-orlando: See if you agree/accept my answers - especially the one you need a shot of scotch :-)22:11
*** vivek-ebay has quit IRC22:12
Sukhdevsalv-orlando: here is the link https://review.openstack.org/#/c/97125/222:12
salv-orlandoSukdev: red wine more than scotch in that case. “Arista” is pretty much the italian name for the pork roast22:16
*** ranger81_ has joined #openstack-neutron22:17
*** ranger81 has quit IRC22:18
*** ramishra has joined #openstack-neutron22:18
*** rolledback has quit IRC22:19
*** _TheDodd_ has joined #openstack-neutron22:20
ihrachyshkakevinbenton: hi. I saw your email re: CentOS packaging for neutron. I'm a neutron packager for redhat, so I may ask people around why that's done as it is now.22:21
ihrachyshkakevinbenton: instead of installing brocade code when installing ML2 package, wouldn't it be more correct to avoid installing any ML2 brocade stuff with pure ML2 package and move the ML2 brocade specific code to brocade plugin package itself?22:22
Sukhdevsalv-orlando: Wow!!  really? I did not know that - I knew it was Italian, but, did not know it meant port roast22:22
salv-orlandoSukhdev: not literally. It’s a piece of pork meat used to do pork roast. and the roast then takes the name from the meat.22:23
*** ramishra has quit IRC22:23
*** doddstack has quit IRC22:23
*** ijw has joined #openstack-neutron22:23
Sukhdevsalv-orlando: what does arastra mean in Italian?22:24
salv-orlandoarastra? never heard such word - at least spelled in this way22:26
kevinbentonihrachyshka: the drivers are expected to be in the ml2 folder22:26
*** dfarrell07 has quit IRC22:26
kevinbentonihrachyshka: moving everything to the plugin folder doesn’t really make sense because it is supposed to be a driver that runs in ML222:27
ihrachyshkakevinbenton: sorry. can you show specific files that do not belong to proper place?22:27
Sukhdevsalv-orlando: ah ha…our original name was Arastra, and was changed to Arista - so, I thought both would mean something in Italian :-)22:27
*** jecarey has quit IRC22:28
kevinbentonihrachyshka: https://github.com/bigswitch/neutron/blob/master/neutron/plugins/ml2/drivers/mech_bigswitch/driver.py#L3022:28
kevinbentonihrachyshka: that driver uses the same servermanager module and base class as the plugin22:29
salv-orlandoSukhdev: or probably they both were not mean to mean anything in Italian22:29
salv-orlandomaybe that’s some old Sanscrite word22:29
ihrachyshkaaha, I see. so how about moving that file to openstack-neutron-brocade package? in that case, those files won't clutter systems that don't use brocade, and at the same time allow anyone willing to use brocade to install one additional package and have all in place22:30
Sukhdevsalv-orlando: well, I leave it as mystery in that case --- since you picked up on Arista, I thought you may find some connection :_)22:30
*** catohornet1 has quit IRC22:30
*** safchain has quit IRC22:31
*** dvorkinista has quit IRC22:31
ihrachyshkakevinbenton: honestly, I don't packagers that set all as it is now long time ago, payed too much attention to each ml2 plugin file. Instead, plugins were just split based on their top level path: neutron/plugins/$plugin_name results in openstack-neutron-$plugin_name package22:32
ihrachyshkas/I don't/I don't think22:32
kevinbentonihrachyshka: right, and that’s why i sent the email. I just assumed they were not interested in special-casing the bigswitch plugin22:33
*** jgrimm_ has quit IRC22:34
salv-orlandoSukhdev: for what is worth it appears to mean “Security” in Sanskrit. http://www.babynology.com/meaning-arista-m55.html - and my bit of research for today is done.22:34
ihrachyshkakevinbenton: I would say, that's 1) not known to d/s - you may just report a bug 2) not in high priority list. But we'll be glad to handle your case once we have spare cycles22:34
Sukhdevsalv-orlando: :-)22:35
Sukhdevsalv-orlando: isn't almost midnight for you? have a good night...22:36
*** leseb has joined #openstack-neutron22:36
*** jckasper has joined #openstack-neutron22:37
karimbhey buddies, the qg interfaces of my router dont show either in br-int or br-ex, they show as tap devices there....is it normal?22:41
*** leseb has quit IRC22:41
marunsalv-orlando: ping22:42
*** mestery has joined #openstack-neutron22:42
*** ranger81 has joined #openstack-neutron22:44
*** ihrachyshka has quit IRC22:44
*** ranger81_ has quit IRC22:44
marunsalv-orlando: so is this ready to merge? https://review.openstack.org/#/c/9573822:44
marunsalv-orlando: I have a few comments but nothing too critical22:44
*** _TheDodd_ has quit IRC22:46
*** pcm_ has joined #openstack-neutron22:47
*** otherwiseguy has quit IRC22:48
openstackgerritAaron Rosen proposed a change to openstack/neutron: NSX: revert queue extension name change  https://review.openstack.org/9948322:49
openstackgerritSoheil Eizadi proposed a change to openstack/neutron-specs: Pluggable IPAM Subsystem in Neutron  https://review.openstack.org/9796722:49
*** nati_ueno has quit IRC22:49
*** xmltok_ has joined #openstack-neutron22:52
*** bandarji1 has joined #openstack-neutron22:52
*** tomoe_ has quit IRC22:52
*** oda-g has joined #openstack-neutron22:52
*** ranger81_ has joined #openstack-neutron22:53
*** bandarji has quit IRC22:53
*** ranger81 has quit IRC22:54
*** mestery has quit IRC22:55
*** alex_klimov has quit IRC22:55
*** WackoRobie has joined #openstack-neutron22:55
*** amotoki_ has quit IRC22:55
*** thedodd has joined #openstack-neutron22:56
*** marun has quit IRC22:56
salv-orlandomarun: I think they’re reasonable questions. I or jlibosva will address them tomorrow morning. I think it’s pretty much bed time for both of us22:56
*** alagalah has quit IRC22:59
*** WackoRobie has quit IRC22:59
*** ijw has quit IRC23:04
*** Sukhdev has quit IRC23:05
*** xmltok_ has quit IRC23:06
*** zzelle_ has quit IRC23:07
*** rkukura_ has joined #openstack-neutron23:08
*** rkukura has quit IRC23:08
*** rkukura_ is now known as rkukura23:08
*** trad511 has quit IRC23:10
*** nati_ueno has joined #openstack-neutron23:10
openstackgerritA change was merged to openstack/neutron: Copy-paste RPC Service class for backwards compatibility  https://review.openstack.org/9728923:11
openstackgerritA change was merged to openstack/neutron: Introduced rpc_compat.create_connection()  https://review.openstack.org/9729023:11
*** zhipeng has quit IRC23:14
*** catohornet1 has joined #openstack-neutron23:14
*** bandarji1 has quit IRC23:14
*** zhipeng has joined #openstack-neutron23:14
*** bandarji has joined #openstack-neutron23:14
*** yamahata has quit IRC23:18
*** thedodd has quit IRC23:18
*** julim has quit IRC23:19
*** ramishra has joined #openstack-neutron23:19
*** rm_work|away is now known as rm_work23:20
*** dvorkinista has joined #openstack-neutron23:21
*** ramishra has quit IRC23:24
*** nplanel has joined #openstack-neutron23:25
openstackgerritPraneet Bachheti proposed a change to openstack/neutron: Opencontrail plug-in implementation for core resources  https://review.openstack.org/9663023:25
*** sbfox has quit IRC23:26
*** bandarji has quit IRC23:27
*** prad_ has quit IRC23:30
*** osnew_ has quit IRC23:31
*** banix has joined #openstack-neutron23:32
*** pcm_ has quit IRC23:37
*** leseb has joined #openstack-neutron23:37
*** a_le has quit IRC23:40
*** dims has quit IRC23:41
*** yamamoto has quit IRC23:41
*** leseb has quit IRC23:42
*** dvorkinista has quit IRC23:42
*** manishg has joined #openstack-neutron23:52
*** leenheer has quit IRC23:54
*** leenheer has joined #openstack-neutron23:58
*** xmltok_ has joined #openstack-neutron23:58
*** suresh12 has joined #openstack-neutron23:59

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